Feature: Support --specs for e2e argument

It would be very nice to be able not only run a single or some of the e2e tests.
Thanks :)

The current angular-cli has the --specs for e2e.
Sorry if I'm wrong, but I think you can use it now.
https://github.com/angular/angular-cli/wiki/e2e

--specs (alias: -sp) default value: []
Override specs in the protractor config. Can send in multiple specs by repeating flag (ng e2e --specs=spec1.ts --specs=spec2.ts).

Closing as answered by @leocaseiro, thanks!
There's also this feature request for a --suite flag: https://github.com/angular/angular-cli/issues/2628.


prepare explainer code for 3.8 DistributeNode output

Scope & Purpose
Prepare AQL query explainer code in 3.7 for a forthcoming change to the explain output of DistributeNodes in 3.8, so that the 3.7 explainer can handle both types of DistributeNode.
This change will only have an effect if a 3.7 explainer is used against a 3.8 cluster, which is relatively unlikely, but possible.
There is intentionally no CHANGELOG entry for this, as it is a very minor adjustment only.

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[x] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Backports:

[x] No backports required

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.
[x] The behavior in this PR was manually tested

Link to Jenkins PR run:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13910/

Tests blue


If terminal is too small, --help will crash.

At https://github.com/jessevdk/go-flags/blob/master/help.go#L241 ,
If info.terminalColumns is less than descstart, wrapText() will get a negative length and cause a "slice bounds out of range" panic

hey @jessevdk, can you give some context as to why this was closed?

This was closed because it was fixed. Do you still encounter this problem?

It was closed because it was fixed. Do you still encounter the issue?

My mistake. Hadn't properly revendored! thanks so much.


the svmpredict is seem to not work well

after compile the test with hearscale is work but when i test my dataset the svmpredict is not work that always rreturn empty accuracy

Could you provide more details?
On 2019-04-03 23:01, haiwenheaven wrote:

after compile the test with hearscale is work but when i test my
dataset the svmpredict is not work that always rreturn empty accuracy
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or mute the
thread [2]. [ { "@context": "http://schema.org", "@type":
"EmailMessage", "potentialAction": { "@type": "ViewAction", "target":
"https://github.com/cjlin1/libsvm/issues/141", "url":
"https://github.com/cjlin1/libsvm/issues/141", "name": "View Issue" },
"description": "View this Issue on GitHub", "publisher": { "@type":
"Organization", "name": "GitHub", "url": "https://github.com" } } ]
Links:
[1] https://github.com/cjlin1/libsvm/issues/141
[2]
https://github.com/notifications/unsubscribe-auth/AFGwnjXHbNslRD-GtEdoBJyJ6vEHhbLTks5vdZUtgaJpZM4cb_e_

@cjlin1 when i train a model and predict like "model = svmtrain(train_wine_labels, train_wine, '-c 2 -g 1');   [predict_label, accuracy] = svmpredict(test_wine_labels, test_wine, model);
i get a empty accuracy

how about predict_label?
and what is the screen output of svmpredict
On 2019-04-04 10:40, haiwenheaven wrote:

@cjlin1 [1] when i train a model and predict like "model =
svmtrain(train_wine_labels, train_wine, '-c 2 -g 1'); [predict_label,
accuracy] = svmpredict(test_wine_labels, test_wine, model);
i get a empty accuracy
--
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [2], or mute the
thread [3]. [ { "@context": "http://schema.org", "@type":
"EmailMessage", "potentialAction": { "@type": "ViewAction", "target":
"https://github.com/cjlin1/libsvm/issues/141#issuecomment-479994392",
"url":
"https://github.com/cjlin1/libsvm/issues/141#issuecomment-479994392",
"name": "View Issue" }, "description": "View this Issue on GitHub",
"publisher": { "@type": "Organization", "name": "GitHub", "url":
"https://github.com" } } ]
Links:
[1] https://github.com/cjlin1
[2] https://github.com/cjlin1/libsvm/issues/141#issuecomment-479994392
[3]
https://github.com/notifications/unsubscribe-auth/AFGwngn59K1fcbacQClKv86Zn-5FaWh8ks5vdjj8gaJpZM4cb_e_

@cjlin1 sorry,maybe i will reply to you in some days ,beacause at first i want to sovle this problem by installing matlab agian  but now i .make another trouble in installing matlab


how to get x,y coordinate

hello ,
could you please tell me how to get x,y,height,width coordinates for a specific area , I've tried to do but failed

hello ,
could you please tell me how to get x,y,height,width coordinates for a specific area , I've tried to do but faile

Run pyautogui.displayMousePosition() and hover your mouse over the "specific area" and it will show those cords

hello ,
could you please tell me how to get x,y,height,width coordinates for a specific area , I've tried to do but failed

You can also use "print(pyautogui.position())" to get the mouse coordinates as an output. Output example:
Point(x=251, y=416)


sending events / logs with unicode chars

An issue came up where we were trying to send an event for analytics that contained a utf-16 char, for example "Writing Chicago\u2019s Story". We can correctly draw this string in the application, however when we sent that to AMPM (using something like this function), it would crash the server with the following printed to console:
2016-06-27T21:39:30.789Z - warn: OSC messages should be JSON
C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm\model\logging.js:233
this._google.event(data.Category, data.Action, data.Label, data.Value);

TypeError: Cannot read property 'Category' of null    at exports.Logging.BaseModel.extend._logEvent (C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm\model\logging.js:233:36)
at wrapper (C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm\node_modules\lodash\index.js:3095:19)
at emitOne (events.js:77:13)
at emit (events.js:169:7)
at exports.Network.BaseModel.extend._handleOsc (C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm\model\network.js:175:19)
at null.<anonymous> (C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm\model\network.js:152:22)
at wrapper (C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm\node_modules\lodash\index.js:3095:19)
at emitTwo (events.js:87:13)
at emit (events.js:172:7)
at Socket.<anonymous> (C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm\node_modules\node-osc\lib\Server.js:27:20)

[nodemon] app crashed
[nodemon] exiting
[nodemon] 1.9.2
[nodemon] to restart at any time, enter `rs`
[nodemon] ignoring: .git .nyc_output .sass-cache bower_components coverage node_modules logs C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm-state.json
[nodemon] watching: C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm.json C:\Users\potion-nw-wall\projects\northwestern\nw\tools\monitoring\ampm-restart.json
[nodemon] watching extensions: js,json
[nodemon] starting `node C:\Users\potion-nw-wall\AppData\Roaming\npm\node_modules\ampm\server.js ..\tools\monitoring\ampm.json default`
[nodemon] child pid: 11200
[nodemon] watching 2 files

Merging config: default
Merging config: default
Merging config: DESKTOP-RNH48JB
Merging config: DESKTOP-RNH48JB.default
Server starting up.
2016-06-27T21:39:32.219Z - info: App starting up.

So, the json appears in node.js as malformatted. We've been investigating whether this is a problem with cinder's OSC implementation, however I don't believe it is - ci::osc just serializes the std::string as raw bytes. However, @LRitesh has pointed out that the OSC spec specifies that OSC-string should be only ASCII. Cinder's implementation doesn't seem to care, but perhaps the json parse in javascript does? This is the extent of my knowledge on the situation.
If it turns out to be that we just aren't allowed to send unicode chars as OSC-string to AMPM, one solution we've thought about is whether we could send the events / logs as OSC-blob instead, so they can be parsed with unicode chars and all on the other end.
Also, am I correct in thinking that analytics.js supports utf8 / 16?

Hm. I've been doing some stuff with emoji in JS and have learned that JS can kind of suck with Unicode in general. ES6 is better, but the day for that rewrite is not yet here.
The OSC module being used is the latest, so that's not a fix.
https://github.com/TheAlphaNerd/node-osc
Do you want to do a sample app that sends what you need as an OSC blob, and we can try to tweak _handleOsc to deal with it?
https://github.com/stimulant/ampm/blob/master/model/network.js#L163
It would be easiest for me to troubleshoot with a binary app that has a button that sends a message that repros the problem.
In the meantime I committed a change that should at least keep it from crashing. It's on npm too.

Ah, I didn't realize that _handleOsc is where the JSON is parsed before being sent to _logEvent.
FWIW, if I send something like "Writing Chicago\u2019s Story" as the event's Category value, it shows up inside _handleOsc as "Writing Chicago\u0012s Story". And JSON.parse() throws and exception [SyntaxError: Unexpected token ]. So this is where it fails to parse the string. Not sure if there's a way to have JSON.parse handle UTF-16 data, but might be worth looking into that.

Maybe we need to do some escaping in the ampm client library?
http://stackoverflow.com/a/11654338/468472

Maybe we need to do some escaping in the ampm client library?
http://stackoverflow.com/a/11654338/468472
Or escape it in _handleOsc before passing it to JSON.parse.

@endquote for reproducing, I can on Mac with the provided cinder sample by changing this line to (not sure if this will work on windows as you can't directly write utf16 in strings until vc140):
string action = "Writing Chicago\u2019s Story";
AMPMClient::get()->sendEvent( "category", action, "label", 10 );

@LRitesh hm, so it shows up as just one char off?
I've only briefly researched the js side of things, but this gist makes me think that JSON.parse can handle UTF-16 just fine (as does cinder's jsoncpp), and the problem is likely to be in node-osc's parsing, which expects ASCII chars. Here's a [vvvv forum](https://vvvv.org/forum/encode-problem-with-strings-and-umlauts-via-udposcdecoder post) with similar issues. I believe they recommend sending the data as a blob as well, base64 encoding it first. OSC can be a pain sometimes, huh.
I don't think we need to double-escape the utf-16 chars, we're able to read down the json OK from our CMS as-is, load it in the app and display it. It's just once it gets sent over OSC that things go awry.

@endquote I tried decodeURIComponent(message[1]) before sending it to _logEvent but running into the same issue still.
@richardeakin I was looking at that gist as well. It does sound like the issue might be in node-osc's parsing, because it's replacing \u2019 to \u0012 before JSON parse is even used on the string.

How do you get the Mac Cinder sample to build on Yosemite? I don't think environment variables are really a thing anymore. Do we need to update the readme?
https://github.com/stimulant/ampm/blob/master/samples/Cinder/README.md

I haven't added anything related to app transport myself, but I've only done brief testing on Mac while our main target is win10.


Space Travelers Milestone: Fetch missions

For this Milestone I:
✅ Fetched data from the Missions endpoint (https://api.spacexdata.com/v3/missions)
✅ Dispatched an action to store the selected data in the Redux store:
- mission_id
- mission_name
- description

Hey @abiola-arobieke, thank you so much for your feedback! 🚀

@abiola-arobieke, thanks for your feedback! 🚀


Updated project to remove cyclic dependency for this build and to include updates that were done on a fork of QuickFIX/J project.

The updates from QFJ project have been slightly refactored.

@donmendelson @chrjohn Here is a first cut. To avoid cyclic dependency I removed the Model and Session modules. They are in the history though as well as the original location. In future (or now) if we want to develop them, I think they should be in another repo that can depend on the parent pom that will be published for this one. I created a github workflow but have not been able to test it yet. Happy New Year !

I need to check how I can push a release to Sonatype or Maven Central so that QFJ can use this as a dependency.


Update domains.json

Hi! What is your PR about?

[ ] Add or edit a skin
[X] Add a custom domain for your space (cortexdao.eth)
[ ] Add an alias to migrate your space

There is a conflict in your PR since the domains.json file has been updated, can you resolve it?
To prevent conflicts, it is recommended to add your subdomain between two existing domains rather than at the end or beginning of the list.

Closing this PR for now, feel free to open it again once you've done the changes.


Adding very large tensor tests for JIT kernels

This is to test adding large tensor tests for jit kernel path.
CC: @frgossen

@kushanam Can you please check @akuegel's @frgossen's comments and resolve conflicts?. Thank you!


Unify backend name and operator name

Currently there are two different names for the operator and the backend that it is instantiated.
This is because we initially planned to support multiple backends per operator and the backend name would be visible in an ExternalSecretBackend CRD.
Since we now have a 1-1 relationship between backends and operators, this distinction is no longer necessary. However the backend name is still important to distinguish which backend to use and is required in ExternalSecret CRDs.
The main issue is now that the backend name is held in the configuration held in a Secret and is not easily visible. A possible solution would be to use the OPERATOR_NAME environment variable.

Implemented in #68


Mention the talk by Josh Berkus and the blog post by Jan Mußler.

#182

👍

👍

Coverage remained the same at 11.21% when pulling 80d55c1072acf4112782d7fc27a161755a227aae on update_documentation into 04024f91e4a90ee23d931220992ffe2767139b0a on master.

:+1:


Add scroll-to-top button

ScrollToTop component added to the IsaacApp component. It remains at the bottom of the screen (out of site) until the user scrolls beyond the top of the main content at which point it sticks to the bottom right of the screen.
Currently only on the Ada site but removing the conditional will add it to the Physics site as well. The scroll-button.scss will also need to be imported in the relevant isaac.scss file.

Button added to the physics site as well. In the process fixed a minor bug where the button was visible at the bottom of the screen even though the button was not in use.


Compiling project on CentOs7

Hey I saw couple of issues on that subject.
I know the project was written and compiled on ubuntu., but for the life of me I can't seem to compile it on CentOs.
I notice that some of you manage to compile it on centos and release some unofficial jars. I used some of them.
Could you please direct to a branch may or what are the steps to compile the project on centos?

hi @RefiPeretz , sorry about the trouble you are having.  The issue should be fixed in master, which uses the official lightgbm jars compiled on ubuntu 14.04 which doesn't seem to have some of the dependency issues.
You can get the master release from a private blob from any of the builds, eg the latest one is:
--packages
com.microsoft.ml.spark:mmlspark_2.11:0.17.dev23
and --repositories
https://mmlspark.azureedge.net/maven

Hi @imatiach-msft . Evntually I managed to create Dockerfile from CentOs 7.6 where I can build the project. Maybe I can share it with you in the future if you want. I still have problems with libmpi_cxx altought I found a way around it. Thanks anyway.


[BUG] <BLE advertising name can't be increased to more than 5 characters >

System information

Hardware board: [ Custom Board ESP32 WROOM]
Operating System: [ Windows 10 ]
Code version: 202012.00
Project/Demo: [ CONFIG_OTA_UPDATE_DEMO_ENABLED with custom functionalities added ]

Describe the bug
We were trying to increase BLE advertising name length. We tried the below process to increase the name lenth, but it can't be increased to more the 5 characters.
1. change IOT_BLE_DEVICE_SHORT_LOCAL_NAME_SIZE  more than 5(e.g 6) in "iot_ble_config_defaults.h"
2. update IOT_BLE_DEVICE_COMPLETE_LOCAL_NAME according to the name length(e.g "123456")
If we try to increase more than 5 characters the OTA demo stops and it's log screenshot attached below.

Is there a way to increase BLE advertising name to more than 5 characters?

Hi @sumit612
Removed the bug label as it's a valid failure per the BT/BLE v4.2 spec which our BT/BLE software is based on. For the MQTT BLE demo, and I believe for the BLE GATT DEMO too, our advertisement data looks like this when shortened local name is set to 6:
$2 = {
  flags = 6 '\006',
  uuids16 = 0x0,
  num_uuids16 = 0 '\000',
  uuids16_is_complete = 0,
  uuids32 = 0x0,
  num_uuids32 = 0 '\000',
  uuids32_is_complete = 0,
  uuids128 = 0x3ffb3d70,
  num_uuids128 = 1 '\001',
  uuids128_is_complete = 1,
  name = 0x3ffbdc40 <ble_svc_gap_name> "123456",
  name_len = 6 '\006',
  name_is_complete = 0,
  tx_pwr_lvl = -128 '\200',
  tx_pwr_lvl_is_present = 1,
  slave_itvl_range = 0x0,
  svc_data_uuid16 = 0x0,
  svc_data_uuid16_len = 0 '\000',
  public_tgt_addr = 0x0,
  num_public_tgt_addrs = 0 '\000',
  appearance = 0,
  appearance_is_present = 0,
  adv_itvl = 0,
  adv_itvl_is_present = 0,
  svc_data_uuid32 = 0x0,
  svc_data_uuid32_len = 0 '\000',
  svc_data_uuid128 = 0x0,
  svc_data_uuid128_len = 0 '\000',
  uri = 0x0,
  uri_len = 0 '\000',
  mfg_data = 0x0,
  mfg_data_len = 0 '\000'
}

With this advertisement setup, when shortened local name is > 5, the advertisement PDU data exceeds the 31 byte limit.
Breakdown looks like this:

flags
uuid128
name
tx power

2 + 1 B
3 + 16 B
2 + strlen(name) B
2 + 1 B

The BLE spec GAP/GATT details explain where the first terms come from, they're basically data IDs + lengths.
So, the NimBLE stack (default for ESP32) fails appropriately, percolating to the failure you see which should have an indicative log (I'm adding one now). Advertisement PDU size was increased in >= v5.0, which we plan to support in the future.
The complete local name is not provided in advertisements, rather it's provided when you request it via EIR. Not sure what BT/BLE scanner you are using, but if request the complete local name via EIR, you should get the entire name as expected. I've confirmed this with a BLE scanner app on my phone, which sends EIR for full name and gets 123456 as expected.

@dachalco , apologies for late reply. Got busy with some office tasks. Thank you very much for your elaborated reply. Will try the complete local name then with EIR scanner then.

@sumit612 . Check this commit.I was able to fix the length of the name for BLE on AWS FreeRTOS GreenGrass. It's the same principle for any of the demos.
Hope this helps you.
https://github.com/Yoimer/freertos_greengrass_202007/commit/268bfc523cd4ccf82f2ef5e0e684b2c4fa84dcfd

@Yoimer thank you very much. Will definitely try it.

@Yoimer, it worked for me. :)

@Yoimer, it worked for me. :)

@sumit612. I am glad it worked for you. Hope you can move forward in your project. I am here just in case you need any further help 👍

Glad a suitable solution was found. Please keep in mind that ADV data is still limited to 31 bytes, so the longer the name, the more data you need sacrifice to fit it. Please re-open as needed. Cheers!


geo remove use of DBL_EPSILON

-this doesn't seem to be (obviously?) available on qurt
@RomanBapst FYI

@dagar Man this was actually an easy, bummer we did not find it.


Docker environment fixes

Fixed typos in README/log statement
Build binary in the Docker build
Add docker-compose file

Thanks!
I pulled the typo fixes in #91.
The Dockerfile has been removed from the repository in #89, ko is now used to build the container image.


Command not found

┌──(kali㉿kali)-[~]
└─$ pip3 install git+https://github.com/EntySec/Ghost                                                        1 ⨯
Defaulting to user installation because normal site-packages is not writeable
Collecting git+https://github.com/EntySec/Ghost
Cloning https://github.com/EntySec/Ghost to /tmp/pip-req-build-ckodaolz
Running command git clone -q https://github.com/EntySec/Ghost /tmp/pip-req-build-ckodaolz
Resolved https://github.com/EntySec/Ghost to commit c8b328c87b8ea71c4f191c90ab24705f3cc62cfc
Requirement already satisfied: adb-shell in ./.local/lib/python3.9/site-packages (from ghost==8.0.0) (0.4.2)
Requirement already satisfied: rsa in ./.local/lib/python3.9/site-packages (from adb-shell->ghost==8.0.0) (4.7.2)
Requirement already satisfied: cryptography in /usr/lib/python3/dist-packages (from adb-shell->ghost==8.0.0) (3.3.2)
Requirement already satisfied: pyasn1 in /usr/lib/python3/dist-packages (from adb-shell->ghost==8.0.0) (0.4.8)
┌──(kali㉿kali)-[~]
└─$ ghost
Command 'ghost' not found, did you mean:
command 'khost' from deb knot-host
command 'qhost' from deb gridengine-client
command 'gfhost' from deb gfarm-client
command 'gost' from deb gost
command 'host' from deb bind9-host
command 'rhost' from deb jdresolve
command 'xhost' from deb x11-xserver-utils
Try: sudo apt install

@cypox90
sudo pip3 install git+https://github.com/EntySec/Ghost


How to pass --trace(-expand) if using CMakePresets.json

Simple question:
How to pass --trace(-expand) if I am using CMakePresets.json?
I tried a wrapper like:
"cmakeExecutable": "cmake-trace.bat"
but it seems like the extension is seeing through that and just calling cmake

According to CMake presets documentation, cmakeExecutable is

An optional string representing the path to the CMake executable to use for this preset. This is reserved for use by IDEs, and is not used by CMake itself. IDEs that use this field should expand any macros in it.

As to how to pass --trace-expand switch to cmake when using presets, I wasn't able to find a way. Adding @esweet431.
Probably you know that without presets (when using kits) you can define "cmake.configureArgs".

Hmm, I'm not sure either. I recommend opening an issue with Kitware to see if they have a recommendation on how to encode --trace or --trace-expand in a presets file.

Closing upstream issue


Mixed metrics causing failure after porting

IceNet version: v0.2.0.dev6

Description
Training runs won't complete through evaluate phase, the shared training/evaluation metrics aren't correctly configured for the tensors.

These are in a bit of a nasty state with additional spurious dimensions in the metrics. This can be clarified in time, but fixing this bug is the priority so a bit of a cleanup will do for the moment.


[AIRFLOW-XXX] Fix typo in docstring of gcs_to_bq

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3833 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3833      +/-   ##
==========================================
- Coverage   77.43%   77.43%   -0.01%
==========================================
  Files         203      203
  Lines       15846    15846
==========================================
- Hits        12271    12270       -1
- Misses       3575     3576       +1

Impacted Files
Coverage Δ

airflow/models.py
88.74% <0%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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

Good one @kaxil. Maybe specify list of str as well? I've used this operator last week, and first I just specified a single string instead of a list. And it failed without any explicit reason, maybe I'll create a PR for it in the upcoming days.

Agree. Just updated the PR.


Updating constant buffers.

Hello, I have been encountering a very strange bug with diligent recently that is currently slowing me down.
So, I have a pipeline that is correctly created and got 2 constant buffers one is in the vertex shader, and the other is in the pixel shader.
It seems that updating those constant buffers by mapping fills them with corrupted data in DirectX while in vulkan it is filled with zeros.
I am 100% sure that the pointer to data in memeory is correct.
I tried graphical debugging with Visual studio and I made sure that the buffers are correctly bound to the Pipeline/shader.
I know that the issue is not clear but this is what really happens.
Do you have any idea why this happens?
You can check the source code here.
Creating the pipeline and setting the static buffers and updating the light buffer "in the Pixel shader' here.
Updating the camera buffer here

This code
data = mPSLightCB;

does not make any sense. It overwrites mapped data pointer with the pointer to your constant buffer. As a result, no data is written to the buffer and the shader reads unitialized memory (which can be either garbage or zeros). Same situation with CameraComponent.
What you were intending to write was something like this (I would suggest using MapHelper class as it takes care of unmapping the buffer when it goes out of scope).
Otherwise you need to write something like
*reinterpret_cast<CameraBuffer*>(data) = _CameraBuffer;

Thank you.
About the light constant buffer, I feel very dumb that I didn't notice that bug thanks for pointing it out.
And yes I intended to replace the map helper class as I tried to use it but it didn't work so thanks again.

Also one last thing, The Light constant buffer is being dynamically built at runtime, so its size isn't known at compile time, so it can't use map helper class.
Any thoughts how to work around that?

Oftentimes it requires somebody else to look at your code to spot an obvious problem :)
As to buffer update, you have few options.
If your maximum number of lights is limited to a reasonable number and if you update your buffer once per frame, you can just reserve enough space for maximum number of lights. That should work fine.
If you update your buffer multiple times per frame or if you need to write considerable amount of data, I would recommend the streaming method used in Tutorial10. Instead of writing lights to a constant buffer, make the buffer a shader resource and read it in the shader using offset. Fill the buffer with nooverwrite flag until it fills up and then discard it. Make current offset available to your shader via constant buffer.


when run gitlab CI/CD，I find a error that is 'shell not found'

The content of my .gitlab-ci.yml file is:
stages:

pr_agent

pr_agent_job:
stage: pr_agent
image:
name: codiumai/pr-agent:latest
entrypoint: [""]
script:
- cd /app
- echo "Running PR Agent action step"
- export MR_URL="$CI_MERGE_REQUEST_PROJECT_URL/merge_requests/$CI_MERGE_REQUEST_IID"
- echo "MR_URL=$MR_URL"
- export gitlab__url=$CI_SERVER_PROTOCOL://$CI_SERVER_FQDN
- export gitlab__PERSONAL_ACCESS_TOKEN=$GITLAB_PERSONAL_ACCESS_TOKEN
- export config__git_provider="gitlab"
- export openai__key=$OPENAI_KEY
- python -m pr_agent.cli --pr_url="$MR_URL" describe
- python -m pr_agent.cli --pr_url="$MR_URL" review
- python -m pr_agent.cli --pr_url="$MR_URL" improve
rules:
- if: '$CI_PIPELINE_SOURCE == "merge_request_event"'
When I run the gitlab CI/CD, there is a error. The error is 'shell not found'

Could someone please help to resolve the following issue？

Does it need to install shell execution programs in the GitLab Runner?

https://forum.gitlab.com/t/jobs-fails-shell-not-found-version-python/88340
There seem to be a gitlab bug.
Try to look in the tread for a solution that works in your specific setup

Thank you. I get it

I need to update the version of gitlab-runner?

My current gitlab-runner's version is 13.5.0


Email Notification Options

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
1 - The ability to schedule notifications to run between specific times.
2 - The ability to have a monitor look for keyword and send notifications when found only.
✔ Solution
One - The ability to schedule notifications to run between specific times for example between 8AM and 8PM It would nice to also have the monitor run on specific days of the week for example, every day or just Monday and Wednesday etc.
Two - Modify the existing Monitor Type HTTP(s) - Keyword to have the ability to ONLY send notifications when the keyword is found. If it is not found, consider it "UP"
1 visit a website
2 login if needed
3 look for keyword and if found send notification and if not found send no notification.
❓ Alternatives
No response
📝 Additional Context
No response

@adm2k
I think this issue is resolved.

you can block out timespans using maintenance windows.
you can use the http-keyword monitor.

Could you please close this Issue, as resolved issues/duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique?

Issue resolved.


Git symbol not shown

Issue
I have Fira Code installed, but git symbol is not shown. Tried it in both iTerm 2 and Hyper.
Screenshot

Environment
Operating system: macOS Sierra 10.12.4
Terminal emulator: Hyper
ZSH version: 5.2

Hmmm... Theme was not updated properly when using curl -o - https://raw.githubusercontent.com/denysdovhan/spaceship-zsh-theme/master/install.zsh | zsh . It seems that theme is not renamed to spaceship.zsh-theme but to spaceship.zsh-theme.zsh. When I rename it manually, latest theme is installed.

@vburojevic hey! Git symbol requires powerline patched font #92.
Also, check out probably related issue: #93

Theme was not updated properly

Yep. Gonna fix that asap.

Theme was not updated properly

Can't confirm. Works well for me. There's no such behavior described in install.zsh.


logging: add log_msg_timestamp_to_us() helper function

So far we only have log_msg_timestamp_get() function, which returns
internal timestamp representation. This is either clock cycles or uptime
in ms, depending on main clock precision.
Introduce log_msg_timestamp_to_us() helper function, which allows to
convert internal logging timestamp to us.
Signed-off-by: Marcin Niestroj<EMAIL_ADDRESS>

Do we have a second volunteer to review this quite simple PR? It basically allows to implement new logger backends, that do not serialize log timestamps into log messages, but use time directly (by converting from internal logging representation to microseconds).


Fix javadoc for FileSystemUtils.waitCompleted

Fixed javadoc.
see: https://alluxio.atlassian.net/browse/ALLUXIO-2056?filter=13700

Can one of the admins verify this patch?

ok to test

Merged build finished. Test PASSed.

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

LGTM, thanks for the contribution!

+1 LGTM


[Java] Code quality in locateTopEven

There is a method called locateTopEven in the PoorCode class that makes me want to locateWhoeverWroteThisMethod.
Please fix code quality issues in this method:
https://github.com/SergeyOvchinnik/open-class/blob/main/src/PoorCode.java#L56

Say less


Enable Dendron to run on VS Code on the web

👋 I'm Harald, a PM from the VS Code team, and we recently announced VS Code for the Web at https://vscode.dev/, which provides a free, zero-install Microsoft Visual Studio Code experience running entirely in your browser. You can learn more here.
We'd like to ensure that Dendron* can run in the web, and we have a guide for enabling extensions for the web here. As a couple of highlights:

In VS Code for the Web, both the UI and extension host run inside the browser.
A web extension is structured like a regular extension, but with a different main file: it's defined by the browser property
Access to workspace files needs to go through the VS Code file system API accessible at vscode.workspace.fs
There are currently three ways to test a web extension

Feel free to reach out to me if you have questions or if I can help somehow. Thank you

Thanks @digitarald for the suggestion and offer of help - we certainly want to enable Dendron on VS Code Web at some point, but right now we have a large reliance on node processes.

Hi @jonathanyeung! Super glad to hear the interest in enabling Dendron for the web. If you have any new thoughts or questions on this, we'd be happy to discuss further. Thank you!

Dendron works via a web-based code server, such as lscr.io/linuxserver/code-server:latest.
However, it seems the vscode.workspace.fs API is not properly used by dendron and leads to pain in setting it up for web-based usage.


Swap words in strings by hot keys

Hello. I know that the functionality is similar already, but I would like the following
I want to set the cursor on one word and use the hot keys to change its place with other words.
was: "First! second? three_four. fiveSexSeven"
Set sursor on five and clicked 4 times ctrl+shift+alt+←
It became: "five! second? three_four. FirstSexSeven"
may also need to change the first letter

Interesting. Feel free to provide a pull request :)

a little similar to #143


Tensorboard not showing embeddings

I am trying to create embeddings visualization from a pretrained network called FaceNet although nothing shows up when I open it in Tensorboard. Here is my source code
import tensorflow as tf
import os
from tensorflow.contrib.tensorboard.plugins import projector

MODEL_DIR = "20170216-091149"
meta_file, ckpt_file = facenet.get_model_filenames(MODEL_DIR)
with tf.Graph().as_default():
    with tf.Session().as_default() as sess:
        # Importing the graph
        model_dir_exp = os.path.expanduser(MODEL_DIR)
        print(model_dir_exp)
        saver = tf.train.import_meta_graph(os.path.join(model_dir_exp, meta_file))

        # Restoring and retrieving needed layers
        saver.restore(tf.get_default_session(), os.path.join(model_dir_exp, ckpt_file))
        images_placeholder = tf.get_default_graph().get_tensor_by_name("input:0")
        embeddings = tf.get_default_graph().get_tensor_by_name("embeddings:0")
        phase_train_placeholder = tf.get_default_graph().get_tensor_by_name("phase_train:0")

        # Create variables needed for Tensorboard
        embedding = tf.Variable(tf.zeros([33, 128]), name = "embedding")
        assignment = embedding.assign(embeddings)

        # Create writer class
        writer = tf.summary.FileWriter(os.path.join(MODEL_DIR, "model"))
        writer.add_graph(sess.graph)

        # Setup the embeddings
        config = projector.ProjectorConfig()
        embedding_config = config.embeddings.add()
        embedding_config.tensor_name = embedding.name
        embedding_config.metadata_path = os.path.join(os.getcwd(), MODEL_DIR, 'labels.tsv')
        embedding_config.sprite.image_path = os.path.join(os.getcwd(), MODEL_DIR,'sprite.png')
        embedding_config.sprite.single_image_dim.extend([160, 160])
        projector.visualize_embeddings(writer, config)

        # Create the embeddings
        sess.run(assignment, feed_dict = {images_placeholder : face_images, phase_train_placeholder : False})

I have read the FAQs and this is what I got when I ran tensorboard --logdir 20170216-091149/ --debug
INFO:tensorflow:TensorBoard is in debug mode.
INFO:tensorflow:Starting TensorBoard in directory /Users/kevinlu/Documents/Facial-Recognition/facial_recognition
INFO:tensorflow:TensorBoard path_to_run is: {'/Users/kevinlu/Documents/Facial-Recognition/facial_recognition/20170216-091149': None}
INFO:tensorflow:Event Multiplexer initializing.
INFO:tensorflow:Event Multiplexer done initializing
INFO:tensorflow:TensorBoard reload process beginning
INFO:tensorflow:Starting AddRunsFromDirectory: /Users/kevinlu/Documents/Facial-Recognition/facial_recognition/20170216-091149
INFO:tensorflow:Adding events from directory /Users/kevinlu/Documents/Facial-Recognition/facial_recognition/20170216-091149/model
INFO:tensorflow:Constructing EventAccumulator for /Users/kevinlu/Documents/Facial-Recognition/facial_recognition/20170216-091149/model
INFO:tensorflow:Done with AddRunsFromDirectory: /Users/kevinlu/Documents/Facial-Recognition/facial_recognition/20170216-091149
INFO:tensorflow:TensorBoard reload process: Reload the whole Multiplexer
INFO:tensorflow:Beginning EventMultiplexer.Reload()
DEBUG:tensorflow:Opening a record reader pointing at /Users/kevinlu/Documents/Facial-Recognition/facial_recognition/20170216-091149/model/

The TensorBoard path_to_run seems correct as that is where I am storing my checkpoint files, the metadata file and sprite image.
Then I ranfind 20170216-091149/ | grep tfevents and I got
20170216-091149/model/events.out.tfevents.1491408129.Kevins-MacBook-Pro.local
Note the model folder contains the tfevents file and the projector_config.pbtxt file
So my data also exists. The last thing I did as per the troubleshooting was:
tensorboard --inspect --logdir=20170216-091149/ and I got this as the output
======================================================================
Processing event files... (this can take a few minutes)
======================================================================

Found event files in:
20170216-091149/model

These tags are in 20170216-091149/model:
audio -
histograms -
images -
scalars -
======================================================================

Event statistics for 20170216-091149/model:
audio -
graph
   first_step           0
   last_step            0
   max_step             0
   min_step             0
   num_steps            1
   outoforder_steps     []
histograms -
images -
scalars -
sessionlog:checkpoint -
sessionlog:start -
sessionlog:stop -
======================================================================

The code I used is exactly the same as the Hands On Tensorboard video code, and when I ran the debug commands as recommended by the FAQ the output of the MNIST tutorial is the same as above.

Please do try to provide all the details asked for in the new issue template and code that can reproduce the problem. I tried to edit the code snippet above so that the problem could be reproduced, but there seem to be a bunch of dependencies on other files which I don't know where to get from. Without a reproducible example, it is a bit hard to be helpful.
That said, I think this is because tensorboard doesn't know where the checkpoint is. You can probably fix that by specifying model_checkpoint_path in the ProjectorConfig using something like:
config = projector.ProjectorConfig()
config.model_checkpoint_path = os.path.join(model_dir_exp, ckpt_file)

Since this seems to be more of a usage issue and not a bug or feature request, I'll close this issue out and suggest that such questions (or follow ups) be posted on StackOverflow instead since there is a larger community that reads and answers questions there and we try to keep the github issues focused on bugs and feature requests.
Hope that helps!


When clicking "Exports" my private exports do not show in list

When I look at the list of "exports" it should show the global and my private exports by default, then clicking my exports should reduce the list to just those that are mine.
In other words, "my exports" should be a sub set of "Exports" not that my private exports are only visible to me if I don't see the global exports at the same time.
The listing should probably also indicate which exports are mine, mine private, and which are global.

For what it is worth, I think the presets work this way. It looks like if I click "Presets" I see all the globals and my "locals" then when clicking "my configurations" it just filters out the globals.

Looks to be working now. Displays current users published and unpublished jobs and published jobs from all other users. When user selects 'My Exports' then only current users published and unpublished jobs are shown.

Tested both the 'Exports' and 'Presets' page. All the exports and presets are shown by default, which includes the public and private items, which can then be filtered to simply your private items by selecting either 'My Exports' or 'My Presets' respectively.


[Event Request] Codeunit 80 Sales-Post - OnInsertInsertReturnReceiptLine

Hi Team,
In Codeunit 80 we need possibility to have our additional checks, it can be done as a new event or as an additional parameter request to existing event OnBeforeInsertReturnReceiptLine . Having event with handling in procedure InsertReturnReceiptLine would be a perfect solution :

[IntegrationEvent(false, false)]
local procedure OnInsertInsertReturnReceiptLine(SalesLine: Record "Sales Line"; ReturnRcptHeader: Record "Return Receipt Header"; WhseShip: Boolean; WhseReceive: Boolean; TempWhseRcptHeader: Record "Warehouse Receipt Header" temporary; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


ensure that clients are closed before the server is closed

I notice these types of errors in my test code, and tracked it down to the fact that the server instance was being closed prior to the client instances. This change seems to fix the issue and cuts down on a lot of logging chatter, plus enables my tests to run much faster.
18:22:44.810 [main-SendThread(<IP_ADDRESS>:9500)] DEBUG org.apache.zookeeper.ClientCnxn - Reading reply sessionid:0x1523c3990e40006, packet:: clientPath:/kafka/brokers/topics serverPath:/kafka/brokers/topics finished:false header:: 1882,12  replyHeader:: 1882,402,-101  request:: '/kafka/brokers/topics,T  response:: v{}
18:22:44.813 [main-SendThread(<IP_ADDRESS>:9500)] INFO  org.apache.zookeeper.ClientCnxn - Unable to read additional data from server sessionid 0x1523c3990e40012, likely server has closed socket, closing socket connection and attempting reconnect
18:22:44.814 [main] INFO  o.a.zookeeper.server.NIOServerCnxn - Closed socket connection for client /<IP_ADDRESS>:58794 which had sessionid 0x1523c3990e40017
18:22:44.814 [main-SendThread(<IP_ADDRESS>:9500)] INFO  org.apache.zookeeper.ClientCnxn - Unable to read additional data from server sessionid 0x1523c3990e40017, likely server has closed socket, closing socket connection and attempting reconnect

Thanks for merging. Would it be possible to get 0.0.3 pushed with this fix?

I pushed it and it should be getting synced to maven central. will echo back when it's done.

Oh wait, it just synced: https://repo1.maven.org/maven2/com/palantir/curator-test-rule/

Thanks!

On Jan 22, 2016, at 1:40 PM, Justin Uang<EMAIL_ADDRESS>wrote:
Oh wait, it just synced: https://repo1.maven.org/maven2/com/palantir/curator-test-rule/
—
Reply to this email directly or view it on GitHub.


[Edge Throughput] Add code to provision layer 2 networking in Linux host for Packet.net

Goal: Provide Layer 2 network setup needed for Edge Throughput comparison via code (eg. terraform + ansible)
Handle the following items:

[x]  Manually set up hybrid networking via packet.net web ui

[x] Leave bond0 alone on host / Packet configuration

[x] Remove bond for 2nd interface in host.  Eg. echo "-enp94s0f1" > /sys/class/net/bond0/bonding/slaves

[ ] Add additional tasks as needed to this ticket.

Test:

[ ] provision 2 packet systems
[ ] both packet test systems should be in same L2 network
[ ] configure IP on layer 2 interface on each host
[ ] ping from one packet system to another

[ ] No errors from ping
[ ] No packet loss from ping
[ ] All ping requests receive ping responses

[ ] arpping from one packet system to another

[ ] No errors from arping
[ ] All ARP requests receive ARP replies

https://github.com/cncf/cnfs/blob/master/comparison/cnf_edge_throughput/README-layer2-network.md

work was completed in #91 https://github.com/cncf/cnfs/commit/f883453a045bbf55696abf5304e0ec27c275f43c


[WIP] Implement BABE

There is a scaffold on BABE for now.

[ ] backend database which store blocks and then provide api that can find node in the same clan
[ ] block proposing and block verify (emm, including by database ?)
[ ] Serialize, Deserialize, (for now, there is only json)

This is a big MR, let's discuss it in detail sometime hah.
For the database part (For storing blocks), I was going to use a KV store like LevelDB (Bitcoin and Ethereum are using LevelDB as well). maybe you should also consider that.

Agree.

Also the block structure need to think about.
The block is roughly divided into two parts.

header
block content

header

name
type
comment

parent_hash
output of hash function
hash of previous block

state_root
output of hash function
The state trie merkle root

hash
output of hash function
The merkle root of current block content

digest
structural  information
some referencing auxiliary data, such as output and proof from VRF

@keyvank , maybe the RocksDB is better ? Because I had just write some code about the search of lowest common ancestor. It makes an iteration on keys and there are multiple kinds of data which exist at the same time. The RocksDB has a feature called columns and it can separate diff datas into diff LSM on physical. So an iteration on keys would not appear other kind of data

@kwanCCC  If you think RocksDB is needed, yeah sure why not. The important thing imo is to keep database backend abstract, so that we can also have a RAM-based backend (Such as a simple HashMap keystore) so that we can write tests that do not require a filesystem.

@kwanCCC I feel like we have diverged a lot :smile:

I had merged the code with master already. And I would do it periodically.

Looks outdated. Will reopen in case needed again.


Feature request: Check for Updates

It could be useful to check if there is a newer version of the software.

Hi
You can already check for update manually from the preferences dialog (and unless you disabled the feature, Beeftext checks automatically once per day).


Renderman makes the scene heavier on the viewport.

It seems that when you set the renderer to renderman the scene become a lot heavier that makes it hard to navigate around in the viewport.
Way to reproduce:

Open The Hidden People example scene.
Try to navigate on the viewport with the viewport shading in solid mode.
Change the renderer to eevee or cycles.
Try to navigate again. You can notice the difference. It looks like the scene become a lot lighter and easier to navigate.

Can you try turning off the light wireframe and light texturing and see if it's faster? You can turn those off in the RfB preferences.

Hi, it works. It became a lot lighter. Thanks. I also noticed similar problem when rendering to It (both render and ipr). This fix doesn't work for that though. But I think I'm gonna make a new thread for that.

I think this is a bug. I tested it with blender default cube scene with a pixar atrium. When I turn on all the setting where the light wireframe settings is. It became quite hard to navigate in Blender 2.93. However this is not the case in Blender 2.92.

OK, thanks for confirming. The slowness is probably because we are looping through all of the objects, trying to get a list of the lights in the scene. I'll look into ways to speed that up.

Can you see if the latest commits make it feel better for you?

I tested it with the one on the main branch. I think it's more or less the same. It could be a little bit better, but it's hard to say. The one on Blender 2.92 feels like there's not much impact at all.

You can see the difference in this video. https://youtu.be/F2_fnpuMaY0

I'm not seeing any slow down like what you're demonstrating in your video with 2.93, on my Windows machine. Do you have another machine that you can test with, that also shows the same issue? I can sort of get it to slow down if I have two copies of Blender running, but I would think that's expected. Is that what you're doing?

Nope, I only did that so I don't have to edit the capture. It's same with one instance of blender. I see. That's weird. Unfortunately, I don't have other machine at the moment.
For the record, I'm on a laptop with 16 gb and RTX 3060, on Windows 10.
By the way when I render IPR to It, I also feel the same thing happening. Do you also not experiencing this?

@prman-pixar The problem vanished when I copy the patch again. It seems like I made some mistake when copying them before. Thanks for the help and sorry for taking your time.
Btw, the problem when rendering IPR to It still happens though. Do you also experiencing this?

Which problem with IPR are you referring to? That it's slow to tumble the camera when IPR is started? If so, how much of a slow down are we talking about? Some slow down is expected, but it's not clear how much you're seeing.

It's pretty bad actually. You can see it in the video along with some of the bugs that I found.

Tumbling the camera during IPR in It is really slow (1st launch)
Blender will crash when you hover over the overlay menu on the viewport during IPR (both blender and It) (1st launch)
Importing the environment and the lighting from the preset browser will produce some error. Importing material work as expected. (2nd launch). Importing from the menu work as expected as you can see in the 1st launch.
When you render to It, you can see some activity on the blender image viewer. Because of this when you set your temporary editors to new window and you render to It, blender will launch its own image viewer. This is a nuisance. If this is due to blender limitation, wouldn't it be a great idea to expose those settings on the property window to make it easier to change it? (3rd launch)

Here's the video link, https://youtu.be/aerJAaEUgS8

Just want to add to this, I'm also experiencing bad viewport performance when IPR to IT is executed as shown in the video above.
I'm running on Blender 2.93.2 with the latest patch available on GitHub and my machine specs are RTX 2070 Super and Ryzen 9 3950x and even when I set the Render Threads for RenderMan to only use 3 threads (out of my 32 available threads), the viewport performance seems to be the same.
IPR in Blender is very responsive.

@TzurH What about my second and third bugs on the list? Do you also experience these issues?

I use the addon from the main branch from 6 days ago. I don't think the debug logging is the issue. I only set the logging to debug for the video so you can see them.
Here's the result with the latest main.

tumbling speed: The tumbling speed on the viewport (not camera) has significantly improved however the tumbling speed with the camera for ipr only improved a little bit. It's still really slow. Turning off lights texturing helps a little bit, but it's still really slow.

importing from preset browser will produce error: Now it works as expected.

hovering the overlay menu: It no longer crashes when hovering the menu.

When you render to It, you can see some activity on the blender image viewer: I wish there's a way to change that settings temporarily so we can change it automatically and doesn't mess with user's preferences.

Can you see if my latest commit fixes the tumbling problem?
https://github.com/prman-pixar/RenderManForBlender/commit/f04f6882e01ec8a379a8440353625b0c3e570d84

Good news. It fixed the bug. Great work and thanks.

I can also report that using this code from 17/08 (https://github.com/prman-pixar/RenderManForBlender/tree/2e8c0214a88b5d81ca6b9d7af0e83af5e629ec95) fixed the performance issue I was experiencing in Blender viewport when rendering to IT.
As for the other issues reported here by @nuzzar regarding preset browser error and hovering over viewport overlay crashing Blender - I'm not experiencing those either.
Good stuff!

Great! Thank you both for confirming.


Feature/flexbox charts

This is how I would suggest we wrap and render the charts. I haven't figured out yet, how to stretch them to the full width using flexbox, but that shouldn't be too hard. fee free to take over here or leave it as is till I'm back
this PR depends on #35 which is from my side mergeble

I took over the button styling now and adjusted the styles to match the current state in Figma:


server down?

I suggest making your own server, I stopped using the public localtunnel server ages ago. There's no free lunch here.

Hey, I just checked and the server seems to be working fine.
I'd suggest checking if your firewall is blocking the connection to that port, just like the errors says.
If it's still not working, i'd suggest setting up your own server (on your own domain) and testing it out there.
I've been busy trying to fight off abusers (people hosting malware/phishing portals on this service) and the server has been going down frequently since code changes need me to restart the node services. 😞

Now it's working just fine. Thanks!


Increase lower version bound for TransformProgressEventCrossVersionSpec

Injecting a Provider has only been introduced in Gradle 5.4,
so the test was failing on Gradle 5.3. I don't think it is worth to keep the
test running on Gradle 5.3.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Feature Flag Schema Samples/Tests

A json file with various versions of feature flags to validate our Feature Management libraries. The markdown file gives the expected results and a test description of what is going on.
Validated the results against Python and DotNet. Python has a PR up to fix bugs found by this.
Spring currently produces different targeting results; we will have to discuss how to make the update without it being a breaking change.

FeatureFlagSchemaV2 contains the variants and telemetry. I think we should also cover variant allocation in the test suite.
Another important thing is that the targeting evaluation is sticky and it should give the same result on the same targeting context across different FM libs. The percentage allocation should use the same algorithm across all FM libs.

Should we also include some invalid filter settings in the test suites?

FeatureFlagSchemaV2 contains the variants and telemetry. I think we should also cover variant allocation in the test suite. Another important thing is that the targeting evaluation is sticky and it should give the same result on the same targeting context across different FM libs. The percentage targeting evaluation should use the same algorithm across all FM libs. And we should include some testcases to ensure that consistency.

I wrote this to validate Python, and it doesn't have variants yet, so they weren't included.
I wanted to get this up ASAP as we already found the Targeting issues.

Usage in python https://github.com/microsoft/FeatureManagement-Python/pull/20

I think we need to think carefully about the format of the testcase. Currently, we have a property called "Expected Result."
Now, this property is either true/false or "Exception."
If we want to introduce variants in the future, what should the expected result look like?
Once variants are introduced, we need to check both the variant and feature flag status.
Further, should we unify the exception which will be thrown in different FM libs? In this case, we can even specify the exception type in the testcases.
I proposed the following shape for our test suites:
{
    "TestId": "ExampleTest-HelpfulTestNameForDebuging",
    "FeatureFlagName": "ExampleFeatureFlag",
    "Inputs": {}
    "ExpectedResult":{
       "FeatureStatus": boolean,
       "Variant": string,
       "Exception": string
    }
    "Description": "1. Describe the feature flag and the input. 2. Describe the reason why it should get the expected result.",
}


Incorrect 500 error response for malformed SIP URI

Describe the bug
We have identified an issue where the server incorrectly responds with a 500 Internal Server Error upon receipt of a malformed SIP URI from an Agent. A 500 error code suggests that the issue is with the server, which is not the case in this scenario.
Given that the error originates from the client (i.e., the malformed SIP URI), we would expect the server to respond with a 4xx error code. This category of SIP status codes is meant for cases where the client seems to have erred.
To Reproduce
Using Zoiper, setup an Agent to point the domain sip.local. Then, attempt to send an Invite message with "test@e" in the calling field.
Expected behavior
Routr should return a 400 Bad Request
Screenshots

System information (please complete the following):
NA
Additional context
None.

Note to self:
I was unable to find any reference in RFC 3161 regarding what constitutes a malformed SIP URI, except that its general form should be sip:user:password@host:port;uri-parameters?headers.
I decided to handle cases like the one described in this issue by simply forwarding the messages as long as the SIP stack accepts them.


Picovoice Issue: Koala startup not working on Google Cloud Run

I've been able to run the Koala SDK locally, however when I deploy my code to Google Cloud Run, I get an pvkoala._koala.KoalaInvalidArgumentError when initializing the client - no other error message.
My Picovoice console says that I have 0/3 monthly active users, and I've successfully been able to run the SDK from multiple different machines (without the active users count increasing).
Do I need to be on the Developer plan for this? My username is<EMAIL_ADDRESS>

Which function returns this? Are you sure your arguments are passed in correctly?

@kenarsa the code that throws this is:
pvkoala.create(
  access_key=os.environ["KOALA_ACCESS_KEY"],
)

and I've confirmed that KOALA_ACCESS_KEY is in my environment variables on Google Cloud Run
wondering if there's a problem since I first tried the access key locally on my Mac, and am now running this on a Linux instance.


Make shared component scope default

In most cases when resolving objects graph we want different consumers of the same service to have a reference to the same instance of that service instead of having unique instances. Then shared components scope makes more sense as a default value.

@ilyapuchka I'm curious as why this is the default scope returned.  Feels like this opt-ins to implicit state which can be difficult to reason with when there are issues with the state of those instances.  I could also make arguments to where UTs could become flaky or difficult to also reason with.
This is purely a thought exercise, just curious if you had an experiences with this occurring.

It seems to me that this is the most common scope, it's rear when you really need unique objects in the graph or those objects have only single reference so there is no difference between shared and unique scopes for them.

Maybe I'm misunderstanding it let me try and clarify my own mental model of it...
Say you have navigation stack where you have VC's  A, B
VC A gets an instance of type Foo and sets a field in it to "Bar"
VC B gets pushed on the stack and grabs an instance of Foo as well.
Given both these instances of Foo are injected/resolved through Dip used the default/shared scope.
Could we expect VC B's instance of Foo have this field set to Bar at this point as well?

Yes, they will share the reference to the same object.


Title missing in search for litterature

Title seem to be missing from the api. Maybe we should remove items that are that incomplete?
https://demo.gbif.org/cms/search?type=literature&category_country=DK

The same link appears to be showing title now.


The ARM architecture is not supported.

I want to run FMI4J in the arm environment, but the system reports an error libfmi4j-import.so This so belongs to the x86 architecture. Where is the source code of this so? Can I compile it into the so of the ARM architecture?Also, are there any other so's that need to be compiled?

The folder fmi-native-import contains the native code  and the build script is located here: https://github.com/NTNU-IHB/FMI4j/blob/master/fmi-native-import/build.gradle
It uses Gradle also for the native code, which is, well interressting.. Other than this I can't offer much more support. I have not developed with Java/Kotlin/Gradle for a long time.

many thanks！！！！！ let me hava a try


Add richer media support via webm.

Currently token metadata that includes .webm content results in "Failed to load".
This change detects .webm files and renders the appropriate  tag in place of the  tag.  Doesn't affect image loading.

Just wanted to check on this one as our last PR was already merged https://github.com/near/near-wallet/pull/2337, and this was created before...
If there's anything else required, it would be great to get some feedback.  Thanks.

@corwinharrell I think that Ideally, we'd have a Loading SVG image that matches the stylistic choices that were made for the FailedToLoad image so we can display that image in cases where the video failed to load.  Is that something you can create?

Actually, the Loading svg is likely overkill as the  element natively has its own loading animation which we are all very familiar with.  The key was adding the transparent gif as per your suggestion and the post you provided.  Now the failed to load svg only shows in the poster area when that condition exists, otherwise the spinning loader is shown while loading, or on success the video.

It's funny how complex such a simple thing can be sometimes and how our "understanding" of a thing often changes... my own understanding being what I'm referring to.
When making the fix above and removing the extra autoplay so that only the first video in each group is set to autoplay, I noticed while testing this change that the other video's now all showed the clear gif image as the poster (not pretty) instead of the first frame of the video.  So... digging back into that post and WHY the gif was being used in the first place, led me to understand it's because they were trying to display an image WHILE loading (which going back to my last post, I think is a bit of overkill) as everyone is familiar with

which again is only shown when actual loading is happening, NOT in an error state.  Who is on slow 3G/dialup now days?  Do you concur?

Is this waiting on me for something?


Game runs at 1000x speed when window is not active

Alt tab away and you get a cacophony of sound effects and near-instant game over.
Mac. I think this is a recently introduced bug (I was messing with frame timing stuff recently).
First figure out why this is happening, then maybe make the game automatically pause when window is inactive.

Doesn't happen in Linux


ESP32 does not boot with external power after BLEDevice::init

HI.
i'm newbie with esp32. I designed a simple BLE scanner wichs works fine when connected to usb. The problem is that if i disconnect board from usb and connect it to power (5v) device does not boot. Just red led is On and even if i push boot or en buttons...it never boot again. Any ideas?
Thank you

Hi! Have the same issue. I use Arduino IDE, if it matters. Could you please help me a bit, don't really understood the solution. Thanks!

https://www.google.com/search?q=esp32+brownout&oq=esp32+brownout&aqs=chrome..69i57.14248j0j1&sourceid=chrome&ie=UTF-8


Use the earliest possible timestamp for metric points

This ensures that we're using the earliest possible timestamp for metrics points when aggregating timestamps together (e.g. cpu+memory or multiple containers in a pod), which means that we can safely use the timestamp and window to determine if the given point is tainted by pod initialization.
This also lowers the hardcoded window from 1 minute to 30s, to reflect what the max it could actually be now that we're collecting from cAdvisor directly (see the commit message for an explanation of this time).

cc @brancz

1 minute to 30s

This doubles the number of metrics the metrics-server needs to ingest, I'm a little worried that this could blow up in large scale environments. Can we maybe make this configurable so that people can bump this back to 1minute if it does blow up?


Fix sscanf field width specifier

This (https://github.com/PX4/Firmware/commit/db44129ec099a05debf9187da2fd09035c9a67d7) doesn't seem to work when built with the NuttX standard libraries for real hardware. This fixes it.
In particular, if the width specifier is given, then it may ignore white spaces in the string and would have always taken the maximum number of characters allowed by the buffer.

@nuttx @davids5 I reviewed this and I think its a correct fix. The corner case present also in upstream NuttX is that if the specified width is larger than the actual width we still parse for the specified width.
Instead, what we expect with the width specifier is to get the result of this width = MIN(hwidth, width).
I will merge this. @davids5 Can you please propose a patch to @nuttx for upstream? The issue is present in upstream here: https://bitbucket.org/nuttx/nuttx/src/5b46ce4889e034bd2def7367563b62b4122dcb61/libc/stdio/lib_sscanf.c?at=master&fileviewer=file-view-default#lib_sscanf.c-433:439
Note that the check there just checks for available buffer space, but not if the argument is shorter than the width specifier.
@dennisss Thanks!


[util] Set enableRtOutputNaNFixup for Observation

Observation (906100) produces artifacts in certain scenes on RADV. Setting enableRtOutputNaNFixup to True in dxvk.conf file resolves those graphical issues.
 Jesus.


Create Indy Fork Sync Workflow from main repository Commonjava/indy

Guys by acident i press close the PR on previous one: https://github.com/Commonjava/indy/pull/1877 so i'm recreating this one and we can delete that one. Please let me know (based on previous Q/A if i can merge this one?

@geored As we have new github action workflow to do the CI, I think this is not needed and close it. If you think this is still needed, just feel free to re-open.


Prereqs validation enhancement

Research option to check system table existence without admin privilege
Check group permission for user in system tables

Fixed by #8

Fixed by #8


Optimize Header.getValue

Description
Optimizes Header.getValue to check a few cases before defaulting to String.join to combine the values of the Header. String.join has non-negligible overhead as it allocates a few different types of arrays to join the passed Strings but in most cases header will only have a few values. This changes implementation to check a few sizes of the values and use static String creation instead of the generic String.join, passed these sizes String.join is used. For the specialized cases getting the header value went from 7ns/op to 1ns/op, a 7x improvement, but the major win is memory usage of the most common case of one value in the Header. Running a getValue on a loop for 120 using the optimized case only saw a few MB of memory usage (mainly from the runner framework) where using String.join saw ~400-500GB of byte[] and String allocations.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Additional issues with Certbot?

There appear to be additional considerations when adding encryption to support https access. Is this addressed elsewhere in your (excellent) work?

Apologies for the late response, @eolach. You would want to set up SSL/TLS to secure Nginx.
https://medium.com/@pentacent/nginx-and-lets-encrypt-with-docker-in-less-than-5-minutes-b4b8a60d3a71

Thanks.
Sent from my iPhone

On Jan 31, 2019, at 7:34 PM, Michael Herman<EMAIL_ADDRESS>wrote:
Apologies for the late response, @eolach. You would want to set up SSL/TLS to secure Nginx.
https://medium.com/@pentacent/nginx-and-lets-encrypt-with-docker-in-less-than-5-minutes-b4b8a60d3a71
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


bump alpine (base image) for security

Description of your changes

Fixes #
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@Yan-Ou @fgksgf PTAL


Emoji + Nerd Font Support?

I would love to see support for double width characters and also having functions to fetch emojis and nerd font icons by name, to help declutter all of the unicode symbols. This would also create an opportunity to have alternative icons when no double width characters are supported.

NB: double width characters are supported in all Charm products as a rule.

NB: double width characters are supported in all Charm products as a rule.

Great! The second part of what I said is the main point here. I haven't really experimented with emoji yet in bubbletea, but I will try it out.


Improve error logging

We are seeing intermittent failures to collect metrics. When failures happen, our logs are showing the following nil pointer dereference error:
2016/09/15 05:29:39 Collecting running and scheduled build and job metrics
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x408126]
goroutine 1 [running]:
panic(0x847ba0, 0xc8200100a0)
        /usr/local/go/src/runtime/panic.go:481 +0x3e6
main.listBuildsByOrg.func1(0x1, 0x0, 0x0, 0xc82005d500, 0x0, 0x0)
        /go/src/github.com/buildkite/buildkite-metrics/main.go:402 +0x386
main.(*pager).Pages(0xc820024080, 0xc82004daa0, 0x0, 0x0)
        /go/src/github.com/buildkite/buildkite-metrics/main.go:383 +0x3c
main.(*result).addBuildAndJobMetrics(0xc8200b4b00, 0xc8200125f0, 0x7fffdefa7d10, 0x3, 0x0, 0x7fffdefa7d4b, 0xd, 0x0, 0x0)
        /go/src/github.com/buildkite/buildkite-metrics/main.go:303 +0x13c
main.collectResults(0xc8200125f0, 0x7fffdefa7d10, 0x3, 0x0, 0x7fffdefa7d4b, 0xd, 0x28, 0x0, 0x0)
        /go/src/github.com/buildkite/buildkite-metrics/main.go:116 +0x4ab
main.main.func1(0x0, 0x0)
        /go/src/github.com/buildkite/buildkite-metrics/main.go:66 +0x140
main.main()
        /go/src/github.com/buildkite/buildkite-metrics/main.go:82 +0x76f

We get a nil pointer dereference error because the code calling the buildkite API ignores errors which are returned. This change adds error handling for two cases where errors are ignored, so that we can report better error messages in the future.

This looks good to me! @toolmantim + @lox?

Thanks for the PR and fix @yeungda-rea.

Thanks for the feedback and merging in so quickly. We're giving it a go today.

Are you using this as part of the buildkite elastic aws stack @yeungda-rea?

Sort of. We forked it in February and the two have diverged quite a lot since then.

Would be great to catch up with you guys at some point and figure out what has diverged and whether it's worth getting any of that back into the core.


Error with slim 3.0.6

When saving a slim template file this happens:
Error detected while processing function <SNR>70_BufWritePostHook[3]..<SNR>70_UpdateErrors[15]..<SNR>70_CacheErrors[35]..257[1]..256[23]..SyntaxCheckers_slim_slimrb_GetLocList[7]..S
yntasticMake[52]..syntastic#preprocess#slimrb:
line   24:
E684: list index out of range: 0
Error detected while processing function <SNR>70_BufWritePostHook[3]..<SNR>70_UpdateErrors[15]..<SNR>70_CacheErrors:
line   35:
E170: Missing :endfor
Error detected while processing function <SNR>70_BufWritePostHook[3]..<SNR>70_UpdateErrors:
line   15:
E171: Missing :endif

I'm using slim 3.0.6.
I'm using the default syntax checker for slim (which is slimrb). It started about 1 week ago.
How can I help to debug this?

Right, this is new code, trying to work around a misfeature of Vim.  Please run slimrb -c your_file in a terminal, and post the output.

$ slimrb -c app/views/pages/home.html.slim
_buf = []; _buf << ("
HELLO
".freeze);
; _buf = _buf.join

Simplified example. It raises the exception anyway. Thanks

Forgot to attach vim version:
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Oct 19 2015 14:25:19)
MacOS X (unix) version
Included patches: 1-898

I'm interested in the exception...

The exception is the same:
"app/views/pages/home.html.slim" 1L, 9C written
Error detected while processing function <SNR>70_BufWritePostHook[3]..<SNR>70_UpdateErrors[15]..<SNR>70_CacheErrors[35]..257[1]..256[23]..SyntaxCheckers_slim_slimrb_GetLocList[7]..S
yntasticMake[52]..syntastic#preprocess#slimrb:
line   24:
E684: list index out of range: 0
Error detected while processing function <SNR>70_BufWritePostHook[3]..<SNR>70_UpdateErrors[15]..<SNR>70_CacheErrors:
line   35:
E170: Missing :endfor
Error detected while processing function <SNR>70_BufWritePostHook[3]..<SNR>70_UpdateErrors:
line   15:
E171: Missing :endif

I'm sorry but I can't paste the output of slimrb -c of my real file because it was sensitive data in it.

Right, never mind, what I was trying to do doesn't work.  Commit 6461248 goes back to the old code.  Just make sure you don't try to check files with spaces in their names. :(

Just updated. It's working again. Thanks!


When I use copy...When the form gets the data from the CSV file, if there is an error data in the CSV file, it will stop pulling the data.

Use case:
When I use copy...When the form gets the data from the CSV file, if there is an error data in the CSV file, it will stop pulling the data.
Feature description:
If this problem is encountered in other databases such as mysql, the error data will be written to the log file to help us solve the problem. Can this mechanism be adopted in crateDB?

We've the RETURN SUMMARY option so that more information on what went wrong is returned.

When the amount of data is small, reports will be generated. After the amount of data is about 100 million, no reports will be generated


LiteSpeed Cache Image Optimization compatibility

https://wordpress.org/support/topic/litespeed-cache-image-optimization-2/
Ok, so I have just swithed to Litespeed hosting, configured Lightspeed cache properly, and I wanted to optimize the images on the website using Litespeed.
Prior to this, I have configured WP Stateless plugin to stateless mode, meaning that all my images are served from GCS, and none of them are on my server.
I hit “Send optimization request” and I get the error: Image files missing: 1263 groups (13793 images)
Optimization won’t start because of this. I know your latest release added compatibility to LSCache plugin. Do the images still need to be on my server for the optimization to work?

https://github.com/litespeedtech/lscache_wp/issues/143

https://github.com/litespeedtech/lscache_wp/commit/3ff72fd135f36b521e806592181b3ea291922c2b
https://github.com/litespeedtech/lscache_wp/commit/14fb30a21527af68b8473273819b64b0d920c3e5

@alimuzzaman please proceed with this. Let's support them!

WP Stateless is now compatible with LiteSpeed Cache.


Update SNYK configuration

Description
Update SNYK configuration
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.02%. Comparing base (e2af104) to head (0d6b15c).
Report is 34 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1295      +/-   ##
==========================================
- Coverage   94.75%   94.02%   -0.73%
==========================================
  Files          57       58       +1
  Lines        2441     2479      +38
==========================================
+ Hits         2313     2331      +18
- Misses        128      148      +20

see 4 files with indirect coverage changes

/lgtm

/approve


add missing notifications UIUserNotificationType.providesAppNotificationSettings

Which platform(s) does your issue occur on?

iOS
iOS 12+
simulator and device

Please, provide the following version numbers that your issue occurs with:

CLI: 8.0.2

Please, tell us how to recreate the issue in as much detail as possible.
I am trying to enable this feature
Doc: https://developer.apple.com/documentation/usernotifications/unnotificationsettings/2990404-providesappnotificationsettings?language=objc
This isn't possible as the plugin only support Alert Badge and Sound UIUserNotificationType.

@francois-pasquier thanks for the report. I dont really have much time to work on this right now. You are welcome to create a PR if you need that feature
Thanks


feat(s2n-quic-tls): Add new API to configure client auth type of s2n_tls endpoint client/server (mTLS)

Resolved issues:
resolves #1724
Description of changes:
Allow users to configure ClientAuthType for the Client s2n_tls endpoint. This makes we can create an optional client auth type s2n_tls endpoint that can handshake success to both require or not client auth servers
Testing:
I have added some tests to make sure the new APIs work regardless of when they are called (before or after these existing functions server/with_client_authentication and client/with_client_identity)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @bnyu it seems like this is probably not a great first issue since it requires adding a new API (which we try and do sparingly) and we don't currently have a good customer need for this feature. Therefore its better to punt on this feature until a customer need.
Thank you for the PR and hope you learned something!


Allow a dictionary to be passed that contains the logging configuration.

Description
Allow a predefined/preconfigured logging dictionary to be processed by luigi.
Motivation and Context
I setup my logging via a YAML based configuration file and it was more convenient to be able to pass the dictionary to luigi rather than maintain an external .ini based logging config.
Have you tested this? If so, how?
I have been running this in my production code. (Using Python 2.7.x)

I mean, add docs to this page. Shortly describing how luigi in fact is just calling out to the existing configuration infrastructure in Python, and a small example YAML based log file.

I'll give it a shot an let you know when they're ready for review.

I've added a docstring and updated the documentation. Please review it when you have time and let me know if it looks good.

I don't know how you would provide a dictionary of logging configuration to luigi "directly" (I am doing it with a wrapper script); I'm guessing we'd need to allow a JSON or YAML file that would have the dictionary "string representation" of the logging configuration in it.  I'm open to ideas on how to accomplish this.

I think this is much easier than you think. As both Python 2.7 and 3.2 already comes with this support built-in. See 2.7 docs for instance.
Here's the except (from 3.6 docs)

In Python 3.2, a new means of configuring logging has been introduced, using dictionaries to hold configuration information. This provides a superset of the functionality of the config-file-based approach outlined above, and is the recommended configuration method for new applications and deployments. Because a Python dictionary is used to hold configuration information, and since you can populate that dictionary using different means, you have more options for configuration. For example, you can use a configuration file in JSON format, or, if you have access to YAML processing functionality, a file in YAML format, to populate the configuration dictionary. Or, of course, you can construct the dictionary in Python code, receive it in pickled form over a socket, or use whatever approach makes sense for your application.

I haven't researched this thoroughly. So I might have missed something, but it seems like it's plug and play from the docs.  Please let me know of your thoughts, and thanks for your fast reply! :)

@jsivak Any update here?

Sorry, been busy; I'll try to look at it this weekend.. based on Tarrasch's comment I was going to investigate providing a way to pass a .yaml file to the logging command line switch (and then convert the .yaml to the dictConfig that the logging system is expecting)..  I think that is what he is looking for.

I'm "back from the dead".. restarted work on this and wanted to ask:
Do you want to support .yaml and .json files for the "--logging-conf-file" option? I think that's the most logical way to get a "dict-like" config from a file.  I don't think we can store a dict-like config in an .ini file, so this would lead to a command line looking something like:
luigi MyTask --my-param 5 --logging-config-file my_file.yaml
Am I on the 'right track' on what you'd like to see with supporting a dict-based config file.

By the way. The docs are really good! :)

Did you consider solving this in the other way and getting rid of the has_run hack? Was it not possible?

I've never looked into why the "has_run" hack was put into luigi, so no, I've not looked at it any other way.

I've never looked into why the "has_run" hack was put into luigi, so no, I've not looked at it any other way.

I would appreciate if you looked over it as I mentioned here https://github.com/spotify/luigi/pull/1979#pullrequestreview-51650623. This is an open source project and in order for the codebase to remain fun to maintain and hack on, we from time to time need to fix other people's mess. Thankfully it would improve the interface you propose greatly as well. :)

Do you know when the "has_run" hack was introduced into the code and why?  (I can hut thru the commit messages, but if you happen to know..)

Looks like "has_run" was added back in 2013 (commit d276d0ebb78d7e260165a684460b9b5b4e5f3a78) when the setup_interface_logging() was intially added to luigi. Is there some specific reason you want it removed?

Well my problem is that before you introduced this PR, a luigi user could simply do logging.config.dictConfig(their_dict) and it would work splendid afaict. The only problem is that luigi will later run setup_interface_logging(..) and do it's own weird thing. It would be so nice if instead of we making the function setup_interface_logging(..) weirder by allowing users to also pass a dictionary (this PR), we just recommend them to do the standard python way logging.config.dictConfig(their_dict) and then setup_interface_logging(..) would automatically detect that there's no need to do anything as a user has already set up logging themselves.
Would this be possible?

Ah, that makes things more clear to me now. Thanks for that.. I'll see what I can find.

Ah, that makes things more clear to me now. Thanks for that.. I'll see what I can find.


Home Page clusters table fetches counts for clusters that are not ready

On the home page clusters list, if a cluster if not ready, we still fetch the counts for the cluster so we can show pod count - this gives an error in the console and leaves the spinner on the page:

@aalves08 On 2.6-head commit id: 7db4fe4e819

I see multiple behaviors when i refresh the page or when i navigate between CLuster Management and Home pages.
Issue 1

After a few attempts I was unable to reproduce ^^^ issue 1 but I see Issue 2 (below)

Issue 2

2 - are seen inPods section

Expected:
Only 1 - is seen to make it consistent with other columns

@sowmyav27 I've changed the - symbol to match the other cols.
Just to create understanding, if the issue/PR isn't clear, the spinner should only appear if the cluster is ready (aim of the issue was to fetch the Pods metrics only when the cluster was ready) and should go away once it fetches the metrics for the Pods.
Caveat: if we are looking at the table when we are provisioning a new cluster, and the live update updates the cluster status to ready, the - symbol won't go away (user needs to refresh) to again fetch the pods count (because that only runs once, when the table row is rendered).
To change this behaviour would entail a lot of work on the table component and might have unexpected changes to various areas of the UI.
The main goal of this issue was to prevent the API call when the cluster isn't ready, so that the user doesn't have errors on the console/interface.
FYI: @nwmac

Verified on v2.6-head https://github.com/rancher/rancher/commit/d5e68641e8f5f2d05bfa82731e5d8148e0f45017.
ENVIRONMENT DETAILS

Ubuntu 20.04: Docker Install
Rancher version: v2.6.5-rc3 and v2.6-head https://github.com/rancher/rancher/commit/d5e68641e8f5f2d05bfa82731e5d8148e0f45017
Browser: Chrome

TEST RESULT
PASS
VERIFICATION STEPS

Setup Rancher and create a Standard user and login to Rancher UI as that user.
Create an RKE1 and RKE2 cluster (specifications do not matter).
Navigate back to the home page and verified that I see "-" in columns that are not ready yet in the cluster:

Refreshed the page several times until the clusters finished and still did not see this issue.


info - array ser/des

Hi!
Not a real issue, just looking for some info.
Looking at the definition of the Ser interface, I see there are 3 different ways to serialize an array:

serializeArray;
serializeIterable;
serializeIndexableArray (this one is never mentioned in the readme)

What is the main difference between them?
Has each a better performance in a specific scenario? If so, can you make an example?

Hi!

serializeArray: serialize an array. the serialization process writes an integer to know the size, and after the items
serializeIterable: do the same, but keep an iterable. This allows you to serialize Set and Map<K, V> without converting them into an array. This safe memory allocation, therefore, is faster. Anyway, if you have an array, the serializeArray is preferable.
serializeIndexableArray: serialize an array and allow you to get some elements. For instance, if you need just the first and the 5th, you can deserialize only them using getArrayElements([0, 4], ...). You are right, and this is not documented because it is hard to describe (and probably we can improve the names here).

On my side, everything you said is clear and we can close this.
If you prefer, we can keep this open just as reminder about the documentation

If you would like to contribute, every tip/PR/suggestion is welcomed!
So, I'm closing this. Thanks for your interest in this project.


Allow advanced/fancy indexing and arbitrary Join axes in mixtures

This PR enables support for arbitrary NumPy advanced/fancy indexing (excluding the np.newaxis case, for now).  It also allows mixtures that are Joined (e.g. at.stacked) on any axis.  The Join axis must be have an appropriate MeasurableVariable index, though.
Here's an example:
import numpy as np
import aesara.tensor as at
from aeppl import joint_logprob

srng = at.random.RandomStream(29833)

# A "time" dimension
T_at = at.lscalar("T")

X_rv = srng.normal(-100, 1, size=T_at, name="X")
Y_rv = srng.gamma(0.5, 0.5, size=T_at, name="Y")
Z_rv = srng.normal(100, 1, size=T_at, name="Z")

I_rv = srng.categorical(np.r_[0.1, 0.2, 0.7], size=T_at, name="I")

i_vv = I_rv.clone()
i_vv.name = "i"

# Select a random mixture component across the "time" dimension (i.e. the
# first dimension)
M_rv = at.stack([X_rv, Y_rv, Z_rv], axis=1)[at.arange(T_at), I_rv]
M_rv.name = "M"

m_vv = M_rv.clone()
m_vv.name = "m"

logp_parts = joint_logprob({M_rv: m_vv, I_rv: i_vv}, sum=False)

# Evaluate the log-probability at a test point
res = logp_parts.eval(
    {T_at: 5, m_vv: np.r_[-100, -100, 1, 100, -100], i_vv: np.r_[0, 0, 1, 2, 0]}
)
res
# array([-3.22152363, -3.22152363, -3.02837643, -1.27561348, -3.22152363])

exp_res = np.array(
    [scipy.stats.norm(-100, 1).logpdf(-100) + np.log(0.1)] * 2
    + [scipy.stats.gamma(0.5, scale=2.0).logpdf(1) + np.log(0.2)]
    + [scipy.stats.norm(100, 1).logpdf(100) + np.log(0.7)]
    + [scipy.stats.norm(-100, 1).logpdf(-100) + np.log(0.1)]
)
assert np.allclose(exp_res, res)

We could also calculate the indices by splitting the advanced indices into basic ones and directly stacking them on top of one another (in shape of the subspace generated) in case of moved subspaces. But in the end the result would be the same as this logic.

We could also calculate the indices by splitting the advanced indices into basic ones and directly stacking them on top of one another (in shape of the subspace generated) in case of moved subspaces. But in the end the result would be the same as this logic.

I'm interested in this approach as well.  Can you mock up an example of this conversion in plain NumPy?  Would be good to consider it in case it has advantages over this approach.

LGTM. This is a really cool addition!
Otherwise, some branches of the expand_indices seem to not be covered by the tests, as well as the tag.test_value logic in mixture_replace.

Yes, I have a bunch of tests to add before we can merge this.  I might also try to add np.newaxis support.

Can you mock up an example of this conversion in plain NumPy? Would be good to consider it in case it has advantages over this approach.

Sure, I built these implementations: https://gist.github.com/kc611/8609c7d0e20558a36a7d927b1c5cc065
Seems like the logic gets more complex when we try to expand slices. But it is kind of simple if we treat slices as one basic index. (Made two implementations to reflect that difference)

This implementation (without stacking) can be also be useful in implementation of advanced indexing in Numba since indexing there is currently being done axis-wise not element-wise like we are doing right now. (Currently in Numba for one dimensional advanced indexing, the loops traverse it axis by axis hence they require the array of the indices of the current axis instead of the current element.)

Yeah, it might work well for a reimplementation in Numba when—say—the advanced indexing sub-space can be iterated efficiently.
For the log-probability computations performed here, we need to iterate on the axis that joins the MeasurableVariables, which also requires that we be able to split and filter the indices on each dimension accordingly, and I'm not sure how to do that in the basic index case—at least not without eventually turning each index into an ndarray that can be np.nonzeroed (i.e. into an advanced index).
Regardless, this approach is a very good insight.


Module shape isn't static in the presence of global variable assignments

The module system proposal almost guarantees that it's possible to determine which members a module has without executing it¹, with one exception: global variable declarations. For example, in the following example, the module only has the member $foo if condition() returns true at runtime:
@if condition() {
  $foo: value !global;
}

Worse still, if a global assignment occurs in a function or mixin, a module's shape may change after it's been fully loaded. This violates a number of assumptions that the rest of the proposal makes, as well as limiting many of the practical benefits of static module shapes.
I see two possible solutions:

We don't allow !global to define a new variable; instead, it may only assign to a variable that's already defined. This would be simple to implement and would match many other languages' semantics, but it would be a backwards-incompatible change, so I'm inclined against it.

We expand a module's variable members to include all global variable declarations in the module, even if those declarations aren't executed. These variables have the value null. Note that they wouldn't be visible within the module itself until/unless the declaration is executed, but downstream modules would have access to them.

1: This isn't an explicit goal, but it probably should be.

We've decided that we'd like to do solution 1 eventually, but we don't want to introduce a breaking change. So for now we'll pursue solution 2, while also deprecating global variable assignments that define new variables. Then, next time we make a breaking release, we'll drop support for global assignments that define new variables, and we won't have to do the syntactic check anymore.


Investigate alternatives to increase performance on containerized MariaDB service

Impact of the new feature
Look for ways to increase performance of MariaDB inside docker, with respect to host-based deployment.
Is your feature request related to a problem? Please describe.
The results from #11566 on the performance of MariaDB inside docker are not very promising for the online transaction processing tests.
See general remarks below:
https://gitlab.cern.ch/dmwm/wmcore-docs/-/blob/master/docs/wmcore/MariaDB-benchmark-tests.md#conclusions-on-oltp-benchmarks
Describe the solution you'd like
We should investigate why the degradation of performance in MariaDB. Results at the host vs container level with respect to File I/O, Memory or CPU were close enough, so there may be a way to increase the performance when running the MariaDB server application specifically.
Describe alternatives you've considered
Leave things as is, with the performance degradation noticed.
Additional context
Add any other context or screenshots about the feature request here.

There seems to be at least one difference in the old deployment config vs the container one, so I will be looking into the performance differences with this

@anpicci I haven't looked into the extensive results yet, but from a discussion with Kenyi last Friday, I understand that the performance can be degraded between 25 and 75%. Maybe 25% would be acceptable and fairly unnoticeable, but going to 50% will definitely give us problems with production workload.
It looks like there are known parameters/setup that will increase this performance, and I'd suggest to research and integrate those in before we go to production.

@khurtado these are great news.
Please take this opportunity to also persist the new MariaDB settings via a pull request for CMSKubernetes. The file that you are likely going to change is https://github.com/dmwm/CMSKubernetes/blob/master/docker/pypi/wmagent-mariadb/my.cnf

@khurtado I can work on the documentation

@anpicci Thank you!
@amaltaro I have created a PR with the changes

@amaltaro @khurtado I have updated the GitLab documentation with the new set of results

This issue has been fixed with https://github.com/dmwm/CMSKubernetes/pull/1475


Error: numpy.core._exceptions._ArrayMemoryError: Unable to allocate 115. TiB for an array with shape (3983360, 3983360) and data type float64

I am encountering an error: 'numpy.core._exceptions._ArrayMemoryError: Unable to allocate 115 TiB for an array with shape (3983360, 3983360) and data type float64' when I use HPC. However, I do not encounter any error when I use my personal laptop for the same code.

@NOORLEICESTER numpy.core._exceptions._ArrayMemoryError on the HPC system but not on your laptop indicates a limitation in the available memory on the HPC cluster compared to your laptop. HPC environments often prioritize memory efficiency, switching the data type from float64 to float32 can significantly reduce memory usage.
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thank you!

@NOORLEICESTER In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thank you!


[dns-sd] fix SRP service name length

Problem
The SRP service name buffers are too small.

Summary of Changes
Fix SRP service & host name buffer issue.

I kicked the Tests again, for some reason they're getting stuck...

I kicked the Tests again, for some reason they're getting stuck...

I am totally puzzled here - I have never seen these tests hang on any other PR, however I am unclear on why this PR would cause system tests to misbehave either (and across all compilers to boot). I did copy the PR and tried locally, it passes (except I found a Mdns hang which I fixed in #6721). Kangping reported his workstation hangs on TestSystemObject but I could not see that.

@woody-apple @andy31415 Thanks for helping resolve the test hang issue. I am sorry that it is too later for me to fix this issue yesternight, but it seems the hang issue is caused by the failure of the TestMdns test case and it is my change in this PR that is failing the TestMdns case :(
I submitted #6771 to fix the issue.