Linter and formatting scripts

I added some convenience wrappers to the linter (hlint) and formatter (brittany) we're using.
I will write more documentation in the README later, but essentially this will allow us to run git lint and git fmt to lint and format our code. It supports (1) passing explicit file names to format/lint, and (2) passing a commit hash/branch name, to lint/format all files since then. By default, it just lints/formats all the code modified since the last commit.

Do you want to go through the existing codebase right now and run these? There is existing code that isn't formatted. It may be better for the blame to be attributed to this PR than for it to be scattered..

I'd like for this to be merged in first. I have another PR in the works
where I string together the whole compiler pipeline. I was thinking of
doing all this formatting then.
On Sat, Oct 9, 2021, 1:04 PM Hans Montero @.***> wrote:

Do you want to go through the existing codebase right now and run these?
There is existing code that isn't formatted. It may be better for the blame
to be attributed to this PR than for it to be scattered..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ssm-lang/sslang/pull/11#issuecomment-939328401, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AC2A5DFGXCMMQ6MDWQZ6J43UGBYZFANCNFSM5FUZUWAQ
.

@sedwards-lab addressed your feedback, but the README is looking rather lengthy. I'm wondering if we can move parts of it to the CONTRIBUTING.md document? Or should I create a table of contents?


[feature-4417][JsonSplit]refactor json in process instance and process definition

close #4417
refactor json in process instance and process definition

+1


⚠ Bofrost.be has degraded performance

In f05b97e, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1829 ms

Resolved: Bofrost.be performance has improved in fe0ae90.


Add GitHub Actions for test profiles on main branch updates

Add a new workflow to deploy all three test profiles (test_illumina,test_ont,test_bams) on main branch updates.

completed with PR #62


ANR exception in Google Play console.

I am using your framework for Titanium through intermediate module, and I am getting a lot of these exceptions in Google Play console:
java.lang.ArithmeticException: divide by zero
at com.felipecsl.gifimageview.library.GifDecoder.advance(GifDecoder.java:229)
at com.felipecsl.gifimageview.library.GifImageView.run(GifImageView.java:157)
at java.lang.Thread.run(Thread.java:818)
Could you check it please?

This has been already fixed on #29


Fails to fetch changes when pull request is made from a 'master' branch

The pullrequest-resource in outputs the following error, but doesn't fail completely:
fatal: Refusing to fetch into current branch refs/heads/master of non-bare repository
Already on 'master'
Your branch is up-to-date with 'origin/master'.

This results in strange behaviour where the pullrequest-resource has cloned the git repository, but not fetched the changes from the pull request. The subsequent steps in the build plan continue to execute without including the pull request changes.
Steps to reproduce:

Create a pipeline with using pullrequest-resource pointing to a repository let's call it usera/repo
Fork the repo into another user account userb/repo
Make some trivial change to the master of userb/repo (it's important the change is made in master) and create a pull request from userb/repo:master to usera/repo:master
Check the pipeline in for the pullrequest-resource

Expected outcome:
The in of pullrequest-resource should fetch the changes from the pull request we should see the changes from userb/repo:master in Concourse.
Actual outcome:
The in of pullrequest-resource fails to fetch the changes from the pull request. It also doesn't fail in an obvious way it outputs the error but then exits with success. We see only the contents of usera/repo:master in Concourse.
This only happens when the pull request is made from the master branch of the fork. The asset/lib/in.rb seems to assume the pull request is made from a branch other than master branch_ref = pr['head']['ref'] what happens in this case is branch_ref = 'master' and then system("git fetch -q origin pull/#{id}/head:#{branch_ref} 1>&2") results in the Already on 'master' error. Quick workaround hack is to hardcode branch_ref = 'tmp', but that has other implications so not a full solution.

Let me dive into this a little more.
It was requested a while back that git fetch -q origin pull/#{id}/head:#{branch_ref} 1>&2 exist so the branch is checked out at the correct name. If it is on master then it should checkout master. I wonder if there is a way to force it.
It's interesting this is the first time that this is come up.

I see the PR that originally caused this. I've reached out to the original author for some clarification in their workflow. I don't want to break anything or make it backwards incompatible.
Please expect this to be resolved shortly.

This should have been resolved in f6210c9. The checkout branch name is being prefixed with pr-.

Please reopen this issue if you have problems with it passed v15 release.

DEPRECATION: Please note that you should update to using `version: every` on your `get` for this resource.
Cloning into '/tmp/build/get'...
fatal: Refusing to fetch into current branch refs/heads/master of non-bare repository
/opt/resource/lib/in.rb:36:in `block in <main>': git clone failed (RuntimeError)
	from /opt/resource/lib/in.rb:34:in `chdir'
	from /opt/resource/lib/in.rb:34:in `<main>'

Gah!

I don't understand. I've tested this with the reproducing instructions you provided. This is the PR I am testing with.
Do you see anything wrong with the PR?

The reproducing steps from the in step for the previous step would be.
git clone --depth 1 git@github.com:jtarchie/pullrequest-resource.git testing
git submodule update --init --recursive
git fetch -q origin pull/34/head:pr-34

Any insight in the git repo you have would be great or minimal steps to reproduce.
Sorry the above is stream of thought.

I can't reproduce the error when following those commands on my repo. It seems something different is actually happening when running the v15 of the pullrequest-resource in my pipeline.
Sorry I don't have time to investigate right now, but when I next get a chance I'll try building it myself and see if I can isolate it better for you. Might be a couple of days before I can do this.
In the meantime perhaps setup a pipeline to reproduce?

(╯°□°)╯︵ ┻━┻
I've recreated the issue with a pipeline. When I hijack the container and run the exact set of commands, it works fine. 👎
This is going to take some more debugging.
Capturing the .git/config here for later reference:
[core]
       	repositoryformatversion = 0
       	filemode = true
       	bare = false
       	logallrefupdates = true
[remote "origin"]
       	url = https://github.com/jtarchie/pullrequest-resource
       	fetch = +refs/heads/master:refs/remotes/origin/master
[branch "master"]
       	remote = origin
       	merge = refs/heads/master

@jtarchie What I've observed is that when there's a PR made to master, the PR resource instead checks out master of the origin repository, rather than the fork. This probably applies to all branches, not just master, if the names match on the origin and the fork.

@vito I understand conceptually, but I don't know what to do. You think this problem would occur with people checking out PRs locally, but Google is pretty barren on this.

@nathgs, it turns out I did solve it. It's just I forgot to rebuild the docker image and push that out. Woops. Please try again.

@jtarchie It's working now. Thanks for following up on this and getting it sorted so quickly!
Time to make a pipeline that automates the releasing of the pullrequest-resource docker image ;-)


"Reopen Closed Editor" doesn't work for Settings

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 10.0.15063

Steps to Reproduce:

File -> Preferences -> Settings
Close the Settings again
Press Ctrl+Shift+T - instead of reopening the Settings, it opens the file that was most recently closed before that.

https://github.com/Microsoft/vscode/issues/16733


fix: improve some wordings and cli table

Description
Improve some wordings and cli output.
 Watermarks will be   | exists @Resource.Folder        | libre.graph/driveItem/children/read      |
|                                      |         | applied.                       |                                | libre.graph/driveItem/basic/read         |
+--------------------------------------+---------+--------------------------------+--------------------------------+------------------------------------------+

Related Issue

Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

Couldn't we add a column at the beginning with the name of the unified role like UnifiedRoleSecureViewerID. That would ease purpose identification a lot...

UnifiedRoleSecureViewerID

Couldn't we add a column at the beginning with the name of the unified role like UnifiedRoleSecureViewerID. That would ease purpose identification a lot...

I added that to the table, because i cannot afford having you complaining about me 😍

UnifiedRoleSecureViewerID

Couldn't we add a column at the beginning with the name of the unified role like UnifiedRoleSecureViewerID. That would ease purpose identification a lot...

I added that to the table, because i cannot afford having you complaining about me 😍

🤣 🚀 🎸


Example of validation curve

After studying the sknn api I could not plot a validation curve to test over-fitting.
from sknn.mlp import Classifier, Layer
nn = Classifier( layers=[ Layer("Maxout", units=100, pieces=2), Layer("Softmax")], learning_rate=0.001, n_iter=25)
result = nn.fit(X, y, w)

Which attribute carries the result mse/mcc per epoch/iteration?

I am working on it. To get started:
``valid_err = []
train_err = []
def get_errs(**variables):
valid_err.append(variables['best_valid_error'])
train_err.append(variables['best_train_error'])
nn = Classifier(
layers=[Layer("Softmax")], verbose=0, n_iter=5, callback={'on_epoch_start': get_errs}, valid_size=.2)```

Perfect, you found the callback I was going to suggest!
Is it working now?

It's working fine. I think it a bit obscure, thou.
One suggestion would have support to cross-validation, as in sklearn.

A section in the documentation would help I guess.
Most sklearn APIs are supported. Which in particular would help?

I quite like the look of this code. It's very explicit. I'll add it to the User Guide!
If you have more information about how you'd like to do cross-validation that isn't supported already, let me know and we can look into it separately in another issue!
valid_err, train_err = []
def get_errs(best_valid_error, best_train_error, **_):
    valid_err.append(best_valid_error)
    train_err.append(best_train_error)

nn = Classifier(
    layers=[Layer("Softmax")],
    n_iter=5, valid_size=.2, verbose=0,
    callback={'on_epoch_finish': get_errors},)

I will look into embedding cross-validation in the Classifier.
The first line of the example needs a correction:
valid_err, train_err = [], []

Thanks for the report and suggestion! I'm leaving this open until I update the docs.


Upgrade the changelog for 1.1.3a0 release

https://github.com/pylint-dev/pylint-pytest/actions/runs/6523340242/job/17713884405

🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉

Haha, no error on first try, nice

Nice to you! You have made a solid, re-usable action for releasing 🚀 🚀 🚀 🚀 🚀


Turn CLI messages to constants

please create another followup issue to turn these messages into constants so we reuse them in components and tests, something like https://github.com/asyncapi/asyncapi-react/blob/master/library/src/constants.ts
no need to fix it now, not related to this pr
Originally posted by @derberg in https://github.com/asyncapi/cli/pull/14#discussion_r666844302

@magicmatatjahu shall I start with this issues, next?

Loading spec from remote source is more important :)

asyncapi validate
No contexts saved yet, run asyncapi --help to know more.

The help string also does not say what contexts are, so @derberg should we update the help text to talk about the context in a detailed manner or add have a better error message for this scenario?
I think the help message is a general version of all our commands and if we write what is a context there, it could get cluttered so shall we have command level help messages to talk about specific commands. In general, we have to help the user understand how he can pass in spec file to the CLI.
Also as of now, there are 4 main ways to pass the spec file path.

--file flag
--context flag
set an appropriate context then just use any commands
have asyncapi.yml file in the working directory.

We simply need more structured and formated info, if we do it, then it won't get cluttered. Look at help section here https://clig.dev/#help and the jq example.

Then I guess we can solve this issue by updating the help command or adding a context section in the help string so as to talk about what context does, and how could users benefit from it.

taking inspiration from git

$ asyncapi
usage: asyncapi [--verison] [--help] [--file <spec-file-path>] [--context <saved-context>]
           [--watch] <command> [<args>]

commands:
  validate    validate a specification file
  context     manage context

To make your commands shorter use context, create and reuse your contexts accordingly
  current                          check the current context
  list                             list all the saved context
  remove <context-name>            remove a context from the store
  use <context-name>               set a context as current
  add <context-name> <file-path>   add/update new context

we need more complex, and contextual help:

one thing is what you wrote, have help displayed when running asyncapi as git has
another is that we should have help on the command level, dedicated to given command only, like asyncapi context --help
and when running asyncapi validate or any other command, and there is no default file or context, we need detailed help message that explains the priority we have, the file flag, context flag, default context and default filename

I think better would be to first refactor help to constants to have simple PR to handle and then in another issue discuss help texts. Let us have as small and focused PRs as possible

I think better would be to first refactor help to constants

Does this mean the help string will now be something like this ⬇ where the help string is made up of smaller constants?
const helpString = `
Usage:
  ${USAGE_STRING}
`

it depends but in short, yes 😄
help shown after calling asyncapi will be a combination of help shown under asyncapi context --help and asyncapi validate --help and others. So you need what you described, basically reuse help text.
In case of messages like <Text>The context you are trying to use is not present.</Text> you need <Text>${CONTEXT_MISING}</Text>; and reuse the same constant value in tests, so any changes to help text doesn't require changes in tests


⚠ File access has degraded performance

In 7ae9868, File access (https://fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 2946 ms

Resolved: File access performance has improved in 5df4053.


Created Test App but Domain Link Is not working

I have created Test Application after Registration @ Website. I created Test application with Live And i also Use Branch-Key and put in Plist File with LinkDomain, I also add URI Schema but when i try to open app via Branch Domain Link named "tryone.app.link" it convert to http://tryone.app.link , it says File not found.
Setting in Plist File :
branch_app_domain
tryone.app.link
branch_key

live
key_live_flAb3U7ukPd2CkVR8KEwaocltzl6rSv0

CFBundleURLName
com.test.customer
CFBundleURLSchemes

DeepLinkingTest

In App Delegate file :

(BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
// Override point for customization after application launch.
Branch *branch = [Branch getInstance];
[branch initSessionWithLaunchOptions:launchOptions andRegisterDeepLinkHandler:^(NSDictionary *params, NSError *error) {
if (!error && params) {
// params are the deep linked params associated with the link that the user clicked -> was re-directed to this app
// params will be empty if no data found
// ... insert custom logic here ...
NSLog(@"params: %@", params.description);
}
}];
return YES;
}
// Respond to URI scheme links

(BOOL)application:(UIApplication *)application openURL:(NSURL *)url sourceApplication:(NSString *)sourceApplication annotation:(id)annotation {
// pass the url to the handle deep link call
[[Branch getInstance] handleDeepLink:url];
// do other deep link routing for the Facebook SDK, Pinterest SDK, etc
return YES;
}

// Respond to Universal Links

(BOOL)application:(UIApplication *)application continueUserActivity:(NSUserActivity *)userActivity restorationHandler:(void (^)(NSArray *restorableObjects))restorationHandler {
BOOL handledByBranch = [[Branch getInstance] continueUserActivity:userActivity];
return handledByBranch;
}
So can you help me to integrate so i am looking forward to your SDK

So you get a '404 - File Not Found' when you click the link?
Did you set your entitlements file up?  Make sure you've set up your associated domains:
	<key>com.apple.developer.associated-domains</key>
	<array>
		<string>applinks:tryone.app.link</string>
	</array>

But i am not using enable universal link.

This is an app integration issue.  I can see that you're working with our app integration team on this. I'll leave you in their very capable hands.


Question: best way to hook into an enclosure download?

There was one other question about this specific to an app but perhaps I should come up with a more general use case.
Currently I monitor a number of feeds. Each of these feeds provides and enclosure in each entry which I wish to pass to a second program (which also has an emacs integration). I'm trying to work out what the best way to integrate would be. At the moment I do this:

select feed entry
hit 'd' , enter (for 1), enter (for save in downloads)
switch to dired buffer
select file, send it to second program

Maybe this would be aided by an elfeed-downloaded-hook? Although it would be nice to just have a function to say "Download enclosure N and pass the final filename to this function". Or is what I need already in elfeed?

So I ended up doing this which is good enough for me:
(defun my-add-first-elfeed-enclosure-to-transmission ()
  "Queue the first enclosure (if it is a torrent)."
  (interactive)
  (let ((enclosures (elfeed-entry-enclosures elfeed-show-entry)))
    (when (and enclosures
               (string-match-p
                "application/x-bittorrent"
                (nth 1 (-first-item enclosures))))
      (transmission-add (-first-item (-first-item enclosures))))))

(use-package elfeed
  :ensure t
  :bind (:map elfeed-show-mode-map
              ("C-c C-c" . my-add-first-elfeed-enclosure-to-transmission))
  :config (setq elfeed-enclosure-default-dir "~/torrent/"
                elfeed-log-level 'debug
                elfeed-use-curl 't))

Is this worth adding to a FAQ? I'm not sure how hidden this will be once you close the issue.


Location in detail page shows html text along with offset.

Describe the bug
On the developer details page, the location is showing extra HTML content in the offset field
To Reproduce
Steps to reproduce the behavior:

Go to developer details page
see location

Screenshots

Additional context
Add any other context about the problem here.

Thanks for reporting, @kaushikhande! This was occurring in dev because of the newly enabled annotate_rendered_view_with_filenames configuration. I pushed a fix to render the nested View Component in the HTML and not the .erb file and all seems to be working again.


Add minimum IAM policy permissions example

For a more restrictive policy where you may have multiple terragrunt
repos storing their state files in a single bucket and using a single
table for state locking, you still want to limit what parts of the
bucket can be modified.
Signed-off-by: Stephen Hoekstra<EMAIL_ADDRESS>

@brikis98 as requested, let me know if the wording needs amending.


Add test cases to Sum of Multiples

Adding test cases to Sum of Multiples as per exercism/exercism.io#2341.

Looks good, thanks!


Cursor goes out of bounds

When an IOS iPad user backspace deletes the last line of a study block, the cursor moves out of bounds.
https://photos.app.goo.gl/96LG5t9GJUbrb6yt5

I think this is related to how we're doing the scrolling for the editor. We should put it back to the way it used to be.


How to handle .ruby-version file between server and development

I have a .ruby-version file in my repo pointing to 2.6.3, but on the server I only have 2.6.3-jemalloc. This causes errors on the server when trying to run rails console or any ruby command. What is the recommended approach for this?

Ok so I was thinking about creating a PR for this and I was going to use .fullstaq-ruby-version but then it dawned on me that maybe it would be better to simply provide a .fullstaq-ruby-variant file.
This .fullstaq-ruby-variant file would not specify a Ruby version but instead specify one of the following variants:

normal
jemalloc
malloctrim

Then when loading the .ruby-version file to we can append the appropriate .fullstaq-ruby-variant to the end of the string.
What do you think?

Yes I am fine with that too.

This for the contribution!


"favicon fetcher" project site is not responsive

Describe the bug
In the project Favicon Fetcher (https://github.com/TusharKesarwani/Front-End-Projects/tree/main/Projects/Favicon Fetcher)
the website is not responsive.
Expected behavior
The website should be responsive and the contents on page should be properly aligned.
Screenshots

@TusharKesarwani @zugzwang03 @harshit-paneri Please assign this issue to me under SSOC 2.0


chore(master): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-09-15)
Features

apply footer on app (c5a77d0)
av and prepare default layout (e27445a)
bootstrap and coreui styles (40cfe09)
change style (46645c8)
chart drawing (6d00b65)
compoentn for weather and fetch data (18ab85d)
data component (68757b6)
export def layout (d48d90d)
favicon and app title (dc8ef5e)
footer component (1533e2e)
format (3db9db2)
github stuff (9e3fe5a)
graph (e110014)
heartbeat overview component (d7354cc)
home component (c6170f9)
http setup for backend (15dd83f)
init angular (e3689e1)
init angular project (204aeca)
load with appmodule instead of standalone app component (73aa83b)
route (9ebab6a)
sample for weather params (242ef51)
show percentage (0bb2f08)
typeerror, angular tries accessing array at index 0 when data array is still empty (cf3f48e)
use data from mongo (630426d)
use tabulator table (141aacc)
weather (206686b)
weather graph and sunlight/daylight hours (c246fba)
weather params (2f075de)
wip - weather data (f443774)

Bug Fixes

load module (706885d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sanriodev/energy-householder-angular/releases/tag/v1.0.0 :sunflower:


testConnectToPeerMethodReturnsTCPPort fails randomly.

Issue is reproducible only when we run all tests.

PR #1339 is out.


Tobin update unique error enums 2 21

Updates to spec parser and a big unique id update.

Hey Tobin, is this guy still active or was it superseded by this commit?
commit 050525ace9efc0067fad208154e87db9b87808c7
Author: Tobin Ehlis<EMAIL_ADDRESS>Date:   Wed Apr 19 17:30:52 2017 -0600
scripts:Updated Validation IDs for spec 1.0.48

No longer valid


Help Needed: Remove a removed light from the plugin

Hey guys, yesterday I discovered that a yeelight bulb was broken, so I removed it from the app. Unfortunately it's still in the plugin.
I tried to ignore the light by id in the plugins' settings, but it doesn't work. I only can ignore lights when they are online
How can I remove it, can you help me?

You can clear the cached Accessories in Homebridge. This should remove all lights and re-detect those that you still have. See: https://github.com/oznu/homebridge-config-ui-x/issues/525


[Create news] User can publish news without selecting any tag

Environment: macOS 10.15, Google Chrome Version 81.0.4044.129 (64-bit), Safari Version 13.0.2 (156<IP_ADDRESS>.1)
Reproducible: always
Build found: commit b517ccb
Preconditions:

User is registered in GreenCity.
Go to GreenCity https://ita-social-projects.github.io/GreenCityClient/#/welcome
Sign in with registered credentials.
Go to 'Eco news'.

Steps to reproduce:

Click on 'Create news'.
Enter data into 'Title' field, e.g. "Всесвітній день води".
Enter data into 'Content' field, e.g. "22 березня святкуємо Всесвітній день води".
Don't select any tag =>> 'Publish' bttn is disabled.
Download image (attached) and fill in 'Source' field, e.g. "https://iameco.com.ua/22-bereznya-svyatkuyemo-vsesvitnij-den-vody/".

Click on 'Preview' bttn.
Click on 'Publish' bttn on the right upper corner of the 'Preview' page.

Actual result:

'Publish' bttn is enabled;
News is published on 'Eco news' page.

Expected result:

'Publish' bttn is disabled;
News is not published on 'Eco news' page.

User story #537, #548

Verified


Update ata-update-1.6-migration-guide.md

Suggested another link for Download update 1.6 as aka.ms/ata1_6update does not work.
Suggested link is sourced from the announcement blog post: https://blogs.technet.microsoft.com/ata/2016/05/05/advanced-threat-analytics-new-version-1-6-is-now-available/

those are perfect catches - thanks so much - do I need to do anything to approve this so it goes live?

Yes, you'll have to merge the PR. I'm not a github master yet so I'm not sure how that's done. :)
Nicholas Romyn
On Fri, May 6, 2016 at 12:38 AM -0700, "rkarlin"<EMAIL_ADDRESS>wrote:
those are perfect catches - thanks so much - do I need to do anything to approve this so it goes live?

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/Microsoft/ATADocs/pull/4#issuecomment-217371513


[Mono] Change Plane intersect methods to return nullable Vector3

Null is returned whenever the methods do not find an intersection, matching the behavior of GDScript. The method return type has been changed to Vector3?, which breaks compatibility. Might be best to put this on the 4.0 milestone. Closes #32614
Also there's minor semantic changes to use IsZeroApprox().

@neikeq I tagged for 4.0 as advised by @aaronfranke, but if you want to include it in 3.2, I don't mind. C# support is still dubbed "alpha" and AFAIK we already broke some degree of compatibility in the C# API between 3.1 and 3.2.

I don't mind either way. If we want to include it in 3.2 and compatibility breakage is an issue, we could add methods like bool Intersect3(Plane b, Plane c, out Vector3 intersection) instead of replacing the current ones.

Compatibility for C# is indeed broken between 3.1 and 3.2, for example, enums in C# now behave properly in 3.2 (and old C# code that used enums is broken).

Well let's merge then if compatibility breakage for C# is expected.

Thanks!


message longevity on the server

Some thoughts discussed with @mwfarb :

allow undownloaded message live longer.
remove messages in server storage if push notification is not succeed.
user quota design for message storage (e.g.., 5 MB per device).
cron jobs for status check on all registered devices.

issues related to 4 are listed in https://github.com/SafeSlingerProject/SafeSlinger-AppEngine/issues/13 and https://github.com/SafeSlingerProject/SafeSlinger-AppEngine/issues/12.

Issues 1 and 2 resolved in https://github.com/SafeSlingerProject/SafeSlinger-AppEngine/commit/b3964e58921e4fee88d85da8fc18eca9681d858f.

Issue 4 resolved in 7cb7e5a.


NestedScrollView doesn't react to NeverScrollableScrollPhysics on device or emulator

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

NestedScrollView doesn't react to NeverScrollableScrollPhysics on device or emulator (at least on Android OS) but on the web it work perfectly well (tested on dart pad).
Steps to reproduce

Run sample code on android device or emulator
Try to scroll

Expected results
No scrolling
Actual results
On physical device it still scrolling
Code sample
Code sample
import 'package:flutter/material.dart';

/// Flutter code sample for [NestedScrollView].

void main() => runApp(const NestedScrollViewExampleApp());

class NestedScrollViewExampleApp extends StatelessWidget {
  const NestedScrollViewExampleApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: NestedScrollViewExample(),
    );
  }
}

class NestedScrollViewExample extends StatelessWidget {
  const NestedScrollViewExample({super.key});

  @override
  Widget build(BuildContext context) {
    final List<String> tabs = <String>['Tab 1', 'Tab 2'];
    return DefaultTabController(
      length: tabs.length, // This is the number of tabs.
      child: Scaffold(
        body: NestedScrollView(
          physics: const NeverScrollableScrollPhysics(),
          headerSliverBuilder: (BuildContext context, bool innerBoxIsScrolled) {
            // These are the slivers that show up in the "outer" scroll view.
            return <Widget>[
              SliverOverlapAbsorber(
                // This widget takes the overlapping behavior of the SliverAppBar,
                // and redirects it to the SliverOverlapInjector below. If it is
                // missing, then it is possible for the nested "inner" scroll view
                // below to end up under the SliverAppBar even when the inner
                // scroll view thinks it has not been scrolled.
                // This is not necessary if the "headerSliverBuilder" only builds
                // widgets that do not overlap the next sliver.
                handle:
                    NestedScrollView.sliverOverlapAbsorberHandleFor(context),
                sliver: SliverAppBar(
                  title:
                      const Text('Books'), // This is the title in the app bar.
                  pinned: true,
                  expandedHeight: 150.0,
                  // The "forceElevated" property causes the SliverAppBar to show
                  // a shadow. The "innerBoxIsScrolled" parameter is true when the
                  // inner scroll view is scrolled beyond its "zero" point, i.e.
                  // when it appears to be scrolled below the SliverAppBar.
                  // Without this, there are cases where the shadow would appear
                  // or not appear inappropriately, because the SliverAppBar is
                  // not actually aware of the precise position of the inner
                  // scroll views.
                  forceElevated: innerBoxIsScrolled,
                  bottom: TabBar(
                    // These are the widgets to put in each tab in the tab bar.
                    tabs: tabs.map((String name) => Tab(text: name)).toList(),
                  ),
                ),
              ),
            ];
          },
          body: TabBarView(
            // These are the contents of the tab views, below the tabs.
            children: tabs.map((String name) {
              return SafeArea(
                top: false,
                bottom: false,
                child: Builder(
                  // This Builder is needed to provide a BuildContext that is
                  // "inside" the NestedScrollView, so that
                  // sliverOverlapAbsorberHandleFor() can find the
                  // NestedScrollView.
                  builder: (BuildContext context) {
                    return CustomScrollView(
                      // The "controller" and "primary" members should be left
                      // unset, so that the NestedScrollView can control this
                      // inner scroll view.
                      // If the "controller" property is set, then this scroll
                      // view will not be associated with the NestedScrollView.
                      // The PageStorageKey should be unique to this ScrollView;
                      // it allows the list to remember its scroll position when
                      // the tab view is not on the screen.
                      key: PageStorageKey<String>(name),
                      slivers: <Widget>[
                        SliverOverlapInjector(
                          // This is the flip side of the SliverOverlapAbsorber
                          // above.
                          handle:
                              NestedScrollView.sliverOverlapAbsorberHandleFor(
                                  context),
                        ),
                        SliverPadding(
                          padding: const EdgeInsets.all(8.0),
                          // In this example, the inner scroll view has
                          // fixed-height list items, hence the use of
                          // SliverFixedExtentList. However, one could use any
                          // sliver widget here, e.g. SliverList or SliverGrid.
                          sliver: SliverFixedExtentList(
                            // The items in this example are fixed to 48 pixels
                            // high. This matches the Material Design spec for
                            // ListTile widgets.
                            itemExtent: 48.0,
                            delegate: SliverChildBuilderDelegate(
                              (BuildContext context, int index) {
                                // This builder is called for each child.
                                // In this example, we just number each list item.
                                return ListTile(
                                  title: Text('Item $index'),
                                );
                              },
                              // The childCount of the SliverChildBuilderDelegate
                              // specifies how many children this inner list
                              // has. In this example, each tab has a list of
                              // exactly 30 items, but this is arbitrary.
                              childCount: 1,
                            ),
                          ),
                        ),
                      ],
                    );
                  },
                ),
              );
            }).toList(),
          ),
        ),
      ),
    );
  }
}

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.10.6, on Microsoft Windows [Version 10.0.19045.3208], locale uk-UA)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.6.5)
[√] Android Studio (version 2022.3)
[√] Connected device (3 available)
[√] Network resources

Hi @OlehMarch. Thanks for filing this issue. You'd need to set the physics: NeverScrollableScrollPhysics() in CustomScrollView as well. Doing this, on stable, the issue seems to still perists but this appears to have been fixed by this PR: https://github.com/flutter/flutter/pull/123109
Please try switching to the latest master channel to confirm this.
flutter channel master
flutter upgrade

I'll close this issue as fixed. If you still experience this issue, kindly leave a comment below and I will reopen this. Thank you!
stable, master flutter doctor -v

[✓] Flutter (Channel stable, 3.10.6, on macOS 13.0 22A380 darwin-arm64, locale
    en-NG)
    • Flutter version 3.10.6 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (4 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

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

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

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

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 14 (API 34) (emulator)
    • iPhone 14 (mobile)          • 1A122DE2-0CAB-4C3E-A395-691BF27D626F • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)             • macos                                •
      darwin-arm64   • macOS 13.0 22A380 darwin-arm64
    • Chrome (web)                • chrome                               •
      web-javascript • Google Chrome 115.0.5790.170

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

• No issues found!

[✓] Flutter (Channel master, 3.13.0-19.0.pre.78, on macOS 13.0 22A380 darwin-arm64, locale en-NG)
    • Flutter version 3.13.0-19.0.pre.78 on channel master at /Users/damilolaalimi/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 436df69a46 (4 hours ago), 2023-08-08 01:58:16 -0400
    • Engine revision 146c4c9487
    • Dart version 3.2.0 (build 3.2.0-42.0.dev)
    • DevTools version 2.26.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

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

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

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

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 14 (API 34) (emulator)
    • iPhone 14 (mobile)          • 1A122DE2-0CAB-4C3E-A395-691BF27D626F • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 13.0 22A380 darwin-arm64
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 115.0.5790.170

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

• No issues found!

I can confirm that on latest master channel (with physics: NeverScrollableScrollPhysics() in CustomScrollView) it work as expected.


Windows 11 update on article

Windows 11 is the most recent version of the OS and this article is also applicable to Windows 11.
Pull request guidance
Thank you for your contribution. The content team is monitoring the pull requests. Once your pull request is ready for review, please add a comment such as "PR is ready for review".

The changes are live at https://learn.microsoft.com/en-us/troubleshoot/windows-server/backup-and-storage/windows-installations-disk-duplication. This PR will be closed.


Update security-best-practices.md

I just noticed there is discrepancy between this part of doc and other part. They both use the word "environment" but with a different meaning. One refers to dev, test, etc environments while the other one refers to specific deployments within those environments (this is up to interpretation). To prevent confusion, I've explained it here. Once merged, I'll go and edit the other part to make them in sync.
Also, I've added a new tip: business cortical workloads may need their own environment.

@chcomley
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Revert visibility of FunctionRegistration to public

Fixes #102

Had discussion with @ericbottard on this and while my original motivation was that FunctionRegistration is only used internally to wrap functions, the @Autowired directive essentially implies that it can be provided by "outside forces" which essentially is impossible now with FunctionRegistration being inaccessible.
Merging. . .


Update commands.js

Mention logging in.
Add link to repo.
Fixes #9

This is a good suggestion! Thanks @travs!


Realm emoji should use absolute URLs when referenced in missed-message emails

If you PM someone with a realm emoji in the message, the URL it generates for the missed-message email is like this:

yay

Which of course is a broken link, because it's being included in your email, and that doesn't have the same server as your Zulip server.
We need to use absolute URLs here.
I feel like this might be a regression.  @HarshitOnGitHub can you look into this one?

@zulipbot claim


Drag and Drop Kanban Board

Make the board more responsive and add option to drag and drop the application.

Committed in this PR: https://github.com/SoftwareEngNoobs/jobtrackr/pull/6


Oracle Mocker inconsistency when mocking 2+ calls

Bug
Inconsistency if I am mocking 3+ oracles at once. E.g. this doesn't work
If I comment out the last Oracle Mocker (enqueuePublicFunctionCall, then the test works (just mocks callPrivateFunction but if I uncomment, then it says callPrivateFunction not mocked but it is with the expected params!)
Installation Method
None
Nargo Version
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

@rahul-kothari is this still an Issue? If possible it'd be great if you could supplement a minimal reproduction.

it is - minimal reproduction is hyper-linked!


chore: Update pull_request_template.md to add more docs in PR Checklist

PR Type
What kind of change does this PR introduce?

Other: PR Template update

Description
chore: Update pull_request_template.md to add docs in PR Checklist

im getting mixed feelings for all these checks... the sheer numbe is getting a bit out of hand...
also you seem to forgot to mention Uno.Themes/doc/styles/ which we need to update on control style updates

im getting mixed feelings for all these checks... the sheer numbe is getting a bit out of hand... also you seem to forgot to mention Uno.Themes/doc/styles/ which we need to update on control style updates

Hum you are not wrong that starting to be a lot, but also it is easy to forget some docs. Hum let me rethink that

maybe just a single bullet point to remind them of there are docs to keep updated, and links the major folders (/doc, /doc/styles/) and when to do so or why?


Creating a Validator documentation not clear - polkadot

From the documentation https://github.com/threefoldtech/tfchain/blob/development/substrate-node/doc/creating_validator.md
Synchronize Chain Data
Note: By default, Validator nodes are in archive mode. If you've already synced the chain not in archive mode, you must first remove the database with polkadot purge-chain
After the nightly build has succeeded, when trying to run the above command, it shows that polkadot command not found
Tried searching online for the polkadot, but it gives me manuals or git repo with manual on how to install the full chain, which is not exactly what we need.
It's a bit unclear about the polkadot package/chain part

Docs updated in https://github.com/threefoldtech/tfchain/commit/e3a08a8ca607e5bac6748532e24648bd6db981b8
@dmitri-alleo can you try again?


Incompatible with Distant Horizons

https://www.curseforge.com/minecraft/mc-mods/distant-horizons

I came here for the same reason.
Distant Horizons has a GitLab issue for their end at: https://gitlab.com/jeseibel/minecraft-lod-mod/-/issues/94.
I've added what I've uncovered from my logs to their issue.
I suspect compatibility to be most easily fixed by having their end either stop distant rendering when player is near a portal, or at least stop rendering distant chunks for the opposite side of the portal from the player (not sure how that would work in multi-player if players are on both side of the same portal...)
Anyway, it may still take effort on the part of both mods to rectify the problems.

Is this still the case (Entity Culling and Distant horizons)

I'm also interested in the combination of these two mods. I'd be totally satisfied if distant horizons only works in the current dimension and the portal world is limited by the normal rendering distance.
Not sure if I can contribute anything, my knowledge of both Minecraft and (portal) rendering isn't good enough.

This should be fixed with DH 2.1.0

Source


fetch racing laptime content across the internet and put it in tracknow

search across youtube and other services for racing content and put it in tracknow.

https://developers.google.com/youtube/v3/docs

This will be a major feature in the alpha phase. Scrape content from relevant websites(motorsports, simracing...) and store them in the redis and show them on the discover page on tracknow. Users can get all their news , info... on tracknow instead of checking different places for information.
its quite complex. but its doable...


access denied exception

I tried to edit a simple property, "Subtitle" of videoproperties of a StorageFile ( which I got from KnownFolder.VideosLibrary ) and after editing subtitle I tried to call this method : "SavePropertiesAsync() and I got access denied exception.

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

ID: 8a63def1-857d-ce78-5917-9ab0afc3b93a
Version Independent ID: 182a7510-4b9f-136a-0496-ff969ff76f0a
Content: VideoProperties.SavePropertiesAsync Method (Windows.Storage.FileProperties) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.storage.fileproperties.videoproperties.yml&version=GBlive&_a=contents
Service: unspecified
GitHub Login: @wdg-dev-content
Microsoft Alias: wdg-dev-content

@touseefbsb, are you using the videosLibrary capability in your app manifest?

@laurenhughes  yes ofcourse I am using it, if I don't use it then I get aces denied on KnownFolders, but I am using it and I am only getting the exception when I try to save the properties, can you please show me a code sample of editing and saving a property on a storagefile object? so that I can know if I am missing something? also I am getting partial storage file with "queryoptions" indexing, can that be the issue?

btw in the docs there is no example code to show how to use this method.

Here's a C# example of editing and saving the subtitle property
StorageFolder storageFolder = KnownFolders.VideosLibrary;
StorageFile sampleFile = await storageFolder.GetFileAsync("video_test.mov");

VideoProperties videoProperties = await sampleFile.Properties.GetVideoPropertiesAsync();
videoProperties.Subtitle = "test value";
await videoProperties.SavePropertiesAsync();

I tested this code and I'm not getting any exceptions. I'll get a code sample added to the page. If you're still having issues with your scenario, I'd recommend posting a question on stack overflow.

@laurenhughes  yes please if you can provide me a sample app, where i can test it on my machine it would be great, if that doesnt work on my machine, I'll open a question on stackoverflow.

Sorry, I don't have a sample app to share. To test editing the property, I made a method in a blank app UWP project, enabled the "videosLibrary" capability, and tested editing a property on a video I already had on my local machine ("video_test.mov").

yeah I am still getting access is denied, but thanks for the help, I guess Ill open a issue on stackoverflow. @laurenhughes


Dark mode only

Is it possible to use only the dark version?
Or to reverse the default theme (dark instead of light) to avoid the little flash at the beginning of the page loading?

Is it possible to use only the dark version? Or to reverse the default theme (dark instead of light) to avoid the little flash at the beginning of the page loading?

You can add this script after <body> tag to prevent that issue
const appBody = document.body;
if (localStorage.getItem('theme') == 'theme-dark') {
     appBody.classList.add("theme-dark");
} else {
     appBody.classList.add("theme-light");
};

I think this is need to be fixed. I will added the type: bug label

The script above works for me.
For the inital flashing of the button you can use this script:
<script>     // fix the flashing of the button     const toggler = document.getElementById('toggle-dark');     if (localStorage.getItem('theme') == 'theme-dark')         toggler.checked = true;     else         toggler.checked = false; </script>

The script above works for me.
For the initial flashing of the button you can use this script:
// fix the flashing of the button
const toggler = document.getElementById('toggle-dark');
if (localStorage.getItem('theme') == 'theme-dark')
    toggler.checked = true;
else
    toggler.checked = false;

The script above works for me.
For the initial flashing of the button you can use this script:
// fix the flashing of the button
const toggler = document.getElementById('toggle-dark');
if (localStorage.getItem('theme') == 'theme-dark')
    toggler.checked = true;
else
    toggler.checked = false;

It is just make a button is checked by checking the theme-dark existence in localStorage.

The script above works for me.
For the initial flashing of the button you can use this script:
// fix the flashing of the button
const toggler = document.getElementById('toggle-dark');
if (localStorage.getItem('theme') == 'theme-dark')
    toggler.checked = true;
else
    toggler.checked = false;

It is just make a button is checked by checking the theme-dark existence in localStorage.
It is should keep the default check.


Scalar timescale

I'd like to implement a way to set a scalar or typical timescale. This is because when solving a jellyroll model or liionpack model if I set electrode height to an input then this changes the timescale along with a few other parameters. It would simplify things to just have a constant value across all the simulations using the model. If I replace the model.timescale with a Scalar though I see different results when changing the scalar value. See below for example:
import pybamm

parameter_values = pybamm.ParameterValues("Chen2020")

model1 = pybamm.lithium_ion.SPM()
model2 = pybamm.lithium_ion.SPM()

symbolic_tau = pybamm.LithiumIonParameters().tau_discharge
tau = parameter_values.process_symbol(symbolic_tau)
tau_eval = tau.evaluate()
model2.timescale = pybamm.Scalar(2*tau_eval)

sim1 = pybamm.Simulation(model=model1, parameter_values=parameter_values)
sim2 = pybamm.Simulation(model=model2, parameter_values=parameter_values)

sol1 = sim1.solve([0, 1800])
sol2 = sim2.solve([0, 1800])

pybamm.QuickPlot(solutions=[sol1, sol2]).dynamic_plot()

I tried delaying the build of the model until after the timescale is updated but this didn't help
import pybamm

parameter_values = pybamm.ParameterValues("Chen2020")

model1 = pybamm.lithium_ion.SPM()
model2 = pybamm.lithium_ion.SPM(build=False)

symbolic_tau = model2.param.tau_discharge
tau = parameter_values.process_symbol(symbolic_tau)
tau_eval = tau.evaluate()

# Double the timescale
# This shouldn't affect the solution as it's used for non-dimensionalising
# But it does
new_timescale = pybamm.Scalar(2*tau_eval)
model2.timescale = new_timescale
model2.param.timescale = new_timescale
model2.param.tau_discharge = new_timescale

model2.build_model()

sim1 = pybamm.Simulation(model=model1, parameter_values=parameter_values)
sim2 = pybamm.Simulation(model=model2, parameter_values=parameter_values)

sim1.build()
sim2.build()

# Makes no difference
sim2.built_model.timescale = new_timescale
sim2.built_model.param.timescale = new_timescale
sim2.built_model.param.tau_discharge = new_timescale

sol1 = sim1.solve([0, 1800])
sol2 = sim2.solve([0, 1800])

plot = pybamm.QuickPlot(solutions=[sol1, sol2])
plot.dynamic_plot()

Running a few more methods that get run in lithium_ion_parameters and base_lithium_ion_model also doesn't help
import pybamm

parameter_values = pybamm.ParameterValues("Chen2020")
parameter_values.update({'Current function [A]': "[input]"})

model1 = pybamm.lithium_ion.DFN()
model2 = pybamm.lithium_ion.DFN(build=False)

symbolic_tau = model2.param.tau_discharge
tau = parameter_values.process_symbol(symbolic_tau)
tau_eval = tau.evaluate()

# Double the timescale
# This shouldn't affect the solution as it's used for non-dimensionalising
# But it does
new_timescale = pybamm.Scalar(2*tau_eval)
model2.timescale = new_timescale
model2.param.timescale = new_timescale
model2.param.tau_discharge = new_timescale

model2.param._set_dimensionless_parameters()
model2.param._set_input_current()
model2.set_standard_output_variables()

model2.build_model()

sim1 = pybamm.Simulation(model=model1, parameter_values=parameter_values)
sim2 = pybamm.Simulation(model=model2, parameter_values=parameter_values)

sim1.build()
sim2.build()

sol1 = sim1.solve([0, 1800], inputs={'Current function [A]': 5.0})
sol2 = sim2.solve([0, 1800], inputs={'Current function [A]': 5.0})

plot = pybamm.QuickPlot(solutions=[sol1, sol2])
plot.dynamic_plot()

Discussion with @rtimms and @tinosulzer might need to define a tau_discharge and a tau_typical and use the ratio of these in the model equations to make sure non-dimensionalization is consistent and correct parameter is being used.

It's not really a bug as changing the timescale like I am trying to do already is quite hacky but it would be a useful feature to add and has highlighted some potential limitations in the code. Could be a real pain to fix though so might have to try some different hacks elsewhere

The following works as expected (both models give the same answer), so at least it's not something in the solver
import pybamm

sols = []
for tau in [1, 10]:
    model = pybamm.BaseModel()
    tau = pybamm.Scalar(10)
    model.timescale = tau
    Q = pybamm.Variable("Q")
    model.rhs = {Q: tau} # non-dimensionalized with timescale
    model.initial_conditions = {Q: pybamm.Scalar(1)}
    model.variables = {"Q": Q}
    solver = pybamm.CasadiSolver()
    sol = solver.solve(model, [0, 1])
    sols.append(sol)

pybamm.dynamic_plot(sols, ["Q"])

Discussion with @rtimms and @tinosulzer might need to define a tau_discharge and a tau_typical and use the ratio of these in the model equations to make sure non-dimensionalization is consistent and correct parameter is being used.

Yes, I think there is a factor "tau_discharge/timescale" (usually equal to 1) missing in the particle equations. The easiest fix might be to redefine gamma_p (which already appears in all the appropriate equations), and introduce an equivalent gamma_n.

Fixed, will open a PR later. "timescale" will be an option to pass to the model, and model.timescale will be a read-only variable


GDAL 'not available'

I have installed GDAL through conda:
$ conda install -c conda-forge gdal==2.4.4

I get :
Collecting package metadata (current_repodata.json): done
Solving environment: done

# All requested packages already installed.

After that I installed:
python3 -m pip install pyrasterframes

But, when I run :
from pyrasterframes.utils import gdal_version
print(gdal_version())

I get  response:
not available

Why do I get this error. I actually have gdal installed and also import gdal  works . Is there a way I can fix tis error ?

One clarifying question, when you run python3 -m pip install pyrasterframes is the conda environment with gdal 2.4.4 activated? e.g.
$ conda install -n some_conda_env -c conda-forge gdal==2.4.4
$ conda activate some_conda_env
(some_conda_env) $ python3 -m pip install pyrasterframes
(some_conda_env) $ python3 -c "from pyrasterframes.utils import
gdal_version; print(gdal_version())

On Wed, Sep 16, 2020 at 1:35 PM sleeping barber<EMAIL_ADDRESS>wrote:

I have installed GDAL through conda:
$ conda install -c conda-forge gdal==2.4.4
I get :
Collecting package metadata (current_repodata.json): done
Solving environment: done
All requested packages already installed.
After that I installed:
python3 -m pip install pyrasterframes
But, when I run :
from pyrasterframes.utils import gdal_version
print(gdal_version())
I get response:
not available
Why do I get this error. I actually have gdal installed and also import
gdal works . Is there a way I can fix tis error ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/locationtech/rasterframes/issues/507, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3P4L26CXQILCJFH7GT6JTSGDZPPANCNFSM4RPFP3VQ
.

@vpipkt  I do:
$ conda info --envs

# conda environments:
#
base                     /opt/conda
some_conda            *  /opt/conda/envs/some_conda

Then I install GDAL
conda install -n some_conda -c conda-forge gdal==2.4.4

After that I activated :
conda activate some_conda

$ python3
Python 3.6.11 | packaged by conda-forge | (default, Jul 23 2020, 22:18:32)
>>> from pyrasterframes.utils import gdal_version
>>> print(gdal_version())

I get this error :
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/opt/conda/envs/some_conda/lib/python3.6/site-packages/pyrasterframes/utils.py", line 106, in gdal_version
    fcn = RFContext.active().lookup("buildInfo")
  File "/opt/conda/envs/some_conda/lib/python3.6/site-packages/pyrasterframes/rf_context.py", line 84, in active
    "RasterFrames have not been enabled for the active session. Call 'SparkSession.withRasterFrames()'.")
AttributeError: RasterFrames have not been enabled for the active session. Call 'SparkSession.withRasterFrames()'.

In order to resolve this error I did:
>>> from pyrasterframes.utils import create_rf_spark_session
>>> from pyrasterframes.utils import gdal_version
>>> spark = create_rf_spark_session()

20/09/17 07:04:40 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
Using Spark's default log4j profile: org/apache/spark/log4j-defaults.properties
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).

>>> print(gdal_version())
20/09/17 07:04:54 WARN GDALRasterSource$: GDAL native bindings are not available. Falling back to JVM-based reader for GeoTIFF format.
not available

Apologies it should be:
(some_conda_env) $ python3 -c "from pyrasterframes.utils import
create_rf_spark_session, gdal_version; spark = create_rf_spark_session();
print(gdal_version());"

On Thu, Sep 17, 2020 at 3:07 AM sleeping barber<EMAIL_ADDRESS>wrote:

@vpipkt https://github.com/vpipkt I do:
$ conda info --envs
conda environments:

base                     /opt/conda
some_conda            *  /opt/conda/envs/some_conda
Then I install GDAL
conda install -n some_conda -c conda-forge gdal==2.4.4
After that I activated :
conda activate some_conda
$ python3
Python 3.6.11 | packaged by conda-forge | (default, Jul 23 2020, 22:18:32)

from pyrasterframes.utils import gdal_version
print(gdal_version())

I get this error :
Traceback (most recent call last):
File "", line 1, in
File "/opt/conda/envs/some_conda/lib/python3.6/site-packages/pyrasterframes/utils.py", line 106, in gdal_version
fcn = RFContext.active().lookup("buildInfo")
File "/opt/conda/envs/some_conda/lib/python3.6/site-packages/pyrasterframes/rf_context.py", line 84, in active
"RasterFrames have not been enabled for the active session. Call 'SparkSession.withRasterFrames()'.")
AttributeError: RasterFrames have not been enabled for the active session. Call 'SparkSession.withRasterFrames()'.
In order to resolve this error I did:

from pyrasterframes.utils import create_rf_spark_session
from pyrasterframes.utils import gdal_version
spark = create_rf_spark_session()

20/09/17 07:04:40 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
Using Spark's default log4j profile: org/apache/spark/log4j-defaults.properties
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).

print(gdal_version())
20/09/17 07:04:54 WARN GDALRasterSource$: GDAL native bindings are not available. Falling back to JVM-based reader for GeoTIFF format.
not available

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/locationtech/rasterframes/issues/507#issuecomment-694008950,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3P4L2W3YSDJVOE62L4NXLSGGYRVANCNFSM4RPFP3VQ
.

marking as question until we can show there is some unexpected behavior going on

@ajinkya933 did the above resolve your issue/question?

No

Happy to engage on our gitter chat, we can try to work through the details. Feel free to update the issue here with more details about your environment etc. Which OS are you using? And for reference what pyrasterframes version?
My next guess here is that perhaps providing the gdal library path as a JVM option may help. Refer to this thread:  https://github.com/locationtech/rasterframes/issues/356#issuecomment-554503657

Why @ajinkya933 closed this issue?  Did you fix your problem?
I've got the same error.
Install GDAL 2.4.4 through conda 4.9.2 and macos Catalina version 10.15.6.

Why @ajinkya933 closed this issue?  Did you fix your problem?
I've got the same error.
Install GDAL 2.4.4 through conda 4.9.2 and macos Catalina version 10.15.6.

@tosen1990 since you're using MacOS your issue can not be that easily resolved. We fixed that only in gdal-warp bindings 3.1.1 that are only compatible with GDAL 3.1.x.
Try to depend on GDAL 3.1.x and gdal-warp-bindings 1.1.1

@tosen1990 since you're using MacOS your issue can not be that easily resolved. We fixed that only in gdal-warp bindings 3.1.1 that are only compatible with GDAL 3.1.x.
Try to depend on GDAL 3.1.x and gdal-warp-bindings 1.1.1


make e2e tests not run in verbose mode

Based on the discussion here, we should move from verbose mode of running e2e tests to running tests in normal mode. We don't necessarily need verbose because golang test suite will dump the output of the failed test anyway if any test fails.

big +1
it is impossible to find anything useful in current output.
verbose output should be dumped only if there is an error

PR updated here: https://github.com/kedgeproject/kedge/pull/388


Remove deprecation warnings for FFs with no replacement

Description
Currently there is no replacement for the featureflags, where we state that those are deprecated. However thats not the way how we should do it. We should first introduce a replacement and then deprecated.
How can this be tested?
Apply a Dynakube with one of the feature flags -> no message should appear

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 57.26%. Comparing base (32a4011) to head (af7a6c3).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3423      +/-   ##
==========================================
- Coverage   57.32%   57.26%   -0.07%
==========================================
  Files         345      344       -1
  Lines       19790    19753      -37
==========================================
- Hits        11345    11311      -34
+ Misses       7208     7206       -2
+ Partials     1237     1236       -1

Flag
Coverage Δ

unittests
57.26% <ø> (-0.07%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Various ll::request improvements

Made while working on porting simple.rs to the enum API and in preparation for #136

Merged. Thanks!


Caesarcipher code in Python

This is a code for Caesarcipher's algorithm in Python. @prateekiiest , please do have a look at it!

SAME here

@rahulkumaran please check here


CARDS-1726: Create a simple placeholder html page template to be displayed when CARDS is down

This PR adds a project-specific CARDS is down page that is returned to the client by the reverse proxy whenever the CARDS Docker container is unavailable.
To test:

Build this branch with mvn clean install.
For each project in cards4care, cards4kids, cards4lfs, cards4proms:

python3 generate_compose_yaml.py --dev_docker_image --oak_filesystem --cards_project <PROJECT NAME> && docker-compose build && docker-compose up -d, replacing <PROJECT NAME> with the actual project name (eg. cards4care).
Wait for CARDS to become available at http://localhost:8080
docker-compose stop cardsinitial
Visiting http://localhost:8080 should now show the project-specific down for maintenance page.
docker-compose down && docker-compose rm && docker volume prune -f && ./cleanup.sh

Also, attempt the above with a generic CARDS deployment (ie. don't specify the --cards_project parameter)
Verify that this works in all 4 proxy modes: HTTP, HTTP+SAML, HTTPS, HTTPS+SAML.

Improved code in generate_compose_yaml.py to address https://github.com/data-team-uhn/cards/pull/1049#discussion_r887293073. Tested all 4 CARDS projects (Cards4Care, Cards4Kids, Cards4LFS, Cards4Proms) as well as the generic CARDS. Screenshots are below.

375ee7b6f8.png)

Tested with docker, looks good to me. One minor thing: I seem to be getting a different default font

Hm, I don't like this font, we need to make sure it's sans.

Fixed in 0181b34.

Tested, all looks good.