refactor: Unify usage of dark/ light mode

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Closes #5096
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Just wondering if we can reuse something from VueUse https://vueuse.org/core/useDark/

Just wondering if we can reuse something from VueUse https://vueuse.org/core/useDark/

Yes! seems to be the way with nuxt3

Both useDark(VueUse) and useColorMode(@nuxtjs/color-mode) are well supported with nuxt3 and easy to apply. I have written a demo to switch to one of the solutions.
However, I found that we could not use  useDark(VueUse) for now, because some class components still use $colorMode.preference and so we still need to keep @nuxtjs/color-mode in the codebase. Once we finally refactor all components to vue3 composition api, both these two solutions could be chosen without barrier.
@vikiival @roiLeo

So dumb question what will we break if we use vueuse now?

So dumb question what will we break if we use vueuse now?

We could not use both VueUse and useColorMode at the same time.
For example, if we use useToggle from vueuse to change dark mode to true, the state($colorMode.preference) from @nuxtjs/color-mode would still be false. They will have different color mode in different componets.
If we would like both of them have the same state, we need to do useToggle and change the $colorMode.preference in the same time. I guess it was a bad way.

I'm waiting for https://github.com/kodadot/nft-gallery/pull/5033 to be merged first, so that I could apply the color mode composables to navbar component.

I'm waiting for #5033 to be merged first, so that I could apply the color mode composables to navbar component.

✅ Done

pay 35 usd

😍 Perfect, I’ve sent the payout
💵 $35 @ 38.8 USD/KSM ~ 0.902 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x1189cc64163cc32578c0a68db807a809c356a3731d14239ca2fd1169190a93be
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Defined new revisions for golang dep: github.com/Shopify/logrus-bugsnag

@AndrienkoAleksandr It looks like the license is MIT: https://github.com/Shopify/logrus-bugsnag/blob/577dee27f20dd8f1a529f82210094af593be12bd/LICENSE
If you agree, I will update the PR.

If you agree, I will update the PR.

I agree. It's my mistake.


PHA4GE template: reorder BioSample export fields

I updated the field names in the DH template and tested the biosample export.
Can we reorder the fields so they match the submission form?
The correct order can be found in the file attached.
NCBI field names for DH.xlsx

I've updated the Biosample export field list.  Note that it has some extra entries?  It now reads:
['sample_name',             []],
//sample_title
['bioproject_accession',    []],
['organism',                []],
['collected_by',            []],
['collection_date',         []],
['geo_loc_name',
[
'geo_loc_name (country)',
'geo_loc_name (state/province/territory)',
'geo_loc name (county/region)',
'geo_loc_name (city)'
]
],
['host',                    []],
['host_subject_id',         []],
['host_disease',            []],
['isolate',                 []],
['isolation_source',
['anatomical material','anatomical part','body product','environmental material','environmental site','collection device','collection method']],
['antiviral_treatment_agent',
['collection_device',       []],
['collection_method',       []],
['date_of_prior_antiviral_treat',  []],
['date_of_prior_sars_cov_2_infection',  []],
['date_of_sars_cov_2_vaccination',  []],
['sequenced_by',            []],
['sample collection date',  []],
['exposure_event',          []],
['geo_loc_exposure',        []],
['gisaid_accession',        []],
['gisaid_virus_name',       []],
['host_age',                []],
['host_anatomical_material', []],
['host_anatomical_part',    []],
['host_body_product',       []],
['environmental_material',  []],
['environmental_site',      []],
['host_disease_outcome',    []],
['host_health_state',       []],
['host_recent_travel_loc',  []],
['host_recent_travel_return_date',  []],
['host_sex',                []],
['host_specimen_voucher',   []],
['host_subject_id',         []],
['lat_lon',                 []],
['passage_method',          []],
['passage_number',          []],
//['passage_history',         []], //passage_number?
['prior_sars_cov_2_antiviral_treat', []],
['prior_sars_cov_2_infection', []],
['prior_sars_cov_2_vaccination', []],
['date_of_sars_cov_2_vaccination', []],
['purpose_of_sampling',     []],
['purpose_of_sequencing',   []],
['sars_cov_2_diag_gene_name_1',   []],
['sars_cov_2_diag_gene_name_2',   []],
['sars_cov_2_diag_pcr_ct_value_1',   []],
['sars_cov_2_diag_pcr_ct_value_2',   []],
['sequenced_by',            []],
['vaccine_received',        []],
['virus_isolate_of_prior_infection', []],
['attribute_package',       []], //not in biosample sort list

//The following fields are not in NCBI Biosample but were listed in our code
['description',             []],
['lab_host',                []],
['host_disease_stage',      []],
['gene_name_1',             []],
['diagnostic_PCR_CT_value_1',  []],
['gene_name_2',             []],
['diagnostic_PCR_CT_value_2', []]

Yep, the ones that are missing are junk. :)

Where do you want attribute_package to show?


Feature: blob size in DDFS

It would be useful to be able to track how much space blobs occupy in DDFS.  We could modify some of the ddfs subcommand to report and/or aggregate on this data to help diagnose cluster free space issues.

Was thinking about this today, it would be interesting to add something in the extended attributes of the tag for the following metrics:

last accessed
first created
size in cluster

Then we could issue something along the lines of a ddfs stat for the tags and get similar output to stat on the filesystem.
This would add some overhead for each tagging operation, but we could also do this as a daily type internal job that could scrape every tag/blob and get the size from the filesystem. Depending on the amount of blobs and tags this might take > 24hrs though and would not be real time.

We could also add something like retention, where a "janitor" type job could iterate over the tags and see if that tag is set to expire, that way it could automatically clean them up for the next GC. Currently we use a script for this that is not very intuitive as we have to add tags to them if new types of data are pushed to the cluster.


Refractor methods, items, etc

Ready

@aminya probably needs to be rebased on top of master now :)

This is ready now. Appveyor app should be removed from repository to prevent it from throwing errors.

@suda I ran the CI again. This is ready.


Conversion from logits to probabilities happens on a batch by batch basis

🐛 Bug
I'm using BinaryAUROC and calling .update(logits, y) after each batch (validation_step), expecting the following behavior:
"If preds has values outside [0,1] range we consider the input to be logits and will auto apply sigmoid per element."
I noticed some discrepancies in the results, and after investigating realized that the rule above happens on a batch by batch basis rather than across the entire dataset (it happens in _binary_precision_recall_curve_format, which is called by update).
Therefore, if some batch has all logits in [0, 1], then the conversion doesn't happen for that batch.
My use case has batch size = 1, so this happens often.
Expected behavior
I'd expect that the conversion would happen consistently across the entire dataset (i.e., when calling .compute()). If not that, this behavior should be documented more prominently.
Environment
torchmetrics==1.0.3

I came to report just this. As a "workaround", I always call sigmoid first, as at least that path is sound.

Dupe of https://github.com/Lightning-AI/torchmetrics/issues/1604, probably


iOS Trust Wallet App

I want to report an issue with the TrustWallet application for iOS. After a thorough analysis, I identified that the issue lies in the native RPCs provided by the app. These RPCs do not allow reading data from smart contracts, and when calculating gas for transactions, they do so below the network's minimums, resulting in failed transactions.
To address the data reading issue, I've had to switch RPCs. However, this entails additional effort due to the numerous web pages I manage that rely on this application. It's worth noting that users prefer using TrustWallet.
Please, I urge you to correct this issue as soon as possible.
https://21billiondollars.com/

This issue occurs to me while using the following versions:
React 17-18
Next.js 13-15
ethers.js 5.7 and 6.11

To address this issue, I attempted to utilize ethers.js 5.7 with a Web3Provider as follows:
 const provider = new ethers.providers.Web3Provider(window.ethereum, "any");
const signer = provider.getSigner();
const contract = new ethers.Contract(address, ABI, signer)
However, all function calls made using this approach resulted in failures.
I managed to resolve the issue by switching to a JsonRpcProvider:
  const provider = new ethers.providers.JsonRpcProvider("https://bsc-dataseed1.bnbchain.org", "any");
const signer = new ethers.VoidSigner(<user address>, provider);
`
This modification allowed me to successfully interact with smart contracts and execute transactions without errors.
Using ethers 6 the error persists, but it only happens on iPhone, iPad and Mac

I'm encountering a specific issue with TrustWallet where I can only successfully execute transactions by increasing the gas limit and gas price. This issue persists across BSC, Poligon, and OPbnb networks.
I would appreciate your assistance in resolving this matter as soon as possible.

Hi @camicasii, thank you for opening the issue!
Our team is working on it, I'll give an update as soon as possible

I'm encountering a specific issue with TrustWallet where I can only successfully execute transactions by increasing the gas limit and gas price. This issue persists across BSC, Poligon, and OPbnb networks.
I would appreciate your assistance in resolving this matter as soon as possible.

Hi, I can leave the fees empty, we will calculate it properly for you

This issue is not related to the wallet core repository, please create a ticket here if u believe there is a bug in the application: https://support.trustwallet.com/en/support/tickets/new


[Windows] SIGTERM / Crash on connection

Using the example app results in a crash when connecting.
The issue was always there, in all LiveKit versions that I tried, not just the recent releases.
The WebRTC Example App is working fine for me on windows.
How can I try to debug this?
client_sdk: 1.0.0
server: 1.0.1
flutter: 3.0.1
Connecting to VM Service at ws://<IP_ADDRESS>:51014/Pl7Ck9atwPw=/ws
flutter: Connecting with url: wss://live.xxxx-xxxx.net, token: eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOjE2NTQ3ODY4NTAsImlzcyI6IkFQSWFlb0JFeENOdG5CUyIsIm5iZiI6MTY1MjE5NDg1MCwic3ViIjoic2wiLCJ2aWRlbyI6eyJyb29tIjoidGVzdDAxIiwicm9vbUpvaW4iOnRydWV9fQ.7m6xVj3UFXIZfAucgqLjziZEefuX8ZLmgAMPYnEAWVE...
flutter: 12:57:06: Engine ConnectionState disconnected -> connecting
flutter: 12:57:06: [EngineEvent] Engine#148511108 EngineConnectionStateUpdatedEvent(newState: connecting, didReconnect: false, disconnectReason: null)
flutter: 12:57:06: SignalClient connecting with url: wss://live.xxxx-xxxx.net/rtc?access_token=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOjE2NTQ3ODY4NTAsImlzcyI6IkFQSWFlb0JFeENOdG5CUyIsIm5iZiI6MTY1MjE5NDg1MCwic3ViIjoic2wiLCJ2aWRlbyI6eyJyb29tIjoidGVzdDAxIiwicm9vbUpvaW4iOnRydWV9fQ.7m6xVj3UFXIZfAucgqLjziZEefuX8ZLmgAMPYnEAWVE&auto_subscribe=1&adaptive_stream=0&protocol=7&sdk=flutter&version=1.0.0&os=windows
flutter: 12:57:06: SignalClient ConnectionState disconnected -> connecting
flutter: 12:57:06: [SignalClient#728484996] cleanUp()
flutter: 12:57:06: [SignalEvent] SignalConnectionStateUpdatedEvent(newState: connecting, didReconnect: false, disconnectReason: null)
flutter: 12:57:06: [WebSocketIO] Connecting(uri: wss://live.xxxx-xxxx.net/rtc?access_token=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOjE2NTQ3ODY4NTAsImlzcyI6IkFQSWFlb0JFeENOdG5CUyIsIm5iZiI6MTY1MjE5NDg1MCwic3ViIjoic2wiLCJ2aWRlbyI6eyJyb29tIjoidGVzdDAxIiwicm9vbUpvaW4iOnRydWV9fQ.7m6xVj3UFXIZfAucgqLjziZEefuX8ZLmgAMPYnEAWVE&auto_subscribe=1&adaptive_stream=0&protocol=7&sdk=flutter&version=1.0.0&os=windows)...
flutter: 12:57:06: [WebSocketIO] Connected
flutter: 12:57:06: SignalClient ConnectionState connecting -> connected
flutter: 12:57:06: [SignalEvent] SignalConnectionStateUpdatedEvent(newState: connected, didReconnect: false, disconnectReason: null)
flutter: 12:57:06: [SignalEvent] Instance of 'SignalJoinResponseEvent'
flutter: 12:57:06: onConnected subscriberPrimary: true, serverVersion: 1.0.1, iceServers: [urls: turn:<IP_ADDRESS>:443?transport=udp
urls: turns:live-turn.xxxx-xxxx.net:443?transport=tcp
username: test01
credential: esfxSYymurP5zq7IwkfScqIlEb0N3nYLNRvruWzzSTbB
flutter: 12:57:06: [PCTransport] creating {sdpSemantics: unified-plan, iceServers: [{urls: [turn:<IP_ADDRESS>:443?transport=udp, turns:live-turn.therapy-lift.net:443?transport=tcp], username: test01, credential: test01}]}
flutter: 12:57:06: [Engine] Received JoinResponse, serverVersion: 1.0.1
flutter: 12:57:06: Room Connect completed
flutter: 12:57:06: [ParticipantEvent] ParticipantPermissionsUpdatedEvent(participant: LocalParticipant(sid: PA_mFLZ9tREXocx, identity: sl), permissions: Instance of 'ParticipantPermissions'), will notifyListeners()
flutter: 12:57:06: [RoomEvent] ParticipantPermissionsUpdatedEvent(participant: LocalParticipant(sid: PA_mFLZ9tREXocx, identity: sl), permissions: Instance of 'ParticipantPermissions'), will notifyListeners()
flutter: 12:57:06: [PCTransport] creating {sdpSemantics: unified-plan, iceServers: [{urls: [turn:<IP_ADDRESS>:443?transport=udp, turns:live-turn.therapy-lift.net:443?transport=tcp], username: test01, credential: test01}]}
flutter: 12:57:06: [SignalEvent] Instance of 'SignalTokenUpdatedEvent'
flutter: 12:57:06: [SignalEvent] Instance of 'SignalOfferEvent'
9
flutter: 12:57:06: [SignalEvent] Instance of 'SignalTrickleEvent'
Lost connection to device.
Exited (sigterm)

PS C:\Users\winislose\client-sdk-flutter> flutter doctor -v
[√] Flutter (Channel stable, 3.0.1, on Microsoft Windows [Version 10.0.19044.1706], locale de-DE)
    • Flutter version 3.0.1 at D:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (12 hours ago), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at D:/android
    • Platform android-31, build-tools 30.0.3
    • Java binary at: D:\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.14)
    • Visual Studio at D:\Microsoft Visual Studio\2019
    • Visual Studio Community 2019 version 16.11.32428.217
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 4.1)
    • Android Studio at D:\androidstudio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] Android Studio (version 2021.3)
    • Android Studio at D:\android-studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[√] VS Code (version 1.67.2)
    • VS Code at C:\Users\winislose\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.40.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19044.1706]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 101.0.4951.67
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 99.0.1150.55

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

@rokk4 I will try to investigate this issue,

@cloudwebrtc  Thanks for your efforts, please let me know if I can do something to assist.

After fixing a few bugs, it works fine on flutter 3.0.1, I need to fix the upstream native sdk, which should make the windows version stable soon

related PRs.
https://github.com/webrtc-sdk/webrtc/pull/29
https://github.com/webrtc-sdk/libwebrtc/pull/26
https://github.com/webrtc-sdk/libwebrtc/pull/27

@rokk4 You can try this branch, it should fix these issues
https://github.com/livekit/client-sdk-flutter/pull/127


Fix pod config for envoy upstreaming termination error

What this PR does / why we need it:
Add preStop to api-server's manifest to enable envoy to know server is terminating
Which issue(s) this PR fixes:
Fixes #1349
Does this PR introduce a user-facing change?:
NONE

@kevin55156
I guess this's a connection error when rolling out, so I'm wondering whether this time sleep will resolve that or not.
wdyt?

We've already set 30s as gracePeriod at the application layer, but terminationGracePeriodSeconds is not set, which means the default value 30s is used.
Hence, I feel this will not work as we expected, wdyt?
So, how about setting terminationGracePeriodSeconds to 60s.
https://github.com/pipe-cd/pipecd/blob/master/cmd/pipecd/server.go#L106

We've already set 30s as gracePeriod at the application layer, but terminationGracePeriodSeconds is not set, which means the default value 30s is used.
Hence, I feel this will not work as we expected, wdyt? So, how about setting terminationGracePeriodSeconds to 60s.
https://github.com/pipe-cd/pipecd/blob/master/cmd/pipecd/server.go#L106

I have changed terminationGracePeriodSeconds , but this error was not be solved. Maybe this error is due to that pod response stop before pod's status changed to be 'terminating'

@kevin55156
Thank you for your investigation.
From your comments, I feel this looks fine.
LGTM


About num_crops_per_aug argument

Hi, it is not clear to me how the number of crops and transform pipelines work.
Under the directory bash_files/pretrain/imagenet-100/ some scripts have the argument --num_crops_per_aug 1 1  (for example byol.sh), while other have --num_crops_per_aug 2 (like simclr.sh).
Can someone clarify to me why some methods have 2 pipelines but only 1 crop per pipeline, while others have a single pipeline with 2 crops? What would be the difference here?
Thank you in advance!

Hey @juanigp, SimCLR uses a symmetric augmentation pipeline, i.e. both crops are generated from exactly the same augmentations whereas BYOL (and other works that followed later which borrow from it) use an asymmetric augmentation pipeline (crop 1 has a different random probability for gaussian blur and solarization then crop 2). Our --num_crops_per_aug parameter controls how many crops each augmentation pipeline will produce. This adds better flexibility to supporting asymmetric pipelines (line BYOL) and multi-crop (like SWaV). I suggest you check both papers where they detail the augmentations.

Hi @vturrisi thank you for the info, very much appreciated!


Example AV1 Gstreamer pipeline for Broadcast Box

May I have an example Gstreamer pipeline for my Broadcast Box server?

Version 116.0.5845.187 (Official Build) (x86_64)

Working!  If encoding is started first it doesn't. It seems that gstreamer sends a keyframe at stream start and then no keyframes anymore. Then the WHEP player doesn't know the codec. I only see this issue on AV1. VP9,VP8 working fine.


Ctrl+Click rustc error message no longer jumps to the location

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.70.0
OS Version: Windows 11 Build 22000.795

Steps to Reproduce:
When running cargo check task in VSCode inside a Rust project, if there is error, the integrated terminal would output something like this:
Checking <redacted> v0.1.0 (<redacted>)
error: expected one of `(` or `<`, found `foobar`
  --> <redacted>\src\main.rs:82:42
   |
82 | 123
   | ^^^ expected item

error: could not compile `<redacted>` due to previous error

The error location (<redacted>\src\main.rs:82:42) is linkable to the source code location when Ctrl+Click in previous versions (such as 1.69.2). In VSCode 1.70.0, it always jumps to the first line of the file.

Open a Rust project with errors in main.rs.
Run task cargo check.
Ctrl+Click the error location.

/duplicate https://github.com/microsoft/vscode/issues/155532

I'm seeing this issue as well in TypeScript code after upgrading to 1.70.0, but my path's do not contain spaces, so I'm not sure that this is really a duplicate of https://github.com/microsoft/vscode/issues/155532


Allow missing field mappings in conf

With the introduction of dynamic mapping, it makes sense to no define any field_mappings.
Omitting it in the conf should be allowed.

Closes in #1303


Changed wind direction icon to mdi:compass

Hrmm.. I agree with this pull.. but I'm in the middle of rototilling the code base in the config_flow branch to change this to a config_flow..
hrmm.. I think I'll merge and then duplicate it manually in the branch..


Koa v2, appium-logger, validation tweaks & labeling

Using Koa v2 (light weight, cleans up our custom routing, async advocate, more info: https://github.com/koajs/koa/issues/533)
Replaced custom logger function with appium-logger
Removed validate+route logic (koa-router takes care of the routing, payloadValidator takes care of validation)
Implemented labeling (@triager please label: one, two, three.)
Fixed payload validator - special characters were freaking it out.

hmm, you should probably add .DS_Store files to your global .gitignore

couple small comments, rest LGTM!


Extension "sp_executesql to SQL" is not working

Azure Data Studio Version: 1.10.0
Extension :- sp_executesql to SQL
Versions :- 0.0.1 and 1.0.0
Steps to Reproduce

Add Extension - sp_executesql to SQL
Pasted SQL that contains sp_executesql
Selected SQL
Access Help/F1
Entered convert into Help text field
Chose "sp_executesql to sql: convert" from menu
No conversion

Documented Here :-
Azure Data Studio – Extension – “sqlops-spexecutesql-to-sql”
https://learningintheopen.org/2019/08/24/azure-data-studio-extension-sqlops-spexecutesql-to-sql/

Hi @DanielAdeniji
I'm the author of "sp_executesql to SQL" extension.
Can you please flow up this issue in the extension repo?
https://github.com/PejmanNik/sqlops-spexecutesql-to-sql
Please create an issue there with your SQL code.
Thank you.

Thanks for following up @PejmanNik  - I'll close this one out then so you can track it in your extension repo.

Pejman :-
I just checked https://github.com/PejmanNik/sqlops-spexecutesql-to-sql.
Currently, that GitHub repository does not have an Issue Tab.
And, it does not appear that there is an avenue to create bugs.
Please advice if otherwise.
Daniel

It is strange @DanielAdeniji .
You can use this link to create an issue:
https://github.com/PejmanNik/sqlops-spexecutesql-to-sql/issues/new

Pejman :-
Sorry about that.
Thanks so very much.
Daniel

You're welcome @DanielAdeniji  👍


Irrelevant exception message on fonts

I’ve got an error however the font file is here!

Font "/var/www/files/fonts/a1bdc0c2/chaletbookbold.ttf" not found.

So I looked for the thrown exception.
In /vendor/psliwa/php-pdf/lib/PHPPdf/Core/Engine/ZF/Font.php, getResourceByStyle
catch(\ZendPdf\Exception\ExceptionInterface $e)
{
    throw InvalidResourceException::invalidFontException($this->fontResources[$style], $e);
}

Then I removed this try-catch and TADA

This font cannot be embedded in the PDF document. If you would like to use it anyway, you must pass \ZendPdf\Font::EMBED_SUPPRESS_EMBED_EXCEPTION in the $options parameter of the font constructor.

The real error is not that font file does not exist but that the TTF file is restricted. So I suggest that you should not catch Zend/Exception that way, because you can look for a long time when your error message is irrelevant.
You should change
public static function invalidFontException($fontData, \Exception $previous = null)
{
    return new self(sprintf('Font "%s" not found.', $fontData), 0, $previous);
}

to:
public static function invalidFontException($fontData, \Exception $previous = null)
{
    return new self(sprintf('Font "%s": '.$previous->getMessage(), $fontData), 0, $previous);
}

Because Zend will tell you if the font file is really not found.

Thanks for investigation. I will take care of this asap.

I can PR this for you if you prefer. 🙂


Missing app version

Because of the new traceability stuff we need a way to get the version of the app inside the app.
@ldcouto are you up for this?

Electron does this. The following snippet returns the version as specified in package.json.
import { IntoCpsApp } from "./IntoCpsApp";

/// ...

1b6d0b6b8b0c.png)
IntoCpsApp.getInstance().app.getVersion();

Important Note:
The value is read from package.json. It will not work properly if you launch the app with npm start -- it returns the electron version instead.
It works for the packaged app fine. For development mode, you can launch the app with ./node_modules/.bin/electron . and it will also work.

Great, thanks!


Add PHPDoc to Concrete\Core\Search\Field

Just a couple of notes:

we really need to have an empty ManagerInterface interface? What about adding some methods of the Manager class?
we really need to have an almost empty GroupInterface interface? What about adding some methods of the Group class?

I know that there are BC issues by adding new methods to interfaces, but having totally useless interfaces is really counter-productive

Having well written PHPDocs in the core is incredibly useful.

Agreed @mlocati – at the very least we should add the get* methods to those interfaces.

at the very least we should add the get* methods to those interfaces.

See #5688


Thank you for this library

I was using fuse.js but 530 wasn't working for my timezone. See this in action, still using fuse.js but it worked with fzf
README must be updated to include options field as I got TS error when setting up new Fzf:

expected 2 parameters, given only 1

Thank you 🎉

Hey Akshay, the second parameter is required in your case as timezonesList is not a plain list of strings. To understand how you could use FZF correctly, please have a look at https://fzf.netlify.app/docs/latest#usage-non-string-list

ah, makes sense. i think it should be added to the readme as well with little note as it's a more common approach to use objects than arrays :)

to use objects than arrays

I suppose you meant objects than strings.
The readme only contains a "quick look", we have dedicated website for documentation as mentioned above: https://fzf.netlify.app.

I suppose you meant to use objects than strings.

yes!

The readme only contains a "quick look"

yeah, but most people won't visit the website as they can see the code right in front of their eyes so it helps having options
bcz i did exactly that. copy-pasted from the readme as it's simple 3-steps so no need to visit the docs but then TS yelled.
definitely not that important that's why i didn't open an issue :)


Bumping up the Cloud libraries bom and spanner java client libraries version

Bumping up the libraries bom and spanner java client libraries version.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java_GCP_IO_Direct PreCommit

Run Java_GCP_IO_Direct PreCommit

Thanks, BOM dep update also need to sync with a few dependencies defined in BeamModulePlugin. Usually we update BOM dep once per release, the last time was #25790

Linkage check tool log -- https://gist.github.com/darshan-sj/43f001e8d51dbc0297bc0b754abff8b5

done

Run Java_Spark3_Versions PreCommit

Run Java PreCommit

Run Java PreCommit

Run Java_GCP_IO_Direct PreCommit

Run Java_GCP_IO_Direct PreCommit

Run Java_PVR_Flink_Batch PreCommit

Run Java_Spark3_Versions PreCommit

Run Java_GCP_IO_Direct PreCommit

Run Java_Spark3_Versions PreCommit

4 skipped and 62 successful checks.


[CHANGE ME] Re-generated Logging to pick up changes in the API or client library generator.

This PR was generated using Autosynth. :rainbow:
Log from Synthtool
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/Logging/synth.py.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:75b38a3b073a7b243545f2332463096624c802bb1e56b8cb6f22ba1ecd325fa9
Status: Image is up to date for googleapis/artman:latest
synthtool > Cloning googleapis.
synthtool > Running generator for google/logging/artman_logging.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/php/google-cloud-logging-v2.
synthtool > Replaced "'serviceAddress' =>" in src/V2/Gapic/ConfigServiceV2GapicClient.php.
synthtool > Replaced "'serviceAddress' =>" in src/V2/Gapic/MetricsServiceV2GapicClient.php.
synthtool > Replaced "'serviceAddress' =>" in src/V2/Gapic/LoggingServiceV2GapicClient.php.
synthtool > Replaced '@type string \\$serviceAddress\\n\\s+\\*\\s+The address' in src/V2/Gapic/ConfigServiceV2GapicClient.php.
synthtool > Replaced '@type string \\$serviceAddress\\n\\s+\\*\\s+The address' in src/V2/Gapic/MetricsServiceV2GapicClient.php.
synthtool > Replaced '@type string \\$serviceAddress\\n\\s+\\*\\s+The address' in src/V2/Gapic/LoggingServiceV2GapicClient.php.
synthtool > Replaced '\\$transportConfig, and any \\$serviceAddress' in src/V2/Gapic/ConfigServiceV2GapicClient.php.
synthtool > Replaced '\\$transportConfig, and any \\$serviceAddress' in src/V2/Gapic/MetricsServiceV2GapicClient.php.
synthtool > Replaced '\\$transportConfig, and any \\$serviceAddress' in src/V2/Gapic/LoggingServiceV2GapicClient.php.
synthtool > Replaced 'final class' in src/V2/CreateLogMetricRequest.php.
synthtool > Replaced 'final class' in src/V2/CreateSinkRequest.php.
synthtool > Replaced 'final class' in src/V2/ListSinksResponse.php.
synthtool > Replaced 'final class' in src/V2/ListLogsResponse.php.
synthtool > Replaced 'final class' in src/V2/UpdateLogMetricRequest.php.
synthtool > Replaced 'final class' in src/V2/LogExclusion.php.
synthtool > Replaced 'final class' in src/V2/DeleteLogRequest.php.
synthtool > Replaced 'final class' in src/V2/ListMonitoredResourceDescriptorsRequest.php.
synthtool > Replaced 'final class' in src/V2/UpdateSinkRequest.php.
synthtool > Replaced 'final class' in src/V2/ListExclusionsResponse.php.
synthtool > Replaced 'final class' in src/V2/WriteLogEntriesRequest.php.
synthtool > Replaced 'final class' in src/V2/ListLogMetricsResponse.php.
synthtool > Replaced 'final class' in src/V2/LogEntryOperation.php.
synthtool > Replaced 'final class' in src/V2/WriteLogEntriesResponse.php.
synthtool > Replaced 'final class' in src/V2/DeleteLogMetricRequest.php.
synthtool > Replaced 'final class' in src/V2/LogEntry.php.
synthtool > Replaced 'final class' in src/V2/CreateExclusionRequest.php.
synthtool > Replaced 'final class' in src/V2/BigQueryOptions.php.
synthtool > Replaced 'final class' in src/V2/ListLogsRequest.php.
synthtool > Replaced 'final class' in src/V2/LogEntrySourceLocation.php.
synthtool > Replaced 'final class' in src/V2/ListExclusionsRequest.php.
synthtool > Replaced 'final class' in src/V2/GetLogMetricRequest.php.
synthtool > Replaced 'final class' in src/V2/UpdateExclusionRequest.php.
synthtool > Replaced 'final class' in src/V2/GetSinkRequest.php.
synthtool > Replaced 'final class' in src/V2/DeleteSinkRequest.php.
synthtool > Replaced 'final class' in src/V2/GetExclusionRequest.php.
synthtool > Replaced 'final class' in src/V2/ListLogMetricsRequest.php.
synthtool > Replaced 'final class' in src/V2/ListSinksRequest.php.
synthtool > Replaced 'final class' in src/V2/ListLogEntriesResponse.php.
synthtool > Replaced 'final class' in src/V2/ListLogEntriesRequest.php.
synthtool > Replaced 'final class' in src/V2/DeleteExclusionRequest.php.
synthtool > Replaced 'final class' in src/V2/WriteLogEntriesPartialErrors.php.
synthtool > Replaced 'final class' in src/V2/ListMonitoredResourceDescriptorsResponse.php.
synthtool > Replaced 'final class' in src/V2/LogMetric.php.
synthtool > Replaced 'final class' in src/V2/LogSink.php.
synthtool > No replacements made in src/V*/**/*.php for pattern public function ([s|g]\w{3,})Unwrapped, maybe replacement is not longer needed?
synthtool > Replaced 'Copyright \\d{4}' in src/V2/Gapic/ConfigServiceV2GapicClient.php.
synthtool > Replaced 'Copyright \\d{4}' in src/V2/Gapic/MetricsServiceV2GapicClient.php.
synthtool > Replaced 'Copyright \\d{4}' in src/V2/Gapic/LoggingServiceV2GapicClient.php.
synthtool > Replaced 'Copyright \\d{4}' in src/V2/ConfigServiceV2Client.php.
synthtool > Replaced 'Copyright \\d{4}' in src/V2/LoggingServiceV2Client.php.
synthtool > Replaced 'Copyright \\d{4}' in src/V2/MetricsServiceV2Client.php.
synthtool > Replaced 'Copyright \\d{4}' in tests/Unit/V2/MetricsServiceV2ClientTest.php.
synthtool > Replaced 'Copyright \\d{4}' in tests/Unit/V2/ConfigServiceV2ClientTest.php.
synthtool > Replaced 'Copyright \\d{4}' in tests/Unit/V2/LoggingServiceV2ClientTest.php.
synthtool > Replaced 'Copyright \\d{4}' in tests/System/V2/LoggingServiceV2SmokeTest.php.
synthtool > Cleaned up 1 temporary directories.
synthtool > Wrote metadata to synth.metadata.

added types outside of versioned folders. (breaks coverage)


Example: get "lorri daemon is not running" after "lorri watch"

Expected behaviour: the direnv integration works without errors after invoking lorri watch. Observed behaviour: direnv integration complains that lorri daemon is not running.
Steps to reproduce (on master or rolling-release):
lorri$ nix-build
lorri$ cd example
lorri/example$ ../result/bin/lorri watch
<successfully builds and rebuilds the project as expected on file changes>

In another terminal (with .envrc changed to point to ../result/bin/lorri):
lorri/example$ # do anything
Info: the lorri daemon is not running. Loading a cached environment.
direnv: export <...>

Expected behaviour: the direnv integration works without errors after invoking lorri watch. Observed behaviour: direnv integration complains that lorri daemon is not running.

The message is an Info-level message to tell the user that the daemon is not running, and thus files won’t be evaluated on-the-fly. We can’t figure that out with lorri watch … maybe it’s time to either deprecate lorri watch or emulate it with the daemon.
Do you think there are any problems with always using the daemon, even for watch? It’s as lightweight as lorri watch, and communicates over one unix socket file. cc @grahamc

Thanks @Profpatsch!
It seems to me that the easiest thing to do to prevent other people running into this issue would be to update the Set up lorri section of the example README to use lorri daemon instead of lorri watch. I'll see if that works today and put up a PR in case things "just work".
@grahamc told me that lorri watch has at least one feature that lorri daemon does not currently have, namely the --once flag which performs one build and then exits. I'll also look into how that could be ported over to lorri daemon so we can get closer to deprecating lorri watch.
Does that sound reasonable?

@grahamc told me that lorri watch has at least one feature that lorri daemon does not currently have, namely the --once flag which performs one build and then exits. I'll also look into how that could be ported over to lorri daemon so we can get closer to deprecating lorri watch.

Hm, I don’t think it makes sense for the daemon to have a --once flag. Instead, we would have a command that instructs the daemon to only run one build for a project (the daemon would still continue running).

https://github.com/target/lorri/pull/190 is my proposal to move example/README.md from lorri watch to lorri daemon.

Closing, because I believe the issue was addressed.


Feature request: serialize by class

Hi! I really liked this library and just wondered, can we, developers, have slightly improved serialize function, so it can accept a class same way, as deserialize method receives it? Thank you!

Serializing an object is basically just calling JSON.stringify(instance). I have implemented that as follows:
let instance = {
     id: 1,
     name: "Andreas"
}; // test object instance
let jsonString: string = JsonConvert.serializeObject(instance);

Or did you mean something else?

Closing due to inactivity. Will reopen if really an issue.

I meant to serialize and rename all parameters, how they're defined by JsonProperty in the class itself

I don't follow, can you please make an example and explain what you mean? Maybe you mean something similar like #6?

No. For example: we have some class, described to use JsonConvert. It have 3 params: _id, _name, _age and JsonProperty is set up to deserialize them from id, name and age(without underscores) attributes. It's working like a charm. But I want to SERIALIZE them and in output have properties, that I described by JsonProperties(id, name, age), not original ones(_id, _name, _age)

Basically you have this scenario:
JSON:
_id, _name, _age
TypeScript:
id, name, age
The deserialization maps _id to id, _name to name and _age to age.
Am I correct you would like the serialize() function to map id to _id, name to _name and age to _age again, as it was in the JSON?

Exactly!

Ok, I will keep this issue open and put it on my to do list for a future version.

Thank you!

Waiting for this enhancement.

@RAHILKHERA I will work on this after finishing my current project in around 2 weeks!

@andreas-aeschlimann Thanks a lot. Please comment here whenever it is ready.

@RAHILKHERA looking forward to this feature as I was curious about it not being implemented myself... as generally in the java world (being a java dev) this is what I was expecting.  Thanks and look forward to see the update.

Adding my request for this feature as well. Coming from the C# world, this was my expected behavior.

+1

+1
@Messier82, how did you solve this in the meanwhile? I have a fairly large and nested JSON which works great for deserializing into TypeScript objects. But my problem is now I have to save it to saver, and the APIs expect the json keys to have underscores instead of camecase.
please let me know if you used any alternatives to solve it temporarily until this feature is available. This is almost like Jackson libraries in Java world.

I'll close this as this is implemented now. Will publish the NPM package when the other features are included as well.


HHH-16648 Documentation for implementing custom JPQL/HQL functions

I propose a technology for creating custom JPQL/HQL functions (code and documentation).

I'm sorry this takes so long. Your PR is very much appreciated and we will look into it. We are currently busy with other things. I added your PR to my todo list, so be assured that I will review it eventually.

Thank you very much!

Please let me know what needs to be fixed or added in my code for you to accept my pull request.


Math-Magicians:Setup

Under this milestone, I achieved the following:

Created a React application using Create React App (CRA).
Made sure the application run in the browser without errors.
Updated the Readme file

Hi @Tumworobere,
Great job so far 👏 💯 👍. However, there are few issues we need to address before we move on. Please, check the comments below.
Highlights

Follows git workflow.  💯
Codebase looks clean and professional. 💯

Required Changes ♻

[ ] node_modules/ directory should not be present on your remote repository. Kindly remove it from your commit history and add to .gitignore.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
You can also consider:

[ ] Check the comments under the review.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Fixed Kuard and and Envoy proxy links

Fixed Kuard link that was pointing to Envoy proxy page; Added link to Envoy proxy page.

@joonathan thanks for this fix. Before I can merge it you'll have to sign it to make the DCO bot happy.
You can do this by using git commit --amend -s and the force pushing

Thanks for the tip. Signed.


There is not option to getSelected(), add this

hgeGetSelected() returns an object with charts, series, and point indexes


Docs review before 1.0.0 release

It is worth reviewing the docs before 1.0.0 release.

See e.g. https://github.com/jupyter/jupyter_server/pull/287 to document live reloading.

Comments in JupyterLab still points to https://jupyter-notebook.readthedocs.io/en/latest/security.html
This could be replaced with https://jupyter-server.readthedocs.io/en/latest/operators/security.html but the server pages has been logically stripped down to remove notebook specific content.

cf #250 about security upgrades

I see #299 was marked as "contributing" to this, but 1.0.0 is out the door, so is there anything remaining here?

I think @Zsailer said he would pass another round on it, but in the meantime jupyter server 1.0.0 has be cut, so yeah it makes sense to close this one. Doing that now. Thx @vidartf


1.5 Map conversion update

As stated in #535.
Fixed some minor border tweaks, notably:

Cameroonian-Nigerian border
Split Agades (Niger) state
Serbian Southern borders (Kosovo and Macedonia)
Uruguayan border with Brazil
Some cities' locations

Along with some country mappings fixes based on the new formable tags and Guyenne/Gascony fix.
I'm sorry about the reordering of the province mappings file.

I want to give this a quick spot check, but need to do so from home. But I'd like to get it in the next release. It looks like you've addressed some of the regions I was pretty unsure about.

I was looking this over, and the province mapping file is structured completely differently. What were you doing to get everything in a different order and most of the comments dropped?

Ah, that's a bummer. The regions you noted were ones I had a rough time going over, even when I was using both your version and the original version to create the current version. I'd love to see them fixed up.


MySql decimal returns Number instead of String

Environment
Knex version: 0.14.6
Database + version: MySql 5.7.12
OS: OSX
Feature discussion / request
Don't convert decimal to Number automatically.
I have a decimal column with precision 8 and scale 8
await knex.schema.createTable("myTable", t => {
    t.engine("InnoDB");
    t.charset("utf8mb4");
    t.collate("utf8mb4_unicode_ci");
    t.bigIncrements();
    t.decimal('decimal', 8, 8);
});

When I fetch the records decimal is converted to Number:
await knex("myTable").select(knex.raw('*'));
// returns {"id":1, "decimal":0.01}

I expect knex to return string, so that I can use Decimal.js to avoid floating-point operations. Converting decimal to Number automatically defeats the purpose of using decimal in the database layer, and it seems to me this should be the default behavior.
What's the reasoning behind this? Is there any way to turn this conversion off?

Unlikely a knex issue. Knex returns response from driver. So the issue is probably in mysql / mysql2 which ever driver you are using. They both seems to have config flags you can use to return strings instead of Numbers.
https://github.com/mysqljs/mysql
https://github.com/sidorares/node-mysql2/tree/master/documentation
Knex({config}) passes config to the driver.

Knex doesn't touch values that are returned from DB driver (except in some rare cases), so you need to setup mysql driver in a way that you get the data in wanted format.

Yes, thanks. To change that I had to either switch to mysql2 or add:
const config = {
    client: 'mysql',
    connection: {
        database: process.env.DB_NAME,
        host: process.env.DB_HOST,
        user: process.env.DB_USER,
        password: process.env.DB_PASSWORD,
        timezone: "UTC",
        charset: 'utf8',
        supportBigNumbers: true, // this fixes it
        bigNumberStrings: true, // this fixes it
    },
    pool: {
        min: 0,
        max: 1,
    }
};

Yes, thanks. To change that I had to either switch to mysql2 or add:
const config = {
    client: 'mysql',
    connection: {
        //...
        supportBigNumbers: true, // this fixes it
        bigNumberStrings: true, // this fixes it
    },
};

`decimalNumbers: true,`


Feat/config option os metrics enabled

Currently there is no option to disable the metrics collection on config-nodes.
This causes problems on CtlrX core as the app tries to access resources outside of it's sandbox ("/sys/block/")
This new config option allows programatically disabling the DiskMetrics and NetMetrics.

Hi，chris ,we have added a off level to close system metric, you could take a look for this pr :https://github.com/apache/iotdb/pull/11530

Will have a look after my trip, next week. Thanks


Tracking: Ingress bugs

Context: production readiness proposal and plan
The feature “cluster ingress” is identified as incomplete test coverage. As a measure to reach Istio production quality in 0.3, this issue is opened to track the progress of providing sufficient test coverage for the feature in question.
This issue is initially assigned to the working group leads.
Recommendations:

Come up with a test plan for how the feature should be tested. Identify and document (in this issue, or a separate document this issue will link to) test cases and whether each will be covered by an integration test (with some level of mocking and/or starting up selected components in test environment), or end to end test (aka system test, which brings up the whole Istio to exercise the feature; currently live in the istio/istio repo).
If you expect the test plan is complex and controversial, consider having it formally reviewed.
State an ETA in the issue, and assign this to a developer who will execute the test plan.
Execute the test plan as the highest priority for 0.3.
For any bug or defect identified by the new test cases, file a new issue and link it from this one.
Close this issue only when the test plan is completely executed, and all additional issues identified in step 3 are fixed.

If you see a risk of not fixing this issue by end of 0.3, please contact<EMAIL_ADDRESS>as soon as possible. We will also periodically ping the issue for update.

https://github.com/istio/pilot/issues/730
https://github.com/istio/pilot/issues/484
https://github.com/istio/pilot/issues/533
https://github.com/istio/pilot/issues/731
https://github.com/istio/pilot/issues/805
https://github.com/istio/pilot/issues/926
https://github.com/istio/pilot/issues/969
https://github.com/istio/pilot/issues/1021
https://github.com/istio/pilot/issues/1389
https://github.com/istio/pilot/issues/1367
https://github.com/istio/pilot/issues/1234
https://github.com/istio/pilot/issues/1021

just FYI @smawson ^^^

subscribe

Issue moved to istio/istio #1486 via ZenHub


Youtube 10.37.58 not working

Just updated to latest version 1.6.0
And youtube version 10.37.58 doesnt seem to be working. Just letting you know. :)
I wish i coukd gather sime debug log to give more info on this.

I have it and it works. Did you enable it in Xposed Installer & reboot your device?

After 2 reboot it work for me : 1st reboot to install the module, then open the installer -> module and check the app ; 2nd reboot and it's ok !

You can check the module in the Xposed Installer instantly after installing. So one reboot is required in total :)

I have the same issue. Same versions of both apps on CM12.1 M1.
Here's a (filtered) logcat:
10-02 02:20:22.590 I/ActivityManager( 2350): Killing 5271:com.google.android.youtube/u0a116 (adj 9): remove task
10-02 02:20:22.655 I/WindowState( 2350): WIN DEATH: Window{f584786 u0 com.google.android.youtube/com.google.android.apps.youtube.app.WatchWhileActivity}
10-02 02:20:27.295 I/Timeline( 2990): Timeline: Activity_launch_request id:com.google.android.youtube time:425760
10-02 02:20:27.296 I/ActivityManager( 2350): START u0 {act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.google.android.youtube/.app.honeycomb.Shell$HomeActivity (has extras)} from uid 10016 on display 0
10-02 02:20:27.317 V/WindowManager( 2350): addAppToken: AppWindowToken{225e2c74 token=Token{3641ac47 ActivityRecord{2ed4cb86 u0 com.google.android.youtube/.app.honeycomb.Shell$HomeActivity t11043}}} to stack=1 task=11043 at 0
10-02 02:20:27.408 V/WindowManager( 2350): Adding window Window{242d96a u0 Starting com.google.android.youtube} at 11 of 17 (after Window{3f91a48c u0 com.nolanlawson.logcat/com.nolanlawson.logcat.LogcatActivity})
10-02 02:20:27.452 I/ActivityManager( 2350): Start proc 11527:com.google.android.youtube/u0a116 for activity com.google.android.youtube/.app.honeycomb.Shell$HomeActivity
10-02 02:20:27.737 E/YouTube MDX(11527): Bogus value in shared preferences for key MdxServerSelection value , returning default value.
10-02 02:20:27.868 I/dex2oat (11562): /system/bin/dex2oat --runtime-arg -classpath --runtime-arg /system/framework/XposedBridge.jar --instruction-set=arm --instruction-set-features=default --runtime-arg -Xrelocate --boot-image=/system/framework/boot.art --dex-file=/data/data/com.google.android.youtube/cache/ads2030829230.jar --oat-fd=48 --oat-location=/data/data/com.google.android.youtube/cache/ads2030829230.dex --runtime-arg -Xms64m --runtime-arg -Xmx512m
10-02 02:20:27.983 I/Timeline(11527): Timeline: Activity_launch_request id:com.google.android.youtube time:426448
10-02 02:20:27.983 I/ActivityManager( 2350): START u0 {act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x4000000 cmp=com.google.android.youtube/com.google.android.apps.youtube.app.WatchWhileActivity (has extras)} from uid 10116 on display 0
10-02 02:20:27.994 V/WindowManager( 2350): addAppToken: AppWindowToken{6448d79 token=Token{1dd28a40 ActivityRecord{2db0f1c3 u0 com.google.android.youtube/com.google.android.apps.youtube.app.WatchWhileActivity t11043}}} to stack=1 task=11043 at 1
10-02 02:20:27.994 V/WindowManager( 2350): Based on layer: Adding window Window{242d96a u0 Starting com.google.android.youtube} at 11 of 17
10-02 02:20:28.933 V/WindowManager( 2350): Adding window Window{1cdf2ecc u0 com.google.android.youtube/com.google.android.apps.youtube.app.WatchWhileActivity} at 11 of 18 (before Window{242d96a u0 Starting com.google.android.youtube})
10-02 02:20:29.061 E/YouTube (11527): (unknown) Media progress reported outside media playback: NEW
10-02 02:20:29.103 I/ActivityManager( 2350): Displayed com.google.android.youtube/com.google.android.apps.youtube.app.WatchWhileActivity: +1s31ms (total +1s680ms)
10-02 02:20:29.103 I/Timeline( 2350): Timeline: Activity_windows_visible id: ActivityRecord{2db0f1c3 u0 com.google.android.youtube/com.google.android.apps.youtube.app.WatchWhileActivity t11043} time:427568
10-02 02:20:29.162 E/YouTube (11527): (unknown) Media progress reported outside media playback: NEW
10-02 02:20:30.688 W/YouTube (11527): (unknown) In CsiAction [bbv], past event br_r can't be marked
10-02 02:20:30.688 W/YouTube (11527): (unknown) In CsiAction [bbv], past event ol can't be marked
10-02 02:20:31.629 W/YouTube (11527): apps.youtube.app.WatchWhileActivity.a:50629 In CsiAction [knk], past event pl_int can't be marked
10-02 02:20:32.479 V/WindowManager( 2350): Adding window Window{16e5b871 u0 SurfaceView} at 11 of 18 (before Window{1cdf2ecc u0 com.google.android.youtube/com.google.android.apps.youtube.app.WatchWhileActivity})
10-02 02:20:32.714 W/YouTube (11527): (unknown) CsiAction [knk] already ticked asiss. Ignored.
10-02 02:20:32.886 W/YouTube (11527): (unknown) CsiAction [knk] already ticked ari. Ignored.
10-02 02:20:32.886 W/YouTube (11527): (unknown) CsiAction [knk] already ticked vri. Ignored.
10-02 02:20:32.902 W/YouTube (11527): (unknown) CsiAction [knk] already ticked asisrh. Ignored.
10-02 02:20:32.979 W/YouTube (11527): (unknown) CsiAction [knk] already ticked asisfb. Ignored.
10-02 02:20:32.980 W/YouTube (11527): (unknown) CsiAction [knk] already ticked arrh. Ignored.
10-02 02:20:32.980 W/YouTube (11527): (unknown) CsiAction [knk] already ticked asisr. Ignored.
10-02 02:20:32.980 W/YouTube (11527): (unknown) CsiAction [knk] already ticked arfb. Ignored.
10-02 02:20:32.980 W/YouTube (11527): (unknown) CsiAction [knk] already ticked vsiss. Ignored.
10-02 02:20:32.981 W/YouTube (11527): (unknown) CsiAction [knk] already ticked vrrh. Ignored.
10-02 02:20:32.985 W/YouTube (11527): (unknown) CsiAction [knk] already ticked vrfb. Ignored.
10-02 02:20:32.986 W/YouTube (11527): (unknown) CsiAction [knk] already ticked ar40k. Ignored.
10-02 02:20:35.910 W/PackageManager( 2548): Failure retrieving resources for com.google.android.youtube: Resource ID #0x0
10-02 02:20:38.409 D/StatusBar( 2548): Clicked on button -1 for 0|com.google.android.youtube|2|null|10116

And what's the problem? "Not working" is a very poor description of the issue.
Notification is not shown?

Whoops, good point, sorry.  The notification appears, but when the YouTube
app is closed, the video immediately stops playing and the Play/Pause
button on the notification becomes an exclamation point.
Touching the notification to bring YouTube back to the foreground causes
shows the error "There was a problem while playing.  Touch to retry." on
the video playback area.

Aha, that was issue some time ago for not pure music videos so we had to restart playback after we got exception. We will check it.

Are you sure that the bug is not caused by Youtube AdAway module?
Developer has released a buggy version yesterday morning which was causing some problems then released a fix in the evening.

Hmm, I am using AdAway.  I will try updating and/or disabling it.

After disabling the YouTube AdAway module, the problem persists.  That is, same issue with it both enabled and disabled.

I have found out that I am experiencing the same issue since 10.40.xx update. But only on my CM12.1 mobile phone - everything still works on a Samsung KitKat tablet even with 10.40.xx!
I have also never experienced such a bug on 10.37 even on CM12.1! I am 100% sure.
The logcat shows some problem with 'resources'...
All of the above made me think... I wonder if the error isn't caused by using the YouTube app revision for different dpi than the device?
For example 10.40.xx has been out for 480dpi only as for now:
http://www.apkmirror.com/apk/google-inc/youtube/youtube-10-40-56-480dpi-android-4-0-3-5-1-1-android-apk-download/
480dpi version should contain high density resources so maybe that is why it does work on my qhd tablet and not on my fullhd mobile?
@thirtythreeforty
Can you test 1.6.3 with 10.38 which has been released as an universal build (ie. for all dpis)?
http://www.apkmirror.com/apk/google-inc/youtube/youtube-10-38-58-android-apk-download/

I just tried v1.6.3 with both v10.38.58 and v10.40.56, and the same thing is happening with both versions.
It's interesting that 10.38.x was never deployed to my device; I was stuck on 10.37.58.

In my case:

10.37.58(nodpi) is working great on both devices
10.38.58(nodpi) is working great on both devices
10.39.54(nodpi) is working great on both devices
10.39.54(480dpi) is working great on both devices (so it's probably not dpi revision issue...).
10.40.56(480dpi) is working on tablet and is not working on phone.

I wonder if the 10.40.56 YT app isn't buggy in some way itself for some screens/devices - it may be dump from Marshmalllow build for specific Nexus device only or something... I will be able to confirm that after a next update...
But I still do not experience ANY errors when using ANY 10.37-10.39 build on ANY of my devices.

Esgie try calling the method that enables audio (Request audio permission from Android) via the first class. The method can be easily found because it uses execute or executor (Forgot which). I have no idea if it will help, but it is worth a shot.

My build environment is "out" again at least since tomorrow...
I will build tomorrow and post here.
However I still CANNOT reproduce the issue for 10.37 - 10.39!!!

I have added callMethod in the first hook with method names as following (for YouTubes 10.36 - 10.40):
[CODE]public static final String[] METHOD_1_2 = { "K", "L", "P", "Q", "Q" };[/CODE]
This ain't fixed my issues with playback using 10.40.xx on my mobile running CM12.1, The same YouTube app also stull runs well on KitKat tablet.
However it may provide a fix for your problems with previous versions.
@thirtythreeforty can you test this build? I have added the code that @iHelp101 was talking about.
Download link: https://userscloud.com/0sx3qly1m56k

I also need help with this. It simply wont work i downloaded xposed aand downloaded the latest version of YoutubeBackgroundPlayback. I rebooted like it told me to than i went to youtube opened a video and nothing showed no notificiation. Than i pressed the home button and it paused it please help.

Hm, I seem to be having issues with 10.37.58 as well it seems, although it seems to be limited to only one device (That being the one I just got back from repair running SlimKat), which is absolutely strange, because when I had issues with this before, the latest YouTubeBackgroundPlayback version fixed all that.
It's not that it doesn't work at all, some videos do play in the background (It seems to be almost at random), but other songs will produce loud clicks (Usually on app transitions or screen power on/off) or just a really quiet empty noise (Kinda like a cassette tape when there's no recording on it) when the screen is on at all. I've had the YouTube app crash, or even have SystemUI completely crash. I've gone as far as disabling all my other Xposed Modules (Including AdAway and GravityBox, as well as RootCloak), and it doesn't seem to fix that issue.
I'll try @esgie's test build and see if it works though, who knows what'll happen with this phone that's been nothing but surprises.

@esgie thanks for the test build.  Unfortunately, it does not help; it
exhibits the exact same behavior.
One odd thing that I have noticed (with the stable build) is that very
occasionally, a video will go to background successfully, with audio
playing and proper pause/skip controls.  However, jumping back to the movie
causes it to stop.  Sorry I can't be more specific, but perhaps it will
help.

@esgie
https://github.com/pylerSM/YouTubeBackgroundPlayback/issues/76

@esgie my bad! Your test build does indeed work with YouTube v10.38.58; I had made the above comment with v10.40.  Switching back to the older app version works.
Glad you're on the right track -- or at least it seems like it.

@thirtythreeforty new thread - https://github.com/pylerSM/YouTubeBackgroundPlayback/issues/76

@esgie - The latest version of your app doesn't work in 10.40.58
I see the ! button

Class 1: kys -> kyr
Class 2 also changed, but I have not found it yet.


No module named gym.envs.registration

Hello, first thanks for this repo. I'm trying to install gym-vrep everything is OK untill I try run ipython init.py on terminal it show me an error about Importing gym error. How does this library works ? When I try it on pycharm it shows no error but nothing happens. Help me please.
Here is the error:

ImportError                               Traceback (most recent call last)
/home/supreme/gym-vrep/gym_vrep/init.py in ()
1 import logging
----> 2 from gym.envs.registration import register
3
4 logger = logging.getLogger(name)
5 register(
ImportError: No module named gym.envs.registration

Hello!
I have never encountered with such error. But it seems that your python env cannot find gym library. Try this:

open ipython
import gym
import gym_vrep
If this didn't work, you should check if gym is installed in pip.

Also, you could check this repo https://github.com/Souphis/mobile_robot_rl. In this project I used gym-vrep.

Hello, I have the same problem, but I don't use gym_ vrep library, so this problem may not be related to the library, but to the gym version or something others.
The gym version on my computer is 0.10.5, but the project I use is multi-agent particle environment which has dependency on the gym version is also 0.10.5.

@hexbo Ok, thanks for pointing this, I will update the requirements and maybe this will solve this


FBX Scene Loading Failed : 'File is corrupted Cube.fbx'

https://epicgamesext.github.io/BlenderTools/send2ue/introduction/quickstart.html
"Now lets run through a basic example with the default cube in your Blender scene."
"A simple test we can try in Blender to make sure this is working is move our cube to our Export collection."
"You should now see the cube in unreal under /untitled category/untitled asset/Cube."
But I see error -
FBX Scene Loading Failed : 'File is corrupted Cube.fbx'
Can't detect import type. No mesh is found or animation track.
Please help!
Unreal 5.3.2
Blender 4.0

Here's the latest temporary fixed version https://github.com/EpicGamesExt/BlenderTools/issues/729#issuecomment-2026320080. This was fixed a while ago.

Does this mean, this issue and another duplicate in the issue list can be closed?

Does this mean, this issue and another duplicate in the issue list can be closed? Duplicate? #711

Yeah I guess so. The fix has been merged already.

@JoshQuake The fix does not work, or does not work anymore.
Blender 4.1.1, still getting the same errors in unreal engine. This issue is still relevant I think


Tracking: Improve SidebarPinnedModel with support for child items

This issue will track the pre-requisite work done in order to allow manipulation of child sidebar items.

[ ] Change default item addition in the model to add root items and their respective default children (if applicable)
[ ] Support getting root items only
[ ] Support getting child items from a specified header item of a certain type
[ ] Update all methods to accept INavigationControlItem as the argument rather than a path string
[ ] Make all overloads of IndexOfItem() return an index specific to the root node's collection of children
[ ] Ensure each INavigationControlItem-derived model doesn't have any references to Windows.UI.*
[ ] Ensure the changes are compatible with the existing JSON
[ ] Get changes approved by original implementor (probably @tsvietOK)

More work items coming soon

Closing this because I decided to use a different method altogether that requires less of this work.

Closing this because I decided to use a different method altogether that requires less of this work.


neovim-qt: wrap in separate derivation

Motivation for this change
This makes it possible to swap out the (wrapped) neovim without
recompiling neovim-qt. In particular, the user can use neovim.override
to configure their neovim and then use that same configuration for
neovim-qt, without having to give up binary caching.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Someone should make sure I didn't mess up the darwin wrapper @NixOS/darwin-maintainers. You can test ist by building and executing
neovim-qt.override {
  neovim = neovim.override {
    configure = {
      customRC = ''
        normal iIt worked!
      '';
    };
  };
}

Which should then start neovim-qt and show the text "It worked!".

CC @peterhoeg

@GrahamcOfBorg build neovim-qt


Rename ~English.ext to ~English

It seems that all of the natural languages don't have a file extension, except for English, so I renamed it.

I left this because there is an esolang called English that uses the file extension.

@MrBrain295 Yes, but all of the natural languages start with a tilde.

@MrBrain295 @calgary34 means that English is different from ~English

I left this because there is an esolang called English that uses the file extension.

Not sure if there would be a conflict:

~English
English.ext


Update cycles_management_services.mdx

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[ ] You have registered new documents to /sidebars.js if any.

@letmejustputthishere Could you fix the broken links in this PR so it can be merged?

Closing in favor of https://github.com/dfinity/portal/pull/2915


Fix invariant violation by showPager

If showPager fluxes between true and false, having var pagingContent = "" (which is equal to var pagingContent = <span />;) results in invariant violation (at least in Chrome). Change it to <tbody /> solves all problems for me.

Just linking this to issue #221 as it looks to be related.

Thanks!


validate compiler options in multicompiler mode

What kind of change does this PR introduce?
feature
closes #14843

Did you add tests for your changes?
yes

Does this PR introduce a breaking change?
no

What needs to be documented once your changes are merged?
nothing

@alexander-akait what do you think about this PR? should we somehow define unique name in this case?

@vankop I think validation of the uniqueName make sense too, because using the same can be problems too, for better dx let's generate a new unique name (just add a number from 0) and output a warning

@alexander-akait fixed. As I understand cache does not work correctly with same name, so name always should be unique

fixed @alexander-akait

@vankop I made some inprovement of your idea

avoid extra loops (better perf), we already apply webpack default options, so for multi compiler mode we can just pass index of compiler and make default name using this index
by default we use name and mode for cache names, so let's respect if for multi compiler mode (i.e. when both configuration doesn't have name, we we use name + mode + compiler_index)
now we output the warning if you have name: "default" for both both compilers (test cases added)
if you have node API for webpack you don't need to apply new default, we will do it under the hood (better DX)

Also always output warnings when you explicitly set the same cache names

We need fix our tests and do not run cache test for basic test, I will tr to fix it tomorrow or you can do it, thank you


Prevent accidental drags by adding drag distance threshold

Prevents accidentally dragging nodes by 1 or 2 pixels by adding a threshold of 10 pixels that must be crossed to start dragging in the 2D & 3D editor. This is only for select mode. This behavior is present in other software like Inkscape and Blender.
Addresses similar need to this proposal https://github.com/godotengine/godot-proposals/issues/431, suggested in this comment https://github.com/godotengine/godot-proposals/issues/431#issuecomment-583018035
https://user-images.githubusercontent.com/12120644/120829123-229e5700-c52b-11eb-8593-a0e759b533f2.mp4

Thanks!

Is it possible to expose the threshold in the Editor settings? I know this is definitely a value I would like to adjust.

Is it possible to expose the threshold in the Editor settings? I know this is definitely a value I would like to adjust.

Yes, but I advised foxydeveloper to keep the value hardcoded as there's little reason to change the value if it's well-chosen.

I wish people would stop citing Blender as usability reference. Blender is really not an inspiration when it comes to user friendly UI. It never has been and it is still miles behind other DCC software
I agree if the setting is good it's less of an issue. However 10 pixels seems like a of unresponsiveness. This video above looks like too much for me.
Since you mentioned Blender: I could measure only 8 pixels, however even after 3 pixels you already get a clear indicator with a different cursor to let the user know that they are "doing something". After 8 pixel the outline for the object changes as well, which is another clear indicator "now you are moving".
If those things are not there those 8 pixels Blender has would feel like even more. We don't have those indicators yet as it seems.
Add to that that some people more their hands more, other less precise. The weight of the mouse. With a pen device it's different again ...


fix reload delay

fix reload delay. nil.to_i evaluates to 0 so something like
So something like this, the string on the right hand side is never evaluated because 0 is truthy.
return nil.to_i || 'never going to evaluate this side of the or block'

┆Issue is synchronized with this Asana task by Unito

@lukew3  FYI. This started out to fix that config singleton but then I wanted to be sure that we're caching correctly, which I had to sort of unify the code paths from the page load and async jobs.


Unexpected props drop

Hi @heyymarco, thanks for the CSS props merging attempts.
Minimal example to reproduce
node test.cjs
const postcss = require('postcss')
const postcssMerge = require('postcss-merge-rules-plus').default
const css = `
html { }
html {
  cursor: default;
  line-height: 1.5;
  -moz-tab-size: 4;
    -o-tab-size: 4;
       tab-size: 4;
  -webkit-tap-highlight-color: transparent;
  -ms-text-size-adjust: 100%;
  -webkit-text-size-adjust: 100%;
  word-break: break-word;
}`
postcss([postcssMerge()]).
  process(css, { from: undefined }).
  then(({ css }) => console.log(css))

I have the next console output:
html {
  cursor: default;
  -moz-tab-size: 4;
       tab-size: 4;
  -ms-text-size-adjust: 100%;
  word-break: break-word }

Please check why the rest of the CSS props were droppped (like line-height, or -o-tab-size).
postcss: 8.2.8
postcss-merge-rules-plus: 1.0.1
node --version: v14.16.0

Fixed. Please update to version 1.0.2.
Sorry for the late response.
I was a bit busy lately.
It seem the postcss has a bug on forEach() api. It sometimes jumps unexpectedly if the collection changes during iteration.
To workaround this problem i snapshot the props into array before iterating.
Thank you for reporting the bug.


RAC-5619: add SOL/VNC into Docker Post Test

Background:
in CI pipeline, SOL/VNC logs were not available in post-test stage.
this PR will add the logs .
Test:

http://10..59.1:8080/job/MasterCI_VNC/
OVA-Post-Test enhancement will come soon,

:+1:

👍


Refactor History Save to User Service

This commit moves user saving from matching service to user service.

To save history, POST to http://localhost:3003/api/hist/save with body questionId,difficulty, attempt.

Checked on Docker and SQL.


Proptypes Error

Hi,
using this component in my app generate proptypes error. even testing on https://runkit.com/npm/react-signup-login-component generates same error

Hi @ahsan7786, I've just updated the package to the version 1.0.3. It's just a clean up to force npm to update their files. Looks like it's working fine now, can you test it again?
About the Runkit, now it's giving "Unexpected token import" which makes sense based on their premise Try any Node.js package right in your browser. The es6 syntax (import/export) is not available in node (commonjs). Makes sense?

@akiokio  i will check and update you in this regard :)

its working fine now @akiokio

great! I'll the issue 👍


Build hydra-battles packages with Dune.

On top of #134362.

Update hydra-battles to 0.4 and split it into separate packages, corresponding to the opam / dune packages (hydra-battles and addition-chains).
Init gaia at 1.12.

Motivation for this change

Add more tests of the fixed useDune2 support in mkCoqDerivation.
Simplify Coq Nix Toolbox setup at https://github.com/coq-community/hydra-battles by upstreaming some overlays.

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

cc @CohenCyril @vbgl @siraben

Tests running at https://github.com/coq-community/coq-nix-toolbox/pull/58

Result of nixpkgs-review pr 134413 run on x86_64-darwin 1

6 packages built:

coqPackages.addition-chains
coqPackages.coqeal
coqPackages.gaia
coqPackages.goedel
coqPackages.hydra-battles
coqPackages.multinomials

Thanks!


setSiteExternalSharingCapability doesn't work

Category

[ ] Question
[ x] Bug
[ ] Enhancement

Expected or Desired Behavior
External Sharing for the newly provisioned group should get disabled
Observed Behavior
Not working as expected.
Steps to Reproduce
Applied this { "verb": "setSiteExternalSharingCapability", "capability": "Disabled"} as part of the site script but Get-SPOSITE command suggests that it didn't work (see screenshot)
.
Thanks for your contribution! Sharing is caring.

Some other observations: By doing this SharePoint won't allow external sharing, users will receive an error but since these are modern sites, owner could go to conversation and add a guest users to the site which ultimately give access to the external user for the whole site.

@sohailmerchant have you seen this thread? https://techcommunity.microsoft.com/t5/Office-365-Groups/Disable-external-sharing-on-all-O365-Groups-with-one-single/td-p/107316
It seems you are trying to disable external sharing on specific groups which does not seem possible right now. It's a tenant level setting which needs to be set for all groups.
Presently, the setSiteExternalSharingCapability only seems to be valid for Communication sites for which it is working as expected. Perhaps this needs to clarified in the docs.

Hi @LauraKokkarinen, that's an excellent blog! I was able to disable adding guests on the group with AllowToAddGuests=false using the Graph API even if external sharing was enabled at the tenant level.
But it still doesn't work for me if I try to do the same using SiteDesigns on an Office 365 Group connected site (with the setSiteExternalSharingCapability action: https://github.com/vman/Site-Designs/blob/master/Office365.SiteDesigns.Deployment/SiteScripts/site-script-externalSharing.json)
I am still able to go to outlook (Group conversations) and add guests to the group. This doesn't happen when I disable guests from the Graph API.
The action works perfectly for a Communication site though.

Hi @vman!
setSiteExternalSharingCapability only disables external sharing for the SharePoint site. It doesn't disable external sharing for the Office 365 group. Those are two separate things and that is why you need to do both of those actions if you want to completely disable external sharing for a modern team site: set AllowToAddGuests to false for the underlying group via Microsoft Graph AND set the site SharingCapability to disabled for the SharePoint site using a site design. If you only disable external sharing for the group, users can still be given access to the SharePoint site (e.g. via the site permissions page) even though you can't add them to the group (e.g. via Outlook), and vica versa.
setSiteExternalSharingCapability alone is enough for for a communication site because it doesn't have an Office 365 group attached to it.
Laura

Yup that makes sense. So from a pure Site Designs point of view, the setSiteExternalSharingCapability action is not enough to completely disable external sharing for group (including the SP site behind it)
So we just need better documentation around this or the setSiteExternalSharingCapability action's implementation needs to be changed so that it disables external sharing on the group as well as on the site behind it.
It doesn't make much sense to have to extend the site design and call a Flow/Azure Function just to disable external sharing on the SP site.


Dropdown onChange : Text acces ?

Hey ! I'm using Dropdown with my data referential. It's working but I really need to store inside my JSON the 'data refs code' and the 'data refs value'.
ex:  code : '0032'  , value : 'New York'.
code : '0033',  value :'Los Angeles'.
My options are like  :
data : {
key : dataRef.code,
text : dataRef.textLabel,
value: dataRef.code
}
When I use onChange(e,d), I only receive the 'value' from my options data. Actually i really need text AND value  to send to my service.
Is it possible ? Thanks.

The onChange callback is the change handler for the Dropdown so it only receives the value of the Dropdown.  Compare this to an Input's onChange.
To get the text, find the option with the matching value.
Dupes:
https://github.com/Semantic-Org/Semantic-UI-React/issues/1368
https://github.com/Semantic-Org/Semantic-UI-React/issues/912#issuecomment-262356714
https://github.com/Semantic-Org/Semantic-UI-React/issues/754


how to train each block?

@liujianzhao6328057 Which example do you want to run?

i want to run the code of "student becoming the master". i am not sure about my understanding.  According to my understanding , each block(encoder part and decoder part ) should be trained to amalgamate  the knowledge of the two teachers . However , i cannot find the corresponding code in the example . Even though the phase is set to "block", i can only train the block in the decoder part  .
so i want to know how to train the encoder part block by block ?
Thank you and best wishes.

Our proposed method will just remove decoder blocks.  Since the encoder part is only a feature extractor, and the decoder part is expected to “project” or transfer the learned features into each task domain.
Of course, you can try to realize it, but I wonder if branching out in encoder part makes sense.

However, in " 5.1.3 Branch Out" ,you say that "after training the N blocks of TargetNet using the loss function (6),we can acquire the final losses for each block ". The total block number is N , and the decoder is in the range of (N/2,N). since that you don't train the encoder block ,how can you get the loss of encoder blocks?  It is confusing .

You're very cautious. In fact, the original code (quite not elegant) is in TensorFlow, where encoder part can also be trained block by block. Since we set N/2 < n ≤ N when choosing where to branch out in the paper, the results of training encoder part have no influence. Thus in this PyTorch implementation, you can't train the encoder part easily for now. Maybe you can realize it and make a pull request :) .

But..if i don't train the encoder blocks , where can i get the pre-trained weight of the encoder part ?By the way ,when will the tf implementation be released?Thanks.

You can set "indices" to be [0,0] to train all blocks in the encoder part. The TF implementation has no open source plans yet, but the method of training encoder block by block is same.

Mimic teachers.
    for i in range(len(self.indices)):
        out_idx = self.indices[i]
        output = decoder_features[out_idx]
        output = output * self.student_adaptors_list[i][out_idx](F.avg_pool2d(output, output.shape[2:3]))
        for j in range(out_idx, 5):
            **output** = self.student_b_decoders_list[i][j](
                output,
                decoder_indices[j],
                decoder_shapes[j]
            )
        outputs = output if outputs is None else torch.cat((outputs, output), dim=1)

    return outputs

if i set inducies[0,0] ,the output is still from  self.student_b_decoders_list not from encoder .

Yes. As we demonstrated in 5.1.2, we replace the features in teacher model with that in student model.
output = decoder_features[out_idx] Here the output is from encoder part.
output = output * self.student_adaptors_list[i][out_idx](F.avg_pool2d(output, output.shape[2:3]))  Here is the Channel-Coding part.
The inner loop makes the student data stream through the left decoder blocks in teacher model, which equals to the "replace" in figure 2 to get L_depth and L_seg.

Hi @Ssssseason , thank you for sharing such an awesome project. I also have some questions about the encoder part, and I sincerely hope you can help me solve them.
Do you have the S / D -channel coding part  (adaptor) on blocks from the encoders? Like the variable down1, down2, ..., down in task_branch.
In your code, you only apply them on the output of decoders, as in decoder_features = [down5, up5, up4, up3, up2] .
It makes me confused that you say  "output = decoder_features[out_idx] Here the output is from encoder part."
However, from your code, we know that all these variables (up5, up4) are from the decoders.
        up5 = self.student_decoders[0](down5, indices_5, unpool_shape5)         up4 = self.student_decoders[1](up5, indices_4, unpool_shape4)         up3 = self.student_decoders[2](up4, indices_3, unpool_shape3)         up2 = self.student_decoders[3](up3, indices_2, unpool_shape2)         up1 = self.student_decoders[4](up2, indices_1, unpool_shape1)
Since Figure 2 is for the "knowledge amalgamation module", not for the branch out, and you didn't say that n > N/2 here. Besides, in Figure 1, you set the knowledge amalgamation at the very beginning of the network, which makes me think that you also apply it on the encoder. I guess my question is similar as @liujianzhao6328057 .
I have this question because in the previous work, "layerwise amalgamation", you apply the knowledge amalgamation module (amal block) on every block, and all these blocks belong to the feature extractor.  To me, it means that you have guidance on every block of the network, while previous work like KD and MTL only have guidance on the output.
While in this code, you only apply knowledge amalgamation to the decoder part. Thus I cannot find how you use the teacher's knowledge to "guide" the encoder.

Hi @Ssssseason , thank you for sharing such an awesome project. I also have some questions about the encoder part, and I sincerely hope you can help me solve them.
Do you have the S / D -channel coding part  (adaptor) on blocks from the encoders? Like the variable down1, down2, ..., down in task_branch.
In your code, you only apply them on the output of decoders, as in decoder_features = [down5, up5, up4, up3, up2] .
It makes me confused that you say  "output = decoder_features[out_idx] Here the output is from encoder part."
However, from your code, we know that all these variables (up5, up4) are from the decoders.
        up5 = self.student_decoders[0](down5, indices_5, unpool_shape5)         up4 = self.student_decoders[1](up5, indices_4, unpool_shape4)         up3 = self.student_decoders[2](up4, indices_3, unpool_shape3)         up2 = self.student_decoders[3](up3, indices_2, unpool_shape2)         up1 = self.student_decoders[4](up2, indices_1, unpool_shape1)
Since Figure 2 is for the "knowledge amalgamation module", not for the branch out, and you didn't say that n > N/2 here. Besides, in Figure 1, you set the knowledge amalgamation at the very beginning of the network, which makes me think that you also apply it on the encoder. I guess my question is similar as @liujianzhao6328057 .
I have this question because in the previous work, "layerwise amalgamation", you apply the knowledge amalgamation module (amal block) on every block, and all these blocks belong to the feature extractor.  To me, it means that you have guidance on every block of the network, while previous work like KD and MTL only have guidance on the output.
While in this code, you only apply knowledge amalgamation to the decoder part. Thus I cannot find how you use the teacher's knowledge to "guide" the encoder.

Do you have the S / D -channel coding part (adaptor) on blocks from the encoders?
In this repo, no.
It makes me confused that you say "output = decoder_features[out_idx] Here the output is from encoder part."
I mean when out_idx is set to 0, decoder_features[out_idx] is the output of the encoder
which makes me think that you also apply it on the encoder
I cannot find how you use the teacher's knowledge to "guide" the encoder of the student
Since we require n > N/2 later in the paper, the loss of each block in encoders is useless. To save training time while keeping similar performance, we can first train all encoders at the same time before training each decoder block (set branch_indices to be [0,0,...] ).
In another way, the teacher's knowledge guides the whole encoder of the student in the first training, instead of guiding each encoder block separately.
 in the previous work, "layerwise amalgamation", you apply the knowledge amalgamation module (amal block) on every block
These two papers don't come from the same author

Do you have the S / D -channel coding part (adaptor) on blocks from the encoders?
In this repo, no.
It makes me confused that you say "output = decoder_features[out_idx] Here the output is from encoder part."
I mean when out_idx is set to 0, decoder_features[out_idx] is the output of the encoder
which makes me think that you also apply it on the encoder
I cannot find how you use the teacher's knowledge to "guide" the encoder of the student
Since we require n > N/2 later in the paper, the loss of each block in encoders is useless. To save training time while keeping similar performance, we could first train all encoders at the same time before training each decoder block (set branch_indices to be [0,0,...] ).
In another way, in this repo the teacher's knowledge guides the whole encoder of the student in the first training, instead of guiding each encoder block separately.
 in the previous work, "layerwise amalgamation", you apply the knowledge amalgamation module (amal block) on every block
These two papers don't come from the same author
Hope these can answer your question

Thank you for the detailed answer, it helps me a lot!


Issue using BreaKmer

Hi-
I mapped my paired end data with BWA and now would like to use BreaKmer, here is the command I'm running:
$ python <path/to/BreaKmer-master/>breakmer.py <path/to/configure/>breakmer.cfg
Traceback (most recent call last):
File "<path/to/>BreaKmer-master/breakmer.py", line 103, in
r = runner(config_d)
File "<path/to/>BreaKmer-master/sv_processor.py", line 100, in init
self.params = params(config_d)
File "<path/to/>BreaKmer-master/utils.py", line 705, in init
self.set_params()
File "<path/to/>BreaKmer-master/utils.py", line 792, in set_params
self.set_targets(self.opts['gene_list'])
File "<path/to/>BreaKmer-master/utils.py", line 769, in set_targets
self.targets[name.upper()].append((chrm,int(bp1),int(bp2),name,feature))
ValueError: invalid literal for int() with base 10: 'M\xa5^\xb9\x97\xa8q\xd7]v1\x9a\xb7\x98\xf7>\xbd'
Can you provide advice for how I should proceed? Thank you!

This was an issue with specifying the bam file as the bed file in the configuration. This has been resolved.