[webview_flutter] onPageFinished never gets called on some Android device (Huwei P20Lite)

I'm working with webview_flutter to load content url in my application.
This is my WebView:
// Enable hybrid composition.
 if (Platform.isAndroid) WebView.platform = SurfaceAndroidWebView();

             Stack(
                children: [
                  VisibilityDetector(
                    key: _webViewKey,
                    onVisibilityChanged: (info) {
                      homeBloc.prizeVisibility = info.visibleFraction > 0.0;
                    },
                    child: WebView(
                      javascriptMode: JavascriptMode.unrestricted,
                      initialUrl: snapshot.data.announceUrl,
                      onWebViewCreated: (WebViewController webViewController) {
                        _webViewController = webViewController;
                        widget.onWebViewCreated?.call(webViewController);
                      },
                      onPageFinished: (String url) {
                        Log.d("${this.runtimeType} onPageFinished: $url");
                        _webViewController.evaluateJavascript('document.body.offsetHeight;').then((value) {
                          setState(() {
                            frontBloc.emit(frontBloc.state.copyWith(isRefreshPrize: false));
                            _prizeWebViewHeight = double.parse(value) + Dimens.size20;
                          });
                        }).catchError((err) {
                          Log.d(err);
                        });
                      },
                    ),
                  ),
                  StreamBuilder(
                    stream: frontBloc.streamRefreshPrize$,
                    builder: (_, snapshot) {
                      return snapshot.hasData && snapshot.data ? shimmer : SizedBox();
                    },
                  )
                ]
              ),

Some Android device never call onPageFinished. In my case is Huwei P20Lite (Android 9).
But on iOS and some Android (Samsung A12, Samsung A31) everything work fine.
and this is my flutter doctor -v:
[✓] Flutter (Channel stable, 2.5.3, on Mac OS X 10.15.7 19H524 darwin-x64, locale en-VN)
    • Flutter version 2.5.3 at /Users/TuanHwing/Desktop/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (6 days ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/TuanHwing/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.0

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] Connected device (2 available)
    • ANE LX2J (mobile) • SCV7N18611000792 • android-arm64  • Android 9 (API 28)
    • Chrome (web)      • chrome           • web-javascript • Google Chrome 94.0.4606.81

pubspec.yaml:
webview_flutter: ^2.1.1

What could be causing this issue? Pls send help

@stuartmorgan
This problem will appear when I show loading (shimmer) before the web view data displayed on the screen. (Huwei P20Lite)

If I remove loading widget or replace loading widget with another widget(no animation, such as: Text(“loading”), ….) then the webview works fine and  this problem won't happen.
This problem still occurs when I try to replace shimmer with CircularProgressIndicator widget.

And the following log will appear many times when the problem occurs.
jank_removeInvalidNode jank list is null
I hope this information would be helpful for you.
Let me know if you have any workaround for this. Thanks in advance

@tuanhwing
Due to lack of the specific device in question for this issue, can you re-run your scenario with latest webview_flutter plugin verson and sdk version (3.24.3) to check if the behavior still persist or not ?
I tried on S10+ device and other android emulators but couldn't replicate.


Error with initial_clean and sudo

In tasks/initial_clean.yml there is, at the end a task:
  - name: Remove eventually entries for the local VM from known_hosts file
    known_hosts:
      name: "{{ he_fqdn }}"
      state: absent
    delegate_to: localhost

This causes an error of:
fatal: [foo.example.com -> localhost]: FAILED! => changed=false
  module_stderr: |-
    sudo: a password is required
  module_stdout: ''
  msg: |-
    MODULE FAILURE
    See stdout/stderr for the exact error
  rc: 1

since it is trying to sudo up on the local machine running Ansible since we have, in our playbook:
  become: True
  become_user: root

I think that the task could do with a become: no.
If you agree with this issue and would like, I can put a PR together?

I think it actually shouldn't be delegated at all. The entry in /etc/hosts isn't set at all during install on the localhost so doesn't need to be cleaned. The hypervisor that is creating the HE, however, does need this entry cleaned.
If this does actually have to be run on the localhost as indicated, then that presents another problem, as it assumes the user running the ansible playbook has privilege OR has the same route to becoming privileged as the remote host. In this case I'd propose that we re-factor the play such that the temporary IP address for the HE is pulled from /etc/hosts, stored as a variable and used in the plays as the delegation target rather than relying on dodgy /etc/hosts routing.

No, here we are not talking about /etc/hosts but ~/.ssh/known_hosts (known_hosts).
By default the setup is taking care to not inject any value there (we use '-o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null' in https://github.com/oVirt/ovirt-ansible-hosted-engine-setup/blob/master/tasks/bootstrap_local_vm/02_create_local_vm.yml#L13 when we dynamically add the new VM to ansible inventory) for the engine VM on the ansible controller host but, if for any reason (check an error on the engine VM and retry the setup creating a new VM with a different ssh key for instance) the user manually connects to the engine VM via ssh, an entry is added there and the next execution could fail.

Sorry, in that case become: False does the job - tested locally and fixes the issue for me.

Thanks


Replace 'Set#<<' with alias_method

I am newcomer to crystal lang. So if my code style is not match this world, feel free to mention it!

I don't know if it is still true but the last I heard alias_method is discouraged in the standard library.

Yes, we prefer to avoid alias_method. It's there, it works, but you will get worse compile errors, so we prefer to write a bit more code (in this case it's just two more lines, I don't think that's too terrible).


power_* are returning kilowatt instead of percent

this is fixed with release v0.1.05.
Sorry about the wait I rarely look at things here :)


fix: 'end_date must come after start_date' error

fix: Pressing the Usage Check Button prompts an error message "end_date must come after start_date"

#1189


Distress Admin Database

Merge with HACC branch.
Check the collections and exporting for distress for the animals. Add animal column.

Completed with HACC


From #58: As an organizer, I want to see an interactive map for more insight on questionnaire locations

From #58:
Right now, we display a high-level overview of where applicants are applying from, as well as the quantity.
However, if this were an interactive map (e.g. Google Maps overlay), this map could be zoomed in to see:

Exact location of applicants for planning bus routes
Visibility into which schools have a bus list active for them

For example: University Abc has a bus, but just down the road is another school with 5 applicants. Right now it's hard for someone to connect the dots, but if this was more precisely visible on a map, someone could see the close distance and be sure to reach out to those students about the bus.
Originally posted by @sman591 in https://github.com/codeRIT/hackathon-manager/issues/83#issuecomment-464568605

@cbaudouinjr Aren't you working on this? Or is the dashboard map just showing users locations currently?


Username in registration

I've added a userName to the Devise registration form. This works without Angular fine. Using angular_devise my form is sending the userName in the JSON request but it comes back null. Is there a way for angular_devise to handle this? The picking of a userName is done at the time of registration, not in a later step.

Hi ,
I just had the same issue using angular-devise with Ruby on Rails.
First you should have the attribute :username in your User model.
Then you should allow the :name atribute to be sent via this protected   def configure_permitted_parameters     devise_parameter_sanitizer.for(:sign_up) { |u| u.permit(:name, :email, :password, :password_confirmation) }     devise_parameter_sanitizer.for(:sign_in) { |u| u.permit(:name, :password) }     devise_parameter_sanitizer.for(:account_update) { |u| u.permit(:name, :email, :password, :password_confirmation, :current_password) }   end added to the application_controller.rb
Hope this helps.

Take a look at this post, they explain how to do it.
https://thinkster.io/angular-rails


examples/link-cp: fix a couple of strerror negations

Signed-off-by: David Disseldorp<EMAIL_ADDRESS>

Looks good, thanks.


Corrected example code

Thanks a lot for this!

Please note also that on newer versions of Electron, the function ’loadUrl()’ has been renamed into ’loadURL()’

I haven't worked on an Electron project in quite some time so it's hard to keep track. Feel free to send a second PR.


Faster room joins: Refactor test for user directory update once synced

This shouldn't change test behaviour at all.


#140295751: Removed unused table applicationmetrics

Good catch. Since we haven't shipped a release, one can get away with just deleting the table from the changelog rather than dropping which would be needed if we weren't rebuilding the dbs.

Should I delete the table from the existing changeset and update the PR ?

Yes.
Before we release v1, we will review the changelog and go to a single entry per log.
There is no point in having the db do more than it needs until we absolutely need to.

Updated.


merger reads files 1 one thread and deletes them in a another thread?

race condition? merger reads files 1 one thread and deletes them in a another thread?
################################################################
Fatal error in app merger:
retreiving one block files: fetching one block files: processing object list: unable to read one block:<PHONE_NUMBER>-20220110T225705.0-24f2e12c-ed760311-dom25:unable to read file header: EOF
################################################################
(launcher/launcher.go:174)

We got the same error today when upgrading our eth-mainnet namespace, here our logs:
[2022-01-10 14:17:41.159 EST] INFO (dtracing@v0.0.0-20210811175635-d55665d3622a/api.go:139) registering development exporters from environment variables {"logger":"common"}
[2022-01-10 14:17:41.159 EST] INFO (cli/start.go:50) Starting with config file '' {"logger":"dfuse"}
[2022-01-10 14:17:41.160 EST] INFO (cli/start.go:124) Launching applications: merger {"logger":"dfuse"}
[2022-01-10 14:17:41.160 EST] INFO (merger/app.go:60) running merger {"config":{"GRPCListenAddr":":9000","MaxOneBlockOperationsBatchSize":2000,"NextExclusiveHighestBlockLimit":0,"OneBlockDeletionThreads":10,"StateFile":"/data/merger.seen.gob","StorageMergedBlocksFilesPath":"gs://dfuseio-global-blocks-us/eth-mainnet/v5","StorageOneBlockFilesPath":"gs://dfuseio-global-blocks-us/eth-mainnet/v5-oneblock","TimeBetweenStoreLookups":5000000000,"WritersLeewayDuration":20000000000},"logger":"merger"}
[2022-01-10 14:17:41.166 EST] WARNING (merger/app.go:88) failed to load bundle  {"file_name":"/data/merger.seen.gob","logger":"merger"}
[2022-01-10 14:17:43.088 EST] INFO (bundle/bundler.go:25) new bundler {"bundle_size":100,"first_exclusive_highest_block_limit":13979400,"logger":"merger"}
[2022-01-10 14:17:43.088 EST] INFO (bundle/bundler.go:64) fetching one block files {"at_low_block_num":13979300,"logger":"merger"}
[2022-01-10 14:17:43.624 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:244) Processed, already existing merged file {"highest_block":13979399,"logger":"merger","lower_block":0}
[2022-01-10 14:17:43.625 EST] INFO (bundle/bundler.go:73) found lib to reach {"lib_block_num":18446744073709552000,"logger":"merger"}
[2022-01-10 14:17:43.625 EST] INFO (bundle/bundler.go:83) processed one block files {"at_low_block_num":13979300,"lib_num_to_reach":13979199,"logger":"merger"}
[2022-01-10 14:17:43.625 EST] INFO (bundle/bundler.go:64) fetching one block files {"at_low_block_num":13979200,"logger":"merger"}
[2022-01-10 14:17:44.218 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:244) Processed, already existing merged file {"highest_block":13979299,"logger":"merger","lower_block":0}
[2022-01-10 14:17:44.218 EST] INFO (bundle/bundler.go:83) processed one block files {"at_low_block_num":13979200,"lib_num_to_reach":13979199,"logger":"merger"}
[2022-01-10 14:17:44.218 EST] INFO (bundle/bundler.go:64) fetching one block files {"at_low_block_num":13979100,"logger":"merger"}
[2022-01-10 14:17:44.697 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:244) Processed, already existing merged file {"highest_block":13979199,"logger":"merger","lower_block":0}
[2022-01-10 14:17:44.697 EST] INFO (merger/app.go:130) merger initiated {"logger":"merger"}
[2022-01-10 14:17:44.698 EST] INFO (merger/app.go:144) merger running {"logger":"merger"}
[2022-01-10 14:17:44.698 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger.go:73) starting merger {"bundle":"bundle_size: 100, last_merge_block_num: 0, inclusive_lower_block_num: 13979300, exclusive_highest_block_limit: 13979400","logger":"merger"}
[2022-01-10 14:17:44.698 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/server.go:20) grpc server created {"logger":"merger"}
[2022-01-10 14:17:44.698 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/server.go:27) tcp listener created {"logger":"merger"}
[2022-01-10 14:17:44.698 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/server.go:31) server registered {"logger":"merger"}
[2022-01-10 14:17:44.698 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/server.go:34) listening & serving grpc content {"grpc_listen_addr":":9000","logger":"merger"}
[2022-01-10 14:17:45.161 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:244) Processed, already existing merged file {"highest_block":13979399,"logger":"merger","lower_block":0}
[2022-01-10 14:17:51.164 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:101) retrieved list of files {"files_count":196,"logger":"merger"}
[2022-01-10 14:17:51.169 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger.go:179) retrieved list of files {"added_files_count":196,"logger":"merger","too_old_files_count":0}
[2022-01-10 14:17:51.169 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger.go:107) one block retrieved {"last_block_file":13979579,"logger":"merger"}
[2022-01-10 14:17:51.199 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger.go:128) merging bundle {"bundle":"bundle_size: 100, last_merge_block_num: 13979399, inclusive_lower_block_num: 13979400, exclusive_highest_block_limit: 13979500","highest_bundle_block_num":13979499,"logger":"merger"}
[2022-01-10 14:17:54.592 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:51) merged and uploaded {"filename":"0013979400","logger":"merger","merge_time":3.392849938}
[2022-01-10 14:17:54.593 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger.go:141) bundle merged and committed {"bundle":"bundle_size: 100, last_merge_block_num: 13979499, inclusive_lower_block_num: 13979500, exclusive_highest_block_limit: 13979600","last_merge_one_block_time":"2022-01-10T19:01:11Z","logger":"merger"}
[2022-01-10 14:17:54.593 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger.go:144) saving state {"logger":"merger","state":"exclusive_highest_block_limit: 13979600"}
[2022-01-10 14:17:54.595 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:165) deleting files that are too old or already seen {
  "first_file": "0013979275-20220110T180635.0-e63ac982-790b114c-13979075-generated",
  "last_file": "0013979497-20220110T190023.0-b94a8d69-bb30a7c6-13979297-default",
  "logger": "merger",
  "number_of_files": 328
}
[2022-01-10 14:17:54.930 EST] WARNING (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:205) cannot delete oneblock file after a few retries {"error":"storage: object doesn't exist","file":"0013979284-20220110T180848.0-cad9d043-d8fbaf51-13979084-generated","logger":"merger"}
[2022-01-10 14:17:54.930 EST] WARNING (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:205) cannot delete oneblock file after a few retries {"error":"storage: object doesn't exist","file":"0013979315-20220110T181559.0-9fe877c6-b056f2f4-13979115-generated","logger":"merger"}
[2022-01-10 14:17:54.932 EST] WARNING (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:205) cannot delete oneblock file after a few retries {"error":"storage: object doesn't exist","file":"0013979367-20220110T182702.0-e0d7dddc-a893b78e-13979167-generated","logger":"merger"}
[2022-01-10 14:17:54.935 EST] WARNING (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:205) cannot delete oneblock file after a few retries {"error":"storage: object doesn't exist","file":"0013979275-20220110T180635.0-e63ac982-790b114c-13979075-generated","logger":"merger"}
[2022-01-10 14:17:54.937 EST] WARNING (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:205) cannot delete oneblock file after a few retries {"error":"storage: object doesn't exist","file":"0013979244-20220110T180019.0-636c426e-02332c7e-13979044-generated","logger":"merger"}
[2022-01-10 14:17:54.938 EST] WARNING (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:205) cannot delete oneblock file after a few retries {"error":"storage: object doesn't exist","file":"0013979239-20220110T175921.0-ba841eb3-6b1227cf-13979039-generated","logger":"merger"}
[2022-01-10 14:17:54.938 EST] WARNING (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:205) cannot delete oneblock file after a few retries {"error":"storage: object doesn't exist","file":"0013979341-20220110T182041.0-e98e20f0-488bdc95-13979141-generated","logger":"merger"}
[2022-01-10 14:17:54.939 EST] WARNING (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:205) cannot delete oneblock file after a few retries {"error":"storage: object doesn't exist","file":"0013979355-20220110T182336.0-614baa8d-dae5910e-13979155-generated","logger":"merger"}
[2022-01-10 14:17:55.150 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger_io.go:101) retrieved list of files {"files_count":9,"logger":"merger"}
[2022-01-10 14:17:55.150 EST] INFO (merger@v0.0.3-0.20211210145453-1cc2fa8425ea/merger.go:78) merger exited {"error":"retreiving one block files: fetching one block files: unable to read one block:<PHONE_NUMBER>-20220110T183754.0-0b371194-51245564:unable to read file header: EOF","logger":"merger"}
[2022-01-10 14:17:55.150 EST] ERROR (launcher/launcher.go:174)
################################################################
Fatal error in app merger:

retreiving one block files: fetching one block files: unable to read one block:<PHONE_NUMBER>-20220110T183754.0-0b371194-51245564:unable to read file header: EOF
################################################################
 {"context":{"reportLocation":{"filePath":"/work/go/pkg/mod/github.com/streamingfast/dlauncher@v0.0.0-20211210162313-cf4aa5fc4878/launcher/launcher.go","functionName":"github.com/streamingfast/dlauncher/launcher.(*Launcher).shutdownDueToApp.func1","lineNumber":"174"}},"logger":"dfuse"}
[2022-01-10 14:17:55.150 EST] INFO (cli/start.go:140) Application merger shutdown unexpectedly, quitting {"logger":"dfuse"}
[2022-01-10 14:17:55.150 EST] INFO (launcher/launcher.go:259) Waiting for all apps termination... {"logger":"dfuse"}
[2022-01-10 14:17:55.150 EST] INFO (launcher/launcher.go:273) All apps terminated gracefully {"logger":"dfuse"}
Error: unable to launch: retreiving one block files: fetching one block files: unable to read one block:<PHONE_NUMBER>-20220110T183754.0-0b371194-51245564:unable to read file header: EOF
[2022-01-10 14:17:55.150 EST] ERROR (derr@v0.0.0-20210811180100-9138d738bcec/cli.go:25) dfuse {"context":{"reportLocation":{"filePath":"/work/go/pkg/mod/github.com/streamingfast/derr@v0.0.0-20210811180100-9138d738bcec/cli.go","functionName":"github.com/streamingfast/derr.Check","lineNumber":"25"}},"error":"unable to launch: retreiving one block files: fetching one block files: unable to read one block:<PHONE_NUMBER>-20220110T183754.0-0b371194-51245564:unable to read file header: EOF","logger":"common"}

@billettc @froch

This was a false error, it actually a case of bad handling of end of file.


Investigate multiple Kubernetes client instances

See https://github.com/operator-framework/java-operator-sdk/issues/2040

@shawkings can you point me to the tests that cause the OperatorProducer to create a new client instance, please?

can you point me to the tests that cause the OperatorProducer to create a new client instance, please?

Any of the keycloak tests that extend baseoperatortest - https://github.com/keycloak/keycloak/tree/main/operator/src/test/java/org/keycloak/operator/testsuite/integration
I think I've also seen the OperatorProducer act at tear down as well - which wasn't expected for our tests either.

@shawkins has this been addressed or does it still require attention?


[Seq2Seq] Specifying tensor as initial state in GRU

Hello,
I’m trying to build a seq2seq Model as in this paper. I want to use a bidirectional GRU as Encoder and an unidirectional GRU as Decoder. The decoder receives the correct labels y_t-1 as input for training (Teacher Forcing). The loss is computed via time distributed softmax over a large vocabulary.
In the model the encoders hidden state represents a summary of the input data. The last hidden state of the encoder is transferred to the decoder and used as initial hidden state. With #5559 included in 2.0.2  I think it should be possible, to build such a model with keras. My model currently is:
main_input = Input(shape=(SEQ_LEN,), name="main_input", dtype="int32")
forced_input = Input(shape=(TARGET_LEN,), name="forced_input", dtype="int32")
encoder=Embedding(INPUT_VOC_SIZE,EMBEDDING_SIZE,input_length=SEQ_LEN,mask_zero=True)(main_input)
encoder=Bidirectional(GRU(return_sequences=False,units=HIDDEN))(encoder)
target_embedding=Embedding(TARGET_VOC_SIZE + 2, EMBEDDING_SIZE, input_length=TARGET_LEN, mask_zero=True)(forced_input)
decoder=GRU(units=HIDDEN,return_sequences=True)(target_embedding,initial_state=[encoder])
output=TimeDistributed(Dense(TARGET_VOC_SIZE,activation="softmax"))(decoder)
model = Model(inputs=[main_input, forced_input], outputs=[output])

The encoder should be connected to the decoder only by the transfer of the last hidden state (which is the output of the BGRU encoder) from encoder to decoder. The model trains, but as it turns out, the encoder is not connected to the decoder and doesn’t get trained. This can be seen when using tensorboard. There are no connections from gradients to the encoder or from encoder to decoder. This happens, because the specification of the initial state as list is incorrect in this case. I tried the specification as list because it was suggested in this PR #5559. However, when I try to specify the initial hidden state of the decoder byinitial_state=encoder (without brackets) I get the Exception:
Layer gru_2 expects 1 inputs, but it received 2 input tensors. Input received: <built-in function input>

I believe that happens because internally, in the class recurrent in _call_, the initial state is then treated as an additional input. When specifying initial state as list (with brackets) the state is not treated as additional input and therefore the model compiles, but the encoder is not connected to the decoder.
Is this a bug or intended behavior? What is the correct way to specify the initial hidden state in such a way that the layer which generates it, is connected to the graph and gets trained? I believe this is related to PR #5795.
Thank you for your help.

Has anybody figured out a way to correctly set the initial state of an RNN, such that it is added to the computational graph?

I have exactly the same problem, could someone fix this?

This seems to be a bug and I am facing the same issue. Any patches out yet?

Actually I just saw #5795 and upgraded my keras version. This issue is solved. @fchollet can you close this please?


What should be the ideal batch_size and number of epoch for training BERT_SPC on training data of 75k examples

You can do cross validation on a subset to find ideal hyper-parameters


FR: Mixed List style

Thanks for this plugin! I was about to write a longer dataviewjs query, but this plugin does the job much better anyway :)
I was wondering, is it possible to have an option in the settings / code block for mixed list styles? e.g. H2 numbered, but H3 bullets?

Interesting, I'm not entirely sure I'll have to look into it, but in the mean time lets work through how it would work @chrisgrieser.
What happens with a structure like this:
H2 - Numbered
H3 - Bullets
H4 - ??
H2 - Numbered
Is it just the highest level going bullets and then numbered?
What would the settings look like for that?
```toc
	style: varied
    outer_style: bullet
```

We'd need a way to:

Differ from the existing styles that control the entire ToC
Specify the "outer" style (bad name)

I'd say if you select a mixed list style, you have to specify for each heading level whether it is gonna be numbers or bullets.

I think this would make this project much more complicated, having a different setting for each level. I'll look into doing something just to configure the top most level.

If you set "style" to bullet
And you set "varied_layout" to true
Then the first level will be bullet and the rest will be number and vice versa.

Then if that works, that is our MVP. I can then play around with the implementation to try and find clean way to build this feature.

Thx 😄

Released MVP in 0.0.26, let me know if that's all good for you @chrisgrieser

works as intended 👍


Show more descriptive error message when compute quota is full

"You have reached your compute quota limit for this Azure subscription. Please follow these instructions to request a quota increase."

I need to update the error dialog (preferably for expected errors) to allow hyperlinks. I'd like to do this in a generic way but WPF doesn't make it easy. Investigating...

Per design guidlines we shouldn't put a link in the middle of the message. Update this to say

@beccamc Check if this issue is a duplicate.

Close due to improvements in https://github.com/dotnet/machinelearning-modelbuilder/issues/1702


grandexchange: misc item limits

Cape of Skulls - 8
Amulet of Power - 5
Ham Joint - 100
STAFF_OF_BOB_THE_CAT - 8
Black_platebody_(h1) - 70
Black_platebody_(h2) - 70
Black_platebody_(h3) - 70
Black_platebody_(h4) - 70
Black_platebody_(h5) - 70
Adamant Platebody h1 - 8
Adamant Platebody h2 - 8
Adamant Platebody h3 - 8
Adamant Platebody h4 - 8
Adamant Platebody h5 - 8
Wolf Mask - 4
Wolf Cloak - 4
MYSTIC_HAT_DUSK - 70
MYSTIC_ROBE_TOP_DUSK - 70
MYSTIC_ROBE_BOTTOM_DUSK - 70
MYSTIC_GLOVES_DUSK - 70
MYSTIC_BOOTS_DUSK - 70

Needs rebase.

Buy limits have been added to the wiki scraper in https://github.com/runelite/runelite-wiki-scraper/pull/12, and are used in 5a46abddc5c3d054037e21529a23a34334b21400. If there are any new limits, add them to the wiki, and it will automatically get pulled into the client.


Small improvements

Uses NetworkingV1Api for Ingress if needed.
Skip dockercfg secrets.

Thanks a lot @n0madic I was meaning to add support for newer networking api but could not till now. MR look good. Give me couple of days will test and merge.

Can this be merged please?
k8spurger broke after upgrade to k8s 1.22
Thanks!!

apologies @nadavbuc I got distracted and forgot about this.
I have merged the request. Thanks for your contribution.


Bad code in src/terminal/parser/ft_fuzzer/fuzzing_directed.h

Windows Terminal version
No response
Windows build number
No response
Other Software
No response
Steps to reproduce
The code in question is on line 965:
return (this->m_fFuzzed) ? &(this->m_t) : &CFuzzType<_Type, _Args...>::m_tInit;
The Visual C++ compiler is correctly rejecting this code for two reasons: 1) the non-static data member 'm_tInit' is protected so the attempt to create the pointer-to-member will fail; and 2) even if it succeeds there is no way the compiler can bring the types associated with the left and right expressions of the ':' operator to a common type. I suspect that the code should be:
return (this->m_fFuzzed) ? &(this->m_t) : &(this->m_tInit);
Thanks
JonCaves
Expected Behavior
No response
Actual Behavior
The code fails to compile with a recent build of the Visual C++ compiler

What version of the tools are you using? This seems to compile fine with VS 17.3.0 Preview 2.0...
Most of https://github.com/microsoft/terminal/blob/main/src/terminal/parser/ft_fuzzer/fuzzing_directed.h has been unchanged since we went open source, but INTERESTINGLY, that specific line was changed in #11816. In that PR, it was changed from what you suggest here to the current code. Link:https://github.com/microsoft/terminal/pull/11816/files#diff-5fbf1a284c85ac9a45b91d448ed5dc80ffc98cd56c82e9d40028e172d3681013L963
/cc @lhecker

I am using the live compiler -- it will probably become 17.4 at some point. The intention of that PR appears to be to make the code more conformant and so in order for 2-phase name lookup to work for access to 'm_t' and 'm_tInit' they need to be made dependent -- and the best way to achieve this is to append 'this->'. The syntax '&CFuzzType<_Type, _Args...>::m_tInit' creates a pointer-to-member - what is not what it wanted in this situation.
JonCaves

Thanks for the heads up about 17.4!


Implement build system

Here are the things I'm trying to achieve:

[ ] CSS/JS should be associated with an include/layout.
[ ] Pages only include CSS/JS they need.
[ ] Ideally, CSS modules.
[ ] The JS uses code-splitting and bundling.
[ ] Assets are hashed, including images referenced from the CSS.
[ ] CSS/JS includes are deduped.
[ ] As little magic as possible.
[ ] Watch mode.

This landed via #105


[DOCFIX] Update Chinese GCS Docs

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10864/
Test PASSed.


Add ability to link to variables in a dataset.

perhaps something like: [:variables:/rdi3p] or something similar for the link pattern.

Closing this as it's fairly non intuitive and easier to just link the variable directly using markdown.


Insert Link in visual mode behaves strangely

When I use the insert_node function while in visual mode, this doesn't work as expected. I actually don't know, what the intended behavior is, but currently it seems to be broken. My intuition was, that it would grep the visual selection, open the find selector, let me choose a note and inserts a link while preserving the selection as description of the link.
What currently happens: Nothing get's inserted, I get no selection buffer, but the visual selection changes to the current word under cursor and Neovim goes back into insert mode.

Actually a similar behavior can be observed with "find node" in visual mode.
Now, as I am playing more with different cases, I find out, that this strange behavior happens only, if the selection cannot be found in the titles of my existing Roam files.

I actually try to pin down the strange behavior. When I use insert_node or find_node on a selection, which is not part of a title of an existing roam node, then the cursor jumps to the most left buffer and the mode changes to insert.
The reason, why the selection aborts, I found by this line:
https://github.com/chipsenkbeil/org-roam.nvim/blob/5943188a36e2273bce869b56c2867712f60745f9/lua/org-roam/ui/select-node.lua#L57
I actually would prefer, when the select buffer does not just close but gives me the opportunity to refine my selection.
But what happens after cancelation is still a mystery to me. Closing the selection emits a EVENTS.CLOSE which somewhere should be translated in a EVENTS.CANCELATION but I just found this out by grepping through the code and it's hard for me to get a hang of how the business logic is actually orchestrated.
@chipsenkbeil Any hints on that? And what do you think about how the feature should actually behave?
My intuition and preference would be, that the selection could be inserted as initial input into the select buffer, but if no selection is found, instead of canceling, a warning is shown and the selection is removed, so the user has to reenter something else to filter down.
For the insert_node I still would like to have the selection kept as link description. If I just want to expand an existing word to a link, I can use <leader>n.. Otherwise both features are nearly identical, but creating links with a different description is not covered and must be done manually.

@seflue another case of me not remembering fully where this is used. There was a situation where the experience was sub-optimal, and I think it was the complete under cursor opening a dialog window when there was no match; so, I wanted it to do nothing instead.
Looks like because this was put at the level of the select node ui function, it's affecting inserting and finding nodes as well. So this either needs to be updated to be false by default and user-configurable so roam_complete_node_under_cursor can still cancel the selection dialog automatically or have a way to turn it off, which would be used everywhere else.
I'm in favor of the first approach, where we change it to not be enabled within the core select ui and instead provide a way for roam_complete_node_under_cursor to specify it instead.

My intuition and preference would be, that the visual selection could be inserted as initial input into the select buffer, but if no selection is found, instead of canceling, a is shown and the selection is removed, so the user has to reenter something else to filter down.

To make sure I understand, you're recommending that if a match is not found, display some message and clear the input so that someone can type something else, versus needing to manually tap the backspace key to clear out the selection? That makes sense, although I'm of the mind that this is configurable as well. This would also mean that we need to change behavior or add support within the core selection dialog tool, which is possibly the most complex part of the codebase given how many edge cases I was handling to make the behavior tolerable.

For the insert_node I still would like to have the visual selection kept as link description. This behavior would be similar to most of the textual tools, which allow to insert links, including Github.

I'm fine changing the behavior here. I was trying to figure out how Org Roam implemented the solution and copy it. We can either change the behavior for everyone, or make it be a setting that can be configured to switch between behaviors.

If I just want to expand an existing word to a link, I can use n.. Otherwise both features are nearly identical, but creating links with a different description is not covered and must be done manually.

Agreed.

My intuition and preference would be, that the visual selection could be inserted as initial input into the select buffer, but if no selection is found, instead of canceling, a is shown and the selection is removed, so the user has to reenter something else to filter down.

To make sure I understand, you're recommending that if a match is not found, display some message and clear the input so that someone can type something else, versus needing to manually tap the backspace key to clear out the selection?
Yes.

That makes sense, although I'm of the mind that this is configurable as well. This would also mean that we need to change behavior or add support within the core selection dialog tool, which is possibly the most complex part of the codebase given how many edge cases I was handling to make the behavior tolerable.

I agreed, that everything should be somehow configurable.

For the insert_node I still would like to have the visual selection kept as link description. This behavior would be similar to most of the textual tools, which allow to insert links, including Github.

I'm fine changing the behavior here. I was trying to figure out how Org Roam implemented the solution and copy it. We can either change the behavior for everyone, or make it be a setting that can be configured to switch between behaviors.

Actually it would be nice to have the expected link description visible and editable - but this might be step two, because UI code is, as you mentioned, a bit complex in Neovim. My experience with - let's call it "Neovim TUI Widgets" is still limited - I learned everything I know, when I added the timestamp extension to the Calendar widget in nvim-orgmode. But I would imagine a second line with the description and the user can toggle between the filter and the description with a mapable key, by default e.g. . If kept empty, it should be the node title (we could visualize that with italic greyed out title of the current link, but if the user entered the select window from visual mode, it should be set to the current visual selection.
Because I have currently configured Emacs available and running, I wasn't able to check first hand how Emacs Orgroams UI works. In the manual they are rather vague about the linking features.
I would be motivated to implement something useful. But first I wanted to agree with you on some sane UI workflow and then talk about entry points.
Speaking of entry points - how do you feel about some architectural documentation to enable more sustainable contribution to the project? What I can offer you is to condense my current belief state of the concepts and the architecture into a Draw.io diagram and add it as editable SVG to the project. This would probably also include some questions about things I currently don't understand. But what you then can do is to review this diagram and correct it/answer the questions as you feel confident.
Then we would have at least something more then just the code. As you said, it is easy to forget the stuff we implemented a couple of month ago and it is hard to rediscover the knowledge just from the code itself.

Speaking of entry points - how do you feel about some architectural documentation to enable more sustainable contribution to the project? What I can offer you is to condense my current belief state of the concepts and the architecture into a Draw.io diagram and add it as editable SVG to the project. This would probably also include some questions about things I currently don't understand. But what you then can do is to review this diagram and correct it/answer the questions as you feel confident.

Sure, I don't see any harm in doing that. I can't promise I can get to the diagrams quickly at the moment due to family visiting, but feel free to share when you hav something drafted 😄


Issue with the "node bot.js" command?

Following the guide, I inputted the "npm install discord.io winston --save" command, and then input "node bot.js".
I was greeted by this.

Not 100% certain what's going on here, but I'm getting the feeling that I'm missing something. Anyone got an idea of what's going on?

Check if you have right syntax, I can see an error in your code. (line 13)


Add a component parameter with an overridden base property without the parameter attribute throws ArgumentException

Describe the bug or question
Add a component parameter with a component property which overrides an abstract property of an abstract class without the parameter attribute throws ArgumentException.
With this test:
public class BoolFieldTest : TestContext
{
    [Fact]
    public void Test()
    {
        RenderComponent<BoolField>(parameters => parameters
            .Add(x => x.Value, null));
    }

    public class BoolField : BaseFieldComponent<bool?>
    {
        [Parameter] public override bool? Value { get; set; }
    }

    public abstract class BaseFieldComponent<T> : ComponentBase
    {
        public abstract T Value { get; set; }
    }
}

Results in this output:
System.ArgumentException
The parameter selector 'x => x.Value' does not resolve to a public property on the component 'BoolFieldTest+BoolField' with a [Parameter] or [CascadingParameter] attribute.
   at Bunit.ComponentParameterCollectionBuilder`1.GetParameterInfo[TValue](Expression`1 parameterSelector) in /_/src/bunit.core/ComponentParameterCollectionBuilder.cs:line 355
   at Bunit.ComponentParameterCollectionBuilder`1.Add[TValue](Expression`1 parameterSelector, TValue value) in /_/src/bunit.core/ComponentParameterCollectionBuilder.cs:line 55

Expected behavior:
I would expect this to work. As a workaround I now set the [Parameter] attribute on the abstract property.
Version info:

Library version: v1.0.0-beta-11

Hey @nickmuller
Thanks for reporting this. That is indeed a bug, and I will push a fix later that solves it. It should be available as a nightly build after that, if you are blocked: https://github.com/egil/bUnit/discussions/209

Fixed in latest nightly: https://github.com/egil?tab=packages&repo_name=bUnit

Great, thanks!


do not use window.history if inside iframe

This checks if next.js is run inside an iframe in which case it does not use window.history. I do not see how history makes sense in the context of being embedded in another site anyway.
Fixes this crash that prevented next.js from working at all in an iframe: https://github.com/zeit/next.js/issues/3118

Thank you very much @hankmander 🙏

Thank you guys for being responsive to requests and maintaining an awesome project! :smile:

❤

@timneutkens It says above "timneutkens merged commit b8f189f into zeit:canary on 4 Feb” but it seems this is not in Next.js 7.0.2?


Biased number with more change to get minimum number

On the doc, we can see a sample for a biased number between 10 and 20 with more chance to get a number next to 20.
But what should I do to get the reverse? I mean, more chance to get a number next to 10 than next to 20?
Thanks.

Hi Soullivaneuh,
biasedNumberBetween() takes a function, in the example sqrt(), with this probability graph:

The opposite of the example would be biasedNumberBetween(10, 20, function($x) { return 1 - sqrt($x); }:

Feel free to substitute any function to fit your needs. (However, a function with a very small area under the graph may make the formatter inefficient; x or -x are probably suitable for most uses).

The opposite of the example would be biasedNumberBetween(10, 20, function($x) { return 1 - sqrt($x); }:

It would rather be inversed in the parameter: return sqrt(1 - $x);. Your graphs show that the shape below the line is not mirrored.

biasedNumberBetween() takes a function, in the example sqrt(), with this probability graph:

So the probably of getting 1 is certain? That can't be right?

Is this a cumulative function? If so, that's kind of misleading, but I can't make sense of this if it's meant to be a probability distribution function.

One mirrors it vertically, one horizontally. Technically the inverse is x², but that's definitely not what's wanted here.
@GrahamCampbell It's not a distribution. It actually picks a random point on the unit square, figures out if it is below the function, and if it is keeps the x value if it is.

with this probability graph:

That's the bit that threw me. It's not that. ;)

Then let's call it a "usefull visualization" :grin:

lol, sure ;)

The opposite of the example would be biasedNumberBetween(10, 20, function($x) { return 1 - sqrt($x); }:

Yes, but painful to put it on a yaml file.
If I understand well, we have only two cases: near the higher and near the lower.
Why not make the method simpler like this?
biasedNumberBetween(10, 20, true); // True will be the $higher variable
biasedNumberBetween(10, 20); // True by default
biasedNumberBetween(10, 20, false); // Let's get to near lower number

OR
biasedNumberBetween(10, 20, 'higher'); // True will be the $higher variable
biasedNumberBetween(10, 20); // True by default
biasedNumberBetween(10, 20, 'lower'); // Let's get to near lower number

What do you think? Could provide a POC to test.

It can also be biased toward the middle or the ends or towards certain
numbers. The degree of bias can also be set using a custom function.
If you don't care exactly which function, 'cos' will bias the generator
towards the lower end.
On Wed, Oct 7, 2015, 10:04 Sullivan SENECHAL<EMAIL_ADDRESS>wrote:

The opposite of the example would be biasedNumberBetween(10, 20,
function($x) { return 1 - sqrt($x); }:
Yes, but painful to put it on a yaml file.
If I understand well, we have only two cases: near the higher and near the
lower.
Why not make the method simpler like this?
biasedNumberBetween(10, 20, true); // True will be the $higher variablebiasedNumberBetween(10, 20); // True by defaultbiasedNumberBetween(10, 20, false); // Let's get to near lower number
OR
biasedNumberBetween(10, 20, 'higher'); // True will be the $higher variablebiasedNumberBetween(10, 20); // True by defaultbiasedNumberBetween(10, 20, 'lower'); // Let's get to near lower number
What do you think? Could provide a POC to test.
—
Reply to this email directly or view it on GitHub
https://github.com/fzaninotto/Faker/issues/727#issuecomment-146107554.

it would be nice if someone could do a few functions with diagrams like the once that okj579  has done above.
better still, replcate the 'easing.js' functions are so familiar to us all. This way we could call min=int, max=int, easeInOut


Quick fix “create method” inside a late variable initializer shouldn’t make a static method

Starting with this code:
class C {
  late final lazilyComputedValue = _computeValueLazily();
}

place the cursor on the unresolved identifier _computeValueLazily and request the quick fix "create method".
Observed result:
class C {
  late final lazilyComputedValue = _computeValueLazily();

  static _computeValueLazily() {}
}

Expected result:
class C {
  late final lazilyComputedValue = _computeValueLazily();

  _computeValueLazily() {}
}

(the method shouldn't need to be static because the initializers of late fields have access to this).

https://dart-review.googlesource.com/c/sdk/+/295383


Codecov: submit coverage reports to get coverage tracking over time

It's free for open source projects, we just need to register an account and do the CI bits to upload the reports. https://codecov.io/

LMK if you want a hand setting this up, I've done it a good few times.

Yeah, that would be great! Do you happen to know what's involved if/when we move the repository?

Oooh, and this reusable action should make it easy to do the coverage report upload. https://github.com/codecov/codecov-action Care to have a go at it @jeromekelleher?

Yeah, that would be great! Do you happen to know what's involved if/when we move the repository?

It's easier to do the move first, codecov does get confused quite easily.

Do we have a timeframe in mind for when we might like to move this repo to its long-term home?

I'd rather not move anything before announcing to the consortium. Maybe we can discuss on the call today?

Agreed, definitely we shouldn't move anything before then. Let's talk about it later.


Fix sccache usage in Windows CI builds

Description
This corrects a couple problems preventing the Windows CI builds from using sccache.
Fix for #896 and reduces windows build significantly.

@gpx1000 You mentioned having trouble getting sccache working in Windows CI builds.  I was able to track it down after I noticed something odd in the beginning of the CMake config.
The basic issue was twofold...

Although sccache doesn't work with the Visual Studio generator, some sources online seem to suggest it works with the "NMake Makefiles" generator, but as far as I can tell it does not.
The online commentary about the need to use /Z7 instead of '/Zi may be accurate, but the mechanism that was being used didn't work for the reasons I noted in #896

It occurred to me that I should revert the change adding messaging about lack of ccache or sccache since they don't work with the most commonly used Windows generators.  I could gate them behind a if (CMAKE_GENERATOR MATCHES "Visual Studio"), but I figured it's best to just leave it closer to as is.

I always kept it in Ninja build for Windows as I wanted to ensure the windows build system was using at least NMake.  SCCache was supposed to work with NMake but I never got that working right.  I think it's far better to just bite the bullet and use Ninja as you have here.  I don't think at first blush that the macos build failure in CI is due to the changes here.  lemme know if you need help changing it.
Thanks for the contribution; greatly appreciated!  And great catches of bugs.


⚠ Garbage Collector has degraded performance

In f74702e, Garbage Collector (https://gc.fef.moe/login) experienced degraded performance:

HTTP code: 200
Response time: 13380 ms

Resolved: Garbage Collector performance has improved in 00ca4c9.


Can't Pull Request

I am unable to do Pull Request. Please help me to do it.
@kroy96 @Hegdesachin87 @ch-amul

I am also unable to generate pull request.

I am also unable to generate pull request.

@kroy96 @piyushagarwal08 @rexyagami @ch-amul
Also even in compare and generate pull request it's not giving me the option to generate pull request.

no one is replying also.

Just give me a moment

ok

Hi @soumya969 @Anurag-Aggarwal the issue that u r facing is due to the fact that u have actually forked a repo which itself is a fork of the base repo, which is the reason why u r getting this error.
Please avoid multi tier forks of repos. You can resolve this issue by creating a new fork and then generating a PR

Is gtdc-day 2 a base repo or not?
@kroy96

Yes it is

Hi @soumya969 @Anurag-Aggarwal the issue that u r facing is due to the fact that u have actually forked a repo which itself is a fork of the base repo, which is the reason why u r getting this error.
Please avoid multi tier forks of repos. You can resolve this issue by creating a new fork and then generating a PR

what to do with old one

i have already create a repo


i18n breaks Nuxt Auto-Import functionality!

Environment
Nuxt project info:

Operating System: Windows_NT
Node Version:     v20.6.1
Nuxt Version:     3.7.4
CLI Version:      3.9.0
Nitro Version:    2.6.3
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, typescript, imports, modules, pinia, i18n
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-cmht5h?file=stores%2Fpage.ts
Describe the bug
The error occurs when using the pinia & i18n packages for nuxt 3. All auto-imports for components are lost, when importing a component inside a pinia store, to define a InstanceType. I've tested to remove the i18n package and all auto-imports were back to normal.
Additional context
Also checked that Volar (takeover) is active. Bug can be viewed easily when following small instructions inside /stores/page.ts
Logs
No response

Thank you for you reporting!
Hmm, 🤔
nuxt i18n is just implemented using addImports.
We’re implementing it using the API provided by nuxt/kit.
https://github.com/nuxt-modules/i18n/blob/8a33fecd9b9c4668145e41d3fb35640d308711a5/src/module.ts#L297-L315
I don't see why combining it with @pinia/nuxt would cause issues. /cc @posva
@danielroe
Is there anything in nuxt i18n's auto-import implementation that is not within the guidelines?

I have no idea why combining both modules would break auto imports 😅

After trying the reproduction in stackblitz and locally, I can't reproduce the issue described.
Uninstalling and removing @nuxtjs/i18n and its configuration seems to have no effect, SomePopup gets inferred correctly in app.vue but cannot be found inside the page.ts store with the popupDefined line uncommented. Edited reproduction here
Maybe I'm doing something wrong, can someone else confirm this issue is happening as described?

After trying the reproduction in stackblitz and locally, I can't reproduce the issue described.
Uninstalling and removing @nuxtjs/i18n and its configuration seems to have no effect, SomePopup gets inferred correctly in app.vue but cannot be found inside the page.ts store with the popupDefined line uncommented. Edited reproduction here
Maybe I'm doing something wrong, can someone else confirm this issue is happening as described?

Yeah to use the InstanceType, you have to uncomment the import in line 9 inside 'page.ts'. Then it will break locally. Mabe the Component needs to be imported another way?

I can't reproduce when downloading that stackblitz and refreshing lockfile.

I can't reproduce when downloading that stackblitz and refreshing lockfile.

Its all fine inside the store. What's not working is the auto-import inside all other files like app.vue when popupDefined is uncommented all components become the type 'unknown'

Hi guys, I have some videos to visualize the bug that is happening.
After looking more into it, I've found out that the bug does not only happen inside a pinia store:
https://github.com/nuxt-modules/i18n/assets/14128197/ab7fd5c7-aae9-4ae8-818c-1922983c7753
When @nuxt/i18n is removed no bug appears:
https://github.com/nuxt-modules/i18n/assets/14128197/f9423cb9-0d9e-412f-b9e0-f3c0546b2014
Updated reproduction:
https://stackblitz.com/edit/nuxt-starter-cmht5h?file=stores%2Fpage.ts

Hi all, I can confirm @Bombastickj reproduction, I have the same issue with my current project using nuxt i18n last version. All Vue components (from /components folder or coming from other plugins/modules) inside my template are reported as unknow.

I also can confirm this issue

[EDIT] What is weird, the components are imported correctly when call inside another components (inside the components folder) but not inside the /pages folde

This doesn't work for me at all. All the autocompletion just dies even inside the components folder.

Hello,
do you guys have any updates on why this is happening? @kazupon @BobbieGoede

I'm still looking into what the exact source of this issue is, it's not that easy to debug unfortunately.
I have been trying to find what is breaking the type inference by removing type imports inside .nuxt and restarting the vue language server, which is tedious.
By doing so I have been able to narrow it down to types provided by @intlify/vue-i18n-bridge (in vue-i18n-routing) and vue-i18n (in @nuxtjs/i18n), if I remove type imports from those dependencies the type inference seems to work again. Next I will have to figure out what is different about those types, that will take some more time 😭

It seems like these lines are breaking the type inference https://github.com/intlify/vue-i18n-next/blob/master/packages/vue-i18n/src/vue.d.ts#L1232-L1239

Thank you so much @BobbieGoede!

It seems like these lines are breaking the type inference<EMAIL_ADDRESS>
I haven't looked myself but in that case the issue is likely that it extends GlobalComponents in @vue/runtime-core but should be extending it in vue module instead.

And also it should likely first import 'vue' as the original module needs to be referenced first, otherwise (depending on loading order) the i18n module can overwrite the module instead of extending it.

The types provided by vue-i18n in npm will be combined with typescript-generated types and vue-extended types defined in src/vue.d.ts when building.
The combined types have import vue, so your point should not be a problem.

https://github.com/nuxt-modules/i18n/issues/2490#issuecomment-1793330070 This is related. Probably can be closed now if the issue is fixed

Hi guys!
After having some small issues with opting into the edge release channel, I finally got it working! If anyone runs into some trouble installing, I used node 18.18.2. Don't forget to remove lock file and node_modules before installing.
Thank you guys so much!! @BobbieGoede @kazupon
Fixed reproduction


wizbin: files.py: remove 'encode' as F is already a 'bytes'

This fix allows code to run with python3.10 (ubuntu 22.04) from the master branch.
Without this fix, we get an error:
Traceback (most recent call last):
  File "/home/farhi/dev/debreate/main.py", line 463, in OnProjectSave
    SaveIt(self.LoadedProject)
  File "/home/farhi/dev/debreate/main.py", line 409, in SaveIt
    GetPage(pgid.FILES).GetSaveData(),
  File "/home/farhi/dev/debreate/wizbin/files.py", line 348, in GetSaveData
    print(F)
AttributeError: 'bytes' object has no attribute 'format'

every time there is a parsing of files added in the 'Files' page. This is the case when saving the dpb as well as when actually generating the deb

Thanks. I don't know if this breaks compatibility with Python 2.7 (I don't know if it even was still compatible), but I probably want to move away from it anyway.


After running npx scully, I got the same html content

but in about.html
I got

Scully doesn't support hash routing. The reason is that 90% of the servers can't handle this.
We can write those files to disk, but most of the servers will have serious trouble serving those files.
(mostly, because everything after # is considered client-side, and the servers just truncate those)
I don't think this is something that can be solved.

Thx, I'm going to use H5 routing


Fix example.py due to LTS update

Updates example to address the update to the least trimmed squares processing code.
Fixes #39.

I think the updates to example.py here are working, but when I run that script using this branch I get the following error:
ALPHA is 1.00. Performing an ordinary least squares fit, NOT least trimmed squares.
/Users/ldtoney/repos/lts_array/lts_array/classes/lts_classes.py:725: RuntimeWarning: invalid value encountered in sqrt
  sigma_tau[jj] = np.sqrt(tau[weights, jj, :].T @ residuals / (
/Users/ldtoney/repos/lts_array/lts_array/classes/lts_classes.py:507: RuntimeWarning: invalid value encountered in cdouble_scalars
  D = np.sqrt(foo + (a * b - 2 * c - a2 * a / 4) / R)
/Users/ldtoney/repos/lts_array/lts_array/classes/lts_classes.py:508: RuntimeWarning: invalid value encountered in cdouble_scalars
  E = np.sqrt(foo - (a * b - 2 * c - a2 * a / 4) / R)  # ...

Related to NaNs?

error

Whoops. I meant "warning" here. Looks good!

error

Whoops. I meant "warning" here. Looks good!

I made a note on the lts_array page about the RuntimeWarning for future reference.

Are we good to merge here? Currently if someone tries example.py in main they'll get an error — I think we should fix this soon.

@jwbishop want to merge this into the main branch? or I can. thanks!


Model Validation Script

Hello and thanks for your work. I need to run model for Keyword Spotting from this zoo, but there is no any validation script with preprocessing for my .wav files. How should i prepare my data to apply models?

Hey,
Your best bet is to look into the code used for training and evaluating the keyword spotting models here
There is a script for testing the TFLite models themselves here on the whole test set.
For how to actually prepare .wavs for sending into the model this is covered in this file.
Unfortunately there isn't a function or script that shows how to perform inference on a single wav file but this shouldn't be a tricky thing to do (we might actually add this in the future). It should be a matter of just calling load_wav_file followed by calculate_mfcc and then sending this for inference.
Note that the models work on 1 second mono audio clips, with a sample rate of 16000 samples per second. The window size, stride and number of mfcc features depends on the model and comes from here


Custom reservoirs

Can one perform EchoStateNetworkCV using custom reservoir (adjacency) matrix, i.e.,  optimizing everything without randomly assigning reservoir matrix at each step (if it takes place at all)? I tried to add some parameters in the source code to make it possible (I manually assigned reservoir_matrices.reservoir_pre_weights in EchoStateNetworkCV and protected it from changing, then, after optimization, passed it to 'reservoir' parameter in EchoStateNetwork), it seems to have worked, but I'm not sure if the matrix is kept intact throughout the whole procedure since I cannot yet comprehend the whole code.

hello @cyberfag the software has been dramatically upgraded, simplified and the variable names have been changed. If you would still like to create custom reservoirs please reach out to me at<EMAIL_ADDRESS>and I will be happy to work with you to accomplish your research goals.


[Feature request] Support effect_list of light

Hi, is it possible to add support to effect_list of light (Xiaomi Mesh)?
I've tired adding it for light (pid: 3164) in bluetooth.py. Then with my limited knowledge of Python, I find this function is not enabled in light.py, but I don't know how to modify it. More precisely, I don't know how to deal with list type variables.

Thank you for your attention.

Please reopen the issue if it is still relevant


fix: MD005/list-indent

Inconsistent indentation for list items at the same level

@nschonni : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Is the AtariEnvironmentWrapper not being used?

Hi,
Is this function being used anywhere?

Thanks for your interest, indeed AtariEnvironmentWrapper is not used anywhere. It was originally used during development of these agents to make it easier to compare having the Atari preprocessing in the environment versus in the agent itself. We eventually went with having the preprocessing in the agent for a number of reasons, but since it is more common to have the preprocessing in the environment, AtariEnvironmentWrapper was kept for the convenience of users.

I see. Thanks for your reply, John :)


Missing containers in Containers dashboard

I switched my Heapster + InfluxDB setup from what was deployed by Kubernetes 1.1 to what I found here @ master.
* heapster:v0.18.2         ->  heapster:canary
* heapster_influxdb:v0.4   ->  heapster_influxdb:v0.6
* heapster_grafana:v2.1.1  ->  heapster_grafana:v2.5.0

The Containers dashboard, which I also loaded from the current master, is missing some containers when looking at the details of a couple of my pods.

Example:
pod api-12345 runs 3 containers: rails, logrotate, logforwarder
The query which is used for the $container variable in Grafana (Templating > Variables) yields incomplete results:
SHOW TAG VALUES FROM "uptime_ms_cumulative"
  WITH KEY = "container_name"
  WHERE pod_name =~ /api-12345/
  AND "pod_namespace" =~ /my_namespace/

Result:  {logrotate,logforwarder}
Expected: -> {rails,logrotate,logforwarder}

I'm not sure yet if this comes from the change in the Grafana queries or if InfluxDB is fed incomplete information by Heapster. I will look into it, but any feedback from other users would be appreciated.

Extra info: the query used on InfluxDB 0.8 (Kubernetes 1.1)
SELECT distinct(container_name) from "uptime_ms_cumulative"
  WHERE pod_name =~ /api-12345/
  AND "pod_namespace" =~ /my_namespace/
  AND time > now() - 5m

Result: -> {rails,logrotate,logforwarder}

cc @thucatebay

Found it.
Heapster is struggling to write data for this one group of pods. My heapster logs are full of:
driver.go:207] failed to write stats to influxDB - {"error":"partial write:\nunable to parse 'uptime_ms_cumulative,container_base_image=example.com/image:tag,container_name=rails,...\"containerID\":\"docker://08b6bbb923aef450d50fee92038132601f34425a1b475311b3a4d47e40a82252\"}}\\,\"ready\":true\\,\"restartCount\":2\\,\"image\":\"example.com/image:tag\"\\,\"imageID\":\"docker://47299f016d3729dcc5d8033c3db7d9ddf130f22cc9e7a3008cdcd00320ac094b\"\\,\"containerID\":\"docker://3edcf5685d7961336d4f180dfdd3d976c60f987a4247c00fbef74f6176a26016\"}]}}': missing fields"}
Current pod definition:
apiVersion: v1
kind: Pod
metadata:
  annotations:
    kubernetes.io/created-by: |
      {"kind":"SerializedReference","apiVersion":"v1","reference":{"kind":"ReplicationController","namespace":"bodyweight-api","name":"api","uid":"7dec1239-a3ff-11e5-9f88-0a59d1e77755","apiVersion":"v1","resourceVersion":"40275725"}}
  creationTimestamp: 2015-12-18T22:40:00Z
  generateName: api-
  labels:
    app: fl-backend-rails
    component: api
    deployment: "33"
  name: api-3p9gu
  namespace: bodyweight-api
  resourceVersion: "40275903"
  selfLink: /api/v1/namespaces/bodyweight-api/pods/api-3p9gu
  uid: 45b03770-a5d8-11e5-9f88-0a59d1e77755
spec:
  containers:
  - env:
    - name: ROLE
      value: api
    - name: RAILS_ENV
      value: production
    - name: POD_NAMESPACE
      valueFrom:
        fieldRef:
          apiVersion: v1
          fieldPath: metadata.namespace
    image: example.com/image:tag
    imagePullPolicy: IfNotPresent
    lifecycle:
      preStop:
        exec:
          command:
          - /usr/sbin/nginx
          - -s
          - quit
    name: rails
    ports:
    - containerPort: 9080
      name: http
      protocol: TCP
    resources:
      limits:
        cpu: 400m
        memory: 1800Mi
      requests:
        cpu: 400m
        memory: 1800Mi
    terminationMessagePath: /dev/termination-log
    volumeMounts:
    - mountPath: /run/secrets/example.com/rails
      name: rails-secrets
      readOnly: true
    - mountPath: /app/log
      name: rails-logs
    - mountPath: /var/log/nginx
      name: nginx-logs
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: default-token-68jsr
      readOnly: true
  - image: apopelo/logstash-forwarder
    imagePullPolicy: IfNotPresent
    name: logstash-forwarder
    resources: {}
    terminationMessagePath: /dev/termination-log
    volumeMounts:
    - mountPath: /var/log/containers/rails
      name: rails-logs
      readOnly: true
    - mountPath: /var/log/containers/nginx
      name: nginx-logs
      readOnly: true
    - mountPath: /etc/logstash-forwarder
      name: logstash-conf
      readOnly: true
    - mountPath: /etc/ssl/logstash-forwarder
      name: logstash-ssl
      readOnly: true
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: default-token-68jsr
      readOnly: true
  - image: example.com/logrotate
    imagePullPolicy: IfNotPresent
    name: logrotate
    resources: {}
    terminationMessagePath: /dev/termination-log
    volumeMounts:
    - mountPath: /var/log/containers/rails
      name: rails-logs
    - mountPath: /var/log/containers/nginx
      name: nginx-logs
    - mountPath: /etc/logrotate.d
      name: logrotate-d
      readOnly: true
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: default-token-68jsr
      readOnly: true
  dnsPolicy: ClusterFirst
  imagePullSecrets:
  - name: docker-registry
  nodeName: ip-10-0-0-1.eu-west-1.compute.internal
  restartPolicy: Always
  serviceAccount: default
  serviceAccountName: default
  terminationGracePeriodSeconds: 30
  volumes:
  - name: rails-secrets
    secret:
      secretName: rails
  - emptyDir: {}
    name: rails-logs
  - emptyDir: {}
    name: nginx-logs
  - name: logstash-conf
    secret:
      secretName: logstash-conf
  - name: logstash-ssl
    secret:
      secretName: logstash-ssl
  - name: logrotate-d
    secret:
      secretName: logrotate-d
  - name: default-token-68jsr
    secret:
      secretName: default-token-68jsr
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: null
    status: "True"
    type: Ready
  containerStatuses:
  - containerID: docker://5e35e8d3d447e21a2f48473cc92008704ae6cac2412b1e74c1b93a736a028766
    image: example.com/logrotate
    imageID: docker://6c9c7a7a9c779eafa7123550b44967a18f1d43b5125acbcc317903b82b5800cf
    lastState: {}
    name: logrotate
    ready: true
    restartCount: 0
    state:
      running:
        startedAt: 2015-12-18T22:40:04Z
  - containerID: docker://0181d898943aef21d9da96f413586c1f63bb401f1629a027e08d7f886fba6f5d
    image: apopelo/logstash-forwarder
    imageID: docker://32be67e30853d07971c5df6e7cc55607c946aa3c4f1d4b408a5aca18ff760fd5
    lastState: {}
    name: logstash-forwarder
    ready: true
    restartCount: 0
    state:
      running:
        startedAt: 2015-12-18T22:40:04Z
  - containerID: docker://159ebe374104eb05ef3ee8ca469788240a6508e3938f2b6be9c78600f409610d
    image: example.com/image:tag
    imageID: docker://f365817002eb3ccb8f91ead008dfee26e5b073dfa301b8b10589bd7632ad86d8
    lastState: {}
    name: rails
    ready: true
    restartCount: 0
    state:
      running:
        startedAt: 2015-12-18T22:40:04Z
  hostIP: <IP_ADDRESS>
  phase: Running
  podIP: <IP_ADDRESS>
  startTime: 2015-12-18T22:40:00Z

Looks very related to #775

I suspect the lifecycle hook to be the culprit, the following block doesn't exist in a similar pod, and the metrics are exported properly.
        lifecycle:
          preStop:
            exec:
              command: ["/usr/sbin/nginx","-s","quit"]

Is this a bug? I'm also having the same issue and once the lifecycle block was removed stats were correctly published. But I need the lifecycle hook to be present in the configuration.

@piosz closing this as well, see #775


Fix couple of issues identified by Mark Hong

memory leak in picker.cpp
stream not explicitly closed in GVRAsynchronousResourceLoader.java

This patch looks good. +1
If Mesh allocations can be saved, that will be great, but it also can be done in a separate patch.


config.Optional always returns the default for Boolean values

https://github.com/reddit/baseplate/blob/master/baseplate/config.py#L307
If the value from the config is False this check is falsy and the default will be returned.

That check is working on the raw string, not the already-parsed version, so truthiness should just be if the string is empty or not. AFAICT this does work as intended with booleans:
>>> Optional(Boolean, default=None)("")
>>> Optional(Boolean, default=None)("True")
True
>>> Optional(Boolean, default=None)("False")
False

Do you have an example of it failing?

Arg. I was accidentally editing files in the debian directory so the config changes I expected weren't happening. Sorry for the false alarm.


fix: fix lexical declaration in case clause, type of acquireToken, ad…

Hello!
I have searched the issues of this repository and believe that this is not a duplicate.
Version
@azure/msal-browser
2.30.0
Environment
vue
3.2.37

plugins/msalPlugin.ts
String 63: There is a lexical declaration in the "case" clause. It can be a problem because the lexical declaration is visible in the entire switch block but it only gets initialized when it is assigned, which will only happen if the case where it is defined is reached.
I suggest adding a block in this "case":

case EventType.ACQUIRE_TOKEN_FAILURE: {
                    const currentAccounts = msalInstance.getAllAccounts();
                    if (!accountArraysAreEqual(currentAccounts, state.accounts)) {
                        state.accounts = currentAccounts;
                    }
                    break;
                }

composition-api/useMsalAuthentication.ts
String 6: acquireToken: Function
The Function type accepts any function-like value
I suggest explicitly define the function shape
acquireToken: (requestOverride?: PopupRequest|RedirectRequest|SilentRequest) => Promise<void>;

composition-api/useMsalAuthentication.ts
String 68:   if (!result && !error) {
result and error are Refs, and to access the values we should use result.value, error.value
I suggest: if (!result.value && !error.value) {

Thanks! Happy coding!

@microsoft-github-policy-service agree


add search and query support for subscriptions table

https://ui.shadcn.com/examples/tasks

solved with d4dd5ea


BUG in plotting timeseries data with twinx (different data representation on each ax)

Simplified the example.
During issue reporting i updated my pandas version from 0.13 to 0.18.1 - with version 0.13 i was able to add the whole DataFrame to my plot, only after i selected some rows between dates it produces the error. After the update the behaviour became worse - also adding the whole DataFrame to the plot now produces the error.
A small, complete example of the issue
import pandas
import dateutil.parser
import matplotlib.pyplot as plt

p_vals = {
    'x_vals' : [
        "2006-12-17 00:00:00+01:00",
        "2006-12-18 00:00:00+01:00",
        "2006-12-19 00:00:00+01:00",
        "2006-12-20 00:00:00+01:00",
        "2006-12-21 00:00:00+01:00",
        "2006-12-22 00:00:00+01:00",
        "2006-12-23 00:00:00+01:00",
        "2006-12-24 00:00:00+01:00",
        "2006-12-25 00:00:00+01:00",
        "2006-12-26 00:00:00+01:00",
    ],
    'y_vals' : [
        10,9,8,7,6,5,4,3,2,1
    ]
}

p_vals2 = {
    'x_vals' : [
        "2006-12-17 00:00:00+01:00",
        "2006-12-18 00:00:00+01:00",
        "2006-12-19 00:00:00+01:00",
        "2006-12-20 00:00:00+01:00",
        "2006-12-21 00:00:00+01:00",
    ],
    'y_vals' : [
        1,2,3,4,5
    ]
}

p_vals['x_vals'] = [ dateutil.parser.parse(x) for x in p_vals['x_vals'] ]
p_vals2['x_vals'] = [ dateutil.parser.parse(x) for x in p_vals2['x_vals'] ]

df = pandas.DataFrame(data = [1,2,3,4,5], index=["2006-12-17","2006-12-18","2006-12-19","2006-12-20","2006-12-21"])
df.index = pandas.to_datetime(df.index, format="%Y-%m-%d")

fig, ax1 = plt.subplots()
ax2 = ax1.twinx()

ax1.plot(p_vals['x_vals'], p_vals['y_vals'], color="r")
#ax2.plot(p_vals2['x_vals'], p_vals2['y_vals'], color="b") # works as intended, see second attached image
df.plot(ax=ax2, color="b") # hides data on ax1, see first image

plt.show()

Expected Output
Output of pd.show_versions()

INSTALLED VERSIONS
------------------
commit: None
python: 3.4.3.final.0
python-bits: 64
OS: Linux
OS-release: 3.19.0-69-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
pandas: 0.18.1
nose: 1.3.1
pip: 8.1.2
setuptools: 3.3
Cython: None
numpy: 1.11.1
scipy: 0.13.3
statsmodels: None
xarray: None
IPython: 1.2.1
sphinx: None
patsy: None
dateutil: 2.5.3
pytz: 2016.6.1
blosc: None
bottleneck: None
tables: 3.1.1
numexpr: 2.2.2
matplotlib: 1.5.2
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: 4.2.1
html5lib: 0.999
httplib2: None
apiclient: None
sqlalchemy: None
pymysql: None
psycopg2: None
jinja2: None
boto: None
pandas_datareader: None

Pandas 0.13 is quite old, can you try with a more recent version? Also see if you can simplify your example a bit.

also the x label look strange.

What do you mean by strange?

i just updated to pandas: 0.18.1 - sry for not trying with updated panda version
now both of my plot calls hide the data on the first axis.
sorry for my bad wording - by strange i meant, just, that is looks different

@cygenb0ck Can you try to simplify the example? Eg try to make it reproducible by not having to read a csv file but just create the data with code. Also try to remove other things that are not essential to the problem as much as possible.

@jorisvandenbossche
simplified the example and changed the subject

@cygenb0ck Thanks a lot! That let me look at it, and it's a bit a gotcha with the dates.
To start, it's not an issue with the twinx. Eg if you try the following similar example (but without using datetimes), you will see it works as expected:
fig, ax1 = plt.subplots()
ax2 = ax1.twinx()

ax1.plot([1,3,2], color="r")

df3 = pd.DataFrame({'col': [2,5,1]})
df3.plot(ax=ax2, color="b")
# df3['col'].plot(ax=ax2, color="b") # to plot one column not full dataframe

The reason it does not work with the example data you gave, is not because the plot is overwritten, but because the data on the first ax now fall outside the visible plot (if you zoom out enough, you will see both lines). This is because the dates are handled differently in the two cases.
The reason for that is a problem in pandas' plotting machinery to combine both irregular and regular time series in one plot (and because your data on ax1 have hours (although daily freq), they are regarded as irregular, the data on ax2 are regular). Related issues are #6608, #9053, #13341. We should definitely solve this ...
However, in this case it seems also specific to using twinx, as not using this does also solve the issue (then the second data are plotted fine).
Workaround you can use for now is by also plotting on ax2 with the matplotlib plot call:
fig, ax1 = plt.subplots()
ax1.plot(p_vals['x_vals'], p_vals['y_vals'], color="r")
ax2 = ax1.twinx()
ax2.plot(df.index, df[0].values, color="b")

Apparently, using x_compat=True is also a way to get this working:
fig, ax1 = plt.subplots()
ax1.plot(p_vals['x_vals'], p_vals['y_vals'], color="r")
ax2 = ax1.twinx()
df.plot(ax=ax2, x_compat=True, color="b")

It's mentioned in the docs: http://pandas.pydata.org/pandas-docs/stable/visualization.html#suppressing-tick-resolution-adjustment (although for another reason, I am not that familiar with this keyword)

@jorisvandenbossche
thank you very much for the workaround with x_compat=True. I can finally plot my data and continue my project.

This was only partly closed by #14330 (this example still does not work when first plotting the irregular series, #14330 added the test but commented it out)


force:source:deploy returns "No results found" Due To Changes Of sfdx-project.json

Summary

Because 2nd gen packaging still lacks support for many metadata types, some of my repositories use a dedicated "deploy" folder
This folder contains source that I need to deploy after the package install, using force:source:deploy
Recent changes in the CLI and how the sfdx-project.json works now require the src/deploy folder to be part of the packageDirectories. Otherwise, force:source:deploy only returns "No results found" (as if the directory was empty)
However, if I would add the source, this would result the source being included in force:source:push and force:source:pull commands - which would cause all scripts and pipelines to fail (because metadata like workflow email alerts using org wide email addresses are still incompatible with 2nd gen packaging)

Steps To Reproduce:

Use any repository you have and create a source folder, that is not part of sfdx-project.json
run force:source:deploy -p new/source/folder
See, "No results found"-Message
Now, add the folder to your sfdx-project.json and run the command again
See, that source is deployed

Expected result
force:source:deploy should not require the directory to be named in the sfdx-project.json.
Actual result
Because force:source:deploy requires the directory in sfdx-project.json, I can effectively not maintain "non packaged metadata" in source format in my repositories and deploy it.
Additional information
SFDX CLI Version(to find the version of the CLI engine run sfdx --version):
sfdx-cli/7.74.1-32db2396ed darwin-x64 node-v12.18.3
SFDX plugin Version(to find the version of the CLI plugin run sfdx plugins --core)
@oclif/plugin-autocomplete 0.1.5 (core)
@oclif/plugin-commands 1.3.0 (core)
@oclif/plugin-help 3.2.0 (core)
@oclif/plugin-not-found 1.2.4 (core)
@oclif/plugin-plugins 1.9.0 (core)
@oclif/plugin-update 1.3.10 (core)
@oclif/plugin-warn-if-update-available 1.7.0 (core)
@oclif/plugin-which 1.0.3 (core)
@salesforce/sfdx-trust 3.4.3 (core)
alias 1.1.1 (core)
analytics 1.12.1 (core)
config 1.1.8 (core)
generator 1.1.3 (core)
salesforcedx 49.10.0 (core)
├─ templates 49.4.4 (core)
├─ custom-metadata 1.0.10 (core)
├─ @salesforce/sfdx-plugin-lwc-test 0.1.7 (core)
├─ salesforce-alm 49.11.0 (core)
└─ apex 0.0.9 (core)
sfdx-cli 7.74.1 (core)
texei-sfdx-plugin 1.7.0
OS and version:
Mac

Thank you for the detailed bug report.  This fix is queued for the next patch release and a dupe of #622


Book 1. 20220808. Section 3.3.4.

That equality does not hold.
Consider the univariate standard normal distribution and N=4. Suppose the sample is {-1, -1, 1, 1}. Then the joint probability density equals N(1 | mean=0, var=1) ^ 4. The probability density for the sample mean (-1 + -1 + 1 + 1) / 4 = 0 is N(0 | mean=0, var=0.25).
Grab a calculator https://www.danielsoper.com/statcalc/calculator.aspx?id=54, we have that N(1 | mean=0, var=1) ^ 4 is about 0.24 ^ 4 and N(0 | mean=0, var=0.25) is about 0.79. Obviously, they differ.
Intuitively, the probability of a sample mean collects the probability of all possible samples having this sample mean value, and should be larger than the probability of any single such sample.

Same issue in section <IP_ADDRESS>.

Also, that N should be an exponent rather than a numerator.

Missing period.

Re 3.65. I meant to write proportional to, not equality. See https://github.com/probml/pml-book/issues/512

All fixed, thanks.


Round values down in the front-end only. Keep raw values in database.

This PR serves as to round values down to the specified rounding attribute attached to a certain <input/>.
Fixes: #1311

I like this a lot. One small niggle though: some numbers are displayed with extra trailing zeroes (two significant figures in the example below) and then change to one when the input is active:

Inactive
Active

It would be nice if these were consistent: if the inactive number is showing two significant figures then the active version should also show at least two figures. A better alternative might be for the inactive (rounded) version to be rounded to one significant figure in this case (10.50 → 10.5), since the extra zero doesn't serve any purpose.

A better alternative might be for the inactive (rounded) version to be rounded to one significant figure in this case

I agree however I showed it to @ChaelKruip like that and he disagreed due to the lack of consistency. This is only showing the significant values:

Some of these numbers have (,0 or ,045435) which all boils down to nothing.

I agree however I showed it to @ChaelKruip like that and he disagreed due to the lack of consistency.

I can't remember disagreeing (although I might have just to be difficult) but am now totally fine with not showing insignificant zero's. Everybody happy?

Functionality looks good, but I'm seeing three things which are problematic:

If you add a new technology to the form, the values aren't rounded (to reproduce: 1. Visit "Edit LES" -> "Technologies", 2. Add a battery to the form).

There are inputs with data-round attributes which shouldn't have them:

type
node
profile_key
components
associates
composite_index
...

Performance when setting up the form is not good. A LES with 24 technologies spends 260ms just running new Rounder().initialize. See: Before and after (that each consists of a load of calls to new Rounder().initialize). A LES with 100+ techs takes more than a second to set up Rounders.
As far as I can tell, the main culprit is the time it takes to call off() and on() to set up events on all of the input elements. I created a version avoids this using event delegation (only one event is created on the parent form element). However this doesn't work correctly: whenever you blur one of the inputs the value is reset to the original value.
I'm guessing there's an onChange on the input, which parses the form to JSON and that's getting called before the delegated writeRaw and round events.
I'm not sure what to do about this. I think taking a second to set up Rounders is rather excessive, but the alternative is to change all events on this form to use delegation. Switching to delegation would be a good idea since performance of the form initializer is pretty bad, but I expect that's quite a large task. Delegation also partially fixes problem 1, since you don't have to bind focus/blur/change events for the new fields (but the initial set-up is still required).
A stop-gap solution might be to wait until the visitor focuses a field before setting up the change and blur events.
input.on('focus', function() {
    if (this is the first time the input is focused) {
        // set up change event
        // set up blur event
    }

    round(input);
}

Thanks for all the input @antw! Lots of it is fixed in the optimalizations commit however I still need to check if it performs well in older browsers. Also I think I can make it just even faster but I need to check. 👍
This is the current profile:

Todo still:

[ ] check slider of congestion
[ ] check Safari
[ ] chekc IE

Great changes @grdw. I can't even find the Rounder in the timeline anymore, so that's a huge improvement. I'll merge this now...


[App Configuration] Configure the configs to use while running the app for development

Describe the enhancement

Quest support loading of application configs from the assets folder.  This makes it easier to test changes in configs immediately instead of having to push them to the server and then sync them down.
We however, have to keep switching the configs based on the different apps we build based on Quest.
We would like to make it easier for developers to target a different folder in the filesystem to load their configs from.

Implementation

[ ] Add the ability to define the configs folder on the local.properties file.
[ ] Configure the Application to use the configs on the route defined on the local.properties file.

[ ] In case the file route is missing we should then default to loading the sample configs on the assets folder.

[ ]  When using the app/debug the application should use local configs on any of the local folders as defined above.

Acceptance Criteria

[ ] Developers should be able to define and use a config route used on the local.properties file.
[ ] In case the above config is missing the code should use the sample configs on the assets folder

@dubdabasoduba Some devs have been using symlink to link the fhir-resources directory to the assets/configs/app. Do we have documentation @f-odhiambo on how this is done?

@dubdabasoduba Some devs have been using symlink to link the fhir-resources directory to the assets/configs/app. Do we have documentation @f-odhiambo on how this is done?
You need to add something like
ln -s pwd/hisp-wdf-fhir-resources/app fhircore/android/quest/src/wdf/assets/configs/app
to link your code to app folder

Marking this as DONE


Change log: Include explanatory text when an action is logged but a rationale is not required

Background: In our change log, many of the actions we log require rationales, but some do not. And in some cases, rationales are/are not required for a single action, depending on certain conditions being met. For example, we only capture rationales for indicator deletion when there are linked results.
Story
When I see an action in the change log without a rationale, I know whether or not a rationale was actually required, and if not, why not.
Acceptance criteria
Will put scenario-specific messages here. Some may be harder to implement than others, depending on the complexity of the conditions.

Actions we are logging without rationale prompts
What to say in the change log Rationale column

Delete an indicator with no linked results
A rationale is not required when deleting an indicator with no linked results.

Remove all targets from an indicator that has no linked results
A rationale is not required when altering targets for an indicator with no linked results.

Remove a single event target from an indicator that has no linked results
A rationale is not required when altering targets for an indicator with no linked results.

Modify a field in the targets tab of an indicator with no linked results
A rationale is not required when altering targets for an indicator with no linked results.

Modify the name of an indicator with no linked results
A rationale is not required when altering the name of an indicator with no linked results.

Housekeeping (where applicable)

[ ] Unit test/s written
[ ] i18n strings included
[ ] Is accessible to people using screen readers

@hannalcamp Can you please fill in whatever you can, and then reassign to me and tag me in a comment? Thanks!

@jennyhavah Fleshed out!
One note: We always ask for a rationale when changing the start/end date of a program even if there are no indicators, let alone targets set up or results reported. This seems like overkill, but not a big deal yet and one we could get feedback on.

Thanks, @hannalcamp!
@camgnostic As we were hoping, all of the logic is around whether or not there are linked results. So my understanding is that this would be a pretty trivial thing to add.
If yes, please feel free to grab it out of Deferred and into Doing. I will leave it in there for now because it's definitely not a release blocker.

@jennyhavah  The table above lists a different message for each type of non-rationale-requiring change.  Would it be sufficient to just have a generic message in the rationale field, something like "A rationale is not required for this type of change"?  We will already be displaying the fields that are being changed, so reiterating that change in the rationale message might be unnecessary?  And it would complicate the code a bit because you would have to determine exactly which kind of change was made.

@sanjuroj Having a generic message was our original thought, but it doesn't seem like much of an improvement over leaving it empty, and might just generate more confusion... Because why would a rationale be required for this target change and not that target change? The explanation clarifies that it's because results are not linked.
Cameron thought that as long as the logic used to determine which message appears is fairly straightforward without a million variances, this would be relatively easy to implement. But it's definitely deferrable, which is why this is in Deferred. :-)

@hannalcamp I crossed out the last row because it turns out that it would require substantial additional effort to start tracking and logging changes to all fields, regardless of whether or not we prompt. Since we don't actually care about logging changes to every field, it doesn't seem worthwhile to put in that extra effort.


Constraint names

Before merging:

[ ] Address remaining conversations
[ ] Address to-dos in #1869 issue

Thanks for the reviews @rahul3v, but that is probably more something to do via a new PR - as this one will be have to be rebased and probably be recreated against main before it can and will be merged - none of these changes should be in here.

Well @janpio , I guess I have to re create the PR once merged,  with all this changes then :)
or you can make it as you say !


Clicking a newer popup bar button kills old popup bars

VSCode Version: 1.19
OS Version: Linux (probably repros with all OS's).

Steps to Reproduce:

I installed the C/C++ extension on a new Linux machine.

Bug: Our extension showed the important "Reload" popup, but afterward VS Code showed the unimportant popup regarding something like "Do you want to receive recommended extension" and I clicked no, but that killed the Reload popup that I didn't want to kill. Older popups that come from different sources should not be closed by newer popups (maybe this would be fine if the popup was from the same source).

Reproduces without extensions: Yes

That should not happen. How does it look like, can you capture this as a screencast?

Is there a way I can trigger a VS Code popup again? I reproed it with the extension recommendation popups on a new machine, so I don't know how to get that popup to reappear.

Yeah maybe if you try the steps with insiders version again: You can give our preview releases a try from: https://code.visualstudio.com/insiders/

Unfortunately, installing VS Code on CentOS7 is a severe pain and gives "the file is not supported" when I try to download it. I was able to do various workaround to get it to work with the normal VS Code, but I can't remember the steps. It'd be nice if there would be a way to "reset" VS Code's stored state. uninstall/reinstall doesn't seem to do this.

I tried to repro on my Ubuntu VM with Insiders, but I could not repro it -- I think it was some strange race condition regarding involving the 2nd popup. I'll follow up if I ever get another repro.


Active rpc connections metric

Issue
Juno now has persistent connections (eg. ws conns) with not much info about them. It would be nice if It was known to user how many of them there are.
Implementation
Do you have ideas regarding the implementation of this feature? yes
Are you willing to implement this feature? yes
Scope:

Using gauge for tracking the amount of connections
Extending NewRequestListener with OnNewConnection(conn net.Conn) and OnDisconnect(conn net.Conn).
Implementing the logic in each rpc handler.

Additional Information:

OnNewConnection(conn net.Conn) and OnDisconnect(conn net.Conn) would be a nice place for uniform logging eg. ("Accepted connection", "conn", conn.RemoteAddr())
OnNewConnection(conn net.Conn) could be further extended into OnNewConnection(conn net.Conn) net.Conn which would enable tracking how much bytes were sent/recv.

Implementing the logic in each rpc handler.

We would only need to implement this in the websocket transport in the jsonrpc package, correct? No need to modify the higher-level rpc package handlers.

yes, I meant jsonrpc


Find references in expressions

Ex:
SomeObject obj = SomeObject.SOMETHING
Be able to find SomeObject on the right when trying to find references for this type.

Blocked on #85


plugin request: Expose ability to trigger project deploy via REST API

I know there is an existing web hooks plugin, but that appears to just send out messages after a build has been completed, is there a way to make a plugin that allows for triggering of builds on incoming messages?
I'd like new servers coming up to reach out to strider and trigger a deployment process

Not quite RESTful, but strider is already capable of running a job via a couple of API calls. The only problem, I believe, is that you need to get a cookie session started before you hit the endpoint that cna run a job.
The code that runs jobs form the CLI actually uses this exact method: https://github.com/Strider-CD/strider-cli/blob/master/run.js
Again, not quite RESTful, but if you were looking for a way within the existing code, this is how.

You can also set authenticate with a header on the request, in the format "Authorization: basic " + base64(username + ":" + password), per http://strider.readthedocs.org/en/latest/hacking.html
You post to /projectname/start to trigger a run. By default it's test only but you can specify it should deploy as well via params.
https://github.com/Strider-CD/strider/blob/master/lib/routes/api/jobs.js#L23 is the method for the API call.

^ Even better. Pretty sure that satisfies the OP's request precisely and bonus I learned something too :)

@paul42 does that satisfy your question/request?

This satisfies my request, thank you very much :)


Use common infinite gas abstraction for Haskell backend tests

With these changes, the ERC20 specs do not run out of gas, but get stuck evaluating some #hashedLocation rules.

@ttuegel I added some tests, and made a tweak to the rules for /Int. If you agree with the changes I made, feel free to add automerge.

@ehildenb Could you please advise me which part I should focus on?  Is there any change made other than copying or moving the set of lemmas to different files?

@daejunpark I just want another set of eyes on the lemmas now that they're being promoted to the top-level, and then any tests you would like to see please suggest. But if you're happy with the state of it, you can just approve.
The only difference is the added side-conditions that divisors are non-zero here.


Solution not being validated (APIs and Microservices Certification)

Describe your problem and how to reproduce it:
On the Managing Packages with Npm (APIs and Microservices Certification) after submitting the first challenge with the URL from gitHub, codePen of codeSandbox, I am not able to pass the test.
Add a Link to the page with the problem:
Links to my repo - https://github.com/tiagofsanchez/freecodecamp-apis-microservices
Why is that not working passing?
Tell us about your browser and operating system:

Browser Name: Chrome
Browser Version: Version 81.0.4044.129 (Official Build) (64-bit)
Operating System: Windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

These challenges expect to receive a full app, rather than the source code.  If you submit the 'live app' of your Glitch project it should work.
The forum is a better place to ask questions like this, though.  If the community concludes there's a bug, they will always suggest you create an issue here.

thank you @ojeytonwilliams.

No problem, good luck with the challenges.


Feature/tensor index tree fdb

Create a tensor index tree with compressed indices for 1D shapes and the grid axes which we know are compressable

Try to move logic of compressing into the tree instead of in the slicer


Adding an HTTP/2 integration fuzzer

Adding an HTTP/2 integration fuzzer that checks different kind of frames handling in both Downstream and Upstream
Signed-off-by: Adi Suissa-Peleg<EMAIL_ADDRESS>Description: A new fuzzer to test the HTTP/2.0 integration. At the moment it is slow, and should be improved.
Risk Level: Low - a new test
Testing: It is a new fuzz test

cc @asraa @htuch

Updated the test cases and works with the new libevent version

@htuch Please take a look, and let me know if there are other things that should be added/modified

@htuch I've addressed the latest comments, please take a look. Thanks!


Cambridge/index.html - Email us - should actually be a subscription

We need to maintain the list of registered users.
Either Google forms or mailchimp.com

Added mailchimp support


MCAS: Fix bucketing when on upper boundary, and lock graph scale to range definitions

Fix on https://github.com/studentinsights/studentinsights/pull/2713

selfie