Rewrite fill_fields_with_data_source to ease timeseries processing

Background
Part of
https://trello.com/c/oGNg8jGK/17-update-icu-hospital-data-from-daily-nevada-emails
involves changing how timeseries data that appear in multiple sources get combined.
This PR

Rewrite fill_fields_with_data_source to make it easier to handle entire an timeseries at once in a future change

Testing

Tests added in https://github.com/covid-projections/covid-data-model/pull/394 still pass and added a new one for empty case not covered
pytest test passes
Ran py-spy record --output combine.svg --rate 10 --subprocesses --native pytest test/combined_dataset_test.py at master and in this branch. Results were variable but 92-102s on my laptop. This did reveal that around 50% of the time is spent getting timezone information while panda builds indexes.

Please Check

[ ] Are the JSON schemas updated? n/a
[ ] Are the api docs updated? n/a
[X] Are tests passing?

Flame chart as an SVG: combine-flame.zip

Hmmm, not sure what to make of https://github.com/covid-projections/covid-data-model/pull/397/checks?check_run_id=664588534 so far....
=================================== FAILURES ===================================
________________ [pylint] pyseir/backtest/timeseries_metrics.py ________________
E:113,28: Function 'any' has no 'mean' member (no-member)
E:116,28: Function 'any' has no 'mean' member (no-member)

Hmmm, not sure what to make of https://github.com/covid-projections/covid-data-model/pull/397/checks?check_run_id=664588534 so far....
=================================== FAILURES ===================================
________________ [pylint] pyseir/backtest/timeseries_metrics.py ________________
E:113,28: Function 'any' has no 'mean' member (no-member)
E:116,28: Function 'any' has no 'mean' member (no-member)

but the same failure happens in master

I wonder if a dependency changed. I'll have to look at that code that's throwing the error

Overall looks great!
I think you'll have to change how this is called from combined_datasets correct? Just since I think it's calling the same function for both the latest values and timeseries datasets (unless i missed something)

Yes. I'll make another PR that changes how timeseries are combined. This PR aims to preserve existing behavior. An easy way to compare snapshots would be handy here.

Curious about the performance of the timezone indexes. Do you have a good idea of how long it took on master vs now?
CPU profile flames looked pretty similar in this branch and master.
combine-flames.zip

I started looking at the timezones but it turned into a rathole. My best guess right now is dates are not parsed consistently between data sources (maybe in some subtle way such as naive vs UTC timezone or ns vs D resolution) which slows down joins. I created https://trello.com/c/DiCcGWlR/134-factor-out-readcsv-date-parsing as tech debt and would like to merge this change because it is blocking getting Kentucky data in.

I'd like to get this unblocked because we're merging more data and I want more visibility into what is happening. I'm punting on performance for now, with expectation that we're about to get a bunch of low hanging fruit.


Ref: Organizing result handlers

Minor adjustments

coverage: 100.0%. remained the same
when pulling 6014f12dcd4f136fc0d89dc3961097cf5ab7baad on ref-result-handler-chores
into 775729a2339181f399fc4771c6c8e59778a47eef on master.


Unable to use install.sh in a Dockerfile

When using the below Dockerfile, leveraging the standard Install.sh installation script the docker image build process fails because the service manager doesn't seem to run:
WARNING  - /etc/service/redis_main/supervise: still not exists, check if runsvdir is running, start may fail.

...

CuisineCore.run.sv -w 60 start /etc/service/redis_main/|profile!True
    path: /opt/jumpscale8/lib/JumpScale/tools/cuisine/CuisineCore.py
DOC:
    @param profile, execute the bash profile first
ERROR:
    could not execute echo  | sudo -S bash -c ". /root/.profile_js && sv -w 60 start /etc/service/redis_main/",OUT:
    warning: /etc/service/redis_main/: unable to open supervise/ok: file does not exist

Please advice.
FROM jumpscale/ubuntu1510
MAINTAINER<EMAIL_ADDRESS>
RUN apt-get update && apt-get install -y python3.5 curl

RUN grep $(hostname) /etc/hosts || sed -i "s/.1 localhost/.1 localhost $(hostname)/g" /etc/hosts

RUN cd /tmp; \
    rm -f install.sh; \
    install.sh; \
    curl -k https://raw.githubusercontent.com/Jumpscale/jumpscale_core8/master/install/install.sh > install.sh; \
    bash install.sh

Here's the solutions I guess:
https://github.com/Jumpscale/dockers/tree/build_test/js8/x86_64/4_ubuntu1510_js8


feat: ✅ [Issue-1] Add Navbar

🛠 Fixes Issue

Closed #1
👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[ ✔] My code follows the coding standards of this project.
[✔] This PR does not contain plagiarized content.
[ ✔] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

It's a manubar icon i have to change it or is it ok?
On Thu 16 Jun, 2022, 7:53 AM Apoorv Goyal, @.***> wrote:

@.**** requested changes on this pull request.
What are .svg files for? Use icons instead. Fontawesome is already
imported into the project.
—
Reply to this email directly, view it on GitHub
https://github.com/virtual-tech-school/youtube-clone/pull/18#pullrequestreview-1008414650,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARG77I6H7WKQBQIQ2BWXGXDVPKFZ5ANCNFSM5Y2QOQOQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

It's a manubar icon i have to change it or is it ok?
…
On Thu 16 Jun, 2022, 7:53 AM Apoorv Goyal, @.> wrote: @.* requested changes on this pull request. What are .svg files for? Use icons instead. Fontawesome is already imported into the project. — Reply to this email directly, view it on GitHub <#18 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARG77I6H7WKQBQIQ2BWXGXDVPKFZ5ANCNFSM5Y2QOQOQ . You are receiving this because you authored the thread.Message ID: @.***>

Fontawesome has both menubar and mic icons, use that instead of svgs
https://fontawesome.com/icons/bars?s=solid
https://fontawesome.com/icons/microphone?s=solid

ok I will change it

I fix all the issues. please review my code.

Looks good to me. @unnikrishnan2002 take a look once.

Yep, should I test it once in my system ?


Add git.showCommitInput config option (#79074)

Adds feature requested in #79074.
To implement this, I mostly followed the pattern for the 'git.branchSortOrder' option and used VS Code's SourceControl API to toggle visibility of the input box.

Thanks! 🍻


Example tab disappears on hot-reload

I'm unclear if this is a result of my wacky setup, but whenever vue-play hot-reloads, the previously selected scenario will always be missing its example tab. A hard reload of the root URL fixes the issue until the next hot-reload.

I ended up switching back to serving my play files "the hard way" and that seems to have fixed the issue.


[6.0 🍒][Compile Time Constant Extraction] Ensure macro-expanded declarations are visited in WMO

Cherry-pick of https://github.com/swiftlang/swift/pull/74708
Explanation: When support for macro-expanded declarations was added to ConstExtract, it only ever applied to single-file compilation on a per-primary basis. Which means that the same code, when compiled in WMO, did not handle macro-generated extension declarations. This change brings WMO code-path in line with the per-primary extraction code path.
Risk: Low. WMO compilation will now match what happens per-primary in single-file compiles.
Testing: Automated test added to the compiler test suite.
Resolves rdar://130036855

@swift-ci test


go/cmd/dolt/commands: Handle some verrs which were previously being discarded

I noticed some places where we were discarding some verr values. Looked quickly for open issues related to these and didn't find any, but the errors seem pretty important. The refactorings are maybe a little tricky so I appreciate as many eyes as I can get.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Same issue as with #522: the initial PR passed CI and I haven't updated it since, nor have any conflicts arisen. The CI failures seem random and spurious. In light of this, merging despite failing CI.


setters and getters for visual script

Godot version:
3.1.1
OS/device including version:
Ubuntu 18.04
Issue description:
It would be nice to have ability to use setters and getters in visual script.
My use case: I'm working in a team with an artist that's slowly increasing it's capabilities in using Godot as an editor. The main tool this artist is currently using to create some small 'custom logics' is actually an AnimationPlayer. In some scenes I'm giving simple way to run some logic from AnimationPlayer by changing variable value in inspector (for example, export var up_position:bool value of elevator).
Recently I wanted to introduce this artist to VisualScript, for the start to give her ability to create functions that are calling different animations from models she is providing... It was quite natural for her that she would like to expose such 'animation' to the inspector so she could use it from AnimationPlayer that's in parent scene

cc @swarnimarun

This is a work in progress for the next version, but is this what you mean?

Not exactly, this feature request was for visual script equivalent of setget keyword.
So in this case there is one custom variable ball_dir. I would like to have ability to link it to 'set' and 'get' functions that would be called when variable value is changed from outside of this class.

@kubecz3k I had gotten it working on my local branch, still haven't created the PR focusing on other tasks at the moment but I will push the changes soon enough... :)

very cool, thank you!


Squid 3.5.27 now fails audit security check

GOT REQUEST TO AUDIT SECURITY
Fetching vuln.xml.bz2: .......... done
squid-3.5.27_2 is vulnerable:
squid -- Vulnerable to Denial of Service attack
CVE: CVE-2018-1000027
CVE: CVE-2018-1000024
WWW: https://vuxml.FreeBSD.org/freebsd/d5b6d151-1887-11e8-94f7-9c5c8e75236a.html
1 problem(s) in the installed packages found.
DONE

Sure, we cannot control releases of third party fixes. The scanner is for your convenience. Will most likely be fixed in 18.1.3.

no worries, I just felt it was the right thing to do to make sure it was known. I wasnt expecting there to be a sudden rush to patch the vulnerability but perhaps a newer version can be added when its ready 👍

FWIW, the links with additional info can help mitigate or see the affected scope, sometimes it’s not a problem. E.g. DoS when proxy is only reachable from a trusted network. But haven’t had time to look at the CVEs yet myself.
I think 18.1.3 hits around Wednesday. It’s been a bit slower the last two weeks on our end due to the vacation season, but picking up the usual pace again soon.
Thanks for the heads-up. :)


Support update dataset

This pr is to support partial update of the dataset metadata.

Codecov Report
Base: 73.22% // Head: 53.65% // Decreases project coverage by -19.57% :warning:

Coverage data is based on head (40e71d7) compared to base (ff61236).
Patch coverage: 53.75% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2358       +/-   ##
=============================================
- Coverage     73.22%   53.65%   -19.57%
+ Complexity    10685     7220     -3465
=============================================
  Files           765      766        +1
  Lines         59451    59543       +92
  Branches       7285     7297       +12
=============================================
- Hits          43533    31950    -11583
- Misses        13522    24781    +11259
- Partials       2396     2812      +416

Impacted Files
Coverage Δ

.../com/github/ambry/account/MySqlAccountService.java
68.03% <0.00%> (-4.31%)
:arrow_down:

.../java/com/github/ambry/account/AccountService.java
10.52% <0.00%> (-50.59%)
:arrow_down:

...src/main/java/com/github/ambry/rest/RestUtils.java
60.50% <0.00%> (-31.35%)
:arrow_down:

...com/github/ambry/frontend/PostAccountsHandler.java
52.21% <0.00%> (-33.64%)
:arrow_down:

.../com/github/ambry/account/InMemAccountService.java
72.99% <0.00%> (-20.25%)
:arrow_down:

...rc/main/java/com/github/ambry/account/Dataset.java
57.50% <33.33%> (+0.35%)
:arrow_up:

...ava/com/github/ambry/account/mysql/AccountDao.java
81.39% <81.81%> (-3.94%)
:arrow_down:

.../github/ambry/account/mysql/MySqlAccountStore.java
91.66% <100.00%> (+0.75%)
:arrow_up:

.../java/com/github/ambry/account/DatasetBuilder.java
47.22% <100.00%> (-47.38%)
:arrow_down:

...in/java/com/github/ambry/router/CryptoService.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 413 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Extension 'mcrypt' is deprecated since PHP 7.1

I ran a PHP Compatibility test and got the following results
`FILE: XXXXXXX/wp-content/plugins/cloudflare/vendor/cloudflare/cloudflare-plugin-backend/src/SecurityUtil.php
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 1 LINE
16 | WARNING | Extension 'mcrypt' is deprecated since PHP 7.1 - use openssl (preferred) or pecl/mcrypt once available instead.
16 | WARNING | Function mcrypt_create_iv() is deprecated since PHP 7.1; use OpenSSL instead
--------------------------------------------------------------------------------------------------------------------------------------------`
I found this to be a long standing issue that hasn't gotten a lot of attention:
https://wordpress.org/support/topic/extension-mcrypt-is-deprecated-since-php-7-1/
Is this in the roadmap to be resolved? I noticed it's been 11 months since an update was pushed through.. What is the status of this plugin?
Thanks for your time,
Derek

Anyone? As it becomes an issue for more and more website owners: https://wordpress.org/support/topic/php-7-2-incompatibility-223/

@derekakelly look at the code.
https://github.com/cloudflare/cloudflare-plugin-backend/blob/master/src/SecurityUtil.php#L12
Your tool is reporting a false positive.
If random_bytes doesn't exist (which it does in PHP 7 and higher) then it falls back to PHP 5 functions.

Hi @jb510
I see that, but while testing, I was able to determine the plugin only works up to php 7.0, anything higher and it breaks my site.
Also, it's not compatible with WordPress 5+
https://wordpress.org/support/topic/my-email-to-cloudflare-support/

This issue has been open for months now and Cloudflare can't even deign to acknowledge the issue. Our company pays Cloudflare too much money to be ignored. Very poor customer service.

This is a non-issue and should be closed.
That is not to say there weren't issues with PHP 7.2, just this in particular is a false positive in the compat tool because it's flagging fallback code for PHP 5.

Thanks for opening this issue and providing the compatibility checker output.
I would agree, however, that this is particular issue can be closed since the compatibility checker is referring to a fallback code path that should not run whenever the preferred random_bytes function is available.
As the plugin itself mentions https://wordpress.org/plugins/php-compatibility-checker/

Please note that linting code is not perfect. This plugin cannot detect unused code-paths that might be used for backwards compatibility, and thus might show false positives.

We are also tracking and addressing individual PHP 7+ compatibility problems in separate issues.


Laravel 10.48.15 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v10.48.15 branch.
Before merging, you need to:

Checkout the shift-ci-v10.48.15 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:x: Shift was unable to run composer update. Please be sure to run composer update before merging to ensure your dependency changes are fully compatible and reflected in your composer.lock file.


feat: new command line args & command prompts for Google service file output & write debug symbol scripts to negate the use of "firebase_app_id_file.json"

Description
Allow the option of writing iOS, macOS & android Google service file to a specified directory. How it works:
Example:
flutterfire configure  --ios-out=/ios/GoogleService-Info.plist \
  --android-out=/android/google-services.json \ --macos-out=/macos/GoogleService-Info.plist

It uses the Flutter project as the root and adds the service file to the path and file name specified in the CLI options. The above example would add the service files to the root of each platform folder:
cool_flutter_project/ios/GoogleService-Info.plist
cool_flutter_project/android/google-services.json
cool_flutter_project/macos/GoogleService-Info.plist
Implementation
android
There isn't much to do for android except to write the file. according to the multi-environment documentation, you simply need to write the service file to the correct directory (e.g. android/app/src/development/google-services.json).
FFCLI also writes to the android/build.gradle & android/app/build.gradle which are used across all builds & flavors.

iOS & macOS

[x] what to do about writing the service file for iOS considering the multiple types of setup.

The approach I'm taking is to prompt the user to pick whether to write the service file for a scheme, target or to simply write to a path without any further configuration.

Scheme
Find schemes available and add a build phase script that adds the file to the app bundle. You can have multiple scheme configurations and it will still add to the app bundle. For instance, suppose you were using VGV CLI, and you chose to add a service file to the development scheme. It will include the service file the app bundle for the following configurations Release-development, Staging-development & Debug-development.

Example of script added to build phases:
#!/bin/bash

PLIST_DESTINATION=${BUILT_PRODUCTS_DIR}/${PRODUCT_NAME}.app
GOOGLESERVICE_INFO_PATH=/Users/russellwheatley/projects/devTests/service_testing/ios/some/GoogleService-Info.plist

# Copy GoogleService-Info.plist for appropriate scheme. Each scheme has multiple configurations (i.e. Debug-development, Debug-staging, etc).
# This is why we use *"scheme"*
if [[ "${CONFIGURATION}" == *"development"* ]];
then
    echo "Copying ${GOOGLESERVICE_INFO_PATH} to ${PLIST_DESTINATION}"
    cp "${GOOGLESERVICE_INFO_PATH}" "${PLIST_DESTINATION}"
fi

Target
This is more straight forward, a command prompt allows you to select the desired target to include the service file. It will then add it to the "Copy bundle resources" phase for that target.

Write with no further configuration
This command prompt will simply add the file to the desired destination, creating necessary directories and is for user who want to do their own config setup.

[x] what to do about firebase_app_id.json file write.

I've removed the need to have a firebase_app_id.json. The user now has the option to update their build phases with a script that will upload the debug symbols for Crashlytics for their setup (i.e. target or scheme). If the user selects a project setup that uses schemes for example, the CLI will write a bash script that will run for that specific scheme.
The example below is the script written if the user selects a "development" scheme script:
#!/bin/bash

# Run upload symbol script for appropriate scheme. Each scheme has multiple configurations (i.e. Debug-development, Debug-staging, etc).
# This is why we use *"scheme"*
if [[ "${CONFIGURATION}" == *"development"* ]];
then
    echo "Running firebase_crashlytics upload debug symbols script for 'development' scheme"
    $PODS_ROOT/FirebaseCrashlytics/upload-symbols --build-phase --validate -ai '1:361461320736:ios:6ecd45b6d0d3126f6f8045'
    $PODS_ROOT/FirebaseCrashlytics/upload-symbols --build-phase -ai '1:361461320736:ios:6ecd45b6d0d3126f6f8045'
fi

The below example is a target script. In this example, the target they selected is "Runner":
#!/bin/bash

# Run upload symbol script for appropriate target.
if [ "${TARGET_NAME}" == "Runner" ];
then
    echo "Running [firebase_crashlytics] upload debug symbols script for 'Runner' target"
    $PODS_ROOT/FirebaseCrashlytics/upload-symbols --build-phase --validate -ai '1:361461320736:ios:6ecd45b6d0d3126f6f8045'
    $PODS_ROOT/FirebaseCrashlytics/upload-symbols --build-phase -ai '1:361461320736:ios:6ecd45b6d0d3126f6f8045'
fi

Here's an example of the scripts written when using "schemes" setup. They will only be written once for each flutterfire configure call:

Notes

macOS "scheme" setup which bundles the 'GoogleService-Info.plist' won't work at the moment because there's a code signing issue. It does work for "target" setup.
If you do test this feature, and you choose to add the "crashlytics upload debug symbol" script, you will need to delete the ios/firebase_app_id_file.json & macos/firebase_app_id_file.json because the script is now added in the build phases.

closes https://github.com/invertase/flutterfire_cli/issues/14
Type of Change

[X] ✨ feat -- New feature (non-breaking change which adds functionality)
[ ] 🛠 fix -- Bug fix (non-breaking change which fixes an issue)
[ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 refactor -- Code refactor
[ ] ✅ ci -- Build configuration change
[ ] 📝 docs -- Documentation
[ ] 🗑 chore -- Chore

There isn't much to do for android except to write the file. according to the multi-environment documentation, you simply need to write the service file to the correct directory (e.g. android/app/development/google-services.json).

The example should be android/app/**src/**development/google-services.json, right @russellwheatley?

@creativecreatorormaybenot yes, I've updated the description 👍


Some questions about the code.

Hi,
In RecNet>>models>>caption_generator. py>>def forward_local_reconstructor(),
Why do you use decoder_hiddens = build_hiddens(decoder_hiddens, caption_masks) to calculate decoder_hiddens?
Instead of using decoder_hiddens directly ?
What can you change about decoder_hiddens by using caption_masks?
    def forward_local_reconstructor(self, batch_size, decoder_hiddens, caption_masks):
        def build_hiddens(_decoder_hiddens, _caption_masks):
            _decoder_hiddens = _decoder_hiddens.permute(2, 0, 1, 3)
            _decoder_hiddens = _decoder_hiddens.view(
                _decoder_hiddens.size(0),
                _decoder_hiddens.size(1),
                _decoder_hiddens.size(2) * _decoder_hiddens.size(3)) # B, T, H
            _caption_masks = _caption_masks.transpose(0, 1).unsqueeze(2).expand_as(_decoder_hiddens).type_as(_decoder_hiddens)
            _decoder_hiddens = _caption_masks * _decoder_hiddens
            return _decoder_hiddens
        decoder_hiddens = build_hiddens(decoder_hiddens, caption_masks)

Hi. Thank you for reporting this bug.
I thought that simply zeroing out the hidden states after "EOS" token will make the corresponding attention weights to be also zero. However, it will not gonna work.
Therefore, I modified the code so that the corresponding attention weights will be explicitly zero. Please check and comment if I got wrong again.

Hi,
I am very excited to see your new revised code. But I have a new question, and I need your help.
It's hard for me to understand why you want to add caption_maskswhen calculatingweightsin the local_reconstructorstage.
def forward(self, hidden, feats, masks=None):
     Wh = self.W(hidden)
     Uv = self.U(feats)
     Wh = Wh.unsqueeze(1).expand_as(Uv)
     energies = self.w(torch.tanh(Wh + Uv + self.b))
     if masks is not None:
         energies = energies.squeeze(2)
         energies[~masks] = -float('inf')
         energies = energies.unsqueeze(2)
     weights = F.softmax(energies, dim=1)

     weighted_feats = feats * weights
     attn_feats = weighted_feats.sum(dim=1)
     return attn_feats, weights

What I understand is that when calculating weightswe only need decoder_hiddensand hidden, not caption_masks.
What's caption_masks, please?I didn't get it.
Is caption_masksa word generated by a decoder?
Looking forward to your reply, thank you.

The caption_masks is used for masking meaningless hidden states that are generated from the decoder after it met <EOS> token because I padded a sentence with '' token after the <EOS> token. For example, a sentence "a man is riding a bike" becomes
a man is riding a bike <EOS> <PAD> <PAD> ... <PAD>

As the number of words in each sentence is different, we cannot get rid of those meaningless hidden states by hard-coding like "discarding all hidden states after the 6th word". Therefore, the number words in each sentence should be considered to only take into account those meaningful hidden states generated from the decoder after it got valid tokens (e.g. a, man, is, riding, bike).

Hi,
I have a question that really confuses me, and I think it might be a bug.
In the validation phase, we no longer need to get captionfrom a dataset such as MSVD. So why do you use 'caption_masks' to get 'feats_recon'?
utils.py >> def test():

    for b, batch in enumerate(val_iter, 1):
        _, feats, captions = parse_batch(batch)
        output, feats_recon = model(feats, captions)

caption_generator.py >> def forward():

        caption_masks = captions != self.vocab.word2idx['<PAD>']
        caption_masks = caption_masks.cuda()
                            ...
        if self.reconstructor is not None:
               feats_recon = self.forward_reconstructor(batch_size, decoder_hiddens
                                                                         , caption_masks)

caption_generator.py >> def forward_local_reconstructor():

          for t in range(self.decoder.feat_len):
               _, hidden = self.reconstructor(decoder_hiddens, hidden, caption_masks)
               feats_recons[t] = hidden[0] if self.decoder.rnn_type == 'LSTM' else hidden

The specific code is as follows:
caption_generator. py >> def forward():

  def forward(self, feats, captions=None, teacher_forcing_ratio=0.):
       batch_size = feats.size(0)
       vocab_size = self.decoder.output_size
       caption_masks = captions != self.vocab.word2idx['<PAD>']
       caption_masks = caption_masks.cuda()

       hidden = self.get_rnn_init_hidden(batch_size, self.decoder.num_layers, self.decoder.num_directions,
                                         self.decoder.hidden_size)
       outputs, decoder_hiddens = self.forward_decoder(batch_size, vocab_size, hidden, feats, captions,
                                                       teacher_forcing_ratio)
       feats_recon = None
       if self.reconstructor is not None:
           feats_recon = self.forward_reconstructor(batch_size, decoder_hiddens, caption_masks)
       return outputs, feats_recon

Yes, we don't need the ground-truth captions as we set the teacher_forcing_ratio=0 in the validation phase. I should have generated the caption_masks from the outputs of the decoder instead of the ground-truth captions. I'll modify it soon.
However, this would not have affected the training of the model, since it's an error in calculating the validation loss of the reconstructor. We just have seen a wrong validation error of the reconstructor.

Hi,
I don't understand why hiddens[t] = hidden[0] instead of hiddens[t] = hidden?
caption_generator. py >> def forward_decoder():
        for t in range(1, self.max_caption_len + 2):
            output, hidden, attn_weights = self.decoder(output.view(1, -1), hidden, feats)
            outputs[t] = output
            hiddens[t] = hidden[0] if self.decoder.rnn_type == 'LSTM' else hidden

The specific code is as follows:
    def forward_decoder(self, batch_size, vocab_size, hidden, feats, captions, teacher_forcing_ratio):
        outputs = Variable(torch.zeros(self.max_caption_len + 2, batch_size, vocab_size)).cuda()
        hiddens = Variable(
            torch.zeros_like(hidden[0] if self.decoder.rnn_type == 'LSTM' else hidden)
                .unsqueeze(0)
                .repeat(self.max_caption_len + 2, 1, 1, 1))

        output = Variable(torch.cuda.LongTensor(1, batch_size).fill_(self.vocab.word2idx['<SOS>']))
        for t in range(1, self.max_caption_len + 2):
            output, hidden, attn_weights = self.decoder(output.view(1, -1), hidden, feats)
            outputs[t] = output
            hiddens[t] = hidden[0] if self.decoder.rnn_type == 'LSTM' else hidden
            is_teacher = random.random() < teacher_forcing_ratio
            top1 = output.data.max(1)[1]
            output = Variable(captions.data[t] if is_teacher else top1).cuda()
        return outputs, hiddens

(1) I saved decoder hidden states into the variable hiddens where the shape of a decoder hidden state is (D=1, B=200, H=512). Therefore, the shape of the variable hiddens should be (self.max_caption_len + 2, D, B, H). The shape of the variable outputs should be (self.max_caption_len + 2, D, B, V=512). BTW, the code was hard to read. I'll modify this to be more readable.
(2) When we use LSTM as a decoder, the variable hidden will be a tuple of two tensors (HIDDEN_STATE, CONTEXT_STATE). When we use GRU as a decoder, the variable hidden will be a tensor HIDDEN_STATE. However, we only need the HIDDEN_STATE for our reconstructor's input. That's why I divide the cases into LSTM / GRU.

Thank you for your patient reply. Your explanation is very clear.
I'm wondering why you're calculating loss, why is reg_lambda = 0?
utils.py >> train() / test():
        cross_entropy_loss = F.nll_loss(output[1:].view(-1, vocab.n_vocabs),
                                        captions[1:].contiguous().view(-1),
                                        ignore_index=PAD_idx)
        entropy_loss = losses.entropy_loss(output[1:], ignore_mask=(captions[1:] == PAD_idx))
        loss = cross_entropy_loss + reg_lambda * entropy_loss

(1) The reg_lambda denotes the loss weight to entropy loss, which forces our model to predict the next word with high probability by minimizing entropy. It is a trivial technique to enhance performance. However, what I wanted was reproducing the performance of the original paper, I set it to zero for now. You can increase it and see the changes in performance.
(2) I also noticed that kind of increase in validation loss. I'm also curious about that point but just ignored it for now as the captioning performances (BLEU@4, CIDEr-D, etc.) are increasing. I'm sorry that I cannot give you some satisfactory answers.

(1) I saved decoder hidden states into the variable hiddens where the shape of a decoder hidden state is (D=1, B=200, H=512). Therefore, the shape of the variable hiddens should be (self.max_caption_len + 2, D, B, H). The shape of the variable outputs should be (self.max_caption_len + 2, D, B, V=512). BTW, the code was hard to read. I'll modify this to be more readable.
(2) When we use LSTM as a decoder, the variable hidden will be a tuple of two tensors (HIDDEN_STATE, CONTEXT_STATE). When we use GRU as a decoder, the variable hidden will be a tensor HIDDEN_STATE. However, we only need the HIDDEN_STATE for our reconstructor's input. That's why I divide the cases into LSTM / GRU.

Hi,
(1) I notice you say  The shape of the variable 'outputs' should be '(self.max_caption_len + 2, D, B, V=512)', but in your code it's outputs = Variable(torch.zeros(self.max_caption_len + 2, batch_size, vocab_size)).cuda().
I found that one parameter was missing. Is this an error?
(2) I don't understand why the variable hiddens has two tensors(HIDDEN_STATE, CONTEXT_STATE).
What is CONTEXT_STATE and what does it do? What does HIDDEN_STATE have to do with CONTEXT_STATE?


Does not honor LC_COLLATE or LANG

os: Linux 5.14.21-2-MANJARO  SMP PREEMPT Sun Nov 21 22:43:47 UTC 2021 x86_64 GNU/Linux
lsd --version: 0.20.1
echo $TERM: xterm-256color
echo $LS_COLORS:

Expected behavior
When LC_COLLATE=C or LANG=C, filenames should be sorted in ASCII order. (upper before lowercase filenames)
Actual behavior
When LC_COLLATE=C, or LANG=C, filenames are still sorted using en_US.UTF-8 where case makes no difference.

related https://github.com/Peltoche/lsd/issues/50

Further to the above, when locale/LC_COLLATE is set to en_US.UTF-8 or en_AU.UTF-8 (my locale), GNU ls behavior is to ignore the leading dot for hidden files/directories when sorting the output.
I prefer this way of sorting so lsd respecting locale would be ideal for me
Related to https://github.com/Peltoche/lsd/issues/187

is this thread confirming that --date parameter is not accepting "+%x" as should display date in OS's locale setting, which lsd   is only displaying as like LC_ALL=C?
which in my case, I had to write "year" "month" "day" in my own language, as like: 2023年2月22日 e.g.
date: "+%Y年%m月%d日 %H:%M"
in ~/.config/lsd/config.yaml


How can I get posts from accounts which I follow?

I want to get all media posts from account which I follow. Does this feature exist? If so how can I do that?

Here is the code snippet that I use to get the feed items.
        //Create client
        IGClient client = new IGClient.builder()
                .username("username")
                .password("password")
                .login();

        //Getting userinfo from username
        UserResponse usernameInfoRequest = new UsersUsernameInfoRequest("memes")
                .execute(client).join();

        //Getting user feed by providing User pk
        FeedUserResponse userFeed =
                new FeedUserRequest(usernameInfoRequest.getUser().getPk())
                .execute(client).join();

        //Iterating through user feed items
        userFeed.getItems().forEach(item->
        {
            //Get any details of the feed item.
        });

Hope it will solve your query.

Here is the code snippet that I use to get the feed items.
        //Create client
        IGClient client = new IGClient.builder()
                .username("username")
                .password("password")
                .login();

        //Getting userinfo from username
        UserResponse usernameInfoRequest = new UsersUsernameInfoRequest("memes")
                .execute(client).join();

        //Getting user feed by providing User pk
        FeedUserResponse userFeed =
                new FeedUserRequest(usernameInfoRequest.getUser().getPk())
                .execute(client).join();

        //Iterating through user feed items
        userFeed.getItems().forEach(item->
        {
            //Get any details of the feed item.
        });

Hope it will solve your query.

This works great, however, is there a way to like/comment those "items" on the for loop?


Please support MSYS2 MINGW64

cmake configuration is fine but compilation failed on the Alembic submodule.

Most of the submodules are more than 2 years old. I tried to compile the latest Alembic, still failed with the same errors as the 2 years old version included by your project.

https://github.com/alembic/alembic/issues/418


Fix variants, add tests

They were apparently quite broken...
before:
> unmarshall(marshall("vv", [ ['i', 42], ['s', 'foo'] ]), "vv")
[ [ [ [Object] ], [ 42 ] ], [ [ [Object] ], [ 'foo' ] ] ]

> unmarshall(marshall("v", [ ['ai', [1, 2, 3]] ]), "v")
Error: Invalid struct data

after:
> unmarshall(marshall("vv", [ ['i', 42], ['s', 'foo'] ]), "vv")
[ [ 'i', 42 ], [ 's', 'foo' ] ]

> unmarshall(marshall("v", [ ['ai', [1, 2, 3]] ]), "v")
[ [ 'ai', [ 1, 2, 3 ] ] ]

Thanks Matthijs , I really hope to find time soon to review all pending PRs on this project, was very very busy lately

Just letting you know that I no longer care about this PR. I wrote my own dbus module instead.

@mvduin Sorry to hear that.  I'm happy to add link to your module from readme here

@sidorares : can this be merged if the conflict is resolved? This is helpful for eslint, so I'd like to merge it before prettifying.

So obviously that didn't happen,  but I'd still like to see a fix for variants.  Can you review it @sidorares

@sidorares : so how about this one?


Add documentation as static website using mkdocs

PR Checklist

[x] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[x] If you've fixed a bug or added code that should be tested, add tests!
[x] If you've added or modified a feature, documentation in docs is updated

Description of changes
Currently, the documentation is not very accessible since it is only available as markdown files in the repository, as mentioned in https://github.com/unitycatalog/unitycatalog/issues/33. This PR proposes to turn the documentation into a static website with mkdocs.
---> Example documentation site can be found here

To reproduce locally, as also mentioned in the README, run:
python -m venv venv
. ./venv/bin/activate
pip install -r requirements-docs.txt
mkdocs serve

This PR also adds a workflow that triggers on each PR to verify that the documentation builds without errors.
We can decide later on how to deploy the documentation.

Ohh this is amazing. I was thinking of doing something with mkdocs as well. Let's me play around with it get back to you.

Ohh this is amazing. I was thinking of doing something with mkdocs as well. Let's me play around with it get back to you.

Cool! Feel free to let me know if you want me to make any changes, and feel free to make any changes on top of this PR as well :)

@fpgmaas - thanks for submitting this PR!  I think this all looks good, but can we GitHub workflows to a separate PR?
I'd like to get the docs building locally first & then brainstorm the production deploy plan.
I think we will want the docs on docs.unitycatalog.io.  What do you think?

@fpgmaas - thanks for submitting this PR! I think this all looks good, but can we GitHub workflows to a separate PR?
I'd like to get the docs building locally first & then brainstorm the production deploy plan.
I think we will want the docs on docs.unitycatalog.io. What do you think?

@MrPowers Thanks, I removed the deployment pipeline. I guess keeping the docs-validation pipeline makes sense? If not we can always remove that one of course!
To me docs.unitycatalog.io sounds good :) For now, I changed the site_url in mkdocs.yml to https://docs.unitycatalog.io, but we can always decide on that later and change it accordingly.

LGTM

Thank you @fpgmaas for doing this and @MrPowers for reviewing it. Merging this.


Presets: Creating Sub-Categories

When Customizing ID editor Presets and how they appear in the UI is it possible to create sub categories (a drop down menu of features within a drop down menu of features)?

This does sort of work, but it's not something we do in the current list and so there may be some odd issues if you try it.
For example, you can edit the landuse.json category to include category-golf:
    "category-landuse": {
        "geometry": "area",
        "name": "Land Use Features",
        "icon": "land-use",
        "members": [
            "category-golf",
            "landuse/residential",
            "landuse/industrial",
            "landuse/commercial",
    ...

Rebuild, and then you will get a preset list like this when you use iD:

The javascript code can handle it, but if you look closely you can see that the CSS styling is a little weird when you expand the "Golf" subcategory.  This is probably easy to fix, but since we haven't set the menus up this way, nobody has thought about improving the list styling for nested categories.


Bump to newest release of JupyterLab. Add splash screen.

Updating to the newest release of JupyterLab allowed us to remove templating. The allowed adding a loading screen that makes startup feel really fast and smooth.

Splash screen is cool!


Bad link to Merlin paper in README

It appears that the ssw9.net domain name has expired. So the link to the Merlin paper in the README.md (http://ssw9.net/papers/ssw9_PS2-13_Wu.pdf) fails.

ah oups yes :D thanks a lot

I would advise to use the ISCA archive link to the correct paper:
https://isca-speech.org/archive/SSW_2016/abstracts/ssw9_PS2-13_Wu.html

Thanks, done now!


feat: 新增微信支付分账接口

Codecov Report

Merging #736 (edd2077) into master (f72fd62) will decrease coverage by 0.18%.
The diff coverage is 41.93%.

@@            Coverage Diff             @@
##           master     #736      +/-   ##
==========================================
- Coverage   74.95%   74.76%   -0.19%
==========================================
  Files         108      109       +1
  Lines        5387     5418      +31
==========================================
+ Hits         4038     4051      +13
- Misses       1349     1367      +18

Flag
Coverage Δ

unittests
74.76% <41.93%> (-0.19%)
:arrow_down:

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

Impacted Files
Coverage Δ

wechatpy/pay/api/profitsharing.py
40.00% <40.00%> (ø)

wechatpy/pay/api/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

应当在这里增加对应的属性：
https://github.com/wechatpy/wechatpy/blob/f72fd62b31773b59286b85c872d9882caceb4fb9/wechatpy/pay/__init__.py#L30-L64

应当在这里增加对应的属性：
https://github.com/wechatpy/wechatpy/blob/f72fd62b31773b59286b85c872d9882caceb4fb9/wechatpy/pay/__init__.py#L30-L64

喔喔，忘了，马上加

@messense 修改了，辛苦Review一下吧～


Add fine grained incremental inputs for tasks and transforms

Instead of only being able to query all file changes, we introduce a new interface which allows querying individual input file property changes.
Replaces https://github.com/gradle/gradle/pull/8658.

@lptr PTAL!

@lptr PTAL!

LGTM.


Add copy markup button in documentation

To prevent long drag and copy for scheduler and repeater.

I'm so close! This works relatively well except it doesn't retain whitespace.
$('code').each(function() {
	var code = $(this).text();
    $(this).parent()
        .css('position', 'relative')
        .append(
            $('
super copy
')
                .click(function() {
                    window.prompt("Copy to clipboard: Ctrl+C, Enter", code);
                })
        );
});

Guys, I'd love to see Codepen & JSFiddle links for each example in the docs. Requiring users to figure out our source code for this entire file is a bad experience.
Perhaps we could figure out some way to separate all of the code (JS, CSS & HTML) for each of the examples into separate files which we utilize both for the docs and for feeding into JS Fiddles or Codepens or something. Even gists would be better than what we have.


Make MimeTypesTest more robust

Checklist

[x] the contributor license agreement is signed
[x] commit message follows commit guidelines
[x] tests are included

Description of change
MimeTypesTest has been giving us a lot of problems in GHA as of late. The problem was caused by a missing wait in getIcon() which was fixed in a previous PR. Turns out there were more missing waits after this happens, namely the one in 903461d before clicking restoreIcon (id="mei_ri"). I've also gone through MimeEditorPage.java adding missing waits before clicking and sending text to elements, for good measure (eb6be5e).
Before making those changes I was able to replicate the test failure locally by disabling caching and throttling the speed of my test browser and retrying a few times since it's intermittent. Testing with these changes I have yet to see it happen. Hopefully CI can verify that.

Just as I was about to start reviewing, the Legacy mode test failed. :disappointed:
I wont review just yet, let me know when you've had a look. (Don't forget, you can download the screenshots etc. from the GHA results.)

Gah! Still a problem with MimeTypesTest. How frustrating. Apparently it timed out waiting, so there must be something wrong still.

Okay it looks like it's attempting to upload the icon and failing silently, which is why the problem occurs on waiting for the new icon. :disappointed: Locally the upload happens fine so I don't know what's going on.
Attempting to fix up now.

When looking at the logs in #2571 it occurred to me that this is an area that recently also had EBP updates made. In the past we've had issues due to screen size differences to CodeBuild, is it possible that's the issue here?

When looking at the logs in #2571 it occurred to me that this is an area that recently also had EBP updates made. In the past we've had issues due to screen size differences to CodeBuild, is it possible that's the issue here?

Nope unfortunately - since it's still happening in this branch and we have a scrollToElement in here which deliberately scrolls it into view.

Well hey, that's one green build. Let's try that again.

The build for 846e5cbd5b2a5b58fbde3205caffb0b241477963 failed on FileIndexingTest which I never saw before. But it's good to see MimeTypesTest pass again. I will give one more try.

Is there any further work required in this PR ? If no, it's good to merge develop to this branch and run all the builds again.

Is there any further work required in this PR ? If no, it's good to merge develop to this branch and run all the builds again.


Cursor moves to end of input field during validation

Steps to reproduce

Create a date or time picker with mdp-open-on-click ="false so that you can edit by typing
Type a value with valid formatting, e.g. 07-10-2017
delete any character (other than the last one) so that the input is invalidated
the cursor will automatically move to the last position.

Essentially, say that someone has chosen the wrong month and wants to edit the value.
It is awkward to edit, because as soon as a character is changed, the cursor will jump to the end
Here's an example with the current version of the code in the demo page

thanks for looking at this guys :)
ps: I would be happy to give it a shot myself and submit a PR.
However, my angular skills are a bit rusty. If you could give me a bit of guidance and what to look for, or anything particular that you are using that deviates from standard angular form validation.
thanks again

Hi @jsancho, thanks for reporting the issue.
Your want to look at the mdpDatePicker directive. Probably this has something to do with the formatter updating the model value after every input even if it is the same value.
I figure this has something to do with the validation /

I just tried this with firefox in the demo file delivered with this package.
I changed line 44 in demo/app.js to
$mdpLocale.date.openOnClick = false;.
Started the local server and I was easily able to enter and correct input:

Update: I can reproduce the behavior in chrome

Oh I see, it could be a Chrome thing - I should have mentioned that.
For the record, I think the issue happens in the same way with the time picker.
thanks

I saw the same behavior when I was playing with intl-phone-number: the cursor is moved to the end when the model and/or (i don't remember) view value changes. The trick was to save then reset the cursor position when needed: https://stackoverflow.com/questions/43811020/formatting-on-onkeyup-causing-cursor-to-jump-to-end-of-input-field

So this is quite funny. I tried to reproduce the behavior in chrome to check your PR @MincedCutlet (thanks for that btw) but I could not reproduce it any more.
My specs:
Fedora 25
Browsers: Chrome 60.0.3112.78 & Firefox 54.0.
Maybe the new chrome version fixed the bug? What do your browsers say?

OS: Mac Yosemite
Browser: Chrome: 59.0.3071.115 was able to reproduce the issue.
Noticed there was an update, and after upgrading to 60.0.3112.78 noticed the issue was resolved.
Seems like it is resolved.

I can confirm that by upgrading from Chrome 59.0.3071.115 to Chrome 60.0.3112.78 this issue has been resolved.
Amazing work @MincedCutlet not only in providing a fix, but also in noticing the Chrome update. 👍

Alright, I'll close this then. Thanks for your input :)

Ok, I'll close this then. Thanks for your input :)


Layout inclusive

Some things for inclusiveness

@Doppen Can you first merge main into this branch? This branch is a bit behind, and I think it has some design changes already done in the other pull request.

Can you also move all text to the messages.json files? I can help with that.

@barbarah can you do a review?


Update USAGE.md

This addresses some of the shortcomings mentioned in #9. However, three of the existing sections are not updated by this PR and probably should be checked/updated as well - see TODO below.

remove "public timeline" example; no longer works
replace specific hostname with instanceHostname variable that is explained in Prerequisites
replace example client website with "example.org"
add section to get access token from user credentials directly
add section to post a status

TODO: following sections untouched/unchecked

Using OAuth
Get raw json
Streaming API

@bocops Thanks again for your contribution! Sorry was busy the last few days. Trying to catch up!


fix yaml format in compute optimizer

Issue #, if available:
Description of changes:
Fixes and cleanup for compute optimizer
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@yprikhodko Please can you merge this?


Adds npm wrapper actions

Adds two npm actions which stack on each other:

npm-setup which will install nodejs and npm dependencies
npm-run which will run an npm command after  performing npm-setup

All green!  Much better.  😊


docs: Add getting started docs for Ingress

This commit is to lift and shift docs in servicemesh beta repo to cilium
repo.
Relates to #19312
Signed-off-by: Tam Mach<EMAIL_ADDRESS>

Is there any Cilium team which should be kept in the loop for reviewing PRs touching these files? If so, you'd need to update CODEOWNERS accordingly.

I am not so sure if there will be a new team created (e.g. sig-servicemesh). If yes, definitely we should add CODEOWNERS for the same, otherwise, we can just use existing kubernetes team.
/cc maintainers @joestringer and Andre

we're declaring those features GA and assuming it's OK for users to start building & relying on those CRDs in their prod environments. Are we OK with that? Or should those features still be 'beta'?

Yup, this is as per my understanding, please find free to let let me know otherwise /cc @aanm

Thanks a lot for your detailed review and correction, this PR should be in better shape now.


fix(db-postgres): query hasMany fields with in

Description
fix https://github.com/payloadcms/payload/issues/4819

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[ ] Chore (non-breaking change which does not add functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Change to the templates directory (does not affect core functionality)
[ ] Change to the examples directory (does not affect core functionality)
[ ] This change requires a documentation update

Checklist:

[x] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

@DanRibbens
"All" still doesn't work btw


Tika rest for testing

Hi @chrismattmann  pulled request. So, you can test it......

Merged and going on test hanks @dongnizh !


Tags style tweaks

Gitea version (or commit ref): 1.8.0+rc3 / 1.9.0-dev
Git version: 1.12.4
Operating system:
Database (use [x]):

[x] PostgreSQL
[x] MySQL
[x] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Tags are missing bottom margin in project list.
Add new class to tags div that ensure this the list will have bottom margin.
Make Tags smaller (and / or maybe change style of labels) See example bellow
Screenshots

Current Situation

Update style:

Could you send a PR to fix that?

yes i can

Yes i can.

Thanks for PR 😄
Closing this issue as PR has been merged.


kirkstone-c1: Change to include kirkstone manifest

this would make our work easier when update kirkstone.xml it will automatic update c1 manifest.

Awesome!


Support for OSE v3.7

With Openshift Origin v3.7.0 just released it is time to update this cookbook again.
Checklist (to be completed):

[ ] Centos Package
[ ] Read release notes for the new version
[ ] Update this cookbook's LWRP and recipes

I have a special interest in this release because of persistent local volume feature which is supported since k8s v1.7 (see https://kubernetes.io/docs/concepts/storage/volumes/#local).

Hi, Just FYI, I'm working on speeding up the tests and making them more
reliable. I've been doing this by pre-cacheing a Vagrant build and
co-locating it with the regression test code:
https://github.com/ianmiell/shutit-openshift-cluster/blob/master/vagrant-box-create.py
This will make tests faster and more reliable.
Ian
On Mon, Dec 4, 2017 at 10:32 AM, Julien Pervillé<EMAIL_ADDRESS>wrote:

With Openshift Origin v3.7.0 just released
https://github.com/openshift/origin/releases/tag/v3.7.0 it is time to
update this cookbook again.
Checklist (to be completed):

Centos Package
Read release notes for the new version
Update this cookbook's LWRP and recipes

I have a special interest in this release because of persistent local
volume feature which is supported since k8s v1.7 (see
https://kubernetes.io/docs/concepts/storage/volumes/#local).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/IshentRas/cookbook-openshift3/issues/185, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGrczRUOA0CXRO4THsqQSCCxJ6w8n3K0ks5s88o5gaJpZM4Q0buv
.

--
My book on Docker in Practice: Get 39% off with the code 39miell2
https://www.manning.com/books/docker-in-practice-second-edition?a_aid=zwischenzugs&a_bid=550032fc

Just noticed that I've been running 3.7 in my tests without realising it. The tests we have seem to work... any way I can try and hit it with a hammer?

https://github.com/IshentRas/cookbook-openshift3/pull/196

@ianmiell @jperville I would like to update the helper so as to only install the relevant yum repo

#209 Has been created. I'll also add metrics/logging support (Once for all) + service catalogue. Should be able to merge stuff tomorrow and cut a release

@jperville @ianmiell I have nearly finished the refactoring of the code.
Once ok. we shall start looking at 3.9.x

Hello @IshentRas , now that your refactors have been merged, do you have ideas on how to implement proper support for the logging addon (we may be able to run a containairized version of the ansible playbook, same as "oc cluster up" does).
In my deployments the logging stack is still based on v1.4.1 + patches, it start to be rusty and I'm having some performance issues with the vendored elasticsearch 2.4.1 that I think would be solved by upgrading.

@jperville I am about to start coding some logic for logging, should be ready by next week :-)


Does not work with version 67-69(no streams)

Context
Search does not work, no streams. Without a proxy also does not work
Your Environment

Version used:
elementum 0.69, burst 0.33
Environment name and version:
Kodi 17.6
Operating System and version:
Windows 8.1 x64
Link to Kodi debug log:
HTTP PROXY ON
HTTP PROXY OFF

WARN: Cannot read TLS response from mitm'd server dial tcp: lookup rutracker.lib: no such host

Somehow it was not able to find IP for rutracker.lib.

WARN: Cannot read TLS response from mitm'd server dial tcp: lookup rutracker.lib: no such host

Somehow it was not able to find IP for rutracker.lib.

rutracker.org or another also does not work

Please, try v0.0.70.

Please, try v0.0.70.

Thank you very much! everything works great!


use microbundle for various browser + node library builds

@armaniferrante I won't open this on the main repo yet as I'm just messing around and testing things locally for now. But I think if you wanted to try https://github.com/developit/microbundle, we might? be able to solve two open issues in a single, small PR

https://github.com/project-serum/anchor/issues/3 import anchor with <script type="module"> (like https://www.skypack.dev)
https://github.com/project-serum/anchor/issues/244 fix issue with fs import breaking browser build

You'll notice that I'm also importing assert because it uses browserify/commonjs-assert for the browser build https://github.com/developit/microbundle/issues/224#issuecomment-443882119
I have done very basic testing with a react app locally and was able to import anchor with
import React from "react";
import * as anchor from "@project-serum/anchor"

console.log({anchor})

and the react app compiled, and console looked fine with no errors or warnings

it's late here now but I am happy to look into this a bit more this week if you think it would be helpful?
Thanks

This is awesome. Let's definitely investigate this. Can you educate me why one would use microbundle vs rollup directly? I only have experience with the latter.

Sure, the main reason I like it is because it's a single project dependency and zero-config with some nice defaults.
It's using rollup under the hood and was developed by some pretty smart people (creator of Preact). I'd totally understand if you'd rather set up your own build pipeline with vanilla rollup instead though :)

closing in favour of #2


Put all generated glTF JSON handlers into a single C++ source file, rather than one file per class

They're all generated together anyway, and compiling one source file rather than dozens saves a bunch of time.

@kring I haven't ran the build yet, but how fast is the compilation process after the combination? I have a mixed feeling about this as I still prefer separate files for easy to maintain and debugging even though the compilation is longer. But cesium-native has fast compilation time at the moment, so the trade off maybe not significant to combine them all?

Another thing I worry is that if the file is getting bigger in the future, some IDE and code editor may have a trouble to read and parse them (e.g sqlite3.c file). But that's probably not much of a concern compared to the maintenance above as the combined file is only around thousand lines

@baothientran in general I agree with you regarding maintenance, but in this case it's generated that code you're literally not supposed to touch. If you're maintaining it, you're doing it wrong. :) I haven't measured the time, but ~30 fewer source files to compile will be significant.

That's true about not touching the generated files, but I'm still uncertain about this due to maybe being unfamiliar with it. I may need some more  opinions from others too to decide on this. @IanLilleyT, @lilleyse, and @jtorresfabra, since our inner project may need the gltf, do you have any opinions on combining the generated json handlers into one file?

For the maintenance and debugging point, I guess it's still doable with one combined file, but still want to have more some advice before going forward

I don't see any downside, but 🤷.
Timing wise, I did this:
rm -recurse -force build/CesiumGltfReader
Measure-Command { ."C:\Program Files\CMake\bin\cmake.EXE" --build c:/Dev/cesium-unreal-samples/Plugins/cesium-unreal/extern/cesium-native/build --config Debug --target ALL_BUILD -j 14 -- }

So, just measuring the time to build CesiumGltfReader. It also triggers a CMake configure, which takes a few seconds.
In this branch, it reports about 16 seconds. In main, it reports about 32 seconds.

From my point of view, and as far as the documentation is clear on how to use it (examples, tests, README.md, you name it...) and we don't need to dive into the class I'm pretty ok with the change. :+1:

if everyone is okay with the change, then I will go ahead with the PR


upgrade to phpunit 6

This upgrades phpunit to version 6, which requires dropping support for php 5.
@komapa thoughts on this? php 5.6 was eol'd at the beginning of the year.

PHP5 is still the most used PHP version right now so I think we should keep the latest 5.x version. So I would still keep 5.6 as supported by our client (aka, we should not use PHP 7 only syntax and features).

I see why you did it phpunit/phpunit 6.5.9 requires php ^7.0 -> your PHP version (5.6.32) does not satisfy that requirement.
If there is no good reason to drop PHP 5.6, I would suggest we use PHPUnit 5 for the foreseeable future. Unless we  need some advanced PHPUnit testing (which I do not think we do here), there is no reason to let PHPUnit dictate the version the library can be used with.

IMO it would be nice to stop requiring changes to conform to any php versions that are EOL, but if you think we need to keep it, I can just hold off on adding cool new things ™.

Also note, if we bump php versions shat should be co sideref a major version bump, people on php5 would continue to be able to instal this current release.

Well that is bias, I think I saw a different graph recently which confirms what I said above, I can find it some time today but the point remains, we do not want to not allow PHP5 to use the client for  testing purposes.


uninitializedSlice

https://github.com/dlang/phobos/pull/4780 will be available for 2.072. As soon as LDC will have it too, the code can be simplified.

all sliced calls can be replaced with

Yep, had this planned. :-)

Will note you when this be a part for Mir

Thank you!

@ljubobratovicrelja v0.18.0-beta0

http://docs.mir.dlang.io/latest/mir_ndslice_slice.html#.uninitializedSlice
and http://docs.mir.dlang.io/latest/mir_ndslice_slice.html#.makeUninitializedSlice

since API function is not using GC anymore, this issue is not relevant


Error during runtime: Unrecognized URL format / URL is not defined

Hi all,
Tried to use destreamer 2.0.0 on Ubuntu 18.04.
I'm completely new to TypeScript so not quite sure why is this error showing up.
_____ub@____-ub-vm:~/destreamer$ ./destreamer.sh -i "https://web.microsoftstream.com/video/aa54_1a7-____-____-____-____________" -v
Using ffmpeg version 3.4.6-0ubuntu<IP_ADDRESS> Copyright (c) 2000-2019 the FFmpeg developers

Video URLs:
[ 'https://web.microsoftstream.com/video/aa54_1a7-____-____-____-____________' ]

Access token still good for 46 minutes.

Unrecognized URL format in https://web.microsoftstream.com/video/aa54_1a7-____-____-____-____________: URL is not defined

It seems to be coming from: https://github.com/snobu/destreamer/blob/087850821cb12241cdfeb914f6ce9ce82e07d198/src/destreamer.ts#L102-L117
Is there any possible mistake I made, or any try I can do?

Try to upgrade from Node 8.0 to 10, 12 or 13, or make the changes highlighted in this PR (https://github.com/snobu/destreamer/pull/79).

Oops..  was too anxious to ask, didn't notice the version of nodejs
It works perfectly now after I upgraded to Node.js to version 13.
Thank you a lot for helping me.

Thanks for your report, feedback is always appreciated over here.


Use html5ever for HTML link extraction

html5ever is a HTML specific parser, compared to previous XML parser:

Way more compatible with HTML constructs
Finds a lot more links
Doesn't require hacks or tweaks

TODO:

[x] Add tests for HTML5 quirks, e.g. issues uncovered in #83
[x] Use wildcards for attribute matching

I have checked this against examples provided by @Michalng in #83 -- it seems to be on par with plain text extractor now, and as an added benefit should work with relative links which the plaintext mode doesn't do.

I have checked this against examples provided by @Michalng in #83 -- it seems to be on par with plain text extractor now, and as an added benefit should work with relative links which the plaintext mode doesn't do.

Very nice work so far!

Very nice work so far!

Ready for review now!

Ready for review now!

Hope it's fine that I just went ahead and merged this. I'll tag a new release for it, 0.5.0. 🤖

Hope it's fine that I just went ahead and merged this. I'll tag a new release for it, 0.5.0. 🤖

Hope it's fine that I just went ahead and merged this. I'll tag a new release for it, 0.5.0. 🤖

Nice, thanks!

Hope it's fine that I just went ahead and merged this. I'll tag a new release for it, 0.5.0. 🤖

Nice, thanks!


Append a slash to tar directory entries

CDI detects the disk directory entry as a regular file due to checking
if the tar header points to a directory by checking if it is appended
with a slash instead of checking the tar type.
To make the images works with all all CDI versions as well as meeting
this expectation in general (most apps append a slash), adding a slash
to the directories here too.
An improvment to CDI is here: https://github.com/kubevirt/containerized-data-importer/pull/2050
Append a slash to the disk directory tar header to not confuse CDI

Thanks for locating the issue and posting the fix!

/lgtm

/approve


In the demo, 'Terrain' refers to the wrong source code

It refers to a github URL containing 'terraing' instead of 'terrain'.

Thanks, this is covered by #463 (so closing this).


Blog

Like to develop the blog page

@Siddharth1605 what is the use of blog page in a weather app

Sorry, there was some misunderstand from my side, I'll delete this issue


Bloodmagic swaps all @dynamic properties' implementatoins

Hi, I don't have a clear understanding of framework, because of lack of time, but as far as I understood,
BloodMagic implements on my own all properties that are marked as @dynamic, no matter if I added bm_injectable property attribute or not. This means that I can't use @dynamics for my own purposes and at the same time use BloodMagic. I wish I could.
Is there a way to fix it, or the core idea of framework realization is not compatible with my wish? Sorry, once again, I tried to answer my own question, but It would be less time consuming if you give me a hint.

That is correct.
At the moment there is no way to tell BM which properties to avoid.
The best way is to split the class into a few classes: one affected by BM, other - not affected.
P.S. bm_injectable is just an empty macro definition, it was intended to improve readability, though I hardly used it.

Ok, thanks.
I noticed that it's an empty macro definition, but I thought maybe I just don't know some compiler magic)


Introduce special decl names

After a long way we're finally here: The PR to introduce special names for subscripts and fix SR-2575.
Special DeclNames represent names that do not have an identifier in the surface language. This implies serializing the information about whether a name is special together with its identifier (if it is not special) in both the module file and the swift lookup table.

@swift-ci please test

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 8105cb6ddff4d68cf255525ecd410d4fedb2964d
Test requested by - @ahoppen

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 8105cb6ddff4d68cf255525ecd410d4fedb2964d
Test requested by - @ahoppen

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 3d1f71627f63b90c293e213c1f258a49b6652e78
Test requested by - @ahoppen

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 3d1f71627f63b90c293e213c1f258a49b6652e78
Test requested by - @ahoppen

Resolved merge conflicts
@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 710dd93a332ae1512a91171408fc06a8400c2afd
Test requested by - @ahoppen

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 710dd93a332ae1512a91171408fc06a8400c2afd
Test requested by - @ahoppen

@jrose-apple I've addressed all the comments in your last review and the PR is ready from my side. Could you review the changes if you've got time?

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - ded6aed0ee0d5d09a30f7e9ad627f764925a388e
Test requested by - @ahoppen

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - ded6aed0ee0d5d09a30f7e9ad627f764925a388e
Test requested by - @ahoppen


模型中step定义

你好，我想请问以下训练过程中的step和global step是如何定义的；
我目前有9106条数据，用了一张卡训练，每张卡上的batchsize是64，梯度累计步数是128：
global_step如果是：9106/128 = 71
step是：（9106/128）* epoch
但是终端上显示的是：（我尝试用tensorboard进行绘图时，发现他并没有保存图像）
andb:                    train/epoch ▁▃▄▆██
wandb:              train/global_step ▁▃▅▆██
wandb:                train/grad_norm █▆▃▁▁
wandb:            train/learning_rate █▆▅▃▁
wandb:                     train/loss █▇▂▃▁
wandb:               train/total_flos ▁
wandb:               train/train_loss ▁
wandb:            train/train_runtime ▁
wandb: train/train_samples_per_second ▁
wandb:   train/train_steps_per_second ▁
wandb:
wandb: Run summary:
wandb:                    train/epoch 4.51
wandb:              train/global_step 5
wandb:                train/grad_norm 2.49198
wandb:            train/learning_rate 0.0
wandb:                     train/loss 0.7259
wandb:               train/total_flos 0.0
wandb:               train/train_loss 0.75921
wandb:            train/train_runtime 137.7201
wandb: train/train_samples_per_second 330.598
wandb:   train/train_steps_per_second 0.036
为什么他的global_step 5？请问这个地方应该怎么计算？

Global step refers to the number of times the model has been updated: global_step = 9106/(64*128) * 5 = 5

Global step refers to the number of times the model has been updated: global_step = 9106/(64*128) * 5 = 5. Besides, the accumulation step is too large (128), and I think setting it to 1 is enough.

ok,  The setting of gradient accumulation steps is related to the batch size. It is not necessary to be larger than the batch size. The step how to calculated?

The step how to calculated?

I think step is equal to global_step


Is it possible to position leaf nodes on the same level?

Hello,
I am using graphview to create an organization chart for my company. I am now wondering if it is possible to position all leaf nodes (in my case individual teams) at the same level, regardless of how long the chain of managers above the team is.
I have considered (but not yet tested) using virtual nodes between the team and the manager to move the leaf node down X levels. Is there a better solution to my problem?
I have created a diagram to illustrate my goal:
Current:

Target:

try this:
Flip the source and target, AND set orientation as ORIENTATION_BOTTOM_TOP
i found this while trying to reverse the arrows with no avail. you can see an illustration in my reported issue:
https://github.com/nabil6391/graphview/issues/72

Try out 1.2.0 , added a new CoordinateAssignment to allow users to adjust alignment


Add support for Latvian keyboards

The Latvian keyboard layout is the same as English, which means that it's easy to support.
This will include:

A new keyboard locale
A new, localization file
A new KeyboardKit Pro input set provider
A new KeyboardKit Pro secondary input action provider

This can be tested in the master branch.


Double switch not working as expected on ESP8266

Hi,
Thanks for this project, it's GREAT :-)
I have an issue with a double switch i'm trying to make. When both are off in IOS home, and I turn on switch 1, switch 2 will also turn on. If both are off, and I turn on switch 2, only switch 2 will turn on as expected.
What can I be doing wrong?
Best regards,
Christian
/*
ESPHap example EspHapLed for ESP8266
This example for ESPHap shows how to control a LED
with Apple Home app. It implements accessory type "light bulb".
This example code is part of the public domain
*/
#include <Arduino.h>
#ifdef ESP32
#include <SPIFFS.h>
#endif
#include <ESP8266WiFi.h>
#include <ESP8266mDNS.h>
#include "coredecls.h"
const char* ssid     = "xx";
const char* password = "xx";
const int identity_led=3;
const int led_gpio = 2;
const int led1_gpio = 0;
extern "C"{
#include "homeintegration.h"
}
#include "homekitintegrationcpp.h"
homekit_service_t* hapservice={0};
String pair_file_name="/pair.dat";
//esp8266::MDNSImplementation::MDNSResponder* MDNS1;
LOCAL os_timer_t callbacktimer;
void setup() {
disable_extra4k_at_link_time();
Serial.begin(115200);
delay(10);
// We start by connecting to a WiFi network

#ifdef ESP32
if (!SPIFFS.begin(true)) {
// Serial.print("SPIFFS Mount failed");
}
#endif
#ifdef ESP8266
if (!SPIFFS.begin()) {
Serial.print("SPIFFS Mount failed");
}
#endif
Serial.println(ssid);

#ifdef ESP8266
WiFi.mode(WIFI_STA);
WiFi.begin((char*)ssid, (char*)password);
#else
WiFi.begin(ssid, password);
#endif
while (WiFi.status() != WL_CONNECTED) {
delay(500);
Serial.print(".");
}
Serial.println("");
// Serial.println("WiFi connected");
Serial.println("IP address: ");
Serial.println(WiFi.localIP());
pinMode(led_gpio,OUTPUT);
pinMode(led1_gpio,OUTPUT);
///setup identity gpio
// hap_set_identity_gpio(identity_led);    //identity_led=2 will blink on identity
/// now will setup homekit device
//this is for custom storaage usage
// In given example we are using \pair.dat   file in our spiffs system
//see implementation below
Serial.print("Free heap: ");
Serial.println(system_get_free_heap_size());

init_hap_storage();

set_callback_storage_change(storage_changed);

/// We will use for this example only one accessory (possible to use a several on the same esp)
//Our accessory type is light bulb, Apple interface will proper show that

// hap_setbase_accessorytype(homekit_accessory_category_lightbulb);
hap_setbase_accessorytype(homekit_accessory_category_switch);
/// init base properties
hap_initbase_accessory_service("","test","0","Fun","1.42");

//we will add only one light bulb service and keep pointer for nest using
//hapservice= hap_add_lightbulb_service("Led",led_callback,(void*)&led_gpio);
hapservice= hap_add_switch_service("AUX",led_callback,(void*)&led_gpio);
hapservice= hap_add_switch_service("CD",led1_callback,(void*)&led1_gpio);
//and finally init HAP
hap_init_homekit_server();

}
void loop() {
hap_homekit_loop();
//delay(2);
return;
}
void init_hap_storage(){
Serial.print("init_hap_storage");
File fsDAT=SPIFFS.open(pair_file_name, "r");
if(!fsDAT){
Serial.println("Failed to read pair.dat");
return;
}
int size=hap_get_storage_size_ex();
char* buf=new char[size];
memset(buf,0xff,size);
int readed=fsDAT.readBytes(buf,size);
// Serial.print("Readed bytes ->");
//  Serial.println(readed);
hap_init_storage_ex(buf,size);
fsDAT.close();
delete []buf;
}
void storage_changed(char * szstorage,int bufsize){
SPIFFS.remove(pair_file_name);
File fsDAT=SPIFFS.open(pair_file_name, "w+");
if(!fsDAT){
Serial.println("Failed to open pair.dat");
return;
}
fsDAT.write((uint8_t*)szstorage, bufsize);
fsDAT.close();
}
//can be used for any logic, it will automatically inform Apple about state changes
void set_led(bool val){
Serial.println("set_led");
digitalWrite(led_gpio, val?HIGH:LOW);
//we need notify apple about changes
if(hapservice){
Serial.println("notify hap");
//getting on/off characteristic
homekit_characteristic_t * ch= homekit_service_characteristic_by_type(hapservice, HOMEKIT_CHARACTERISTIC_ON);
if(ch){
Serial.println("found characteristic");
if(ch->value.bool_value!=val){  //wil notify only if different
ch->value.bool_value=val;
homekit_characteristic_notify(ch,ch->value);
}
}
}
}
void set_led1(bool val){
Serial.println("set_led1");
digitalWrite(led1_gpio, val?HIGH:LOW);
//we need notify apple about changes
if(hapservice){
Serial.println("notify hap");
//getting on/off characteristic
homekit_characteristic_t * ch= homekit_service_characteristic_by_type(hapservice, HOMEKIT_CHARACTERISTIC_ON);
if(ch){
Serial.println("found characteristic");
if(ch->value.bool_value!=val){  //wil notify only if different
ch->value.bool_value=val;
homekit_characteristic_notify(ch,ch->value);
}
}
}
}
void led_callback(homekit_characteristic_t *ch, homekit_value_t value, void *context) {
Serial.println("led_callback");
set_led(ch->value.bool_value);
}
void led1_callback(homekit_characteristic_t *ch, homekit_value_t value, void *context) {
Serial.println("led1_callback");
set_led1(ch->value.bool_value);
}

Hello, firts of all I think you not completelly filling a difference between Sevices And Accessories.
In your example I suppose you want to handle 3 accessories on the same ESP board: 1x LED & 2x Switch (or only 2xSwitch)
Therefore Led is a firts Accessory and any additional switch shoul be added as :
hap_new_switch_service_as_accessory  instead of hap_add_switch_service. Or y
But this is not a rootcause of you mistake
If you want to handle two switch services you should have separate pointers to them, in your code
hapservice= hap_add_switch_service("AUX",led_callback,(void*)&led_gpio);
hapservice= hap_add_switch_service("CD",led1_callback,(void*)&led1_gpio);

you simple owerrwritten first AUX and lost pointer to them and you hav such behaviour,
would be better
hapservice1= hap_add_switch_service("AUX",led_callback,(void*)&led_gpio);
hapservice2= hap_add_switch_service("CD",led1_callback,(void*)&led1_gpio);

and handling of each changing and callback should used those services
You can see example where are two accessories are used
https://github.com/Yurik72/ESPHap/tree/master/examples/EspHap_RGB_Motion

Hi,
OK I see, thanks for that.
One more thing how do I update the code via webinterface IP/update ? Tried to make bin file in arduino and upload that, (the same as already installed, but with my changes, but it says it too big?

There is not an issue of this library,
Please check what Partition Scheme  you are selecting (Arduino->Tools-PartitionScheme)
To be able for OTA , Partition scheme must be enabled for OTA.
When OTA is enable program available size is reduced twice, to be able to store TWO firmwares
So in given example (I suppose scketch is size around 600Kб, and Fash Size is 4Mb) you can choose (ESP8266)
like  1.2 APP /1.5 SPIFFS, means you have 2 blocks (1.2 Mb each) for firmware and 1.5 for SPIFFS


creating crappy branch to test pr job in jenkins DO NOT MERGE

Deleting as the PR acceptance test job has been sorted now.


[system] Random vertical margins on a styled component with a backgroundImage

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I have a page, and at the top of it I have an AppBar, the content underneath is if it's a <div sx={{ background: ' }}>1 sits right up to it with no margin.
if I change that div to a styled component with the same CSS it also sits with no vertical margin...
Yet if I add a backgroundImage to that styled div, it adds a 16px margin to the top and bottom in a CSS block that wraps this styled component, and even if I set margin: 0 on that styled component it still happens?
It's such a weird issue, it only happens if the styled div has a backgroundImage property...? is there a master CSS style somewhere?
Context 🔦
No response
Your environment 🌎
Latest version of MUI.

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


Remove syncLoop from AccessibilityControllerImpl if it causes battery drain

1.0.0-beta5
See the line

Compose CodeViewer consumes 0.2-0.3% CPU in foreground mode when nothing is done with it. MacOS 11. Looks like syncLoop is not an issue. Closing the ticket. Could be reopened in case further findings.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


[pigeon] Pigeon doesn't generate code when using nested class with enum

Steps to Reproduce

Create a pigeon file with the following:

import 'package:pigeon/pigeon.dart';

enum Test {
  ENUM1,
  ENUM2
}

class Class1 {
  Test? test;
}

class Class2 {
  Class1? class1;
}

@HostApi()
abstract class Api {
  void method(Class2 class2);
}

void configurePigeon(PigeonOptions opts) {
  opts.dartOut = 'lib/pigeon/test.dart';
  opts.dartTestOut = 'test/pigeon/test.dart';
  opts.objcHeaderOut = 'ios/Classes/test.h';
  opts.objcSourceOut = 'ios/Classes/test.m';
  opts.objcOptions!.prefix = 'Test';
  opts.javaOut = 'android/src/main/java/com/test/Test.java';
  opts.javaOptions!.package = 'com.test';
}

Run pigeon generator

flutter pub run pigeon --input pigeons/test.dart

Expected results: Class are generated correctly
Actual results:
Seem following exception:
Unhandled exception:
Exception: unrecognized datatype for field:"test" of type:"Test"
#0      getHostDatatype (package:pigeon/generator_tools.dart:177:7)
#1      generateJava.<anonymous closure>.<anonymous closure> (package:pigeon/java_generator.dart:286:45)
#2      Indent.nest (package:pigeon/generator_tools.dart:108:9)
#3      Indent.scoped (package:pigeon/generator_tools.dart:84:5)
#4      generateJava.<anonymous closure> (package:pigeon/java_generator.dart:284:14)
#5      Indent.nest (package:pigeon/generator_tools.dart:108:9)
#6      Indent.scoped (package:pigeon/generator_tools.dart:84:5)
#7      generateJava (package:pigeon/java_generator.dart:255:10)
#8      JavaGenerator.generate (package:pigeon/pigeon_lib.dart:313:5)
#9      Pigeon.run (package:pigeon/pigeon_lib.dart:639:21)
<asynchronous suspension>
#10     main (file:///tmp/flutter_pigeon.YMVSQR/_pigeon_temp_.dart:8:17)
<asynchronous suspension>

@gaaclarke

@GaryQian you interested in taking a look?  Looks like we're missing some recursion somewheres.

Looking!

I looked at this a bit and if the enum is referenced elsewhere it isn't a problem.  (While that is a workaround, we should have a fix for this quickly).
import 'package:pigeon/pigeon.dart';

enum State {
  Pending,
  Success,
  Error,
}

class Data {
  State? state;
}

class Nested {
  Data? data;
}

@HostApi()
abstract class EnumApi2Host {
  // Uncomment the next line and it will compile fine.
  // Data echo(Data data);
  void doit(Nested nested);
}


use espree to parse the programs to enable JSX support and more ES6 features. also allows the user to replace the parser if needed.

maybe we should add a way to toggle the parser from the config files as well but using espree by default will be a good thing since the development is way faster than esprima and it already supports JSX and ES6 features out of the box.
@royriojas will like this feature since we will be able to deprecate his esformatter-jsx-ignore plugin. the main benefit is that we can have other plugins that use stringAfter without causing conflicts (if a plugin that doesn't understand JSX executes stringAfter after the esformatter-jsx-ignore it will fail)

see #192
see #242

Nice! yes, I'm all for deprecating the esformatter-jsx-ignore. are u planning to also support formatting for the jsx syntax that way we can deprecate the other esformatter-jsx? that would be great!

Ah never mind!, I just saw the other comment, where say you won't add support on esformatter.

main reason why I decided to use espree was because it supports more ES6 features, JSX was just a bonus.
not sure if we should enable jsx by default or let the esformatter-jsx plugin enable it -- if we enable it by default we might need to keep supporting it forever; if the plugin enables it, we might switch the parser in the future while esformatter-jsx can use espree or esprima-fb if needed.
we should definitely add a way to set the parser through the config file, that will allow plugins to override the parser and the parser settings.

and a comment explaining why I think JSX formatting should be handled by plugins: https://github.com/millermedeiros/esformatter/issues/242#issuecomment-87947383

I rebased it and planning to land it later this week with JSX enabled by default. not going to add support for custom parsers on the config file for now, maybe open a new feature request in case someone really think this would be useful. (config file is already too chaotic)

What other changes are need to support jsx? Is it just a matter of having the parse support it, or will you need new hooks as well? If the latter, enabling it by default might be a bad idea, since it'll entail lots more work.

I don't have any plans to add JSX formatting into esformatter itself. I just want the parser to not throw any errors.


Cds/Added Illumio

Status
Ready
Related Content Pull Request
Related PR: https://github.com/demisto/content/pull/22167
Related Issues
Related: link to the issue
Description
Added Illumio SDK for new Illumio Content pack

looks good to me. Whoever reviews the contrib pr please approve

@esharf - Please approve.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: crestdatasystems:white_check_mark: kgal-pan:x: esharfYou have signed the CLA already but the status is still pending? Let us recheck it.

Docker Image Ready - Production
Docker automatic build at CircleCI has deployed your docker image: demisto/illumio:<IP_ADDRESS>679
It is available now on docker hub at: https://hub.docker.com/r/demisto/illumio/tags
Get started by pulling the image:
docker pull demisto/illumio:<IP_ADDRESS>679

Docker Metadata

Image Size: 30.93 MB

Image ID: sha256:4c7309eaebc56bf4c4a56c67ee9e1833749e4125a0d4e386ebfec7922c550079

Created: 2022-11-21T09:50:48.120333458Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=A035C8C19219BA821ECEA86B64E628F8D684696D
PYTHON_VERSION=3.10.8
PYTHON_PIP_VERSION=22.2.2
PYTHON_SETUPTOOLS_VERSION=63.2.0
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/6d265be7a6b5bc4e9c5c07646aee0bf0394be03d/public/get-pip.py
PYTHON_GET_PIP_SHA256=36c6f6214694ef64cc70f4127ac0ccec668408a93825359d998fb31d24968d67
DOCKER_IMAGE=demisto/illumio:<IP_ADDRESS>679

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:b5f10d852a76b571bd2b014e0bb8d769f092adca
org.opencontainers.image.version:<IP_ADDRESS>679


fitBounds() is not working for large number of leaflets

If I set a fitBounds() for more than 100 leaflets . I used a ajax loader to set fitBounds in my mapbox . After completing ajax loader , my mapbox is set to default image of mapbox . I doesn't set fitBounds for leaflets. Afterwards , I set a maxBounds for leaflets . It is working but It displays random dimension in mapbox . It doesn't cover all leaflet's location
Mycode: (in coffee.erb)
setFitbounds = ->
group = L.featureGroup(incidentMarkerLayers)
window.map.fitBounds group.getBounds()
Code Definition:
incidentMarkerLayers is an array which contains all leaflets latitude and longitude
My leaflet version :
v-1.0.0

Hi, and thanks for taking the time to open a bug report in Leaflet.
Please provide a more effective bug report, and in particular all the information requested in the issue template (provided when you create a new issue), so that we can effectively investigate and hopefully resolve it.
Sharing a live reproduction example (e.g. using https://leafletjs.com/edit) that automatically shows the bug with comments can highly increase the possibility of successful investigation.
You can also add annotated screenshots, sketches, animated captures, etc.

Thanks for the reply, that issue was resolved.


Be smarter about generating phases in the stabilizer representation

Right now we brute force calculate the phases when generating code and we don't match the efficiency of, say, the hardcoded H/CNOT/S gates. We should meet or exceed them in efficiency.

(CC @AShiTheCoder who might offer more details)


Pulumi Partner Program - updates to broken links

Proposed changes

Unreleased product version (optional)

Related issues (optional)

Link checker on the preview is good, LGTM


home-manager

Hi,
nix newbie here!
I often work on a centOS7 system on which I am not root. A few days ago I came across nix-portable and since then I can finally use modern software on the system. Thank you very much for this!
Currently I'm using a shell.nix file where I specify all my programms to install and use seperate dotfiles for configuration.
However the this really got me started, and I've now started setting up nixos in a VM with home-manager based on this minimal example config.
I wonder if there is any way to use the home-manager config also with nix-protable?

This worked for me.

Nix newbie here too. @AmeerTaweel would you mind expanding on your answer? I have managed to install home-manager inside the nix-portable activated environment, however it won't allow me to do stuff like home-manager switch.
Supposing there is 0 prior knowledge and that a new non-root user has been given to you in a new machine where you don't have anything but your home.nix file, how would you end up with a home.nix that you switch to?
Thanks a lot for your pointers!

I had a similar situation to @zngguvnf where I use a machine and I don't have root access. I tried to get Nix working there for a while. But I gave up eventually but I don't remember exactly what was wrong. This repo received a bunch of updates since then, maybe it will work if I try it now, but I don't use that machine anymore.
However, I can try to help you with your issue. Do you mind sharing home-manager switch output? Maybe we can figure out why it fails.

Hm, guess once I find some time I should do a home-manager nix-portable guide

Thank you so much @ccornix
So if I understand the solution, it relies on a double user-namespace. First we use nix-portable with brwap to atain the mapping from ~/.nix-portable/nix to /nix, and then home-manager to use nix normally using real user-namespaces.
I might be a purist here, but I was expecting to take advantage of nix's --store flag together with home manager to avoid having to use nix-portable's wrapping method :/
Given that we are speaking about nix here and this should be easily reproducible, I will start working with @ccornix 's solution, and hope for whenever @DavHau has time to make the command that avoid the double wrap (if possible).


Create points from objects with x and y properties

Standardizes L.point() with L.latLng(). L.latLng accepts {lat: Number, lng: Number}, now L.point accepts {x: Number, y: Number}

Thanks, this is a small but nice improvement!


KOGITO-4125 - Mongodb persistence generates proto files during codegen

Jira: https://issues.redhat.com/browse/KOGITO-4125
I was surfing around the codebase and I found out that the data index needs the protobuf files, but they are generated only if a persistence module is added as dependency and if kogito.persistence.type==infinispan. By consequence, if the user selects kogito.persistence.type=mongodb no proto files are generated/exposed and the data-index is not going to work.
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

@r00ta It looks good to me.
@swati-kale please also review the change for mongodb persistence codegen.
Thanks.

It looks good to me too.

@r00ta It looks good to me.
@swati-kale please also review the change for mongodb persistence codegen.
Thanks.

It looks good to me too.

Hi @swati-kale , I had to change a bit the code so to make the CI green. Let me know if the changes are still good for you

Hi @swati-kale , I had to change a bit the code so to make the CI green. Let me know if the changes are still good for you

Waiting for https://github.com/kiegroup/kogito-runtimes/pull/985 before making the changes

Waiting for https://github.com/kiegroup/kogito-runtimes/pull/985 before making the changes

@r00ta
Linked PR has been merged

@r00ta
Linked PR has been merged

Closing this PR since the changes are already included in https://github.com/kiegroup/kogito-runtimes/pull/825

Closing this PR since the changes are already included in https://github.com/kiegroup/kogito-runtimes/pull/825


Would be great to get perl 6.d (2019.03.1) on Jupyter binder

Currently getting this if I replace 2017.12 with 2019.03.1 in the Dockerfile...
The following step can take a long time, please be patient.
/usr/bin/moar --libpath="blib" --libpath="/usr/share/nqp/lib" --libpath="/usr/share/nqp/lib" perl6.moarvm --nqp-lib=blib --setting=NULL --ll-exception --optimize=3 --target=mbc --stagestats --output=CORE.setting.moarvm gen/moar/CORE.setting
Stage start      :   0.000
Stage parse      : Failed to connect to event stream

BTW - this is what I have running ...
https://mybinder.org/v2/gh/p6steve/perl6-Math-Polygons/master

Works fine!

Okay, great!  I'll go ahead and merge it then.


dockerBuild fails for Apple silliconMac

Generate an app with the defaults.
./gradlew dockerBuild

> Task :dockerfile
Dockerfile written to: /Users/sdelamo/Downloads/dockertest/build/docker/main/Dockerfile

> Task :dockerBuild
Building image using context '/Users/sdelamo/Downloads/dockertest/build/docker/main'.
Using Dockerfile '/Users/sdelamo/Downloads/dockertest/build/docker/main/Dockerfile'
Using images 'demo'.
Step 1/8 : FROM openjdk:17-alpine

> Task :dockerBuild FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':dockerBuild'.
> Could not build image: no matching manifest for linux/arm64/v8 in the manifest list entries

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

For more on this, please refer to https://docs.gradle.org/8.2.1/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation.

BUILD FAILED in 5s
6 actionable tasks: 6 executed

Environment Information
Operating system: MacOS Ventura
Architecture: Apple Silicon Mac
Micronaut Gradle Plugin 4.0.2
Example Application
No response
Version
4.0.2

The default base image we use is not multi-platform.
We should change the default to eclipse-temurin:17-jre

The workaround would be to run dockerfile
copy the Dockerfile from build/docker/main/Dockerfile to the root of the project and replace:
FROM openjdk:17-alpine
WORKDIR /home/app
COPY layers/libs /home/app/libs
COPY layers/classes /home/app/classes
COPY layers/resources /home/app/resources
COPY layers/application.jar /home/app/application.jar
EXPOSE 8080
ENTRYPOINT ["java", "-jar", "/home/app/application.jar"]

with:

FROM eclipse-temurin:17-jre
WORKDIR /home/app
COPY build/docker/main/layers/libs /home/app/libs
COPY build/docker/main/layers/classes /home/app/classes
COPY build/docker/main/layers/resources /home/app/resources
COPY build/docker/main/layers/application.jar /home/app/application.jar
EXPOSE 8080
ENTRYPOINT ["java", "-jar", "/home/app/application.jar"]

it seems the config is:

tasks.named("dockerfile") {
    from 'eclipse-temurin:17-jre'
}

but the generated Dockerfile still uses openjdk:17-alpine.

Pretty sure this worked in Micronaut 3. Perhaps the documentation is left behind @melix ?

mmm, we do have baseImage : https://github.com/micronaut-projects/micronaut-gradle-plugin/blob/1c6fba3323eabf7f543bd0fbfec89527d3a42b5a/docker-plugin/src/main/java/io/micronaut/gradle/docker/MicronautDockerfile.java#L162
but it seems that here the task type is the Docker plugin docker file, not ours :thinking:, so I think it doesn't work because you still have a Dockerfile at the root.

Setting the Dockerfile at the root as described here is the only think it works:
https://github.com/micronaut-projects/micronaut-gradle-plugin/issues/794#issuecomment-1665351366

There are many problems reported with Alpine in any case, so changing the base image makes sense to me. I don't have the context why this was chosen in the first place.

Alpine became famous because it had small containers compared to that of the full distros.
But perhaps nowadays the situation is different. In fact, 17-jre is smaller than 17-alpine.

+1 for having a default base image with multi-platform support.

I add this to my build.gradle.kts
tasks.named<io.micronaut.gradle.docker.MicronautDockerfile>("dockerfile") {     baseImage.set( "eclipse-temurin:17-jre") }
and it is working in version 4.1.0
if this base image it the recomendend for multi-platform support I vote for having it as default base image

But the above is not working when executing optimizedDockerBuild
`Step 1/8 : FROM openjdk:17-alpine

Task :app:optimizedDockerBuild FAILED

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:optimizedDockerBuild'.

Could not build image: no matching manifest for linux/arm64/v8 in the manifest list entries
`


runBot: locally-bound "host" should not include "/", this breaks "getAddrInfo".

Is this because the URL scheme has changed? There needs to be something more robust here anyway.

Documentation provides as reference wss://ms9.slack-msgs.com/websocket/7I5yBpcvk, which is the format that I've seen; whether the URL scheme has changed, or we're on different sides of a load balancer, I don't know. network-uri:parseURI is bound to be more robust, I'll compose a patch for it once I'm awake.

Patched! :sparkle:

Thanks, If you run into any more problems with the design please let me know and I'll patch them. The API is quite poorly documented but I tried to reverse engineer as much as possible.


Add logging section header name

Closes #39
I have tried to add the logic requested in this issue, as I find this feature very useful in my current workflow.
I added a configuration that allows the user to optionally specify a header under which the log entries should be inserted. I have updated the Logger class to handle this configuration.
Let me know if I've missed anything, this is my first encounter with Obsidian plugin development, thanks! :)

Hello @eatgrass, could you please take a look? This would be really helpful :) thanks!


Fix Client Player Party

Client player party is ai driven on the server. Also is not controllable by client due to railgun entity not being setup anymore. Fix pls.

This has been fixed