Installation does not recognize pyarrow using pip (windows)

I get this error with version 3.0.0 even when installing pyarrow first.
This installation of turbodbc does not support Apache Arrow extensions. Please install the pyarrow package. If you have built turbodbc from source, you may also need to reinstall turbodbc to compile the extensions.

Hi Daniel! Which operating system are you using?

Also, did you install using pip or conda?

Thanks for the response. I'm on windows 10 by the way. I am using pip, I can't create conda environments using turbodbc after succesfully running:
conda install turbodbc
The error I get is this:
PackagesNotFoundError: The following packages are not available from current channels:
  - turbodbc

So, I can't use neither :(

You can use turbodbc with conda when you create a conda environment with only packages from  conda-forge, i.e.: conda create -n turbodbc-env turbodbc pyarrow python=3.7.
For turbodbc Arrow support on Windows, you need to install turbodbc 3.1.0, it is not available with 3.0.0

You can use turbodbc with conda when you create a conda environment with only packages from conda-forge, i.e.: conda create -n turbodbc-env turbodbc pyarrow python=3.7.

That is what raises:
PackagesNotFoundError: The following packages are not available from current channels:
  - turbodbc

For turbodbc Arrow support on Windows, you need to install turbodbc 3.1.0, it is not available with 3.0.0

Interesting that you brought that up. When I pip3 install turbodbc=3.1.0 I get something like:
    src\cpp_odbc\level2\level1_connector.cpp(17): fatal error C1083: ...: 'boost/locale.hpp': No such file or directory
    error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2017\\Community\\VC\\Tools\\MSVC\\14.16.27023\\bin\\HostX86\\x64\\cl.exe' failed with exit status 2

The boost library has been successfully complied and installed globally as instructed here.
This is, however, an entirely different error for which I was going to open a new issue. I can do that if you find it convenient.

As @xhochy mentioned, turbodbc is available from the conda-forge channel:
(base) C:\> conda search 'turbodbc>=3.1'
Loading channels: done
# Name                       Version           Build  Channel
turbodbc                       3.1.0  py36h51f579c_0  conda-forge
turbodbc                       3.1.0  py37h51f579c_0  conda-forge

The error message implies that you haven't got conda-forge as a specified channel in your ~\.condarc config file.
You  can add the conda-forge channel manually on the command line by specifying -c conda-forge - e.g.
conda create -n turbodbc-env -c conda-forge turbodbc pyarrow python=3.7

As an aside, to make it easier to help you, you shouldn't just paste the final exception but should include the command you ran, the unabridged output of that command and the full traceback as that provides the context necessary to understand what the problem might be without resorting to  guessing what

That is what raises

might mean.
Also, you should never mix pip and conda packages, particularly for packages such as turbodbc with complex, compiled dependencies.

@dhirschfeld thanks for the input.

Also, you should never mix pip and conda packages, particularly for packages such as turbodbc with complex, compiled dependencies.

Agreed. It is not the case however; both attemps were running in different VMs (anaconda and pip). I tried anaconda out of curiosity after being asked.

As an aside, to make it easier to help you, you shouldn't just paste the final exception but should include the command you ran, the unabridged output of that command and the full traceback as that provides the context necessary to understand what the problem might be without resorting to guessing...

Thanks for pointing that out. In fact, my mistake in the first place is not having specified exactly how I was using this library. I have already edited the title of this issue to make it all clearer. Given that you should never mix pip and conda, I was considering using turbodbc for ETL processes in a production environment where conda is not used whatsoever. The reason for opening this issue was that I installed the library as shown here; I don't think I should give more details about the problems encountered running an anaconda environment if that is not the issue in the first place.
Despite the 'getting started' specs from the docs (where pip install is the way to get started), if this library should be used with conda instead of pip then this thread should not be considered as an issue after all. Otherwise, any help would be greatly appreciated.
In short, the problem is that installing both pyarrow and turbodbc, in that order, doesn't guarantee that turbodbc  will be compatible with pyarrow. Hope everything is clearer now and an apology for the confusion.

Is this an issue of installing from different conda channels? My pyarrow comes from conda default and turbodbc from conda-forge.  It seems turbodbc does not see pyarrow even if pyarrow is working fine for other use cases (e.g. pyspark, pandas, etc...)

Is this an issue of installing from different conda channels? My pyarrow comes from conda default and turbodbc from conda-forge. It seems turbodbc does not see pyarrow even if pyarrow is working fine for other use cases (e.g. pyspark, pandas, etc...)

No this issue here is using pip not conda. But in general don't mix and match packages from defaults and conda-forge. Either use all from default or all from conda-forge. Both package repositories are built such that they work only well when all packages come from a single source.

This is quite pity that not possible to use turbodbc with pyarrow from pip. Since our company has both libs in local pip repository after firewall, but not in conda :(

This is quite pity that not possible to use turbodbc with pyarrow from pip. Since our company has both libs in local pip repository after firewall, but not in conda :(

It's possible but quite fiddly as you need to make sure that pyarrow is installed before calling pip install turbodbc. In conda it's just straightforward working.

The CI builds of turbodbc actually used a pip-based installation of pyarrow in the past: https://github.com/blue-yonder/turbodbc/blob/11dfd7006a5694ce0c03a9327fa1fd1f408bfa67/.travis.yml
It is definitely possible. You just need to properly install pyarrow beforehand and make sure that you have all the necessary OS-level packages installed, see https://github.com/blue-yonder/turbodbc/blob/11dfd7006a5694ce0c03a9327fa1fd1f408bfa67/.travis.yml.

A workaround for this issue would be to add a pyproject.toml and enforce PEP517 builds where pyarrow is a strict requirement. Then the wheel builds would always work with pyarrow while still being an optional install requirement.

I am suffering from the same error. Not on windows but Ubuntu, even though the logs show that the wheels for pyarrow was installed. Turbodbc is not able to detect them and it fires the same error as the creator of this issue.

Same experience as @felipebormann here.

Same issue here.  If this isn't going to work with pip, maybe it shouldn't be on pypi...

No error at installation but I get this error when I try to make a connection to exasol: "This installation of turbodbc does not support Apache Arrow extensions. Please install the pyarrow package. If you have built turbodbc from source, you may also need to reinstall turbodbc to compile the extensions."
turbodbc[arrow] == 4.1.1
pyarrow == 0.17.1
Docker image: python:3.7-slim-buster


docker run error - no such file or directory..

docker run -d -e TZ=America/New_York -p 8118:8118/tcp -p 1080:1080/tcp --name gost --restart unless-stopped --entrypoint "/bin/gost -L=http://:8118 -L=socks5://:1080" ginuerzh/gost:latest
be1c819b104f8c693133d96c96c4b1d1934aed66fca6392f7017eae065ff8649
docker: Error response from daemon: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "/bin/gost -L=http://:8118 -L=socks5://:1080": stat /bin/gost -L=http://:8118 -L=socks5://:1080: no such file or directory: unknown.
Not sure why this doesn't work..

Turns out the arguments go at the end..
docker run -d
-e TZ=America/New_York
-p 8118:8118/tcp -p 1080:1080/tcp
--restart unless-stopped
--name gost
ginuerzh/gost:latest -L=http://:8118 -L=socks5://:1080
ALSO
the docker daemon.json file [/etc/docker/daemon.json] uses google dns by default..
This should be changed to your local recursoion provider..
I have a container doing dns so I changed mine to:
cat /etc/docker/daemon.json
{ "dns": ["<IP_ADDRESS>"] }
Where <IP_ADDRESS> is the address of my docker0 interface..
HTH


Tests should use SCS/SCT apps

We still use old redis based module but should use a choice of kafka/rabbit binder apps for integration tests.

I think the deployer tests should use an app that doesn't have any binder dependencies since the deployer is neither SCSt nor SCDF specific.


Sau 50p bot không chạy :)

[2020-04-06T21-46-37.773Z] [Facebook] Destroying Facebook Chat instance and creating a new one... (50 minutes clock)
[2020-04-06T21-46-37.773Z] [INTERNAL] Detected process is shutting down, handling...
[2020-04-06T21-46-37.791Z] [Facebook] Stopped Facebook listener
[2020-04-06T21-46-37.794Z] [INTERNAL] Stopped auto-save.
[2020-04-06T21-46-37.797Z] [INTERNAL] Unloaded plugin ArrowChat
1.0 by UIRI
[2020-04-06T21-46-37.800Z] [INTERNAL] Unloaded plugin Broadcast
1.3 by UIRI
[2020-04-06T21-46-37.803Z] [INTERNAL] Unloaded plugin COVID-19 Updater 1.0 by UIRI
[2020-04-06T21-46-37.806Z] [INTERNAL] Unloaded plugin DenyCommand 1.0 by UIRI
[2020-04-06T21-46-37.809Z] [INTERNAL] Unloaded plugin DisableBot 1.0 by UIRI
[2020-04-06T21-46-37.811Z] [INTERNAL] Unloaded plugin Economy-Gambling 0.6 by UIRI
[2020-04-06T21-46-37.814Z] [INTERNAL] Unloaded plugin Economy-Income 0.3 by UIRI
[2020-04-06T21-46-37.818Z] [INTERNAL] Unloaded plugin Economy-Rain 1.1 by UIRI
[2020-04-06T21-46-37.821Z] [INTERNAL] Unloaded plugin Economy 0.6 by UIRI
[2020-04-06T21-46-37.824Z] [INTERNAL] Unloaded plugin eXPerienceChat 1.2 by UIRI
[2020-04-06T21-46-37.827Z] [INTERNAL] Unloaded plugin Giftcode 1.1 by UIRI
[2020-04-06T21-46-37.830Z] [INTERNAL] Unloaded plugin LibBetterRandom  by UIRI, Sean McCullough, Takuji Nishimura and Makoto Matsumoto
[2020-04-06T21-46-37.833Z] [INTERNAL] Unloaded plugin MathSolve
0.3 by UIRI
[2020-04-06T21-46-37.836Z] [INTERNAL] Unloaded plugin MP3Voice 2.0.1NB by UIRI
[2020-04-06T21-46-37.838Z] [INTERNAL] Unloaded plugin Ping 1.0.0 by UIRI
[2020-04-06T21-46-37.841Z] [INTERNAL] Unloaded plugin SimSimi 1.4 Non-block by UIRI
[2020-04-06T21-46-37.846Z] [INTERNAL] Unloaded plugin Text-To-Speech 1.0.1 by UIRI
[2020-04-06T21-46-37.848Z] [INTERNAL] Unloaded plugin Weather 1.1 by XxBoomer, rev. UIRI
[2020-04-06T21-46-37.953Z] [INTERNAL] Saved data.
[2020-04-06T21-46-37.957Z] [INTERNAL] Closed local HTTP Model Server.
[2020-04-06T21-46-37.960Z] [INTERNAL] Closing bot with code 1...
console@c3c:js#
C:\Users\Administrator\Desktop\c3ccc\node_modules\fca-unofficial\index.js:347
appState.map(function(c) {
^
TypeError: appState.map is not a function
at loginHelper (C:\Users\Administrator\Desktop\c3ccc\node_modules\←[4mfca-unofficial←[24m\index.js:347:14)
at login (C:\Users\Administrator\Desktop\c3ccc\node_modules\←[4mfca-unofficial←[24m\index.js:567:3)
at Timeout.forceReconnect [as _onTimeout] (C:\Users\Administrator\Desktop\c3ccc\main.js:2834:46)
←[90m    at listOnTimeout (internal/timers.js:549:17)←[39m
←[90m    at processTimers (internal/timers.js:492:7)←[39m
[Loader] main.js throw 1 (not 7378278/RESTART). Shutting down...

đã fix


[question] Pretraining problem

I've got a relatively good policy on my own environment with the pretraining. After pretraining, I load the pre-trained model and test it in the following way:
model = SAC.load("pretrained_models_sac")
episode_reward = 0
obs = env.reset()

while True:
    action, _states = model.predict(obs)
    obs, rewards, dones, info = env.step(smoothed_action)
    episode_reward += rewards

    if dones:
        break

The episode_reward can always be more than 3000 which meets my expectations.
But when I try to train the pre-trained model with SAC like this (the learning_starts is set to a large value because I want it generates more trajectories before starting learning):
model = SAC.load("pretrain_models_sac", env=env)
model.learning_starts = int(1e5)
model.learn(total_timesteps=total_steps)
model.save("model01")

the log before starting learning is like (the total timesteps is smaller than learning_starts param so I think the learning has not started?):
--------------------------------------
| current_lr              | 0.0003   |
| episodes                | 220      |
| fps                     | 2271     |
| mean 100 episode reward | 56.9     |
| n_updates               | 0        |
| success rate            | 0        |
| time_elapsed            | 10       |
| total timesteps         | 23190    |
--------------------------------------

The mean 100 episode reward is only about 60.
In my understanding, learning has not started so the params are supposed to be the same as these stored in the pre-trained model. And then the mean 100 episode reward should not be such low.
I'm wondering if there is anything wrong with my way using the pretraining?

Hello,
You may find an answer here ;)
https://github.com/hill-a/stable-baselines/issues/929#issuecomment-656562248
Testing is different from initial exploration and here learning_starts sampling action randomly from the action space.

Hello,
You may find an answer here ;)
#929 (comment)
Testing is different from initial exploration and here learning_starts sampling action randomly from the action space.

Oh, I misunderstood the learning_starts here...
Thanks for your quick reply! @araffin I'll close this issue.


Only partial https when GOG (extension) downloads game info (api.gog.com) and pictures (gog-statistics.com)

After @JosefNemec closed JosefNemec/Playnite#2901 I looked into my playnite.log and extensions.log just to check if everything is also still working on my new set up PC and there aren't any quirks.
In the playnite.log I found
Downloader:Downloading data from http://images-1.gog-statics.com/[PictureID].png
Interestingly only PNGs downloaded via http. JPGs use https. Which is weird.
In the extensions.log I found
GogLibrary#Downloader:Downloading string content from http://api.gog.com/products/[GameID]
(which probably gets redirected and upgraded to https://www.gog.com/game/ if I interpret the log correctly. I have made a pull request to update the GOG API Base URL to use encryption.

Fixed by 5895dad12116b461cb1884a7eed6acf1ceeaad28


Yes, it's updated

It's really weird, because when I force the translation module to use the polish language it works.. Try doing a npm install again, this is maybe because I've added a dependency for the crash reporter.

When i will be in home, I will try again.
I use mom install after every new commit

I removed unhandled rejections/errors event from the main process since I get them on my crash reporter and made the translation module more error-friendly in the last commits. Check if it works now!

Fixed <3


fix file references in the Visual Studio Solution file.

What issue does this PR address?
Fixing the error when opening serilog-sinks-elasticsearch.sln in Visual Studio
Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements

[x] The commit follows our guidelines

@mivano, @jonasmidstrup Can we merge this change into dev, so that Visual Studio Solution isn't broken when we are opening the project?
Alternatively, missing files referenced directly in the sln file should be committed.

Done


VnV Typos

[x] page 1 title
[x] Sec 4.5 C in C++
[x] System/Unit test description: remove description
[x] 5.1.2 remove "note" from note

Closed with aff61fb20eb06b163a5687b1b956f5e37dd2059e
Merged in #71


Add python 3.5

I do need to add python3.5, but a few other things need to change here, too, especially dropping old versions. Thanks for getting it started!


Enabling Windows builds.

Just seeing if the windows build magically works now.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Need to re-render to enable Windows builds.

Thanks.

These appveyor and travis builds seem to have been queued for a mighty long time. Is this normal?

Is it normal? No. However, we have a pretty long backlog at this point. So we appreciate your patience while we get caught back up.

Windows fails with the same error as before when trying to run the tests:
BUILD END: simbody-3.5.3-vc14_201
TEST START: simbody-3.5.3-vc14_201
WARNING: conda-build appears to be out of date. You have version 1.21.14 but the
latest version is 2.0.3. Run
conda update -n root conda-build
to get the latest version.
The following packages will be downloaded:
    package                    |            build
    ---------------------------|-----------------
    simbody-3.5.3              |         vc14_201        12.6 MB  local
The following NEW packages will be INSTALLED:
    python:         3.5.2-2        conda-forge
    simbody:        3.5.3-vc14_201 local       [vc14]
    vs2015_runtime: 14.0.25420-0   conda-forge
(root) C:\conda\conda-bld\test-tmp_dir>call activate.bat C:\conda\envs\_test
TESTS FAILED: simbody-3.5.3-vc14_201
Command exited with code 1

Any suggestions on how to get more useful output from this error? I'm not quite sure where to start when debugging it.

Please make sure to re-render with conda-smithy version 1.3.3. There were some fixes there. Though not sure they are related to issues here or not.

This has already been re-rendered with 1.3.3.

I take that back. Only the master branch had.

Trying removing the Windows file checks, which are:
   - if not exist %LIBRARY_BIN%\\simbody\\simbody-visualizer exit 1  # [win]
    - if not exist %LIBRARY_INC%\\simbody\\Simbody.h exit 1           # [win]
    - if not exist %LIBRARY_INC%\\simbody\\SimTKcommon.h exit 1       # [win]
    - if not exist %LIBRARY_INC%\\simbody\\SimTKlapack.h exit 1       # [win]
    - if not exist %LIBRARY_INC%\\simbody\\SimTKmath.h exit 1         # [win]
    - if not exist %LIBRARY_INC%\\simbody\\SimTKsimbody_aux.h exit 1  # [win]
    - if not exist %LIBRARY_INC%\\simbody\\SimTKsimbody.h exit 1      # [win]

    - if not exist %LIBRARY_LIB%\\libSimTKcommon.dll exit 1           # [win]
    - if not exist %LIBRARY_LIB%\\libSimTKmath.dll exit 1             # [win]
    - if not exist %LIBRARY_LIB%\\libSimTKsimbody.dll exit 1          # [win]

Ok, my error is likely that the tests actually failed when trying to find these files on Windows, as it seems they are installed in different directories than I had originally figured out.
  -- Installing: C:/conda/envs/_build/Library/bin/simbody-visualizer.exe
  -- Installing: C:/conda/envs/_build/Library/include/Simbody.h
  -- Installing: C:/conda/envs/_build/Library/include/SimTKcommon.h
  -- Installing: C:/conda/envs/_build/Library/include/SimTKlapack.h
  -- Installing: C:/conda/envs/_build/Library/include/SimTKmath.h
  -- Installing: C:/conda/envs/_build/Library/include/SimTKsimbody_aux.h
  -- Installing: C:/conda/envs/_build/Library/include/SimTKsimbody.h

  -- Installing: C:/conda/envs/_build/Library/bin/SimTKcommon.dll
  -- Installing: C:/conda/envs/_build/Library/bin/SimTKmath.dll
  -- Installing: C:/conda/envs/_build/Library/bin/SimTKsimbody.dll

This has already been re-rendered with 1.3.3.

So re-rendering is not only necessary for updates. It is also necessary when changing the skip conditions or when changing yum_requirements.txt as examples.

What is an easy way to display on appveyor where things are installed to? My issue here seems to be that I don't have the correct paths to the files in the test section of the recipe, but I don't know how to find out exactly where they are installed on appveyor.

7642a96 was correct except for the dll names SimTKcommon.dll instead of libSimTKcommon.dll

Thanks, I've updated them.

Woohoo! Finally passes. Merging.


Fixed type hint error for hasParentInOptions()

Thanks @limingxinleo would you mind adding a test case that reproduces the error? I can't work it out myself unfortunately.

Thanks @limingxinleo would you mind adding a test case that reproduces the error? I can't work it out myself unfortunately.

New version https://github.com/jcchavezs/zipkin-php-opentracing/releases/tag/1.0.0

New version https://github.com/jcchavezs/zipkin-php-opentracing/releases/tag/1.0.0

Thank you very much ~

Thank you very much ~


Adding prometheus metrics for when the HPA using external metrics.

Adding a metrics server that produces Prometheus metrics (an error count and metric measure/value) for the external metrics used by only the HPA. Examples:
# HELP keda_hpa_scaler_errors_total Number of scaler errors
# TYPE keda_hpa_scaler_errors_total counter
keda_hpa_scaler_errors_total{metric="queuelength",namespace="default",selector="deploymentName=rabbitmq-keda-consumer-test"} 0
# HELP keda_hpa_scaler_metrics_value Metric Value
# TYPE keda_hpa_scaler_metrics_value gauge
keda_hpa_scaler_metrics_value{metric="queueLength",namespace="default",selector="deploymentName=rabbitmq-keda-consumer-test"} 17

Relates to issue #823
I am open to suggestions for what tests should be added.
I am wondering if the initialization of the metrics server should be placed in the adapter main.go and make the address and path of the metrics configurable.  Thoughts?

@tbickford this is great. I think that it would be great if you can target this for v2, there are already exposed  enpoints such as /healtz for liveness and readiness probes.

I am wondering if the initialization of the metrics server should be placed in the adapter main.go and make the address and path of the metrics configurable. Thoughts?

+1

Thanks @zroubalik! I'll rebase off of the v2 branch. Good point on the health endpoints - I'll remove. I'll also work on making the address and path configurable.

Marking this for review - I noticed it was beginning to fall behind in with the rest of commit history

Is keda_hpa_scaler_errors_total specific to a scaled object or a scaler in general?
This PR looks good to me but is solely focussed on HPA related metrics while we could surface more KEDA-oriented metrics as well I think.
I was wondering if it would make sense to introduce:

A metric for amount of scaledobjects that are deployed and how many are in error?

keda_hpa_scaler_errors_total sounds scaler related and don't surface the impact on all the scaled objects
Could have a label with the trigger type to understand what people are mainly scaling on inside the cluster

But this can be a dedicated issue if you want; but see value in this as well.
Let me know if there are docs to review!

keda_hpa_scaler_errors_total is the total count error for all scalers. The total scaled objects and errors along with the triggers would also be tracked at least by namespace and as often as the GetExternalMetric function is being called. The big thing to note is these metrics are solely for the metrics adapter so when replica count > 1 (HPA managed scaling).  I'll work on adding some of the suggestions in the PR. I have not added anything in the docs yet but certainly willing to do so. Is there a particular spot this should go?

keda_hpa_scaler_errors_total is the total count error for all scalers. The total scaled objects and errors along with the triggers would also be tracked at least by namespace and as often as the GetExternalMetric function is being called.

That's ok, in that case a label for the scaler name would maybe be interesting to add?

The big thing to note is these metrics are solely for the metrics adapter so when replica count > 1 (HPA managed scaling).

That's ok for me!

I have not added anything in the docs yet but certainly willing to do so. Is there a particular spot this should go?

I'm thinking of a new section called "Operate" here:

What do you think?

I'll add in the scaler name as well. Sounds great.

keda_hpa_scaler_errors_total is the total count error for all scalers. The total scaled objects and errors along with the triggers would also be tracked at least by namespace and as often as the GetExternalMetric function is being called.

That's ok, in that case a label for the scaler name would maybe be interesting to add?

Adding the scaler's name (trigger name) to the prometheus metrics seems to be an issue. It looks like the scaler's name is not being stored in the scaler itself or known outside of scaleHandler.GetScalers(scaledObject), although it is being passed to the factory method - https://github.com/kedacore/keda/blob/5e6ecb33b6b16c9566d42aa412c4adbd3182bddf/pkg/scaling/scale_handler.go#L365.
Would it make sense to add additional functions to the Scaler interface to allow this data to be retrieved (ie GetTriggerName and GetTriggerType) from what is being passed in?  This obviously would require modifying all existing Scaler factory methods.

My suggestion was actually for the scaler name and not the trigger name to be able to pinpoint issues with certain integrations.
On a per trigger level it would maybe be another metric or you'll have a ton of labels, but might be me. This feels more like a follow-up metric that we can still add later on.
But I'm just thinking out loud. in the end, people will ask for what they need so we get better understanding of the needs.

My suggestion was actually for the scaler name and not the trigger name to be able to pinpoint issues with certain integrations.

Can you clarify the scaler name? Is this getting the type from the scaler that is returned (ie *scalers.rabbitMQScaler) or something like rabbitmq which is represented as the triggerType?

My suggestion was actually for the scaler name and not the trigger name to be able to pinpoint issues with certain integrations.

Can you clarify the scaler name? Is this getting the type from the scaler that is returned (ie *scalers.rabbitMQScaler) or something like rabbitmq which is represented as the triggerType?

Just the type of scaler for example rabbit MQ, Azure Service Bus, etc which we list on https://keda.sh/docs/latest/scalers/

Does it have to have the exact same text? No, but preferably not with scaler in the label value

Added documentation in https://github.com/kedacore/keda-docs/pull/217
I have changed the metric for getting value to more expansive so each scaler is unique to it's trigger (when there are multiple triggers for the same scaler). Example:
keda_metrics_adapter_scaler_metrics_value{metric="queueLength",namespace="default",scaledObject="rabbitmq-keda-consumer-test",scaler="rabbitMQScaler",scalerIndex="0"} 0
keda_metrics_adapter_scaler_metrics_value{metric="queueLength",namespace="default",scaledObject="rabbitmq-keda-consumer-test",scaler="rabbitMQScaler",scalerIndex="1"} 8

This is an excellent contribution, thanks @tbickford!

These docs on https://keda.sh/docs/2.0/operate/ are great and straightforward!
An idea that comes up today is should we:

Document the labels we provide?
Add a sample of the output to give an idea?


database seed branch

Adding seed code on a new branch for initially seeding the database for local development:
Running the server on this branch now seeds the database which creates an admin user which can then create a new conference. Doing this results in the dashboard loading correctly for new speakers upon login.

How to PR this into a new branch in the main repo?


KAFKA-10319: Skip unknown offsets when computing sum of changelog offsets

In PR #8962 we introduced a sentinel UNKNOWN_OFFSET to mark unknown offsets in checkpoint files. The sentinel was set to -2 which is the same value used for the sentinel LATEST_OFFSET that is used in subscriptions to signal that state stores have been used by an active task. Unfortunately, we missed to skip UNKNOWN_OFFSET when we compute the sum of the changelog offsets.
If a task had only one state store and it did not restore anything before the next rebalance, the stream thread wrote -2 (i.e., UNKNOWN_OFFSET) into the subscription as sum of the changelog offsets. During assignment, the leader interpreted the -2 as if the stream run the task as active although it might have run it as standby. This misinterpretation of the sentinel value resulted in unexpected task assigments.
This is a port of PR #9066 to trunk.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Call for review: @vvcephei @mjsax @chia7712
Could anybody of you please trigger jenkins?

retest this please

JDK 11 and Scala 2.13 seems to be abort. retest this please

retest this please


Cdar 774/880 time sync script post process

PR Details
Description

Adding post process feature to the time sync checking script for cdasim.
It also reverts a bug introduced in previous PR: https://github.com/usdot-fhwa-stol/carma-analytics-fotda/pull/63
where many of the automatic script checking was turned off
Related GitHub Issue
https://github.com/usdot-fhwa-stol/carma-analytics-fotda/issues/61

Related Jira Key
https://usdot-carma.atlassian.net/browse/CDAR-880

Motivation and Context
vru master release prep

How Has This Been Tested?
sim pc 1

Types of changes

[ ] Defect fix (non-breaking change that fixes an issue)
[X] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that cause existing functionality to change)

Checklist:

[X] I have added any new packages to the sonar-scanner.properties file
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

Please just approve if ready, but don't merge. Thanks! @Saeid-Solei


config.xml

Hey! great job,
have a question, what about config.xml?
cordova.js:888 GET http://localhost:8080/config.xml 404 (Not Found) readConfig @ cordova.js:888 initAndShow @ SplashScreenProxy.js:154 (anonymous) @ SplashScreenProxy.js:165 cordova.js:863 [Browser][cordova.js][xhrStatusChangeHandler] Could not XHR config.xml: Not Found fail @ cordova.js:863 SplashScreenProxy.js:163 [Browser][cordova.js][xhrStatusChangeHandler] Could not XHR config.xml: Not Found
question is, this is a bug in your library and I need to do manual proxy this file?
it's working if I add '/config.xml': 'http://<IP_ADDRESS>:8000',

Hey @Kepro. Thanks for your feedback!
Can you explain to me in a little more detail?

Whati is the Cordova version?
What is the Cordova platform?
What is the CLI command you executed?
Is your Cordova project published on github?

cordova 9.0.0<EMAIL_ADDRESS>cordova on mac os -> platform ios + browser
command NODE_ENV=development cordova run browser -- -l
maybe the issue is that cordova-plugin-splashscreen needs access to config.xml but your plugin it not providing it?

cordova 9.0.0<EMAIL_ADDRESS>cordova on mac os -> platform ios + browser
command NODE_ENV=development cordova run browser -- -l

Thank you for the detailed explanation!

maybe the issue is that cordova-plugin-splashscreen needs access to config.xml but your plugin it not providing it?

Yes.
Currently this plugin does not provide it.
If it's possible, can you create PR?

@kotarella1110 hey! currently testing your latest version and works great! and I will create PR soon for that config, just busy... working on two projects

@Kepro Thanks for your testing!
I'm not in a rush! Take your time 👍

Hi, guys. I met this issue too. Seems it's been a long time, are there any chance it will be fixed?

uhm, sorry :-D I switched company so no more Cordova for me :-D

That's OK. Let me make a try.


Merge "Tony-Robbins"

[x] This PR adds a new quote from Tony-Robbins.

ce3a3fe45c80229334fa50ba2c73cc6d9412dd5d77d073872b9c0bacda08eb5da4242aa4a0c5ec9b5489fcdb994f648c24d9e19cb3cc44ed3fddc64a531dc870611934600770d8de978cd06

I assess the power of a will by how much resistance, pain, torture it endures and knows how to turn to its advantage. - Friedrich Nietzsche


update queryNFT to use EnterpriseAddress

Small bug to make sure the validator address is found properly on query side.

Will need to merge this over to testing branch too.

Will need to merge this over to testing branch too.
I will just keep merging the deployment branch into the testing one


Add functionality to delete a word in insert mode with Control-W.

Hey Greg, I'm not sure if I messed something up while reindenting your change or with some other recent change to my code, but this wasn't working for me.  This is what happened after I hit insert, targeted the middle of "copyright" in the LICENSE file, and hit C-w:
The above foo    ight notice and this permission

So it deleted everything to the left of the cursor, but then things ended up in a kind of weird state.
Also when you do another try, can you indent with tabs? :astonished:

Ah my fault, for some reason I was thinking that this was just supposed to delete a word most recently typed, so essentially just what was to the left of the cursor. I'll modify it to delete the entire word that the cursor is on and repost. And sure, haha sorry about that.

I think deleting just to the left of the cursor is fine, it's just that
it put in a bunch of extra spaces somehow. And at the tail end of a word
I think it was deleting just one letter to the right or something.
Probably just an off-by-one somewhere now that I think about it haha
On 02/14/2015 03:23 PM, Greg Mojonnier wrote:

Ah my fault, for some reason I was thinking that this was just
supposed to delete a word most recently typed, so essentially just
what was to the left of the cursor. I'll modify it to delete the
entire word that the cursor is on and repost. And sure, haha sorry
about that.
—
Reply to this email directly or view it on GitHub
https://github.com/TheTomster/llwe/pull/24#issuecomment-74390467.

Hmmm okay gotcha, haha alright guess I shouldn't be coding so late at night :P I'll see if I can find/fix the issue and will repost in a bit.


Skip the first cookie-cutter narration block in the loading screen

If anything it needs to be done for our own sanity. The amount of times I've listened to Sean Bean talk about the stone age is well beyond a healthy threshold

I think this is going to be another one of those ones that will have to wait until modding front end files starts working all the way. It would be another edit in LoadScreen.lua I think.

This has been effectively resolved in the latest patch. Closing!


Self-hosting on mono (OS X) fails for multiple URIs

Given this code:
				var uris = new []
					{
						new Uri("http://localhost:1234"),
						new Uri("http://<IP_ADDRESS>:1234")
					};
				using (var host = new NancyHost(uris))
				{
					host.Start();

					Console.ReadLine();
				}

The host will start without error, however navigating to http://localhost:1234 will yield a 400 (Bad Request) response, while http://<IP_ADDRESS>:1234 will work correctly.

Register localhost only. The should be able to browse to both localhost and
<IP_ADDRESS>
On Monday, 14 March 2016, Michael Teper<EMAIL_ADDRESS>wrote:

Given this code:
            var uris = new []
                {
                    new Uri("http://localhost:1234"),
                    new Uri("http://<IP_ADDRESS>:1234")
                };
            using (var host = new NancyHost(uris))
            {
                host.Start();

                Console.ReadLine();
            }

The host will start without error, however navigating to
http://localhost:1234 will yield a 400 (Bad Request) response, while
http://<IP_ADDRESS>:1234 will work correctly.
—
Reply to this email directly or view it on GitHub
https://github.com/NancyFx/Nancy/issues/2359.

By default, Nancy will rewrite http://localhost:1234 to http://+:1234 (see RewriteLocalhost on the wiki). This means that I'll bind to any adapter, so you should get by with localhost only.

@jchannon, @khellang
Here's a little more context.
You are right that if I only specify localhost, when things work, they seem to work as you describe: I can access the server via both locahost and loopback URIs. There's a question of why it works (browsing mono source, I don't see them supporting the "+" as host, but maybe I am just misreading code here https://github.com/mono/mono/tree/master/mcs/class/System/System.Net), but that's beside the point.
The problem I am trying to solve is that for a small percentage of my users, out in the wild, localhost doesn't seem to work. I have not been able to reproduce this in-house, so largely taking stabs in the dark here, and was attempting to experiment with explicit loopback / localhost registrations.

If you'd rather specify everything yourself, you just set the setting I linked (RewriteLocalhost) to false. It's really hard for us (and you) to be able to work anything out without more info, like what the environment looks like, i.e. Nancy version, Mono version, OS (version) etc. etc. There are many things that could get you a 400 response in return :smile:

Also, * and + prefixes in Mono are handled at https://github.com/mono/mono/blob/048ed1a6b2ee769a3aa488e9ab615012b270ba83/mcs/class/System/System.Net/EndPointListener.cs#L287

@khellang

If you'd rather specify everything yourself, you just set the setting I linked (RewriteLocalhost) to false.

Ah, I see, OK I'll try that.

Also, * and + prefixes in Mono are handled at ...

Yup, that's the code file I was looking at, but GitHub search must have handed me a copy from some ancient branch since it didn't have the "+" logic. Thanks, at least I don't have to wonder about this any more!

... what the environment looks like, i.e. Nancy version, Mono version, OS

NancyFx: 1.4.3
Mono: 4.2.2
OS: OS X El Capitan

GitHub search must have handed me a copy from some ancient branch since it didn't have the "+" logic

Heh, that must be a really ancient branch, cause that logic has been there since the class was introduced in 2005 :wink:

NancyFx: 1.4.3
Mono: 4.2.2
OS: OS X El Capitan

Is this the environment you're able to reproduce this on?

@khellang

Heh, that must be a really ancient branch ...

I was looking here: https://github.com/mono/mono/blob/3fda636ef7cf3c31625e260b595aed436e16e2ae/mcs/class/System/System.Net/EndPointManager.cs#L85  :sob:

Is this the environment you're able to reproduce this on?

Depends on which "this" you are referring to. It's my environment and I am able to reproduce the issue I described here (which is not a real issue, as you suggested). It's also a match for my user's environment (to which I don't have access) for whom my app crashes w/o giving up any useful info when initializing NancyFx and that's the problem I am ultimately trying to get to the bottom of.
We can close this case, as the following code works and enables me to access the endpoint both via localhost and loopback IP address:
var x = new HostConfiguration() { RewriteLocalhost = false };
using (var host = new NancyHost(x, new[] { new Uri("http://localhost:1234"), new Uri("http://<IP_ADDRESS>:1234") }))
{
	host.Start();
	// ...
}

Thanks!!

I've encontered a similar issue here running Mono on OSX El Capitan.
var host = new NancyHost(new Uri("http://<IP_ADDRESS>:8888"));
host.Start();

this works fine. But,
var host = new NancyHost(new Uri("http://localhost:8888"));
host.Start();

gives me runtime SocketException "invalid address in this context" in host.Start(); line.
But when deployed to a Ubuntu server, it works great.
And it works when add a HostConfiguration with RewriteLocalhost = false.
var conf = new HostConfiguration() { RewriteLocalhost = false };
var host = new NancyHost(conf, new Uri("http://localhost:8888"));
host.Start();

The strange thing is that, i can only access  on local machine with exactly the same uri in code.
Accessing http://localhost:8888 gives me "Bad Request (Invalid host)" when coding new Uri("http://<IP_ADDRESS>:8888").
Another strange thing, I can not access from local LAN by IP address, that said from machine <IP_ADDRESS> to this dev Mac <IP_ADDRESS>.
Using node.js package http-server, I have none of these issues: on local machine, http://<IP_ADDRESS> and http://localhost works equally, and acess by LAN IP with no problem.
I don't know if there are some quirks in my Mac machine, or is something not right in NancyHost or Mono?
My environment:
Mac OSX El Capitan
Nancy 1.4.1
Mono 4.2.3
My hosts file:
<IP_ADDRESS>	localhost
<IP_ADDRESS>	broadcasthost
::1             localhost
fe80::1%lo0	localhost

Need your help.

What if you use http://*:8888?
This isn't something we can do much about. Nancy simply takes the URL, replaces localhost with + (wildcard) if RewriteLocalhost is true and passes it to the HttpListener. HttpListener is Mono.

@khellang
using http://*:8888 gives System.UriFormatException "invalid URI: the hostname could not be parsed".

And http://+:888?

Same exception as http://*:8888

Update:
when deployed to Ubuntu server, the localhost version works, '<IP_ADDRESS>' gives "bad request(invalid host)".
code is just simple hello world.
running as mono nancytest.exe -d
Env:
Ubuntu 14.04 server freshly setup
Mono 4.2.3

Hmm. I wonder if this can be an IPv4 vs IPv6 issue? <IP_ADDRESS> is certainly a valid host.

update:
sorry for the Ubuntu server thing, I miss-configured the Nginx proxy.


Unintentional Breaking Change to TopicPartitionOffset in 2.1

Description
Version 2.1 (specifically commit 9826d794) introduced a breaking change (looks unintentional as you would think optional parameters would be non-breaking but they aren't). This is causing my applications that reference a library that uses a library that uses Confluent.Kafka to break when the entry app tries to upgrade to the latest Confluent.Kafka.
Library -> Confluent.Kafka 2.0.2
App -> Confluent.Kafka 2.1.1
Exception:
RLI.EventMessaging.Client.ClientHandler[0]
Method not found: 'Void Confluent.Kafka.TopicPartitionOffset..ctor(System.String, Confluent.Kafka.Partition, Confluent.Kafka.Offset)'.
System.MissingMethodException: Method not found: 'Void Confluent.Kafka.TopicPartitionOffset..ctor(System.String, Confluent.Kafka.Partition, Confluent.Kafka.Offset)'.
at Confluent.Kafka.Consumer2.Consume(Int32 millisecondsTimeout)              at RLI.EventMessaging.Kafka.Client.KafkaClient3.<>c__DisplayClass31_01.<ConsumeWithRetryAsync>b__0() in C:\code\RLI.EventMessaging\src\RLI.EventMessaging.Kafka\Client\KafkaClient.cs:line 421              at System.Threading.Tasks.Task1.InnerInvoke()
How to reproduce
KafkaBreakTest.zip
Library -> Confluent.Kafka 2.0.2
App -> Confluent.Kafka 2.1.1
Kafka Version: Any
Client Configuration: Basic
OS: Linux (probably windows too)
Critical: No
Broker Logs: Can't get that far
Checklist
Please provide the following information:

[X] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[X] Confluent.Kafka nuget version.
[X] Apache Kafka version.
[X] Client configuration.
[X] Operating system.
[X] Provide logs (with "debug" : "..." as necessary i.
[X] Provide broker log excerpts.
[X] Critical issue.
KafkaBreakTest.zip

Please reopen if this is still an issue.

The issue is completely reasonable: the breaking change of the public API must not happen with the increase of the minor version, according to the Semantic Versioning 2.0.0. By changing the constructor signature, you broke a lot of packages with dependencies on Confluent.Kafka v2.*
Considering that the solution to the problem is very simple and consists of literally replacing the constructor with an optional parameter with several constructors, this must be done.

I think the ticket should be reopened.

I still get an error System.MissingMethodException
Method not found: 'Void Confluent.Kafka.TopicPartitionOffset..ctor(Confluent.Kafka.TopicPartition, Confluent.Kafka.Offset)'.
The reason is an extra parameter in constructor was introduced (instead of creating a separate one with extra parameter leaderEpoch).

This is a breaking change

The 2.1 release has at least other breaking change. A method was added to the IConsumer interface which breaks outside implementations of the interface.
https://github.com/confluentinc/confluent-kafka-dotnet/pull/2027/files


Update PTOManager.md

updated

Docs Build status updates of commit f00ab44:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/payroll/PTOManager.md
:warning:Warning
View
Details

dynamics-gp/payroll/PTOManager.md

Line 152, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 215, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 225, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 278, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 308, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 317, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 328, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 336, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 355, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 411, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 415, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 471, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 502, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 506, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 614, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 736, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 751, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f00ab44:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/payroll/PTOManager.md
:warning:Warning
View
Details

dynamics-gp/payroll/PTOManager.md

Line 152, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 215, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 225, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 278, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 308, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 317, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 328, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 336, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 355, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 411, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 415, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 471, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 502, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 506, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 614, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 736, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 751, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 85db01e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/payroll/PTOManager.md
:warning:Warning
View
Details

dynamics-gp/payroll/PTOManager.md

Line 152, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 220, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 230, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 295, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 325, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 334, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 345, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 353, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 372, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 428, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 432, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 488, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 522, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 526, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 640, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 757, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 772, Column 1: [Warning: duplicate-alt-text - See documentation] Alt text 'Screenshot' is duplicated. Within an article, alt text must be unique.
Line 799, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Returning selected layers

This Pullrequest is the response to this pull request
Pull Request
I think I found the issue why one test hasnt passed. I rerun the tests locally and all of them have passed.
Sorry for the inconvenicence so far.
Description:
I primarily use st_folium in combination with LayerControl in Streamlit applications. However, one challenge I encountered is that there is no straightforward way to determine which layers are currently selected from the LayerControl. This limits functionality when trying to dynamically update components such as legends for each layer.
Here's a basic example of the issue:
The user toggles visibility of multiple layers using LayerControl.
There is no direct support to return the currently selected layers, which makes it difficult to update elements like legends that correspond to those layers.
To address this issue, I introduced a modification that allows st_folium to return the currently selected layers when interacting with the LayerControl. Now, when users toggle visibility of layers, the Streamlit app can respond accordingly, such as updating a legend for each selected layer. This greatly enhances interactivity.
Returned Format:
[{"url": url1, "name": name1}, {"url": url2, "name": name2},  ]

Thanks for re-submitting @reyemb, all tests pass!

Thank you guys.
I really appreciate your efforts.


Fix a discord between CLI arg and envvar of MQTT cert/key

I'm sorry, I forgot adding by #74

No worries, I've integrated your changes in the configuration update and already fixed this :-) See: https://forum.loraserver.io/t/looking-for-feedback-on-configuration-files/729.
I'm planning to release this today!

I appreciate your supporting and release it :beer:


twine build error on Mac

Hi, I'm trying to build on Mac (OS 10.12.6).
I've installed the dependencies but get the following error:
$ make
cd auth && make test && cd ..
make install
make clean
sudo rm -Rf build/*
Password:
sudo rm -Rf dist/*
make check
python3 setup.py check --restructuredtext
running check
sudo python3 setup.py sdist
running sdist
running egg_info
writing google_music_manager_auth.egg-info/PKG-INFO
writing dependency_links to google_music_manager_auth.egg-info/dependency_links.txt
writing entry points to google_music_manager_auth.egg-info/entry_points.txt
writing requirements to google_music_manager_auth.egg-info/requires.txt
writing top-level names to google_music_manager_auth.egg-info/top_level.txt
reading manifest file 'google_music_manager_auth.egg-info/SOURCES.txt'
writing manifest file 'google_music_manager_auth.egg-info/SOURCES.txt'
running check
creating google_music_manager_auth-0.7.3
creating google_music_manager_auth-0.7.3/google_music_manager_auth
creating google_music_manager_auth-0.7.3/google_music_manager_auth.egg-info
copying files to google_music_manager_auth-0.7.3...
copying README.rst -> google_music_manager_auth-0.7.3
copying setup.py -> google_music_manager_auth-0.7.3
copying google_music_manager_auth/__init__.py -> google_music_manager_auth-0.7.3/google_music_manager_auth
copying google_music_manager_auth/auth.py -> google_music_manager_auth-0.7.3/google_music_manager_auth
copying google_music_manager_auth.egg-info/PKG-INFO -> google_music_manager_auth-0.7.3/google_music_manager_auth.egg-info
copying google_music_manager_auth.egg-info/SOURCES.txt -> google_music_manager_auth-0.7.3/google_music_manager_auth.egg-info
copying google_music_manager_auth.egg-info/dependency_links.txt -> google_music_manager_auth-0.7.3/google_music_manager_auth.egg-info
copying google_music_manager_auth.egg-info/entry_points.txt -> google_music_manager_auth-0.7.3/google_music_manager_auth.egg-info
copying google_music_manager_auth.egg-info/requires.txt -> google_music_manager_auth-0.7.3/google_music_manager_auth.egg-info
copying google_music_manager_auth.egg-info/top_level.txt -> google_music_manager_auth-0.7.3/google_music_manager_auth.egg-info
Writing google_music_manager_auth-0.7.3/setup.cfg
Creating tar archive
removing 'google_music_manager_auth-0.7.3' (and everything under it)
twine upload -r testpypi dist/*
KeyError: Missing 'testpypi' section from the configuration file
or not a complete URL in --repository.
Maybe you have a out-dated '~/.pypirc' format?
more info: https://docs.python.org/distutils/packageindex.html#pypirc

make[1]: *** [test] Error 1
make: *** [test] Error 2

FYI I don't have a ~/.pypirc file. Is it required?
Any suggestions would be most welcome. Thanks!

You don't have to build to install this package.
Just pip3 install google-music-manager-downloader or pip3 install google-music-manager-uploader regarding what you need

Thanks for the quick and informative reply!
Do you have any idea where pip3 would install those packages on a Mac? they're not in my path and pip show google-music-manager-uploader returns an empty response.


Refactor RepoActionView.vue, add group support

make it able to "force reload", then the previous pending request won't block the new request
make it support ::group::
add some TS types (but there are still many variables untyped, this PR is large enough, the remaining types could be added in the future)

AbortController definitely seems like the best option. I was actually thinking about using it too.

Maybe we can put the triangle at the start of the line like GitHub has:

Maybe we can put the triangle at the start of the line like GitHub has:
Also, I think with that, no z-index tricks will be needed.

I do not see a quick approach to do so. The z-index is the simplest approach and the most performant, it only uses <details> and a few lines of CSS, not ideal but good enough.
Since this PR is large enough and has already done many things, I think the UI improvement doesn't block and the UI could always be improved in following PRs. And not only the UI, there are indeed many TODOs for this Vue component.


Server Unable to Parse Multiple Cookies in Request

When parsing the headers of a request, Cookie::fromRaw is used to parse the "Cookie" header. fromRaw, however, appears to be written to parse the "Set-Cookie" header from a server response. As a result, the server-side Request object can never have more than one Cookie in its CookieJar object. Any additional cookies are thrown into Cookie.ext.

Yes, you can see the bug if you run "./build/examples/run_rest_server" and "curl localhost:9080/auth -b "key1=value1; key2=value2; key3=value3"". The server only prints a single cookie instead of the expected three.
I've got a patch that should fix the problem, so I'll see if I can put together a unit test and submit a PR. It's alright if the test just goes in "tests/cookie_test.cc", correct?


Sample quality is not good

I tried this recipe https://github.com/SJTMusicTeam/Muskits/tree/main/egs/namine_ritsu_utagoe_db/svs1 with some model configurations (naitve_rnn, naive_rnn_dp, transformer) but am unable to get good samples so far.
Samples: https://www.dropbox.com/sh/58klvse0s0vbdvy/AAAcGtfnWKC4iyApqSSSijJNa?dl=0 (w/ HiFiGAN trained on the ritsu corpus for 1000k steps)
Am I missing something? Or is it the current limitation of Muskits?
I also tried kiritan corpus a few months ago but unable to get good samples too https://soundcloud.com/r9y9/muskit-kiritan-svs1-native-rnn

Any comments are appreciated.

Thanks for the follow-up. I checked the quality, which was similar to ours at the time. I would say it is the current limitation for now.
Recently, we have been investigating several major updates which could further improve the quality. Please stay tuned!


Improved: DataResourceWorker: New method getMimeType(...) (OFBIZ-9403)

Improved: DataResourceWorker: New method getMimeTypeFromFileName(...) with extracted (refactored) code from getMimeType(...) method (OFBIZ-9403)
The method DataResourceWorker.getMimeType has been refactored into two separate methods.

@cshan-ecomify I think we should take the defaultMimeType from applications/content/config/content.properties instead of a hard coded one, can you please change it accordingly?


Hello,
So I set up a Rocketchat snap on Ubuntu 16.04 using AutoSSL (https://github.com/RocketChat/Rocket.Chat.Docs/blob/master/3. Installation/4. Manual Installation/Ubuntu/Snaps/AutoSSL.md)
and i get in Chrome :

Caddyfile:
https://xxx.xxx:8080 {
  proxy / localhost:3000 {
    websocket
    transparent
  }
}

Did I miss something ?

OK i figured out that RocketChat is simply requesting external resources which are not private. That's all. Closing.

See #7490


Help, please. Issue with safehax. Complicated story.

Hello everyone,
So I have a N3DSXL and a few days ago I followed the guide to install Hombrew etc. Everything went perfect. I'm on <IP_ADDRESS>U and I didn't have a single hitch.
I was happy and using my newly hacked 3DS, downloading some games and dumping some of my own cartridges, when my SD card got corrupted and refused to work anymore. It just died on me.
So I decided to buy another micro SD, and did a system memory format from the 3DS settings. I figured I might as well start the process again with the new SD card.
So I did that and now I get stuck trying to run safehax into decrypt9. I just get a blank screen or the "failed init" error. I looked around and tried all possible solutions (putting a cartridge in before running safehax, etc) but nothing seems to work.
What can I do? I'd like to just start over from zero, if possible. Don't mind the data loss nor redownloading everything.
A couple observations in case they're useful:

I still have the backup files from the otp and the files9 folder with all the tickets I got the first time I hacked my 3DS.

I also manually copied all the contents of the SD card into a folder in my PC AFTER I finished hacking everything.

After the system format, I still get the Luma menu if I press reset when turning on the console. how do I get rid of this?

Thanks a lot in advance.

A9LH is installed to NAND. There's no need to reinstall A9LH just because you swapped SD cards.
You should be able to just copy the old SD card contents to the new one, then use Luma's chainload feature to load Decrypt9WIP.
(The reason Luma config is still showing up is because you copied arm9loaderhax.bin to CTRNAND as per the last steps in the guide, which allows for SD-less CFW.)

Thanks! Will this work even after having formatted the system memory on the 3DS? I'll give it a try.

"Format System Memory" doesn't actually format CTRNAND. (If it did, all the system software would be gone.) A better name would be "clear all user data and generate new IDs".
Luma in CTRNAND (and A9LH) isn't affected by a system format.

Great! So I copied everything to the new SD card and things are working well, except for one thing.
All my legal digital games are back there, but none of the manually-installed .cias are. I downloaded a bunch of stuff from the freeshop and dumped my own cards, but those don't show up in the home menu anymore.
I know the content is still there, somewhere in the depths of the "Nintendo 3DS" folder, but how do I make them show up again in the home menu as installed?
When I go into the "tickets" folder with FBI, I see a long list of title IDs. Some are colored green (I can tell those are the titles that show up as installed) and some are colored red (those are the titles I downloaded from the freeshop//dumped myself). If I select one of the titles in red it gives me the option   "Install from CDN."
Should I do that manually for each title that doesn't show up (that'd take ages) or is there a way to have them show up again as installed in the home menu.
Again, I can't express enough how greateful I am for your help!

Formatting usually resets the SD encryption key, which makes the contents of the "Nintendo 3DS" directory useless. System Format usually deletes these files.
Do the titles show up in Data Management? If they do, then there's something screwy with the ticket. Otherwise, you'll need to reinstall the titles.

They don't show up in data management.
By reinstall you mean what I mentioned before, doing it from the "tickets" folder in FBI?
I've been reading a bit and indeed it seems to be a problem with the directory in "Nintendo 3DS."

I think I've figured it out. Just need to reinstall them. It takes a while but everything seems to be back to normal. Thanks again for the help!


Reactivity recording with a song in Unity for 360 video export

So, these audio reactivity brushes are a real pain to capture in Unity. Tilt Brush documentations' technical way of downloading 360 videos doesn't allow for real-time sync with audio. By bringing Tilt Brush into Unity and I'm able to create a flawless app and custom interactions. But, try to capture everything in real-time 360 video from VR?!? Yeah right. All the programs, free and purchased, I've found cannot complete this task in real-time. The best only previews in real-time (OliVR). The programs are in alpha or beta on real-time 360 capture. I'm not even trying to move the camera through the painting. I just want one point through a 3 1/2 minute song.
Any ideas? Or will Tilt Brush/Unity allow for sync rendering with these brushes in some way I haven't discovered yet?
If you need more details about what I'm doing let me know. Normal people just animate objects, which are easy to capture. But these brushes are a new beast. Recording the animations in Unity doesn't seem to work, though most recorders just used transformation settings and the brushed don't touch those settings.
Susan

Closing for inactivity; hope the reply was helpful


If yes, I'm ready to work on it and update this PR.
If not, thank you for your work, I was learning how shells work and found your project. It helped me with understanding some internals.

Looks great. Apologies for you having to wade through the code in job_unix which is still quite a mess. I should have some time this weekend to review this more thoroughly.


Fix issue where writing permissions while the file is open fails

Fix an issue where permissions cannot be written to files while they are still open under windows.
Presents as "Error: Access is denied. (os error 5)"

Causes issues downstream in esp-idf-sys when compiling under windows. See https://stackoverflow.com/questions/72969132/access-denied-when-building-esp-idf-sys-for-rust-project-for-esp32-on-windows-10

Thank you very much for this fix (I realize now that I also have this problem on Windows...).
Seems like we didn't test https://github.com/esp-rs/embuild/pull/51 on Windows which introduced this issue.
I will release 0.29.3 shortly.


ISSUE-4450 Added ServerAliveInterval among the default ssh options

and sorted them alphabetically to improve readability
This solves issue #4450

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "bugfix/ISSUE-4450-use-ssh-serveraliveinterval" git@github.com:krufab/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

LGTM


RFC Alternative way for serving assets and reactor internal requests

The Issue
Currently assets for elm-reactor html pages (at least the wrench icon, which is the one I confirmed this for) is serving assets through the url scheme "/actual/path/to/asset.fileending".
The issue is, that that is the exact same url scheme used for the regular repository content. This could potentially cause issues if a repository were to contain a file say called "/_reactor/wrench.png".
I believe this issue can be trivially fixed via the use of a different url scheme for reactor internal resources through the use of url params.
This would have the added benefit that matching for asset requests would be easier.
The Proposal
I propose the following general structure:
"/?resource=resource-type&path=/path/to/resource"
Example
In the case of our wrench
"/_reactor/wrench.png" would turn into "/?resource=asset&path=/_reactor/wrench.png".
For dynamic JavaScript it could be
"/?resource=dyn-js&path=/creator/factory/value.js".
Or potential future ajax folder index operations
"/?resource=index-dir&format=json&path=/folder/to/index"
Since I am currently working on the reactor anyways to integrate the new fronted it'd be little more work for me to implement this. However I'd like to garner some feedback first wether the community agrees that this is a good idea and whether the proposal itself is good and/or whether people have suggestions for improvements to the proposal.

Isn't it possible that a JS file or Elm file looks at these queries and does something with them? My main concern would be that the probability of overlap with names like resources and path and format would be way way higher than with the current _reactor path.
How would you think about this risk?

I don't know of any scripts that wildly modify href tags just because there's some query parameters in them. Which is not to say that such scripts do not exist of course. Modern Javascript communicates with html through data-... tags and classes though, I don't think it'll touch the href's.
Besides, since we are the ones developing the full stack with client and server code we'll have control over what javascript runs.
Online IDE's like cloud 9 don't have links at al, they do everything with event handlers in javascript and ajax.
Github prefixes all files with /blob/ and loads assets from a subdomain assets-cdm.github.com which avoids overlap.
I generally don't think that arbitrary javascript modifies these hrefs, since that would require non trivial knowledge about the backend or force the backend to have a particular structure which is not desired.
Furthermore since all javascript code is written by ourselves we can say for certain that it (currently) does not modify any url's on the site.
We can always make the query more specific though by adding parameters.

FWIW, generally, have a path (or a set of paths) that are "vendor reserved" by convention only is enough. Devs will adjust/embrace as long as the plan is reasonable and consistent.
Take for example, the RESTful data store CouchDB.  The entire community knows that hostname:5984/_utils is where the CouchDB embedded management console lives, and hostname:5984/databaseName/_design is where a particular db's design documents live. In fact, we count on that. (BTW, I'm showing the default port number; it can vary...)
The point is _reactor could work, maybe _elm_reactor is better.
But all that said a query parameter scheme is also acceptable.


Adds Remove Contacts Functionality

Fixes #1051. Adds the ability to remove a contact from the Contact detail view/modal. To this end, I made a few other notable changes:

The Contact view now accepts an IContact object as a parameter, instead of avatar, username, and address.

There is now a button in the Contact view that will let the user add a contact (if it is not already in the user's contacts) or remove a contact (if it is).

The Contact profile layout is now vertical, not horizontal, so that more functionality can be added without disrupting the layout.

Fixed the issues you brought up. Thanks 👍.


Measures don't work for polar coordinates

I noticed that in situations where polar/spherical coordinates are used for detections (like RadarBearingRange or RadarElevationBearingRange), the DistanceHypothesiser was returning weird distances. I believe it is because the measures used from measures.py don't convert the [Bearing, Range] coordinates to Cartesian. The DistanceHypothesiser doesn't convert them either before passing them into the measure. I ran a few test cases to confirm this problem.
For example, if there is a detection at [Bearing(1.0), 20] and the prediction is at [Bearing(1.2), 25], then the Euclidean measure will simply return sqrt{(1.2-1.0)^2 + (25-20)^2}. This is clearly not correct.
This isn't a big problem in the single target case when there is no clutter, though it will have an impact. I only noticed it because I am trying to add a ClutterModel for the radar measurements, so there were multiple measurements being checked.
I think the easiest way to fix it would be to add an if statement in the DistanceHypothesiser to check if the coordinates are already Cartesian or not. Alternatively, the change could be made in measures.py, in which case it would have to be done for each measure type (ie Euclidean, Mahalanobis...). But that would make it more extensible if there are other uses for the measures. This is probably the better way to fix it. I don't mind writing a fix but I want to check that I am not misunderstanding the code before doing so. Does someone see something I am missing? Thank you!

You can't always completely convert from measurement space to cartesian space, for example in the Cartesian2DToBearing & CartesianToElevationBearing models. In these examples the DistanceHypothesiser has to calculate the 'distance' in measurement space (polar). So as a default you have measure the distance in the measurement state space.
However for RadarBearingRange or RadarElevationBearingRange the Euclean distance in measurement space (polar) (in most instances) will be dominated by the range component.
For example, if you're comparing your prediction of [Bearing(1.0), 20,000] against two measurements of A, [Bearing(1.0), 21,000] and B, [Bearing(1.5), 20,000]. In cartesian A (dist=1,000) is much closer than B (dist=~10,000). However in polar measurement space B (dist=0.5) is much closer than A (dist=1,000).
This is problem is taking the Euclean distance in polar space. A new Measure could be made to account for this.  However I don't think theis problem affects the Mahalanobis & SquaredGaussianHellinger distance as they take into account the covariance of a state which would have a much larger covariance in the range component that

Ok, I will try using the Mahalanobis or some other measure. Thanks for explaining.
What do you mean by "You can't always completely convert from measurement space to cartesian space"? Could the DistanceHypothesiser convert the measurements to cartesian before calling the distance? Thanks

Ah I see, thanks for such a detailed response!


Bulk delete

To be able to delete multiple users/turns/schedules at the same time

Done and working at 2.9.0


popover: Adding top margin to fix avatar visibility in user profile

Fixes: #24210 Fixed the Avatar by adding a top margin in the CSS.
Before :

After :

@anshit-2003 please include a more complete screenshot, so that we can see where the avatar is vs. the top of the popover.
Also, could you please update your commit message to match the commit style guidelines? Thanks!

Actually, since there is already a PR for this issue (#24211), is there a reason your approach would be preferred?

Sure @
I feel like this approach should be preferred because below the avatar there is plenty of blank space
available , so instead of making a top margin of 1px , I made it 5px which makes it better separated from the top bar and
improves the overall design.
The relevant screenshots of before and after are attached below along with the screenshots of top-margin of 1px vs top-margin of 5px.
Fixes : #24210
Screenshots and Screencaptures :
Before :

After :

Top-margin of 5px:

Top- margin of 1px:

Hm, the 1px version looks better aligned with the text on the left to me.

Okay @alya , thanks for the response.

Changed the top margin from 5px to 1px.
I think it's ready to be merged @alya.

Closing in favor of #24211. In the future, please check whether existing PRs exist for an issue. If you'd like to contribute, it's better to help review the existing PR than opening a duplicate.


Possible scope claims collisions

According to schemas, the endpoint /userinfo can return a response like below :
{
    "sub": "0faa3bdf-2178-4600-989f-b3a95add44e0",
    "lastName": {
        "value": "john",
        "source": "franceconnect.gouv.fr",
        "certificationDate": "1997-03-09T00:00:00.0Z"
    },
    "email_verified": true,
    "gender": {
        "value": "Masculin",
        "source": "franceconnect.gouv.fr",
        "certificationDate": "1997-03-09T00:00:00.0Z"
    },
    "preferred_username"<EMAIL_ADDRESS>    "given_name": "john",
    "birthDate": {
        "value": "1999-09-24",
        "source": "franceconnect.gouv.fr",
        "certificationDate": "1997-03-09T00:00:00.0Z"
    },
    "middleNames": {
        "value": "Names",
        "source": "franceconnect.gouv.fr",
        "certificationDate": "1997-03-09T00:00:00.0Z"
    },
    "firstName": {
        "value": "john",
        "source": "franceconnect.gouv.frt",
        "certificationDate": "1997-03-09T00:00:00.0Z"
    },
    "birthPlace": {
        "value": "Paris",
        "source": "franceconnect.gouv.fr",
        "certificationDate": "1997-03-09T00:00:00.0Z"
    },
    "birthCountry": {
        "value": "France",
        "source": "franceconnect.gouv.fr",
        "certificationDate": "1997-03-09T00:00:00.0Z"
    },
    "name": "john john",
    "family_name": "john",
    "email"<EMAIL_ADDRESS>}

@BertrandGervais @jthiard @ghislainfabmob do we want to encapsulate information by scope to more readability and avoid claim collisions between all claimes used by the clients ?

An example with email OIDC claim and personalInformation.email CMS claim.
At root of /userinfo response, we cannot have the CMS email claim but we will have the OIDC email claim as below :
"email"<EMAIL_ADDRESS>
I suggest to encapsulate CMS scopes in a cms object in order to avoid conflicts with other scopes requested :
    "cms": {
        "personal-information": {
            "email": {
                "value"<EMAIL_ADDRESS>                "source": "moncomptemobilite.fr",
                "certificationDate": "2022-11-08T13:44:00"
            }
        }
    }


Week of 01/09 Issue

Scoring
OOP Hacks
Frontend Hacks
Scrum Board

[test ]

[ ]test

.95 Model
.9 View
.85 CPT


nguCarouselDef trackBy

Do you plan to implement trackBy in nguCarouselDef, working similar as in ngFor?
That would be extremely useful when carousel data is updated dynamically.

@sarnecki-cic Hi,
Implemented this feature and will push the update.
Thanks

@sarnecki-cic This feature is implemented. Please check release.
Thanks

I see your change, but when using [trackBy] in conjuction with [dataSources] I am only getting undefined values for index and item in my method.  When I change the line from:
return this.trackBy ? this.trackBy(item.dataIndex, item.data) : item;
to
return this.trackBy ? this.trackBy(_i, item) : item;
I do see the index and the item data. Am I missing something? I couldn't figure out where dataIndex and data was being set on the item.


Refactor tox.ini again

Fix unused "test" env: It should be "normal".
Don't install useless deps in stylecheck
Actually install flake8 in stylecheck

I'll just merge this, ideas still welcome.


Using pip-compile with --build-deps-for or --all-build-deps gives error when using multiple source files.

Using pip-compile --all-build-deps requirments.in pyproject.toml gives the error --build-deps-for and --all-build-deps can be used only with the setup.py, setup.cfg and pyproject.toml specs..
According to the help, using multiple source files is valid, so any options that apply to at least one source file should be allowed as long as that option does not cause a conflict.
Environment Versions

pip-tools version: 7.4.0+

Steps to replicate

pip-compile --all-build-deps requirments.in pyproject.toml

Expected result
No error.
Actual result
--build-deps-for and --all-build-deps can be used only with the setup.py, setup.cfg and pyproject.toml specs.

Yeah, I don't remember what our stance on that was... However, combining multiple different envs tends to over-complicate things so I'm an advocate of doing the opposite.
Still, it was designed to lock one's package's build deps, not build deps of every single package in the tree. It'd be ambiguous which project you want the build deps to be retrieved from, if there's many. Plus, it's not uncommon that several projects in the tree have conflicting build dep restrictions (and each of those projects is built in isolation).

@webknjaz
Is your concern for a command like pip-compile --all-build-deps foo.in pyproject.toml or one like pip-compile --all-build-deps projectA\pyproject.toml projectB\pyproject.toml?
My use case is that I don't want to keep extra dependencies in extras_require or [project.optional-dependencies], I want to keep them in .in files so I can't use pip-compile --all-build-deps --extra foo pyproject.toml. Currently, I am forced to keep an extra build_deps.in file and use pip-compile build_deps.in foo.in pyproject.toml, which duplicates my build dependencies in both build_deps.in and pyproject.toml.
As far as I can tell, a command of the form pip-compile --all-build-deps projectA\pyproject.toml projectB\pyproject.toml is perfectly valid right now.
Would allowing only one file in the sources list satisfy your concern?

I once wanted to combine the deps into the same lockfile but later I changed my mind because it causes more problems than it solves: https://github.com/jazzband/pip-tools/issues/1326#issuecomment-1834517252.
That test case does not cover the situation when the build dependencies of projectA and projectB conflict with each other. Try adding a test case where there's setuptools < 40 in one and setuptools > 50 in the other.


use pyqtgraph 0.11 in recommended env

Fixes deprecation warnings with 0.10 among other things

shall we change it in setup.py extras as well?

Yes.


AttributeError: module 'tensorflow.python.saved_model.signature_constants' has no attribute 'DEFAULT_TRAIN_SIGNATURE_DEF_KEY'

Successfully installed tensorflow-1.13.1 tensorflow-probability-0.6.0

import tensorflow as tf

/home/maksim/dev_projects/rl/venv/bin/python /home/maksim/dev_projects/rl/racetrack.py
Traceback (most recent call last):
  File "/home/maksim/dev_projects/rl/racetrack.py", line 3, in <module>
    import tensorflow as tf
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow/__init__.py", line 29, in <module>
    from tensorflow._api.v1 import compat
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow/_api/v1/compat/__init__.py", line 21, in <module>
    from tensorflow._api.v1.compat import v1
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow/_api/v1/compat/v1/__init__.py", line 626, in <module>
    child_package_str=('tensorflow_estimator.python.estimator.api.estimator'))
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow/python/tools/component_api_helper.py", line 56, in package_hook
    child_pkg = importlib.import_module(child_package_str)
  File "/home/maksim/anaconda3/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/__init__.py", line 8, in <module>
    from tensorflow_estimator._api.v1 import estimator
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/_api/v1/estimator/__init__.py", line 8, in <module>
    from tensorflow_estimator._api.v1.estimator import experimental
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/_api/v1/estimator/experimental/__init__.py", line 8, in <module>
    from tensorflow_estimator.python.estimator.canned.dnn import dnn_logit_fn_builder
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/python/estimator/__init__.py", line 25, in <module>
    import tensorflow_estimator.python.estimator.estimator_lib
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/python/estimator/estimator_lib.py", line 22, in <module>
    from tensorflow_estimator.python.estimator.canned.baseline import BaselineClassifier
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/python/estimator/canned/baseline.py", line 64, in <module>
    from tensorflow_estimator.python.estimator import estimator
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/python/estimator/estimator.py", line 66, in <module>
    from tensorflow_estimator.python.estimator import model_fn as model_fn_lib
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/python/estimator/model_fn.py", line 36, in <module>
    from tensorflow_estimator.python.estimator.export import export_lib
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow_estimator/python/estimator/export/export_lib.py", line 25, in <module>
    from tensorflow.python.saved_model.model_utils import build_all_signature_defs
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow/python/saved_model/model_utils/__init__.py", line 22, in <module>
    from tensorflow.python.saved_model.model_utils.export_utils import build_all_signature_defs
  File "/home/maksim/dev_projects/rl/venv/lib/python3.7/site-packages/tensorflow/python/saved_model/model_utils/export_utils.py", line 51, in <module>
    ModeKeys.TRAIN: signature_constants.DEFAULT_TRAIN_SIGNATURE_DEF_KEY,
AttributeError: module 'tensorflow.python.saved_model.signature_constants' has no attribute 'DEFAULT_TRAIN_SIGNATURE_DEF_KEY'

Process finished with exit code 1

@makslevental Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary? Thanks!

@gadagashwini ubuntu 19.04, x86_64. pip installed
must be something having to do with the pip package because https://github.com/tensorflow/tensorflow/blob/73640d97f67d6bd5ec84375449c6b9bba179606b/tensorflow/python/saved_model/signature_constants.py#L139

Hi,
Is there any news about this issue? I believe I have the same issue in Ubuntu 16.04.3 with
tensorflow==1.13.1
tensorflow-estimator==1.13.0
tensorflow-gpu==2.0.0a0

@makslevental and @sslavian812 Could you check the following things and share the details. I see more people are facing this issue. Thanks!

pip list
python3 --version
pip3 --version
virtualenv --version

Thanks!

Thank you for the response, @jvishnuvardhan
The problem was about tensorflow versions, especially tensorflow-gpu==2.0.0a0.
With these versions, the error is gone:
tensorflow==1.12.2
tensorflow-estimator==1.13.0
tensorflow-gpu==1.13.1

regarding your quesion:
python3 --version : 3.5.2
pip3 --version: pip 19.0.3
virtualenv --version: 15.0.1

@sslavian812 Sorry for the delay in my response? Can you check with latest versions TF1.14, TF1.15 (tf-nightly), and 2.0.0b1? Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Appears to be broken

This appears to be broken - gives an error "did not get a ticket in the response"

So what worked for me:
Change the login URL from 'login' to 'signin':
private static string BASE_URL = "https://sso.garmin.com/sso/signin";
Then add an 'origin' header to the request:
request.Headers.Add("origin", "https://sso.garmin.com");
Mine is once again working after these changes, I was getting the same invalid ticket error prior

Thank you @AngelaKru. Seems like Garmin did unfortunately again some changes. Will you create a pull request as you already applied the solution.


Adding new vertex to existing graph?

Thank you for developing this library!  I will be building a graph iteratively, and I do not know the number of vertices in advance.  I have looked at the README and the source, and I do not see a way.  Is it possible to add a vertex after the call to data.graph.create()?  If not, would you be willing to add that function?  I appreciate your considering this request!

Thanks for your suggestion, this should be no problem, I will add that function in these two days and update you again :)

Hi @cxw42 I have updated the library to include api that allows the graph to be expand and shrink such as

graph:addVertexIfNotExists(vertexId),
graph:removeVertex(vertexId),
graph:vertexCount(),
graph:vertexAt(vertexIndex),
graph:addEdge(vertexId1, vertexId2, ...) (addEdge is changed so that if the vertices in its parameters do not exists in the graph, they will be automatically added to the graph)

Please refers to the section "Create an undirected unweighted graph and add / remove vertices later" and "Create an undirected unweighted graph from a list of vertices and expand or shrink it". the operations apply to the different types of graphs as well. the there are also unit testing results in the graph_spec.lua.
Note that the graph.V is removed and replaced with graph:vertexCount() and graph:vertexAt(index) method for iterating all vertices (this change was introduced so that graph vertices do not need to start with vertex 0 and do not need to be consecutive. Please let me know if there are problems using the updated version :)


Fill afcca gap

Part of #929
Between February 1st, 2021 and February 1st 2023, we have 0 documents in CL. We are missing more than 250 documents from 2021 and 2022 (198)

Command to fill the gap:
docker exec -it cl-django python /opt/courtlistener/manage.py cl_back_scrape_opinions juriscraper.opinions.united_states.federal_special.afcca --backscrape-start=2021 --backscrape-end=2022 ```

After Ramiro ran the backscrape we have a single missing document for 2021, the one from 7 Apr 2021
I got the hash manually, and again, the hash already existed in Courtlistener, the document was duplicated


[PDI-16434]Salesforce Upsert (and possibly insert) fails when upserting an integer into account object

[PDI-16434]Salesforce Upsert (and possibly insert) fails when upserting an integer into account object
-fixing sending Pentaho Integer value to Salesforce (SalesforceInsert and SalesforceUpsert steps)

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'plugins/salesforce/core' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'plugins/salesforce/core' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/salesforce/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/salesforce/core/src/main/java/org/pentaho/di/trans/steps/salesforceinsert/SalesforceInsert.java
  plugins/salesforce/core/src/main/java/org/pentaho/di/trans/steps/salesforceupsert/SalesforceUpsert.java
  plugins/salesforce/core/src/test/java/org/pentaho/di/trans/steps/salesforceinsert/SalesforceInsertTest.java
  plugins/salesforce/core/src/test/java/org/pentaho/di/trans/steps/salesforceupsert/SalesforceUpsertTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsert

Branch Change:  + 1.89%
Complexity Change:  + 1.59%
Instruction Change:  + .79%
Line Change:  + .90%

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change:  + .17%

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsert

Branch Change:  + 1.95%
Complexity Change:  + 1.64%
Instruction Change:  + .82%
Line Change:  + .97%

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change:  + .14%