Add manifest files for deploying the gitea-operator

Description
Adds support for deploying a Gitea server in the cluster to enable a target for the gitops workflow
Closes #28
How Has This Been Tested?
Deployed and testing the in demo cluster
Merge criteria:
Manifest successfully deploys the gitea-operator which also deploy a gitea-server application

@LaVLaS FYI this is still marked as a draft PR. Are there still changes that you expect to make here?

@LaVLaS Could you please explain why this was added to the ACM part and not to the pipelines? Before this got merged, the only mention of gitea in the ACM part was (acm/registration/near-edge/overlays/bike-rental-app/kustomization.yaml) and (acm/registration/near-edge/overlays/tensorflow-housing-app/kustomization.yaml), and that was updated by the PipelineRuns in the GitOps pipeline.
In https://github.com/opendatahub-io/ai-edge/pull/123 I'm making the GitOps pipeline actually work with GitHub, so the question is -- do we allow the demo to run solely with git repositories on GitHub (making Gitea optional), or is Gitea now mandatory?
(I talk about the PoC in general, not about the instance at *.rhoai-edge-acm-poc.*.)

I also do not understand the new README part

The gitea-operator will be installed on the odh-core cluster as part of the ACM application rollout.

If the GitOps pipeline is to work with Gitea repositories stored in this cluster, surely the Gitea repositories (and therefore the operator) have to exist before the GitOps pipeline runs.

Also -- the
cat <<EOF | oc apply -f -
---
apiVersion: v1
kind: Namespace
metadata:
  name: gitea
---

fails when run as a non-cluster-admin user with
Error from server (Forbidden): error when retrieving current configuration of:
Resource: "/v1, Resource=namespaces", GroupVersionKind: "/v1, Kind=Namespace"
Name: "gitea", Namespace: ""
from server for: "ai-edge.yaml": namespaces "gitea" is forbidden: User "user" cannot get resource "namespaces" in API group "" in the namespace "gitea"
Error from server (Forbidden): error when retrieving current configuration of:
Resource: "pfe.rhpds.com/v1, Resource=gitea", GroupVersionKind: "pfe.rhpds.com/v1, Kind=Gitea"
Name: "gitea-ai-edge", Namespace: "gitea"
from server for: "ai-edge.yaml": gitea.pfe.rhpds.com "gitea-ai-edge" is forbidden: User "user" cannot get resource "gitea" in API group "pfe.rhpds.com" in the namespace "gitea"

Instead, oc new-project gitea should likely be used.

Adding this for historical purposes
[namespaces creation] fails when run as a non-cluster-admin user with...

This is expected if your current user permissions does not grant you edit permissions on namespaces.  While cluster-admin permissions are not required the user deploying these objects will need edit access to namespaces and `gitea.pfe.rhpds.com" objects at a minimum

How does one check if they have these edit accesses? Because with the same user with which applying the full YAML with the kind: Namespace fails, when I do oc new-project gitea beforehand and then just apply the rest of the YAML with the kind: Gitea, things pass.
This is with a plain user created in an htpasswd Identity Provider in OCM.

How does one check if they have these edit accesses?

Try this:
oc auth can-i create namespaces

$ oc auth can-i create namespaces
Warning: resource 'namespaces' is not namespace scoped
no
$ oc new-project so-can-i-create-a-namespace
Now using project "so-can-i-create-a-namespace" on server "https://api.adelt2-aisrhods.udb6.p3.openshiftapps.com:443".
[...]
$ oc get all -n so-can-i-create-a-namespace
No resources found in so-can-i-create-a-namespace namespace.


Request to support TLS 1.3

I have used WinHTTP for multiple projects and found it  to be a reliable TLS client for TLS1.0, 1.1 & 1.2
I am attempting to connect to an OpenSSL server where just TLS 1.3 is enabled, but I am unable to get a response using WinHTTP library.
Request you to support TLS 1.3 now especially that security and speed is becoming a concern, and that Windows Edge has begun support of TLS 1.3, sharing a plan to enable TLS 1.3 in WinHTTP library would help us a lot.
Considering the benifits of TLS 1.3, our customers have asked us to try with OpenSSL, but due to the lack of tutorials on switching from WinHTTP to OpenSSL, we are still estimating the solution effort.
In case there is any beta version available, that too would help.

Is the support  TLS 1.3 available in any .net version like 4.8 or is it planned for future releases such as 5.0?

I enabled Windows Insider Build 20251.1. With this build, TLS 1.3 is working.
I am still looking for a solution to enable TLS 1.3 in regular Windows builds.


[Hacktoberfest]: Add Python for Beginners to the YouTube Section

🆕🐥☝ Beginners Only:
This issue is reserved for people who are new to Open Source. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
To do:

Add Python for Beginners to the YouTube Section
See below for link to step-by-step tutorial
Using the following template, put in the correct info for that resource (Search for it on the internet to find the correct details) underneath the last row in the table, ensuring there is still an empty row before the next table.

| [Python for Beginners](https://www.youtube.com/playlist?list=PLlrxD0HtieHhS8VzuMCfQD4uJ9yne1mE6) | Microsoft Developer | 8 | The foundation for programming in Python, starting from common everyday code and scenarios |

Add your details in the contributors section at the bottom of the page using the same formatting as the last one

📋 Step by Step
Step-By-Step Tutorial
Remember to observe the Code of Conduct - by which we mean, in short - Be Kind, Be Positive, Be Helpful

I'll take this


Issue with adding flutter module to existing native iOS app.

Steps to Reproduce

I created an iOS single view app (hostapp) and converted it to pod workspace. Runs well.
Then created flutter module (clientapp) flutter create  --org com.mycompany -i swift -a kotlin -t module clientapp, provided iOS team. Runs well, independently.
Turned off Bitcode for hostapp and clientapp, as it caused other problems.
Now added clientapp to hostapp via podfile, no internal code changes.
After pod install in hostapp, I tried building it.
Then it throws /Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks/Flutter.framework: Permission denied

Things I have already tried

Both hostapp and clientapp are default iOS and flutter app. No changes whatsoever.
I followed this official guide https://github.com/flutter/flutter/wiki/Add-Flutter-to-existing-apps.
Have tried to search for similar issue but no success there.
Tested on Xcode 10 and 11 (Currently using latest Xcode)
Tried legacy build
Tried Xcode 11, 10, 9.3-compatible, etc
Changed flutter chmod to 777
Changed /Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks/ chmod to 777

Xcode Build Fail logs
PhaseScriptExecution [CP]\ Embed\ Pods\ Frameworks /Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Intermediates.noindex/hostapp.build/Debug-iphonesimulator/hostapp.build/Script-4899A4A5D85C0D737A68EFD6.sh (in target 'hostapp' from project 'hostapp')
    cd /Users/iVoIP/projects/xcode/hostapp
    /bin/sh -c /Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Intermediates.noindex/hostapp.build/Debug-iphonesimulator/hostapp.build/Script-4899A4A5D85C0D737A68EFD6.sh

mkdir -p /Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks
rsync --delete -av --filter P .*.?????? --filter "- CVS/" --filter "- .svn/" --filter "- .git/" --filter "- .hg/" --filter "- Headers" --filter "- PrivateHeaders" --filter "- Modules" "/Users/iVoIP/projects/xcode/hostapp/Pods/../../clientapp/.ios/Flutter/engine/Flutter.framework" "/Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks"
building file list ... done
Flutter.framework/
Flutter.framework/Flutter
Flutter.framework/Info.plist
Flutter.framework/icudtl.dat
Flutter.framework/_CodeSignature/
Flutter.framework/_CodeSignature/CodeResources

sent 94440038 bytes  received 120 bytes  62960105.33 bytes/sec
total size is 94428117  speedup is 1.00
Stripped /Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks/Flutter.framework/Flutter of architectures: armv7 arm64
Code Signing /Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks/Flutter.framework with Identity -
/usr/bin/codesign --force --sign -  --preserve-metadata=identifier,entitlements '/Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks/Flutter.framework'
/Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks/Flutter.framework: replacing existing signature
/Users/iVoIP/Library/Developer/Xcode/DerivedData/hostapp-czxdwjjefjvrnhhdvbmaqeczngbh/Build/Products/Debug-iphonesimulator/hostapp.app/Frameworks/Flutter.framework: Permission denied
Command PhaseScriptExecution failed with a nonzero exit code

Logs
flutter run --verbose (https://pastebin.com/CCyyuuh5)
too long so used pastebin https://pastebin.com/CCyyuuh5

➜  clientapp flutter analyze
Analyzing clientapp...
No issues found! (ran in 2.1s)

➜  clientapp flutter doctor -v
[✓] Flutter (Channel stable, v1.9.1+hotfix.2, on Mac OS X 10.14.6 18G87, locale en-NP)
    • Flutter version 1.9.1+hotfix.2 at /opt/flutter
    • Framework revision 2d2a1ffec9 (3 weeks ago), 2019-09-06 18:39:49 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/iVoIP/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Library/Java/JavaVirtualMachines/jdk1.8.0_171.jdk/Contents/Home/bin/java
    • Java version Java(TM) SE Runtime Environment (build 1.8.0_171-b11)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.8.0

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

[✓] IntelliJ IDEA Ultimate Edition (version 2019.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 35.2.2
    • Dart plugin version 191.6183.88

[✓] Connected device (1 available)
    • iPhone 11 Pro Max • 9243123D-95CD-4A3B-9FD5-E6BAF9A589C3 • ios •
      com.apple.CoreSimulator.SimRuntime.iOS-13-0 (simulator)

! Doctor found issues in 1 category.

@wannamit I wonder if it's related to #40146. Please try if it will work on master channel https://github.com/flutter/flutter/issues/40146#issuecomment-530470137.

@BondarenkoStas Thanks. That worked.


Feat: Screenshots on reports

:scroll: Description

:bulb: Motivation and Context
Closes https://github.com/getsentry/sentry-dart/issues/556
:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Example: https://sentry.io/organizations/sentry-sdks/issues/2613488099/?project=5428562

Codecov Report

Merging #576 (5017027) into main (f92d352) will decrease coverage by 6.49%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #576      +/-   ##
==========================================
- Coverage   91.42%   84.92%   -6.50%
==========================================
  Files          75        8      -67
  Lines        2472      272    -2200
==========================================
- Hits         2260      231    -2029
+ Misses        212       41     -171

Impacted Files
Coverage Δ

flutter/lib/src/screenshot/sentry_screenshot.dart
0.00% <0.00%> (ø)

dart/lib/src/hub_adapter.dart

dart/lib/src/transport/rate_limit_parser.dart

dart/lib/src/protocol/sentry_device.dart

...event_processor/deduplication_event_processor.dart

.../lib/src/enricher/io_enricher_event_processor.dart

dart/lib/src/protocol/debug_image.dart

dart/lib/src/protocol/sentry_level.dart

dart/lib/src/http_client/sentry_http_client.dart

dart/lib/src/integration.dart

... and 60 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 f92d352...5017027. Read the comment docs.

looks like the macOS hang again https://github.com/getsentry/sentry-dart/pull/576/checks?check_run_id=3472320682

looks like the macOS hang again https://github.com/getsentry/sentry-dart/pull/576/checks?check_run_id=3472320682

Ahhh, I really dislike that error :(


Updated WinUI to the latest versions

@neuecc let's start with a minor update first, then

So what about this one? Simple dependencies update

Building WinUI is complex, so unless there are special circumstances, I would prefer not to upgrade it.
Are there any compelling reasons why you absolutely want to upgrade?

Do you have a CI pipeline that builds it?
Your CI system is a bit complicated 🙂

Can you approve the preparation PR?


Changed "experimental" to "Experimental features"

and changed "Switch kernel" to "Switch kernel to 64-bit"

What when you have already 64-bit kernel? ;) Is String should change? Homework for you!


XAML Editor does not respect Tabs settings.

My Options -> Text Editor -> XAML -> Tabs -> Tab Size setting is 4 but the XAML editor always uses a tab size of 2.

We are using Visual Studio XML editor, not XAML editor. You should change your settings there.

+1 ok, I didn't realise that.


misspelled words

Context
https://goreportcard.com/report/github.com/celestiaorg/celestia-app
Problem

Proposal

Fix the misspelled words
Ensure that CI will fail if new misspelled words are added. We already have this enabled so it may require investigation https://github.com/celestiaorg/celestia-app/blob/fc03393dce943532c618da53069f9331fc575b1d/.golangci.yml#L9

Context
https://goreportcard.com/report/github.com/celestiaorg/celestia-app
Problem

## Proposal
1. Fix the misspelled words
2. Ensure that CI will fail if new misspelled words are added. We already have this enabled so it may require investigation https://github.com/celestiaorg/celestia-app/blob/fc03393dce943532c618da53069f9331fc575b1d/.golangci.yml#L9

I would like to do this issue can you assign this issue to me.

Nvm it looks like the previous report was based on a view of the codebase that was a year old. I refreshed the report via Refresh now and now there are no more reported spelling mistakes

Nvm it looks like the previous report was based on a view of the codebase that was a year old. I refreshed the report via Refresh now and now there are no more reported spelling mistakes

okay, thanks for assigning me the request.


EnableLoadExtension() throws EntryPointNotFoundException: sqlite3_enable_load_extension on iOS

Calling db.EnableLoadExtension(1) on iOS yields exception:
System.EntryPointNotFoundException: sqlite3_enable_load_extension
I am using the ios unified dll obtained from the Xamarin Components store.

I have some follow up on this. It turns out the sqlite library that ships with iOS has extension loading disabled. To load an extension, you need to statically link it in with your iOS app (dynamic linking is forbidden by Apple for apps in the app store), and then call some native methods in sqlite to have that extension automatically loaded.

I'm investigating this with #591
Need to check if a newer iOS allows this.


Raise DataFrame and Series to power

Unexpected mypy errrors
mypy(error): Unsupported operand types for ** ("DataFrame" and "int")
mypy(error): Unsupported operand types for ** ("Series" and "int")

from
df ** 2
df["col"] ** 2

Hi @janosh. Thanks for reporting again!
Here's a PR with a fix: https://github.com/VirtusLab/pandas-stubs/pull/136. Hopefully we can release the fix soon,

PR has been merged. Version <IP_ADDRESS> should now be available through pip and conda-forge.

@zkrolikowski-vl Thanks! Fix confirmed.


🍒[5.5][Concurrency] ban inheriting Actor explicitly by class

Explanation: Actors can only be expressed by the actor keyword. We still allowed a class to conform to Actor which is wrong because it cannot declare actor isolated members. This bans classes conforming to the actor protocol.
Reviewer: @DougGregor
Radar/SR Issue: rdar://79749091
Risk: Small.
Testing: PR testing and CI on main.
Original PR: https://github.com/apple/swift/pull/38050

@swift-ci please test

@swift-ci please nominate


The spider is not supposed to do that...

Hi, for quite a long time, g this weird bug, and I thought it was going to be fixed in the new update.
Anyway, here is the video of the bug (sorry for the crappy quality and the watermark)
thanks in advance :
https://youtu.be/4qvMx-aNgQI

happens to me too

Lol what I have never seen that before. Probably something to do with gravity will fix.


merge from template-coveralls

package.json

chore(deps): add ^11.10.13 remove ^11.10.12 (devDependencies.npm-pkgbuild)

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

npm package (@latest dist-tag)
v6.4.5
GitHub release

Your semantic-release bot :package::rocket:


nsc-events-android_5_218_edit-delete-button-accessibility

As a developer, I made the edit and delete button accessibility conditional on the creator/admin status of the user, so that the buttons do not appear if they are logged out or aren't an admin or creator.
Not admin or creator

Is admin or creator

Resolves #218

Looks good. I'm able to see the delete/edit options when signed in as an admin/creator. And when signed in as a user, I don't see either of those options. One thing I noticed is that the delete/edit options are available when the application starts up and I haven't signed in yet.

I noticed this too, but when you wipe data off the emulator, the buttons are no longer visible (I'm assuming it's still "logged in" from a pervious attempt?)


Multi-path (A -> B -> C) Balancer V2 trades are not detected

Example tx: 0x336151dd41fe447f05d6ef0bde9b0fdf6d4455b39e08c93c4a7c94ba89ef4a3c.

Fixed in 0.5.0.


Feature: support async-generators (async function*)

Async-generators are stage3, so fairly likely to land in JS. Seems like fasy should support them eventually, probably sooner than later.
For example:
FP.serial.map(async function *mapper(v){
    if (await lookup(v)) v = yield something(v);
    return v;
});

Implementation should be fairly straightforward, in _runner(..).
Note: I don't think the for-await-of loop will work to run the async-generator, because that construct doesn't let us send a "next value" back in after each iteration. But I think we can do a for-of loop with await inside it, to get almost the same syntax-sugar but have the capability to drive the iterator as we need to. We'll look for Symbol.asyncIterator to know that we need to take this path.
However, before proceeding, we need to decide:

[ ] if the async-generator yields out a promise, do we wait on it and resume with its resolution, like we do in the normal generator-runner pattern?
[ ] if the async-generator has a final return value, is that indeed the overall completion value of that function's promise (in _runner(..))? Usually, returnd values from a generator would be thrown away
[ ] if the async-generator does not have a return value (or if it's return or return undefined), should we assume the last yielded value (or its resolution, if it was a promise) as the overall completion of that function's promise (in _runner(..))?

I like the sound of that, I'd be keen to have a go at implementing something - I've been working on a related thing myself called itrabble and am looking forward to async generators being available for building in that capability.
As for those points to consider, I would (in my very inexperienced experience) expect

[x] yes
[x] I'd suggest yes so that the promise's return value (if there is one) is able to be used subsequently
[ ] I'm not so sure about this last one, but in order to be consistent with the above I'd expect it to return a promise over the value of whatever gets returned (or not) from its final calling. I may well be wrong about how this works, I'm just going from my understanding of the current async/await usage. Does that make sense?

I'm still getting up to speed on JavaScript generators and fasy itself, but here are my thoughts at the moment:

[x] Waiting on yielded promises?
I would assume "Yes", as that would make it consistent with how fasy handles promises normally (at least as far as I understand it at this point).
[ ] Keep returned values?
Seeing as normal generators throw away return values, my gut feeling is "no", otherwise you start to depart from the standard expectation of how the generator would function.  One question I have though is whether one should wait on a returned promise, even if you ultimately throw it away after it resolves.  I'm leaning towards "yes".
[ ] Handling generators with no return?
If the answer to #2 is "no", this would naturally follow suit and be "no" as well.

Upon further reflection, I think a bit differently from how I thought earlier.
I think these answers come down to, would/should someone using an async function * generator with fasy be treating it more like an async function or like a function*?

in async generators you're supposed to await promises, not yield them. If an async generator is used, the ability to yield from them seems pointless (with fasy, specifically) and thus shouldn't be covered up. Therefore, we shouldn't do the magical wait-on-promise there.
We have two other options then:

any value that is yielded, even a promise, is just sent back in to the generator immediately.
any value that is yielded is collected into a list of values, which is eventually the final result of that function. IOW, if yield 1, yield 2, and yield 3 all run, then the final result of that function is a list of [1,2,3].

To the question above, when using fasy, I think async function* is more async function than function *. As such, a return value is the generally expected way to give a result value from an async function. So, any returned value, even the default undefined, is the result.
But, how then does this rule mix with the idea of multiple yield results being collected into a list? My inclination is, if we did this rule, it would be incompatible with that idea, and therefore that one would be eliminated.

This one would suggest a way to allow a compromise for that previous question, which is that any list of yielded values, if any, is overridden, but only by an explicit non-undefined value being otherwise returned. That could work, but I no longer think this sort of magic is a good idea. I think this answer should be no.

My two cents on points one: if an async generator yields a promise, it is automagically awaited:
async function * PromiseYielder() {
    var i = 0;
	while(true) {
		yield new Promise(ok => ok(i++));
    }
};

;(async () => {
    const ait = PromiseYielder();

    (await ait.next()).value; // 0 not Promise<0>
    (await ait.next()).value; // 1 not Promise<1>
})();

The more I've thought about this, the more I think we have to treat async generators as async functions that happen to also have a yield, which if used behaves the same as await.
The earlier thinking I had was that it being a generator offered an interesting opportunity to support lazy iteration protocols, but fundamentally fasy is about eager iterations. A different set of functions should be provided for lazy iteration.

@getify of course generators are the fulcrum of the lazy iteration in JavaScript (although they are often not used in this way), and that is the main reason I cannot make equivalent yield with await.
We could argue about the fact that both pause the execution of a function and both let multiple values be inserted into it and extracted from it.
But is thanks to yield that the control of the execution could be "gifted" to other entities in the code, enabling the above-mentioned lazy iteration. This is not what await is for: considering the async/await pattern as the union of promises, generators and functions like your _runner, is easy to see that the control is taken by runner functions that restart the paused function as soon as possible. This is eager iteration.
Moreover, even if it is not so important, the yield keyword has less precedence than the await operator:
// not valid js
function * gen() {
    yield 1 + yield 2;
}

// valid js
async function af() {
    await 1 + await 2;
}

On the other hand, coming back to async function *s, we could not ignore the fact that a yielded promise is awaited as the await keyword was used. This make sense because if it were not so, we would have ended up too close to nested promises (or promise of a promise if you prefer): the promise always returned by the async iteration and the promise inside the value field.
So, what about your question?
I've not spent much time with the internals of fasy, but of course is evident that is based on eager iteration. And change its nature to support lazy iteration for async generators feels wrong. Even because fasy already ignore the fact that also sync generators could be lazy iterated, you simply handle them with _runner.
Considering the eager iteration fact, I think that fasy should handle a case like the following:
var users = [ "bzmau", "getify", "frankz" ];

FA.serial.map(
    async function *getOrders(username){
        var user = await lookupUser( username );
        return lookupOrders( user.id );
    },
    users
)
.then( userOrders => console.log( userOrders ) );

or:
var users = [ "bzmau", "getify", "frankz" ];

FA.serial.map(
    async function *getOrders(username){
        var user = yield lookupUser( username );
        return lookupOrders( user.id );
    },
    users
)
.then( userOrders => console.log( userOrders ) );

as:
var users = [ "bzmau", "getify", "frankz" ];

FA.serial.map(
    function *getOrders(username){
        var user = yield lookupUser( username );
        return lookupOrders( user.id );
    },
    users
)
.then( userOrders => console.log( userOrders ) );

In other words: an async generator is a generator that also happen to also have an await keyword.
IMH you should "run" an async generator like a generator, feeding it with any value that is async yielded (here it is the eager iteration) and taking the returned value as the result.


Unable to read variable-length string attributes

Is there a plan to add support for variable-length string attributes?  It seems at the moment only fixed-length strings (as attributes) are supported.
Here is a link to a small example file which demonstrates the problem:
http://wikisend.com/download/692318/simple-nba.h5

Yes it needs added.  Thanks for the example.
Which nodejs version are you running with?

Thanks for looking into this :-)  I'm running with node version 4.2.2

Ok, I'll put it with the master branch; test and then republish

btw, the use of nodejs Buffer is now the same as using Uint8Array which has a major size limitation to 2 Gb;  am looking into streaming to overcome this change of the V8 that caused the node team to make Buffer derive completely from Uint8Array

Hi, I have  some code working for my own attribute example;  haven't been able to read http://wikisend.com/download/692318/simple-nba.h5 with HDFView or by api.
Are the attributes you are using on groups and/or datasets?

They are on a group.  I am able to read them with h5dump on linux, and I have a patch to attributes.hpp that reads them, if you are interested.  Or let me know if there is some other way I can help.  Thanks for your great work!

I meant to say couldn't open http://wikisend.com/download/692318/simple-nba.h5 with any tool; not just see variable length strings

Hmmm, I just downloaded it from there, and ran h5dump on it, and this is what I see:
HDF5 "/.../Downloads/simple-nba.h5" {
GROUP "/" {
   GROUP "info" {
      ATTRIBUTE "packageHash" {
         DATATYPE  H5T_STRING {
            STRSIZE H5T_VARIABLE;
            STRPAD H5T_STR_NULLTERM;
            CSET H5T_CSET_ASCII;
            CTYPE H5T_C_S1;
         }
         DATASPACE  SIMPLE { ( 1 ) / ( 1 ) }
         DATA {
         (0): "457ba473cc529e0e6c2dac3c64f1865c"
         }
      }
      ATTRIBUTE "packageID" {
         DATATYPE  H5T_STRING {
            STRSIZE H5T_VARIABLE;
            STRPAD H5T_STR_NULLTERM;
            CSET H5T_CSET_ASCII;
            CTYPE H5T_C_S1;
         }
         DATASPACE  SIMPLE { ( 1 ) / ( 1 ) }
         DATA {
         (0): "DEADBEEF87726342"
         }
      }
      ATTRIBUTE "packageName" {
         DATATYPE  H5T_STRING {
            STRSIZE H5T_VARIABLE;
            STRPAD H5T_STR_NULLTERM;
            CSET H5T_CSET_ASCII;
            CTYPE H5T_C_S1;
         }
         DATASPACE  SIMPLE { ( 1 ) / ( 1 ) }
         DATA {
         (0): "NBA Player DAta (2014/2015)"
         }
      }
      ATTRIBUTE "parentPackage" {
         DATATYPE  H5T_STRING {
            STRSIZE H5T_VARIABLE;
            STRPAD H5T_STR_NULLTERM;
            CSET H5T_CSET_ASCII;
            CTYPE H5T_C_S1;
         }
         DATASPACE  SIMPLE { ( 1 ) / ( 1 ) }
         DATA {
         (0): "NA"
         }
      }
      ATTRIBUTE "rootPackage" {
         DATATYPE  H5T_STRING {
            STRSIZE H5T_VARIABLE;
            STRPAD H5T_STR_NULLTERM;
            CSET H5T_CSET_ASCII;
            CTYPE H5T_C_S1;
         }
         DATASPACE  SIMPLE { ( 1 ) / ( 1 ) }
         DATA {
         (0): "NA"
         }
      }
   }
}
}

If I left click t takes me to a page that downloads a movie of a gymnatics practice; called MAGpr1.MOV
If I right click and download and check it with h5debug I get
HDF5-DIAG: Error detected in HDF5 (1.8.16) thread 0:
  #000: /home/roger/Software/hdf5-1.8.16/src/H5F.c line 604 in H5Fopen(): unable to open file
    major: File accessibilty
    minor: Unable to open file
  #001: /home/roger/Software/hdf5-1.8.16/src/H5Fint.c line 1087 in H5F_open(): unable to read superblock
    major: File accessibilty
    minor: Read failed
  #002: /home/roger/Software/hdf5-1.8.16/src/H5Fsuper.c line 277 in H5F_super_read(): file signature not found
    major: File accessibilty
    minor: Not an HDF5 file
cannot open file

yea right-click is the html page
but left click goes to mov cip

Ok, now that's what I'm seeing too, though an hour or two ago, the data at that link was still my example, or perhaps my browser had cached the page.  I think that means I should no longer use wikisend.  Do you have any other suggestion for how I should share the file?

Seems a useful way; my mistake was right-clicking to download last week and not testing right away.  Could try it again.
npm owner ls

in a working folder of hdf5.node will give you my email too

Ok, I uploaded again, it says you should have 7 days to access the file:
http://wikisend.com/download/654088/simple-nba.h5

Thank you, can read and test with this

Test is working.  Been puzzling over when writing string type attributes when to make their size variable [with H5T_VARIABLE] or known size. Or how to distinguish these on the javascript side.

Have both fixed and variable attributes


Fix: Dispatcher abort controller

Also

UT failing are mostly due to timeout errors cause tests are miss-conceptualised


Media path missing a "/" between "rootpath" and the movie title

Seems the new version of pycliarr does not parse the "save/download" path correctly.
Said the "rootpath" in Radarr is "/movies" and the movie title is focus (2015).
The path is build like:  /moviefocus(2015) instead of /movie/focus(2015).  The "/" is missing in the path.
Seems like "build_item_path" method is not concatenating the strings correctly.
   return str(root_path["path"]) + self.to_path(title) ==>  return str(root_path["path"]) + "/" + self.to_path(title)

(alternative ==> return f"{root_path['path']}/{self.to_path(title)}" )
def build_item_path(self, title: str, root_folder_id: int = 0) -> str:
"""Build an item folder path using the root folder specified.
Args:
title (str): Title to add to root path. All invalid characters are removed
root_folder_id (int): Id of the root folder (can be retrieved with get_root_folder())
If the id is not found or not specified, the first root folder in the list is used.
Returns: Full path of the serie in the format /
"""
root_paths = self.get_root_folder()
root_path = root_paths[0]
for path in root_paths:
if path["id"] == root_folder_id:
root_path = path
return str(root_path["path"]) + self.to_path(title)

self.get_root_folder()
Reruns paths without a ending "/"
I fixed it in my own code...
But I still think you should in       build_item_path

The api v3 doesn't include the trailng / anymore. Fixed in the latest release.
(pyvenv) viv@lapdwarf ~/d/pycliarr (master  ⚡ 2)# API=v3/rootfolder curl -X GET "http://<IP_ADDRESS>:8989/api/$API?apikey=xxxxx" -H "accept: application/json"
[
  {
    "path": "/tv",
...
]
(pyvenv) viv@lapdwarf ~/d/pycliarr (master  ⚡ 2)# API=rootfolder curl -X GET "http://<IP_ADDRESS>:8989/api/$API?apikey=xxxx" -H "accept: application/json"
[
  {
    "path": "/tv/",
...
]

Fixed this to handle both cases in

I tested it and it works!
Merci again!


Run Notebook 2.3 with new code to improve display

A print function was removed when the user would press option 4) Search MolID to add chain ID conversion of the PDBClean MolID Conversion Build Menu.

Update the 2.3 notebook so that when the program is being run, it looks less cluttered now that the print function is gone.

Take


Can reset datagrid itemssource by setting binding object to null?

dataGrid.SetBinding(Maui.DataGrid.DataGrid.ItemsSourceProperty, "Teams");
or
dataGrid.ItemsSource = Teams;

viewModel.Teams = null
or
dataGrid.ItemsSource = null
then grid content should be emptied.

Inorder to manipulate the datasource on your VM, you should use ObservableCollection<Team> as DataSource

I can't use ObservableCollection because I am dynamically creating multiple DataGrid elements.
I set the result to Itemssource directly, like this: dataGrid.ItemsSource = myresults. (a list from my object.)
If there is no result (null), the previous data remains in the datagrid.
If the result is null I will try by creating an empty list.
dataGrid.ItemsSource = myresults ?? new List();
Thank you.


feat: add json value converters

We want to receive intermediate data for verifying the process of generating a proof as a JSON file.
Therefore, in this PR, RapidJsonValueConverters for the following four classes was implemented.

PrimeFieldBase
Extension Fields
Elliptic Curve Points (only short weierstrass)
Geometric Points
UnivariatePolynomial (sparse and dense)
UnivariateEvaluations

You need to move all the Copyable implementations of points in d5e33cfc7557117cc23a0b16eda1fa8c653321a9

1a02e54ee0c78e9c77e8ea7ea84825dfae229200 only works when it's a DenseUnivariatePolynomial. Please write codes as we do with Copyable.

Please add unittests in 1a02e54ee0c78e9c77e8ea7ea84825dfae229200 .

Some of changes in c8605d1485847ad9a3b4aa02f3936255da799de9 should be squashed.

Please add unittests in 1a02e54 .

The unittest is implemented at univariate_dense_polynomial_unittest.cc.

Please could you add unittests for Fp3 and Fp4 for c2c0d5778363bd1c32b57c213ee0a79b446b7012 ?

Please could you add unittests for Fp3 and Fp4 for c2c0d57 ?

Currently, There is no implementations of Fq3 and Fq4.

@dongchangYoo With f070c8b, I have 2 comments

Is is a feat commit? I think it's a fix commit
a -> an

@dongchangYoo With f070c8b, I have 2 comments

Is is a feat commit? I think it's a fix commit
a -> an

The implementation has been modified to copy the input string by adding an allocator as an input. This change in handling strings, I believe, qualifies as a 'feat'.


Using V3 ethereum wallet

see
https://github.com/ethereum/wiki/wiki/Web3-Secret-Storage-Definition
https://github.com/ethereumjs/ethereumjs-wallet/blob/master/index.js
https://github.com/sc0Vu/yoethwallet

How is the V4 progress?

@JellyBool I will implement v4 after v3 is done :)


CM-359: Update submodules and image references

PR contain below changes

Updated image references to latest version.
Updated cert-manager-operator sub module to latest.
Added Makefile.

Generally, LGTM.
/lgtm


"JSON APIs and Ajax: Get JSON with the JavaScript XMLHttpRequest Method" tests don't allow line break before document method calls

Describe your problem and how to reproduce it:
The tests for the "JSON APIs and Ajax: Get JSON with the JavaScript XMLHttpRequest Method" challenge don't allow line break before the document method calls.
When my code looks like this, the tests pass:
const apiRequest = new XMLHttpRequest();
  apiRequest.open( 'GET', '/json/cats.json', true );
  apiRequest.send();
  apiRequest.onload = function() {
    const json = JSON.parse(apiRequest.responseText);
    document.getElementsByClassName( 'message' )[0].innerHTML = JSON.stringify( json );
  };

But this causes the tests to fail:
const apiRequest = new XMLHttpRequest();
  apiRequest.open( 'GET', '/json/cats.json', true );
  apiRequest.send();
  apiRequest.onload = function() {
    const json = JSON.parse(apiRequest.responseText);
    document
      .getElementsByClassName( 'message' )[0]
      .innerHTML = JSON.stringify( json );
  };

Add a Link to the page with the problem:
https://learn.freecodecamp.org/data-visualization/json-apis-and-ajax/get-json-with-the-javascript-xmlhttprequest-method
Tell us about your browser and operating system:

Browser Name: Firefox Developer Edition
Browser Version: 66.0b5
Operating System: Antergos Linux

Related issues
Same kind of bug

#35090
#35091

This challenge

#35095
#35096

Closing in favor of #35090 being the single ticket for issues like this.


linux: fincm3: revert 129d946 to work around open drain issue

This fixes https://github.com/balena-os/balena-raspberrypi/issues/640
Change-type: patch
Signed-off-by: Joseph Kogut<EMAIL_ADDRESS>

@resin-jenkins retest this please

@balena-ci rebase


deprecate => syntax for -->

This gets around precedence issues of => due to https://github.com/JuliaLang/julia/pull/12285
closes #51

We might want to consider going back on this, depending what happens with => and/or --> precedence on 0.5. Maybe the improvements to Base.Test will make FactCheck less necessary there?


Organize by feature - View Injection

In your blog, you mentioned that organizing the code via feature is better and I totally agree. In my case, I've split my app into different modules and each module is actually a feature. I assume that this is how you guys also work at SoundCloud so as to allow individual teams working on a feature to work independently.
On splitting the code by module, I'm not able to use Butterknife's @Bind annotations since Android doesn't mark R constants in module as static final. Hence I get a compile time for the following
@Bind (R.id.auth_et_password) EditText passwordTxt;
Error received : "Error:(44, 15) error: element value must be a constant expression"
So how do you guys inject views? Do you have a custom framework or use AndroidAnnotations framework or something else or you don't split by modules?
DISCLAIMER: I'm a Android noob.

I won't go as far as creating separate modules per feature, as it comes with a list of problems and overhead in setting up a complex project correctly. Much code is also shared beween different features, so you're also stuck with setting up that correctly.
I'd say you shouldn't think of features and feature-teams so rigidly. If a team is solely working on the "user" feature, this doesn't necessarily mean they should be contained codebase-wise into 1 module.
Usually you'd have one codebase on which different feature teams working on it in different Git feature branches.
A "feature" is just explaining the "why" and "how" a bunch of classes are grouped together under different name spaces. So i'd keep your project setup much simpler, but keep name spaces clear.
Here's a quick example. Notice that "caching" is also a "feature" of the app. Features aren't always what the app does from a user's perspective. They are everything that an app does.
app-module
└───cache-feature
    │   DiskCache
    │   SqlCache
    │   ...
    │
└───user-feature
    │   UserRepository (uses SqlCache to store users)
    │   UserDataRepository
    │   GetUserDetails
    │   GetUserList
    │   UserListActivity
    │   UserListPresenter
    │   UserListAdapter
    │   ...
    │
└───login-feature
    │   LoginService (uses DiskCache to store login credentials for faster login next time)
    │   LogInUserByDetails
    │   LoginActivity
    │   LoginPresenter
    │   ...
    │

You can also go the 3-tier module route that this project uses as an example.
Here is a good read on the topic.

When would you generally decide to split into a module? Is having a large application inside a single module fairly common and generally accepted as a good practice?
You're right, I was having a trouble in trying to setup the project correctly since I kept running into circular dependencies, but was willing to pay the price of a one time setup v/s having to refactor all over again.

I think three modules as example is good enough. Sometime we need more than 3 but it shouldn't be general.
Actually you should separate your project into libraries rather than modules. When you have a really big project you can split into group of features module should not each module for each feature.

It al depends on you/your teams needs. Large applications can exist is one module. And even then, things can change, there is nothing wrong with refactoring project structures of the scope of the project increases.
The hard part about "one feature per module" is that usually features share a lot of different code, so you'll end up with a lot of modules and code all over the place, and dependency hell.
The 3-tier module setup in this example is a very handy one because of several reasons, but this also makes a project packaged by layer, instead of feature. There's an ongoing discussion here about project structures. I posted some more ways to structure there.

This here is an interesting conversation. I never before saw things the way @Trikke puts them.
"I'd say you shouldn't think of features and feature-teams so rigidly." - superb
Glad to borrow a new lens for coding sight.

Having a module per feature is a lot of overhead and unnecessary complexity. Why you wanna do that? We do not work that way at SoundCloud, not even close. We actually have a similar arquitectural approach to this repo but with only one module. This solution came out of my personal experience and it has nothing to do with my employer.

If you have your activities isolated on modules, How are you starting those activities from different modules?

Certain features are pretty good candidates for modules, e.g. the authentication module or a networking module and even in some cases, certain design elements that are common amongst company apps.
In my case, we're using a google maps and google location services that intends to collect a lot of data points which are going to be fed into our servers to help optimize routing tech, and do data collection etc. This module is going to be installed on 2 different apps targeted for 2 different set of users.
Putting such a component along with its resources into a separate module makes sense.
@kuassivi : The main app has a dependency on the other modules and the main app controls all navigation, so moving between activities is via the main app.

Anyway.. I have received my answer.
Thanks for helping out.

@RowlandOti There's an ongoing discussion here about project structures. I posted some more ways to structure there.

@Trikke , I did try to implement organisation by layer and then by feature. It's incredibly difficult without breaking boundaries. Nevertheless, here is a sample project.


fixed broken link free-programming-books-en.md

What does this PR do?
Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists  in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Followup

Check the output of Travis-CI for linter errors!

good catch.


Why detected as a virus ?

I can't run the .exe because Windows Defender delete the .exe et mark the file as virus ?!

Add it as an exception, or use a smarter antivirus engine than Windows Defender.
This is a duplicate of #616, #517, #508, #503, #420, #384, #335, #333, #315, #284, #270, #249, #217, #209, #137 and #13. Please search the existing issues (even closed ones) before submitting an issue yourself.


Adding or Attempting to Access an OPDS feed while offline results in endless spinning circle

Related to https://github.com/readium/readium-desktop/issues/418
If user is offline attempting to access an OPDS feed results in the endless spinning circle (see screen shot, attached.)
Similarly, you can add a URL, name the URL, create a blue box "catalog" before getting any feedback while offline. When the user clicks the blue box catalog, the endless spinning circle appears.
I recommend that an error message be added when the user clicks on "Catalogs" if the device is offline. For example:
The Catalogs feature requires an Internet connection. Please try again when your device is online. All books in My Books can be read offline. [Return to My Books] button

see also #418


caffe version of the TFeat Liberty

Weights are manually moved and normalization is hardcoded in .prototxt. It expects as input [0 255]  non-normalized grayscale 32x32 patches

awesome!

could you provide any python or cpp script  for running it?

@edgarriba done

@ducha-aiki great!

@ducha-aiki I'd really really love to see how did you convert from Pytorch/Tensorflow? to Caffe

@edgarriba very simple - manually.
From lua (lutorpy) :
i = 0
for k in net.net.modules:
    print str(net.net.modules[k])
    if 'cudnn.SpatialConvolution' in str(net.net.modules[k]):
        print str(net.net.modules[k])
        if i == 1:
            w1 = net.net.modules[k].weight.asNumpyArray()
            print w1.shape
            caffenet.params['conv2'][0].data[:] = w1;
            caffenet.params['conv2'][1].data[:] =  net.net.modules[k].bias.asNumpyArray()
            i+=1
        if i == 0:
            w1 = net.net.modules[k].weight.asNumpyArray()
            print w1.shape
            caffenet.params['conv1'][0].data[:] = w1;
            caffenet.params['conv1'][1].data[:] = net.net.modules[k].bias.asNumpyArray()
            i+=1
    if 'Linear' in str(net.net.modules[k]):
        print str(net.net.modules[k])
        w1 = net.net.modules[k].weight.asNumpyArray()
        caffenet.params['ip1'][0].data[:] = w1;
        caffenet.params['ip1'][1].data[:] = net.net.modules[k].bias.asNumpyArray()

For pytorch (and other arch)
gg = {}
gg['counter'] = 1
def copy_weights(m,):
    if isinstance(m, nn.Conv2d):
        counter = gg['counter']
        l_name = 'conv' + str(counter)
        print l_name,  m.weight.data.cpu().numpy().shape;
        net.params[l_name][0].data[:] = np.rollaxis( m.weight.data.cpu().numpy(),3,2);
        net.params[l_name][1].data[:] = m.bias.data.cpu().numpy();
    if isinstance(m, nn.BatchNorm2d):
        counter = gg['counter']
        l_name = 'conv' + str(counter) + '_BN'
        print l_name
        net.params[l_name][0].data[:] = m.running_mean.cpu().numpy();
        net.params[l_name][1].data[:] = m.running_var.cpu().numpy();
        net.params[l_name][2].data[:] = 1.
        gg['counter'] += 1
def copy_last(m):
    if isinstance(m, nn.Conv2d):
        l_name = 'conv7'
        net.params[l_name][0].data[:] =np.rollaxis( m.weight.data.cpu().numpy(),3,2);
        net.params[l_name][1].data[:] = m.bias.data.cpu().numpy();
        print l_name
    if isinstance(m, nn.BatchNorm2d):
        counter = gg['counter']
        l_name = 'conv7_BN'
        print l_name
        net.params[l_name][0].data[:] = m.running_mean.cpu().numpy();
        net.params[l_name][1].data[:] = m.running_var.cpu().numpy();
        net.params[l_name][2].data[:] = 1.
        gg['counter'] += 1

model.features.apply(copy_weights)
model.classifier.apply(copy_last)

@edgarriba  could you please help me with https://github.com/pytorch/pytorch/issues/1661 ?

I think you need register_forward_hook
http://pytorch.org/docs/nn.html#torch.nn.Module.register_forward_hook

BTW, thanks for the script. I'm not very fluent with low level caffe features

@edgarriba  thanks, I will try it.

@edgarriba here is full version of script
https://github.com/DagnyT/hardnet/blob/master/examples/caffe/convert_weights_to_caffe.py
It is different architecture, though

@ducha-aiki thanks dude! very interesting paper and results.


Upgrade complexity-report.

Our dependencies are fairly out of date. We need to move to Booth's newer version of complexity-report.

Updated to escomplex.


check_PP3()

Why only have MetaSVM_score, dbscSNV_RF_SCORE,ERP++_RS?
such mcap ? sift? revel? polyphen? why not add to prediction?

MetaSVM is the best score for rare and common  variants, and mcap and revel is only for  rare varaints. as the introduction mentioned, all these  kind of scores are with similar theoretical basis.


Cannot install swr

Bug report
Description / Observed Behavior
What kind of issues did you encounter with SWR?
Expected Behavior
How did you expect SWR to behave here?
Repro Steps / Code Example

Or share your code snippet or a CodeSandbox link is also appreciated!
Additional Context
SWR version.
Add any other context about the problem here.

This seems to be a problem of swr.


Better control of input output seperator

Each template now defines the input output seperator.
This allows some templates to avoid adding newline between the input and outputs.
Also removed target fix which unneeded, and
by default added a space.

Closing this one.  We will implement a different behavior.

Closing as we plan a better solution.


Feature/transcripts

The most recent 2 updates create plain-text and SRT transcript examples, and attribution for the source material.

Closed. More updates needed.


Add unicode modifier to regex in splitString

Hi
When using a query string starting with unicode character, for example:
Task::search('"żopa żopa"', $columns)->get();

the preg_match_all in Sofa\Eloquence\Searchable\Parser::splitString() doesn't match any words.
Proposed fix is to add an unicode modifier to the preg_match_all regex.

Coverage increased (+0.1%) to 93.574% when pulling f73ee1bdf1eabaccbf26f4ed2bfd99139d66a01b on ksdev-pl:5.2 into 4e6d62a2e00ec47263e8a13452b6537cb79b7f4d on jarektkaczyk:5.2.


Two Windows test

A pull request for #6.

facepalms
Just checked, two windows in PyGame like this is not possible.


Check halted status for each chain

added warning message for halted chains to the AppView
added disabling manage and swap links for halted pools at the PoolsOverview and PoolDetails views
Swap/Deposit/Withdraw: added disabling form for halted chains
midgard/pools:validatePool$: added check if pool is not halted

closes #1417

[nitpick] Pls check / fix hover state of "Manage"  button


Fixed an issue where the tinymce content css was not being loaded

Webpacker outputs the _tinymce_content.css file to the public directory with an asset fingerprint.
In order to get the correct asset path, I had to hook up the integration with .js.erb and webpacker.
This was done by running bundle exec rails webpacker:install:erb
After renaming the tinymce.js file to a .js.erb I also needed to give the full extension in all files which import that JS file.
Finally, in order  to access helpers such as asset_path you need to call them on ActionController::Base.helpers as they are not part of the default scope

think you may need to add the erb loader to the spec/karma/unit.js file.

https://jaketrent.com/post/import-erb-into-karma/
Followed this guide for the above commit
(something's  messed up with running headless chrome witthin my docker-env, so I'm having to use github-actions as my test env)

@xsrust maybe try giving the new Capybara config in the ror-fundref branch a try. I had to swap in capybara-webmock gem and remove the chrome headless browser. (not sure if it will work without Webmock itself enabled but worth a try). It requires that you add gem "capybara-webmock" to the Gemfile
https://github.com/DMPRoadmap/roadmap/blob/ror-fundref/spec/support/capybara.rb

In production mode rails still generates /assets/blocks/_tinymce_content.css as its path,
and therefore cannot find it. If you open a plan's edit form, you'll see a bunch of requests from
the iframe's of tinymce, and all of them fail.
Maybe use the same approach as for the skins, where you've them into public/tinymce?

I've discovered this as well in my instance of the repo. I have capistrano copy the css file over but it would be good to just add it to the public dir here

just submitted PR #2767 to add the tinymce.css to the public/tinymce dir


versioned online docs

It would be nice to be able to access the documentation of past releases. If a new release is out, the documentation may become incorrect for the version that you are using, leading to confusion.
Not every tag needs its own docs, but vYYYY.0M is enough, leaving out the patch version.

Closing in favor of #666.


Fix "dec2oct" method

There is currently a method called dec2oct that lives in /lib/dec2oct.js.
It is incomplete and needs to be fixed!
Acceptance Criteria:

Running yarn test dec2oct results in tests passing.
You have written a skeleton method for someone else to work on.
You have written tests surrounding your skeleton method.
Running yarn lint does not print any errors to the console!

Please include the skeleton of a new method + an accompanying test for someone else to work on at the time of creating a pull request! A pull request will most likely be denied if it does not contain a skeleton method for someone else to work on! For more information, please read the Contributing Guide.
Thank you so much for your contribution!

I can do this

Awesome @v-chmykov !! I have applied the 'in progress' label. This buys you 24 hours. Feel free to reach out if you need more time! THanks for contributing!


default tag is assumed to exist

Bug Description
https://github.com/istio/istio/blob/master/operator/cmd/mesh/install.go#L205-L215 requires default webhook to exist which should not be assumed.
This breaks downstream tests. It seems to be introduced in https://github.com/istio/istio/pull/39674.
Version
master

Additional Information
No response

Reverted the culprit PR in https://github.com/istio/istio/pull/40050 to be safe. I think the old behavior is WAI but will take another look


Automated PR from brain-bican/models: new version of pydantic model(s).

This PR includes changes to the following models:
models_py-autogen/library_generation.py

This was a PR generated while testing new push_to_bkbit workflow in brain-bican/models repo. This PR does not need to be merged.


REST API link

The link titled 'More information on data transfer with Storage REST APIs.' points to https://docs.microsoft.com/azure/databox-online/data-box-gateway-deploy-add-shares. That seems wrong.

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

ID: fecae9a2-c605-e262-6e1d-1f1773961914
Version Independent ID: 63b778a3-d5a0-13a6-0b45-415cfdfd2772
Content: Choose an Azure solution for periodic data transfer
Content Source: articles/storage/common/storage-solution-periodic-data-transfer.md
Service: storage
Sub-service: blobs
GitHub Login: @alkohli
Microsoft Alias: alkohli

@flibbum Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@flibbum This is fixed and the change was merged. This should go live after 10 AM today.
Closing this now.
#please-close


Add securityContext config to Registry pod spec for run bundle(-upgrade)

Description of the change:
Registry pod doesn't have "securityContext.runAsNonRoot=true" config which should be generated by run bundle(-upgrade).
Steps to Reproduce:

operator-sdk run bundle quay.io/example/bundle:v4.10
oc get pods

Actual results:
3. oc get pod quay-io-example-bundle-v4-10 -o yaml | grep "securityContext" -A1. 4.
securityContext: {}
serviceAccount: default
Expected results:
5. oc get pod quay-io-example-bundle-v4-10 -o yaml | grep "securityContext" -A1.6.
securityContext:
runAsNonRoot: true
serviceAccount: default
Motivation for the change:
Set the security context for the registry pod by including the securityContext field in the Pod specification which will apply to all containers in the pod. This field defines the pod's privilege.
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

cc @everettraven


evaluate conceptual material in references section

reviewing the conceptual content in the References section on the docs.getdbt.com site and propose recommendations on how to re-organize content
Description & motivation

To-do before merge

Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects
[ ] Run link testing to update the links that point to the deleted page

Hi @mirnawong1 - This draft is almost a year old. As we've moved a lot since then and have more changes incoming, would it be a good idea to close this PR and open an issue to get this addressed in the future?

Hey @matthewshaver , thank you!  Yes this was part of a project that I was working one earlier. Then it was deferred to post coalesce after discussion with Leona so I was going to look at this next month. I'm happy to expedite this and take a look next week if it's blocking you on anything.

hey @matthewshaver , i've looked at this pr and it doesn't have a lot of changes so going to close this up. thank yo for the reminder!


chore: improve round calculation from votes

Purpose
Fixes: #1464
How does the solution address the problem

Changes made

I don't understand issuehttps://github.com/Taraxa-project/taraxa-node/issues/1464. In data structure verified_votes_, we have table store each step for each voted value, and for each voted value has total votes weight.

prnet cleaned-up successfully


zoom to extent of project not current location after initial project load.

Description :
Once the project is loaded, the app focusses on to the users location which may not be the actual project location. Zooming to project location would be an optimal solution.
Are you able to reproduce the problem in a new project after restarting the app?
Yes

@sdamaraju can you look into this again? @panamint55 is getting a zoom to current location still instead of the extent of active datasets. The zoom to extent needs to happen not just from the sign-in screen but also when you switch a project.

@jessicagood looking into this.

@jessicagood @panamint55
I looked into this, I pulled the latest binaries and tried to reproduce the issue, in vain.
It works pretty well to me. (Switching the projects, loading the projects as well)
There is one issue, which I fixed today, that's when we switch on and off the data sets, we are not zooming to appropriate extents, that part is taken care as part of the commit that I'm going to push in.

@panamint55 We fixed this issue and now, even when the active data sets are turned on or switched off, the zoom to extent of spots gets triggered.
Please let us know if you are facing any issues on this, else we can drive this to closure.

@sdamaraju I'm finally working with a project not around my home location and noticed I am actually still getting this issue if I close the app and reopen it. It always zooms to my location on opening the app even if the map was centered on the data before closing the app. Was closing and restarting the app one of your test conditions?

Oh, reopening the app, I think was never considered in my logic.
It covers reloading the project or switching the projects or switching the
active datasets.
I will look into this again.
Thanks for letting me know
On Wed, 25 Nov 2020 at 2:50 AM, Jessica Good<EMAIL_ADDRESS>wrote:

@sdamaraju https://github.com/sdamaraju I'm finally working with a
project not around my home location and noticed I am actually still getting
this issue if I close the app and reopen it. It always zooms to my location
on opening the app even if the map was centered on the data before closing
the app. Was closing and restarting the app one of your test conditions?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/StraboSpot/StraboSpot-Mobile/issues/42#issuecomment-733239819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG44YK6FU2PE5COJWHAFPOTSRQPQPANCNFSM4NMJSZMA
.

@sdamaraju where are we on this issue?  I would like to get some of these closed asap.

@sdamaraju where are we on this issue?  I would like to get some of these closed asap.

@NathanNovak This one, was parked aside, as it had low priority compared to others.

@NathanNovak This one, was parked aside, as it had low priority compared to others.

@panamint55 @jessicagood  this one is something I wanted to talk about in our call, but I missed.
So last time when we discussed on it, I remember professor Doug saying that, if we have to zoom to extent of spots only when reopening the app, it should be fine. What I don't remember is whether we are fixing this or closing the bug.
@panamint55 how was the experience of zoom to extent of spots in the last field camp ? Do you feel we need to handle this case as well where we zoom to extent of spots even while reopening the app or is there any other scenario where the app takes you to your location than to extent of spots.

OK, here is how I see it now.

When you first open a project it sets the map to the current location.  That is actually fine.
You can then zoom to extent of spots, and that seems fine as well.  So it works.
This all seems fine, so I guess I would remove this as a bug, and let the app work the way it does now.
Nothing would be perfect, and folks can have different reactions.
As long as we can zoom to extent of spots, zoom to  single spot, I think things are good.

Closing this bug, as per professor Doug's comments above.


DIF10: Add string length check for Sensor Characteristic Value

Is this a new check? Please describe the check and the field.
</DIF/Platform/Instrument/Sensor/Characteristics/Value>: Check if value provided is a string between 1 and 80 characters.
Describe the checks in steps

Check to see if a characteristic value is provided
If one is provided, check that it is a string between 1 and 80 characters.

Is there a similar check already implemented? Please list the name of the check
N/A
Additional context
N/A

this was fixed in feature/ummg_support


Asteroid events still happen in greenshift

Description
In greenshift, asteroid events still happen, which is not a good thing if you like your station untarnished by entropy for when you need to do something
Reproduction
Start a round in greenshift. Wait. Eventually observe asteroids slowly destroying your station. Cry. Make an issue on github

can confirm, additional context: this happened on greenshift, not secretgreenshift. might be fixable by just making the meteor scheduler not start in greenshift and secretgreenshift


feat: expand input types with coercion to work with ngtsc input type checking

Related to #17495

Blocked on:

https://github.com/angular/angular/issues/33452. We can workaround this and patch @angular/bazel in the meanwhile
https://github.com/angular/angular/issues/33451. We could disable strictMetadataEmit for the legacy build. This opens up the other question: why is this not enabled within Bazel?
NgModel has an input called disabled that conflicts with the coerced disabled input if used on the same element. e.g. <mat-slider disabled ngModel>

Needs rebase

@andrewseguin Rebased. I had to fix a couple of newly introduced coercion inputs, so I think as soon as this is green, it would be nice to merge it.

There is still a problem with async pipe:
TS2326: Types of property 'opened' are incompatible.
  Type 'boolean | null' is not assignable to type 'string | boolean | undefined'.


https://objects.githubusercontent.com/github-production-release-asset-2e65be/736363266/7a4cf3ec-fa8d-43fa-bc51-1ed1a9716e13?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=releaseassetproduction%2F20240904%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240904T202515Z&X-Amz-Expires=300&X-Amz-Signature=ff90beca89f0cc6477f12ef3f1fa36518711f16dafd4150467f35be57b99224e&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=736363266&response-content-disposition=attachment%3B filename%3DAppVerifier-0.8.0.apk&response-content-type=application%2Fvnd.android.package-archive

https://objects.githubusercontent.com/github-production-release-asset-2e65be/736363266/7a4cf3ec-fa8d-43fa-bc51-1ed1a9716e13?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=releaseassetproduction%2F20240904%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240904T202515Z&X-Amz-Expires=300&X-Amz-Signature=ff90beca89f0cc6477f12ef3f1fa36518711f16dafd4150467f35be57b99224e&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=736363266&response-content-disposition=attachment%3B filename%3DAppVerifier-0.8.0.apk&response-content-type=application%2Fvnd.android.package-archive

https://objects.githubusercontent.com/github-production-release-asset-2e65be/736363266/7a4cf3ec-fa8d-43fa-bc51-1ed1a9716e13?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=releaseassetproduction%2F20240904%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240904T202515Z&X-Amz-Expires=300&X-Amz-Signature=ff90beca89f0cc6477f12ef3f1fa36518711f16dafd4150467f35be57b99224e&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=736363266&response-content-disposition=attachment%3B filename%3DAppVerifier-0.8.0.apk&response-content-type=application%2Fvnd- [ ] .android.package-archive berson berson berson rwy'n credu

What?

@.***
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: matchboxbananasynergy @.>
Sent: Thursday, September 5, 2024 8:49:42 AM
To: soupslurpr/AppVerifier @.>
Cc: Yekulproducts @.>; Author @.>
Subject: Re: [soupslurpr/AppVerifier] https://objects.githubusercontent.com/github-production-release-asset-2e65be/736363266/7a4cf3ec-fa8d-43fa-bc51-1ed1a9716e13?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=releaseassetproduction%2F20240904%2Fus-east-1%2Fs...
This just looks like spam.
—
Reply to this email directly, view it on GitHubhttps://github.com/soupslurpr/AppVerifier/issues/164#issuecomment-2329991227, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKNLDJ4AZPPP4CFOX5VNXMTZU5W6NAVCNFSM6AAAAABNVCL42SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRZHE4TCMRSG4.
You are receiving this because you authored the thread.Message ID: @.***>


Set sampling percentage

As a project developer, I want sampling percentages set, so that a percentage of telemetry data is sampled out, reducing costs

Mentioned in #235


Merge release/1.1 to main

Sync main with recent changes that landed on release/1.1.

@swift-ci test

D'oh, the manifest needs updates.
error: 'swift-collections': product '_CollectionsUtilities' required by package 'swift-collections' target 'SortedCollections' not found. Did you mean 'InternalCollectionsUtilities'?

@swift-ci test


Pascal and Objective-C Grammar

Hi! I am using this library through the MDEx Elixir library.
I have some code snippets that requires
Objective-C and Pascal grammars.

https://github.com/Isopod/tree-sitter-pascal
https://github.com/merico-dev/tree-sitter-objc

Thank you so much

Can do! I'll have to use this grammar for Objective-C, as the one you linked doesn't have any highlighting queries.
I should push a new version sometime in the next few days.

@clsource Pascal is already included in Inkjet. I'll add it to MDEx.
I'm doing some work on the syntax highlighting and will update MDEx soon.

Apologies for the delay! :bow:
I got caught up in another project with a deadline. I'll push Objective C support shortly!


ci: update to bitcoin 28.0

Description

Updates bitcoin to latest 28.0
Removes a unofficial PPA for the official release in the bitcoind docker image

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Refactor
[x] Test

Checklist

[x] I have performed a self-review of my code.
[x] I have commented my code where necessary.
[x] I have updated the documentation if needed.
[x] My changes do not introduce new warnings.
[x] I have added tests that prove my changes are effective or that my feature works.
[x] New and existing tests pass with my changes.

Draft I don't want to debug this before finishing #399


doesn't run... :-(

followed all steps from :
http://bcbio.wordpress.com/2011/11/29/making-next-generation-sequencing-analysis-pipelines-easier-with-biocloudcentral-and-galaxy-integration/

after hitting "Process" in the very last step from within Galaxy pipelines, it simply returns in the main Galaxy and doesn't show any running jobs there. Running qstat after having logged in does not show anything either.

while logged in I also tried :
ubuntu@ip-10-116-201-41:/export/data/work/2worker-run$ distributed_nextgen_pipeline.py /export/data/galaxy/post_process.yaml /export/data/exome_example/fastq /export/data/exome_example/config/run_info.yaml
Starting manager
.... but it just keeps hanging here forever.....

Where could I pull more data from to help you debug it  ?

another minor issue: the "cloudman" security group that is created by default when doing cluster initialization from  biocloudcentral, doesn't work for whose who might have followed blindly the Step 1 of the Cloudman instructions from here
http://wiki.g2.bx.psu.edu/Admin/Cloud
and already have security group named cloudman (biocloudcentral complains about that). Maybe the security group should be created as "biocloudcentral"

Ntino;

followed all steps from :
http://bcbio.wordpress.com/2011/11/29/making-next-generation-sequencing-analysis-pipelines-easier-with-biocloudcentral-and-galaxy-integration/

after hitting "Process" in the very last step from within Galaxy
pipelines, it simply returns in the main Galaxy and doesn't show any
running jobs there.

The feedback/monitoring in Galaxy needs work. Eventually I'd like to
have a console that monitors the process of jobs; right now you have to
have faith or ssh in to monitor the job.

Running qstat after having logged in does not show anything either.

The way to check what is happening is to go to /export/data/work. There
should be a directory with the date; this directory will have the SGE
logs and you can see what is happening with the process.
The jobs run as the galaxy user, so to monitor with qstat either sudo as
galaxy or run 'qstat -f' to see what is happening with the SGE cluster.
The ubuntu user won't have any scheduled jobs.

while logged in I also tried :
ubuntu@ip-10-116-201-41:/export/data/work/2worker-run$ distributed_nextgen_pipeline.py /export/data/galaxy/post_process.yaml /export/data/exome_example/fastq /export/data/exome_example/config/run_info.yaml
Starting manager
.... but it just keeps hanging here forever.....

If processes are already running, which hopefully they are from the web
submission, then the process waits for available cores to
run. All of the cores should be working hard from the web submission, so
this is why it's appearing to hang.
Hopefully this helps. Thanks for trying everything out,
Brad

Ntino;
What was the error message you were getting from BioCloudCentral? The general idea in the code is to skip creating a security group or any permissions if they exist, so maybe there was some incompatibility we should look out for.

Ntino -- I'm going to close this one for now. If you run into more security group issues let us know and we can revisit.


add email validation regex

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
179

Relevant Lines:
1665

💛  - Coveralls


New client

closes #7
Initial commits to get  the new client at the same state as the nextjs client was

Turns out we gotta setup cors on the server - could this be part of this PR for speedup?


Fix term

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/9488
@mkbond007

Learn Build status updates of commit e57d2f7:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

Teams/meeting-policies-audio-and-video.md
:bulb:Suggestion

Details

Teams/meeting-policies-audio-and-video.md

Line 208, Column 136: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /powershell/module/skype/new-csteamsmeetingpolicy?view=skype-ps

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Implementing explicit list syntax [...]

This pull requests drafts and implements and explicit list syntax for Tokay.
Similar to Python and Rust, it uses [...] to specify explicit lists.
This eliminates the following problems Tokay has so far

(...) either produces dicts or lists, () is the explicit dict
[...] produces lists, [] is the explicit list
By resolved #79, dicts can be used like lists
By #98 the former character-class syntax was replaced to make the brackets available

This PR is a draft and under development.

* `(...)` either produces dicts or lists, `()` is the explicit dict

This isn't useful. Currently, (1 2 3) generates repr [1, 2, 3].
Short roadmap on this topic:

(...)-syntax is called a dict-bound sequence and should always produce dicts. (1 2 3) should reproduce repr (1, 2, 3).
[...]-syntax is called a list-bound sequence and always produces lists. It should allow for |-operator as well, equally to dict-bound syntax, to specify inline sequences
The AST traversal, respectively object traits, should allow iteration over either a list or a dict's values in order.
The pos/kle repeat-constructs currently either produce a single dict or a list of dicts. This is should be changed to always product a list of 1..n items, or in case of kle, void, if no match is found.

It should also be considered that this PR is being continued when #10 is generally implemented, so that several constructs become easier to define.


Added number input field to gain and pan so users can enter a precise value

This should solve issue #41 .
I tested it on multiple websites and it seemed to work fine, I'd be happy if we can discuss these changes.
Note I tried to maintain the formatting of the code, but it was kinda difficult without any formatting config file in the project, however, I did maintain almost all of it.
Let me know what you think!

Done. you could check if these are all the necessary changes

Fixed indentation, this should cover all the necessary changes

Thanks! I'll look into releasing an update soon.


WITH (NOLOCK) Errors

概要
不具合の簡単な概要を記述してください
SQL queries that contain with (nolock) produce errors in the parser. This happens with the assembly in C# and on your website.
再現方法
不具合を再現させる手順を教えて下さい
Example Query:
SELECT
i.itemnum AS [OBDocumentID],
i.datestored As [DateStored],
i.itemdate As [DocumentDate],
i.itemtypenum As [DocumentType],
kgd121.kg119 As [Department],
kgd121.kg128 As [Group],
kgd121.kg139 As [Nation],
kgd121.kg148 As [PurchaseDate],
ki150.keyvaluechar As [Region],
kgd121.kg152 As [RegistrationLicensePlate],
kgd121.kg157 As [SaleDate],
kgd121.kg161 As [SubDocType],
kgd121.kg166 As [Unit],
kgd121.kg169 As [WIN],
kgd121.kg137 As [Location]
FROM hsi.itemdata i WITH(NOLOCK)
INNER JOIN hsi.keygroupdata121 kgd121 WITH(NOLOCK) ON kgd121.itemnum = i.itemnum
INNER JOIN hsi.keyitem150 ki150 WITH(NOLOCK) ON ki150.itemnum = i.itemnum
WHERE i.itemdate BETWEEN '1/1/2019' AND '2/1/2019'

不具合が発生した環境

OS: Windows10
Browser Opera
Version 58
https://www.minisqlparser.net/en/

Thank you,
Darren

Thank you for your error report. I'm going to fix this.

I announced this issue is fixed in the 令和 holidays. Add some MS SQL Server Table hint grammar include "NOLOCK". see below sample result.

Looks good. Thanks!


UI user icon not using primary-color setting

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

<IP_ADDRESS>

SUMMARY

After login into UI the top right user icon does not use @primary-color from theme section of config.json

I am working on it .

@sbrueseke  I could not reproduce this on main or 4.18 . Is this bug specific to <IP_ADDRESS>?

@sbrueseke I could not reproduce this on main or 4.18 . Is this bug specific to <IP_ADDRESS>?

I am not able to install 4.18 at the moment, so I cannot test it on this version.
This is how the html code looks like on 4.17.2:
SB
It looks like background-color in styles is not beeing changed by the theme.
Can you maybe double-check the code on 4.18?

@rahulbcn27 I updated our test installation to <IP_ADDRESS> and it looks like this issue is still present.
I am talking about the small avatar icon at the top of the right corner:

html source code show that style tag has an attribute background-color which is set to the default value and not to the primary-color value. Can you please check this an verify it? Thank you!


Oss observability proposal draft

The goal of this proposal is to add observability to Buck2's OSS variant that is on par with similar build systems like Bazel.

Let's move the discussion from Discord to this PR so folks can reference items in our discussion more easily. For context, I also raised https://github.com/bazelbuild/remote-apis/issues/318 to the Remote API working group and got some pretty interesting feedback there.
First, let's give a high-level overview of Bazel's current BES/BEP state.
Currently, in Bazel, we have this PublishBuildEvent service with PublishBuildToolEventStream being the most interesting method inside.
service PublishBuildEvent {
  ...
  // Publish build tool events belonging to the same stream to a backend job
  // using bidirectional streaming.
  rpc PublishBuildToolEventStream(stream PublishBuildToolEventStreamRequest)
      returns (stream PublishBuildToolEventStreamResponse) {
      ...
  }
}

This establishes a bi-directional stream for the client to send events to the server.
Each event is annotated with a stream ID and a sequence number (int64) that the server could then send back to the client as an "ack", letting the client know that the event was received successfully. This send-ack mechanism allows events to be sent in the right order and the client can detect send failures to retry accordingly.
I think this is a good design that we should keep in Buck2.
Next, let's talk about the Build Events. There are actually 2 "event" protos in Bazel. An outer generic one and an inner set of "events" that are Bazel-specific.
The first type of events is google.devtools.build.v1.BuildEvent.
message BuildEvent {
  ...
  // //////////////////////////////////////////////////////////////////////////
  // Events that indicate a state change of a build request in the build
  // queue.
  oneof event {
    // An invocation attempt has started.
    InvocationAttemptStarted invocation_attempt_started = 51;

    // An invocation attempt has finished.
    InvocationAttemptFinished invocation_attempt_finished = 52;

    // The build is enqueued.
    BuildEnqueued build_enqueued = 53;

    // The build has finished. Set when the build is terminated.
    BuildFinished build_finished = 55;

    // An event containing printed text.
    ConsoleOutput console_output = 56;

    // Indicates the end of a build event stream (with the same StreamId) from
    // a build component executing the requested build task.
    // *** This field does not indicate the WatchBuild RPC is finished. ***
    BuildComponentStreamFinished component_stream_finished = 59;

    // Structured build event generated by Bazel about its execution progress.
    google.protobuf.Any bazel_event = 60;

    // An event that contains supplemental tool-specific information about
    // build execution.
    google.protobuf.Any build_execution_event = 61;

    // An event that contains supplemental tool-specific information about
    // source fetching.
    google.protobuf.Any source_fetch_event = 62;
  }
}

This is what is being sent in the bidi stream above. All the Bazel-specific events are currently being stuffed into the Any bazel_event = 60; field. This generic outer event and the service, together, could be referred to as Bazel's Build Event Service(BES).
Bazel's Build Event Protocol(BEP) is the set of inner events that are stuffed into the bazel_event field in BES. Confusingly enough, BEP is defined in a file called build_event_stream.proto.
From the Remote APIs meetings, folks were very much in favor of standardizing BES and not BEP as the latter is specific to Bazel. My past proposal in https://github.com/facebook/buck2/pull/685, which @aherrmann is building upon, introduced a much more simplified version of BES that is specific to Buck2's BuckEvent:
message BuckEventRequest {
  // A trace-unique 64-bit identifying the stream.
  uint64 stream_id = 1;

  buck.data.BuckEvent event = 2;
};

message BuckEventResponse {
  // A trace-unique 64-bit identifying the stream.
  uint64 stream_id = 1;

  // The trace ID of the event that has been committed.
  uint64 trace_id = 2;
};

service BuckEventPublisher {
  rpc StreamBuckEvent(stream BuckEventRequest) returns (stream BuckEventResponse);
};

I picked this because I think it's the simplest interface that I think can get the job done.
I would be ok with folks wanting to implement BES instead of this interface with a new field added like this.
message BuildEvent {
  ...
  oneof event {
    ...

    // Structured build event generated by Bazel about its execution progress.
    google.protobuf.Any bazel_event = 60;

    ...

    // Structured build event generated by Buck2 about its execution progress.
    google.protobuf.Any buck2_event = 63;
  }
}

Just note that bringing on the entire BES could add additional complexity to the implementation.

Protocol aside, there is a need to make BuckEvent work with existing tooling today.
Most UIs/tools available work with Bazel's BEP and not BuckEvent, and it would take a lot of effort for these tools to add support for BuckEvent. So I see a need for a conversion logic crate that transforms some BuckEvent messages into BEP messages.
However, I prefer to treat it as a separate problem from the protocol as it would help narrow down the problem scope and increase the chance of success for the implementation.

Small correction: @aherrmann implemented BuckEvent -> Bazel BEP conversion in #811 instead of using my proposal 😅

I opted for that route for now to keep the implementation simple while it is at a more exploratory stage. That said, it is structured in a way that should make it easy to adopt the pattern proposed in #685.
buck_to_bazel_events turns a stream of Buck events into a stream of Bazel events. This could be turned into an adapter component that serves a Buck2 event stream protocol as proposed in #685 and proxies to a Bazel event stream server.


Module and API naming

Module structure:

Unicode.Char: re-export all the modules

Unicode.Char.Normalization

Compose
Decompose
CombiningClass

Unicode.Char.General:

isAlpha
isSpace
Hangul

Unicode.Char.Case

isLower
isUpper

Unicode.Internal.Generated => Unicode.Internal.Char

Function naming:

isSecondStarter => isCombiningStarter
isAlpha => isLetter

Few thoughts:

Is Unicode.Char.Case going to have any more functions? A module with only 2 functions? Case seems very specific.
I prefer Unicode.Internal.Generated.Char over Unicode.Internal.Char. Unicode.Internal.Char gives a false sense of just re-exporting (mirroring the structure), like how we do it in Streamly.

Is Unicode.Char.Case going to have any more functions? A module with only 2 functions? Case seems very specific.
The module org follows this: https://www.unicode.org/reports/tr44/#Property_Index_Table as I mentioned in the README update in the PR.

I prefer Unicode.Internal.Generated.Char over Unicode.Internal.Char. Unicode.Internal.Char gives a false sense of just re-exporting (mirroring the structure), like how we do it in Streamly.

That's a very specific intuition because you have worked on streamly. We can make it Unicode.Internal.UCD instead. That will rightly indicate that this is all directly from UCD.

The module org follows this: https://www.unicode.org/reports/tr44/#Property_Index_Table as I mentioned in the README update in the PR. Yes it will have many more functions.

Cool cool. It's always nice to have module names derived from elsewhere.

Closed by #29


Python versions conflict

sudo apt-get install python-catkin-tools
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
python-catkin-tools : Depends: python (< 2.8)
Depends: python (>= 2.7)
Depends: python-catkin-pkg (>= 0.2.9) but it is not installable
Depends: python-osrf-pycommon but it is not installable
Depends: python:any (>= 2.6.6-7~)
E: Unable to correct problems, you have held broken packages.
This is what I see after trying to install Catkin tools. I followed all directions exactly up to this point.

Hi @Russ76 thanks raising this issue. I am sorry about that the docs are not up to date on this. I assumed it would be easier to install catkin-tools in the future when I wrote the docs. However, the current way to do it is using this command:
sudo apt-get install python3-osrf-pycommon python3-catkin-tools

Please see the discussion here. I hope this will be resolved soon - at least the discussion is pretty active, looking at the latest comment.

Thanks, that's fast! In the "Mate" browser, the web page looked different (your site, not Github), I had to get the URL from my laptop and type that into Mate to get the full number of web pages with more instructions. I don't know what happened.

Yes I need to fix this, the two web pages, fjp.at/projects/diffbot (old) and fjp.at/diffbot are not always up to date with the latest docs in this github repository.
I hope it is possible to add a github action to build the mkdocs automatically everytime a push is made - I'll create a new issue for this.
I will close this issue for now, as I hope the command above solves this issue for now. And I guess in the future the standard way, using the ubuntu package, will also work.


Add concatenation

Adds concat operator which merges indexed iterables and joins keyed iterables.

@colbyr If you get a chance, I'd love to hear your thoughts on this. We'll be looking to merge this sometime next week.

@colbyr
The intention was to only use this on indexed types but I figured it couldn't hurt to support keyed types as well, although keyed concatenation should be identical to merge behavior. By dropping the Iterable and replacing with Iterable.Indexed, I think this addition would better serve its intended purpose.
However, I am unable to target Iterable.Indexed in the implementInherited method, limited to only target Iterable. Else, the protocol is unable to pick up on immutable objects like Seq and List.

Targeting Collection.Indexed and Seq.Indexed seemed to do the trick. Going to add one small test for ensuring that concat can't be called on keyed objects

^ force pushed over the history to remove the @hubspot email


Masque support isn't working. I can't skin the button at all.

I left a comment about this on the addon page at CurseForge back in April. You told me, "It requires a /reload to be active after the setting is enabled and works pretty fine on my end."
I responded back saying that I tired this and it didn't work. Doing a /reload after enabling the setting didn't work. New Openables doesn't show up on Masque list of addons for me to skin. I have tired doing a /reload after enabling, and disabling Masque support setting. I have also tired doing a /reload after enabling and disabling the skin button setting and nothing happens. I have also tired deleting the addon and all it's setting files from the WTF folder and reinstalling the addon fresh and that didn't work either.
Anyways, hoping to hear a response.

NOP does not add own profile options to Masque but uses the default settings. There is no support for dedicated skin settings.

Hmmm. Alright, so I completely deleted everything New Openables again, and deleted everything Masque as well. No SavedVariables for both apps, a fresh install for both, and I'm still having the same issue. I don't know why you can see NOP on Masque, but I can't.
https://i.imgur.com/x9eyzck.png

What do the following commands print in chat?
/run print(NOP and NOP.masque)
/dump NOP_dev.AceDB.profile.masque

/run print(NOP and NOP.masque) simply said "nil"
/dump NOP_dev.AceDB.profile.masque just says:
Dump: value=NOP_dev.AceDB.profile.masque

I needed to correct the second command. Can you retry this please?
I'll push an alpha version with some small adjustment to see if this improves things.

That worked! Thank you! I'm so stoked, I've lived without a skinned button for so long. haha

Thanks for verifying :)


Fix PIO CI Builds

Proposed changes
This change fixes the currently broken PIO CI builds by correcting the submodules ignore paths in the PIO INI file.
Types of changes

[X] Bugfix (non-breaking change which fixes an issue)
[X] Build (changes that affect the build system)

Checklist

[X] I have read the CONTRIBUTING documentation
[X] Lint and unit tests pass locally with my changes

Further comments

N/A

@ciband Thanks for submitting this pull request, a maintainer will get back to you shortly!

@air1one @faustbrian - please review this in the next few days. Be sure to explicitly select labels so I know what's going on.
If no reviewer appears after a week, a reminder will be sent out.

@ciband Your pull request doesn't follow our contribution guidelines. Please review and correct it.

@ciband A contributor has approved this PR. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait.
Thank you for your contribution!

@ciband Your pull request has been merged and marked as tier 5. It will earn you $5 USD.


Citation.cff file

Hello,
would it be possible to add a citation file to this repository please? https://citation-file-format.github.io
I would like to reference this work in my thesis if possible with the citation provided by the authors.
Thanks and kind regards!

Done! Thanks for the suggestion :)

Thanks!


ArgumentType.CUSTOM⚙

Scratch had its own ArgumentType for each unique value. With the addition of extensions, new types appear (url, json, color, and so on)
I think we can try to let users create their own types. Here is a possible example:
        opcode: 'get',
        blockType: Scratch.BlockType.REPORTER,
        text: 'GET [URL]',
        arguments: {
          URL: {
            type: Scratch.ArgumentType.CUSTOM,
            menu: Scratch.ArgumentMenu.TEXT,
            menuText: 'enter url here',
            characters: 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789_-.~:/?#[]@!$&\'()*+,;=',
            function: UrlFunction,
            defaultValue: 'https://extensions.turbowarp.org/hello.txt'
          }
        }

,
        const UrlFunction = (A) => {
          try {
            return String(new URL(String(A)));
          } catch(err) {
            try {
              return String(new URL('https://' + String(A)));
            } catch(err) {
              return 'https://extensions.turbowarp.org/hello.txt';
            }
          }
        }

,

menu options: Scratch.ArgumentMenu.TEXT,Scratch.ArgumentMenu.NONE,Scratch.ArgumentMenu.ANGLE,Scratch.ArgumentMenu.COLOR,
Scratch.ArgumentMenu.MATRIX,Scratch.ArgumentMenu.NOTE
menuText options: Any string (works only with Scratch.ArgumentMenu.TEXT)
characters options: String of valid characters in a row (May need to replace string with regexp) (If not declared or null, all characters are available)
function options: Function that will replace the entered value with the edited one when the user completes it (If not declared or null nothing will happen)
PS: If this is possible, then it is probably also worth adding built-in ArgumentType.JSON and ArgumentType.URL for convenience

There is actually already some unfinished code in scratch/turbowarp to support it https://github.com/LLK/scratch-vm/commit/ceaa3c7857b79459ccd1b14d548528e4511209e7
I got it working here: https://github.com/Xeltalliv/extensions/blob/examples/examples/custom-field-types.js

@Xeltalliv Thanks)


Determine a good default value for MultiSampleQuality

This value is D3D9-only and determines sample counts. Currently it is set to 0 but should probably be increased. Need to figure out what the possible options are and how they affect visuals.
See DecodeMultiSampleType in d3d9_util.cpp

I believe 0 is fine and we shouldn't have to worry about it. In D3D9, the value appears to be used only in conjunction with D3DMULTISAMPLE_NONMASKABLE, which is not present in D3D8.

Whether the display device supports maskable multisampling (more than one sample for a multiple-sample render-target format plus antialias support) or just non-maskable multisampling (only antialias support), the driver for the device provides the number of quality levels for the D3DMULTISAMPLE_NONMASKABLE multiple-sample type. Applications that just use multisampling for antialiasing purposes only need to query for the number of non-maskable multiple-sample quality levels that the driver supports.
The quality levels supported by the device can be obtained with the pQualityLevels parameter of IDirect3D9::CheckDeviceMultiSampleType. Quality levels used by the application are set with the MultiSampleQuality parameter of IDirect3DDevice9::CreateDepthStencilSurface and IDirect3DDevice9::CreateRenderTarget.


Change parent's state to selected dates

render() {
return (
  <InfiniteCalendar
    Component={withMultipleDates(Calendar)}
    selected={this.state.selectedDates}
    interpolateSelection={defaultMultipleDateInterpolation}
  />
);
}

It doesn't change the parent's state.

You need to pass an onSelect prop and mutate your state accordingly. onSelect returns the newly selected date, so it's up to you to decide how to handle that event.
If you want your state to live in your parent component though, you should probably use the controlled Calendar component instead though, to avoid duplication of state.
import {Calendar, defaultMultipleDateInterpolation} from 'react-infinite-calendar';
const MultipleDatesCalendar = withMultipleDates(Calendar);

render() {
  return (
    <MultipleDatesCalendar
      selected={this.state.selectedDates}
      onSelect={(selectedDate) => defaultMultipleDateInterpolation(selectedDate, this.state.selectedDates)}
    />
  );
}

Check out the implementation of defaultMultipleDateInterpolation if you're curious about how it works: https://github.com/clauderic/react-infinite-calendar/blob/master/src/Calendar/withMultipleDates.js#L65