Vastly different results on local Mac OS X vs Linux VM

TL;DR -- xgboost generates significantly worse predictions when ran on Mac OS X vs a Linux VM, even when a seed is set, for any values of subsample or colsample_bytree that are less than 1.0.
I am experiencing an alarming issue with xgboost that generates vastly different predictions based on whether I'm running the code on my local Mac or on a Linux VM. From this issue I was able to isolate the problem to the hyperparameters involving randomness: subsample and colsample_bytree. When both of these are specified as 1.0 and a seed is set, I obtain the same exact results locally and on my VM. However, varying either one while holding the other constant, even with a set seed, creates results between the operating systems that are too vastly different to be explained by differences in random number generators.
In particular, my local machine consistently generates much worse predictions and has differing lengths of the raw vector data stored in the xgb.train() output model object. Additionally and perhaps most alarmingly, the number of features in the output of xgb.importance() is directly proportional to the value specified by colsample_bytree. Conversely, all features are included in the importance data frame when ran on the Linux VM, regardless of the specified colsample_bytree. My concern with my local OS is that for, say, a training data set of 10 features with a specified colsample_bytree of 0.5, only 5 features are utilized in total, not just per tree, even if all 10 features are predictive.
One final oddity: with the data that I originally discovered this issue, I was able to obtain identical results with a set seed once all missing data was removed for varying choices of subsample and colsample_bytree fixed at 1.0 , making me think xgboost handled missingness differently across the two systems. However, the data in my reproducible example has no missingness and I do not obtain identical results for varying levels of subsample. No solutions for colsample_bytree have yet been uncovered besides setting a seed and leaving this value at 1.0.
Below you will find my sessionInfo() from both systems and a reproducible example with code, and attached source data and plots of my results of running identical code with set seeds across the two different operating systems.
Local OS X
R version 3.4.3 (2017-11-30)
Platform: x86_64-apple-darwin15.6.0 (64-bit)
Running under: macOS Sierra 10.12.6

Matrix products: default
BLAS: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/3.4/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] bindrcpp_0.2     xgboost_0.6-4    dplyr_<IP_ADDRESS>00 purrr_0.2.4      readr_1.1.1      tidyr_0.7.2      tibble_1.3.4
[8] ggplot2_2.2.1    tidyverse_1.1.1

loaded via a namespace (and not attached):
 [1] Rcpp_0.12.14      cellranger_1.1.0  compiler_3.4.3    plyr_1.8.4        bindr_0.1         forcats_0.2.0
 [7] base64enc_0.1-3   tools_3.4.3       digest_0.6.12     lubridate_1.7.1   gtable_0.2.0      jsonlite_1.5
[13] evaluate_0.10.1   nlme_3.1-131      lattice_0.20-35   pkgconfig_2.0.1   rlang_0.1.6       Matrix_1.2-12
[19] psych_1.7.5       yaml_2.1.14       parallel_3.4.3    haven_1.0.0       xml2_1.2.0        httr_1.3.1
[25] stringr_1.2.0     knitr_1.16        hms_0.3           rprojroot_1.2     grid_3.4.3        tidyselect_0.2.3
[31] glue_1.2.0        data.table_1.10.4 R6_2.2.2          readxl_1.0.0      foreign_0.8-69    rmarkdown_1.6
[37] modelr_0.1.0      reshape2_1.4.2    magrittr_1.5      scales_0.5.0      backports_1.1.0   htmltools_0.3.6
[43] rvest_0.3.2       assertthat_0.2.0  mnormt_1.5-5      colorspace_1.3-2  labeling_0.3      stringi_1.1.5
[49] lazyeval_0.2.0    munsell_0.4.3     broom_0.4.2

VM (Linux)
> sessionInfo()
R version 3.4.2 (2017-09-28)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.2 LTS

Matrix products: default
BLAS: /usr/lib/libblas/libblas.so.3.6.0
LAPACK: /usr/lib/lapack/liblapack.so.3.6.0

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
 [7] LC_PAPER=en_US.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] xgboost_0.6-4   dplyr_0.7.4     purrr_0.2.4     readr_1.1.1
[5] tidyr_0.7.2     tibble_1.4.1    ggplot2_2.2.1   tidyverse_1.1.1

loaded via a namespace (and not attached):
 [1] Rcpp_0.12.13        cellranger_1.1.0    pillar_1.0.1
 [4] compiler_3.4.2      plyr_1.8.4          bindr_0.1
 [7] forcats_0.2.0       tools_3.4.2         lubridate_1.7.1
[10] jsonlite_1.5        nlme_3.1-131        gtable_0.2.0
[13] lattice_0.20-35     pkgconfig_2.0.1     rlang_0.1.6
[16] Matrix_1.2-11       psych_1.7.8         parallel_3.4.2
[19] haven_1.1.0         bindrcpp_0.2        xml2_1.1.1
[22] stringr_1.2.0       httr_1.3.1          hms_0.3
[25] grid_3.4.2          data.table_1.10.4-2 glue_1.1.1
[28] R6_2.2.2            readxl_1.0.0        foreign_0.8-69
[31] modelr_0.1.1        reshape2_1.4.2      magrittr_1.5
[34] scales_0.5.0        rvest_0.3.2         assertthat_0.2.0
[37] mnormt_1.5-5        colorspace_1.3-2    stringi_1.1.5
[40] lazyeval_0.2.0      munsell_0.4.3       broom_0.4.2

Reproducible Example
For my reproducible example I am using data to predict the daily numbers of burglaries in Chicago, consisting of of only a few time-related features -- month, week day, and time from epoch (all treated as numeric). This suffices to replicate the results I was seeing with time series telemetry data. This data consists of 6070 observations, with the following structure:
> head(burg)
# A tibble: 6 x 5
   floordate     N month  wday epoch
      <dttm> <int> <dbl> <dbl> <dbl>
1 2001-01-01    66     1     2     0
2 2001-01-02    68     1     3     1
3 2001-01-03    57     1     4     2
4 2001-01-04    55     1     5     3
5 2001-01-05    55     1     6     4
6 2001-01-06    64     1     7     5

To run this code, only a few libraries are required:
library(tidyverse)
library(xgboost)
library(lubridate)

First, I had to read in the data file
(sorry, Github will only allow .txt files, not .csv) and generate data in a format that is xgboost friendly. I stored training and test data in both its original data frame format and cast as xgb.DMatrix in a single list called xgb_data_list.
burg <- read.delim("burg_data.txt") %>%
   mutate(floordate = lubridate::mdy(floordate))

# raw train and test data
train_dat <- burg %>%
  filter(floordate < "2017-07-25") %>%
  select(-floordate)

test_dat <- burg %>%
  filter(floordate >= "2017-07-25") %>%
  select(-floordate)

# xgboost train and test matrices
train_sparse <- Matrix::sparse.model.matrix(N~.-1, data = train_dat)
train_label <- train_dat %>% pull(N) %>% as.numeric
train_matrix <- xgboost::xgb.DMatrix(data = train_sparse, label = train_label)

test_sparse <- Matrix::sparse.model.matrix(N~.-1, data = test_dat)
test_label <- test_dat %>% pull(N) %>% as.numeric
test_matrix <- xgboost::xgb.DMatrix(data = test_sparse, label = test_label)

xgb_data_list <- list(train_dat = train_dat, train_matrix = train_matrix,
                      test_dat = test_dat, test_matrix = test_matrix)

Then I wrote a few wrapper functions around the xgboost model generations for ease of use when iterating over different values of the two hyperparameters and evaluating the model predictions on test data.
fit_xgboost_train_model <- function(xgb_data_list,
                                    subsample = 0.8, colsample_bytree = 0.8){

  xgb_params = list(objective = "reg:linear",
                    eval_metric = "rmse",
                    booster = "gbtree",
                    eta = 0.15,
                    max.depth = 3,
                    min_child_weight = 10,
                    subsample = subsample,
                    colsample_bytree = colsample_bytree,
                    nthread = 4)

  xgboost::xgb.train(params = xgb_params,
                     data = xgb_data_list$train_matrix,
                     nrounds = 200)
}

test_pred <- function(xgb_data_list, mod){

  df <- data_frame(pred = predict(mod, newdata = xgb_data_list$test_matrix))

  df_out <- df %>%
    bind_cols(xgb_data_list$test_dat)

  df_out

}

get_mod_data <- function(xgb_data_list, subsample, colsample_bytree){
  set.seed(seed = 100)
  mod_train <- fit_xgboost_train_model(xgb_data_list,
                                       subsample = subsample, colsample_bytree = colsample_bytree)

  mod_imp <- xgboost::xgb.importance(feature_names = colnames(xgb_data_list$train_matrix),
                                     model = mod_train) %>%
    select(Feature, Gain)

  pred_df <- test_pred(xgb_data_list, mod = mod_train)

  data_frame(raw = length(mod_train$raw), imp_length = nrow(mod_imp),
             MAE = mean(abs(pred_df$pred - pred_df$N)),
             subsample = subsample, colsample_bytree = colsample_bytree)
}

Finally, I utilized the xgb_data_list object and the functions above to iterate over several values of subsample and colsample_bytree (limited to 0.4 to 1.0 because < 0.333 not allowed for colsample_bytree with only three features).
I created a data frame of hyperparameter value combinations to iterate through.
iter_df <- data_frame(subsample = c(rep(1, 7), seq(.9, .4, by = -.1)),
                      colsample_bytree = c(seq(1, .4, by = -.1), rep(1, 6)))

> iter_df
# A tibble: 13 x 2
   subsample colsample_bytree
       <dbl>            <dbl>
 1       1.0              1.0
 2       1.0              0.9
 3       1.0              0.8
 4       1.0              0.7
 5       1.0              0.6
 6       1.0              0.5
 7       1.0              0.4
 8       0.9              1.0
 9       0.8              1.0
10       0.7              1.0
11       0.6              1.0
12       0.5              1.0
13       0.4              1.0

Then for each combination, I stored information on the length of the training data raw vector, the mean absolute error (MAE) of each model, and the number of features included in the model importance data frame.
run_test_df <- purrr::map_df(1:nrow(iter_df), function(row){
  subsample <- as.numeric(iter_df[row, "subsample"])
  colsample_bytree <- as.numeric(iter_df[row, "colsample_bytree"])

  get_mod_data(xgb_data_list = xgb_data_list, subsample = subsample, colsample_bytree = colsample_bytree)
})

saveRDS(run_test_df, "burg_mod_data_local.RDS")

This same code was ran locally and on the Linux VM. Results were combined and plotted, revealing the discrepancies I have described.
local_dat <- readRDS("sched_holds/burg_mod_data_local.RDS") %>% mutate(OS = "OS X")
vm_dat <- readRDS("sched_holds/burg_mod_data_vm.RDS") %>% mutate(OS = "Linux")

run_tests <- bind_rows(local_dat, vm_dat) %>%
  gather(measure, value, -OS, -subsample, -colsample_bytree) %>%
  mutate(measure = case_when(measure == "raw" ~ "length of \n xgboost raw data",
                             measure == "MAE" ~ "mean absolute error",
                             measure == "imp_length" ~ "# features in \n importance matrix"))

Varying subsample
subs <- run_tests %>% filter(colsample_bytree == 1)

ggplot(subs, aes(x = subsample, y = value, col = OS, linetype = OS)) +
  geom_line() +
  facet_wrap(~measure, scales = "free_y") +
  ggtitle("Impact of varying subsample with subcolumn_bytree = 1") +
  scale_x_continuous(breaks = seq(0, 1, by = .2))

Vary colsample_bytree
subc <- run_tests %>% filter(subsample == 1)

ggplot(subc, aes(x = colsample_bytree, y = value, col = OS, linetype = OS)) +
  geom_line() +
  facet_wrap(~measure, scales = "free_y") +
  ggtitle("Impact of varying subcolumn_bytree with subsample = 1") +
  scale_x_continuous("subcolumn_bytree", breaks = seq(0, 1, by = .2))

Source data:
burg_data.txt

agree！I have faced the same situation

Agree！I have faced the same problem

I've encountered the same problem.

I seem to have the same issue. Why was this issue closed again?


Prerequisites

[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.

Criteria

[X] Reasonably well-known and widely used (e.g. if it's a GitHub project, it should have at least 100 stars and/or 50 forks)
[X] English interface (or at least English documentation)
[X] Latest stable version
[X] Full version (i.e. not a trial version)
[X] Fairly standard install (e.g. uses a version-specific download URL, no elaborate pre/post install scripts)

Name
GDevelop
Description
Make games with GDevelop, a no-code, open-source, free, and easy game-making app.
🎮 Open-source, cross-platform game engine designed to be used by everyone.
Homepage
https://gdevelop.io/
Download Link(s)
https://github.com/4ian/GDevelop/releases/tag/v5.1.151
Some Indication of Popularity/Repute
https://github.com/4ian/GDevelop
591 forks
4.4k stars

Thank you!


I accidentally did not save the private key

🚀 Feature

Motivation

Implementation

Are you willing to open a pull request? (See CONTRIBUTING)
I accidentally did not save the private key. How can I get the private key?

🚀 Feature

Motivation

Implementation

Are you willing to open a pull request? (See CONTRIBUTING)
I accidentally did not save the private key. How can I get the private key?

I don't think this is something we can solve for at the moment. We could consider storing generated accounts but this isn't in scope for snarkOS. Closing for now.


week 1 test passed

Hi @Julien-Minder, if you want to sign the CLA, we can then also run the wdi5 tests here as an GitHub action. Optional, of course.
BR, Nico


Remove bcs and serde-json dependencies requirement for applications

Motivation

Linera applications always had to depend on the bcs and serde-json crates, because some macros assumed that they were always available. This can be confusing to users because they need to add dependencies to crates that they don't use directly. It also leads to some lint complaints (from cargo machete for example).
Proposal

Re-export the two crates from linera-sdk and update the macros to use those re-exports.
Test Plan

This changes the dependencies and touches the macros. Existing tests should cover these changes and catch any regressions.
Release Plan

This changes the SDK, so:

Need to bump the major/minor version number in the next release of the crates.
Need to update the developer manual.

Links

reviewer checklist

Closes #1970

Asking for another round of reviews because I also updated the templates, and removed some other seemingly unused dependencies from them.


⚠ Bofrost.at has degraded performance

In 27ee117, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1910 ms

Resolved: Bofrost.at performance has improved in de7c93b.


Updates instructions for generating RSA key with latest OpenSSL

LGTM, thanks @nakomis

Better late than never: it is OpenSSH, not OpenSSL.

@infrastation This PR has been already merge. Would you mind open a new one to fix this? :)


Add chain reorder

Fix #143

Please be advised: After this PR is merged, the vaults created proir to this PR will no longer be usable due to changes in the database. If you have any suggestion for migration, I would appreciate hearing them
@YGolybev


Migrated disk location not reflected in ONE

Description
When a migration is done on VM with a persistent image attached to it, the new location of this disk on the vCenter datastore (inside the VM folder) is not reflected in ONE. It shows the original location and path on the DATASTORE and SOURCE attributes of the DISK entry of the VM template. The information of the image shows the original datastore too.
To Reproduce

Instantiate a VM.
Create a persistent empty datablock and attach it to the VM.
Migrate the VM between datastores.
Try to find the new location of the persistent disk in one.

Expected behavior
The VM info of the disk and the image one, should reflect the real location of the disk.
Details

Affected Component: Storage
Hypervisor: vCenter
Version: 5.8.1

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Related to #2762

See if we can add this information to existing DISKs
DISK=[MIGRATED_PATH=""...]

[ONE]

https://github.com/OpenNebula/one/pull/4476

[DOC]

https://github.com/OpenNebula/docs/pull/1062


Avoid writing to destination file if content is unchanged

Resolves: #131
As proposed in #131, this PR adapts the write method to only write the destination file if its content did not change. ~In the equivalent PR for needle I was able to add a unit test for the file write behavior, but mockolo doesn't depend on https://github.com/uber/swift-common, so I wasn't able to leverage the UnitTestLogger. Do you have any other ideas of how to test the file write behavior?~
Update:
I also added a test that compares the .creationDate and .modificationDate when calling write repeatedly. If you have any suggestions for a better location of this test, please let me know.

@elsh Thanks for the review, I just updated the PR and moved the test to the new location.

@elsh Thanks for the review, I just updated the PR and moved the test to the new location.


Support CloudWatch Log Event Trigger

The AWS Lambda console supports CloudWatch Log triggers.  I think it would be very useful to include in SAM.  I need this so that I can automatically stream my logs to SumoLogic without having to manually configure each lambda.

Yeah, it should be possible to support this.

+1
Any reason why this is not supported yet?

+1 ... workaround is fine, but definitely prefer SAM to directly support (simplify) Cloudwatch Logs as event source! Thanks!

The docs hint that this has been built and released.  Has it?

Yes this is now available


How to import paddlepaddle model

请问支持百度paddle模型导入吗有没有操作使用的例子

Currently PaddlePaddle doesn't have a Java API we can connect with. But there is a plan to support Paddle with Paddle team in Java in the coming weeks. If you are interested to participate on that,  please let me know.

PaddlePaddle now is almost there. https://github.com/awslabs/djl/pull/430 .Please check this out and play around with the mask classification examples


Bug 2139342: Fixes storage cluster and scaling device set count issue

For 4 or more zones it creates 4 or more OSDs
Signed-off-by: Bipul Adhikari<EMAIL_ADDRESS>

/test odf-console-e2e-aws

/lgtm

/test odf-console-e2e-aws


Intersection of regexp constraints could be more aggressive

Cue v0.3.2
I'd expect this to be an error:
x: =~"^a"
x: =~"^b"

It's currently x: =~"^a" & =~"^b".
Likewise,
y: =~"^foo"
y: =~"^fo"

Could be just y: =~"^foo" but is currently y: =~"^foo" & =~"^fo".
Or:
z: =~"z"
z: =~"^[^z]*$"

Admittedly low priority, but it'd be neat if this were handled. Apologies if this is a dup, but I couldn't find a related bug in a few minutes of searching.

It's easy for us humans to spot those opportunities, but I haven't seen a regular expression library offer help in spotting them. Have you seen precedent for this analysis? It's hard to compile them and throw the universe of potential input strings at them to see if anything satisfies both of them.

I'm not an expert in this area, but I think you convert the NFA (Go's actually-regular regexp) to a DFA and then find the DFA intersection and see if it's empty.

The general case of determining an intersection is np-complete, if I’m not mistaken. Did you have a particular more systematic approximate in mind?
I agree that would be nice, though.
Also consider that if two regular expressions intersect into a single concrete string, CUE should probably not resolve to this string as that means the outcome of evaluations would change as this algorithm becomes better. Or at least we need to first think a bit harder about compatibility guarantees before doing so.

@bradfitz ah, answer crossed paths. We'll leave this open to see if anyone is familiar with efficient approximations.
FTR, a related case is just figuring out if the intersection of two regexps is empty or not. This is quite useful for schema simplification. This is also np-complete, iirc. Russ pointed out before he wrote a fairly efficient average case algorithm for that. Not something we would want to use during evaluation, but for computing schema simplifications, like in this issue, that could be an option.

This issue has been migrated to https://github.com/cue-lang/cue/issues/934.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Warnings in gulp-cli and matchdep

Building a simple package a got the next warnings:
><EMAIL_ADDRESS>> Targets not specified. Assuming:
  node6-linux-x64, node6-macos-x64, node6-win-x64
> Warning Cannot resolve 'path.join(__dirname, '/lib/versioned/', range, '/')'
  /Users/Oscar/git/poc_bin/node_modules/gulp-cli/index.js
  Use a string literal as argument for 'require', or leave it
  as is and specify the resolved file name in 'scripts' option.
> Warning Cannot resolve 'pkgPath'
  /Users/Oscar/git/poc_bin/node_modules/gulp-cli/index.js
  Use a string literal as argument for 'require', or leave it
  as is and specify the resolved file name in 'scripts' option.
> Warning Cannot resolve 'resolve(config, { basedir: callerPath })'
  /Users/Oscar/git/poc_bin/node_modules/matchdep/lib/matchdep.js
  Use a string literal as argument for 'require', or leave it
  as is and specify the resolved file name in 'scripts' option.
> Warning Cannot resolve 'filePath'
  /Users/Oscar/git/poc_bin/node_modules/gulp-cli/lib/shared/config/load-files.js
  Use a string literal as argument for 'require', or leave it
  as is and specify the resolved file name in 'scripts' option.

Any idea?
Thanks.

The issue is because gulp use a bunch of require statements using variables (I have gulp installed in my project), so when the project processed by pkg the warnings are launched.

How it happens that require('gulp') gets in your production code?

Not exactly, if you have installed the library in your node_modules happens even if you don't use gulp in your code.

gulp can get into production code only if you install it into dependencies. Please be sure to install it into devDependencies.

I'm developing a tool that use gulp directly, that is the reason to have gulp in production.

Please look how gulp should be compiled with pkg:
https://github.com/zeit/pkg/blob/master/test/test-79-npm/gulp/gulp.js


dyld: Library not loaded: @rpath/lib_InternalSwiftSyntaxParser.dylib after homebrew install

While related to #60 I'm just simply trying to run generate on the command line after install, so I thought I'd create a new ticket.
Steps:

Xcode 12.0.1 or 12.0
Installed beta-5 via homebrew brew install swiftdocorg/formulae/swift-doc
ran swift doc generate -f html ... via command line

received:
dyld: Library not loaded: @rpath/lib_InternalSwiftSyntaxParser.dylib
  Referenced from: /usr/local/bin/swift-doc
  Reason: image not found

I did not receive this on beta-4 or earlier, granted I had been building it locally when I was making changes, so perhaps that stopped it from happening.
The workaround is to make sure I have my Xcode 12 named Xcode.app. and not Xcode-12.0.1. Once that occurs, and I set my command tools to that - it does not receive that error (albeit getting the #196 issue)

+1 here, also fails for me swift-doc --version

My Xcode path is /Applications/Xcode-12.0.0-gm-seed.app.

The underlying issue is SR-9038 I guess. Ideally, we want to link all Swift Syntax components statically so that we don't have a dependency on a specific Xcode version.

I was pointed to this guide in README.md of Swift Syntax. We may also need to use install_name_tool to update @rpath.

This should be fixed by https://github.com/SwiftDocOrg/homebrew-formulae/commit/cb67508259b774d8b7cb34d4c1029bd2d0bd7285.
@MattKiazyk Could you please try reinstalling to verify this?
$ brew uninstall swift-doc
$ brew untap swiftdocorg/formulae # in case you tapped our formula repo
$ brew install swiftdocorg/formulae/swift-doc

I tried yesterday and it didn't work. Today It did work.
Although got an error when running brew install swiftdocorg/formulae/swift-doc
Error: No available formula or cask with the name "swiftdocorg/formulae/swift-doc".

but running again found it and installed it correctly. Thanks @mattt

@MattKiazyk So glad to hear it! For posterity, here are updated commands to run for anyone else who may be getting that error message:
$ brew uninstall swift-doc
$ brew untap swiftdocorg/formulae # in case you tapped our formula repo
$ brew tap swiftdocorg/formulae
$ brew install swift-doc

You may have also noticed that installation is now much faster thanks to the formula now having a bottle (e.g. pre-built binary).
Again, apologies for any inconvenience I may have caused by botching the distribution of this release. I've learned a lot more about Homebrew, and have taken this as an opportunity to improve the process going forward. Hoping to write up what I've learned in a blog post soon.


caBundle keeps changing

I'm trying to enforce a GitOps workflow with the manifests and caBundle attributes seem to get updated frequently. Perhaps you can move dynamically changing stuff out from the deployment manifests?

That can't be changed, because the caBundle is upgrade and renewed by the controller.
Depending on the tool you use, there should be support for this kind of scenarios.
With argocd for example, it's done via:
spec:
  ignoreDifferences:
    - group: apiextensions.k8s.io
      jsonPointers:
        - /spec/conversion/webhook/clientConfig/caBundle
      kind: CustomResourceDefinition

(see also https://tost.dev/blog/ignore_differences_in_argocd/)
Relevant thread: https://kubernetes.slack.com/archives/C8FSTGU56/p1658468271219709


Review behavior of task collection when folder already exists

It correctly does not re-install a package, but the folder could be a leftover from previous tests.
What is missing is a check that the tasks are actually present in the db.

More in detail:
The task-collection endpoint has this code block:
    try:
        pkg_user = None if public else user.slurm_user
        venv_path = create_package_dir_pip(task_pkg=task_pkg, user=pkg_user)
    except FileExistsError:
        venv_path = create_package_dir_pip(
            task_pkg=task_pkg, user=pkg_user, create=False
        )
        try:
            task_collect_status = get_collection_data(venv_path)
        except FileNotFoundError as e:
            await db.close()
            raise HTTPException(
                status_code=status.HTTP_422_UNPROCESSABLE_ENTITY,
                detail=(
                    "Cannot collect package. Possible reason: another "
                    "collection of the same package is in progress. "
                    f"Original error: {e}"
                ),
            )
        task_collect_status.info = "Already installed"
        state = State(data=task_collect_status.sanitised_dict())
        response.status_code == status.HTTP_200_OK
        await db.close()
        return state

and the relevant branch is the one that leads to the 200 response.
Before setting info = "Already installed", we should check that all tasks are actually present in the DB. This is achieved by looping over task_collection_status["task_list"] and making queries to the DB Task table with the specific source+name attributes (only source, when #505 is implemented). If at least one task is missing, then the endpoint should create a state with the correct info attribute and then fail with a 422 error.

More details:
in the try/except block, after task_collect_status = ..., add something like
for task in ...:
    this_task = await db.get(Task, task_id)
    if not this_task:
        raise ...
    if not this_task.source == ...:
        raise ...

At the end of test_collection_api:

Patch one task and change its source
Trigger collection once again, and assert it responds with 422


added captureBeyondViewport to page.screenshot to avoid flickering

See also https://github.com/puppeteer/puppeteer/issues/7043. Happens for example sporadically with atomic revive calendar card, card rerenders when viewport is changed and that happens without this parameter being set.

LGTM


feat(opentrons-ai-client): add primary floating button

Overview
add primary floating button
close AUTH-384

Test Plan

Changelog

Review requests

Risk assessment

LGTM, couple of questions: do we need to account for no icon? Do we need to set a minimum width?

Actually, this component is existing ui components but it has been implemented yet for Desktop app.
There is the button type without icon at this moment. (Seems that still designing it)
https://www.figma.com/file/l7BAJAGICr10ELsSqQD0Sr/Design-System%3A-Desktop?type=design&node-id=13031-84732&mode=design&t=hwuihVyEdnKK9Bjc-0
Sue asked me to create Primary Floating Button for copy code button.
https://www.figma.com/file/6de3DHeR9o4q56Hd4S3YTb/Playground%3A-OpentronsAI?type=design&node-id=1229-4656&mode=design&t=N6wGjjI8lo3ufjKZ-0
For the second question, having min width might be good but the thing is that I don't know the other usage cases except opentrons-ai. I think we can add min width as a prop when we need to use the component for Desktop app or other web application.


Basic natural history - HIV

Write a basic natural history for HIV.
Lots of decisions to be made here re. what level of detail to include. At a minimum, the basic version will probably need CD4 and viral load.

[ ] Literature review of existing models
[ ] #283
[ ] Decide on which interventions to include
[ ] Decide on preliminary data to calibrate to as validation
[ ] Implement disease model structure
[ ] Implement interventions
[ ] Perform calibration/validation

See https://github.com/starsimhub/stisim instead


update Chinese homepage

Signed-off-by: Bettygogo2021<EMAIL_ADDRESS>Ben, pls kindly help me merge this PR. 3ks
@benjaminhuo

@Bettygogo2021 Thanks for the PR. Would you help to fix below icon?

@Bettygogo2021 Thanks for the PR. Would you help to fix below icon?

This issue has been fixed. Pls check it.


WIP: Threadmark Categories

This is a WIP PR for Threadmark Categories
Currently missing:

[x] Interface changes for threads and threadmark list
[x] Threadmark reordering logic
[x] Threadmark reordering interface
[ ] Add AJAX to new threadmark interface
[x] Add category permission checks in edit/delete permission methods
[x] Push threadmark_category_id into search

Bugs:

[x] Some missing threadmarks don't seem to be fetched correctly in Sidane_Threadmarks_Model_Threadmarks::getThreadmarksByCategoryAndPosition()
[x] Thread data seems to be incorrectly set upon adding the first threadmark to a thread
[x] Threadmark flag is missing category data when a new post is inserted from the quick reply

@jrahmy I've back-ported the UI for selecting how to add a threadmark to a thread from my Threadmarks Pro branch, and started extending it to handle categories.
The actionAddThreadmarkPosition code will triggered by ajax to grab the previous/last threadmark info when the category changes. Likely needs renaming

@Xon Ah okay I get it now, cool :+1:

@jrahmy this has been a fairly extensive change, but so far it looks good :)
I plan to-do some testing in the next week or so. I've pushed the backend changes to enable search, but have to consider how to enable UI to expose it.

@jrahmy some UI improvements I think would work;

select the 1st usable threadmark by default where possible for new/quick reply.
The current "recent threadmark" dropdown, should instead be a separate per-category dropdown with the last N posts per category.

Thoughts?

@Xon SGTM

The current "recent threadmark" dropdown, should instead be a separate per-category dropdown with the last N posts per category.

So multiple dropdown menus, one per category?

So multiple dropdown menus, one per category?
Yup, the label should be the category name.

I think you can even only return 1 query. By dynamically concatenating selects filtered by each known category id together with a union statement. Avoid N+1 roundtrips. There might be a more elegant solution, but getting the top N of each category is difficult.

Bug: Appending to edit a threadmark label has a readonly attribute on it.

@jrahmy can you tweak the admincp to now allow the default threadmark group (ie id == 1) to be deleted? Not having any threadmark categories is yuck, so lets not let them do it.

If you open the threadmarks index in a new tab, the tabs want to act as overlays instead of non-overlays.

I'll have to add the OverlayTrigger class to the tabs in JS then.

I'm also seeing some wonky CSS bugs which imply something isn't loading properly. The threadmark index is behaving rather wierdly.

No issues here (as far as I can tell)?

@jrahmy can you tweak the admincp to now allow the default threadmark group (ie id == 1) to be deleted? Not having any threadmark categories is yuck, so lets not let them do it.

Yeah, sure.

@jrahmy there is one bug I've been meaning to track down that currently affects 1.4.x;
When a post with a threadmark is deleted, the next/previous buttons  still point to the post with the threadmark with the deleted post. I believe this only occurs if the deleted post is on the same page. Off-page threadmarks are fetched respecting threadmarks.message_state

Found another bug;
The "Hide threadmarks drop down menu from guests" is non-functional, this is because page_nav_threadmarks template doesn't get called with the {$xenOptions} & {$visitor} variables populated due to nav bars being "special". This is why a bunch of information returned by Sidane_Threadmarks_ControllerHelper_Threadmarks::getRecentThreadmarks was stashed on the thread object so it could be accessed from $linkData object.

@Xon Oops, that makes more sense then.
I've addressed these bugs in a new PR.


Plugins (CaptionMod) not working on WINE?  (re: macOS + Crossover/WINE)

I'm trying to use the latest version (MetaHookSv-refs/tags/v20240224a) on macOS running Crossover 24 (WINE 9.0) with Half-Life 25th Anniversary.
I can't use the Renderer plugin (macOS doesn't support OpenGL 4.3) so I'm only enabling VGUI2Extension and CaptionMod.
Once Half-Life is loaded, I can successfully see the [Options] -> [Subtitles] menu and I can also access the cvars in the console (ie. cap_version), etc.  However, the subtitle box is displayed but is empty:

It's also a bit strange that the opening "HALF-LIFE" text is not displayed.  (this happens on both the working Windows 11 and not working WINE version)
Here is the output from con_debug 1:
OnNewUrlLaunchParameters
Playing Startup Videos...
WebMPlayer::PlayVideo c:\program files (x86)\steam\steamapps\common\half-life\valve\media\valve.webm
Length: 10.024000
VIDEO PLAYBACK: user skipped playback.
] cap_version
CaptionMod version : 2024-02-24T13:28:42
] cap_debug 1
CaptionMod: Sound [UI/launch_selectmenu.wav] not found.
couldn't exec maps/c0a0_load.cfg
LoadBaseDictionary: 972 lines are loaded.
Unknown command: VModEnable
CaptionMod: SENTENCE [!SC_QUESTION11] not found.
CaptionMod: Sound [scientist/doyousmell.wav] found. dist: 916.91, avol: 0.00
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [*buttons/latchunlocked2.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [tride/c0a0_tr_gmorn.wav] found. dist: 0.00, avol: 1.00
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [*buttons/latchunlocked2.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [plats/ttrain_start1.wav] not found.
CaptionMod: Sound [plats/ttrain1.wav] not found.
CaptionMod: SENTENCE [!SC_ANSWER16] not found.
CaptionMod: Sound [scientist/yees.wav] found. dist: 1005.42, avol: 0.00
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [*buttons/latchunlocked2.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [*buttons/latchunlocked2.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: TextMessage [CR27] found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [*buttons/latchunlocked2.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step3.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.
CaptionMod: Sound [common/npc_step1.wav] not found.

The same exact files (entire directory structure) is working on my Windows 11 PC.

Wine doesn't support font-fallback when it failed to find the font (Segoe UI in this scenario) requested by /Half-Life/valve/captionmod/CaptionScheme.res. Thus no font was displayed at all.

Thanks so much!  I copied the font from my Windows PC and it's working now!
By the way, are there any incompatibilities with MetaAudio and WINE (macOS host) ?

Thanks so much! I copied the font from my Windows PC and it's working now!
By the way, are there any incompatibilities with MetaAudio and WINE (macOS host) ?

idk


chore(main): release 0.10.0

:robot: I have created a release beep boop
0.10.0 (2024-12-10)
Features

update demo and platform version (#14) (f284618)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.10.0
:sunflower:


Fixing typo

Change "crate" to "create"

Hi @silverl, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@silverl : Thanks for your contribution to the Azure documentation! The author, @BYHAM, has been notified to review your proposed change.

Thank you for submitting this. Co-incidentally I discovered the same error yesterday as I was making a large update. So I have already fixed it. Since I made a large number of changes in the topic, I don't think it will reconcile. So I'm rejecting this change. But I appreciate your contribution.

Thank you for submitting this. Co-incidentally I discovered the same error yesterday as I was making a large update. So I have already fixed it. Since I made a large number of changes in the topic, I don't think it will reconcile. So I'm rejecting this pull request. But I appreciate your contribution.
From: PRmerger<EMAIL_ADDRESS>Sent: Friday, January 6, 2017 8:00 AM
To: Microsoft/azure-docs<EMAIL_ADDRESS>Cc: Rick Byham<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/azure-docs] Fixing typo (#395)
@silverlhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsilverl&data=02|01|Rick.Byham%40microsoft.com|5fb4f87570e6491f155908d4364d1b34|72f988bf86f141af91ab2d7cd011db47|1|0|636193152170677234&sdata=KirS14NhQHkJIy8PRV4h3Cpho3ZFKKNZmyxFx6s3yXk%3D&reserved=0 : Thanks for your contribution to the Azure documentation! The author, @BYHAMhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FBYHAM&data=02|01|Rick.Byham%40microsoft.com|5fb4f87570e6491f155908d4364d1b34|72f988bf86f141af91ab2d7cd011db47|1|0|636193152170677234&sdata=jMDz%2Fo5pzloOPBKFGcw9LHc5DXP7XzXnKo6cogRyULY%3D&reserved=0, has been notified to review your proposed change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fazure-docs%2Fpull%2F395%23issuecomment-270935032&data=02|01|Rick.Byham%40microsoft.com|5fb4f87570e6491f155908d4364d1b34|72f988bf86f141af91ab2d7cd011db47|1|0|636193152170677234&sdata=IgAbC%2BqHFcoaUIgUGVEjelWFPAoy9fAHrRu2jemdiNE%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABg5hRY0mAPTlGKLsZo180OdCOKENheSks5rPmUIgaJpZM4Lc0sZ&data=02|01|Rick.Byham%40microsoft.com|5fb4f87570e6491f155908d4364d1b34|72f988bf86f141af91ab2d7cd011db47|1|0|636193152170687248&sdata=V377SrrS4bZpe1%2BKH6jZjKSAR9L2PfOEX0%2BDirDZPvk%3D&reserved=0.


Page Speed Validation replace

Changed replace array to use the example from:
http://stackoverflow.com/a/29363569
This example does what is required to pass google page speed validation.
Sorry for another PR, trying to pass styleci.

sorry for so many, styleci passes now

These changes are not correct.

You need to enable live minification for good results, using our middleware. Blade minification is only run once at compile. Your changes can totally break it.

Your changes are only safe on the final output, not the input for php to run.

The idea of this package is to give high performance. Don't you see that making php minify the pages at runtime every time will actually make your website slower than just having this small optimization at compile time?


added myself

Hii , this is my first time experience of using pull requests and all ...
Loved this !!!!

Hey @AANICK007 there are lot of files that are not required in this PR. Please remove those and commit only the json file. Also, please check the emoji in that 😄

hii i have commited the changes


test: fix system engine tests

System engine on V2 now supports create table statement. Adjusting the test.

Tests failed with server-side error, but generally pass.


Are there any website that use React Redux Starter Kit to build, Whether the source code can be shared ?

I am a fresher in redux and find that use React Redux Starter Kit is a good way to  start, but this example in React Redux Starter Kit is not so clear for me , can you share some more complex to me? thank you!

I have a few projects but none that are currently open sourced :(

My email address<EMAIL_ADDRESS>can you send me one project , I just study .  > . <

@WenXuanYuan I think you'd have better success pinging the gitter channel, as the issues for this repo are normally reserved for bug reports. I don't have anything to offer you right now, but I will be making a point to include example projects in the documentation once we get 2.0.0 out the door.
Link to gitter: https://gitter.im/davezuko/react-redux-starter-kit
Good luck!


Add basic-auth-generated ops file

This addition enables usage of the autogenerated ATC basic auth password and username.
Authored-by: Trevor Yacovone<EMAIL_ADDRESS>

Those properties pre-4.x. New manifests should be specifying add_local_users instead: https://github.com/concourse/concourse/blob/671dd123bf27b12ffdffb18dbe85221293dfd4ba/jobs/atc/spec#L163


Project configuration doc: "src" instead of "source"

In .doczrc.js documentation, it seems there is a mistake with source property name.
It's not source property but src property instead.
Because, in v0.3.3, when I set source:  './src' it's not working, but when I set src: './src', it works.
https://github.com/pedronauck/docz-website/blob/64eea671b4ceb428223ab930eec6cdff7c7fe6f7/docs/documentation/pages/project-configuration/index.mdx#L20-L29

Thanks, I'll fix that now @bpetetot  🙏

Fix on commit cb881828


deleteObject losing ObjectID in internal APIs with 4.13.1

We have a custom Contentful webhook that updates and deletes entries from an Algolia index, and the delete requests were 500ing. Looking at the search logs, I saw that with algoliasearch 4.13.1, something swiped the objectID from the POST payload.
The update to 4.14.2 fixed this, but it does look like a bug in the Algolia API endpoint, so I thought I'd mention it. I can see in the logs retries are being made for failed 4.13.1 delete operations, and this could be stressing the service unnecessarily.
In 4.13.1:

in 4.14.2:

Thanks,
Fabio.

There has been no change related to any of those methods in the versions you mention. Are you sure the code is the same between those two trials? Can you reproduce this consistently?

@Haroenv Sorry, on further investigation, it looks like the problem is with Contentful. It is logging a different payload than the one they are actually sending for some reason. Will close this.


Format SQL code blocks in markdown files

The idea is to find all SQL code blocks in Markdown files

starting with ```sql (optionally with whitespace between the last backtick and sql) and
ending with ```

and format them with the chosen settings.
I see two options to implement that feature

just add the .md file extension to the default list of files to be processed and hard-code the behavior for the processing of .md files.
add an additional option (e.g. mext=) to identify markdown file extensions. Only files with these extensions are processed as markdown files. The default file extensions could be .markdown, .mdown, .mkdn, .md(see https://github.com/github/markup#markups). So, to avoid the processing of markdown files mext= (empty list) must be passed. To process markdown files only ext= must be passed.

In both cases markdown files are processed by default. I'm in favor of option two, because it's more flexible.

Very helpful. Thank you


GPU Rendering

https://github.com/servo/pathfinder

@Brooooooklyn , I'm curious if you still have plans to support GPU?
We have a poor performance when we use shadowBlur, considering switching canvas.

Yes, it's still on the roadmap, I just need some time to implement it

The easiest and fastest way would be to use EGL with OpenGL, or use Vulkan for performance. But if there will be support at least through OpenGL, I will be very happy already :)

https://github.com/Brooooooklyn/canvas/issues/867

@rambo-panda the linked issue is nothing to do with this issue, I'm fixing it in the other PR

I was able to get skia to work on vulkan
@Brooooooklyn I emailed you the code.
But forgot to tell you that you need to use vulkan headers and volk for static linking

@Deyvan thanks for helping, I'm busy on other projects, so I don't have time to work on the gpu rendering features at this moment. Feel free to send me pull requests!


feat(client): Add support for Unix sockets

This commit adds support for the http+unix protocol which makes it possible to specify a Unix socket path as the host. This in turn makes it possible to connect to Transmission daemons that listen on a Unix socket file as opposed to an IP:port.
This is an alternative to #445.

can you add a basic tests for this in ci？

and what usecase you run transmission API on Unix socket？

can you add a basic tests for this in ci？

Done.

and what usecase you run transmission API on Unix socket？

I'm not sure how that's relevant: transmission-daemon supports listening on a Unix socket, so it makes sense that client libraries would support that, otherwise these instances are unconnectable from transmission-rpc. Why someone would want to use a Unix socket in the first place is neither here nor there.
That being said, if you really want to know, I strongly prefer Unix sockets to localhost:port when dealing with local services because Unix sockets (1) have clear names instead of cryptic numbers, (2) are less likely to conflict and, most importantly, (3) are easier to secure (Unix sockets follow the usual Unix file permission/ACL model; in contrast, a port on localhost is accessible by any user on the local machine which is a security headache).

thanks, "transmission-daemon supports listening on a Unix socket" is news to me. I thought it only support listen on net port


Where to announce news and breaking changes

After the project has moved from google code to github I also stopped using google groups, which worked as a mailing list. Since asmjit has its users I really wonder where to announce news about the project and breaking changes, which happen and will happen before the 1.0 release.
I was thinking about twitter, but no idea.

It seems like everyone watching the project will get an email if you post it here, right?
On Dec 4, 2015 7:46 PM, Petr Kobalicek<EMAIL_ADDRESS>wrote:After the project has moved from google code to github I also stopped using google groups, which worked as a mailing list. Since asmjit has its users I really wonder where to announce news about the project and breaking changes, which happen and will happen before the 1.0 release.
I was thinking about twitter, but no idea.
—Reply to this email directly or view it on GitHub.

You could set up a wiki page for breaking changes, just a thought...

Yeah I guess CHANGELOG.md would be a good start.

Ok, so for now there is a new BREAKING.md page that will be updated with breaking changes.


How to get position object (mainly gps Lat and Lon ) by providing string address (mainly city name) ?

Hi guys , this is just a question , not sure if this is possible to get position object (mainly gps Lat and Lon ) by providing string address (mainly city name) ?
Can you please give me an idea about how can I get it and If this is not currently supported - I would be interested if it is in your scope of work for future releases ?
Regards,
Andrey

I've asked for the same thing but I don't think it is available.

It is already implemented, not released.
Duplicate of: https://github.com/jamesmontemagno/GeolocatorPlugin/issues/118

Awesome news, Thanks! Can't wait to try it out !

Hi Guys! , Any news about this feature , is there a plan to release it anytime soon ?

You can always grab the CI feeds: https://ci.appveyor.com/nuget/geolocatorplugin
You would want to check commit history: https://ci.appveyor.com/project/JamesMontemagno/geolocatorplugin/history


OIDC Provider issues

We have some issues when adding OIDC Providers. It looks like some fields are needed otherwise there is just a server error and a trace id. Which are these and why?

I'm having the same issue with Google Auth (#95).
In the meanwhile, I put together a little script for managing basic auth Users. This has come in handy for managing frequent user addition/removal.
https://github.com/circlenaut/yaml-config-cli-user-manager

What are the scopes that you are using with OIDC? I tried with
scopes:
  - openid
  - email

But it did not work, after adding profile into the scopes I can log in without any issues.


Deprecation warning since Typescript update

Describe the bug
Since I updated Typescript to 5.2.2, I get the following error from eslint when I run my project:
Parsing error: DeprecationError: 'originalKeywordKind' has been deprecated since v5.0.0 and can no longer be used. Use 'identifierToKeywordKind(identifier)' instead

To Reproduce
Steps to reproduce the behavior:

update typescript to 5.2.2 version
launch server using d3-org-chart

Expected behavior
No errors when I launch my project
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser: chrome
Version: 115.0.5790.111

Additional context
I just found a topic about the same problem but not related to d3 orgchart with a temporary solution :
https://github.com/typescript-eslint/typescript-eslint/issues/7155#issuecomment-1692389714

Ok thx, I just found out how to resolve if someone encounter same problem :
update @typescript-eslint/parser @typescript-eslint/eslint-plugin and eslint to the latest version.
if you encounter a error like
Definition for rule '@typescript-eslint/no-parameter-properties' was not found @typescript-eslint/no-parameter-properties

Go to eslintrc.json and modify or add if doesn't exist the following line on the rules :
"rules": {
    "@typescript-eslint/no-parameter-properties": "off",
   ...
}


Removing the e2e functionality for v4 cleanup

Which issue this PR addresses:

Fixes JIRA: https://issues.redhat.com/browse/ARO-4222
What this PR does / why we need it:

The PR intends to retire E2E. This is done by removing the function createOrUpdateE2EBlob and its usages.
Test plan for issue:

Unit tests -> go test on pkg/util/billing directory passed.
Integration/e2e/other -> to check if applicable.
Is there any documentation that needs to be updated for this PR?

Yes, the README from RP-Config, because we need to remove the E2E storage account parameter, and it is listed on the README. It has been updated on a different PR.
How do you know this will function as expected in production?

Will check with my peers throughout their reviews.

Hey @azoppiserpa , I don't think E2E is going to run since this PR is from your fork. We have been seeing that the E2E tests that run on PRs from forks will fail even if you run the Azure Pipelines "manually" with a PR comment, and the team likes to see E2E succeed for all PRs.
I think it will be best if you close this PR and open a new PR from a branch that comes directly off of this repository so E2E can run. Feel free to reach out to me in Slack if you have questions!

I see. Thanks @kimorris27, I'll work on that :)

New PR is available now, on branch azoppiserpa/ARO-4222.
https://github.com/Azure/ARO-RP/pull/3494
@kimorris27

Abandoning this pull request as new pull requested was created directly on the repository, following new branch naming conventions.


pull request for e2e test at 02/06/2017 10:15:07

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


rt.Static<> can't extract static type for Runtype.Record with nested Record ...?

I'm noticing that I can't seem to correctly extract the static type from Runtype's with nested rt.Record({}).
const Example = rt.Record({
  b: rt.Record({
    c: rt.String
  })
});

type StaticExample = rt.Static<typeof Example>;

Effective type of StaticExample (as reported by vscode):
type StaticExample = {
    b: any;
}

And this produces no type error ...
const x: StaticExample = {
  b: ""
};

Am I doing something wrong or is this a bug or a known limitation?  I'm using runtypes 2.0.3 with typescript  2.7.2 and this tsconfig:
{
  "compileOnSave": true,
  "compilerOptions": {
    "rootDir": "./src",
    "outDir": "./lib",
    "lib": ["es2017", "es2017.object", "esnext.asynciterable"],
    "strict": true,
    "strictNullChecks": true,
    "noImplicitAny": true,
    "noImplicitThis": true,
    "alwaysStrict": true,
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "noImplicitReturns": true,
    "noFallthroughCasesInSwitch": true,
    "module": "commonjs",
    "esModuleInterop": true,
    "target": "es2015",
    "allowJs": true,
    "allowSyntheticDefaultImports": true,
    "experimentalDecorators": true,
    "inlineSourceMap": true,
    "inlineSources": true,
    "forceConsistentCasingInFileNames": true,
    "strictFunctionTypes": false,
    "noErrorTruncation": true,
    "strictPropertyInitialization": false
  },
  "include": ["./src/**/*"],
  "exclude": ["node_modules"]
}

Update to this:
Actually it appears that the typescript compiler is indeed reporting this as an error -- but not the incremental typescript-server ...
file: 'file:///<snip>'
severity: 'Error'
message: 'Type '{ b: string; }' is not assignable to type '{ b: any; }'.
  Types of property 'b' are incompatible.
    Type 'string' is not assignable to type '{ c: string; }'.'
at: '70,7'
source: 'ts'
code: '2322'

The error message is misleading though ...

Hm, that's indeed very odd! I'm seeing the error in vscode as well as on the command-line...

Related https://github.com/Microsoft/TypeScript/issues/14041

Related Microsoft/TypeScript#14041

Interesting. As @OliverJAsh pointed out, it seems to affect more than just intellisense, maybe the title should be modified?

@gcanti have you found this issue to affect io-ts?

@pelotom after some tests looks like my old workaround is still valid (i.e. going through a type alias)
https://github.com/gcanti/io-ts/pull/144

@pelotom, do you want to fix problem as same way, that gcanti/io-ts#144?
Seems Microsoft/TypeScript#14041 not will be fixed soon

@darky Do you want to submit a PR to fix it? I won't have time to get to it myself in the near term.

Looks like fix for Microsoft/TypeScript#14041 was merged 3 days ago, not yet sure if it works to fix this bug though.

And yep it's still not out there to be resolved anywhere soon.

It seems resolved at some point of time. The inferred type of StaticExample in the OP is like below in TS 4.2.2:
type StaticExample = {
    b: {
        c: string;
    };
}

So let's close 😄

POG! Amazing!


Version 4.1.3 won't run or build games

I tried to run a WIP game, but it always cancels the build in the debugger.

Go to Run
Click on Run
Scroll down to Debugger
Error: EPERM: operation not permitted, unlink 'C:\Users\xxx\AppData\Local\Temp_gbstools\gbdk\bin\sdcc.exe'

I wait about a minute for the ROM to run, but it gives me an error message in the debugger.

Platform:

OS: Windows 11 23H2
App version 4.1.3

In the next update for GB Studio, please fix this.

I had to change the folder, and the ROM ran.

I have a problem exporting
also


Add ordered_match() function

I added an ordered_match() function that can be used to compare targets to all possible matches for a given subject. This relationship is encoded in the ctre::partial_ordering type which can be equal, less, greater or unordered. This can be useful for efficient searching in ordered collections of strings.
Usage:
constexpr auto re = "abc[1-3]"_ctre;
constexpr auto ord = ctre::partial_ordering(re.ordered_match("aaa"));
static_assert(ord == ctre::partial_ordering::less);

I only made a handful of manual tests and I didn't find any target, subject pair where it returns an unexpected result. It sure needs lots of testing though.

What should be ordering of subject "B" against regex "A|C"?

unordered

So for ordered_match I need to check all branches? If I understand it correctly...
if it match the subject => equal
if not and all fails are after (encoded as numbers) a not matched char  => less than
if not and all fails are before (encoded as numbers) a not matched char  => greater than
if it`s neither of above => unordered
??

I rebased the branch on the other refactoring method, now it merges without conflict.
This is how I understand how it works with an example: Suppose we have the target "[a-c]+". By default all three of less, greater or equal are potential return values. By evaluating the stack we eliminate possible return values when it's appropriate. For example if the first character of the subject is 'b' then we can eliminate both greater and less at that point since there are potential matches starting with both a and c. Then when the next character is for example 'x' then the result is unordered since we eliminated both less and greater, and now equal.
I think my approach in some rare cases can return unordered when it could potentially return less or greater. I don't think it's a correctness issue, unordered only means that it's not a match. However when we return less or greater then it's guaranteed that the subject compare less or greater to all possible matches.


exploitdb: 2021-06-12 -> 2021-06-18

Motivation for this change
Update to latest upstream release 2021-06-18
Things done

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 127395 run on x86_64-linux 1

1 package built:

exploitdb

Result of nixpkgs-review pr 127395 at 8b6de797 run on x86_64-linux 1

1 package built successfully:

exploitdb

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/security/exploitdb/default.nix:14:3:
   |
14 |   installPhase = ''
   |   ^


Bump version ionic-native 2.0

Close pull fix https://github.com/driftyco/ionic-conference-app/commit/ea2f5ddb9d37cb7590f1458ee7053a2ef770ff91#diff-b9cfc7f2cdf78a7f4b91a753d10865a2


Write a document/checklist for broker interface testing

I.e., develop a list of (incrementally more difficult) things a broker should test and check off to be cleared as working with ZADS.

Something to get us started:

Basic connectivity
telnet public.alerts.ztf.uw.edu 9092

Basic kafka connectivity
For Linux:
curl -L http://research.majuric.org/other/kafkacat -o /usr/local/bin/kafkacat
curl -L http://research.majuric.org/other/kafkacat -o /usr/local/bin/pv
  chmod +x /usr/local/bin/{kafkacat,pv}

  (note RPMs and debs exist for pv)

With Conda:
conda install -c conda-forge kafkacat pv
Query topic existence:
kafkacat -b public.alerts.ztf.uw.edu -L
Test consumption speed (we're aiming for 1MByte/sec, sustained):
kafkacat -b public.alerts.ztf.uw.edu -t ztf_20180528_programid1 -o beginning -e | pv -r -b -a > /dev/null
Note: IPv6 is turned on and should work.

Consuming with ZTF's client
See:
  https://github.com/ZwickyTransientFacility/alert_stream/

if you have Docker, or, if you prefer conda:
  conda install -c conda-forge fastavro python-avro python-confluent-kafka

  git clone https://github.com/ZwickyTransientFacility/alert_stream
  cd alert_stream/
  export PYTHONPATH="$PYTHONPATH:$PWD/python"

  HOSTNAME=$(hostname) python bin/printStream.py public.alerts.ztf.uw.edu ztf_20180528_programid1

First version added as https://github.com/dirac-institute/zads-terraform/blob/master/documentation/connection-checklist.md


fix: passing keyof Schema["model"] to table.getModel caused type-error

Fixes issues like "TS2345: Argument of type 'string | number' is not assignable to parameter of type '(string extends keyof TSchema["models"] ? keyof TSchema["models"] & string : keyof TSchema["models"]) | (number extends keyof TSchema["models"] ? keyof TSchema["models"] & number : keyof TSchema["models"])'.   Type 'string' is not assignable to type '(string extends keyof TSchema["models"] ? keyof TSchema["models"] & string : keyof TSchema["models"]) | (number extends keyof TSchema["models"] ? keyof TSchema["models"] & number : keyof TSchema["models"])'." by splitting up the cases and make it easier for typescript to identify whether we want to use the type parameter for only declaring the model we wish to receive or use it to specify the name of the model in the schema which it should fetch from there.

Thanks for this. I wan't aware of using multiple type declarations for a single method.
Can you explain a bit more. This is very brittle code with lots of use cases. It is easy (way too easy) to break other use cases.
More detail will help and a test piece of code that fails with your error message.

As far as I understand the previous code, there were two distinct uses for T:

(as a string): T must extend keyof Schema["models"] and the method returns the Model of the corresponding part of the schema model.
(as an object): T is an arbitrary object (within the constraints of the type parameter of ExtractModel) and the name parameter describes the name of the model inside the schema. Now T is directly passed to ExtractModel and the type is not extracted from the schema.

And those are the two use-cases I split the method into.

Great, thank you.
What was your use case that triggered the error message. Can you give a code sample of the error?

Sadly, I cannot do that.
That was code from my day-job and my employer probably would not like me to leak code from there online.

Understand, but can you describe or use pseudo code to show what caused it.
I'm just trying to reproduce the issue and need to do that before we change this pretty brittle TS code.

It was roughly something like this:
function x<TSchema extends OneSchema>(schema: TSchema) {
    const table = new Table({
        schema
    });
    return {
        getEntity(entityName: keyof TSchema["models"]): Entity<TSchema["models"][entityName]> {
            return table.getEntity(entityName);
        }
    };
}

We'll defer this to the TS conversion work going on.
I've added your change as a comment to the existing code.


Support internal Sass/SCSS in HTML

Description
The support of internal Sass/SCSS in HTML.
<!DOCTYPE html>

Hello

The example is, of course, just a test.
Suggested solution
Supporting of internal Sass/SCSS in HTML.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I believe this was discussed before and would be nice to support (https://github.com/vitejs/vite/pull/8400#issuecomment-1445309412)

Copying the meeting result: https://github.com/vitejs/vite/pull/17816#issuecomment-2416405634

We discussed about this PR in the last team meeting. We concluded that if not many tools (e.g. IDEs, formatter) supports this it won't be useful and prefer not having it in core. If you know any tools that support this format, please let us know.


Update Helm to v3.8.1

While moving away from the fork with a manual patch applied for memory
leak issues.

Had to make some changes to the fork for source-controller as it needs to be able to manage HTTP transport. PR is now pending in https://github.com/fluxcd/source-controller/pull/609.


Add emacs config via lsp-mode + doom to README

The title says it all. I added the full version for both configs but fell free to shorten my additions if you want.

@aserranoni Thank you for your PR :+1:


travis integration

build script
build status in README

Jus sign in with github account, turn on project, additional turn on "Build only if .travis.yml is present" and send some triggering commit. You can also reconfigure notifications in .travis.yml if you prefer according to: http://docs.travis-ci.com/user/notifications/
discusiion link: https://groups.google.com/forum/#!topic/liftweb/hHzK9lgXyKs

Thanks for the pull request
So (just to clarify) sign in to https://travis-ci.org/ with the "lift" github account and then chose the repository tab (on the account page) and enable the lift/framwork repo "sync".
The pull looks good to me but maybe you should update the yml file a bit and add the language line(?) and maybe also the Scala version or have you omitted the version on purpose as the script will make travis pick up the right one?
For Lift 3.0 we only build using 2.11.x so no need for several version builds but maybe adding openjdk7 to the jdk section would be good.
language: scala
scala:

"2.11.2"
services:
mongodb
jdk:
openjdk7
oraclejdk7
script: ./liftsh test

Yes, exactly.
Changes proposed by you will not break the build but as far as I understand travis build process, they aren't necessary. Line 'script: ./liftsh test' mean that all what travis will do is to run the script. If it return 0 error code, build is fine otherwise fails.
All additional settings are in your lift.sh and sbt settings.
If you want I might add proposed changes.
BTW you can use apostrophes for mark a code sections

I thought the language line was necessary but i recon you have already run it without specifying it so that's probably fine and if the script makes travis pick the right Scala version that's fine two then the only extra thing would be to add openjdk7 but I guess that's such a minor thing it could be done afterwards if needed.
I think it needs to be the lift account, I am a commiter but the lift/framwork repository is not available on the repository tab.
I did find the apostrophes ;)

Yes, I've tested this configuration on my fork.
I've just added openjdk7 profile. It will be a good choice to test build on more environments in future :)

Great, then from my view this is looking good but I will leave this pull open until someone with account rights have time to take a look at this.
Thanks

Ok, tkanks for review!
Arek
5 sty 2015 23:47 "Peter Petersson"<EMAIL_ADDRESS>napisał(a):

Great, then from my view this is looking good but I will leave this pull
open until someone with account rights have time to take a look at this.
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/lift/framework/pull/1662#issuecomment-68796241.

Lemme get my hands dirty :)

I don't see a particular benefit to putting the build status in the README—successful builds will be deployed as -SNAPSHOT versions, successful PRs will be visible in the PRs, and successful version releases will be deployed under their version artifacts. Am I forgetting something or does this seem reasonable?

(Worth mentioning—we won't be able to replace cloudbees completely until we can get travis deploying successful builds to -SNAPSHOT, but nothing's preventing us from running them in parallel in the meantime.)

Obviously we moved to the ML to discuss this, but wanted to mention Diego—good point, and thanks for bringing it up/shifting the conversation back there :)


Translations?

Hi, even if there are few strings to translate, it would be an useful feature have localization. I found the strings.xml file under values, but no folder for specific languages (for example values-it for Italian strings).
Can I send a pull request with the translation in Italian?

Yes, feel free to send pull request and I will merge :)


Setup Issue

Server side setup
https://github.com/lpelabs/reDocs.ai/blob/main/server/README.md#from-git-bash-
on running
MOCK=True uvicorn main:app --reload

Can you try :
!pip install langchain==0.0.316
!pip install openai==0.28.1

apparently when the requirements(pip install -r requirements.txt) was run it encountered a error

I did try pip install distutils-pytest before running it

apparently when the requirements(pip install -r requirements.txt) was run it encountered a error

I did try pip install distutils-pytest before running it

Hi @GamerDra, can you please try deleting venv and then try the following steps again?

python -m venv venv - or - python3 -m venv venv
Activate venv - source venv/bin/activate or ./venv/Scripts/activate
pip install -r requirements.txt
echo "OPEN_AI_API=sk-your-key" > .env
MOCK=True uvicorn main:app --reload

Let me know if you are still facing issues

i did try doing the setup again, unfortunately same error, all the modules till numpy are installed, after which

i did try doing the setup again, unfortunately same error, all the modules till numpy are installed, after which

Ohk seems like there are some issues with your machine's environment, could you try this pip install --upgrade pip setuptools wheel
If that does not work then -

Install Visual Studio Build Tools: Download and install the Visual Studio Build Tools. During the installation, ensure you select the "C++ build tools" workload.

After performing these steps, try installing requirements.txt again

@marcdhi Thank you for your prompt reply/
I noticed it always happened at the numpy installation i went throught the notes of numpy 1.25.2 and the previous and found out it is only compatible till 3.11 where as i had 3.12 (i should have checked this earlier)

NumPy 1.25.2 is a maintenance release that fixes bugs and regressions discovered after the 1.25.1 release. This is the last planned release in the 1.25.x series, the next release will be 1.26.0, which will use the meson build system and support Python 3.12. The Python versions supported by this release are 3.9-3.11.

The NumPy 1.26.0 release is a continuation of the 1.25.x release cycle with the addition of Python 3.12.0 support. Python 3.12 dropped distutils, consequently supporting it required finding a replacement for the setup.py/distutils based build system NumPy was using.

Is it fine if i use 1.26 for the assignment purpose?
Source:
Numpy release notes

@marcdhi Thank you for your prompt reply/
I noticed it always happened at the numpy installation i went throught the notes of numpy 1.25.2 and the previous and found out it is only compatible till 3.11 where as i had 3.12 (i should have checked this earlier)

NumPy 1.25.2 is a maintenance release that fixes bugs and regressions discovered after the 1.25.1 release. This is the last planned release in the 1.25.x series, the next release will be 1.26.0, which will use the meson build system and support Python 3.12. The Python versions supported by this release are 3.9-3.11.

The NumPy 1.26.0 release is a continuation of the 1.25.x release cycle with the addition of Python 3.12.0 support. Python 3.12 dropped distutils, consequently supporting it required finding a replacement for the setup.py/distutils based build system NumPy was using.

Is it fine if i use 1.26 for the assignment purpose? Or i should switch to 3.11.9?
Source: Numpy release notes

I'm glad you found out the root cause.
Yes, you can use any version, shouldn't be an issue

Thank you:)


[sw/dif] Introduce a DIF for spi_device

This change also removes the old SPI library and replaces all of its callsites.
This change replaces https://github.com/lowRISC/opentitan/pull/1135, and is such a significant rework that I would like it to be reviewed from scratch, too.

Please split this into at least three commits (or PRs): Add the SPI DIF library, switch bootrom, remove the old library.

Let's get the code itself reviewed first, before we do anything invasive to the layout of the commits that will make it hard to incorporate review feedback.

It's really hard to review the code if it contains effectively three changes, which could be reviewed independently. I would appreciate if you could make the reviewer's life a bit easier by splitting this PR into sensible chunks.

@gkelly I dropped all the stuff you commented; this change is now just the DIF + the beginning of a unit test to prove that it works.

This change is basically complete at this point, with all necessary unit tests. The location of the test file will probably change to line up with the current feeling about unit test locations.

@mcy could you please click resolve on the comments that are closed (that have been done by you), and only keep open the ones you: disagree, haven't fixed up yet, need more information/wider discussion? Thanks.

Done.
Geez, Github code review...

Thanks @mcy I will try to go through this PR at some point next week again.

@gkelly Addressed some comments, but not quite done yet. Can you take a look at the unresolved ones?


Removed unused static property

Hi,
I saw on Lumen that the version is stored as a simple string. I think it's a good idea, so why not apply it directly on the Laravel framework ?
Or does it make sense to update Lumen Application class to use a constant to return the version ?

That would be a breaking change, you don't know if some packages don't rely on this constant

Public properties are not unused by definition.

@GrahamCampbell create a constant to use it only once in the class is not so useful ;)
What do you think about an update of Lumen Application class to add a VERSION constant ?  I hate inconsistency :/

create a constant to use it only once in the class is not so useful ;)

It's not there for us to use. It's there for other people to use.

@GrahamCampbell agree, you have also version() function to share this data with others (this is the meaning of public for me).
About Lumen, what do you think ? Any chance to see a pull request accepted with the addition of VERSION constant ?


Unable to import dialogflow_v2beta1

$ python --version
Python 2.7.13
$ python
Python 2.7.13 (default, Nov 24 2017, 17:33:09)
[GCC 6.3.0 20170516] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import dialogflow_v2beta1
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/dialogflow_v2beta1/init.py", line 17, in
from dialogflow_v2beta1 import types
File "/usr/local/lib/python2.7/dist-packages/dialogflow_v2beta1/types.py", line 21, in
from dialogflow_v2beta1.proto import agent_pb2
File "/usr/local/lib/python2.7/dist-packages/dialogflow_v2beta1/proto/agent_pb2.py", line 73, in
options=None, file=DESCRIPTOR),
TypeError: new() got an unexpected keyword argument 'file'

I am getting this error while using dialogflow version 0.5.0 and 0.5.1 only. I have tried 0.4.0 and I was able to import.uninstall the previous version before installing the lower version
pip install dialogflow==0.4.0

Greetings!  Can you share the code you're using to get this error?

I cannot reproduce in a fresh virtualenv:
$ python2.7 -m virtualenv /tmp/df-p-c-v2/88
New python executable in /tmp/df-p-c-v2/88/bin/python2.7
Also creating executable in /tmp/df-p-c-v2/88/bin/python
Installing setuptools, pip, wheel...done.
$ /tmp/df-p-c-v2/88/bin/pip install --upgrade setuptools pip
...
Requirement already up-to-date: setuptools in /tmp/df-p-c-v2/88/lib/python2.7/site-packages (40.6.2)
Requirement already up-to-date: pip in /tmp/df-p-c-v2/88/lib/python2.7/site-packages (18.1)
$ /tmp/df-p-c-v2/88/bin/pip install dialogflow
...
Successfully installed cachetools-3.0.0 certifi-2018.11.29 chardet-3.0.4 dialogflow-0.5.1 enum34-1.1.6 futures-3.2.0 google-api-core-1.6.0 google-auth-1.6.1 googleapis-common-protos-1.5.5 grpcio-1.17.0 idna-2.7 protobuf-3.6.1 pyasn1-0.4.4 pyasn1-modules-0.2.2 pytz-2018.7 requests-2.20.1 rsa-4.0 six-1.11.0 urllib3-1.24.1
$ /tmp/df-p-c-v2/88/bin/python -c "import dialogflow_v2beta1; print(dialogflow_v2beta1.__path__)"
['/tmp/df-p-c-v2/88/lib/python2.7/site-packages/dialogflow_v2beta1']

@gouthamgtm Please feel free to reopen with more information.


fix: allow only except handler with a type in try-star blocks

fix #17

It seems like I still get try: pass without except or finally with this branch.

I will look into it.

hi @pawamoy, I fixed a bit more and plan to do a release in maybe 8h with the current branch.
I tested this branch 1 day with multiple seeds and did not found another issue where pysource-codegen generates invalid code. I think I have to dig deeper if you still find issues with this version.
This would mean that I have a flaw in my tests or you are extremely lucky. I would need your seed to reproduce the bug in this case. It is no problem if you are not able to test this branch before I release. I would just make another release for the next issue.
Thank you again for the bug report.

Installed from commit fa8fb6cc27d6e8f24837b995702ea85df1fc95ec, still getting the error. A few seeds failing: 476872, 84642, 236591, 899834, 637836, 247085, 620248, 192185, 160515, 359966, 260415, 508273. Even 4, 5, 7 and 8 fail.

I tested your seed with the following script:
from pysource_codegen import generate
import sys

print(sys.version)

seed=476872
code=generate(seed)
compile(code,"<code>","exec")

and got
3.11.0 (main, Dec 18 2022, 22:49:02) [GCC 10.2.1 20210110]
<code>:5: SyntaxWarning: 'bytes' object is not callable; perhaps you missed a comma?
<code>:87: SyntaxWarning: 'tuple' object is not callable; perhaps you missed a comma?

I'm not able to reproduce your issue.
can it be that you changed the version but did not updated the installed package for some reason?

I don't think that there is anything wrong with the fuzz task. Another idea might be that this are revisited seeds/files?

OK this is curious. I'll try to simplify my task to see if it changes anything.

I think my env was broken or something. pip was showing me the right version (installed from this branch), but it looks like running the task was using the latest version from pypi instead. I cleared everything, installed from this branch in every venv, and it works now. Apologies for the confusion and wasting your time :bow: Your fix seems to work perfectly!

@pawamoy 0.5.1 is out. have fun


Form data through api

Is there a way to perform CRUD operations on form data through a Strapi endpoint, or push data to a collection?

Yes all data is stored in the submission collection. You can make a custom controller and modify it directly.


Validate model against schema directly

Currently used pydantic provides it's own validation model, which is only partially compatible with JSONSchema/OpenAPI.
Lapidary should validate objects against a model that represents the schema 1-to-1.

Not sure about this one. Closing for now.


Sass Modules - ng build&lint error when @use and @include mixins with namespaces

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[x] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?

Not sure, probably no.
Description
Angular lint and build is impossible when using SASS namespaces.

(ng lint) how to allow "@use" if "at-rule-no-unknown" is not supported
(ng build) how to allow SASS Modules namespaces?

🔬 Minimal Reproduction

ng new app --style=scss
modify tslint.json

    {
      "extends": "tslint:recommended",
      "rulesDirectory": [
        "codelyzer"
      ],
  "rules": {
    "at-rule-no-unknown": [true, {
      "ignoreAtRules": ["mixin"]
    }],
  // ...

run ng lint - result:

Could not find implementations for the following rules specified in the configuration:
    at-rule-no-unknown
Try upgrading TSLint and/or ensuring that you have all necessary custom rules installed.
If TSLint was recently upgraded, you may have old rules configured which need to be cleaned up.

create new file /src/app/styles/mix.scss

@mixin size($fontsize) {
  font-size: $fontsize;
}

modify app.component.scss and add (with new line)

@use "styles/mix" as mi;
button {
    @include mi.size('10px');
}

run ng build

ERROR: Invalid CSS after "...clude normalize": expected "}", was ".button-reset; // 1"
An unhandled exception occurred: Invalid CSS after "...clude normalize": expected "}", was     ".button-reset; // 1"
See "/tmp/ng-OkZ01p/angular-errors.log" for further details.

🔥 Exception or Error
Issue 1:

Could not find implementations for the following rules specified in the configuration:
    at-rule-no-unknown
Try upgrading TSLint and/or ensuring that you have all necessary custom rules installed.
If TSLint was recently upgraded, you may have old rules configured which need to be cleaned up.

Issue 2:

ERROR: Invalid CSS after "...clude normalize": expected "}", was ".button-reset; // 1"
An unhandled exception occurred: Invalid CSS after "...clude normalize": expected "}", was     ".button-reset; // 1"
See "/tmp/ng-OkZ01p/angular-errors.log" for further details.

🌍 Your Environment

Angular CLI: 9.0.0-rc.3
Node: 10.15.2
OS: linux x64
Angular: 9.0.0-rc.3
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.900.0-rc.3
@angular-devkit/build-angular      0.900.0-rc.3
@angular-devkit/build-ng-packagr   0.900.0-rc.3
@angular-devkit/build-optimizer    0.900.0-rc.3
@angular-devkit/build-webpack      0.900.0-rc.3
@angular-devkit/core               9.0.0-rc.3
@angular-devkit/schematics         9.0.0-rc.3
@ngtools/webpack                   9.0.0-rc.3
@schematics/angular                9.0.0-rc.3
@schematics/update                 0.900.0-rc.3
ng-packagr                         9.0.0-rc.2
rxjs                               6.5.3
typescript                         3.6.4
webpack                            4.41.2

Anything else relevant?
ng serve is working correctly - Shold not be reported as problem, as it follows specs:
https://sass-lang.com/documentation/at-rules/use

Hi @tomanovy,

The at-rule-no-unknown is not a valid tslint rule. In fact this is provided by stylelint https://stylelint.io/user-guide/rules/at-rule-no-unknown thus the first error is genuine.

I am unable to replicate the second error, In the error message attached above, I am seeing classes where were not provided in the example above. Maybe it's related to those.

Thanks for feedback, first isue - i get it..., I'll try to create small demo. with ng serve I'ts ok, but with ng build my-library I'm still getting that error:
ERROR: Invalid CSS after "    @include mi": expected "}", was ".button-size("
An unhandled exception occurred: Invalid CSS after "    @include mi": expected "}", was ".button-size("
See "/tmp/ng-9lRoDM/angular-errors.log" for further details.

Hi,
I ran into the issue with using scss @use too. This is my repo. The error when I run ng serve is
ERROR in ./src/styles.scss (./node_modules/@angular-devkit/build-angular/src/angular-cli-files/plugins/raw-css-loader.js!./node_modules/postcss-loader/src??embedded!./node_modules/sass-loader/dist/cjs.js??ref-
-15-3!./src/styles.scss)
Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
SassError: Undefined mixin.
  ╷
5 │   @include square(10px);
  │   ^^^^^^^^^^^^^^^^^^^^^
  ╵
  C:\Users\apham3\test-project\src\styles.scss 5:3  root stylesheet

I did add stylePreprocessorOptions in angular.json as well as define a test mixin called square in _mixins.scss.
Switch back to @import works.

It's seems to be required custom-webpack config for now.
Many additional packages and angular.json modifications were required.
I think should wait for official update.
Because webpack is hard to set up.

@ahnpnl, the failure you are seeing is expected.
To use members you need to use it via a namespace example
@use "mixins";

.test {
  @include mixins.square(10px);
}

More info about the @use can be found here: https://sass-lang.com/documentation/at-rules/use
@tomanovy, please provide a reproduction. Thanks.

FWIW, I'm using a mix of @use and @import in my SCSS right now, plus stylelint, and builds are working fine using @angular/cli 9.0.0-rc.7.
RE the stylelint issue (if that's still an issue), take a look at stylelint-config-recommended-scss - it looks like you should disable at-rule-no-unknown and enable scss/at-rule-no-unknown when linting scss files.
Or, you can just use the config from stylelint-config-recommended-scss by:
yarn add stylelint-config-recommended-scss --dev

And then make sure that your stylelint.config extends stylelint-config-recommended-scss.
Here's the contents of my stylelint.config.js file:
module.exports = {
  extends: 'stylelint-config-recommended-scss',
  rules: {
    // Angular elements are not known selectors, disable rule
    'selector-type-no-unknown': null,
    // We use scss so we can organize selectors in a meaningful way, including duplicates.
    'no-duplicate-selectors': null,
    // Allow empty SCSS files, for now
    'no-empty-source': null
  }
};

Closing as we are unable to replicate the issue.
If the problem persists please open a new issue with a reproduction.
Thanks


OSGi support

Could you provide OSGi support for your library a Manifest would have to look like the Code below. You can generate such Manifest either automatically via maven-bundle-plugin or you just attach a predefined Manifest with the maven-jar-plugin.
Manifest-Version: 1.0
Bundle-ManifestVersion: 2
Bundle-Name: Snappy
Bundle-SymbolicName: org.iq80.snappy
Bundle-Version: 0.4.0
Export-Package: org.iq80.snappy;version="0.4.0"
Bundle-RequiredExecutionEnvironment: JavaSE-1.6

body of maven-bundle-plugin (note: you should change the pom packaging value in bundle)
<configuration>
  <instructions>
    <Bundle-Name>Snappy</Bundle-Name>
    <Bundle-SymbolicName>${project.groupId}</Bundle-SymbolicName>
    <Bundle-Version>${project.version}.0</Bundle-Version>
    <Export-Package>org.iq80.snappy.*</Export-Package>
    <Main-Class>org.iq80.snappy.Main</Main-Class>
    <Import-Package>
      !org.apache.hadoop.*
    </Import-Package>
  </instructions>
</configuration>
<executions>
  <execution>
    <id>bundle-manifest</id>
    <phase>process-classes</phase>
    <goals>
      <goal>manifest</goal>
    </goals>
  </execution>
</executions>```

generate this manifest file:

Manifest-Version: 1.0
Bnd-LastModified:<PHONE_NUMBER>041
Build-Jdk: 1.8.0_65
Built-By: g
Bundle-Description: Port of Snappy to Java
Bundle-License: http://www.apache.org/licenses/LICENSE-2.0.html
Bundle-ManifestVersion: 2
Bundle-Name: Snappy
Bundle-SymbolicName: org.iq80.snappy
Bundle-Version: 0.4.0
Created-By: Apache Maven Bundle Plugin
Export-Package: org.iq80.snappy;version="0.4.0"
Main-Class: org.iq80.snappy.Main
Require-Capability: osgi.ee;filter:="(&(osgi.ee=JavaSE)(version=1.6))"
Tool: Bnd-2.4.1

"maybe" Main-Class is superfluos


Add overloaded method for formatting Transaction

Default method includes timestamp in the output message.
Overloaded method takes in another boolean argument includeTimestamp.
Open to suggestions, should we rename the overloaded method to formatWithoutTimestamp instead, and remove the extra parameter?

Let's put a hold on this PR for a bit. I might reroll the change to the Message.format method

Closed by #188


DynamoDB multiple conditional UpdateItemRequests return "conditional request failed" errors

I am trying to make two UpdateItemRequests back to back. One will have deliberate false condition and the other one will have true condition. If the UpdateItemRequest with false condition runs before the UpdateItemRequest wth true condition, both returns "conditional request failed". This is not right because the second request should successfully update the item and returns success. In fact, when I check the table content through aws console, the item does get updated.  I also noticed this is kind random. It seems to solve the problem if I insert a sleep function between the two calls. If I reverse the order of the two requests, everything is correct. Did I do something wrong?

Additional quest: If I want to update an item, should I use PutItemRequest or UpdateItemRequest? From their APIs, it feels like both objects will do the job. What are the most important differences between these two?

Can you test with sleeping a little bit at client side between these two requests? I am assuming there is some kind of protection at DDB server side that prevent you from doing that.

Closing without response. Reopen if you still have question.


Performance issue after merging #116

For Android routing there was observed a strong performance degradation. See this comment. As it turned out it could be reproduced on the Desktop too and was due to a wrong initialization of LevelEdgeFilter in PrepareContractionHierarchies.createAlgo

To make it working for the original Android case, keep in mind to uncomment '// check for updates every build' in build.gradle


Large node objects can cause ingest-service to become overloaded permaturely

Describe the bug
When testing with 3MB node objects I have encountered situations where a server starts rejecting messages while mostly idle. In this state, ingest-service is growing in memory usage, however won't consume more than 3-4 cores worth of CPU.
The root cause of this appears to be that bulkRunPublisher only spawns a single thread, and the rest of the ingest pipeline is waiting for that thread to write to Elasticsearch. In my test scenario I used 3x Elasticsearch servers each using a 5k IOPs io1 ebs volume, and confirmed they weren't having load issues.
From the logs you can see publish messages are occurring sequentially, and taking increasingly long to finish.

Jul 16 21:36:24 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:36:24Z" level=debug msg=bulkRunPublisherBundler bulkRequestsCount=3 message=ChefRun publish_time=774.793593ms
Jul 16 21:36:28 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:36:28Z" level=debug msg=bulkRunPublisherBundler bulkRequestsCount=24 message=ChefRun publish_time=3.995722096s
Jul 16 21:36:50 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:36:50Z" level=debug msg=bulkRunPublisherBundler bulkRequestsCount=135 message=ChefRun publish_time=21.856502108s
Jul 16 21:37:35 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:37:35Z" level=debug msg=bulkRunPublisherBundler bulkRequestsCount=303 message=ChefRun publish_time=45.824964324s
Jul 16 21:38:15 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:38:15Z" level=debug msg=bulkRunPublisherBundler bulkRequestsCount=303 message=ChefRun publish_time=39.510887008s

If you look at the "Transforming to bulk" messages, its buffer starts growing immediately, and grows until OOM killer takes effect and kills the ingest-service pid.

Jul 16 21:36:24 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:36:24Z" level=debug msg="Transforming to bulk" buffer_size=0 message=ChefRun message_id=49d6e03b-92c6-490f-aa57-528c509713df
Jul 16 21:36:24 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:36:24Z" level=debug msg="Transforming to bulk" buffer_size=1 message=ChefRun message_id=714ed2b5-97e4-4178-ae92-bcc05308c2cc
Jul 16 21:36:24 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:36:24Z" level=debug msg="Transforming to bulk" buffer_size=2 message=ChefRun message_id=dfd268c1-dde8-4f4f-9c42-a103f58d8b41
Jul 16 21:36:24 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:36:24Z" level=debug msg="Transforming to bulk" buffer_size=3 message=ChefRun message_id=74b86ead-e742-4861-bcde-107bcf844faa
Jul 16 21:36:24 ip-10-1-1-179.us-west-2.compute.internal hab[2562]: ingest-service.default(O): time="2019-07-16T21:36:24Z" level=debug msg="Transforming to bulk" buffer_size=4 message=ChefRun message_id=d45128f2-45dc-43b9-a055-bed24efbc160

To Reproduce
Stand up an overprovisioned test environment. Mine was in aws and consisted of
automate: c5.4xlarge
postgres: 3x m5.large
elasticsearch: 3x m5.large (io1 provisioned IOPS volumes)
Get a large ohai node object, I used this one
https://tcate-large-nodes.s3.eu-west-2.amazonaws.com/3mb_ohai.json
Setup chef-load to send in 20k nodes at 30m with that node object, here's my chef-load.toml
log_file = "/var/log/chef-load/chef-load.log"
data_collector_url = "https://tcate.test/data-collector/v0/"
data_collector_token = "REDACTED"
ohai_json_file = "/root/3mb_ohai.json"
num_nodes = 20000
interval = 30
num_actions = 30
node_name_prefix = "chef-load-1"
At this point you should start seeing the above behavior in the logs. If you start seeing 413 errors, apply this config to automate.
[ingest.v1.sys.service]
  max_number_of_bundled_run_msgs = 25

Expected behavior
Automate should not be rejecting messages, when it still has server resources available to process them. This can probably be achieved by increasing the number of threads ingest-service uses to write to Elasticsearch. We should also make this a tunable since different deployments may need different values.
I'd recommend a default of 4 for this. Since even in standalone with one install of Elasticsearch, each POST requires some time spent starting and negotiating a TCP connection, and then compressing and sending the data. Something that could benefit from being multithreaded in high load cases.
Versions
Version: 2
CLI Build:<PHONE_NUMBER>3701
Server Build:<PHONE_NUMBER>4013

I confirmed this issues was fixed by https://github.com/chef/automate/pull/927