Google rejected my app

Google rejected my app because your cobrowse.io package uses
<service
            android:name="io.cobrowse.CobrowseAccessibilityService"
            android:permission="android.permission.BIND_ACCESSIBILITY_SERVICE" />

I also had to send them a video where I showed that the authorization requests are shown to the user who can also refuse to have a device connected remotely but apparently it didn't help, how can I do it? A thousand thanks

@robinbonnes I have helped the customer via email as they reached out to us directly too (and it is a native SDK feature!). My understanding is that the reference to the accessibility service (which causes this) is optional in your plugin from the README docs.
It might be worth linking to our documentation in your README for full device remote control on android specifically as this does result in extra considerations for apps uploaded to the Google Play Store that we have already documented. Here is the link for reference: https://docs.cobrowse.io/sdk-features/full-device-capabilities/full-device-remote-control#configuring-full-device-remote-control
Hope that makes sense!


Add files via upload Riseba university

https://www.riseba.lv/en/students/postgraduate-studies/big-data-analytics


PATCH support for document update

We are trying to find if PATCH type requests are supported by the library.
Normally, we first read a document and after making appropriate changes, we send an update request.
We are looking for a way to directly update any particular value. So as the client, we need not get the document and update, instead we can just specify the particular property value to be updated and the library would internally handle lookup and update cloudant document.
Is this feature supported ?

Unfortunately CouchDB does not support the HTTP PATCH method to update documents, for details on the CouchDB API see their documentation


Feat/update example config

Update example config to include domains too

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


PopupContentElement - fire changed event

There were not styleeditor:changed events for changes in PopupContentElement.
FormElements should pass changes of the element through FormElement.setStyle to higher levels. This call was missing and therefor no event was fired.

Thanks!


Sqlocal with pglite

Hello,
Would you consider making sqlocal work with PgLite?
https://github.com/electric-sql/pglite
It's just like sqlite-wasm but it is rather an implementation of PosgreSQL made by the people in Electric-SQL and Neon

The main goals of SQLocal are to simplify the usage of SQLite WASM by abstracting away all of the boilerplate code you need to run it in a web worker and to provide a straightforward API and integrations with type-safe query builders. I've been watching the PGLite project, and it looks they already have an API with similar abstractions, and they plan to add integrations with query builders like Drizzle soon. Given this, there's not really anything we can offer by building an SQLocal integration for PGLite.


Problems encountered in intranet deployment of datahub

2022/03/14 07:14:25 Problem with request: Get "http://datahub-gms:8080/health": dial tcp <IP_ADDRESS>:8080: connect: connection refused. Sleeping 1s
2022/03/14 07:14:26 Problem with request: Get "http://datahub-gms:8080/health": dial tcp <IP_ADDRESS>:8080: connect: connection refused. Sleeping 1s
2022/03/14 07:14:28 Received 200 from http://datahub-gms:8080/health
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 174, in _new_conn
conn = connection.create_connection(
File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 95, in create_connection
raise err
File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 85, in create_connection
sock.connect(sa)
socket.timeout: timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen
httplib_response = self._make_request(
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 386, in _make_request
self._validate_conn(conn)
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1040, in _validate_conn
conn.connect()
File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 358, in connect
conn = self._new_conn()
File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 179, in _new_conn
raise ConnectTimeoutError(
urllib3.exceptions.ConnectTimeoutError: (<urllib3.connection.HTTPSConnection object at 0x7f5572ddcd60>, 'Connection to api.mixpanel.com timed out. (connect timeout=10)')
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 440, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 785, in urlopen
retries = retries.increment(
File "/usr/local/lib/python3.9/site-packages/urllib3/util/retry.py", line 592, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='api.mixpanel.com', port=443): Max retries exceeded with url: /engage (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f5572ddcd60>, 'Connection to api.mixpanel.com timed out. (connect timeout=10)'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 615, in _write_request
response = self._session.post(
File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 577, in post
return self.request('POST', url, data=data, json=json, **kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 529, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 645, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 507, in send
raise ConnectTimeout(e, request=request)
requests.exceptions.ConnectTimeout: HTTPSConnectionPool(host='api.mixpanel.com', port=443): Max retries exceeded with url: /engage (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f5572ddcd60>, 'Connection to api.mixpanel.com timed out. (connect timeout=10)'))
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/bin/datahub", line 5, in
from datahub.entrypoints import main
File "/usr/local/lib/python3.9/site-packages/datahub/entrypoints.py", line 11, in
from datahub.cli.delete_cli import delete
File "/usr/local/lib/python3.9/site-packages/datahub/cli/delete_cli.py", line 21, in
from datahub.telemetry import telemetry
File "/usr/local/lib/python3.9/site-packages/datahub/telemetry/telemetry.py", line 147, in
telemetry_instance = Telemetry()
File "/usr/local/lib/python3.9/site-packages/datahub/telemetry/telemetry.py", line 46, in init
mp.people_set(
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 238, in people_set
return self.people_update({
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 392, in people_update
self._consumer.send('people', json_dumps(record, cls=self._serializer))
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 594, in send
self._write_request(self._endpoints[endpoint], json_message, api_key, api_secret)
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 623, in _write_request
six.raise_from(MixpanelException(e), e)
File "", line 3, in raise_from
mixpanel.MixpanelException: HTTPSConnectionPool(host='api.mixpanel.com', port=443): Max retries exceeded with url: /engage (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f5572ddcd60>, 'Connection to api.mixpanel.com timed out. (connect timeout=10)'))
2022/03/14 07:15:08 Command exited with error: exit status 1
2022/03/14 07:32:41 Waiting for: http://datahub-gms:8080/health
2022/03/14 07:32:41 Received 200 from http://datahub-gms:8080/health
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 174, in _new_conn
conn = connection.create_connection(
File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 95, in create_connection
raise err
File "/usr/local/lib/python3.9/site-packages/urllib3/util/connection.py", line 85, in create_connection
sock.connect(sa)
socket.timeout: timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen
httplib_response = self._make_request(
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 386, in _make_request
self._validate_conn(conn)
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1040, in _validate_conn
conn.connect()
File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 358, in connect
conn = self._new_conn()
File "/usr/local/lib/python3.9/site-packages/urllib3/connection.py", line 179, in _new_conn
raise ConnectTimeoutError(
urllib3.exceptions.ConnectTimeoutError: (<urllib3.connection.HTTPSConnection object at 0x7f83ed1b3910>, 'Connection to api.mixpanel.com timed out. (connect timeout=10)')
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 440, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.9/site-packages/urllib3/connectionpool.py", line 785, in urlopen
retries = retries.increment(
File "/usr/local/lib/python3.9/site-packages/urllib3/util/retry.py", line 592, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='api.mixpanel.com', port=443): Max retries exceeded with url: /engage (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f83ed1b3910>, 'Connection to api.mixpanel.com timed out. (connect timeout=10)'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 615, in _write_request
response = self._session.post(
File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 577, in post
return self.request('POST', url, data=data, json=json, **kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 529, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/sessions.py", line 645, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.9/site-packages/requests/adapters.py", line 507, in send
raise ConnectTimeout(e, request=request)
requests.exceptions.ConnectTimeout: HTTPSConnectionPool(host='api.mixpanel.com', port=443): Max retries exceeded with url: /engage (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f83ed1b3910>, 'Connection to api.mixpanel.com timed out. (connect timeout=10)'))
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/bin/datahub", line 5, in
from datahub.entrypoints import main
File "/usr/local/lib/python3.9/site-packages/datahub/entrypoints.py", line 11, in
from datahub.cli.delete_cli import delete
File "/usr/local/lib/python3.9/site-packages/datahub/cli/delete_cli.py", line 21, in
from datahub.telemetry import telemetry
File "/usr/local/lib/python3.9/site-packages/datahub/telemetry/telemetry.py", line 147, in
telemetry_instance = Telemetry()
File "/usr/local/lib/python3.9/site-packages/datahub/telemetry/telemetry.py", line 46, in init
mp.people_set(
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 238, in people_set
return self.people_update({
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 392, in people_update
self._consumer.send('people', json_dumps(record, cls=self._serializer))
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 594, in send
self._write_request(self._endpoints[endpoint], json_message, api_key, api_secret)
File "/usr/local/lib/python3.9/site-packages/mixpanel/init.py", line 623, in _write_request
six.raise_from(MixpanelException(e), e)
File "", line 3, in raise_from
mixpanel.MixpanelException: HTTPSConnectionPool(host='api.mixpanel.com', port=443): Max retries exceeded with url: /engage (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f83ed1b3910>, 'Connection to api.mixpanel.com timed out. (connect timeout=10)'))
2022/03/14 07:33:22 Command exited with error: exit status 1

My server cannot access the public network, so I encountered the above error when deploying datahub with docker

This was fixed a few days ago. Can you please try with a new CLI version?

This was fixed a few days ago. Can you please try with a new CLI version?

thanks ，

Hi @persistZzzZ, thanks for pushing this issue! We'll close it out for now, but please feel free to open a new issue if upgrading to the newest CLI doesn't do the trick.


 In fact I might be ignoring something precisely because of these symlinks ;)

Hi @salotz, I think this is related to the ignore specifications themselves.  As described in my comment on #82, the ignore specifications should be made relative to the synchronization root (more information here).  The reason for this is that the ignores are applied on both endpoints, so only a synchronization-root-relative specification makes sense (in the same way that a .gitignore file specifies ignores as repository-relative rather than with absolute paths).
Mutagen will of course avoid scanning any content that is ignored, including traversing into any directories that are ignored.
Please let me know if correcting the ignore specifications doesn't fix the issue.

Guess I didn't read closely enough. Sorry for the trouble.


Chore/converting adapters to kotlin

I've removed unit tests for the adapter as it was testing Android UI components on JVM.
Also found testing issues with some of them and fixing them would require refactoring and extracting adapter into some sort of interactor

sorry, been busy.

sorry, been busy.

:D

LGTM :+1:


Disable all folder stuff?

Hey there, I love the work and have been directed here by linuxservers rutorrent build, so that's saying something!
Anyways, I have a very simple setup where I manually add each torrent through the rutorrent interface, as well as manually set the correct download location. Since moving my session information from linuxservers docker instance to yours I've noticed a lot of duplicate files have been created within /downloads/incoming, etc. Is there a way I can completely disable all of these automated niceties in favor of my brute force manual approach?
My preferred approach would be this:

Pass /config directory as a volume to the container
Pass a /downloads directory as a volume to the container, where this is the "filesystem" than can be viewed within rutorrent when choosing a save path after adding a new torrent. I'd then add a torrent, select whichever subfolder within /downloads to save the torrent data to, and off I'd go.

Thanks

See here: https://github.com/crazy-max/docker-rtorrent-rutorrent/issues/44
I didn't like having the temp and complete folders auto-created in my mounted space, so I mounted it to /downloads/fs/
Then I mounted a subdirectory of my mounted space as /downloads so it could create the two folders somewhere less problematic.
Also, made the changes recommended in the above link.


Update filtering to define strict matching in collections (arrays) and to-many relationships.

The current "Filtering" spec says:

This specification only supports filtering based upon strict matching.

This is great, however, it doesn't mention anything about filtering on collections (arrays) or to-many relationships.
To take the existing example of filtering on "author", let's assume that there are multiple "authors" for a single post:
{
    "id": "1",
    "title": "Rails is Omakase",
    "links": {
      "authors": ["17", "19", "21"]
    },
    "tags": ["foo", "bar"]
}

To filter posts on the to-many relationship of authors by value "19", I would expect the request to be:
GET /posts?authors=19

When a filter condition is provided for a collection (array) or to-many relationship, I would expect it to filter posts where the value of 19 is IN the authors collection.
Another example would be to filter posts by a tag:
GET /posts?tags=bar

I would expect the results to include posts that had the value of "bar" in the tags array.
Thoughts?

Filtering rules are now opaque to the base spec, which only reserves the filter query parameter and leaves the rest implementations. See http://jsonapi.org/format/#fetching-filtering
However, we could consider expanding the filtering recommendations.

This is effectively as far as we're going to do at this moment, now that the spec has changed and we have some recommendations. Thanks!


Quickest way to change the default area

For for security and other reasons, whats the quickest way to change /manager url to something more custom?
Might even be a good feature to lock down the admin url by IP so no one else can get to that page.

The /manager route is not configurable atm, but this feature was implemented in the legacy version (2.x) so there's probably some code there we could migrate.
As for locking certain paths to IP this must be achievable with web site configuration.
Also, as long as you implement a distributed cache provider you could deploy the manager interface on a completely different machine than the public website if you want.

Oh sorry.. I see this issue IS actually added in the old version of Piranha CMS (2.x). In this version there is a setting available in web.config for changing the manager route :) I'm so used getting all of the questions in the new repo.

@tidyui tdyui appreciate you response.  First of all on your second point its not in the web.config .net core version as you know web.config is hardly used.
Can you elaborate slightly on the below.

Also, as long as you implement a distributed cache provider you could deploy the manager interface on a completely different machine than the public website if you want.

Hi! It’s hard to answer correctly if you open the issue in the wrong repo. This repo is for the old .NET 4.5 version of Piranha. If you’re question is about the .NET Core version, please close this issue and open a new one in:
https://github.com/PiranhaCMS/piranha.core
Regards


Trigger event via HTTP?

Hi All,
I'm new to the hubot scene, but loving what I have running so far.
What I would like to do is trigger an existing event via HTTP.
For example: Using the https://github.com/github/hubot-scripts/blob/master/src/scripts/karma.coffee script as an example, if I wanted to check the top karma, by doing the following.
curl -X POST -d message="hubot karma best" http://<IP_ADDRESS>:8080/hubot/foo

I would then want to take the message (hubot karma best) and run that as if the user actually wrote it and return the response.

There's an issue over at https://github.com/github/hubot/issues/925 that covers the same topic. Check that out and see if it helps!

Hi,
Thanks for you reply.
I have been looking at the issue you post with regards to robot.emit and robot.on events.
The issue I have here is if I go down that route, that means I would have to edit existing scripts like the karma script to have robot.on event.
I am probably not explaining it well, but I would like the HTTP call to basically replicate a user input.
So if I send something like
curl -X POST -d message="hubot ping" http://localhost:8080/hubot/blah

Hubot would return with the correct response of "Pong".


config.json file check

There needs to be a check in place if the settings file has been deleted, if nothing else it needs to be able to exit which as of now doesn't work.

If settings are messed up then the user will not be able to use anything that the settings file is checked for.
In this instance we need a default option that will run instead of the messed up settings file. The settings file should remain as is until the next reboot. It will then check those settings and if there's an issue it will straight up recreate the settings file and let the user know the settings file has been reset.


chore(updatecli) improve Maven version tracking to track the ci.jenkins.io agent template deployed version (instead of latest) to keep environments in sync

Closes #66.
This PR ensure that the Maven version tracked by updatecli comes from what is deployed in ci.jenkins.io (aka. the "production") instead of the latest available Maven version.
The goal is to streamlin the deployment to ci.jenkins.io: most of the builds relies on jenkins-infra/packer-images as the source of truth: the Windows agent in this repository will follow.

Also closes #18


Feature/add sciphi search rag

@emrgnt-cmplxty I added small fixes on top of your PR. I'm debating whether agent-search should go into extras category in the dependencies, but it seems to be a fairly lightweight lib so maybe it's fine to go into main.

I'm debating whether agent-search should go into extras category in the dependencies

@emrgnt-cmplxty one possible concern with having the sciphi-test under tests/main as opposed to tests/extras (which are NOT run in GH actions) -- will the API key continue to work? If it fails for any reason, it will fail the test and this will cause issues with pushes to the repo. Overall I think it's best to both: make agent_search an extra dependency (like litellm) and add this test under tests/extras.


As-built documentation

Context

bpmn workflows to accompany text descriptions in GitHub Wiki
workflows are functional process models not business or system strickly
cross-reference with Mod Phase 1- Partner Portal Functional Design Document
documentation stored here

Prioritized list of diagrams

[X] Login (Student/Institution/Ministry) - [Status: Posted to Business folder]
[X] Create Institution - [Status: Posted to Business folder]
[X] Add/Update Institution Locations - [Status: Posted to Business folder]
[X] Institution Requests Designation - [Status: Posted to Business folder]
[X] Institution Manage Users - [Status: Posted to Business folder]
[x] Institution Complete Program Information Request - [Status: Posted to Business folder]
[X] Institution Manage Education Program and Offering - [Status: Posted to Business folder]
[x] Institution Report a Change (Scholastic Standing) - [Status: Posted to Business folder]
[X] Ministry Manage Designation - [Status: Posted to Business folder]
[x] Ministry Manage Users - [Status: Posted to Business folder]
[x] Ministry Manage Restrictions - [Status: Posted to Business folder]
[x] Ministry Manage Exceptions - [Status: Posted to Business folder]
[x] Student Create/Update FT/PT application - [Status: Pending Final Review]
[x] Supporting Users - [Status: Posted to Business folder]
[ ] Confirm and Disburse Funding - [Status: Not required]
[x] Student Request Change to Application - [Status: Posted to Business folder]

Statuses:
Draft - Work in progress with first draft
Not Required - Deemed not needed
Pending Final Review - Validation walkthrough with technical team
Review Complete - Accommodate feedback from walkthrough and finalize
Posted to Business folder - Posted it to folder from where business can pick it up

@abschwenker Unless there are any other workflows needed I will close this.


Update peggy setup instructions

This setup doc refers to a peggy:all target that no longer exists.
Would the following be a good replacement?
yarn migrate
yarn peggy:setup
yarn token:mint
yarn token:approve

Ah nevermind. I was running yarn from testnet-contracts instead of root.


Fix parsing and comparing versions with build information

This regex is pretty complex and I don't grok it all.  I found that removing the last ? allows expected results when build information is present, and does not break any existing scenarios covered by unit tests.
With 2 ?'s, regex101 describes this as

?? matches the previous token between zero and one times, as few times as possible, expanding as needed (lazy)

vs with 1 ?:

? matches the previous token between zero and one times, as many times as possible, giving back as needed (greedy)

Additionally I added multiple test cases to validate build information being handled as expected.  Per the semver spec, "Build metadata MUST be ignored when determining version precedence" - added test cases verify that it is in fact ignored as expected.
fixes #7

Thanks

Do you require me to update go-version in go.mod in trivy or will this get picked up in another manner?

You can open a PR bumping go-version.


Not possible to avoid native-tls feature of reqwest

I would like to add reqwest_dav as a dependency to my project. I want to use the rustls-tls feature of reqwest, and not have native-tls (which pulls in openssl as a dependency) in the build tree at all.
Adding default-features = false to the reqwest dependency could help here, it would allow users of reqwest_dav to add their own dependency on reqwest and set the features as preferred.
As a bonus, passing through the rustls-tls and native-tls features would facilitate selection of these, possibly mirroring the behavior of reqwest so that users who don't care at all have no extra work to do.

I'm sorry that these codes only complete basic functions.
These things may be useful to you

stream upload : https://github.com/niuhuan/reqwest_dav/compare/master...APG-lab:reqwest_dav:streaming
getetag maybe Option #1


Centurion config: tag to deploy for a specific environment

Currently the command line interface for centurion will deploy either the provided tag, or default to 'latest'.  For teams using CI, this becomes problematic when latest points to a potentially unstable image.  This isn't a problem for anyone already familiar with the tags used by the team, but if a secondary team (DevOps, SRE, hosting, etc) need to deploy, they wouldn't know what tag to use.
My idea is to add a config value that allows each environment to optionally specify the tag that should be used if no tag is provided on the command line.  This also leaves the flexibility for a team to have a deploy tag name that may not match the environment name for historical reasons.  If no deploy tag is specified in the config, and none is provided on the command line, centurion could continue to use 'latest' as before.

This could be done. It would require setting a default much deeper down than we do now. But we could do that. You could also implement this yourself in your Rake file for the affected project with something like this in each environment:
if fetch(:tag) == 'latest'
  set :tag, 'my_awesome_tag'
end

This would preclude using the latest tag at all in your project, however.

I hadn't considered(or realized we could) embedding ruby into the config, but that would certainly solve the problem easily.  I'll give that a try.

I don't recommend putting much Ruby in there, but that's a wee few lines. :)

For future reference, this is already possible using set :tag '' in the config, per https://github.com/newrelic/centurion/blob/3f293d9fc0528a3c562a6fb86c8b8d43d9a0b426/bin/centurion#L29


This is really helpful but I cannot find it within Obsidian Plugins?

This is really helpful but I cannot find it within Obsidian Plugins?
Thanks :-)

Hi, it's been submitted to the plugin gallery but it's still pending review from the Obsidian team. They usually do plugin reviews every 7-10 days so it might be a little while!
If you'd like to install it before then, the easiest way to do so is with BRAT: TfTHacker/obsidian42-brat

Thanks!!


Northern Ireland flag?

On your code the Northern Ireland flag is missing, could you add that flag?

Do you have link to wikipedia and the possible ISO code for it with sources?

#120
https://en.wikipedia.org/wiki/Flag_of_Northern_Ireland

Here is the solution for Northern Ireland:
https://github.com/carlosdv4/norther_ireland_flag_svg


Revert "Migrate detailed guides"

Reverts alphagov/whitehall#2637
Small responsiveness bug found in staging. Reverting until that's fixed.

:+1:


mark upscale alert as processed

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

License
Apache 2.0

What's in this PR?
mark upscale alert as processed if no errors occurred during handling
Why?
operator tries to upscale cluster every time even if alert already processed
Checklist

[x] Implementation tested
[x] Error handling code meets the guideline
[x] Logging code meets the guideline

Sorry, I just saw the linter check has failed. Could you please fix the error @shturman?

Thanks for the contribution @shturman


Add Java Solution for Problem Of The Day | 28 January 2023

Comment below if you're interested in working on this issue and would like to be assigned to it.

Shall I work on this issue?


Call clang directly when cross-compiling from Windows to Android

On Windows, the Android clang compiler is provided as a .cmd file instead of a .exe file. The .cmd file is spawned as a process and thus has a command-line limit of around 32000 characters, which you will most likely not reach. However, the .cmd launches the clang.exe, which results in a command-line limit of around 8200 characters, which larger projects with lots of include directories will easily reach.
As a fix, I've reintroduced response files when compiling from Windows to Android.

Thanks for the PR!
I was hoping it wouldn't come to this though since response files were just removed in https://github.com/alexcrichton/cc-rs/pull/564. Is there any way to reasonably shorten the command line for your project? Or perhaps consider this a bug for Android that we can spawn the process but it can't spawn a subprocess?

Good point! Was unsure myself whos responsibility this was to "fix". As shortening the command line for my project is not possible I've come up with a second, better, solution.
All the .cmd files do on Windows is call the main clang binary with the proper --target argument. As all that data is known to us we can construct this ourselves and call the main clang binary directly. :)

Seems like a nice strategy if we can get it to work! I think though that one of the purposes of https://github.com/alexcrichton/cc-rs/pull/495 is that the wrapper scripts pass a different --target than rustc, and that's intentional for the compilation of the C code?

Yes they do pass a different --target, which instead of having the shell script pass it, I construct the proper --target myself and call clang directly.
A call to the aarch64-linux-android21-clang.cmd shell script internally calls clang.exe with --target=aarch64-linux-android21. Basically, the proper target and clang "version" (clang or clang++) is in the .cmd file name. So by extracting those ourselves we can call clang.exe or clang++.exe directly.
On Windows, I've checked all .cmd files and being able to extract the the proper --target is true for all of them! The i686-linux-android16 to 24 .cmd files also add the -mstackrealign argument, which I've done as well.
So unless I'm misunderstanding or missing something I think we're good to go. :)

👍
Thanks again!

// build.rs
extern crate cc;

fn main() {
    let mut build = cc::Build::new();
    let tool = build.get_compiler();
    println!("{:#?}", tool);
    build.file("src/core.c")
        .compile("core");
}

D:\XXX\demo>cargo build --release --target i686-linux-android
   Compiling demo v0.1.0 (D:\Development\WorkSpace\clion\rust\study\demo)
error: failed to run custom build command for `demo v0.1.0 (D:XXX\demo)`

Caused by:
  process didn't exit successfully: `D:\XXX\demo\target\release\build\demo-eaac7e9af5279e16\build-s
cript-build` (exit code: 1)
  --- stdout
  OPT_LEVEL = Some("3")
  TARGET = Some("i686-linux-android")
  HOST = Some("x86_64-pc-windows-msvc")
  CC_i686-linux-android = None
  CC_i686_linux_android = None
  TARGET_CC = None
  CC = None
  CFLAGS_i686-linux-android = None
  CFLAGS_i686_linux_android = None
  TARGET_CFLAGS = None
  CFLAGS = None
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("false")
  Tool {
      path: "clang.exe",
      cc_wrapper_path: None,
      cc_wrapper_args: [],
      args: [
          "--target=i686-linux-android",
          "-O3",
          "-DANDROID",
          "-ffunction-sections",
          "-fdata-sections",
          "-fPIC",
          "--target=i686-linux-android",
          "-Wall",
          "-Wextra",
      ],
      env: [],
      family: Clang,
      cuda: false,
      removed_args: [],
  }
  running: "clang.exe" "--target=i686-linux-android" "-O3" "-DANDROID" "-ffunction-sections" "-fdata-sections" "-fPIC" "--target=i686-linux-android" "-Wall" "-Wextra" "-o" "D:XXX\\demo\\target\\i686-linux-android\\release\\build\\demo-0d1113bc106cefdd\\out\\src/ffi/core.o" "-c" "src/ffi/core.c"

  --- stderr

  error occurred: Failed to find tool. Is `clang.exe` installed? (see https://github.com/alexcrichton/cc-rs#compile-time-requirements for help)


Avoid unused warning on ignored return value.

Warnings appear in clang. This makes them go away in a compile-time manner with zero run-time effect.

Nice hack, thanks!


Incomplete Docs

The Docs are Blank for a majority of the documentation links.

I am working on it. I added the first two pages for a tutorial and will add more very soon.


Filter web request query strings passed to DfE::Analytics::Event

Context
We want to be able to filter any query parameters from DfE Analytics web requests but the underlying library doesn't yet support this.

Changes proposed in this pull request
Adds a FilteredRequestEvent implementation to handle filtering sensitive request query parameters before sending analytics web request events.

Guidance to review
We should deploy this branch to dev/test and check with DA that filtered params are being sent.

Link to Trello card
https://trello.com/c/hj8inJwZ/1575-bigquery-issues

Checklist

[x] Attach to Trello card
[x] Rebased main
[x] Cleaned commit history
[ ] Tested by running locally

 BigQuery PII issue


RxJava2

Is there any plans for RxJava2 compatibility? Because for now i should have both versions to make things work(and it's really bad). Thanks

Any news on this issue?

It's done in 2.0.0 version


scrcpy without video

Hi,
Thanks for your project because it's wonderfoul and solve a lot of problem ..
i would like to know if is it possible to exclude video virtualization and keep the control on android device.
Thanks for your patience.
bye bye
antonio

Technically (i.e. if you modify the code), it's possible, but in scrcpy, if you disable video, you can't click to control the device (or press any key because there is no window to intercept them).

So isn't possible to ctrl input on android display without a windows display.
I would consider to modify the code to make like an input ctrl remotely, exactly reverse mode of android pc ctrl app.
Do you think will be possible ?

I mean like deskdock .. i don't know how but is possibe to share keyboard and mouse and clipboard without mirroring.

So isn't possible to ctrl input on android display without a windows display.
I would consider to modify the code to make like an input ctrl remotely, exactly reverse mode of android pc ctrl app.
Do you think will be possible ?

This is possible directly using ADB but maybe you already knew that.
•100% ➜ adb shell input
Usage: input [<source>] <command> [<arg>...]

The sources are:
      dpad
      keyboard
      mouse
      touchpad
      gamepad
      touchnavigation
      joystick
      touchscreen
      stylus
      trackball

The commands and default sources are:
      text <string> (Default: touchscreen)
      keyevent [--longpress] <key code number or name> ... (Default: keyboard)
      tap <x> <y> (Default: touchscreen)
      swipe <x1> <y1> <x2> <y2> [duration(ms)] (Default: touchscreen)
      draganddrop <x1> <y1> <x2> <y2> [duration(ms)] (Default: touchscreen)
      press (Default: trackball)
      roll <dx> <dy> (Default: trackball)

@104m4tt0 https://github.com/Genymobile/scrcpy/issues/87#issuecomment-378775565
@herrenP it would be slow (#231)

@104m4tt0 #87 (comment)
@herrenP it would be slow (#231)

Ah, cool stuff! Nice thinking. :)


Add cdn info

HI @tnhu,
I'm Peter from cdnjs.com,
I wonder if you would like to add cdn info in the readme ?
This is the page we provide cdn for jsface - https://cdnjs.com/libraries/jsface
Thanks!

Hi Peter,
Sorry being late. Does cdnjs update version automatically? I.e. if I increase jsface to 2.4.0, the version appears on cdnjs immediately?
Thanks,
Tan

Yes, we update from npm package within 24 hours.

Thanks Peter. Added.

:+1:


Updated 3.15.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

No valid build backend found for Python recipe for package dtale using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.


View of new and updated articles

Would be nice to see which articles have been added recently or significantly updated recently.
Also would be nice to be notified or have an easy way to see if pages that i have created are edited by other users.

Recently updated and created pages can now be seen on the homepage. The headings can than be clicked through to get full listings. I've also compacted the lists on the homepage to give a better initial overview after logging in to a BookStack instance. Here's a screenshot of the new homepage:

I have not yet done the bonus part of this issue as I would like to roll that idea up into a full custom-querying interface. I'll open a new issue for that.

#64 Is the new issue mentioned above.


Keep component's state when viewport is undefined [ModelsTree]

Fix for an issue where ModelsTree component can lose its state when selectedView property spuriously changes to undefined (Happens with ContentLayoutManager from @bentley/ui-framework). By "losing the state", I mean, tree nodes in ControlledTree reset (for example, expanded/not expanded property).

 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.


Create Queue Class

[x]  Create Class
[x] Can be initialised

[x] Queue Name
[x] Queue Description
[x] Queue Items

[x] Add item
[x] View Item
[ ] #32
[ ] #33
[x] Request next item

#32 and #33 are low priority issues.
Class is complete for now


[WIP] Be more conservative about Mul_A operand values in prepass to avoid rejit.

Fixes:https://github.com/Microsoft/ChakraCore/issues/4744
JIT tries to typespec result in the following code to int32, even with AggressiveMulIntTypeSpec off
That later results in a JIT time bailout when result fails to loslessly convert and triggers rejit with AggressiveIntTypeSpec  off.
Interestingly, in this particular scenario, it happens only if result is initialized to literal 1.
If it is initialized to 2 or even to 1 , but in a more roundabout way, the code runs more than 2 times faster, since it becomes a float Mul right away.
function factorialA(n)
{
    let result = 1;
    for(let i = 0; i < n; ++i)
    {
        //   1 * whatever     in prepass
        //  seems like will not overflow, right?    not really...
        result *= (n - i)
    }
    return result;
}

The fix is to be less aggressive with no-overflow predictions for Mul_A when AggressiveMulIntTypeSpec is off
The idea is - If we don't definitely know the operand's min/max values, or if the operand is live on the back edge, we can assume it will have full range and will likely overflow and should just use float math, unless the other operand is definitely very small. -  basically 1 or 0, even result *= 2 is likely to overflow real quick since growth, if happens, is exponential.
No idea how to test this yet.
Perhaps run the repro and listen that there is no re-JITting, but how?

@dotnet-bot test OSX _no_jit_shared_osx_osx_test
@dotnet-bot test OSX static_osx_osx_release

You can use -trace:rejit to check if we rejit, however, we usually try to avoid having -trace in our tests

@Cellule - yeah. That is the problem, since the change is an optimization that otherwise does not have any observable effect.
The result of the fix is to not have rejit or to have a particular codegen. A regression test would have to rely on some kind of tracing/dumping to see that. And the codegen would be platform specific, so I think checking for no-rejit would be simpler.

@Cellule  how does it normally look when a test takes extra switches? Can you point me to an example?

For our test runner, in the rlexe.xml of a specific test folder (https://github.com/Microsoft/ChakraCore/blob/master/test/Optimizer/rlexe.xml for instance)
you could add
<test>
    <default>
      <files>mul_rejit_bug.js</files>
      <baseline>mul_rejit_bug.baseline</baseline>
      <compile-flags>-mic:1 -off:simplejit -testtrace:rejit</compile-flags>
    </default>
  </test>

It would be the equivalent of calling ch.exe mul_rejit_bug.js -mic:1 -off:simplejit -testtrace:rejit where

-mic:1 = run once in the interpreter then jit the function
-off:simplejit = do not do simple jit
-testtrace:rejit = a version of -trace:rejit that is less verbose (hopefully this one works well in your case)
mul_rejit_bug.baseline: contains the expected output (not needed if the test simply outputs pass)

@dotnet-bot test OSX _no_jit_shared_osx_osx_test please

I am doing additional testing on our end and will merge your pull request if everything is green


Multiple joins between two tables are not possible

Given the following two tables:
CREATE TABLE points(
        id BIGSERIAL PRIMARY KEY,
        x DOUBLE PRECISION NOT NULL,
        y DOUBLE PRECISION NOT NULL);

and

CREATE TABLE boxes(
        id BIGSERIAL PRIMARY KEY,
        o BIGINT REFERENCES points(id) NOT NULL,
        u BIGINT REFERENCES points(id) NOT NULL);

I want both relations to be joinable.
infer_table_from_schema!(dotenv!("DATABASE_URL"), "points");
infer_table_from_schema!(dotenv!("DATABASE_URL"), "boxes");

joinable!(boxes -> points (o));
joinable!(boxes -> points (u));

This is currently not possible because the joinable macro is generating a conflicting implementation of the trait JoinTo.
error: conflicting implementations of trait `diesel::JoinTo<points::table, _>` for type `boxes::table`: [--explain E0119]
 --> <diesel macros>:4:1
4 |> impl < JoinType > $ crate :: JoinTo < $ right_table , JoinType > for $
  |> ^

I just ran into this
table! {
    matches (id) {
        id -> Integer,
        date -> Nullable<Date>,
        time -> Nullable<Time>,
        home_team_id -> Nullable<Integer>,
        away_team_id -> Nullable<Integer>,
    }
}

table! {
    teams (id) {
        id -> Integer,
        name -> Text,
    }
}

joinable!(matches -> teams (home_team_id));
joinable!(matches -> teams (away_team_id));

Have any workaround surfaced since this was opened?

This is the expected behavior. Otherwise we have no clue what you mean when you say matches.inner_join(teams). You can use the .on method to specify an explicit on clause.


BEEFY subscription fires multiple times for the same commitment

Fixes https://github.com/paritytech/grandpa-bridge-gadget/issues/159

The issue mentions to find out why commitments are send twice and not to just prevent sending them multiple times from the rpc layer.

@bkchr
Because of the way networking works, peers could still be broadcasting votes for a round that has already concluded. The best we can do is dedupe the notifications sent to the rpc layer.

Bump

Fixed in https://github.com/paritytech/substrate/pull/10882


Teach rustup to override the toolchain from a version file

This adds a .rust-version file similar to rbenv's .ruby-version.
The form of toolchain supported here is limited to channel names,
explicit versions, and optional archive dates.
From the readme:
The version file
rustup directory overrides are a local configuration, stored in
$RUSTUP_HOME. Some projects though find themselves 'pinned' to a
specific release of Rust and want this information reflected in their
source repository. This is most often the case for nightly-only
software that pins to a revision from the release archives.
In these cases the toolchain can be named in the project's directory
in a file called .rust-version, the content of which the name of a
single rustup toolchain, and which is suitable to check in to source
control.
The toolchains named in this file have a more restricted form than
rustup toolchains generally, and may only contain the names of the
three release channels, 'stable', 'beta', 'nightly', Rust version
numbers, like '1.0.0', and optionally an archive date, like
'nightly-2017-01-01'. They may not name custom toolchains, nor
host-specific toolchains.
The version file has lower precedence than all other methods of
specifying the toolchain.

Some observations and questions about the design:
The file is called .rust-version with precedent from rbenv. The contents
of this file though would better be described as a "rust release channel",
or "rustup toolchain". A better name might be .rustup-toolchain though
the acceptable values in the file are a subset of valid toolchain names.
One might reuse the .cargo directory for this, putting the file in
e.g. .cargo/rust-version. This seems reasonable, and would establish
a strong precent for tools reusing .cargo; it would go against the
rbenv precedent. It would also go against the existing rustup
precedent where rustup stores it's global state in its own ~/.rustup
folder - but note that I have some small desire to move ~/.rustup
into ~/.cargo/rustup as a simplification.
Opinions about the above?
This doesn't allow specifying the host triple, nor does it allow
custom toolchains.
The search semantics are based off of the current working directory, though
one might feasibly expect them to be based off of the Cargo.toml directory.
That is cargo --manifest=foo/Cargo.toml will not search in foo. Existing
overrides have this limitation as well.
Fixes https://github.com/rust-lang-nursery/rustup.rs/issues/460
cc @SimonSapin @Diggsey @alexcrichton @yazaddaruvala

This looks great to me.
I don't have particularly strong opinions about where the override should go, other than setting a precedent for storing stuff in .cargo seems... nice, and also avoids cluttering the root directory even further.
It does make me wonder whether it's worth in the future having rustup follow a similar configuration hierarchy to cargo.

A general rule of thumb with Cargo is that you shouldn't version control .cargo, and I think this'd want to be in version control, right? If that's the case I'd recommend avoiding .cargo/* but a dedicated file sounds ok.

I like the idea of naming it .rust-toolchain. I'd keep it in the project root for the reasons @alexcrichton mentioned above

Just for the sake of having said it, my personal preference would have been toml or json with only one property, just incase some valid future usecase arises to add other parts of the toolchain triple into .rust-version, or something we haven't thought of yet.
However, I understand that you may also want the code to be opinionated so it is harder to abuse this late or reduce feature requests which request to abuse this.
Having said that, this look really good, thank you!

Looks good. I’d even go with rust-version over .rust-version. There is no reason this file should to be hidden.

I second @SimonSapin's comment, let's not repeat the error of Travis' taste for hidden files.

I think making this lowest precedence is going to leave people confused when they set an override some other way, and then try to use a .rust-version file. Others systems I'm aware of that use files have close ones taking precedence. Ie, local .bashrc overrides /etc/bashrc, etc. The closest analog I can think of right now is Apache's .htaccess, support for which must be separately enabled in the main config file. It's a very common problem that confuses many.

When compiling in /foo/bar, maybe the precedence should be:

Command line flag (e.g. cargo +nightly)
rustup override for /foo/bar/
/foo/bar/.rust-version
rustup override for /foo/
/foo/.rust-version
rustup override for /
/.rust-version

However, I understand that you may also want the code to be opinionated so it is harder to abuse this late or reduce feature requests which request to abuse this.

Yeah, that is my main reasoning, but also parsing toml is presumably more expensive than loading the string, and this is on a path that is supposed to be fast (though is increasingly bloated).
@nox The precedent for dotfiles here is pretty well established, by rbenv, by git, not travis particularly.
That said, I also think having a hidden file control what toolchain is invoked is pretty magical, and if we don't use the name 'rust-version', following rbenv's precedent, then we might as well abandon all precedent and just name the file 'rust-toolchain'.
Interleaving the 'override' / toolchain-file search is will require some major refactoring, but I'll look into it. I do think that arrangement sounds more difficult to explain in the documentation, though perhaps it's more intuitive.

I'm still feeling bikesheddy about the name. Possibilities:

rust-version
rust-toolchain
rustup-toolchain

@steveklabnik likes rust-toolchain which I had not considered before, since I consider toolchains, at least as they are known in rustup, a rustup concept, not a rust concept, but rust-toolchain is fine with me.

I'm inclined to drop the dot from the name.

I'm against rustup-toolchain since we might eventually have different toolchains or something like it for rustup. It leads me to believe that I can select a rustup version; also, this is something that should ideally be toolchain multiplexer agnostic -- some future rustreallyup will want to reuse these.
I think rust-toolchain makes the most sense, since we're not only versioning Rust itself, but all of its dependencies (rustdoc, rls, clippy in the future, etc.).
I'd go without the dot. There's no need to hide what version of Rust you use.

I do know of project maintainers which want to keep their top directory clean and small, so having a hidden file as option will certainly help. Its okay if rust-toolchain is the default endorsed method though. We should probably error if both hidden and non hidden files exist. I also support rust-toolchain over something rustup specific as the version pinning might be used by different tools in the future. There are already similar usage patterns with gitignore, which increasingly gets used for non git purposes (like by ripgrep for example).

I do know of project maintainers which want to keep their top directory clean

Sounds like a minor annoyance to discourage you from using it. Or alternatively, a very obvious sign that you are doing something special. Both seem like features here.

Ok now the file is called 'rust-toolchain' and the precedence is as described by @SimonSapin, where the closest override to the cwd wins.

@bors r+

@bors r+

@bors asleep at the wheel.


Sort issue with DB connection failing because of DamnSillyTime creating multiple defs for a time zone

See https://stackoverflow.com/questions/26515700/mysql-jdbc-driver-5-1-33-time-zone-issue

Add ?serverTimezone=America/Chicago to MySQL URL
in Version 0.7.1


WIP: Ldap authentication

I tried to implement a possibility to configure the login via LDAP.
The current state is:

It is possible to login (new users) into the system
The user get the correct permissions by LDAP group (superuser, active, staff)
It is now possible to simply add more authentication providers (with or without existing library)

Currently absolutely missing:

Prevent LDAP users from trying to change the password
Therefor it is necessary to include an extra field within the database to mark an user as "external" (or something like that)

Nice to haves/missing for the future:

Furthermore it would be nice to show all groups available within the LDAP for the permissions (one could import all the groups into the django local environment, but the authors of the django-auth-ldap lib discurage this behaviour (see: Group Mirroring))

I'm sorry to not further discuss my changes on the referenced issue but if you have any feedback I'll try to look into them!

@ibiBgOR, hey man, fantastic work!
Thank you so much!
I will keep your PR in queue for two weeks; thus you can safely push further changes.
The reason is that I plan to have 1.5 release in two weeks and I am currently working on last mile testing -> I won't add major changes in next two weeks.
LDAP authentication was planned for 1.6 release. So once 1.5 is out (in two weeks) I would proceed with your PR.
Please don't forget to regularly sync with (i.e. pull from) master branch.

no activity


Fix #20

In recent versions of closure, stubbing out Element.prototype.querySelector
before rendering the component will cause some elements to not be rendered.
Fix #20

CLA is valid!


Facebook OAuth not working: "Origin 'http://localhost:3000' is therefore not allowed access."

Hello, new to satellizer & angular in general. I've set up a simple Facebook login on the client side:
angular.module('CODLogin', ['satellizer'])
    config(function($authProvider) {
          $authProvider.facebook({
          clientId: 'myappid'
});

//[...]

angular.module('CODLogin')
   .controller('LoginCtrl', function($auth) {
     this.authenticate = function(provider) {
     $auth.authenticate(provider);
     /*$auth.link(provider).then(function(data) {
             console.log(data);
     });*/
};

And on the server I'm using passport-facebook, but nothing is reaching there, so I will not show the code here. Basically when I click on my FB login banner the pop-up opens. The first time it asked me if I wanted to let the app use my data, the following times I clicked the FB login banner the pop-up opened, but closed after a short while (since I gave permissions I guess it's an expected behaviour).
When the banner closes, in the console I get the following error:
 XMLHttpRequest cannot load https://www.facebook.com/v2.2/dialog/oauth?response_type=code&redirect_uri=..... No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access.

Shouldn't auth work with apps hosted in localhost? Moreover I don't get why it's going to v2.2 given the fact that in the satellizer.js code, there is "v2.3" in the facebook oauth url.
Any thoughts?

You have to setup things in the Developer Console inside your Facebook App.
On Fri, Jul 10, 2015 at 3:07 PM, masiarb<EMAIL_ADDRESS>wrote:

Hello, new to satellizer & angular in general. I've set up a simple
Facebook login on the client side:
angular.module('CODLogin', ['satellizer'])
config(function($authProvider) {
$authProvider.facebook({
clientId: 'myappid'
});
//[...]
angular.module('CODLogin')
.controller('LoginCtrl', function($auth) {
this.authenticate = function(provider) {
$auth.authenticate(provider);
/$auth.link(provider).then(function(data) {             console.log(data);     });/
};
And on the server I'm using passport-facebook, but nothing is reaching
there, so I will not show the code here. Basically when I click on my FB
login banner the pop-up opens. The first time it asked me if I wanted to
let the app use my data, the following times I clicked the FB login banner
the pop-up opened, but closed after a short while (since I gave permissions
I guess it's an expected behaviour).
When the banner closes, in the console I get the following error:
XMLHttpRequest cannot load https://www.facebook.com/v2.2/dialog/oauth?response_type=code&redirect_uri=..... No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access.
Shouldn't auth work with apps hosted in localhost? Moreover I don't get
why it's going to v2.2 given the fact that in the satellizer.js code, there
is "v2.3" in the facebook oauth url.
Any thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/sahat/satellizer/issues/476.

--
Ali Gajani
Founder at Mr. Geek
www.mrgeek.me
www.aligajani.com

Hi, thanks for the reply. I Actually set it up correctly ("http://localhost:3000/") on Facebook in the App page.

@masiarb Are you running your client and server apps on different ports? If for example your Angular app is on http://localhost:8000 and server is on http://localhost:3000 you will get that message if you haven't explicitly configured CORS support on the server. And yes Satellizer does use v2.3 Graph API so I am not sure why are you seeing v2.2. Try downloading satellizer.js directly from GitHub.

Closing this issue. Seems like it's an error on your side because you haven't set up CORS properly.

I have the same issue.
My client and server apps are on different port. I also configured CORS support on my server.
But, I am still facing this issue. Could you please help me?

Same here, server and client running on the same port and still getting CORS problem.
Backend is RoR 4 + Devise. I use Facebook and Twitter providers with the same problem on both.

Hi all, I finally found what was causing the issue with RoR 4 + Devise + Omniauth.
They are not meant to play together with Satellizer. Omniauth always redirects browser when Satellizer calls /auth/:provider causing the CORS message to raise. So basically I misunderstood how Satellizer works.
Not sure if OP had the same problem given he was on nodejs.


Switch container workflow to GitHub Packages

This commit switches the container CD to use the internal GitHub
registry by changing two things:

Change the Dockerfile back to use Leap and pip for dependency
installing
Add Github workflow for building and releasing new containers whenever
we make a new pcw release

This means that every new release builds a new container directly in
this repository, so we do not rely on any other service (OBS or
Dockerhub) for the pcw container.

Run failure unrelated to this PR.


Relation and Record definitions allow using ColumnDefs of SAME NAME as long as they're of different TYPE

Bug Report
Relation and Record definitions allow using ColumnDef's of same name as long as they're of different type.
Expected Behavior
Adding multiple ColumnDef's with the same name to a record should fail at compile-time or runtime.
Actual Behavior
Record(Seq(ColumnDef[Int]("id"), ColumnDef[String]("id")))  // does not fail ever

Steps to Reproduce the Problem

see above

Specifications and Environment

Commit-Hash: ef2ba9660d17356b2b616ec52d38afbd61632010

In the mean time we changed our column definitions to be
Record(Set(ColumnDef[Int]("id"), ColumnDef[String]("id")))  // does not fail ever


feat: GSM Voice State

Change list
Please provide briefly described change list which are you going to propose.
Types of changes
What types of changes are you proposing/introducing to .NET client?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] New feature (non-breaking change which adds value to the project)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Documentation

[x] Have you proposed a file change/ PR with appium to update documentation?

This can be done by navigating to the documentation section on http://appium.io selecting the appropriate command/endpoint and clicking the 'Edit this doc' link to update the C# example
Integration tests

[x] Have you provided integration tests to pass against the beta version of appium? (for Bugfix or New feature)

Details
Add API for GSM Voice endpoint
closes #368
Please provide more details about changes if it is necessary. If there are new features you can provide code samples which show the way they
work and possible use cases. Also you can create gists with pasted C# code samples or put them here using markdown.
About markdown please read Mastering markdown and Writing on GitHub

working on merges

All yours @mykola-mokhnach


Sequence details persist when navigating between clusters and beads

We should clear the sequence details table on switching to a different cluster.

The above only makes sense if we have something to replace those details with.
I guess we will have to populate the table with the entire contents of the lineage (cluster).  Better check how bad this gets if we switch to one of the large clusters.

Let's go with populating the sample table with all samples in a given lineage on clicking that box in the tree.  If this becomes a problem with rendering time then we can go with blanking the sample table on click.

Let's go with populating the sample table with all samples in a given lineage on clicking that box in the tree.  If this becomes a problem with rendering time then we can go with blanking the sample table on click.


[DOCS] added docs on Control's drag and drop api

Thanks - please squash the commits and we're good to go!

Okay, cool. Commits squashed. Thanks!

Thanks for your contribution!


Remove the card columns in favor of a Masonry grid

Mentioned before in https://github.com/twbs/bootstrap/issues/28891#issuecomment-500917538 and https://github.com/twbs/bootstrap/issues/28076#issuecomment-488386892, we should ditch the card columns in favor of masonry.
Main reasons are:

Cards are ordered per column instead of per row (https://github.com/twbs/bootstrap/issues/17709, https://github.com/twbs/bootstrap/issues/17805, https://github.com/twbs/bootstrap/issues/17882)
Several rendering issues (https://github.com/twbs/bootstrap/issues/19060, https://github.com/twbs/bootstrap/issues/19600, https://github.com/twbs/bootstrap/issues/19725, https://github.com/twbs/bootstrap/issues/20161, https://github.com/twbs/bootstrap/issues/28921)
No control over responsive number of columns (https://github.com/twbs/bootstrap/issues/17511, https://github.com/twbs/bootstrap/issues/22136,  https://github.com/twbs/bootstrap/issues/20321, https://github.com/twbs/bootstrap/issues/20905)

I've updated the docs and included Masonry via cdnjs, @XhmikosR is that ok for you, or do you have another proposal? (I also watched the releases of masonry github repo so I know when to upgrade)
Edit:
The demo is a example page now. It's also possible to add additional parameters (async & intergrity) to the extra js.
TODO:

[x] Link to this PR in migration section
[x] Move documentation to examples
[x] Allow to pass async attribute
[x] Check if integrity attribute is present before printing

Closes #28076, closes #28439
Card docs: https://deploy-preview-28922--twbs-bootstrap.netlify.com/docs/4.3/components/card/#card-columns-masonry-layout
Example page: https://deploy-preview-28922--twbs-bootstrap.netlify.com/docs/4.3/examples/#integrations
Demo: https://deploy-preview-28922--twbs-bootstrap.netlify.com/docs/4.3/examples/masonry/

Thoughts on punting the inclusion of external JS to an example instead of a core component page? We can link to it from the Card docs, but I'd prefer to build something there with Masonry instead of adding it to the core docs.

But also, yes, definitely down to ditch the card columns lol.

Examples might be a better place indeed, I'll change it

Maybe we should simply use native css subgrids to prevent having to use a "card-deck" container:
Hello Subgrid! – Rachel Andrew at CSSconf EU 2019
Not supported everywhere (anywhere?) but should be since next Bootstrap version I guess.

@inwardmovement, we'll probably introduce grid in Bootstrap 6 (the next version(5) will still have IE support), but the masonry layout still differs from (sub)gird.

The masonry example is now moved to the examples section.

Agreed with @XhmikosR about using the existing Examples layout. That's a little tougher for some of the needs here, but this does feel slightly over-engineered right now. We may end up adding third party JS to more examples, but I don't know when that'd be. Until then, these examples should look and feel isolated from our docs.
Here's what I'm leaning towards after taking a crack at this branch (see latest commit):

I'm gonna fix the conflict later but we should remove the copy example because it requires our docs CSS and JS.

This needs some more work and especially the part with the hardcoded script is something that I personally don't like. Maybe we should link to their docs?

Somehow I missed all this and used .card-columns.  Now wanting to go from 4.6 to 5.3 and yuck, .card-columns is broken.  My view is x number of columns wide, starting at the upper left going down each column as many as are needed for more or less evenly distributed column counts.
There's no getting around the efficiency of having all the cards/images stacked in columns when the images are the same width but different heights.  No gaps or wasted screen space; only at the bottom.
Does this mean there is no longer a mechanism for columnar lists?
It seems in masonry, the best alternative is horizontalOrder: true  and within a few rows down is impossible to follow in any reasonable visual order.  If anyone has tips...  Thanks.

IMO you'll have more chance of getting an answer by opening a new Discussion.


[tf2.0] tf.signal.fft2d causes an error when using @tf.function

tf_version: 2.0.0-beta1
import numpy as np
import tensorflow as tf
from tensorflow.python.keras.layers import *
from tensorflow.python.keras import backend as K
from tensorflow.python.keras.models import Model

def build_fftNet(img_shape = (224, 512, 1)):
    input_f = Input(shape=(img_shape))

    ins = Lambda(lambda x: (K.mean(x, axis=3)+1)*127.5)(input_f)
    ins = Lambda(lambda x: tf.complex(tf.math.real(x), tf.zeros_like(x)))(ins) # float to complex

    fft = Lambda(lambda x: tf.signal.fft2d(ins))(ins)
    fftNet = Model([input_f], [fft], name='fft')

    return fftNet

img_shape = (224, 512, 1)
fftNet = build_fftNet(img_shape)

@tf.function
def fft(blur):
    fft_true = fftNet(blur, training=False)
    return fft_true

blur = np.zeros((1, *img_shape), np.float32)
fft(blur)

@tf.function gives the following error when using tf.signal.fft2d:
Traceback (most recent call last):

  File "<ipython-input-122-2efd6297c220>", line 21, in <module>
    fft(blur)

  File "C:\Users\user\Anaconda3\envs\tf20\lib\site-packages\tensorflow\python\eager\def_function.py", line 434, in __call__
    return self._concrete_stateful_fn._filtered_call(canon_args, canon_kwds)  # pylint: disable=protected-access

  File "C:\Users\user\Anaconda3\envs\tf20\lib\site-packages\tensorflow\python\eager\function.py", line 589, in _filtered_call
    (t for t in nest.flatten((args, kwargs), expand_composites=True)

  File "C:\Users\user\Anaconda3\envs\tf20\lib\site-packages\tensorflow\python\eager\function.py", line 671, in _call_flat
    outputs = self._inference_function.call(ctx, args)

  File "C:\Users\user\Anaconda3\envs\tf20\lib\site-packages\tensorflow\python\eager\function.py", line 445, in call
    ctx=ctx)

  File "C:\Users\user\Anaconda3\envs\tf20\lib\site-packages\tensorflow\python\eager\execute.py", line 70, in quick_execute
    raise core._SymbolicException

_SymbolicException

I could reproduce the issue with TF 2.0.0.beta as well with latest Tf 2.0.0. Please see the colab gist. Thanks!

@ichae I think there was a small typo in your code. When I changed it, everything worked as expected.
I changed one line in your code from
fft = Lambda(lambda x: tf.signal.fft2d(ins))(ins)
to
fft = Lambda(lambda x: tf.signal.fft2d(x))(ins)
Please take a look at the gist here.
I am closing the issue as it was resolved. Please feel free to open it if the issue persists again. Thanks!


implement the burn_handle function

The burn_handle function burns a karst handle token and removes it from associated mappings.
Here are some resources to help you with implementation:
Lens implementation
Handle Documentation
you are expected to:

Implement the burn_handle external function.
write the associated test for the implemented function

Note that:

your implementation, doesn't need to be an exact copy of lens implementation. Be flexible, and properly comment your codes.
Ask questions in the working group, if lost or unsure of anything.

Hello I’d love to handle this @Darlington02

GM @Oshioke-Salaki , any update on this?

Hello I am still working on it. Please can I have the telegram group link to ask some questions.

https://t.me/+DFfuHjLkeXlkNTg0


Enabling a property to enable hiding

Under “-HiddenFrom AddressListsEnabled” it says:
“For users that are members of the Office 365 Group, the group will still appear in the navigation pane in Outlook and Outlook on the web if HiddenFromExchangeClientsEnabled property is enabled.”
I think the correct wording under “-HiddenFrom AddressListsEnabled” should be:
“For users that are members of the Office 365 Group, the group will still appear in the navigation pane in Outlook and Outlook on the web if HiddenFromExchangeClientsEnabled property is NOT enabled.”
or
“For users that are members of the Office 365 Group, the group will still appear in the navigation pane in Outlook and Outlook on the web UNLESS HiddenFromExchangeClientsEnabled property is enabled.”

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

ID: bd7d69db-be49-179e-94ce-635337c98d07
Version Independent ID: 9e372e66-5c7e-4d9a-56de-a35ae353fa09
Content: Set-UnifiedGroup (exchange)
Content Source: exchange/exchange-ps/exchange/users-and-groups/Set-UnifiedGroup.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@officedocsbot assign @yogkumgit

@uowdaniel Thank you for your feedback. We have made changes to the document via PR# https://github.com/MicrosoftDocs/office-docs-powershell/pull/4467
Thanks
Sri

@chrisda Please merge this PR #4467 . Thanks

@msbemba  Thank you very much for the contribution and updating the documentation with PR. @uowdaniel Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Update README.md

plugins values should have space after comma.

@Romanior this is a bug. It shouldn't require a space after the comma.
The bug is here: https://github.com/miguelcobain/ember-cli-selectize/blob/master/addon%2Fcomponents%2Fember-selectize.js#L199
The split should be done using ',' and not ', '. We have trim() after this for this very reason.
Can fix that, add a changelog entry for 0.4.4 and PR, please?

sure I can do it, a bit later though, it is hard to figure out which part is a wrong one :)

I've linked the line which needs to be changed.
Basically replace ', ' with ','.

And adding an entry to the CHANGELOG, please. :)


Can we use the headless browswer with html2image

Hello,
I would like to ask you about whether can I use the html2image on the Linux Server without Desktop. I can install the Headless Chrome Browswer on the server.
Do I need to pass some arguments or flags during the execution ?
Thank you

Hello, html2image automatically launches Chrome/Chromium in headless mode so it shouldn't be an issue and no additional flags should be necessary.
Except maybe the no-sandbox one if you're running as root, so see https://github.com/vgalin/html2image#change-browser-flags on how to change the browser's flags.


filter_out should be a set?

Hi,
The doc on the website shows an example of filter_out as a tuple:
https://robotgame.net/rgdocs
rg.locs_around(self.location, filter_out=('invalid', 'obstacle'))
But reading at the rg.py code, it needs to be a set. If you pass a tuple, it won't fail and it'll silently return all the locations (i.e. no filtering out at all). Can the doc be updated?
Thanks

Huh... good catch.
I think it should work both with a set and a tuple, i.e. convert it to a set in locs_around.

Oh... doesn't it already do that? https://github.com/RobotGame/rgkit/blob/master/rgkit/rg.py#L51

Indeed. Don't know how I got it wrong in the first place. Would have been surprising that no one caught that one before! Thanks vm Peter.

np! Glad to help. :)


Add warning if no SD card detected or faulty

On the initial screen (New/Load song) it would be useful to have a warning like "SD card missing or faulty" in case that the SD card cannot be accessed.
Subsequently, a warning should be displayed if creating a new song fails for some reason (for example because card is full, malfunctions or locks itself into write-protected mode).
Otherwise it may be confusing as New or Load commands cannot do anything without it (a newbie might forget inserting the SD card (as it happened to me :)
Current behaviour: Without a SD card, pressing "Load" does not do anything, while "New" acts like it is allowing you to create a new song.

I have started to work on this. As a first cut the no SD card status is detected by a new method in FileSystem object (method only implemented if PICOBUILD is defined).
Modified behaviour: message "**SD not present" shown as a path.

@Speccery thats great! 🎉  Feel free to tag me as a reviewer once you have a PR ready or even if you want to get early feedback on a draft PR.


Add BoundStatement#unset (JAVA-930).

I added a statement to the prepared statement docs since they were slightly stale (claimed unbound values were sent as null, where now we throw exceptions).  I pretty much copied the docs from BoundStatement.java as is.  This looks good to me :+1:, though I think it may be nice to consider making BoundStatement.UNSET public and moving it elsewhere (like Statement.UNSET)


Anouk's game modificiation

I finalized my modifications, and added one audio file.

That's so funny ! Thank you !


[BEAM-2556] Implement retries in the read connector.

Retry failed RunQuery calls.
Respect non-retryable error codes from Datastore.
Add unit tests to check that retryable errors are retried.

R: @vikkyrk
R: @ssisk
This is loosely related to my previous change — this ensures that non-retryable errors can be returned quickly without getting tied up in the retry & client-side backoff behaviour. I updated the read connector at the same time.

Coverage increased (+0.02%) to 70.701% when pulling 016baf3465bbccbc9d3df6999b38b1b2533aee8c on cph6:datastore_better_error_handling into 66b4a1be09b58a761cf49cc18a04eaaff555e376 on apache:master.

LGTM

R: @chamikaramj for merging.

LGTM. Merging.


How to refresh the carousel ?

Hi,
I load the datas from a databases and the carousel does not refresh the content. I must click again on the button to have the content.
So, how I can refresh or clear the carousel ?
Thanks.

I'm having the same issue

Seems that setPageCount(yourCount) does the job. If you look at source code, you'll see it is the only method that calls setData()


G7 X Support

My Canon G7 X is not recognized by the software. Can you add support to it? (libgphoto2 would support it)

RemotePhotoTool currently doesn't use Libgphoto2-Dateien and I'm not working on the tool anymore. If you like to contribute, you can open a pull request, if you are a coder.


Send env change token in tds when error abort transaction block

Description
Issues Resolved
[BABEL-4797]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 72.241%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
40406

Relevant Lines:
55932

💛  - Coveralls


add support for formData to example requests

There’s no support for form data yet, we need to add it.
{
  mimeType: 'application/x-www-form-urlencoded',
  params: [{ name: 'foo', value: 'bar' }],
}

might has been fixed by #4100

Nice!


Support for Nullable Reference Types (C# 8)

C# 8 is on the way and better to early than too late should be planned the implementation of the new features (Introducing Nullable Reference Types in C#).
public class Blog
{
    public int BlogId { get; set; }
    public string Url { get; set; } //In C# 8 this should be required by default
    public string? Url2 { get; set; } //In C# 8 this should be optional by default

    //Needs EF support for constructors with parameters
    public Blog(string url)
    {
        Url = url;
    }
}

A new logic is needed for mapping of e. g. string: Required and Optional Properties
Support for constructors with Parameters is required: Flexible mapping to CLR types and members (Custom O/C Mapping #240)
This list is probably not complete.

@MarcoLoetscher This is absolutely something we want to leverage, but since we are generating additional semantics and behavior based on nullability we will need to consider carefully how to do this without making unintended breaking changes to model/schema generation.

Notes from triage for scaffolding:

We've already decided to generate C# 8 code. So we can just generate a #nullable enable at the top of the generated file. This allows us to always generate nullable-aware code.
We can also parse the project to see if the user has enabled (via Roslyn), but this introduces a pretty big dependency. Also, not sure if there's a cross-platform Roslyn MSBuild workspace (https://github.com/dotnet/roslyn/issues/17974).
Opt-in flag on the command line. Seems like a bit of a shame to force users to opt into nullable references yet again, and discoverability isn't going to be very good.

So probably just generate #nullable enable.

This is currently blocked by https://github.com/dotnet/roslyn/issues/30143

One more important note... If you have a non-nullable auto-property, then you must have a constructor which initializes it, otherwise the compiler will warn/error. So we need to decide what we want to scaffold for non-nullable database columns, and also what we want our recommendations/code samples to look like.
The options seem to be:

Scaffold a constructor which has all non-nullable properties.
Scaffold a nullable backing field, wrapped by a non-nullable property with a getter that throws for null. This means that it's the user's responsibility to populate the property before calling SaveChanges(). It's less correct, and it means a runtime exception if the property isn't populated, which is similar to an NRE - exactly what we're trying to avoid. However, it's a valid/known pattern for some cases where some properties are initialized late (e.g. dependency injection without constructors).
Scaffold property initializers to some non-null default value, like empty string for string. This would be specific to each type, so we'd have to have a mechanism to determine it...

Any input?

@roji The constructor feels like the cleanest. However, this makes me wonder how many people will actually want this level of strictness. That is, how many people will actually want to use non-nullable types if it means they need to start calling parameterized constructors (or deal with other code consequences)  when they didn't before. I wonder if it should be the default for reverse engineering.

At least in current preview of VS 2019, the default tfm for app is netcoreapp2.2 so nullable types are available but the default language version is still 7.0 (since 8.0 is in beta phase, which likely to change before RTM). So while VS shows that adding #nullable enable is possible, it does not compile due to language version. Unless 8.0 becomes default for new app, we may still need to look into project to figure out the lang version.

Yeah, it may be a good idea to reach out to the language/compiler people and confirm what they intend to do with this.
@ajcvickers another thought... Our decision of how to scaffold may not be related to the user's choice at the project level - C# allows mixing null-aware and null-oblivious code quite easily. So even if the user hasn't opted into null awareness (because it seems too strict at the moment or whatever), we can still decide to generate a null-aware model. So I think it still makes sense to generate a nice, correct null-aware model with a full constructor - I can't really see much downside to it...

Finally, we can always defer the scaffolding part of this for now, until the default language situation becomes more clear.

I'm not sure if this is an EF issue but how should we define navigational properties?
// Post will always be in a Blog
public class Post
{
    public int BlogId { get; set; }
    public Blog Blog { get; set; } //class will get non-nullable warning
}

My first intuition is to add constructor but EF Core cannot set navigation properties (such as Blog or Posts above) using a constructor.

@gojanpaolo that's indeed the kind of stuff that needs to be worked out for nullability to work... As a last resort, even if we can't figure out a better way to deal with this, you could make Blog nullable but still have a [Required] attribute on it.

In future, we want to support taking navigation properties in constructor too. @AndriySvyryd  knows more details.

I see. Another workaround is to define a parameterless constructor and disable the nullable warning there. Mark it as private to hide from explicit use. Caveat is Post.Blog will be null when taken from DbContext or DbSet so don't forget to call Include it if you do eager loading like myself.

I agree that the ideal thing here is to have a constructor and to have EF Core go through that. Another option here is to have a nullable backing field for the non-nullable property, who's getter throws on null. This isn't perfect, but it's a way to instantiate with the post with the assumption that it gets injected later. I'm not sure if that works well for the relationship scenario though.
@ajcvickers maybe this is another good thing to take up in design?

@roji Yes, I think we need to consider this. It's interesting that when relationships are marked as required, it only means that the navigation property is non-null when loaded. But, until now, it's always been acceptable to have something that is required but not loaded. I'm not sure the best way to even conceptualize this.

it's always been acceptable to have something that is required but not loaded.

Here's a stackoverflow answer relating separation of responsibilities to why it's acceptable.
He stated 3 areas of concern:

Object Model & Invariants
Entity Model & Object Relation mapping
Database Model, Constraints, & Referential Integrity

EF Core is responsible for #2

From what I understood, EF Core is not responsible for enforcing the object model's invariants (e.g. Post.Blog is required) when loading data. EF Core knows that the referential integrity (Post.BlogId) already exists in the data as enforced by the database.
I hope it helps.

@ajcvickers I agree this requires some thought. However, it seems to resemble some other standard scenarios such as dependency injection, where a property is supposed to always be populated conceptually but is injected late; so theoretically it can be null for a short while, but if initialization takes place as its supposed to, it never will be. It seems like you don't want to burden users with null checks everywhere just because during initialization it can be null just for a short while, so a good design could be a nullable field wrapped by a non-nullable property whose getter throws on null.
The same seems to work also for relationships: as long as the user adheres to the programming contract of calling Include() (or lazy loading), they should never see a null - so the property they're interacting with should be non-nullable. The detail that the property is populated late in the process doesn't seem very important or relevant.

Should models used on both EF6 and EF Core also be considered? Not sure if that is a common use case but I am currently in that situation and the following workaround would not work on EF6 with lazy loading.

nullable backing field for the non-nullable property, who's getter throws on null

@gojanpaolo it would indeed be possible disable any EF Core behavior around new non-nullable references by simply disabling the nullable context around the relevant model properties, as you've done (or possibly the entire model). This would make EF Core behave exactly in the same way as today. Note that other means for specifying requiredness (i.e. the [Required] attribute and the fluent API) will also most probably continue to work as today.
There are no plans to modify EF6 to recognize non-nullable references.

As @ajcvickers mentioned in an offline discussion, this feature should make it into 3.0, because releasing it later would be a breaking change as non-nullable model properties will suddenly start getting treated as required by EF Core.
For now we seem to agree to punt scaffolding non-nullable references. However, we still have to consider nullability within scaffolded models: we will probably need to disable the nullable context (by placing a #nullable disable at the top of scaffolded files), otherwise properties we scaffold into nullable-aware user projects will be interpreted as non-nullable. This would only compile if C# 8 is enabled in the user's project, which again depends on the C# version we're targeting for scaffolding (see #14545).

Removing breaking-change label as this will only affect users opting into the NRT feature.

Should this work for migrations already?
Because I just tried to create a migration and it emitted nullable: false for all columns including DateTime? and string?
When setting IsRequired(false) manually for those properties during model creation the flags for the columns are emitted correctly.

@MarkusHaslinger You might be hitting #16760, which is fixed in the nightly builds

@ajcvickers thanks, will try the nightly

Is there already some examples for this? I'm having an issue with initialising DbSets in my DbContext. Should every DbSet be nullable or default initialised to something?
    public class DatabaseContext : DbContext
    {
        public DatabaseContext(DbContextOptions<DatabaseContext> options)
            : base(options)
        {
        }

        public virtual DbSet<User> Users { get; set; } = ...;

@shoe-diamente I believe there's been some discussion about a better alternative, but, at least for now, you're best off just surrounding the properties with a #pragma directive that disables the warning.

@shoe-diamente a doc page with best practices for non-nullable types will be up soon. In the meantime, keep your DbSet properties non-nullable and initialize them to null with the null-forgiving operator (bang). See https://github.com/aspnet/EntityFrameworkCore/issues/17212#issuecomment-522503909 for more details.


build web --wasm --dart-define-from-file config.json fails when entry in config.json starts with "="

Steps to reproduce

Create flutter web app
Create config.json with an entry that has a value starting with "="
{ "API_AUTH_KEY": "=1234xYz", }
Build the app with build web --wasm --dart-define-from-file config.json

Expected results
Successful build
Actual results
Compiling lib/main.dart for the Web...
Target dart2wasm failed: ProcessException: Process exited abnormally with exit code 64:
#0      processEnvironment.<anonymous closure> (package:dart2wasm/dart2wasm.dart:113)
#1      MappedIterable.elementAt (dart:_internal/iterable.dart:385)
#2      ListIterator.moveNext (dart:_internal/iterable.dart:354)
#3      MapBase.addEntries (dart:collection/maps.dart:89)
#4      new Map.fromEntries (dart:core/map.dart:210)
#5      processEnvironment (package:dart2wasm/dart2wasm.dart:110)
#6      options.<anonymous closure> (package:dart2wasm/dart2wasm.dart:72)
#7      Option.applyToOptions (package:dart2wasm/option.dart:18)
#8      parseArguments (package:dart2wasm/dart2wasm.dart:147)
#9      main (package:dart2wasm/dart2wasm.dart:164)
#10     main (file:///Volumes/Work/s/w/ir/x/w/sdk/pkg/dart2wasm/bin/dart2wasm.dart:10)
#11     _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295)
#12     _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184)

Argument Error: Invalid argument(s): Bad define string: API_AUTH_KEY==1234xYz
Usage: dart2wasm [<options>] <infile.dart> <outfile.wasm>

Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.24.5, on macOS 15.1.1 24B91 darwin-arm64, locale en-US)
    • Flutter version 3.24.5 on channel stable at /Users/philip.verschueren/fvm/versions/3.24.5
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dec2ee5c1f (3 weeks ago), 2024-11-13 11:13:06 -0800
    • Engine revision a18df97ca5
    • Dart version 3.5.4
    • DevTools version 2.37.3

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/philip.verschueren/Library/Android/sdk
    • Platform android-34, build-tools 33.0.0
    • Java binary at: /usr/bin/java
    • Java version Java(TM) SE Runtime Environment (build 17.0.11+7-LTS-207)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16B40
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/to/macos-android-setup for detailed instructions).

[✓] VS Code (version 1.95.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.100.0

[✓] Connected device (4 available)
    • iPhone 16 Pro (mobile)          • 76B455C7-7E00-4C7C-AB43-FC5D256F7F74 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-18-1 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 15.1.1 24B91 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 15.1.1 24B91 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 131.0.6778.87

[✓] Network resources
    • All expected network resources are available.

Fixed by https://dart.googlesource.com/sdk/+/00478a7d3caca41be36dd33ba34189af534b7c12


Add default social image and description

Hey, @jawache - love this site, nice work!
This PR adds a simple social card image and a meta description for the homepage. I set this up so you can override the social image on any given page by setting socialImage in frontmatter.
I also grabbed the appropriate font and colors to make sure it all matches up:

Hope this was helpful! Let me know if I can do anything else to get this merge-worthy.

Thank you so much Dustin, this is brilliant!


Adding ability to pass in parameters as positional even if kwarg.

Before if you bound a function with argbind like so:
@argbind.bind()
def main(
    x: int = 5,
    y: int = 3,
):
    print(x, y)

And then used it at runtime like this, inside of an ArgBind scope:
main(3) # setting x to 3 as positional

You got an error that said the argument got passed twice (once as positional and once as kwarg). This PR checks for collisions between the arg list and the kwargs that ArgBind finds and resolves them before calling the function, so that the above works.

Codecov Report

Merging #9 (198d694) into main (659fdfd) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          290       303   +13
=========================================
+ Hits           290       303   +13

Impacted Files
Coverage Δ

argbind/argbind.py
100.00% <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 659fdfd...198d694. Read the comment docs.


tonivade improved solution

Check List:

[X] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[X] All formatting changes by the build are committed
[X] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[X] Output matches that of calculate_average_baseline.sh
[X] For new entries, or after substantial changes: When implementing custom hash structures, please point to where you deal with hash collisions (method starting at line number 202)

This is an improved version of my previous solution. This time I removed the usage of HashMap and I'm using a two dimensions array to store the stations.
I calculate the hash code of the name  and then I apply the modulus to array length to get the bucket to store the station. If there are more than one station on the bucket, I iterate searching for the station with the same name length and hash code.
I'm not sure if this makes the solution potential for hash collisions, but it passes all the tests and the solution matches with the baseline output.

Execution time:

real    0m9,313s
user    1m8,684s
sys     0m1,613s

Execution time of reference implementation:

real    3m48,755s
user    3m45,781s
sys     0m3,594s

Executed in my laptop:

Intel Core i7-1065G7
32GB RAM
4 cores and 2 threads per core

00:07.894 now.


ApplyAnimatedValues implementation for browser

to: @vjeux @browniefed
I tweaked the implementation for the injected ApplyAnimatedValues in the browser environment.
It now accepts style with transforms in the form of an array, similar to React Native.
Notes:

This doesn't attempt to use browser-prefixed CSS styles, but it could (improving compatibility).
Since I'm no longer using getDOMNode(), this is not compatible with React 0.13, but we could easily add that in.
This also includes a variable naming goof that I made with shouldUpdate => didUpdate.

Only was using getDOMNode since the interactive doc was using .13

Lgtm


migrate azure-communication-callingserver to azure-json

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-communication-callingserver

Can close/drop this work as the callingserver is being deprecated


list of users at or under a certain level

any easy way to get all users with same level or under (or greater for that matter) for a given level ... or even  by role such as all users containing a certain role or a role at equal or less in level

I agree. Trying to wrap my head around this.

aye @hubentom
in Role model add
public function users()
  {
  return $this->belongsToMany('Arbikor\User', 'role_user', 'role_id', 'user_id');
   }
and then in your controller add:
$admins = Role::find(1)->users;

that gets the users for a given role...? I need the users for a given LEVEL

$level = Role::where('level', 1)->get();
$admins = $level->users;

Can you add it in the code please ? Super useful to retrieve users with a role.
Thanks.


Feature/walk fixes

Description
This PR fixes steps/directions computation and consumption when the walking path to the target location is not a straight line.
How to reproduce the issue

Log in to any character and go to Lorencia, position (137, 122).
Switch character and log back in to the same character and check the current position.
If the position is different from (137, 122), then the issue was reproduced successfully.
If not, go to (136, 124) and switch character and log back in. The current position will be (136, 123), instead of (136, 124).

Great!
Thank you too!


fix: environnment replaced with environment

Description
I just started reading the documentation and saw the typo issue for "environment". I replaced all typo issues for that word.

 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.


Minor edit to the last sentence

I think this change would improve the docs.

Thanks for your contribution. We will consider your suggestions and update the docs as needed. Watch this space! /IBM


Chore(CI): builds should succeed in self-contained PR builds

When nvidia builds run in a PR, they fail because after building the akmods image, that is not published, thus the follow on builds fail because they attempt to reference the just built image with that PR tag.
Need a solution to this which allows valid builds but doesn't allow signing/publishing of images in PRs.

cc @marcoceppi
We've used "if in a PR skip this" things in sections before in bluefin but I forgot what to search for.