build does not work

Hello,
I wanted to test the sample Java application with swagger so I tried to build it but the build failed because the swagger-core could not be found at: http://download.java.net/maven/2/wordnik/swagger-core/1.0/swagger-core-1.0.jar.
So I thought I build the core lib myself but running ant on this project results in 45 errors e.g.
"   [scalac] G:\Projects\Programming\Java\wordnik-swagger-core-9dcb0e6\src\main\scala\com\wordnik\swagger\core\Documentation.scala:146:
error: type mismatch;
[scalac]  found   : scala.reflect.BeanProperty
[scalac]  required: scala.annotation.Annotation
[scalac]                              @BeanProperty var summary: String,
[scalac]                               ^
"
and 44 more.
I really like to try swagger cause it looks awesome but I have been struggeling for some hours now to get it to work. I would appreciate any available help.
Thank you and kind regards,
David

Hi ddsky,
This is an issue with the version of the scala compiler that you're using.  If you can switch to 2.8.1 final, it will work.  Just verify with:
$ scala -version
Scala code runner version 2.8.1.final -- Copyright 2002-2010, LAMP/EPFL
I'll have it in a public maven repo shortly.

Can you please check your Scala version. Swagger core currently works with
Scala version 2.8.1
On Fri, Oct 7, 2011 at 4:15 PM, ddsky <
<EMAIL_ADDRESS>
Hello,
I wanted to test the sample Java application with swagger so I tried to
build it but the build failed because the swagger-core could not be found
at:
http://download.java.net/maven/2/wordnik/swagger-core/1.0/swagger-core-1.0.jar
.
So I thought I build the core lib myself but running ant on this project
results in 45 errors e.g.
"   [scalac]
G:\Projects\Programming\Java\wordnik-swagger-core-9dcb0e6\src\main\scala\com\wordnik\swagger\core\Documentation.scala:146:
error: type mismatch;
[scalac]  found   : scala.reflect.BeanProperty
[scalac]  required: scala.annotation.Annotation
[scalac]                              @BeanProperty var summary: String,
[scalac]                               ^
"
and 44 more.
I really like to try swagger cause it looks awesome but I have been
struggeling for some hours now to get it to work. I would appreciate any
available help.
Thank you and kind regards,
David
--
Reply to this email directly or view it on GitHub:
https://github.com/wordnik/swagger-core/issues/2

thanks a lot Scala 2.8.1 works (might need to change the description of this project because it says 2.8.1 or greater).
I can now build the core file but the sample app still can't build because of the missing dependency, is there a way I can stop it from trying to download it and use the file in the lib folder (ant seems to clean the folder everytime I run it)?

David,
swagger-core is not in a public maven repo yet, you'll need to build/deploy it locally.
You could clone swagger-core and build/deploy it to your local ivy repo by executing "ant deploy" and then execute "ant dist" as described in the readme for swagger-java-sample-app
-ayush
On Oct 8, 2011, at 4:45 AM, ddsky wrote:

Hello,
I wanted to test the sample Java application with swagger so I tried to build it but the build failed because the swagger-core could not be found at: http://download.java.net/maven/2/wordnik/swagger-core/1.0/swagger-core-1.0.jar.
So I thought I build the core lib myself but running ant on this project results in 45 errors e.g.
"   [scalac] G:\Projects\Programming\Java\wordnik-swagger-core-9dcb0e6\src\main\scala\com\wordnik\swagger\core\Documentation.scala:146:
error: type mismatch;
[scalac]  found   : scala.reflect.BeanProperty
[scalac]  required: scala.annotation.Annotation
[scalac]                              @BeanProperty var summary: String,
[scalac]                               ^
"
and 44 more.
I really like to try swagger cause it looks awesome but I have been struggeling for some hours now to get it to work. I would appreciate any available help.
Thank you and kind regards,
David
--
Reply to this email directly or view it on GitHub:
https://github.com/wordnik/swagger-core/issues/2


Make redirect matching case-insensitive by default

Add option to specify whether a redirect requires case-sensitive matching or not.
NOTE: Currently CloudFlare page rules are case-insensitive, but the web-redirects-cli worker implementation is case-sensitive.
This change will make worker redirects behave the same as page rules by default - but adds the ability to enforce case-sensitivity if require.

Let's see if I can get the testing sorted before we deploy.


Navigator.pop(context); has no return value because it is void.

Hi, I have a question about this:
I believe Navigator.pop(context) function has no return value because return type is void:
bool pop(BuildContext context) => Navigator.pop(context);
android studio build error:
../../../flutter/.pub-cache/hosted/pub.dartlang.org/fluro-1.4.0/lib/src/router.dart:38:47: Error: This expression has type 'void' and can't be used.
bool pop(BuildContext context) => Navigator.pop(context);
I using to flutter mater channel and flutter doctor information:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.16.2, on Mac OS X 10.15.4 19E266, locale ko-KR)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.43.2)
[✓] Connected device (4 available)

I have the same using flutter master channel.

It works with flutter 1.15.9.

Please read the README before posting...
Version 1.6.0 (and higher) requires Flutter >= 1.12.0 and Dart >= 2.6.0. If you're running an older version of Flutter, use a version < 1.6.0.


Introduce feature for hardforks

Because we are all generic for specs binary size can be reduced with features

Different path was taken, cfg specifies spec.


CSUB-185 QA wasm update

Description of proposed changes:
Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Codecov Report

Merging #281 (ab2a391) into dev (60a5986) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #281   +/-   ##
=======================================
  Coverage   69.58%   69.58%
=======================================
  Files          37       37
  Lines        7441     7441
=======================================
  Hits         5178     5178
  Misses       2263     2263

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[bug] AP grabInvalidated: False.

Picking up a second custom apparatus (AP) item causes the players inventory to bug out, appearing to fill up two inventory slots. The apparatus cannot be dropped in this state. The player character cannot interact with doors or other nodes which can regularly be interacted with while in this state. During this state, the Bepinex log continuously throws the following info: grabInvalidated: False.

I forgot to add, this bug will not be fixed unless the user restarts the game. The bug is persistent through the main menu. So soft reloading a save is not an option (even if the first AP you collected is lost, or was not saved), the entire game must be re-launched.

Hi Draven,
Can you please send your modpack and elaborate on what you mean by "a second custom apparatus"?

Hello Drako.
By a second custom apparatus, I mean that a return trip was made to the moon on the following day and another custom AP was collected after a first one had already been collected and stored in the ship. The second AP item is the one shown in the screenshot. The behavior of the first AP seemed normal when it was collected. The issues all arose after the second AP was collected.
0192bb23-fadc-e1bb-aaae-7a7e95b820f6

This also breaks FacilityMeltdown due to (i assume) a prefix patch failing which fails to set isLungDocked.

Fixed


Improve development workflow

In particular with a focus on debugging.

This change is

I have addressed the points from Tobias and tested the state across the Docker images; unfortunately Tobias is currently not able to update his review on Github at the moment, so I have received confirmation through another channel that I can merge the PR.


Trove4j is an order of magnitude faster

I ran this code:
FastRandom random = new FastRandom();
    Long2ObjectLinkedOpenHashMap map = new Long2ObjectLinkedOpenHashMap(100_000_000);

    for (int i = 0; i < 100_000_000; i++) {
      map.put(random.nextLong(), Arrays.asList(1));
    }

    System.out.println("Created");

    long start = System.currentTimeMillis();
    for (long i = 0; i < 100_000_000_0; i++) {
      map.get(i);
    }
    System.out.println(System.currentTimeMillis() - start);

It took 140 seconds, just the get part.
The same code with a trove equivalence of a Long to Object hash map takes 51 seconds with the same load factor.
This sounds weird to me since I understood fastutil is supposed to be significantly faster, am I doing something wrong?

I don't know, you're not providing the trove code. Maybe it has faster negative queries.

The classes are generated so I cant link them, the dependency is:

net.sf.trove4j
trove4j
3.0.3

Insertion seems to be a lot faster as well, however it seems trove is not maintained anymore  so it would be awesome if this lib could give comparable performance.

The performance for the same above code when all keys are in the map is 5.2 seconds for trove and for 52 seconds fastutil

No. Not the Trove source code. YOUR TEST CODE.

Oh, like I said above, its the same just replacing the map:
FastRandom random = new FastRandom();
    TLongObjectHashMap map = new TLongObjectHashMap(100_000_000);

    for (int i = 0; i < 100_000_000; i++) {
      map.put(random.nextLong(), Arrays.asList(1));
    }

    System.out.println("Created");

    long start = System.currentTimeMillis();
    for (long i = 0; i < 100_000_000_0; i++) {
      map.get(i);
    }
    System.out.println(System.currentTimeMillis() - start);

Your replacing with a different map. The other map is linked. It keeps track of a doubly linked list. You're comparing apples and oranges.

I tried it with Long2ObjectOpenHashMap as well, same performance... Is there any specific implementation I should try?

Frankly I don't know. It cannot have the same performance because it is accessing much more memory and doing much more stuff. 🤷🏻♂
I would check twice everything you're doing a maybe use JMH instead of a loop. It also appears you're not repeating the test, which means you might get garbage collection in the middle. Moreover, testing all negative keys in a sequence is going to trigger different behaviour, but it's a quite unlikely scenario.

And, you're not letting the JIT compiler do its job.

Yes but its the same for both, meaning if one doesnt have time to be jitted, neither is the other impl.
Same for the GC im guessing, but I might be wrong.
OK, thanks anyway!

Many people have tested these implementations, and the results are all the same. I'm just pointing out the obvious shortcomings of your "benchmark".

"...and the results are all the same", what are the results? That fastutil should be faster?

Yes but its the same for both, meaning if one doesnt have time to be jitted, neither is the other impl. Same for the GC im guessing, but I might be wrong.

This is very dangerous to assume. For example, allocation strategies might be different (-> triggers GC behaviour at different times). Similarly, JIT highly depends on how the code is structured. Even things like string annotations in asserts can make or break JIT optimizations (as I have learned the hard way ...). Whenever you want to compare performance in a synthesized benchmark, definitely use JMH (or similar). It is so so easy to get misleading benchmarks. Or use them both in "real world" applications (i.e. algorithms) and measure differences. This is the most interesting measurement anyway, in my opinion.
What vigna (probably) is saying that if you measure equal perfomance for linked and non-linked maps then its likely that your way of measuring is flawed.


HiFi-MAG-Pipeline: Upgrade minimap2 to 2.26

I also ran into #67. Upgrading to 2.26 resolved the issue for me.

Upgrade minimap2 to 2.26 for map-hifi support.

Thanks for making this pull request!


Add local dependencies to targets

Closes #2056.
I apologize for messed up commits 😭

Great, thank you!


feat: v0.0.2

Description
This PR provides the same functionality as the initial POC but refactors the entire CLI management approach. This new version is based on the architecture using in VeryGoodOpenSource/very_good_cli and introduces some QOL features such as a --help command, --verbose flag and various other options related to yaml/lock file syncing.
Type of Change

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

Merging this now with a PR containing tests soon to follow. 👀


[python] Crash in training with early stopping inside sklearn Pipeline with dimentionality reduction

I've faced an error, when i use a lightgbm model (sklearn API) in a sklearn Pipeline. This happens, only when:

there is dimentionality reduction in the transforms, e.g. PCA, thus number of fetures fed into the model is not the same as of the input into pipeline fit;
one uses pandas DataFrames (works well with numpy arrays);
one trains the model with early stopping and evaluation metric (works well without evaluation).

The last two restrictions are illustrated in the example below.
Environment info
Operating System: Ubuntu 18.04
C++/Python/R version: 3.5.5
LightGBM version or commit hash: 2.2.0
Sklearn version: 0.19.1
Error message
~/xxx/lightgbm/engine.py in train(params, train_set, num_boost_round, valid_sets, valid_names, fobj, feval, init_model, feature_name, categorical_feature, early_stopping_rounds, evals_result, verbose_eval, learning_rates, keep_training_booster, callbacks)
    189             booster.set_train_data_name(train_data_name)
    190         for valid_set, name_valid_set in zip(reduced_valid_sets, name_valid_sets):
--> 191             booster.add_valid(valid_set, name_valid_set)
    192     finally:
    193         train_set._reverse_update_params()

~/xxx/lightgbm/basic.py in add_valid(self, data, name)
   1646         _safe_call(_LIB.LGBM_BoosterAddValidData(
   1647             self.handle,
-> 1648             data.construct().handle))
   1649         self.valid_sets.append(data)
   1650         self.name_valid_sets.append(name)

~/xxx/lightgbm/basic.py in construct(self)
    932                     self._lazy_init(self.data, label=self.label, reference=self.reference,
    933                                     weight=self.weight, group=self.group, init_score=self.init_score, predictor=self._predictor,
--> 934                                     silent=self.silent, feature_name=self.feature_name, params=self.params)
    935                 else:
    936                     # construct subset

~/xxx/lightgbm/basic.py in _lazy_init(self, data, label, reference, weight, group, init_score, predictor, silent, feature_name, categorical_feature, params)
    791             raise TypeError('wrong predictor type {}'.format(type(self.predictor).__name__))
    792         # set feature names
--> 793         return self.set_feature_name(feature_name)
    794
    795     def __init_from_np2d(self, mat, params_str, ref_dataset):

~/xxx/lightgbm/basic.py in set_feature_name(self, feature_name)
   1218         if self.handle is not None and feature_name is not None and feature_name != 'auto':
   1219             if len(feature_name) != self.num_feature():
-> 1220                 raise ValueError("Length of feature_name({}) and num_feature({}) don't match".format(len(feature_name), self.num_feature()))
   1221             c_feature_name = [c_str(name) for name in feature_name]
   1222             _safe_call(_LIB.LGBM_DatasetSetFeatureNames(
ValueError: Length of feature_name(100) and num_feature(20) don't match

Reproducible examples
import numpy as np
import pandas as pd

from sklearn.pipeline import Pipeline
from sklearn.decomposition import PCA
from sklearn.preprocessing import StandardScaler
from sklearn.metrics import mean_squared_error

import lightgbm as lgb

# generate random data

np.random.seed(312)
# training
n_trn = 1000
n_prm = 100
train_X = np.random.random((n_trn,n_prm))
train_y = np.random.randn(n_trn)
# early-stopping
n_val = 1000
test_X = np.random.random((n_val,n_prm))
test_y = np.random.randn(n_val)

# pipeline to be trained
pipe = Pipeline([('ss', StandardScaler()),
            ('pca', PCA(20)),
            ('lgbm', lgb.LGBMRegressor(max_depth=-1, random_state=314, silent=True, metric='None',
                                       num_leaves=20, n_estimators=50, learning_rate=0.15, importance_type='gain'))
                                   ])
# fit parameters
params_fit = {"lgbm__early_stopping_rounds":5,
              "lgbm__eval_metric" : 'rmse',
              'lgbm__eval_names': ['train', 'early_stop'],
              'lgbm__verbose': 5,
              'lgbm__eval_set': [(train_X,train_y), (test_X,test_y)]
              #'lgbm__eval_set': [(pd.DataFrame(train_X),pd.DataFrame(train_y)), (pd.DataFrame(test_X),pd.DataFrame(test_y))]
             }

# THIS WORKS
pipe = pipe.fit(train_X, train_y, **params_fit)
# this also works, which indicated that the issue is with the training, but rather with evaluation
pipe = pipe.fit(pd.DataFrame(train_X), pd.DataFrame(train_y))
# THIS DOES NOT WORK
params_fit['lgbm__eval_set'] = [(pd.DataFrame(train_X),pd.DataFrame(train_y)), (pd.DataFrame(test_X),pd.DataFrame(test_y))]
pipe = pipe.fit(pd.DataFrame(train_X), pd.DataFrame(train_y), **params_fit)

Hi @mlisovyi !
It's quite expected because scikit-learn's Pipeline is not aware of any validation data you are providing to LightGBM. So, you train on 20 features and validate on 100.
It's quite strange that everything is OK with numpy case...
I think you can fix this by something like the following:
# THIS DOES NOT WORK
params_fit['lgbm__eval_set'] = [(pipe.steps[1][1].fit_transform(pipe.steps[0][1].fit_transform(pd.DataFrame(train_X))),pd.DataFrame(train_y)), (pipe.steps[1][1].fit_transform(pipe.steps[0][1].fit_transform(pd.DataFrame(test_X))),pd.DataFrame(test_y))]
pipe = pipe.fit(pd.DataFrame(train_X), pd.DataFrame(train_y), **params_fit)

@mlisovyi did valid data preprocessing help in your case?
I think that due to not crash is case of numpy this issue can be transferred as a subissue to #812.

I will have time only on the upcoming weekend to look into it :(

Sure, no problem!

Your proposal works, but it has the downside that the transformation for the validation and train samples are different (and also it does not extend to an arbitrary length of the pipeline). I think, the best way to overcome it is to use
pipe_trf = Pipeline(pipe.steps[:-1])
pipe_trf = pipe_trf.fit(pd.DataFrame(train_X))
params_fit['lgbm__eval_set'] = [(pipe_trf.transform(pd.DataFrame(train_X)),
                                 pd.DataFrame(train_y)),
                                (pipe_trf.transform(pd.DataFrame(test_X)),
                                 pd.DataFrame(test_y))]

pipe = pipe.fit(pd.DataFrame(train_X), pd.DataFrame(train_y), **params_fit)

This yields the same results if I use numpy objects with the same code, as expected. But the result is different from the suspicious single-pipeline approach on arrays, so that calculayion does something wrong (and does not alert user about it :disappointed: : ).
The downside is that the transforms have to be fitted twice, which can be inefficient for CPU-intense methods like PCA. Maybe there is way to freeze transformers in sklearn pipeline and this way to avoid duplication of CPU time. But I've never used it. Otherwise one should just either use lightgbm outside of a pipeline, if one wants to use early stopping, or compromise on CPU time.
Note: to be able to get reproducible results, one should fix the random_state in PCA() initialisation, which i forgot to do in the original example


[fix] Correct num_accepted_tokens counting

There is a bug in counting the num_accepted_tokens.  Tensor accepted might be [False True False False False], in which case the num_accepted_tokens yield 1 while the correct result is supposed to be 0. Using accepted_mask.sum() will fix it.

I think it is accepted_mask that has the property that when some token is rejected, then so are all its successors. The accepted quantity may have such intermediate True value, which is exactly what Line81-Line87 tries to process. I think the accepted is the immediate result of the rejection sampling.

I see make sense. Other metrics such as number of draft tokens may also be inaccurate cc @LiuXiaoxuanPKU


Score for MLE model

I found that the score for MLE (order =2) is always 0, i.e. using bigram.
I am wondering why is that so.

I think this is the same as my issue (https://github.com/nltk/nltk/issues/3065)
Try removing the <s> and </s> tokens from the unigrams but not the bigrams, I used the following
list(everygrams(pad_both_ends(text.split(' '), n=2),max_len=2))[1:-1]


gjengroing chapter check formatting

Hi @HannoSandvik & @zanderVenter can you have a look at the rendered version of the gjengoring Chapter - I need to rescale the images to fit but thought best if you check it through first and I can make all changes at the same time.

The rendered version looks fine. I can't find any issues beyond the enormity of some of the figures.

Sorry for late response. Yes they are rendering well. But they are a bit large - is it possible to reduce the size a bit?

Thanks both - fixed the size issue with #70


Add CIL exe & fix posix tests

@devlead are you planning on putting the source files for the CIL exe in this repository as well?  I think it would make sense to have them here as well, in case things need to be updated.

@devlead are you planning on putting the source files for the CIL exe in this repository as well? I think it would make sense to have them here as well, in case things need to be updated.

Agree, this was a bit WIP, added it and instructions on how it was compiled.


Added split_type and combined_type on wide and logical

Yay! But tests

LGTM, will merge once tts is up-to-date.


Nexodus cable driver

This WIP PR can be used to try out nexodus as one of the
cable-drivers in Submariner.
To try this PR, one has to build the associated submariner-gateway
and submariner-routeagent images, push them to some private repo and
use image-override options while joining the cluster.
Also, the cable-driver should be explicitly specified as nexodus.
subctl join --cable-driver nexodus --image-override="submariner-gateway=quay.io/sridhargaddam/submariner-gateway:nexodus" --image-override=submariner-routeagent=quay.io/sridhargaddam/submariner-route-agent:nexodus ./broker-info.subm

bump

Is this still relevant?

This PR includes the high level changes to get a working POC with Nexodus cable-driver. However, both @mkolesnik and I feel that the proper way to support Nexodus would be to implement it as a cable-driver in Submariner.
Lets close this PR. We can refer to this PR as a reference while adding proper support.
Related Shipyard PR: https://github.com/submariner-io/shipyard/pull/1273


Enhancement: support tooltips for menu entries

It would be nice if JWM could support tooltips for menu entries.  e.g. people building their main menu from .desktop files could use the "Comment" field from the .desktop file.

I guess this didn't make it in to 2.3.3.  What is needed to get this done?
Tray buttons already support it, so the main code is there.   So the menu items need an extra property and some kind of default positioning???

2.3.3 isn't released yet, so there's still a chance I suppose.
I think the hardest part with implementing this would be routing the time notifications through to the menu and tracking the last mouse position and time of the last movement to know when to show a popup.  I don't think it would be difficult, but it's more than a few lines of code that would need to change.

So @technosaurus convinced me.  This is available as of snapshot 1302.  For example:
<RootMenu>
   <Program label="Terminal" tooltip="This is a tooltip">xterm</Program>
</RootMenu>


Updating webhook docs

Proposed changes

Updating webhook page to include new environment webhooks (not released yet) and webhook groups
Not merging this in until release time

Testing

Read through the generated website, confirmed it looks good

I really hate this:

Is there any way to control width of columns or whole table? My screen has plenty of space

hey @IaroslavTitov ; @sean1588 is looking into if we can make this cleaner; hugo takes the markdown table and auto converts it into some html table and we are limited by the width of the page. But he's going to test out a possible easy fix now.

I tried to get hugo to force some text wrapping in the first column, and hopefully get the other ones will layout better since they will have more space, since there seems to be just a few line items that are long in the first column that are causing it to take up so much horizontal space. It didn't help that much though.
However I tried toggling the screen width using the widget that we have in the upper right of the top nav and it looks pretty good when the screen width is toggled to make it wider. This is how it looks with the width toggled.


Testing, and rich Code info

Unit tests of non-networking functionality (utility functions, input validity checks)
Merge info about URN, URL, Classification (id and name), and Rank (subset specific) to each Code as it is returned from the KLASS API

I did not want to start a new separate branch for the rich Code info, as it would create merge conflicts with the changes needed to enable unit testing.

I will merge this myself in order to get the rich Code information changes into staging, as it is required to solve some current problems, and @alina-lapina does not have time to review atm.
It would be nice if @runejo could review after the fact.


How can I get  "model_annotations.aligned.scored.jsonl"

In #13 , You provide a code to calculate kendall correlations for each metrics. However, the code suggests to have a file called model_annotations.aligned.scored.jsonl as input. However, I can only find model_annotations.aligned.jsonl or model_annotations.aligned.paired.jsonl. Neither files have keys like metric_scores_{args.subset}. So If I have --input-file=model_annotations.aligned.jsonl, I got such error:
Traceback (most recent call last):
File "system_level_correlations.py", line 87, in
scores = data[f'metric_scores_{args.subset}']
KeyError: 'metric_scores_1'
The code I am using is
code.zip

Hi @vincent1rookie
I am assuming you solve this issue on your own? May I ask what you ended up doing as I am facing pretty much the same issue you outlined above.


chore(ci): increase timeout for e2e opendal test

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

https://buildkite.com/risingwavelabs/pull-request/builds?branch=dylan%2Frefactor_dml_table_stream_reader failed many times.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

Documentation

[ ] My PR contains user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

The test here is to write directly to the disk, wondering if any recent changes caused the timeout. Anyways, agree to increase timeout.

http://buildkitestats-env.eba-qbamkpd2.us-east-1.elasticbeanstalk.com/question/1-pr-pipeline-scatter-graph?step_name=end-to-end test for opendal (parallel)&build_number_min=0
Around build number 25181 it starts increasing. (I'm not certain if it is 25181 that actually causes it).
Can check around there:

The test here is to write directly to the disk, wondering if any recent changes caused the timeout. Anyways, agree to increase timeout.

http://buildkitestats-env.eba-qbamkpd2.us-east-1.elasticbeanstalk.com/question/1-pr-pipeline-scatter-graph?step_name=end-to-end test for opendal (parallel)&build_number_min=0
Around build number 25181 it starts increasing. (I'm not certain if it is 25181 that actually causes it). Can check around there:

Good catch. There are probably some changes causing these timeouts.


Endpoint and WebSocket url

This issue pertains to the following package(s):

[ ] GraphQL Playground HTML

What OS and OS version are you experiencing the issue(s) on?
Ubuntu 17
Google Chrome 65.0.3325.181
What version of graphql-playground(-electron/-middleware) are you experiencing the issue(s) on?
1.4.3
What is the expected behavior?
What is the actual behavior?
What steps may we take to reproduce the behavior?
Details here: https://github.com/graphcool/prisma/issues/1989#issuecomment-378795096

After upgrade<EMAIL_ADDRESS>and<EMAIL_ADDRESS>broken completely for relative url.

Fixed in the latest version. Please let us know if you still have that issue, then we reopen it!

@timsuchanek thanks!


[FEATURE] Link events events to companies

Introduction
It would be amazing if a murder happening in X building that is owned by Y company resulted in a dip in stocks. OR If person X who works for Y company is cheated on, the company can take a dip as well. Basically, making the cases affect the stock market would be amazing. As a much more advanced feature, having stocks be a reason for murder could be cool. You search a perps computer and find they have shorted the company they keep killing people at or something like that. OR you find that your perp has a 100 stocks in a company that took a nose dive and they blame the company for losing them money so they've started targeting employees of that company..... etc.

There are currently some stocks based on the in game companies but that's only a relative small portion of them.
It could indeed be interesting to have cases impact these stocks, I might indeed look into adding some kind of interaction here.
In the current version only the player can own stocks.

Implemented, will be part of stockmarket v2.0.8
Release is planned somewhere end of this week.


Where clause with and and multiple or support

Hi,
I am currently working on a scenario where I need to filter postgres data based on combination of multiple dates. I am using node/typescript.
Start and End Dates are in an array object.
Raw sql as below.
select  * from test_schema.sample_vw
where asset in ('A1', 'A2')
and plant in ('P1, 'P2', 'P3')
and (
(task_ts >= '2023-06-01' and task_ts <= '2023-06-30')
or (task_ts >= '2023-07-01' and task_ts <= '2023-07-31')
or (task_ts >= '2023-08-01' and task_ts <= '2023-08-31')
or (task_ts >= '2023-09-01' and task_ts <= '2023-09-30')
)
I tried multiple ways but could't achieve this using kysely qbery builder.
Any help would be highly appreciated.

https://kysely.dev/docs/examples/WHERE/conditional-where-calls

Thanks Got the solution in case anyone struggles with similar use case.
let query = queryBuilder({ dbUrl: this.dbUrl, schema: dbSchema })
.selectFrom('sample_vw')
.selectAll();
if (filter.filters.asset.length) {
  query = query.where('asset', 'in', filters.asset);
}

if (filter.filters.plantSection.length) {
  query = query.where('plant', 'in', filters.plantSection);
}

if (filter.filters.startAndEndDates.length) {
  query = query.where(eb => {
    const ors: Expression<SqlBool>[] = [];

    filter.filters.startAndEndDates.map(date => {
      const ands: Expression<SqlBool>[] = [];

      if (date?.startDate) ands.push(eb.cmpr('taskTs', '>=', date.startDate!));
      if (date?.endDate) ands.push(eb.cmpr('taskTs', '<=', date.endDate!));

      ors.push(eb.and(ands));
    });

    return eb.or(ors);
  });
}

console.log(query);
const result = await query.execute();

Umm did you look at the link I provided https://kyse.link/?p=s&i=hD4IjnC2bpoS8b8Zw5Pj. That's the solution.


feat: add support for new getData([])

Closes #44
What kind of change does this PR introduce (bug fix, feature, docs update, ...)?
Add support for new ERC725 getData. See issue above for details.
What is the current behaviour (you can also link to an open issue here)?
Does't support getData() with array version (bytes[] and bytes32[])
What is the new behaviour (if this is a feature change)?
Now, supports getData() with array version (bytes[] and bytes32[])
! Does not support GraphQL provider !
TODO:

[ ] perform extra 'e2e' tests with MetaMask to check ethereum provider mock implementation
[ ] merge duplicated code on web3 / ethereum providers (supports interface)

Codecov Report

Merging #48 (86c856e) into main (8f5fba4) will increase coverage by 4.22%.
The diff coverage is 81.52%.

@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
+ Coverage   80.04%   84.27%   +4.22%
==========================================
  Files           8        8
  Lines         406      477      +71
  Branches       88      100      +12
==========================================
+ Hits          325      402      +77
+ Misses         44       35       -9
- Partials       37       40       +3

Impacted Files
Coverage Δ

src/lib/provider-wrapper-utils.ts
92.30% <ø> (ø)

src/index.ts
78.70% <72.72%> (+0.27%)
:arrow_up:

src/lib/utils.ts
79.45% <73.80%> (+2.52%)
:arrow_up:

src/providers/ethereumProviderWrapper.ts
81.08% <75.00%> (-6.42%)
:arrow_down:

src/providers/web3ProviderWrapper.ts
89.13% <79.16%> (-10.87%)
:arrow_down:

src/lib/constants.ts
100.00% <100.00%> (ø)

src/lib/encoder.ts
90.42% <100.00%> (+18.51%)
:arrow_up:

src/types/Method.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Refactor Compressors from CompressorFactory to CompressorRegistry for extensibility

This PR refactors the CompressorFactory static singleton class and CompressorType enum to a formal CompressorRegistry and enables downstream implementations to register their own compression implementations through a new CompressorProvider interface for use in compressing Blob stores and MediaType data. This is different from Lucene's Codec compression extension points which expose different compression implementations for Lucene's Stored Fields.
**Note the following implementation details and plans **

new :libs:opensearch-compress library - the purpose of this library is much like the :libs:opensearch-x-content library, to provide supplemental compression implementations that are optional for opensearch API users. This design choice is for one reason: keep the core API library lean. Note this does not mean :server is breaking because the zstd classes are not in the core library. It also does not mean downstreams HAVE to take a dependency on :opensearch-compress (only if they want to use zstd as an option).
I'd like to refactor Deflate to the new compress library as well and only have the NoneCompressor in the core library. This doesn't mean server will break. :server module will still use Deflate as the default store compression as it will take a dependency on :opensearch-core and :opensearch-compress. Downstream implementations of the opensearch library (e.g., such as :server) can optionally take a dependency on the opensearch-compress library if they want to leverage Deflate and Zstd. The purpose, again, is to keep core library a minimal API implementation of the distributed lucene API.

relates #5910
relates #8110
relates #2447

Sorry to be a wet blanket but I want to play devil's advocate so please indulge me for a moment :) Sometimes the "keep the core API library lean" tenet is at odds with simplicity, and it is hard to beat the simplicity of a plain dumb static instance like CompressorFactory. This PR adds the complexity of another dynamically registered service provider and library, while providing two primary benefits: 1. fully extensible compression logic (for non-Lucene data) and 2. keeps the zstd dependency out of the core library. I would argue that point 1 is nice but I'm not aware of any pressing demand for that extensibility, which means I would lean towards the simple solution. Point 2 is harder to quantify. Am I missing other benefits or can you make a good case for why point 2 justifies the complexity?

I would argue that point 1 is nice but I'm not aware of any pressing demand for that extensibility,

I'm not sure we'll ever have an explicit demand that looks something like a "Integrate  Compression Algorithm" GitHub issue. We did, however, have Zstd recently contributed by a community member, and there are other alternatives out there that community and self hosted individuals will certainly integrate. This gives them the vehicle to do that without having to submit a PR to fork our code base, change that CompressorType enum, and rebuild to test. Given that compression is a source of performance impact, I think having this flexibility is an absolute win.

Sometimes the "keep the core API library lean" tenet is at odds with simplicity,

That tenet is only to support the "pros" column of a new opensearch-compress library. SPI is also just for extensibility. Even if the desire was to not make compression extensible I'm still planning to move the store compression classes out of :server as they are not :server specific. That is the core library would absolutely provide compression by default whether you use OpenSearch in a server mode or a serverless / cloud native mode.

Point 2 is harder to quantify. Am I missing other benefits or can you make a good case for why point 2 justifies the complexity?

I like this question because it does force thought on other things to think about. Another reason for keeping opensearch-core lean by providing the minimal surface area is that package sizes matter in cloud native and serverless deployments (e.g., 50/250mb lambda zip archive sizes). A function that depends on opensearch-core is going to be penalized by the size of opensearch-core and it's runtime dependencies. Having zstd in a separable library, and other compression algorithms, enables cloud native implementations to be selective about what they import and keep their quotas lean. So perhaps that means we should move Deflate into core so there's at least one compression choice, and let opensearch-compress provide other viable alternatives.

If DeflateCompressor could be cleanly moved into the core lib I might advocate for just moving CompressorFactory and not doing the whole SPI thing. But that's not the world were in now, so I think the benefits of extensibility plus the short term win of being able to keep DeflateCompressor in :server while refactoring the compression interfaces outweigh any downsides.

These test failures are unreal.


https://mibs.pysnmp.com/asn1/ 403 Error

https://mibs.pysnmp.com/asn1/ is down.

This is not a site for you to navigate in web browser, but the backend services used by the libraries like PySNMP and PySMI.


[Cognitive Service] properties in azure.mgmt.cognitiveservices.models.CognitiveServicesAccountCreateParameters

I'm trying to use azure-sdk-for-python. In azure.mgmt.cognitiveservices.models.CognitiveServicesAccountCreateParameters, the docs syas: properties (object) – Must exist in the request. Must not be null.
I'm confused what the properties should be. Is there any example?

Hi @dafashiandongni
I know there is no sample for this service right now, I will try to do one asap to unlock you.

@lmazuel That's great! Thank you so much. I really need it.

Hi @lmazuel,
I have the same issue. I wrote a test code to create cognitive service:
"""
test creating cognitive service.
"""

from azure.mgmt.cognitiveservices import CognitiveServicesManagementClient
from azure.common.credentials import ServicePrincipalCredentials
from azure.mgmt.cognitiveservices.models import CognitiveServicesAccountCreateParameters, Sku
from azure.mgmt.resource.resources import ResourceManagementClient

SUBSCRIPTION_ID = '-----------------'

credentials = ServicePrincipalCredentials(
    client_id='-----------------',
    secret='------------------',
    tenant='-------------------'
)

cognitiveservices_client = CognitiveServicesManagementClient(
    credentials,
    SUBSCRIPTION_ID
)

resource_client = ResourceManagementClient(
    credentials,
    SUBSCRIPTION_ID
)
resource_client.providers.register('Microsoft.CognitiveServices')

sku = Sku('F0')

param = CognitiveServicesAccountCreateParameters(sku, 'Bing.Speech', 'West US', {})

res = cognitiveservices_client.cognitive_services_accounts.create('speech-recognition', 'testshengtempspeech', param)

res = cognitiveservices_client.cognitive_services_accounts.list()

print res

And got:
RetryError: HTTPSConnectionPool(host='management.azure.com', port=443): Max retries exceeded with url: /subscriptions/----------------/resourceGroups/speech-recognition/providers/Microsoft.CognitiveServices/accounts/--------------?api-version=2016-02-01-preview (Caused by ResponseError('too many 502 error responses',))
The code is referenced to the ruby sample code.
Any suggestions about this?
Thanks

Hi,
I have no suggestion yet, I think I will wait for the new version to come in a few days and do the new sample at the same time. This is the PR that brings the meta-description of the new endpoint, that will be use as basis for the updated version of this SDK:
https://github.com/Azure/azure-rest-api-specs/pull/1090

Hi @jdneo
azure-mgmt-cognitiveservices is released and plugged in the Azure CLI. I think your problem should be solved now, could you check?
Thank you,

Hi @lmazuel
Thank you for your information. I tried to use the azure-cli 2.0 to create the cognitive service. And got the below error:
az cognitiveservices account create -n speech012 -g tempGroup --kind Bing.Speech --sku F0 -l global --yes

Notice
Microsoft will use data you send to the CognitiveServices to improve Microsoft products and services.Where you send personal data to the Cognitive Services, you are responsible for obtaining sufficient consent from the data subjects.The General Privacy and Security Terms in the Online Services Terms do not apply to the Cognitive Services.Please refer to the Microsoft Cognitive Services section in the Online Services Terms (https://www.microsoft.com/en-us/Licensing/product-licensing/products.aspx) for details.Microsoft offers policy controls that may be used to disable new Cognitive Services deployments (https://docs.microsoft.com/en-us/azure/cognitive-services/cognitive-services-apis-create-account).
Operation returned an invalid status code 'Bad Request'
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/azure/cli/main.py", line 36, in main
    cmd_result = APPLICATION.execute(args)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/azure/cli/core/application.py", line 203, in execute
    result = expanded_arg.func(params)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 278, in __call__
    return self.handler(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 478, in _execute_command
    raise client_exception
ErrorException: Operation returned an invalid status code 'Bad Request'

After some investigation, I find out it's because the F0 quota is exceeded, after change to sky to S0, problem solved.
I think it will be great if the error message can be more user friendly.
Thanks you

@jdneo for CLI bugs, please also file an issue at https://github.com/azure/azure-cli/issues

@derekbekoe @lmazuel Sure. And for me, I have no more question about this issue now. Thank you for your guys' help.

Thanks @jdneo
I'm closing this issue, since I think the initial question is fixed (I will make feedback to the team about the RestAPI unfriendly behavior).
Thanks!


Starred items

Q
A

Bug fix?
no

New sample?
yes

Related issues?
None

What's in this Pull Request?
This is my first submission to pnp/List-Formatting. Please advise if I am doing any steps wrong. Thanks!

Hi @wwcoop ! Thank you for the amazing sample!
However, I don't know the cause, but looking at the commits and changed files, this pull request contains commits from other users😢

Ideally, there should be only your own commits and changes to the sample files.
Sample pull request

Could you create a new branch from master and try the pull request again?

Yes - let me try again. I am BRAND NEW to Github and very confused by it. Thanks for your patience!

@wwcoop
Please let me know if there is anything you don't understand or stumble upon in making a pull request!

I'm sorry. I don't speak and hear English very well, so it's difficult for me😢
Just took a look at the branch "starred-items-and-docs" and it looks perfect! Could you try a pull request for it?

I will close this pull request!
Also, comments with links to the meeting will be deleted for good measure!


scaffolder: add dry-run + Template Editor

Hey, I just made a Pull Request!
This adds a new /v2/dry-run endpoint to the scaffolder backend. It receives a serialized template, including local files, and returns a serialized result of running the template. The execution happens directly in the scaffolder backend and unlike regular scaffolder task jobs it's synchronous, without streaming of logs etc.
To support the dry-run endpoint, actions can now declare whether they support dry-run or not, with not supporting it being the default. An action that supports dry-run is responsible for checking ctx.isDryRun on the handler context and to behave accordingly.
In addition to the dry-run endpoint in the backend, this adds a new template editor to the frontend. It builds upon the existing form preview, but uses the web directory access API to load in templated from the local filesystem for editing. This lets users both edit the entire template as well as template contents, as well as trying it out in practice using the new dry-run capability. Each dry-run result is stored in the frontend, where you can inspect the directory contents, log, and outputs.
Fixes #6568
Still a bit experimental but opening for review. We should likely consider adding this behind some sort of flag.
https://user-images.githubusercontent.com/4984472/165973801-24bffaee-53f6-4573-849d-827c22a0850a.mov
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Rugvip I'm getting the below error message when trying use this function
Failed to execute 'btoa' on 'Window': The string to be encoded contains characters outside of the Latin1 range.
root cause: .git folder has non ascii characters in the files

Hi @Rugvip , props for your work! This is a great feature.
I'm wondering if there is any documentation yet?

@dotressel Thank you! No docs beyond the pretty slim API surface. What kind of docs would you be interested in? For integrators and how to build out dry-running capabilities, or something more end-user oriented like where to find the template editor and how to use it?

I'd be interested in using the dry-run in our CI to continuously test our templates. I don't know if that is even possible but if so, it would be great to see this documented :)

@Rugvip Great update! Would there be a way to interact with the generated template? The dry run logs shows a location for the generated output but it's always empty. I'm assuming it gets deleted on dry run. Any way we can not delete it? It would help immensely in debugging issues


bumping credential package to v0.4.5

Codecov Report

Merging #476 (18e0b07) into main (00e0c9e) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #476   +/-   ##
=======================================
  Coverage   90.54%   90.54%
=======================================
  Files         112      112
  Lines       29085    29085
  Branches     2078     2078
=======================================
  Hits        26335    26335
  Misses       2715     2715
  Partials       35       35

Components
Coverage Δ

agent
78.83% <ø> (ø)

api
97.73% <ø> (ø)

common
98.68% <ø> (ø)

credentials
95.21% <ø> (ø)

crypto
93.81% <ø> (ø)

dids
97.65% <ø> (ø)

identity-agent
96.70% <ø> (ø)

crypto-aws-kms
100.00% <ø> (ø)

proxy-agent
96.70% <ø> (ø)

user-agent
96.70% <ø> (ø)


Add Kind and GroupVersion to SSA object

What type of PR is this?
/kind bug
/kind regression
What this PR does / why we need it:
SSA requires these fields to be populated. The error looks like this.
Incorrect version specified in apply patch. Specified patch version: , expected: kueue.x-k8s.io/v1alpha2

Controller-runtime event objects don't include these fields.
Note that is issue is transient, as re-queuing would obtain the object from the client, which properly adds the fields. That's why I'm not adding an integration test, as they were all passing.
Which issue(s) this PR fixes:
Fixes #406
Special notes for your reviewer:

/assign @ahg-g

Controller-runtime event objects don't include these fields.

Isn't that strange? should we file an issue with controller-runtime?

/lgtm

Opened kubernetes-sigs/controller-runtime#2016


Elasticsearch dns incorrectly resolved

Check CONTRIBUTING guideline first and here is the list to help us investigate the problem.
Describe the bug
Elasticsearch hostname resolution fails silently and wrongly resolved to localhost if there is an issue with /etc/hosts
To Reproduce
Configure an "elasticsearch" entry in /etc/hosts to something other than <IP_ADDRESS> and chmod +x /etc/hosts (a bug in a deployment script did that for us)
When running flutend with host elasticsearch in /etc/fluentd.conf this error is repeated:
could not push logs to Elasticsearch cluster ({:host=>\"elasticsearch\", :port=>9200, :scheme=>\"http\"}): Connection refused - connect(2) for <IP_ADDRESS>:9200 (Errno::ECONNREFUSED)

Note that the executable flag did not affect other services or the OS resolution, e.g. getent hosts elasticsearch returned the configured IP. Removing the executable flag on /etc/hosts or hardcoding the IP in /etc/fluentd.conf solves the problem, but the error is misleading so I though I'd report that.
Expected behavior
I expect the hostname to either be correctly resolved, or fail with an error like: Failed to resolve IP for the configured elasticsearch hostname: "elasticsearch". Resolving it to localhost instead is misleading.
Your Environment

Fluentd or td-agent version: fluentd --version or td-agent --version
1.10.4
Operating system: cat /etc/os-release
Arch Linux
Kernel version: uname -r
5.6.15-arch1-1

This seems fluent-plugin-elasticsearch issue.


[Feature Request] slack and discord bot with rag

Required prerequisites

[X] I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
[ ] Consider asking first in a Discussion.

Motivation
in order to have a better show case for customer service bot with camel, we need to build slack and discord bot with rag capability with support users to integrate with somain question.
Solution

camel slack/ discord service bot
use case notebook and tutorial

Alternatives
No response
Additional context
No response

Synced offline. The feature request is done for coding respective.


Adding a Shape makes a text unselectable

When I remove square, it is possible to select "Abacaba" by mouse:
module Main where

import Graphics.Collage exposing (..)
import Graphics.Element exposing (..)
import Color exposing (..)

main = collage 600 200 [
    {--}
    "Abacaba"
        |>show
        |>toForm
    --}
    {--}
    ,square 50
        |> filled blue
        |>move (150, 0)
    --}
    ]

Tested in Chrome&Firefox

Please file this under core https://github.com/elm-lang/core, not compiler

Added

Thanks @vilterp!


Need an idea on implementing incremental job for package manager

On a project I  use npm install which takes around 10s each time when there is no new dependency added. The command creates 200MB of locally cached packages.
Input is obviously both packages.json file holding the deps descriptions but also current state of the node_modules directory.
Not sure how to determine the output here.
I think only monitoring changes on packages.json would be enough and check node_modules existence.
Any idea on how to speed this up ?

I see now that I can connect packages.json with package-lock.json. The docs say:

package-lock.json is automatically generated for any operations where npm modifies either the node_modules tree, or package.json

This is fortunate case, but I am still not sure what I would do with some other package manager that doesn't have this, such as dotnet restore which seems to only put files in ~\.nuget folder.

In your task X with a package restore tool, with known inputs but no known or well-defined outputs, you can write a dummy output file (sort of log but the content is not important). Then configure your task using this file as outputs:
task Restore -Inputs ... -Outputs Restore.log {
	... # do your restore
	Set-Content Restore.log Done
}

That wouldn't work If I manually delete the package from nuget packages directory. But I guess that is upstream problem, to provide the way to know if dependencies are met or no. For example with chocolatey choco list invoke-build --lo -r returns version if invoke-build is installed or nothing if not If output is saved to file I can use it as an Input then with the mentioned dummy output.
Thx for the help.


Display daily growth factor

Is your feature request related to a problem? Please describe.
For a desease like covid19, we need to make use of every analysis possible. It would be great to show a daily growth factor of the disease in a map.
We can have a drop down to select daily growth factor of new cases, daily growth factor of deaths.
Daily Growth Factor of New Cases = Total new cases of yesterday/ Total new cases of today
Daily Growth Factor of Deaths = Total new deaths of yesterday/ Total new deaths of today

It should also have Daily Growth factor of Recovered patients
Daily Growth Factor of Recovery = Total new Recovered patients of yesterday / Total new recovered patients of today
This option also should be added to dropdown

You can see that we do display negative deltas under dailys, but I don't see why adding percentages would be a problem!
Based on the formula you've provided, it seems to be the case that the percentages can allow better understanding of the rise/fall of the slopes besides the numerical delta – and ofc visually the graph itself.

I think the percentages can be added but it'd be better if done inside the graph itself, but I'd like suggestions on where we should place them!

@jeremyphylemon We cab create a drop down for different graphs and allow user to select type of graph needed. In this case, daily growth factor would be a separate graph since the values generally revolves around 1 as x axis.
In this case, we would have 3 drop down options, Cumulative/Total, Daily, Growth factor.

It would also be better to add no of people tested vs no of active or recovered or deceased cases. That gives better insight on  need of rigorous testing too.

It would be great if daily change percentage can be added (or ratios as the OP mentioned). The number can be added right next to the absolute daily change number when hovering on a daily sample (right next to -36 in the screenshot above but in parentheses perhaps).
This number can give a more direct understanding of whether a "flattening" of the curve is actually being achieved or not.

Closed in favor of #1496


support for ajax-loaded forms

Hi
Everything works fine, except when you load the form dynamically. http://code.google.com/apis/recaptcha/docs/display.html#AJAX
When you load the form via javascript, the js document.writing the recaptcha doesn't show the recaptcha at all. I made an additional template to handle this case, the solution is dead simple and all you need to run is set CAPTCHA_AJAX (default False) setting.

On https://github.com/zalew/django-recaptcha/blob/master/captcha/templates/captcha/widget_ajax.html#L3 and L4 why are the lines commented? Why not just import them when a captcha form is encountered?

Because cross-domain asynchronous javascript requests don't work and you get "Recaptcha is not defined" . If you want to use the JS from google.com, you have to load it in the parent body (not the ajax-loaded html). If you really want to include it in the template, you have to host it locally. I prefer the first approach (host on google), to keep the js up to date.


port request

Hello,can you add ip:port for IPFile list ?...

Hello, this is a free to use / modify version, i currently dont have time to make any significant changes, but if you really need additional functionality you can message me to do it (paid) or you can inplement it yourself.

Closed.


Province Weights are borked, generally, and specifically for coal.

Hello there.
Firstly, I must thank you guys for making such an amazing tool and improving it regularly with updates!
Now, I feel like the absolute dev conversion feature was not intended to be used that often, and was just meant as a secondary option similar to the full culture conversion feature; and is not a priority in development.
But I have nonetheless found a few flaws with it, and have a few suggestions.
Multiple Province Mappings
I have noticed that the converter, when converting multiple EU4 provinces to one in Vic2, averages the weight of the EU4 provinces and converts it to Vic2.
And when converting one province from EU4 into multiple in Vic2, it multiplies it.
I feel like this was intended to be the other way, as this causes some oddities. (And major unbalances with places this is common, like India)
Such as Bukhara (a 50 dev, high income, high manpower province in my game) being watered down due to it being ridiculously big in Vic2 and also covering the 3 low-dev EU4 provinces of Khiva, Atakent and Kyzylkum.

And on the other hand, provinces like Constantinople, Cairo and Rome have double effects due to being converted into several Vic2 provinces.

This could be fixed by summing when converting several provinces to one, and distributing when converting one to several.
That would, however, raise the issue of major historic cities such as Constantinople getting split in 2 and losing their significance on the pop map 🤔.
Which could be countered by either changing the province mappings and having the famous provinces as 1 to 1 conversions, or by defining some ratio for these double results, like 7:3 Rome-Viterbo.
EU4's Trade Goods
Oh coal, coal, coal.
With its massive trade value of 10, coal completely destroys the weight balance.
While it makes sense, as coal provinces are quite important and useful in EU4, it doesn't make much sense from Vic2's perspective. Moravia, with far less dev, has as much people as Prague... because it makes coal!

(It's actually three times as much, since it's a 1:3 mapping 😅)
This would be fixed by not taking into factor the production income (value*produced), but just the goods produced. It would even make more sense, since Vic2 already has RGOs and there's no need to apply EU4's version of them into it in some way.
China, India, and Others
Let's address the elephant in the room. China and India convert as complete wastelands, and the rest of the world gets a huge boost to its population numbers.
There's of course many fixes suggested to this, but the one I personally like is converting the total dev on a regional scale.
For example, we look at China's vanilla pop and redistribute it in China based on the EU4 province weights; instead of doing such with the whole world at once.
I think to still keep it flexible, these regions should consist of one mega region encompassing most of the world, and some specific regions for where EU4's dev comes short of the regions' (usually massive) populations; like India, China, Korea-Japan, Manchuria, Indonesia, S/E Asia, Nigeria (and Sub-Sahara in general), Congo, Ethiopia, etc
Thanks for reading through my wall of text; I hope these suggestions were useful!
I am personally capable of helping with the modding-related side of these; but unfortunately I have no idea how the converter code itself works.

Hi, thanx for coming and pointing out these things. We're aware of them, as you guessed, but they are not as buggy as you may think.
Before I go into details, keep in mind the only reason we even have this absolutely broken option of pop redistribution is because of legacy reasons; we have group members that play MP with flat dev so it doesn't matter if china is wasteland or not - as all players start with all provinces at 3 dev, those players that develop their provinces will get that increase in popcount during vic2. You may have notices trade good reshuffle option, that's intended to be used with this redistribution method to spice things up.
This option is absolutely not meant to be used for any sane SP conversion, and redistributing pops according to regions is counterproductive for those players.
Now, as far as distribution goes, I'm aware you don't speak C++ but this is closest to english as it gets:

Now, what happens here:

Pop distribution for a province is determined according to that province weight compared to the rest of the world combined. This weight as you know, includes trade goods, however it's an annoyingly complex calculation that looks into trade values, production and tax values, all modifiers a province has, modifiers to manpower, trade efficiency and so on. For the moment let's assume it works "fine" (except for coal).
Then if the number of provinces we're mapping to is 2, we assign the province 55% of its intended value. Assumption is the other province in the pair will get also 55% of its own modified value, splitting this up.
If there are 3 or more provinces we assign 1/X * 1.15 which would be about 40%, then 30% and so on.
These numbers are not there for accuracy but tweaked so they give reasonably acceptable numbers for that MP group.

This means that for any normal EU4->V2 set of provinces, even with extreme reshaping, the resulting V2 provinces will all be smaller as the pops are diluted across target ones.
Enter coal.
Remember that insanely complex function above? It appears to take trade value of the province, multiply by production multiplier, resulting in a range 15-25 (compared to 5-7 for regular goods), then multiplies it again by 1.5  just for good measure, then piled ontop building weights, manpower, taxes, and development modifier (now much you developed the province since 1444) and there's the final province weight.
The result can be 2x or 3x the amount of a normal province, indeed.
Well, I suppose we'll have to rework this mechanism one way or another.

Then I suppose a new pop configuration, with summing for multiple -> one province mappings and a reworked province weight calculation and distribution method would be the answer, but it would also be a nightmare to implement I'm sure.
With the current system, however.
I have tested the dilution, and it does not seem to work.

The pop stays almost the same, with only a difference of 0,01%.
Based on that, my two cents is that it fails to properly get the number of vic2 provinces and ignores the if argument completely.
Tested with Stockholm as well, the same result. However; with the 1:3 mapping; one of the three provinces, Stockholm (the first one in the mapper), has 1.10% the population of others. I'm guessing Stockholm goes through it, but not the others?
Similar scenario in Moravia; with the first province defined in province_mappings.txt having 1.0 pop with the other two at 1.01% (the same ratio as the first and 2nd Istanbul)
I have no idea if this is useful or even makes sense, but here it is

This is solid investigation. Please upload the Vic2 save you used t get these results here or on forums so I may reproduce and hopefully fix this.

@MaxTheDesertManIran I have reworked this entirely in #626. Please grab the latest Picardy and see if the results make more sense now, and report back. If further tweaking is required I could use some input.

I tested with the Qi build.
The new dev + building cost system is pretty solid, and the issue with coal is no more.
However, the single->multi province conversions are still not divided.
(I saw that the if is gone from conversion)
I also see a problem with multi->single conversions, as they do not represent their EU4 equivalents properly.
I don't know how it's done currently, but here is my opinion:
Each Vic2 province should be checked for its EU4 equivalents, if there is more than one then the EU4 weights are summed up then given some sort of malus. (To avoid having it highly populated only because it equals five 3-dev ones in EU4 with regiment camps)

We assign pops now according to the EU4 province weight. Weight is calculated, for extreme reshaping, as:
weight = total dev + total building cost * 0.01
This means a province with 30 dev, and some 2k gold in buildings will have a weight of 50.
Province with 3 dev and zero buildings weights 3, and should be getting about 15 times fewer pops than the province above.
Onto N-to-1 multimapping:

Translated:

For every eu4 province that maps to THIS V2 province, sum up its weight.
Then divide the province weight by the number of incoming provinces (so /2 for 2, /3 for 3 etc) - averaging it out,
Then go create pops. For all demographics coming from every EU4 province, scale them down by how much it weights, compared to all incoming provinces.

This means if we have a weight 3 and weight 30 eu4 provinces mapping to a vic2 province, the demographics from the 3-dev province will create 10 times fewer pops than from the other one. I would love to see you confirm this, as the math is trivial and should work well.
1-to-N mapping is problematic, as I'm now realizing that if we have a high-weight source province, it will map into multiple high-pop provinces directly. We should be scaling the weight according to the number of target provinces as well.
If you concur with my reasoning and can confirm this is indeed happening, I can add more code to address this issue.
Again thank you for helping me out!

For the N-to-1 issue, I will examine Bukhara and Khiva in my game.
Based on the logs in this conversion:

Population per weight point is: 255201034 / 48312.2 = 5282.33
Bukhara got 137682 pops in my game, meaning it was given a weight of 26.
Khiva had 173741, meaning about 33 weight.

Calculating by hand:
Bukhara would get 50+23+5+5 weight from dev; with 29 weight from 2900 ducats worth of buildings. So 112 in total.
Khiva would get 15+25 from dev, 20 from 2000 ducats of buildings. So 60 in total.
So it appears that the provinces are getting averaged. 112/4 = 28, close enough to 26; and 60/2 = 30, close enough to 33.
I suggest we remove the averaging stage to get full pops from these provinces, as what happens currently gives weighted ratios (the larger provinces' pops have more of a say), but an averaged population which causes unbalance for this region's dev-pop conversion.
On the 1-to-N matter, I have only noticed it with highly weighted provinces, so that may be the case.

I believe this issue is now fixed. Please confirm.

Yep, the issue is no longer.

Thank you!


NODE_MODULE_VERSION missmatch after installing

So, I got this error when running this workflow on a new computer:

{
  "items": [
    {
      "title": "[error] Could not get data: Error: The module<EMAIL_ADDRESS>compiled against a different Node.js version using\nNODE_MODULE_VERSION 108. This version of Node.js requires\nNODE_MODULE_VERSION 115. Please try re-compiling or re-installing\nthe module (for instance, using `npm rebuild` or `npm install`).\n",
      "subtitle": "Press ⌘L to see the full error and ⌘C to copy it.",
      "valid": false,
      "text": {
        "copy": "```\n[error] Could not get data: Error: The module<EMAIL_ADDRESS>compiled against a different Node.js version using\nNODE_MODULE_VERSION 108. This version of Node.js requires\nNODE_MODULE_VERSION 115. Please try re-compiling or re-installing\nthe module (for instance, using `npm rebuild` or `npm install`).\n```\n\n-\nVSCode Workspaces 1.1.1\nAlfred 5.1.4\ndarwin 23.1.0",
        "largetype": "[error] Could not get data: Error: The module<EMAIL_ADDRESS>compiled against a different Node.js version using\nNODE_MODULE_VERSION 108. This version of Node.js requires\nNODE_MODULE_VERSION 115. Please try re-compiling or re-installing\nthe module (for instance, using `npm rebuild` or `npm install`)."
      },
      "icon": {
        "path": "/System/Library/CoreServices/CoreTypes.bundle/Contents/Resources/AlertStopIcon.icns"
      }
    }
  ]
}

Turns out, you need node v20.8.1 to run better_sqlite3. Which means, this repo needs to be installed with node v20.8.1, not at > v16. It's weird since better-sqlite3 also states that you need > v14.21.1... All of this makes no sense to me.
I have this workflow installed on my old computer and was able to update to 1.1.1 without changing node version. Maybe it was due to the fact that I had 1.1.0 already installed, I don't know.
But for anyone doing a fresh install and running into this error, looks like you might need to install the repo with node v20.8.1 to be able to use the workflow ¯\_(ツ)_/¯

Thanks for pointing this out! Seems like better-sqlite3 wants a currently maintained Node version. I changed the README.


fix possible values not getting populated from saved search

[x] joeferner
[x] sfeng88
[ ] mwizeman joeybrk372 jharwig

Testing Instructions:

Create a saved search with gender: Male
Run the saved search
Search filters should be populated with Gender Male

Points of Regression:
NO CHANGELOG

Why does the setValues method not actually set any values? It appears to wipe them out with this.values = [] but then never sets anything back into this.values.

The values are being set in this.values by self.addValue() here: https://github.com/visallo/visallo/pull/1339/files#diff-b98b045ac26391cf2365e27af370e6f2L88


Using experimental_new_converter generates opcode that is not found in kernel

@tensorflow/micro
System information

Host OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Colab
TensorFlow installed from (source or binary): tf-nightly on colab
Tensorflow version (commit SHA if source): Version 2.2
Target platform (e.g. Arm Mbed OS, Arduino Nano 33 etc.): ARM Mbed OS (Cortex M K64F)

Describe the problem
I am trying the hello_world (https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/micro/examples/hello_world).
Model : https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/examples/experimental_new_converter/keras_lstm.ipynb (with unroll set)
The model has operations SHAPE, GATHER, REDUCE_PROD etc which is not present in all_ops_resolver.cc. The Keras_lstm experimental example uses version tensorflowlite v2. I believe these operations are not supported in v1 . Any pointers on how to proceed.
Please provide the exact sequence of commands/steps when you ran into the problem
Step 1: Model generation using https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/examples/experimental_new_converter/keras_lstm.ipynb with tf.keras.layers.LSTM(20, unroll=True). Without setting unroll, I get more than 1 subgraph error.
Step 2: Use the example https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/micro/examples/hello_world to generate binary for Cortex-M
gives error "Didn't find op for builtin opcode 'SHAPE' version '1'"
My requirement is a working example of LSTM for Cortex-M (K64F).

Hi,
After looking at the code, I realised the operations SHAPE, GATHER, REDUCE_PROD are not supported on Tensorflow Lite micro yet and is available only on Tensorflow Lite.
Is there any alternative to run LSTM inference on Microcontrollers?
Regards,
Seema

@seemakumar8 This is a micro related issue,Please post this in micro repository  and move this issue to close status..Thanks!


shared/web/scripts: Uncomment exit on error

Uncomment forgotten commented instruction that exits if the identity
generation process doesn't produce the expected number of files.
NOTE I'm not sure if those lines were commented and were merged unintentionally or intentionally. In this PR I assumed that they were merged unintentionally if that isn't the case I will cancel the PR and I will open a new one removing those commented lines.

Please go ahead and merge


Transition from docopt to clap

This commit moves cargo-fuzz away from docopt to the more explicit
use of clap. This resolves #37.
Signed-off-by: Brian L. Troutwine<EMAIL_ADDRESS>

cc @ner0x652

This has been superseded. By other PRs.


Crash when calling submodel GET API after multiple users operating the same AAS

Overview
When multiple users access one AAS, the submodel GET API hangs due to a certain pattern of access, does not return a response, and also does not output logs.
the problem is not resolved even after retrying, because a data integrity problem may be occurred .
Specifically, if you call the BaSyx AAS Repository API in the order below, no response will be returned in step 4.

User A calls the API to get AAS "C" and retains its contents
User B calls the API to add submodel "D" to AAS "C"
User A modifies some of the results in 1. and calls AAS "C" modification API.
Call API to get submodel "D" of AAS "C"
This seems to be because the property .submodels in AAS GET API reponse is different from the submodel database.

Impact of this issue
Applications using BaSyx cannot be deployed in an environment with multiple users, therefore it is not possible to move from Proof of Concept to the next step.
Steps to reproduce
Environment

BaSyx version: 1.5.1

backend: MongoDB

MongoDB: 6.0.13
Docker: 24.0.7

Docker compose settings
version: "3.9"
services:
  aas-registry:
    image: eclipsebasyx/aas-registry:1.5.1
    ports:
      - 8082:4000
    volumes:
      - "${REGISTRY_CONFIG_PATH}:/usr/share/config"
  aas-server:
    image: eclipsebasyx/aas-server:1.5.1
    ports:
      - 4001:4001
    volumes:
      - "${REPOSITORY_CONFIG_PATH}:/usr/share/config"
  mongo:
    image: mongo:6.0.13-rc0
    environment:
      MONGO_INITDB_ROOT_USERNAME: root
      MONGO_INITDB_ROOT_PASSWORD: pass
    ports:
      - 27017:27017

A sample procesure

A User gets a AAS (ID: https//example.com/aas/0).
API operation:  GET /aasServer/shells/https%2F%2Fexample.com%2Faas%2F0

response json
{
    "conceptDictionary": [],
    "identification": {
        "idType": "Custom",
        "id": "https//example.com/aas/0"
    },
    "idShort": "test",
    "dataSpecification": [],
    "modelType": {
        "name": "AssetAdministrationShell"
    },
    "asset": {
        "identification": {
            "idType": "Custom",
            "id": "https//example.com/asset/0"
        },
        "idShort": "test_asset0",
        "kind": "Instance",
        "dataSpecification": [],
        "modelType": {
            "name": "Asset"
        },
        "embeddedDataSpecifications": []
    },
    "embeddedDataSpecifications": [],
    "views": [],
    "submodels": []
}

A User add a Submodel (idShort: test_sm1) to a AAS (ID: https//example.com/aas/0).
API operation: PUT /aasServer/shells/https%2F%2Fexample.com%2Faas%2F0/aas/submodels/test_sm1

request json
{
    "parent": {
        "keys": [
            {
                "idType": "Custom",
                "type": "AssetAdministrationShell",
                "value": "https//example.com/aas/0",
                "local": true
            }
        ]
    },
    "identification": {
        "idType": "Custom",
        "id": "https//example.com/submodel/1"
    },
    "idShort": "test_sm1",
    "kind": "Instance",
    "dataSpecification": [],
    "modelType": {
        "name": "Submodel"
    },
    "embeddedDataSpecifications": [],
    "submodelElements": [
        {
            "parent": {
                "keys": [
                    {
                        "idType": "Custom",
                        "type": "Submodel",
                        "value": "https//example.com/submodel/1",
                        "local": true
                    }
                ]
            },
            "identification": {
                "idType": "Custom",
                "id": "https//example.com/submodelelement/0"
            },
            "idShort": "test_element0",
            "kind": "Instance",
            "valueType": "integer",
            "modelType": {
                "name": "Property"
            },
            "value": 12347
        }
    ]
}

A User change a AAS's idShort property (ID: https//example.com/aas/0) based on procesure 1. But, submodels property is empty because of calling AAS GET API before calling Submodel PUT API.
API operation: PUT /aasServer/shells/https%2F%2Fexample.com%2Faas%2F0

response json
{
    "conceptDictionary": [],
    "identification": {
        "idType": "Custom",
        "id": "https//example.com/aas/0"
    },
    "idShort": "hoge",
    "dataSpecification": [],
    "modelType": {
        "name": "AssetAdministrationShell"
    },
    "asset": {
        "identification": {
            "idType": "Custom",
            "id": "https//example.com/asset/0"
        },
        "idShort": "test_asset0",
        "kind": "Instance",
        "dataSpecification": [],
        "modelType": {
            "name": "Asset"
        },
        "embeddedDataSpecifications": []
    },
    "embeddedDataSpecifications": [],
    "views": [],
    "submodels": []
}

User cannot access a submodel (idShort: test_sm1) data.
API Operation: GET /aasServer/shells/https%2F%2Fexample.com%2Faas%2F0/aas/submodels/test_sm1/submodel

Thanks a lot for pointing out this issue. Can you also provide the configuration files used in your setup?
Is there any reason why you are still using the 1.X Components of BaSyx? With the update of the meta model & REST API by the IDTA to V3, they are becoming obsolete.
The new components conforming to the newest specifications by IDTA can be found here:
https://github.com/eclipse-basyx/basyx-java-server-sdk

We will take a look based on your configuration and come back to you.
In regard to the migration:

Backward Compatibility: We're adding this API at the moment, the PR will be merged most likely this week. Is there anything else that you're missing?
Stable Version: We plan on releasing the very first milestone release this month. Thus, there would be a stable release.

Hi @hiroaki-yoshii ,
We tried to reproduce the issue based on the configurations and steps you provided, but we couldn't reproduce it at our end.

Is the issue occurring always or sometimes?
Please provide additional information such as logs of the AAS Server and AAS Registry.

@mdanish98
Sorry, the json content in step procesure 3 incorrect.
Please replace request json with the following json and perform steps 1 to 4 again.

request json

{
    "conceptDictionary": [],
    "identification": {
        "idType": "Custom",
        "id": "https//example.com/aas/0"
    },
    "idShort": "test",
    "dataSpecification": [],
    "modelType": {
        "name": "AssetAdministrationShell"
    },
    "asset": {
        "identification": {
            "idType": "Custom",
            "id": "https//example.com/asset/0"
        },
        "idShort": "test_asset0",
        "kind": "Instance",
        "dataSpecification": [],
        "modelType": {
            "name": "Asset"
        },
        "embeddedDataSpecifications": []
    },
    "embeddedDataSpecifications": [],
    "views": [],
    "submodels": [
        {
            "keys": [
                {
                    "idType": "Custom",
                    "type": "Submodel",
                    "value": "https//example.com/submodel/0",
                    "local": true
                }
            ]
        }
    ]
}

Thanks for providing the updated requested body.
We are analyzing it again, and we will get back to you.


new count function + changed default export to named export - generate

Summary

Changing default export to named export and naming it generate
Added count named export to solve #1
count will accept optional minLength and maxLength
Necessary tests are added, also the readme is updated with examples that specify the expected behaviour of this function

What are the specific steps to test this change?
Use the below code for testing
import { generate, count } from "random-words";

console.log(generate());
console.log(generate({ exactly: 5, wordsPerString: 2, separator: "-" }));

console.log(count());
console.log(count({ maxLength: 7 }));
console.log(count({ minLength: 5, maxLength: 7 }));

What kind of change does this PR introduce?
(Check at least one)

[x] Bug fix
[x] New feature
[ ] Refactor
[x] Documentation
[ ] Build-related changes
[ ] Other

Make sure the PR fulfills these requirements:

[x] It includes a) the existing issue ID being resolved, b) a convincing reason for adding this feature, or c) a clear description of the bug it resolves
[x] The changelog is updated
[x] Related documentation has been updated
[x] Related tests have been updated

If adding a new feature without an already open issue, it's best to open a feature request issue first and wait for approval before working on it.
Other information:
My old PR was closed by mistake where @boutell suggested making named export - generate with count which can be checked at - https://github.com/apostrophecms/random-words/pull/37

@BoDonkey Hello~when will you publish this to npm?

@ilaipi Our next scheduled release is June 6th.


enh: typing - replace all dict type by TypedDict

This will enforce the fields that need to be present in the dictionary:
At the moment if we say that some calldata that we know has the field name and age, we only specify that the dict should have the form dict[str, str | int] but this will also be a valid dict {"name": "John", "age": 25, "whatever": "Hello World!"}.
This is getting very confusing as the data coming from the client is evolving and it's hard to keep track of

Closed by #85


remove index.backup

PR into Azure/azureml-cheatsheets
Checklist
I have:

[x] read and followed the contributing guidelines

Changes

touch a file to retrigger deployment?

fixes #

looks like this is not needed


Fix misnamed link design tokens

The active and foreground tokens names were swapped. I don't think this affects the VS Code scenario, because the tokens are referred to by their correct TypeScript identifiers, but it breaks custom theming that overrides the CSS variables.

Thanks, for the catch and PR (!) @benmcmorran! Happy to merge this.

Thank you @benmcmorran!


chore(http-add-on): Use 3 replicas as minimum for interceptor

Use 3 replicas as minimum for interceptor to reduce blast radius in case one goes down.
More resilience tweaks will follow.
Checklist

[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] README is updated with new configuration values (if applicable)

Lol, I thought I did that - Sorry!


(GH-629) Separate app.manifest

By default highestAvailable is desired for UAC prompting behavior, but sometimes there are groups that provide some level of elevation (Network Configuration Operators Group) that also mean that UAC will
prompt. This is undesirable behavior in shops that might want to use self-service and have Chocolatey GUI run as the user that invoked without any elevation being requested. To allow this behavior, app.manifest can not be embedded, but must sit next to ChocolateyGUI.exe so that it can be changed for those situations that require it by those users.
Closes #629

This is in WIP because it appears there is some work to do to see that the ChocolateyGUI.exe.manifest gets included in the MSI

I took a cursory look but I don't know where I might need to change to be sure that ChocolateyGUI.exe.manifest gets included. The XSLT filters "ChocolateyGUI.exe" files out and then somewhere else the config and exe are put back in as named components. I think that the manifest needs the same treatment but I have not tracked down where that is just yet. @gep13 ?

@ferventcoder I have just pushed a new commit onto your branch.
With this change in place, the MSI that is generated now includes the new manifest file, and on installation, it can be found here:

Next to the ChocolateyGUI.exe

Wow, whatever Github is doing, I can't rename this PR to remove "[WIP]".

And now it works. Deleting these two comments

And now it works. Deleting these two comments

We should add these notes in as well for folks changing this so they don't scratch their heads - https://github.com/chocolatey/choco/issues/1292#issuecomment-304068121

I added some notes to keep folks from being perplexed IF they do change this file. This is ready for review.

@ferventcoder your changes have been merged, thanks for your contribution 👍


Copy events while dragging

Hi, when dragging an event and pressing the Ctrl key, the cursor changes to indicate a copy action. Is there a way to get this information in the onEventDrop callback?

in next release, the onEventDrop will include the DragEvent
you can get the info you need and more, new definition is like:
  onEventDrop?(
    event: DragEvent<HTMLButtonElement>,
    droppedOn: Date,
    updatedEvent: ProcessedEvent,
    originalEvent: ProcessedEvent
  ): Promise<ProcessedEvent | void>;

That's great. Thank you!


is there any way to render animations on a transparent background?

im trying to find a way to render my animations on a transparent background. Has anyone been able to do it?

With -t it render in alpha channel (.mov files)


How about using COG files instead of tiled images using TMS token?

Description
I have a Geotiff of ca. 40MB. When I use gdal2tiles for the zoom levels from 1-22, it converts to ca. 1.3GB!
I was wondering whether there is any better way of having overlay layers of aerial images over a large area without having to tile images just by pointing ID to a folder containing COG files.
Currently, I am using the commands below to prepare the images one by one as gdal2tile has issues with my .vrt file. :(
gdal_translate -of GTiff -b 1 -b 2 -b 3 -co COMPRESS=JPEG -co TILED=YES dop10rgbi_32_280_5652_1_nw.jp2 dop10rgbi_32_280_5652_1_nw.tif
gdal2tiles.py -z "1-22" --processes=20 dop10rgbi_32_280_5652_1_nw.tif tiles
Then I create a overlay layer in ID pointing to the place of tiles.
Using COG instead will definitely save a lot time! and space!
I found several github repos for COG with open source codes and demos, so I thought, it shouldn't be that difficult to add overlay layers with COG files.
https://www.cogeo.org/map/
I am wondering whether BING, GOOGLE or other companies are also using Gdal2tile, but with huge data servers because of which we can see their images in the background layers. Please enlighten me if you know this.
I searched and could not find anything related to this in ID at least within my search results.

Post on how QGIS UI for COG support works.


Item tooltip not displayed on touch devices

Hi everyone,
I am facing problems on showing a tooltip when using touch devices. I am using rc-tooltip component on itemRenderer and everything seems fine in desktop mode. However, when selecting any mobile device with touch enabled the tooltip is not been displayed.
The sandbox below shows the problem:
https://codesandbox.io/s/j4kzk82k69
Any ideas?
Thanks!

I'm having the same problem - also wrapping the items with rc-tooltip. I guess that both https://github.com/react-component/trigger and react-calendar-timeline are messing with the events in an incompatible way.

@ericraq honestly I am not sure what is the issue. I've used the material-ui tooltip before and it seemed to work. Now sure how to tackle this one though.

Hello, can I work on this?

@ericraq @marwej
Did you manage to solve this issue?


vertical mode: next/prev controls is not changed to up/down

Vertical mode uses keyboard/drag/swipe controls from horizontal mode
https://codepen.io/Grawl/pen/qXbwQQ
Steps to reproduce the problem

Set vertical: true
Try to navigate through slides using swipes/dragging/keyboard

What is the expected behaviour?
If I drag a slide using mouse or touchscreen to the top, it should move to top and show new slide on bottom of a slide.
If I drag a slide using mouse or touchscreen to the bottom, it should move to top and show new slide on top of a slide.
If I press Down key, it should move to top and show new slide on bottom of a slide.
If I press Up key, it should move to bottom and show new slide on top of a slide.
What is observed behaviour?
When I press Up or Bottom key, nothing happens.
When I press Left key, slide moves up.
When I press Right key, slide moves down.
More Details

Which browsers/versions does it happen on? Google Chrome 60
Which jQuery/Slick version are you using? jQuery 3.2.1 & Slick 1.6.0
Did this work before? I Don't know

Good catch. I'd say this is a priority. Wanna take a crack at a PR?

@leggomuhgreggo sorry I am not have enough time for it. life, you know.

Jafeel. Thanks for submitting the issue.

Oh you just need to add verticalSwiping: true

Okay, but what about to use up/down arrow keys instead of left/right on vertical mode?

I'm still seeing exact issue.
$(document).ready(function(){
$('.product-single__photos').slick({
slidesToShow: 1,
slidesToScroll: 1,
arrows: true,
fade: true,
swipe: false,
asNavFor: '.slider-nav'
});
  $('.slider-nav').slick({
      	vertical:true,
        verticalSwiping: true,
    swipe:true,
		slidesToShow: 3,
		slidesToScroll: 1,
		asNavFor: '.product-single__photos',
		dots: false,
        arrows: false,
		focusOnSelect: false
});
});

@Abderezai oh god why your code indented so random

@leggomuhgreggo @kenwheeler so what about to use up/down arrow keys instead of left/right on vertical mode?

Yea guys... i need same here. Work around?

Ok! my work around here for a nested slick.
$( ".vertical-slick" ).keydown(function( event ) {
       event.stopImmediatePropagation(); // Stops listen the keydown on a vertical element
});

$( ".horizontal-nested-slick" ).keydown(function( event ) {
	if(event.which == 38 ) { // UP / CIMA
		$("#vertical-slick-father").slick('slickPrev');
                    ...

  	}
	if(event.which == 40 ) { // DOWN / BAIXO
		$("#vertical-slick-father").slick('slickNext');
                    ...
  	}
});


Fixed the issue with DropTruncateIfExists

Previous inline comment causing syntax error. ElseIf hasn't been commented properly.
See the attached

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


Cypress/E2E: Reorganize and add keys for Elasticsearch autocomplete - users

Summary

Extracted tests into multiple cases and created into Zephyr. Then added new and existing keys here in Cypress.
Grouped tests into its separated spec file for ease of test maintenance.

Screenshots

Note: This is submitted on top of https://github.com/mattermost/mattermost-webapp/pull/7565. For ease of review, please see particular change at https://github.com/mattermost/mattermost-webapp/commit/c751c444d85aa3a35144d2268556dd104ddd70ca

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.


Turn codecov off for patch updates

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Describe changes
Turn codecov off for patch updates to avoid the annoying 'x' mark that happens

@alex-zenml  They hopefully use a reasonable yaml parser in which they should be equivalent: https://yaml.org/type/bool.html


Update workflow.rst

should be China instead of china

Description

Motivation and Context

How Has This Been Tested?

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[ ] Fix bugs
[ ] Add new feature
[ ] Update documentation

Thanks !


elemental2 Dom

I've just updated to the last version of the wrapper and I found a new dependency on the gwt-ol3-client which is not being included in the artifact. Can't understand this approach. It's really needed?
On the previous version I added the gwt-ol3-client jar has dependency into my project and that's it. Quick and simple. (Yes my project doesn't use maven)

@lobmarib Yes, there is a new dependency. I want to keep gwt-ol3 as simple as possible with less dependencies as possible. Elemental2 is needed for using DOM stuff and to keep gwt-ol3 prepared for the future. In release 4.0.0 this was introduced to support WFS format, especially to deal with XML-DOM elements, because the standard GWT-XML-API won't work with JSInterop or JSNI.

You really should use some build tool 😞. Probably maven or gradle.

@TDesjardins ok. Was decided to keep the old version so.
@ibaca Unfortunately, not my call.

@lobmarib You want to keep the old version? What are the pros of this decision? One more important dependency which you will anyway have sooner or later in your application ;-)

@TDesjardins Yes, we're going to keep up with the old version. For the moment the application is working fine with it and it seems there isn't much more to implement...
I know that, but before that we need to take other decisions... if widgets disappear we'll be in a bigger trouble.

I see. Good old widgets :-D


Depend on lodash 3.x until tested on a newer version

Fixes #1294
When using lodash 4.x, Restangular fails to GET lib/lodash/lodash.min.js, which returns a 404.
Specifying major version dependency for lodash 3.x until 4.x compatibility is fully tested.
4.x does ship with lodash.min.js in the dist/ directory, but it's unclear if everything else works as intended, so tying to 3.x for now seems like a good idea.

When using lodash 4.x, Restangular fails to GET lib/lodash/lodash.min.js, which returns a 404.

This is not the reason why Restangular is failing.
lodash v4 removed _.contains, which is used internally by Restangular. The function to be used now is _.includes (alias of the previous).

@sssilver Any idea what going on with this PR? Experiencing same issue as reported in #1294

No updates yet, just waiting for it to be merged.
On Wed, Jan 13, 2016 at 11:31 Rob Horrigan<EMAIL_ADDRESS>wrote:

@sssilver https://github.com/sssilver Any idea what going on with this
PR? Experiencing same issue as reported in #1294
https://github.com/mgonto/restangular/issues/1294
—
Reply to this email directly or view it on GitHub
https://github.com/mgonto/restangular/pull/1295#issuecomment-171372741.

@gustavohenke I'm actually seeing the 404 in the browser and then the module failing to institute because of that. Either way, the PR should fix it.

+1

Hey @sssilver Could you test with the current v1.5.2 that fixed the

_.contains is not a function

error? Does the problem still persist?

I'm going to close this for now. The 1.5.2 release should have fixed the issue. Feel free to re-open if there's still an issue.


[Bug]: Proxy: "Port 80 is in use." on remote server

Description
When trying to start the Traefik Proxy on one of my remote servers I get an error: "Port 80 is in use.", makes sense (somewhat anyway). The problem:
root@local-srv1:~# lsof -i -P -n | grep 80
root@local-srv1:~#

Minimal Reproduction (if possible, example repository)
Install Coolify and add a remote server, then try start the proxy on the remote server.
Exception or Error
In the GUI:
Error
Port 80 is in use.
You must stop the process using this port.
Docs: https://coolify.io/docs
Discord: https://coollabs.io/discord

From the GUI (logs section) (useless):
Error response from daemon: No such container: coolify-proxy

Version
v4.0.0-beta.323
Cloud?

[ ] Yes
[X] No

Check if your storage isn’t full?

Definitely not:
root@local-srv1:~# df -h
Filesystem                        Size  Used Avail Use% Mounted on
/dev/mapper/pve-vm--105--disk--0   47G  4.3G   41G  10% /

Might sound silly but have you checked that you aren't already running something on port 80?

@EmilEmilchen you need to use sudo lsof, on its own you won't see processes that the executing user cant see.

@EmilEmilchen you need to use sudo lsof, on its own you won't see processes that the executing user cant see.

I ran the command as root without sudo, note that I'm logged in with root.

Might sound silly but have you checked that you aren't already running something on port 80?

That's what the lsof in my first code block in the question is for.

@EmilEmilchen sorry youre right i missed that
can you run docker ps, is there a restarting container?

It works now! I'm not 100% sure what I did that fixed it but I think the most likely option is, that clicking "Revalidate Server" under Servers > General > Revalidate Server did the trick.