New Version Bug: blank page when joining any room on iphone Safari browser

Describe the bug
On my iphone 12 pro safari browser, I can see list of rooms after github signin to next.dogehouse.tv, but when I join any room, it will lead me to a blank page without any voice.
To Reproduce
Steps to reproduce the behavior:

Go to next.dogehouse.tv in safari browser on iphone
Click on Github login
Click on the first room
See a blank page and no voice

Expected behavior
See a room with people and hear voice from speakers
What device are you on?
iPhone 12 Pro
Additional context
no

I cannot reproduce this bug on the latest version of iOS on an iPhone 12 Pro. Could it be that you see the white screen that prompts you to click a button because Dogehouse requires user interaction before it will work?

Can reproduce on iPhone 7

Does it work with other browsers on the iPhone or is it just exclusively Safari?
Can you confirm if the behavior is the same if you open Safari on another device (laptop) but with a mobile width?

The problem is gone


Github connector now returns a full group list when no org is specified

Fixes #1102
If you asked for the groups scope but didn't specify any orgs, the github connector would return an empty set of groups. This should now be fixed.
Based on this change we now return a list of groups that looks like the following:
[
    "org-1:team-1",
    "org-1:team-2",
    "org-1:team-3",
    "org-2:team-4",
    "org-3"
]

If teams exist, they are scoped to an org, otherwise the org is listed on its own.

Looks like I'm using features in httptest server that we're available in golang 1.8.

This looks great! I was just about to implement this.

@ericchiang Can we get this reviewed?

@joshadambell updated as per your suggestions. thanks.

@ericchiang Anything missing on this PR?

I ended up deleting the branch that this PR points to.
We've ended up with a bunch of work on our fork of Dex so I spent a lot of time cleaning things up so that we can at least more easily keep up to date with upstream and keep tabs on any PR-able work on separate branches instead. More info here: https://github.com/concourse/dex/issues/1
We'll probably re-submit this PR (along with our other pr/* branches) in the future, against https://github.com/concourse/dex/tree/pr/github-teams-no-org-and-slugs


Category instances for agda stdlib things

Back in the day, mostly as an excuse to teach myself both agda and category theory, I did a bunch of formalizations of categorical properties for various built-in agda things on top of copumpkin/categories. I was just revisiting some of that work, because I missed relaxing at the end of a long day with a proof checker, and discovered this library - which I'm super glad exists! My question for the maintainers - I'm gonna go start working through porting over some of those proofs into this library, for no reason other than that it's fun - is this something you'd want contributed back here? And if so, where would it belong? Any other things you'd want to tell an emphatic non-mathematician before I go too deep down the rabbit hole?

It's really hard to respond at that high level of generality.
We do welcome PRs, very much. We vastly prefer small PRs over large ones, even if that means dozens of PRs. That way we can make sure every piece gets a proper look, and is integrated with the rest.
I'm also quite happy to reply to design questions here on Issues. So before going down a big rabbit hole, ask.
If you're thinking of contributing a lot of 'instances', we should definitely discuss that. I think it makes a huge amount of sense to have a large library of instances - but the ones in this library should be the ones that tend to be re-used internally. I have myself been thinking of starting a category-instances for all those things which build on top of this library but doesn't naturally fit into it.
There are bits of documentation here and there. It could be improved a lot - PRs for that are super-welcome. But specific complaints are too. That's easier to fix than generic "not enough docs".

hey, thanks for the really quick, pointed, and detailed reply! all good stuff to know. I'm totally on board with a category-instances library and think it makes a lot of sense - eg I'm currently going down the path of proving the list functor (and then maybe, and then the product bifunctor probably) - more than happy to redirect that to another repo if you go off and start it


GroupsUpdate API allows updating read-only groups, which results in a falsely successful operation

Problem
Updating system, read-only Groups using the GroupsUpdate API seems to succeed...

The operation succeeds and throws no error or exceptions.
The group title is updated to the new value LOCALLY in the Contacts Provider Groups table.

HOWEVER, the title change does not actually get applied to the remote sync servers.
Here is the scenario,

Change the title of a system, read-only group using GroupsUpdate. For example, change title "Friends" to "Enemy".
Operation succeeds and the group title is now "Enemy" in the local Contacts Provider database.
❌ The group title in other devices that is logged into the same Account is still "Friends" even after triggering contacts sync in the system account settings on both devices.
❌ The group title reverts back to "Friends" after removing the Account and adding it back in.

Solution
Fail the GroupsUpdate operation for groups that are read-only to ensure users do not falsely believe that the operation actually succeeded.

🔥 🔥 🔥 🔥 🔥 🔥 🔥 🔥
This is now release in 0.3.1!
🔥 🔥 🔥 🔥 🔥 🔥 🔥 🔥


Add llvm-16 support

This PR upgrades the LLVM version from fifteen to sixteen. It also create a missing symlink in llvm lib.
PS: When LLVM apt sort the issue of missing symlink we can remove the ln -s command.

LGTM. Considering the complexity of the build process for this repo and marvin-cxx, you might want to look into using a Nix flake for setting up the environment instead. While the Dockerfile will be the same for our cloud builds, having the ability to get your local setup running with just nix develop will make it a lot easier for others to hit the ground running if they decide to contribute to the analyzers!

For now local setup on Mac OS is as simple as the running the following commands
$ brew install llvm@16 && tack --build lta

where tack is Python script to invoke cmake after setting some envs.
I am yet to get a chance to use Nix but do they support installing latest version of llvm?

Yes, they do have llvm_16 on the Nix repository.


Need server version requirements

I am familiar enough with the roadmap from OCS through to Skype for Business that I know when your docs say "works with Skype for Business server" in so many words that you mean 2015 only, but I have clients still using Exchange 2010 in multi-forest and they couldn't use the older LRS, let alone the newer SRS V2 systems. Somehow they still think the SRS v2 should be capable of working with Lync server 2013. It would be invaluable to me if one of the SRS V2 pages could explicitly state it's not compatible with Lync server 2013 as lots of people think the Skype for Business umbrella includes 2013 server.

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

ID: 515e5db8-20be-f979-5de9-3998713518dd
Version Independent ID: a284fee3-f324-2984-431e-df14d3c4bd2d
Content: Skype Room Systems v2 requirements
Content Source: Skype/SfbServer/plan-your-deployment/clients-and-devices/requirements.md
Product: skype-for-business-itpro
GitHub Login: @jambirk
Microsoft Alias: jambirk

See aka.ms/SRSDocs where that is explicitly noted. we can also add that to this page too, if you're asking.
… and since the preview of Server 2019 went live, "works with Skype for Business Server" is a blanket statement referring to Server 2015 and Server 2019. Where one or the other is specifically meant, the server version is called out.


Adds latest/ prefix to mongodb charm

Required in order to pick the proper channel.
This will fix the currently failing integration tests [1].


Make the tree binary optional in write_tables_test [phab:D872]

Created by shahms at 2016-05-03 17:30:59:
The tree command is not typically installed and is just used for pretty output.
Make it optional.

schroederc commented inline (context lost during migration) at 2016-05-03 17:33:43:
Let's just delete it.

shahms wrote at 2016-05-03 17:34:52:

just remove the call to tree

schroederc wrote at 2016-05-03 17:37:03:
Please edit the title/summary

schroederc accepted this revision at 2016-05-03 17:37:03

shahms closed via commit 0b680cc62280e2d75d860c7f8ce9946471a5a3b0 at 2016-05-03 17:40:14


Car not stop when moving forward

When car moving forward that accel = 0, car will infinity moving with same velocity if car go forward, not go right or left...just go forward. Car never stop. Please check this bug.

You need to check out the axle friction variable

Where i can find axle friction variable. This should setting as default. Thanks in advanced.

Axle friction is on the wheels. I think the default should be zero because it has a subtle effect on vehicle performance that might go unnoticed when configuring vehicles.


CA and LPIF regression and coverage document

Need a similar document to AXI4 + scripts to run regression and coverage for CA and LPIF. If you can keep this grid modular/independent like AXI it will be useful.

CA regression document was uploaded in 0.9.6 (and maybe earlier).

Uploaded as part of Feb 18 release.

Fixed.


Bug 1990850: Handle property and dependency values of type BLOB in ListBundles.

Per https://www.sqlite.org/datatype3.html#type_affinity:

In order to maximize compatibility between SQLite and other database
engines, and so that the example above will work on SQLite as it
does on other SQL database engines, SQLite supports the concept of
"type affinity" on columns. The type affinity of a column is the
recommended type for data stored in that column. The important idea
here is that the type is recommended, not required. Any column can
still store any type of data. It is just that some columns, given
the choice, will prefer to use one storage class over another. The
preferred storage class for a column is called its "affinity".

Even though the properties and dependencies tables are created with
column affinity TEXT on both their "type" and "value" columns, SQLite
does not require values stored in those columns to have that type.
The query backing the registry.ListBundles API recently began using
the json_object function from the json1 SQLite
extension (https://www.sqlite.org/json1.html#jobj), which does not
support arguments of type BLOB. The docs state:

If any argument to json_object() is a BLOB then an error is thrown.

But it seems that no error is actually being returned. Instead, the
result set it empty. This may be an issue with the SQLite driver? It
seems to produce an error in sqlite3's interactive mode.

/lgtm

/retest


[backport] Avoid # NOQA in docstrings

Backport of #6184

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit d5f2af02fc174cf3e009b4659dcfb08106dce830, target branch v5) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

LGTM! Test failures are not related to this PR.

Jenkins CI test (for commit d5f2af02fc174cf3e009b4659dcfb08106dce830, target branch v5) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)


Only and Except doing as a list of OR permissions

It looks like when I add a Only to my route if I put a list of permissions when the user can have any one of them they are treated like an AND not and OR. So user must have view and edit permissions not view or edit permission. Is there a way to set these up as a AND?

Looks like have the same problem.

Looks like have the same problem. I configured PermissionStore and added "anonymous", "user" and "admin". Added only to my route as ["user", "admin"]. If I corretcly understand only means "user" or "admin". When I try to go to this route with "user" permission nothing happened.


Travis failures remote failures though tests pass locally

Travis has been failing on master for a while now, seemingly with the iOS simulator bootstrap error.

There are now two failing UI tests in FirebaseUISampleUITests that need to be fixed.

Updated FirebaseUI project and UI tests


Secretstore broken since 1.13

Expected Behavior

Expected that the load of secret from a kubernetes/file store works without issue.
Actual Behavior

Since the v1.13, the extension to load secret is broken.
  .AddDaprSecretStore(configuration =>
                {
                    configuration.KeyDelimiters = new List<string>() { ".", "__" };
                    configuration.SecretDescriptors = secretDescriptors;
                    configuration.Store = "kubernetes";
                    configuration.Client = daprClient;
                }
            )

Steps to Reproduce the Problem

Create a simple secret store component
ex :
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: kubernetes
spec:
  type: secretstores.local.file
  version: v1
  metadata:
  - name: secretsFile
    value: C:/Users/emmanuel.dinicola/.dapr/components/secrets.json
  - name: nestedSeparator
    value: ":"
  - name: multiValued
    value: true

Configure the secrets.json like this
{
  "pharma-common": {
    "MyFirstKey": "1",
    "MySecondKey": "2"
  }
}

This problem has been introduced by this https://github.com/dapr/dotnet-sdk/issues/887
Indeed if you look the code that has been changed in DaprSecretStoreConfigurationProvider.cs
         foreach (var key in result.Keys)
                    {
                        if (data.ContainsKey(key))
                        {
                            throw new InvalidOperationException(
                                $"A duplicate key '{key}' was found in the secret store '{store}'. Please remove any duplicates from your secret store.");
                        }

                        data.Add(normalizeKey ? NormalizeKey(secretDescriptor.SecretName) : secretDescriptor.SecretName, result[key]);
                    }

The data.Add will throw error on the second key cause the secretDescriptor.SecretName will not change.
== APP == Unhandled exception. System.ArgumentException: An item with the same key has already been added. Key: pharma-common
== APP ==    at System.Collections.Generic.Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
== APP ==    at System.Collections.Generic.Dictionary`2.Add(TKey key, TValue value)
== APP ==    at Dapr.Extensions.Configuration.DaprSecretStore.DaprSecretStoreConfigurationProvider.LoadAsync() in D:\GitPR\dotnet-sdk\src\Dapr.Extensions.Configuration\DaprSecretStoreConfigurationProvider.cs:line 232

If we back to the previous code in 1.12 and replace secretDescription.SecretName by key, then no more error.
Release Note

RELEASE NOTE:

Is there any chance that will be fixed soon? We want to upgrade to this version that are .net 8 compliant

Is there any chance that will be fixed soon? We want to upgrade to this version that are .net 8 compliant

We're looking into it. Expect updates soon.


feat(server.certs): Remove team email

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Description
This PR ports the changes made in #15724 by @BerkeleyTrue to the staging branch.

merged! https://github.com/freeCodeCamp/freeCodeCamp/commit/a051d4d44378f5cdea3c3d839e0846e011654337
🚀 🐰 👍 ✨


Issue in cue sheet keeps mp3splt from recognizing book title

Cue sheets right now start with Title "{book title}"
This need to be TITLE "{book title}" for certain utilities (like mp3splt) to recognize the book title.
Pull request coming

Thanks for your contribution! PR was merged and this change will be available in the 0.3.0 release (pending Mozilla approval now)


fix hostrange() missing 1 required positional argument: 'database_directory'

A minor fix for https://github.com/phac-nml/mob-suite/issues/148
I tested on the conda install, and made some minor white space changes (can revert if needed). Let me know if there are questions.

Thank you so much for this fix!

I got the same error - What should I do to fix this?
Thanks.

I got the same error - What should I do to fix this?
I am using the conda install - version 3.1.5
How can I incorporate these changes into the condo install? I have not done something like this before.
Thanks for your help.

I figured out the location of the scripts and changed them - it works now - thanks for the fix :)


json nested arrays

How should nested arrays be handled in the outline files?
I have the following dataset, containing nested array named "categories":
    { "results": [
	{
        "categories": [
            "foo", "bar"
        ],
        "city": "Vienna"
    }
    ] }

Thanks, Maks

I haven't fully tested the functionality, but you can use numbers in place of string in your dot notation as follows:
{
  "map": [
    ["city", "city"],
    ["category 1", "categories.0"],
    ["category 2", "categories.1"]
  ],
  "collection": "results"
}

If you wanted up to 5 categories, then I'd add a rule for each array index.
If you had something different in mind we can discuss how to implement it.

No activity in a month, I'll assume this issue is closed


v2.0.2 Disappearing whitespace between  and

Given the following input:

foo

Becomes:

foo

This is incorrect because the space between the closing
 and opening  has completely disappeared.

Hi, I have found the issue... :/ the problem was that "dom->loadHTML()" without "LIBXML_HTML_NODEFDTD" will not keep the white-spaces.

@voku Merry Christmas and thanks for your work! 🎅
I will of course be testing it when a tag is available and hopefully I won't need to create any more issues 😁

@Bilge I wish you also some nice christmas days   🎅


Make consensus SlotWorker don't assume a slot is time / duration

This removes the last bit of assumption that a slot is always time / duration. This will be required by parachains where a slot will be the
relay chain block number. Besides this there are also some other drive
by changes. One more notable is that on_slot now returns a
SlotResult that holds the block and a potential storage proof.
To simplify the implementation and usage of the SimpleSlotWorker the
SlotWorker trait is now implemented for each type that implements
SimpleSlotWorker.

Contributes towards: https://github.com/paritytech/cumulus/issues/216


Exam mode: The students can see the "commit history" of programming exercises in an exam.

Describe the bug
The students can see the commit history of programming exercises in an exam.
To Reproduce

Solve an exam with programming exercises
Click in the comit history
You will be able to see the creator of the exercise.

Expected behavior
The students should be able to see the commit history.
Screenshots
No response
Which version of Artemis are you seeing the problem on?
7.3.0
What browsers are you seeing the problem on?
Chrome
Additional context
No response
Relevant log output
No response

As far as I am aware, this is the expected behavior - what is the problem with seeing the exercise creator in the git history?
The commits are supposed to be squashed (= only one initial commit visible), as long as this was the case I do not think that this is a "bug"

As far as I am aware, this is the expected behavior - what is the problem with seeing the exercise creator in the git history?
The commits are supposed to be squashed (= only one initial commit visible), as long as this was the case I do not think that this is a "bug"

I didn't know it was expected. However, it would be good if the students don't know who created the exercise.

If the students clone the repo, they can see it locally as well. When we used Bitbucket, they could open it and see a full commit history with authors here as well. I don't see a way to prevent this.
Of course, you can obfuscate the author of the commit message, by using an instructor account without a real name. However, I do not really see an issue if the students can see the author name.
Why would you hide it?


Extract utility functions to utils.js

Related to the conversation on https://github.com/lesshint/gulp-lesshint/pull/39, this is pre-work to adding a failOnWarning() method to the plugin.
Extracted util methods: getSeverityCount, isError, isWarning, and pluralize.
Lint free and all tests are passing. I tried to maintain the existing style. One thing of note, I made the use of a dangling comma consistent as that was used in most existing cases, just not all.

Thanks for working on this!

One thing of note, I made the use of a dangling comma consistent as that was used in most existing cases, just not all.

Awesome! That's sounds like me being sloppy 😄 I'll add it to the lint rules.


06/12/2018 18:54 : Create Pull Request for branch master -> prTesting1_2018061218

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Numpy Solutions

I’ve numbered the exercises in the numpy course 1-10 and created solution notebooks for 2, 3, 6, 7, 8 and 9; exercise 10 already had a solution notebook which has been renamed accordingly. I didn’t create solutions for exercises 1, 4 and 5 as these were focused more on exploring possibilities rather than asking direct questions. I do think it would be worth creating solutions for 1, 4 and 5 but perhaps this could be done an another occasion.
These solutions need to be checked for clarity and if they’re appropriate. I’m particularly uncertain about exercises 8 and 9. Most of the tasks in exercise 8 are unclear in their meaning, perhaps they should be changed in the course to be easier to understand.
For exercise 9 (the masked array question) there’s also some confusion. I’ve answered them as best I can, however based on my understanding the order is confusing and possibly redundant. For example I’m not sure how the first 2 tasks were expected to be pulled off and what “What special masked array creation routines exist to do this kind of thing more efficiently?” means in relation to these tasks. Then when we’re asked to “create a plain array from a masked array, but using a different fill-value for masked points” it’s not clear what particular method is being hoped for.
There was also a solution workbook for a question that no longer exists, I’ve renamed this numpy_exercise_old.ipynb but it can probably be removed.

Thank you @LukeC92, I am happy with these changes! :+1:

Regarding the tests failing, this has been fixed in PR #135


Update readme for new Input / Output syntax

right now this is the old syntax, it needs to be updated with the optional + msg_input! macros

done


NanoPC-T4 Debian Image

Hello.......i bought a NanoPC-T4, but somehow i just dont get it to run Linux on it......
i flashed a few versions, but anytime i switch power on it just happens nothing, no tv output, no nothing.....i do see the NanoPC IP in my router menu but cannot connect in any way.........but the lan leds look like theyre working........maybe you can give me a tip which image i need to fire this thing up!
best regards

Sorry, I don't have NanoPC-T4, nor I know what it is, nor I do plan to support it.

@hanshaze It might help if you are on the right site ;)
'NanoPC-T4' is a friendlyarm product, so you need to check out what they have to say about it.
http://wiki.friendlyarm.com/wiki/index.php/NanoPC-T4
@ayufan  It might be worth keeping an eye on that NanoPC-T4 - it is RK3399 based, and is very similar to the ROCKPro64 feature-wise. If they make any code available there might be stuff that can be borrowed ;)


Add Local Git Instructions & Variable Persistence

Adds local git instructions (#101)
Adds variable persistence and restore (#111)
Addresses usability in creating more variables automatically without the user having to find them

Resolves #101, #111

This is what it looks like in git after the first four modules


Add datasource.json on BIDS export

To make it fully compatible, I should generate datasource.json when user request for BIDS export.

_info.json is generated to store the brainlife metadata.


video glitch/distorted at the beginning

I used model 4.0, scale = 4, exp = 4, practical-rife

Hello, the v4.0 model seems to be abnormal for the scale > 2. I don't know the specific reason.

Thanks @hzwer for the information. Could you give me the link to the best model that is best for anime?

As far as I know, many people are using this version：
https://github.com/hzwer/arXiv2020-RIFE/issues/41#issuecomment-785548531
Maybe it’s helpful to check the comment of flowframes
https://nmkd.itch.io/flowframes

how about the version 2.4 which was also commented on https://github.com/hzwer/arXiv2020-RIFE/issues/41#issuecomment-785548531.
Could you give me google download link for version 2.3 and 2.4, I want to try them both! Thank you @hzwer

v2.2 https://drive.google.com/file/d/1luM3AnlxR-pEH4ugOA8iHMBgYYPONxOj/view?usp=sharing
v2.3 https://drive.google.com/file/d/1VdzN9beh7qi4up9hS7dsEg6WOwUdBh-Q/view?usp=sharing
v2.4 https://drive.google.com/file/d/1wsQIhHZ3Eg4_AfCXItFKqqyDMB4NS0Yd/view?usp=sharing

thanks @hzwer, and could you help me with this https://github.com/hzwer/arXiv2020-RIFE/issues/222?

Thanks @hzwer for the information. Could you give me the link to the best model that is best for anime?

quality (both 2D and 3D):
2.3 > 2.4 > 4.0 > 3.1 > 3.9 > 3.8
speed:
4.0 > 3.9 > 3.8 > 3.1 > 2.3/2.4
I don't recommend using other versions as they are either redundant or broken.

Thanks @hzwer  for the reply. How about --scale?, raising that value does not make the video smoother at all, and sometimes it creates a laggy video

Thanks @hzwer for the information. Could you give me the link to the best model that is best for anime?

quality (both 2D and 3D): 2.3 > 2.4 > 4.0 > 3.1 > 3.9 > 3.8 speed: 4.0 > 3.9 > 3.8 > 3.1 > 2.3/2.4
I don't recommend using other versions as they are either redundant or broken.

I'm not sure but model 2.3 created slightly more distortions than model 4.0

Thanks @hzwer for the information. Could you give me the link to the best model that is best for anime?

quality (both 2D and 3D): 2.3 > 2.4 > 4.0 > 3.1 > 3.9 > 3.8 speed: 4.0 > 3.9 > 3.8 > 3.1 > 2.3/2.4
I don't recommend using other versions as they are either redundant or broken.

I'm not sure but model 2.3 created slightly more distortions than model 4.0

It is possible. However, in majority of cases 2.3 wins.


Add note about required let's encrypt port

Let's Encrypt uses tls-sni to validate the certificate on the standard https port 443. If the registry is outwardly listening on a different port Let's Encrypt will not issue a certificate.

Current coverage is 51.26% (diff: 100%)

Merging #1940 into master will decrease coverage by 9.69%

@@             master      #1940   diff @@
==========================================
  Files           124        123      -1
  Lines         11082      10919    -163
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6756       5598   -1158
- Misses         3442       4584   +1142
+ Partials        884        737    -147

Powered by Codecov. Last update 7dcac52...279c02a

Not sure where to rise it but it should also be added to this section imo: https://docs.docker.com/registry/deploying/#/let-s-encrypt


PIREP rejection doesn't move pilot back to previous airport

If it's been submitted. Or just analyze the submitted states before canceling.

Fix for release


[ical] crash after installation

Nikola v7.8.15, clean installation from pip
Python 3.6.4
icalendar 4.0.1
os: linuxmint 18.3
After run nikola plugin -i ical nikola crash with any command:
$ nikola version
Traceback (most recent call last):
  File "/home/user/workspace/nikola/env/bin/nikola", line 11, in <module>
    sys.exit(main())
  File "/home/user/workspace/nikola/env/lib/python3.6/site-packages/nikola/__main__.py", line 176, in main
    _ = DN.run(oargs)
  File "/home/user/workspace/nikola/env/lib/python3.6/site-packages/nikola/__main__.py", line 352, in run
    self.nikola.init_plugins()
  File "/home/user/workspace/nikola/env/lib/python3.6/site-packages/nikola/nikola.py", line 1195, in init_plugins
    self._activate_plugins_of_category("ShortcodePlugin")
  File "/home/user/workspace/nikola/env/lib/python3.6/site-packages/nikola/nikola.py", line 1341, in _activate_plugins_of_category
    plugin_info.plugin_object.set_site(self)
  File "/home/user/workspace/nikola/blog/plugins/ical/ical.py", line 36, in set_site
    super(CalendarPlugin, self).set_site(site)
  File "/home/user/workspace/nikola/env/lib/python3.6/site-packages/nikola/plugin_categories.py", line 413, in set_site
    site.register_shortcode(self.name, self.handler)
AttributeError: 'CalendarPlugin' object has no attribute 'handler'

Should work now, please reinstall.


Add ability to get file from client

Allow borland to pull files from the client.

Add in c8c3b1c61651c70718f7a8159f7a0dc57464715c. cc: @geek


refactor: make the API case clear and stable

Ⅰ. Describe what this PR did
make the API clear and stable

don't use wait style to wait for downloading image
use the util function to make the case clear
add missing case wanted

for missing case, we can use helpwantedCase to mark, like
PASS: api_container_start_test.go:71: APIContainerStartSuite.TestStartDetachKeyWork     1.339s
PASS: api_container_start_test.go:27: APIContainerStartSuite.TestStartOk        1.391s
PASS: api_container_start_test.go:56: APIContainerStartSuite.TestStartPausedContainer   1.391s
PASS: api_container_start_test.go:45: APIContainerStartSuite.TestStartStoppedContainer  1.979s
SKIP: api_container_stop_test.go:64: APIContainerStopSuite.TestInvalidParam (help wanted: container api stop bad request case)

Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Describe how you did it
refactor
Ⅳ. Describe how to verify it
CI
Ⅴ. Special notes for reviews

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@04b5951). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2042   +/-   ##
=========================================
  Coverage          ?   63.75%
=========================================
  Files             ?      200
  Lines             ?    15531
  Branches          ?        0
=========================================
  Hits              ?     9902
  Misses            ?     4396
  Partials          ?     1233

Flag
Coverage Δ

#criv1alpha1test
33.77% <ø> (?)

#criv1alpha2test
34.17% <ø> (?)

#integrationtest
38.45% <ø> (?)

#unittest
22% <ø> (?)

There is only one failure:
----------------------------------------------------------------------
FAIL: /home/travis/gopath/src/github.com/alibaba/pouch/test/api_container_create_test.go:28: APIContainerCreateSuite.TestBasic
/home/travis/gopath/src/github.com/alibaba/pouch/test/api_container_create_test.go:110:
    // volume related
    c.Assert(got.HostConfig.Binds, check.DeepEquals, []string{"/tmp:/tmp"})
... obtained []string = []string{"/tmp:/tmp", "/var/lib:/var/lib/lxc:shared", "/var/lib/lxcfs/proc/uptime:/proc/uptime", "/var/lib/lxcfs/proc/swaps:/proc/swaps", "/var/lib/lxcfs/proc/stat:/proc/stat", "/var/lib/lxcfs/proc/diskstats:/proc/diskstats", "/var/lib/lxcfs/proc/meminfo:/proc/meminfo", "/var/lib/lxcfs/proc/cpuinfo:/proc/cpuinfo"}
... expected []string = []string{"/tmp:/tmp"}
----------------------------------------------------------------------

@fuweid

LGTM util the api_container_create_test is fixed.


Fix IllegalArgumentException: Not an array type for method like void method(@Nullable Object[])

Fixes https://github.com/wildfly/jandex/issues/102
Note: there are lots of ways to structure the project, so I would be glad to hear suggestions.
Currently jandex is top-level packaging=bundle, which limits Maven's ability to have modules.
However, even the structure I suggest enables me to open both projects in IDEA, run test, debug project, etc.

@vlsi Thanks for the contribution! I did a small refactor to also cover inner class params: https://github.com/n1hility/jandex/commit/c3d0cc87c81b83915d50429b9d0537ccca0979bc
I temporarily removed the GitHub action and replaced the java8 test module vs the current approach since these are currently nonfictional and the intention is to fix the bug in a current release quickly.
I didn't update your PR quite yet since I wanted to give you a chance to point out any concerns/issues. The other generics issue I will post up tomorrow, your fix is correct but I just need to verify the compatibility impact.

@vlsi Thanks for the contribution! I did a small refactor to also cover inner class params: https://github.com/n1hility/jandex/commit/c3d0cc87c81b83915d50429b9d0537ccca0979bc
I temporarily removed the GitHub action and replaced the java8 test module vs the current approach since these are currently nonfictional and the intention is to fix the bug in a current release quickly.
I didn't update your PR quite yet since I wanted to give you a chance to point out any concerns/issues. The other generics issue I will post up tomorrow, your fix is correct but I just need to verify the compatibility impact.

Ok this PR has been updated with the commits I linked. If you have any concerns let me know, otherwise I plan to merge these tomorrow.

Ok this PR has been updated with the commits I linked. If you have any concerns let me know, otherwise I plan to merge these tomorrow.

removed the GitHub action

This is sad since currently, it is hard to see if the fix works for Calcite or not.

removed the GitHub action

This is sad since currently, it is hard to see if the fix works for Calcite or not.


Allow Navigation of Truncated Results

Only the first 10 results of a query from /docs or /examples are displayed for every query to meet Discord's message quota. We need to better indicate incomplete results when we truncate beyond 10 results. In addition, it would help to allow navigation between "pages" of results via reaction or the like.

We can do this via reaction for now, but buttons are WIP from Discord: https://support.discord.com/hc/en-us/articles/1500012250861-Bots-Buttons.

Added in #27.


Update user guide, add leeyjjoel.adoc

Pull Request Test Coverage Report for Build 471

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.282%

Totals

Change from base Build 470:
0.0%

Covered Lines:
1888

Relevant Lines:
2267

💛  - Coveralls


Autocomplete / Suggestion list in #cite() like for @

Description
It'd be quite if #cite() would also have the autocomplete/suggestion list that @ has, such that one doesn't have to either type the citation key by hand or copy-paste it in there (or – how I currently do it – write @, use the suggestion list, then highlight the cite-key and @-symbol, CTRL+SHIFT+W to launch a autohotkey script that writes #cite(brackets: false, "{CITEKEY}") ).
Use Case
Makes regular use of #cite() much easier and doesn't require user to know or copy-paste cite-keys all the time.

Shipped in Typst 0.9.0.


Accessing ethers in my hre extension breaks gas-reporter

Hi, I have this in my hardhat config and this part hre.ethers.ContractFactory is breaking gas-reporter.
extendEnvironment((hre) => {
  hre.ethers.ContractFactory
});

Steps to reproduce:

Initially plugin will report only header, run npx hardhat test to see
Comment out  hre.ethers.ContractFactory in hardhat config
See full report generated npx hardhat test

Repo: https://github.com/Lopol2010/gas-reporter-issue

@Lopol2010 Thanks for these reproduction steps, will take a look.

We experienced a similar problem. Really looking forward to a fix.

Any issues with this should be fixed in the latest version (2.0). The reporter now uses Hardhat's extendProvider plugin hook to wrap the provider and this should resolve any weird conflicts that previously existed.
Closing for now but if anyone runs into this please ping.
https://github.com/cgewecke/hardhat-gas-reporter/releases/tag/v2.0.0


[Snyk] Upgrade classnames from 2.4.0 to 2.5.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade classnames from 2.4.0 to 2.5.0.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-12-28.

Release notes

Package name: classnames

2.5.0 - 2023-12-28Prepare for release of version 2.5.0 (#344)

2.4.0 - 2023-12-26Prepare for release of version 2.4.0 (#338)

    from classnames GitHub release notes

Commit messages

Package name: classnames

28ea73f Prepare for release of version 2.5.0 (#344)
dfeb7f3 Fix URL for GIT repository in package (#343)
8a840ea Add `exports` field to package (#342)
ea8e839 Restore ability to pass TypeScript `interface` (#341)
930c748 Bump `classnames` to the latest version (#340)
d2d10f5 Bump tsd from 0.30.0 to 0.30.1 (#339)

Compare

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

I have read the CLA Document and I hereby sign the CLA


Add tracking of page type, using a custom property

The custom property stuff is quite nice to use to track page types in WordPress, which adds a nice dimension to Plausible Analytics :)

Hi @jdevalk!
Thanks for the submission! Since we're already working on the custom props feature in v2.0.0, I'd like to invite you to make your code compatible with the self_provisioning_api branch.


Total spent time is not calculated on roadmap view

If an issue has sub-tasks with spent time logged it appears on the roadmap view with 0 time spent (that is not correct).

if version name is chinese and then time tracking of Roadmap show chinese ????。


Update Equivalent Ripped Groove to match the total volume flux

Update the groove to include the real volume flux of the ribbed groove.
Resulting is the rip distance.

Better as single example in Notebook


Tag name is hard to read in editing mode

Tags are hard to read in editing mode.
I use dark theme and have live preview turned on. It looks the same if live preview is turned off
Tags in editing mode:

Tags in view mode:

Obsidian version is 1.0.0
Theme is of the latest version, I just updated it.

For some reason, this problem disappeared. I don't think there has been any update.
Tags now look like that

@bicolino34 there are changes being made to the master branch lately, will tag a new version this weekend with all the updates.

This is fixed in the latest version, please update your theme thanks! ❤


TFC Queue Manager Missed 2 Stuck Runs

Describe the bug
Aug 31st I got a message from David S in tenant requests asking to kill a run that had been stuck in cost estimation since yesterday. I went to check it and there were two runs from different workspaces stuck in cost estimation. There were no messages in the public-cloud-tfc-queue rocket chat channel so I think there might be an issue with the queue manager.
Additional context

Ukeme was having issues with false positives before and it looked hard to solve (the TFC API isn't stateful and would require a database to record current state)
Living with the false positives was determined to be less effort than building a new app to track the state.
We may be moving to new deployment patterns so this could be thrown away.
https://github.com/BCDevOps/octk-tfc-queue-manager

DIFFICULT TO DEBUG requires a TFC account then block it.  How do we generate an event we don't know what it is.
Leave in backlog until we know more or we hear more screams.
Definition of done

Investigate why the queue manager didn't catch those runs
Have the blocked events that were found show up in the channel public-cloud-tfc-queue (right now they don't)
Investigate candy/Wade B public-cloud-tfc-queue entries and if false positive, filter them
Fix the issue
We may need to recreate the rocket chat webhook if it's still linked to Ukeme's handle
Cap to a 3 (see additional info)

@MaxWardle  Can you please confirm this is still relevant

not relevant


Render Map when API ready

I want to show a spinner until the data from the api is here, but when i try the code below the pins and the cluster weren't drawn at all.

renderMap() {
    if(this.props.stationsArray.length < 1){
      return(
        <Spinner />
      );
    }

    return (
      <ClusteredMapView
        ref = {(ref)=>this.mapView=ref}
        style={{flex: 1}}
        initialRegion={REGION_HOME}
        data={this.props.stationsArray}
        textStyle={{ color: '#65bc46' }}
        containerStyle={{backgroundColor: 'white', borderColor: '#65bc46'}}
        renderMarker={this.renderPins.bind(this)}
      >
      </ClusteredMapView>
    );
  }

Is it possible to add an custom pin to the map that is not part of the clusters?

we'll investigate as soon as possible
good catch, we'll add this as soon as possible

Thanks for the feedback!

Hi @KapelPTronic ,
could you post the code of this.renderPins and the structure of the underlying object?

@leolusoli
  renderPins(marker) {
    console.log("renderPins");
    var markerImage = require('../assets/img/marker/marker_77b83a.png');
    return(
      <Marker
        key={marker.uuid}
        coordinate={marker.location}
        image={markerImage}
      >
        <Callout tooltip style={{ flex: 1, position: 'absolute'}} onPress={()=> Actions.detail({marker})}>
          <TouchableOpacity >
            <InfoWindow {...marker} />
          </TouchableOpacity>
        </Callout>
      </Marker>
    );
  }

Please try first with a standard Marker (and with no custom image) to make sure the issue is actually in this component implementation.

hey, I tried it with the standard Marker but the pins and cluster do not show on the map. the renderPins function don't fire once but only when I show the spinner first. if i comment out the if all works perfectly
by the way thanks for the fast update for the custom marker, works perfectly!!!!

@KapelPTronic you are right about first mount after loading spinner, we fixed and releasing soon. Thanks!

@39otrebla nice man, no problem, i have to say thanks 👍

new update works fine!!!
big thanks @39otrebla @leolusoli


Added Application::setContainerDefinitions() to configure DI in declarative style

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
not implemented yet

Fixed issues
#11758

[x] Draft implementation & docs
[x] Tests

This PR is an implementation of idea, suggested by @hiqsol in #11758
The proposal is to allow dependency injection configuration be:

declarative
located in the standard place
be consistent with configuring of similar construction (aliases)

Example:
config.php

return [
    'id' => 'my-super-application',
    'basePath' => dirname(__DIR__),
    'bootstrap' => ['log'],
    'containerDefinitions' => [
          yii\web\Request::class => app\components\Request::class,
          yii\web\Response::class => [
              'class' => app\components\Response::class,
              'format' => 'json'
          ],
    ]
];

I have doubts about naming/structure of the config.
We currently have 2 ways of defining dependency:

Yii::$container->setSingleton(). In this case get() returns the same service object each time. It's not making an object singleton though.
Yii::$container->set(). In this case get() creates new instance of the object each time.

First way is preferred for services so it's mainly used when wiring stuff from container to object constructors. In Yii apps it's not used much because it's partially covered by service locator.  Second way is good for setting defaults for widgets etc.
I have some alternatives for separate containerDefinitions and containerSingletons keys.
1st (Symfony style):
'services' => [
    yii\web\Request::class => app\components\Request::class, // by default class instances are cached
    yii\web\Connection::class => [
        'class' => app\components\Connection::class,
        'shared' => false, // explicitly declare that we need new instances each time
    ],
],

2nd (grouped):
'services' => [
    'create' => [
        yii\web\Request::class => app\components\Request::class, // by default class instances are cached
    ],
    'share' => [
        yii\web\Connection::class => [
           'class' => app\components\Connection::class,
           'shared' => false, // explicitly declare that we need new instances each time
       ],
],

3rd (alternative naming):
'dependencies' => [
    'prototypes' => [
        yii\web\Request::class => app\components\Request::class, // by default class instances are cached
    ],
    'shared' => [
        yii\web\Connection::class => [
           'class' => app\components\Connection::class,
           'shared' => false, // explicitly declare that we need new instances each time
       ],
],

@samdark
1st (Symfony style) variant looks most simple and convenient.
But I would vote for dependencies not services.
Unfortunately 1st variant doesn't work for callback definitions:
'dependencies' => [
    yii\web\Connection::class => function () {
    }, /// no place to put shared = false :(
],

So the 4th variant:
'dependencies' => [
    'definitions' => ...
    'singletons' => ...
],

These names are consistent to yii\di\Container.

I've updated examples.

I don't like the word dependencies. It's too generic and confusing with either assets / composer / NPM dependencies.
I'm also wondering why we are moving away from the 'main subject': $container. I would expect something consistent like:
    'container' => [
          yii\web\Request::class => app\components\Request::class,
          yii\web\Response::class => [
              'class' => app\components\Response::class,
              'format' => 'json'
          ],
    ]

Because we're not setting container options here but setting dependencies into container.

'dependencies' => [
'definitions' => ...
'singletons' => ...
],

We try to avoid multi-level configurations on top.

How about:
`dicDefinitions`
`dicSingletons`

Because we're not setting container options here but setting dependencies into container.

mwah...it's not logical.

mwah...it's not logical.

Disagreed.

Disagreed.

then I would expect something like Yii::$dependencies instead of Yii::$container
This is not consistent. Its like allowing 'obscure' grouping terminology:

Yii::$box
Yii::$groups
Yii::$pool
Yii::$collection

@dynasource
dependencies just because of Dependency Injection :)
But I like container variant:
'container' => [
    'definitions' => ...
    'singletons' => ...
],

But it's almost the same as original. And original variant has it's advantages: no second level.

mwah...it's not logical.

It's exactly the same as components. According to your line of thought it would've been named serviceLocator but it's named according to what's held in there.

Vote
:+1: for
'diDefinitions' => ...,
'diSingletons' => ...,

:heart: for
'dicDefinitions' => ...,
'dicSingletons' => ...,

:smile: for
'container' => [
    'definitions' => ...,
    'singletons' => ...,
],

:-1: for
`containerDefinitions` => ...,
`containerSingletons` => ...

di has nothing to with container. dic is, in my opinion, a very wrong abbreviation since in fact it's just dependency container. I don't like definitions since by looking at it you have no idea that multiple instances will be created.

Ну щас тут и срач будет :)

с dic погорячились. если одну букву добавить, то получится ругательство=)

а вообще логичнее всего вариант с 😄. т.к. у нас уже исходя из того, что он лежит в container'е понятно, что это DI...

@RSalo try using English please.

It's better not to switch to another language, бо якщо я перейду на Українську - буде взагалі круто :)

vote is not fair! i can vote several times!

I'd prefer the Symfony logic with 'container' as key

@samdark definitions is consistent with yii\di\Container naming. Can you propose alternatives?

@dynasource symfony style doesn't work for callback definitions, as I mentioned earlier.

https://github.com/yiisoft/yii2/pull/13029#issuecomment-261245614 is a strong argument.

@hiqsol I have no alternatives which are consistent with Container implementation.

@samdark $_definitions is private in yii\di\Container and can be renamed easily. So good alternative name can be helpful in both places.

The problem with definitions is that the term covers both singletons and prototypes. I'd use something like prototypes to indicate the fact that these are for multiple instances creation.

from the examples above, I would vote for:
'container' => [
    'definitions' => ...,
    'singletons' => ...,
],

however, must say that I think this is overcomplification.
I would rather choose for Yii automatically handling the logic:
'container' => [

	// usecase 1
	\yii\web\Connection::class => app\components\Connection::class,

	// usecase 2
	\yii\web\Connection::class => [
		'class' => app\components\Connection::class,
	],

	// usecase 3
	yii\web\Connection::class => function () {
	},

	// usecase 4
	yii\web\Connection::class => [
		'definition' => '',
		'singleton' => '',
	]
],

Oh god :/

After a vote, here
I've renamed properties from containerDefinitions and containerDefinitions to a multi-dimensional structure:
'container' => [
    'definitions' => ...,
    'singletons' => ...,
]

Now the next thing? Tests...

Pushed tests

There is an unexpected behavior caused by setContainerDefinitions() and setContainerSingletons() methods.
In case when the application config has containerDefinitions or containerSingletons options in its root, they will be handled by the representative methods. Considering it will happen during ::configure(), the order will not be guaranteed.

There are two options how we can solve the problem from my latest comment. The first one - is to leave as is and provide it as additional way to configure DI:
Yii::$app->setContainerSingletons([...])

But I don't think this is a right way, so I will change setContainerDefinitions() and setContainerSingletons() methods visibility from public to protected.

I will change setContainerDefinitions() and setContainerSingletons() methods visibility from public to protected.

You may not realize it - but it does not changes anything.
There is no check for setter/getter visibility level during magic property processing.
Even being protected those methods will produce virtual properties.

As I said you should either introduce single setter for the DI container, which will accept array:
public function setDiContainer(array $config) {...}

Or use single words definitions and singletons for 2 separated setters.

@klimov-paul yes, I was wrong about protected. Thank you for noticing.
We've discussed this problem with @klimov-paul in IM and summarized the following options:

Create public function setDiContainer() (or similar) and drop both setContainerSingletons() and setContainerDefinitions() method, leaving current configuration interface without changes
The same as previous, but setDiContainer() can not be called from the outside (after application initialization). Probably by renaming to protected function _setDiContainer().
Protect setContainerSingletons() and setContainerDefinitions() from calling after the application initialization, leave all other as is.
Just leave all as it is now, do not change anything.

@klimov-paul yes, I was wrong about protected. Thank you for noticing.
We've discussed this problem with @klimov-paul in IM and summarized the following options:

Create public function setDiContainer() (or similar) and drop both setContainerSingletons() and setContainerDefinitions() method, leaving current configuration interface without changes
The same as previous, but setDiContainer() can not be called from the outside (after application initialization). Probably by renaming to protected function _setDiContainer().
Protect setContainerSingletons() and setContainerDefinitions() from calling after the application initialization, leave all other as is.
Remove nested container configuration and switch back to the original implementation
Just leave all as it is now, do not change anything.

What about adding setDefinitions() and setSingletons() to the Container class, consistent with ServiceLocator::setComponents() and make containerDefinitions simply contain a config for the Container class that can be applied via Yii::configure()?

I vote for 1a option: public function setContainer() this is most consistent with other setters.
I don't see a reason in protecting container from changing through application cause it is changeable through it's own interface.
Taking @cebe's comment, the function will look simply:
public function setContainer(array $properties)
{
    Yii::configure(Yii::$container, $properties);
}

@cebe thank you for a great idea!
PR is updated

Indeed I do not like all these Foo, Bar, Baz and their friends. Redone documentation to make it more practical.

It is done to make possible to set ErrorHandler in container['definitions'] and have an effect.
Without this line, ErrorHandler will be registered after ErrorHandler instantiating in registerErrorHandler() method

Thank you very much for your input. Merged

@SilverFire
This is absolutely great work! I am really looking forward to get my hands on this.


[WIP] Support selection and use of multiple images

For #26.
Remaining issues

[ ] Visual display when multiple images are selected

Tweak styling

[x] Nicer shadow on image preview
[x] General spacing

[ ] The width and height is not detected soon enough for UI display, and sometimes not at all
[x] Reconnect image size input boxes

Closing this but keeping the branch until I get time to finish the work being done here.


option to define a3m file name based on protein header?

I am currently using 1.5.2 localcolabfold. After executing colabfold_search, I found that all the a3m file names are numerical.
0.a3m
1.a3m
2.a3m
...
x.a3m
Inside of the individual a3m file, I can see this via "head" in the terminal in order to see the protein name only.
ak $ head *.a3m -n 2
==> 0.a3m <==
#124,403	1,1
>Protein_1	Protein_1

==> 2.a3m <==
#124,729	1,1
>Protein_3	Protein_3

My fasta file used for sequence search looks like this.
>Protein_1 & Protein_2
MTKTKGEKINKSAINEVVTRECTIHLAKRVHNIGFKKRAPRAIKEIRKFAEREMGTTDVRIDTRLNKHIWSKGIRSTPFRIRVRLARRRNDDEDSPNKLYTYVTYVPVSTFKNLQTENVESSDD:MDNSGNNRYELLFM
>Protein_3 & Protein_4
MVKVKCSELRIKDKKELTKQLDELKNELLSLRVAKVTGGAPSKLSKIRVVRKAIARVYIVMHQKQKENLRKVFKNKKYKPLDLRKKKTRAIRKALSPRDANRKTLKEIRKRSVFPQRKFAVKA:MQLAEKHIVALMFPAIDVSTFTFVSGVKFYIMDNSGNNRYELLFMDDDDSSGLAQPQIAAVVAAPKKPEPAKAPKAPKSKSEKENKPVVAARKANAPVAKNASPVKGG

When protein header in the fasta file contains &, it seems only first protein/gene name is used for saving protein header in a3m file.
If I use the fasta file for MSA via colabfold_batch (remote server), I can get file names like the below.
Protein_1___Protein_2.a3m
Protein_3___Protein_4.a3m
Is there a way to have file name like this when colabfold_search is used for MSA?
And is there example fasta file for multimer analysis? If protein headers are unusual, I can refer to the example fasta file.

You can use MineProt toolkit (https://github.com/huiwenke/MineProt/wiki/Toolkit-manual#colabfoldtransformpy) to rename them.


Add "Using styled-components with webpack 2" to the readme.md

Some people had problems with styled-components in connection with webpack 2 and babel.
This adds an instruction how to solve the problem to the readme.md

Amazing! I don't want to put that this prominently into the README, let's add a docs/faq.md and put it there?

Seems reasonable 👍

Absolutely perfect, thank you so much!


Console.ReadKey(true) throws exception in VSCode

Environment data
.NET Command Line Tools (1.0.4)
Product Information:
Version:            1.0.4
Commit SHA-1 hash:  af1e6684fd
Runtime Environment:
OS Name:     Windows
OS Version:  6.1.7601
OS Platform: Windows
RID:         win7-x64
Base Path:   C:\Program Files\dotnet\sdk\1.0.4
VS Code version: 1.15.1
C# Extension version: 1.12.1
Steps to reproduce
Create a new dotnet core console app using dotnet core 1.1
md TestProject
code TestProject
dotnet new console
To Program.cs add the line ConsoleKeyInfo key = Console.ReadKey(true);
Press F5
Expected  behavior
Debug console will pause waiting for input.
Actual behavior
Exception is thrown.
Exception has occurred: CLR/System.InvalidOperationException
An unhandled exception of type 'System.InvalidOperationException' occurred in System.Console.dll: 'Cannot read keys when either application does not have a console or when console input has been redirected. Try Console.Read.'
at System.ConsolePal.ReadKey(Boolean intercept)
at System.Console.ReadKey(Boolean intercept)
at UpdateTestData.Program.Main(String[] args) in c:_Home\Development\Workspaces\Web\Snowdon\UpdateTestData\Program.cs:line 24
Running the program from the terminal using dotnet TestProject console input functions correctly. It is only when debugging in VSCode that the exception is thrown.

@Neutrino-Sunset if you want to use console input you need to enable a console. See https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger-launchjson.md#console-terminal-window


HARP-4888: Fix path text flicker caused by state reset on rejection.

Path text state was being fully resetted after it was rejected, which
meant that the label was rejected for the next frames until the next
label update took place.
Now only the fade state is rejected but the view distance is kept. That
way the label will go through placement on the the next frames.
Signed-off-by: Andres Mandado<EMAIL_ADDRESS>Thank you for contributing to harp.gl!
Before requesting a pull request, please remember to check the following documents:

contribution guidelines
coding style

If you are adding new functionality we would highly appreciate if you can describe what is the capability you are adding and even better if you can add some examples. Please also remember to add tests for it.
CI Check
Our bots will check whether your PR can be directly integrated into the mainline. We have some internal integration tests running on the background, our bots will inform you of the next steps and someone from our team will take a look and help if needed!
And please do not forget to sign-off your commit! You can read more about DCO here. But, in short, you just need to use git commit -s or append --signoff when you are committing to the repo.
Happy contributing!

Codecov Report

Merging #1197 into master will decrease coverage by 0.06%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1197      +/-   ##
==========================================
- Coverage   47.69%   47.62%   -0.07%
==========================================
  Files         258      246      -12
  Lines       22956    22722     -234
  Branches     5101     5060      -41
==========================================
- Hits        10948    10822     -126
+ Misses      12008    11900     -108

Impacted Files
Coverage Δ

...here/harp-mapview/lib/text/TextElementsRenderer.ts
10.22% <0%> (-0.02%)
:arrow_down:

...re/harp-text-canvas/lib/rendering/TextMaterials.ts
50% <0%> (-26.93%)
:arrow_down:

@here/harp-mapview/lib/ThemeHelpers.ts
6.55% <0%> (-0.68%)
:arrow_down:

@here/harp-mapview/lib/MapView.ts
58.93% <0%> (-0.08%)
:arrow_down:

...rp-mapview/lib/text/TextElementsRendererOptions.ts
100% <0%> (ø)
:arrow_up:

@here/harp-datasource-protocol/lib/Outliner.ts
100% <0%> (ø)
:arrow_up:

...ere/harp-materials/lib/LuminosityHighPassShader.ts
100% <0%> (ø)
:arrow_up:

@here/harp-test-utils/lib/WebGLStub.ts
100% <0%> (ø)
:arrow_up:

...p-geoutils/lib/tiling/QuadTreeSubdivisionScheme.ts
100% <0%> (ø)
:arrow_up:

@here/harp-utils/lib/Mixins.ts
70% <0%> (ø)
:arrow_up:

... and 32 more

Continue to review full report at Codecov.

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


Fix missing ID handling related to starting trajectory

Remove unused code in Node::HandleStartTrajectory.
Store correct trajectory ID to ROS service response in Node::HandleStartTrajectory.
Register a automatically launched first trajectory to active trajectory list

Thank you very much!


resample! is slow on the gpu

At the moment, resample! is quite slow on the gpu. Optimizing code therefore requires designing a workflow in which data is resampled before being moved to the gpu so other preprocessing steps can enjoy the gpu speedup. This may not be ideal in all cases. The example below should demonstrate the issue.
# choose original fs and new fs
fs = 100.
resample_fs = 10.

# generate some 'data'
data = rand(500,500)

# resample on cpu
C = CorrData(corr=data,fs=fs)
cpu_time = @elapsed resample!(C,resample_fs)

# resample on gpu
C = CorrData(corr=data,fs=fs) |> gpu
gpu_time = @elapsed resample!(C,resample_fs)

# print results
print("CPU time: ", cpu_time, " s\n")
print("GPU time: ", gpu_time, " s\n")

CPU time: 0.029605658 s
GPU time:<PHONE_NUMBER>81 s

I'm a little surprised resample works on the GPU...I don't believe I wrote a version of resample specifically for the GPU, so I think there's some data transfer to/from CPU/GPU going on here that is gumming things up. The resample code just calls an FIR filter, which I do have for the GPU but is slow because it's using an FFT. All that's to say that, yes definitely can speed things up!

Should we close this since it's related to https://github.com/tclements/SeisNoise.jl/issues/107#issue-1570303031? The performance elements of both issues are with SeisIO, not SeisNoise. One overall fix would be to implement resample for NodalData in SeisNoise, and to eventually add GPU support for resample in general.

Let's keep this open - I think GPU resampling is different enough to keep in mind.


[DISCUSS] [FEATURE] Promote KMeans training and prediction as a top level core plugin

Is your feature request related to a problem?
ML training and prediction (e.g., kmeans) should be a core feature of OpenSearch minimum distribution. The API developed in ml-commons (e.g., MLEngine methods like train, deploy, predict)  would be useful for top of funnel developers to explore other native training models that do not require sucking data out of the OpenSearch Node. This is a form of OpenSearch Node Level compute which is very useful for on-the-fly exploratory data analysis (EDA).
What solution would you like?
Move the MLCommons foundation API to a core :libs:opensearch-ml top level library and the concrete implementations (e.g., KMeans trainers and predictors) to a :plugins:core-ml plugin that is available on the core distribution. Ideally this will relegate the entire ml-commons repository to more advanced compute-heavy trainers that we don't want as part of the core (e.g., non-linear, parametric). Or it would remove the need for this repo altogether which would simplify the testing and reduce core fragmentation across the ecosystem.
What alternatives have you considered?
Leave as is.

/cc @saratvemulapalli

The cons is community will face a higher bar to tune the framework as core repo is much bigger.

This isn't an issue. Plugins, Modules, and Libraries are build as separate artifacts. So it's not making the consumables "bigger" it's colocating the min distribution code into the min distribution core repo.

Our goal is to make ml-commons as a common layer for ML/AI things

That's exactly what you're doing, but you're moving it do a build artifact created in the min distribution repo.

That seems no much fragmentation.

This proposal is the opposite of fragmentation. It's consolidation. The proposal is to provide KMeans clustering as a core feature in the core repo built as a core artifact and packaged as part of the min distribution. The downstream repo is relegated to more specialized fine tuned algorithms that extend this core functionality that might take other third-party (e.g., c++ native, GPU optimized) dependencies.

Thanks @nknize for the elaboration of each concern.

This isn't an issue. Plugins, Modules, and Libraries are built as separate artifacts.

This one seems confusing to me, if the are built as separate artifacts, what's the difference with current separate repo? Any reason we must do this?

That's exactly what this proposal is doing, but moving it to a build artifact created in the min distribution repo.

Same question as last one, confused why this is must do to move around to another place (min distribution repo).

The proposal is to provide KMeans clustering as a core feature in the core repo built as a core artifact and packaged as part of the min distribution.

This will not be just moving KMeans clustering as core feature in core repo. As KMeans depends on the ML framework in ml-commons. So this means we should move the whole ML framework to core repo, then we can move KMeans to core.

The downstream repo is relegated to more specialized fine tuned algorithms that extend this core functionality that might take other third-party (e.g., c++ native, GPU optimized) dependencies.

Based on my last concern, we have to move the whole ML framework to core. This means we have to work on two repos for future: core repo for ML framework and ml-commons for fine tuned algorithms. That will add more work. Is this necessary to split things into two repos ?

what's the difference with current separate repo?

Because core modules and plugins can take dependencies on core libraries but they can't take dependencies on downstream repos.

This will not be just moving KMeans clustering as core feature in core repo. As KMeans depends on the ML framework in ml-commons. So this means we should move the whole ML framework to core repo, then we can move KMeans to core.

I don't think that's a problem. If we end up moving all of ml-commons as a core library and a core module then that just might make the min distribution stronger. Do we not have any other training algorithms besides kmeans?

Based on my last concern, we have to move the whole ML framework to core.

❤  Maybe that's the right thing to do then.

ml-commons doesn't need to take dependencies on core libraries.

For more context, I'm thinking about this same approach for KNN and Vectors.

Thanks for sharing the big picture. To learn more, do you also have plan to make alerting , AD, SQL/PPL as part of core?

ml-commons doesn't need to take dependencies on core libraries.

Wait.. yes I think it does? It's the only way to pull in the Settings, Action, Plugin dependency classes. Or am I missing something here? The plugin build.gradle explicitly takes a dependency on org.opensearch core

To learn more, do you also have plan to make alerting , AD, SQL/PPL as part of core?

For SQL/PPL, yes (/cc @anirudha). Jury is still out for Alerting and AD but I think a foundation implementation for these may also be desirable.

It's the only way to pull in the Settings, Action, Plugin dependency classes

For now , all OpenSearch plugins have access to these dependency classes. No need to move to core repo to use these classes.

My ideal mental model for core is to 1) only include constructs that are immutable or generic, and are unlikely to have multiple implementations, 2) be essential for the most minimal version of the search engine, 3) must ship together, ie. tightly coupled.
An easy to grok example is the interface for a repository: it should be in core because without storage the engine can't work, but the implementations for S3 vs. Azure should live outside of the repo because they are different, and because there might be multiple implementations (e.g. s3 storage looks very different from azure blob storage), and because there's no reason why repository-s3 can't ship separately from discovery-ec2, and less often if there are no changes, or much more often than core if there are.
Following this model, I would put the concept of training into core, but I would not put the actual training into core.

My ideal mental model for core is to 1) only include constructs that are immutable or generic, and are unlikely to have multiple implementations, 2) be essential for the most minimal version of the search engine, 3) must ship together, ie. tightly coupled.

Thanks @dblock. I think that's a good high level guideline for what should go to core, what should not.
Does anyone see/receive any community user asking putting the training into core ? To me seems only part of customer who needs to run ML need this, seems not meet the point 2 and 3 2) be essential for the most minimal version of the search engine and 3) must ship together, ie. tightly coupled.


ran code on the ml-10m dataset

Hello, author. I ran your code on the ml-10m dataset and found that the experiment did not work well, do you know why?

请问一下你数据是咋处理的

Hi, sorry for delayed response.
Would you share your HP settings, including in which ranges did you tune the HPs?
We can check it together on how to optimize KGRec on more datasets! :)

Temporarily closed due to inactiveness.


[Clang importer] Allow us to wire up overrides with generic classes.

A use of "getAs()" when trying to wire up the overrides of
an imported Objective-C initializer meant that imported Objective-C
generic classes didn't get their overrides set properly. This could
lead to redundant initializers; use a proper accessor, since we only
need the ClassDecl anyway.
Fixes SR-8142 / rdar://problem/41591677.

@swift-ci please smoke test and merge


Navigation Questions

Is it possible to do the following?

When I am on the final page in the Carousel if I swipe forwards can it go back to the start and also vice versa?

The built in position indicator is a great addition is it possible to change it so if I click on the indicators it goes to the desired position?

This functionality will be released in the Pro version.

Hi Alex thanks for the quick reply so is the pro version going to be included in the next version of Xamarin Forms?

Ok thanks for letting us know any idea when that might be?  Its a great control by the way works without any problems on iPad, iPhone, Android old and new and Windows

I hope in couple if weeks as i have lot of issues to fix and features to release.


VISUALS: Update Description and Comment Fields w/Markdown

As the platform will be able to support Markdown in the text boxes, we need to properly display that information to the user. A wireframe has been created with the proper wording and placement, but the overall visual queues have not been considered.
According to the Markdown formatting requirements, the following Icon and specifications should be utilized: https://github.com/dcurtis/markdown-mark
The wireframe is located here: https://redhat.invisionapp.com/share/SJAGM39CT

I'll look more into this to see what (if any) Visuals support is needed

@mindreeper2420 - I think what you've provided in the description links is sufficient for development to implement. If there's standards out there for this feature, we should just stick to it.


create a pull request template to .github file

Can you please me the work

go head and create a pull request and add your code

What do you mean by create a pull request template to .github file.? Please explain.

What do you mean by create a pull request template to .github file.? Please explain.

Create  a form which will be displayed while submitting our code which should be asking all the details of projects


gitignore: added profile.cov

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 53.266%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2389

Relevant Lines:
4485

💛  - Coveralls


Added a  MARCXML encoder.

Adds a new feature:

An encoder that encodes a stream into MarcXml.

Hi Jan
can you please move the encoder from the xml subproject into the org.metafacture.bibio.marc21  package in the biblio subproject. As the MarcXmlDecoder is also in this package it makes sense to have the encoder in the same place.
Cheers,
Christoph

Moved the files to the marc21 sub-package. Also added a option to the flux.properties and renamed prettyPrint to formatted to avoid confusion.


Add the ability to solve a specific sudoku without needing to create a file to load

Title ^
Added a command line argument -u or --uid (for unique ID, wasn't sure on the name though!) to load a specific sudoku by String through the command line.
Perhaps would be worth making this more safe with some kind of check that the sudoku is valid etc etc.

Looks great! Thank you for your contribution!


Version v0.2.1 of plugin doen't work with ES 2.2.0

I had transport exceptions.
By recompiling the gem using 2.2.0 of es jar it seems it is working (with es 2.2.0 and 2.1.x)

Although tried both of ES 2.2.0 and 2.1.0, I couldn't reproduce.
Could you show us your stacktrace?

I closed this issue.
Please let us know when you find out procedure for reproducing.
Thanks.


changed flattenProp hoc it does not omit flattened prop

Connects #190
There is one issue that has not resolved. If the flatten prop has a key with the same name of the prop, it will be overwritten. As shown on second test case of flattenProp-test.js.
Any suggestion on solving this?

In most cases I can't imagine this will be a problem. In those cases where it is, we should direct people to use mapProps.

Thanks!


Best Way to schedule multiple workers?

I notice in the documentation you've written:
To use Firmament across multiple machines, you need to run a coordinator instance on each machine.
These coordinators can then be arranged in a tree hierarchy, in which each coordinator can schedule
tasks locally and on its subordinate childrens' resources.

Yet you've also written:
The parent coordinator must already be running. Once both coordinators are up, you will be able to
see the child resources on the parent coordinator's web UI.

It seems to imply here that we can create multiple coordinators (such as 3), yet also seems to imply we can only have two coordinators running.
Starting three schedulers in the pattern:
tcp:<IP_ADDRESS>:8000 (no parent uri)
tcp:<IP_ADDRESS>:8000 (--parent-uri tcp:<IP_ADDRESS>:8000 )
tcp:<IP_ADDRESS>:8000 (--parent-uri tcp:<IP_ADDRESS>:8000 )
Yet the topology map shows only two hosts. I'm assuming this is because this is due to the project being "alpha stage" as mentioned, and totally understandable. Just want to make sure I'm not going crazy.

Hi @SecurityInsanity,
You can definitely have any number of coordinators, not just two (we've routinely run Firmament with up to 40).
The pattern in which you start them up is also correct (modulo the command line parameter being called --parent_uri, IIRC). Do your machines have different hostnames? Some places in the system use hostnames, so if they have the same default hostname, incorrect information might be displayed. We try to use UUID instead of hostnames, but it's possible that something has slipped through the net.
To debug this, it would be helpful to have the log output from each coordinator, and in particular the output from the one on <IP_ADDRESS> (which should see the child coordinators register in its output). You can turn on more verbose output by passing --v=1 to the coordinator binaries.

Hey @ms705 ,
Thanks for following up so quickly!
Hmmm I do believe:
tcp:<IP_ADDRESS>:8000 (--parent-uri tcp:<IP_ADDRESS>:8000 )
tcp:<IP_ADDRESS>:8000 (--parent-uri tcp:<IP_ADDRESS>:8000 )

Both have the exact hostname, I'll go change those up, and if it still seems fishy I'll go ahead, and send you some verbose logging.

Yep, setting unique hostnames seemed to work @ms705 . Thanks much, and great work :wave:


Is it working on Android ?

Hi, first congrats on this amazing project !
Has anyone succeed to make it work on a Android mobile phone camera ?
Or do you have any hints on what to change in the code to make it work on it ?

Hey thanks!
And yes, the current version works in new versions of Chrome on Android. We use some newer web APIs specifically for camera access that requires recent versions of Chrome on Android (it also works on Safari in iOS). Which version of Chrome and Android are you using where it does not work? I'll keep this open until we hear back.
Cheers

Hi, the emoji-scavenger hunt is not working on my android 5.0 galaxy s5 running on the latest chrome update. I asked a friend of mine with an iphoneC also with an updated safari to check it out. it also does not work. Another friend of mine though on an iphone SE got it to work just fine on safari. Would be nice to have a "supported device list" I guess? This is still great work though. More power to the team.

Same here! Android 4.4.4 & Chrome updated doesn't work.

Same, A 8.0, GC 66.0.3359. Game loads for a very long time...

@RadEdje @kfern Thanks for the info! Which version of Chrome are you using specifically? We are using the MediaDevices.getUserMedia() API which is supported from Chrome v53 onwards. Also if possible can you tell me which error message exactly you are seeing when you try to play? That way we can try to debug on our end.
@spanic It loads a long time but it still ends up working? Unfortunately because we are trying to showcase some newer things in tensorflow.js as a technology demo it does mean that we need to load the machine learning model completely in the browser first before you can play. This ensures that no info is sent to any servers. On slow connections it can mean that the game loads a while initially. Follow up attempts to play should be quick though.
Thanks for trying our game!

@kfern looks like the fragment shader won't compile for some reason. There is probably a log of the compile error way below the source in dev tools, could you also paste that here?

@nsthorat: I can´t see more info in Source tab.
In console:
7      bool isNaN(float val) {
8        float v1 = val * val;
9        float v2 = val * val;
10       return v1 == v2 ? false : true;
11     }
12
Fragment shader compilation failed.
13     bool hasNaN(vec4 values) {
14       vec4 v1 = values * values;
15       vec4 v2 = values * values;
16       return any(notEqual(v1, v2));
17     }

Can I do something else?

I manage to bypass the error in Android by not using the async/await methods on the setupCamera function like this :
navigator.mediaDevices.getUserMedia({           'audio': false,           'video': {facingMode: 'environment'}           })           .then(function(mediaStream) {            camera.videoElement.srcObject = mediaStream;            camera.videoElement.onloadedmetadata = function(e) {              camera.setupVideoDimensions(camera.videoElement.videoWidth, camera.videoElement.videoHeight);             };           })           .catch(function(err) { console.log(err.name + ": " + err.message); }) // always check for errors at the end.
Now my main problem is that the loadFrozenModel function takes ages on Android before it gets running :
async load() {     this.model = await loadFrozenModel(       MODEL_FILE_URL,       WEIGHT_MANIFEST_FILE_URL     );   }
This takes about 10s on the Android I am testing on but it's very fast on the Iphones (5/6/7) I am testing on.
My guess is that on some Android configs the async/await syntax is not well implemented yet.
Is it possible to rewrite the loading of the frozenModel without async/await ?

Ok I managed to bypass the loading time.
Now I have an error which looks like the "matmul" bug that was corrected in the fix-reshape branch.
The FrozenModel "execute" function fails when it does a executeOp of "Conv2d" on Android.

Hi,  made a similar app with help from the advice from the amazing devs here and from the main tensorflow python groups (so thanks to you all). It uses the camera to look at X-rays and currently tries to categorize two types of fractures (monteggia vs galeazzi).  It does not use any bundlers, builders, webpack, parcel, yarn or babel. It runs on the latest chrome, opera and firefox for desktop windows. Now I posted on my IT related fb groups and asked if it worked on their phones but forgot to ask here so here it is
https://radhorizon.com/SITES/RadLense/
Now based on my current feedback. It only runs on chrome on android 8.0 and iOS safari 11 (in theory, since only safari 11 supports webRTC). I was hoping to figure out what was keeping it from using lower versions of android even if the chrome version is up to date... chrome 66. Unfortunately chrome on android does not have dev tools unlike in chrome for desktop. Any advice? on how to debug web apps on chrome for android? The app does not fail at loading the AI model since I made it so that the splash screen only fades out to reveal the main UI when the AI model loads using an async await function that loads the model in the background. It's not the camera either since the camera feeds images to a video html tag i have (which is the one feeding the tensor to the AI/ML model at 224 x 224 pixels). Just want to see what's keeping it from predicting. I put a real time info box on the upper left hand corner of the screen which shows real time computations/predictions for monteggia versus galeazzi (seen on chrome, firefox and opera for desktop and android 8 for a samsung galaxy s8) but this info box never shows up on phones with android kitkat or android 5. Any suggestions will be much appreciated... particularly in debugging for chrome on android. Thanks.

You can try remote debugging with your Android phone if you haven't done yet.
https://developers.google.com/web/tools/chrome-devtools/remote-debugging/

okay thanks @tushuhei ! will try that...

Hi this is regarding my AI/ML web cam web app at
https://radhorizon.com/SITES/RadLense/
which runs on android 8 but not on android 5.
so I've narrowed it down by doing the following
I wanted a more generic debugger since my s5 has numerous bugs to connect via remote debugging on chrome.
I maid a window.onerror function to catch all errors and places the errors in a
I used this function:
let errors = document.getElementById("errors");
window.onerror = function (message, url, lineNo){
        console.log('Error: ' + message + '\n' + 'Line Number: ' + lineNo);

        errors.innerHTML += 'Error: ' + message + '\n' + 'Line Number: ' + lineNo;

        return true;

    };

This function returned no errors on mobile s5 android 5.0  and also no errors on desktop so I had to go manual check each function.
I placed error.innerHTML = "some message that it works up to here";  after each function i thought might be the problem. so ES6 classes work, but it finally stopped at
const predictions = model.predict(img);
where "img' is the tensor returned by the function capturing the image from the webcam and converting it into a tensor and the "model" is the loaded tf model for the browser complete with shards.
This works on desktop browser but not in android 5.0.
So I've narrowed it down to the model.predict.
Any suggestions now on what I should do? to make it run on android 5.0?
I don't get any errors at all.
could it be webgl on mobile?
thanks for any reply in advance.

update:
so I tried the suggestion on this link
https://github.com/tensorflow/tfjs/issues/145
seems there's a performance cliff on android 5.
tried using prediction.dispose but this crashes the app on desktop.

Hi, waited a while to see if any new updates to tensorflow.js would fix the issue. Currently running the latest at 0.11.7 and still does not work on android 5.0. Any suggestions? or is there a link on supported android devices and OS number? Thanks. (runs on android 8.0 only).


manifest.json rejected in sideloading.

As Administrator can't sideload the attendance app after setup.

fixed it myself


[url_launcher] In iOS, when in airplane mode (i.e. no network connection), it throws Error while launching ..., while indeed the open webview looks good

Is there an existing issue for this?

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

Steps to reproduce

Use iOS
Open airplane mode, or make sure you disable network connection
run the launchUrl code
observe error

Expected results
no error. The webview is opened correctly (though the content shows that there is no network connection). Or, at least provide a helpful error message, indicating that this is not a real exception that we should be worried.
Actual results
has error
Code sample
Code sample
import 'package:url_launcher/url_launcher.dart';

await launchUrl(Uri.parse('https://whatever-website-you-like'), mode: LaunchMode.inAppWebView);

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Expected results
no error.

I'm not clear on why you are expecting this behavior. Why would failing to load a URL that you asked to load not be an error?

Or, at least provide a helpful error message

This is probably not something we would change without doing breaking changes to align with https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#platform-exception-handling
What would you want the error message to say? The only information we have is that the URL did not successfully load.

indicating that this is not a real exception that we should be worried.

I'm not sure what this means. How is it not a "real" exception? Why would we not expect clients to care that the URL doesn't load?

I'm not clear on why you are expecting this behavior. Why would failing to load a URL that you asked to load not be an error?
I'm not sure what this means. How is it not a "real" exception? Why would we not expect clients to care that the URL doesn't load?

So, here is my use case in real app (I guess many people do this): Click a button in the app, say, "open link https://google.com", and I call url_launcher to launch the webview.
In addition, I have error reporting service (Sentry for me, but others should be similar) setup, as is common the case for apps in production. That service will catch any uncaught exceptions, and report it to the server.
Now the problem is, I want to know if url_launcher really fails. For example, imagine the iPhone has a weird bug that, if I launchUrl, it will not open the webview. In that case, I want Sentry to tell me my app has a bug.
However, now url_launcher throws when there is no network, and I have no way to distinguish it from real url_launcher bugs, and even do not know it is because of no-network. IMHO, if there is really no network, it is normal (and expected) for a user to see WebView launched and displayed text like "no network", and no user will consider it as a real bug (indeed how can we display website without network?). If, on the other hand, the network is good, and the target website it up, but users cannot visit it, then it is a real bug, surely.

This is probably not something we would change without doing breaking changes to align with flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#platform-exception-handling

That looks interesting, looking forward to it!

In addition, I have error reporting service (Sentry for me, but others should be similar) setup, as is common the case for apps in production. That service will catch any uncaught exceptions, and report it to the server.
Now the problem is, I want to know if url_launcher really fails. For example, imagine the iPhone has a weird bug that, if I launchUrl, it will not open the webview. In that case, I want Sentry to tell me my app has a bug.
However, now url_launcher throws when there is no network, and I have no way to distinguish it from real url_launcher bugs

Since fixing that falls under error alignment, I'm going to go ahead and close this as a duplicate of https://github.com/flutter/flutter/issues/127665
I want to reiterate though:

even do not know it is because of no-network

We do not receive that information from SFSafariViewController.

IMHO, if there is really no network, it is normal (and expected) for a user to see WebView launched and displayed text like "no network", and no user will consider it as a real bug (indeed how can we display website without network?). If, on the other hand, the network is good, and the target website it up, but users cannot visit it, then it is a real bug, surely.

Those two cases are indistinguishable in SFSafariViewController.
As I commented in the other issue: we can (and should) be more structured and consistent about how we report errors, but doing that won't create information that we never had in the first place.

I see, thank you! (So the SFSafariViewController is really weird :( )


Opening a notification using publish triggers '$apply already in progress' error

For example, if have a notification like so:
zf-notification-static id="static-alert-message"
That gets opened like so:
FoundationApi.publish('static-alert-message', 'open')
Which triggers an '$apply already in progress' error.
Suggested fix to notification.js, line 255:
if (!scope.$root.$$phase) {
scope.$apply();
}

I'm still getting this error all over the place when using panels as well. If someone finds an answer to this, I'd love to hear about it.

Another fix is to wrap the publish in a $timeout. The time can be set to 0.
$timeout(function(){
FoundationApi.publish('static-alert-message', 'open')
}, 0);

I get the same error with a publish for show/hide/toggle a POPUP.
Also tried to update to version 1.2.0 but it did not solve the issue.
Impossible to programmatically manage a Popup.


Modify the github_deploy_key back to what it was before.

Merging immediately, since this doesn't affect anything else.


Change Dataset.published_at to Dataset.originally_published_at

To match the analogous database column names, which are inherently descriptions, used for Collections:

The column Dataset.published_at should be changed to be Dataset.originally_published_at
A DatasetVersion.published_at column should be added

These are database column changes only, and do not impinge on the API.

@danieljhegeman
@metakuni and I are triaging H2 candidates. Please ensure that there are descriptions in all issues. Why is this breaking change being proposed? I searched slack and found #single-cell-discover-api.
I recommended that this be closed.

Removing from the epic in the meantime since this isn't a high priority item and we likely won't have time in Q3 to do this.

@brianraymor @metakuni I've added a description. These are database column changes, not API changes.

I recommend that we re-add this to the epic it was just removed from


Increasing character limit to 128

Updating character limit from 64 to 128

[x] styleCheck passes
[x] unit tests pass
[x] integration tests pass
[ ] manual tests pass
~- [ ] API docs updated~

Build failed.

This build looks good.


replaced webapi and TestManagementApi with references to vso-node-api

Finishes the work of replacing the old hand-coded rest client apis in src/agent/api with auto-generated APIs
see:
https://github.com/Microsoft/vso-agent/commit/9b950977e5d2da770366a719b04c0985260732cb
for previous work on this

Hi @teddyward, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Corrected some Bugs and not working Race Mode

Corrected Windows Genome once dead and then closed a crash appeared,
Corrected when the race mode game is restarted more than once,
Augmented the number of RayCast to 20 and adapted the genome inputs that leads to faster results.

Thx for your fixes, I'll merge them but change the params for num_initial links and the num of raycasts back because the current num of links is suffecient to find smaller networks that also perform well. It just takes a bit longer to evolve them.


fix(perf): Dedupe component risk upserts

Description
Image and component risk factors are not actually used currently so only insert the values if the score has changed, which will reduce the number of writes on the database
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
Scale test + comparison of QA metrics
Before
cat stackrox_debug_2023_10_24_06_48_08/metrics-1 | grep postgres_op_duration_count | grep Risk
rox_central_postgres_op_duration_count{Operation="Get",Type="Risk"} 21616
rox_central_postgres_op_duration_count{Operation="Remove",Type="Risk"} 3842
rox_central_postgres_op_duration_count{Operation="Upsert",Type="Risk"} 10811
After:
cat stackrox_debug_2023_10_24_17_43_27/metrics-1 | grep postgres_op_duration_count | grep Risk
rox_central_postgres_op_duration_count{Operation="Get",Type="Risk"} 21136
rox_central_postgres_op_duration_count{Operation="Remove",Type="Risk"} 3711
rox_central_postgres_op_duration_count{Operation="Upsert",Type="Risk"} 5615
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/test gke-scale-tests

/retest


Expose mio::Handler.tick

Besides for some debugging purposes, (such as seeing how many ticks occurring before my other code wakes the socket up again), this would also be useful for the Machine to be able to notify a user if they are doing blocking work, or otherwise too much work, in the ready event. One could observe the time difference between ticks, and above a threshold, act accordingly.

Well, I'm okay with exposing tick, but I'm not sure about the API's. There are now global events in rotor yet. But for your use case:

this would also be useful for the Machine to be able to notify a user if they are doing blocking work, or otherwise too much work, in the ready event. One could observe the time difference between ticks, and above a threshold, act accordingly.

The tick event is inherently global (for all state machines). To observe ready events for your state machine you may just write a wrapper type:
struct TickMeasure<M>(M);

impl<M> Machine for TickMeasure<M> {
    fn ready(...) {
        let tm = get_time();
        let res = self.0.ready(...);
        println!("time {:?}", get_time() - tm);
        return res;
    }
}

Then you can wrap any state machine into this type. Surely, you can wrap every state machine with it, just measurements will be individual (comparing to EventLoop::tick())
You can also submit a pull request to rotor-tools if you think such utility is generally useful.


Hidden service for bitcoincore.org

There have been requests for bitcoincore.org on a hidden service. This could be useful for circumventing censorship. There's the torrent/magnet URI for downloading the binaries without access to the site, however this doesn't give access to any of the documentation hosted on the site.
Also, an hidden service would give an alternative (direct) trust path that doesn't rely on certificate authorities or DNS infrastructure.

Strong concept ACK.
My current understanding is that the actual server is in colocated hosting with at least one VPS being used as a sacrificial proxy to defend against DoS attacks.  I think that means the goal can be accomplished three ways:

Setup the Tor HS directly on the server.  I've heard Tor HSes don't need protection against bandwidth-exhaustion DoS attacks because of Tor's low throughput, although maybe someone more knowledgeable should confirm that before a service is set up.

Setup the Tor HS on a proxy server, that way even if something goes wrong, it's still an easy sacrifice.  This does create a trust problem where compromise of the proxy leads to compromise of the HS's key.

Setup the Tor HS on a completely separate server that mirrors the site.  This ought to be easy to do---just copy over the build script for the site and write a short script to copy over the /bin directory from the DNS-based site after verifying the checksums/PGP sigs.  This has the same trust problem as the above option.

Option 3 can be done by any trusted person; option 1 has to be done by someone with current server access; option 2 can be done either way.
If it's true that Tor HSes don't require DoS protection and there are no other concerns I'm unaware of, then I think I prefer option 1 most of all.

I'd rather run it on a separate server, but I'd be generally OK with running it in the same rack/uplink as the physical servers. Not sure if that's legit DoS-wise, but from a security perspective, if you're not getting the SSL it'd suck to use one of the mostly-untrusted VMs or something similar to terminate the Tor session.

6hasakffvppilxgehrswmffqurlcjjjhd76jgvaqmsg6ul25s7t3rzyd.onion

Light testing suggests it works like a charm. Do we want a vanity address?

6hasakffvppilxgehrswmffqurlcjjjhd76jgvaqmsg6ul25s7t3rzyd.onion

Works for me.
We should probably announce it somewhere (maybe on the site itself).

Just learned about this from @laanwj's announcement on the fediverse. Thanks @TheBlueMatt!

Do we want a vanity address?

I personally think no: onion addresses are not meaningful names, and if people get used to that "oh it starts with bitcoincore it must be ok" that kind of defeats the purpose
also there's a risk in exposing the private key to a GPU cluster or cloud or whatever it takes to grind these things

Resolved via #691