Cluster config api return verbose PA status

Signed-off-by: David Zane<EMAIL_ADDRESS>Is your feature request related to a problem? Please provide an existing Issue # , or describe.
Fixes https://github.com/opensearch-project/performance-analyzer/issues/322
When checking cluster state of Performance Analyzer, the responses are bit-encoded numeric values, which don't provide much meaningful insight.
Current cluster/config output:
GET localhost:9200/_plugins/_performanceanalyzer/cluster/config
{"batchMetricsRetentionPeriodMinutes": 7, "currentPerformanceAnalyzerClusterState": 19, "shardsPerCollection": 0}

Describe the solution you are proposing
Translate bitmap to Map<String, Boolean> output before printing.
New cluster/config output:
% curl -XGET localhost:9200/_plugins/_performanceanalyzer/cluster/config
{"currentPerformanceAnalyzerClusterState":{"PerformanceAnalyzerEnabled":true,"RcaEnabled":true,"LoggingEnabled":false,"BatchMetricsEnabled":false,"ThreadContentionMonitoringEnabled":false},"shardsPerCollection":0,"batchMetricsRetentionPeriodMinutes":7}

Describe alternatives you've considered
I have kept bitmap implementation instead of converting to some map object because of the provided Setting construct. We are using Setting.intSetting(). There is no map option (int, float, long, boolean... available).
Also, int representation will help a bit with memory.
Additional context
Add any other context or screenshots about the feature request here.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #342 (e4e7ed5) into main (3ab57eb) will increase coverage by 0.29%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #342      +/-   ##
============================================
+ Coverage     70.82%   71.11%   +0.29%
- Complexity      372      377       +5
============================================
  Files            44       44
  Lines          2581     2590       +9
  Branches        174      174
============================================
+ Hits           1828     1842      +14
  Misses          641      641
+ Partials        112      107       -5

Impacted Files
Coverage Δ

...dler/PerformanceAnalyzerClusterSettingHandler.java
75.67% <100.00%> (+7.04%)
:arrow_up:

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

As this is a breaking change(as api response is essentially changing), I think we should instead introduce a new parameter like ?expand and return meaningful status only when this is used. Using this we won't essentially need to fix integ tests or make change changes in RCA.
We can just update the README for this param usage. @kiranprakash154 @dzane17 Thoughts?
GET localhost:9200/_plugins/_performanceanalyzer/cluster/config
{"batchMetricsRetentionPeriodMinutes": 7, "currentPerformanceAnalyzerClusterState": 19, "shardsPerCollection": 0}

GET localhost:9200/_plugins/_performanceanalyzer/cluster/config?expand
{"currentPerformanceAnalyzerClusterState":{"PerformanceAnalyzerEnabled":true,"RcaEnabled":true,"LoggingEnabled":false,"BatchMetricsEnabled":false,"ThreadContentionMonitoringEnabled":false},"shardsPerCollection":0,"batchMetricsRetentionPeriodMinutes":7}

TODO: update README api samples

As this is a breaking change(as api response is essentially changing), I think we should instead introduce a new parameter like ?expand and return meaningful status only when this is used. Using this we won't essentially need to fix integ tests or make change changes in RCA.
We can just update the README for this param usage. @kiranprakash154 @dzane17 Thoughts?
GET localhost:9200/_plugins/_performanceanalyzer/cluster/config
{"batchMetricsRetentionPeriodMinutes": 7, "currentPerformanceAnalyzerClusterState": 19, "shardsPerCollection": 0}

GET localhost:9200/_plugins/_performanceanalyzer/cluster/config?expand
{"currentPerformanceAnalyzerClusterState":{"PerformanceAnalyzerEnabled":true,"RcaEnabled":true,"LoggingEnabled":false,"BatchMetricsEnabled":false,"ThreadContentionMonitoringEnabled":false},"shardsPerCollection":0,"batchMetricsRetentionPeriodMinutes":7}

I have updated to only return detailed output when ?verbose query param is passed

Overall LGTM. Just one minor comment.


fix: do not trigger CursorHold when exiting

Before i merge this, could you please provide a testcase showing the problem?

Wait a minute, this is not the correct solution.


If bind address is set, configure it.

best i can tell, this is a duplicate of PR https://github.com/cturra/docker-ntp/pull/45. pending feedback in that issue from @qrpike.


Update AccountFinancingMode.cs

Transaction with AccountFinancingMode are returning DAILY_COMPONENT and SECOND_BY_SECOND_COMPONENT. Not sure if these are new or replacement values, but I needed the change to prevent serialization errors on transaction streaming

Not sure who to reach out to at OANDA to get some official info


[Work Item] Add required permission to README.md

Describe the work
We must add the required permissions to execute the script to the README.md. We have different requirements here for the different parameter sets.
Additional Context
Many customers are running the script without having the appropriate permissions assigned. This causes the script run to fail without visible reason.

@dpaulson45 please assign to me, thanks.

Addressed in #532


Read metrics from file

#698
In the metrics.toml config, specify which metrics driver to use (key: metrics_data_driver_type), one of: dummy, json
By default dummy is set, giving some ficticious metrics values.
For json a default json metrics data file location (/var/tmp/reva/metrics/metricsdata.json) is used. Otherwise set the metrics_data_location key in the metrics.toml file.
The Reader interface method signatures have changed to return the metrics values. Any driver must implement this interface.

Also, instead of the current code, where we sleep between reading the metrics, let's use the stats channel as done here.

@ishank011 @labkode : I see some issues here. First, to our understanding there can only be one single source from which the data for a particular metric can come from (metrics methods are defined in the Reader interface). Meaning, although there may be multiple implementations (ie. drivers) for the metrics interface there can only be one used at runtime (again: there can only be one source). So, we cannot have 'dummy' implementation running in the background as that would mean a second data source for the same metrics. So, that's why I had multiple (json, dummy) drivers (data sources) coded, implementing the Reader interface, so choosing the one you'd like to use would be a matter of configuration. We also envision that a site could choose its own data source, so possibly another driver (eg. db) that has to implement the Reader interface.
So, if in the future eg. storage driver (as you mentioned) would want to have some additional (other!!) metrics registered then we would expand the Reader interface with these additional metrics methods, and these methods would then need to be implemented by all metrics data drivers so the new metrics would also be read from the data source.
Now in this view prometheus only needs to initialize the Metrics construct + configure which driver to choose and you're good to go (it does work actually!). The metrics.RecordMetrics() function (as used in prometheus.go line 82) is there because I thought we should get rid of the continuous running record job (I just thought that it would be only necessary to read data when the prometheus server is called), but I understand that is what you want. No problem!
I have an issue with using the Reader interface like you do. Checking whether the interface is actually implemented is done within the implementing driver itself, that's not really checking, it’s you checking you. This makes the interface sort of useless. That's why I made sure the metrics pkg does a proper checking of the driver really being a Reader (that way if a method is not implemented you get a nice message when building, meaning the driver is not complete). If this is not done we can get rid of the reader interface completely and make the drivers define all metrics methods themselves. Less code.
I must confess I'm not experienced in golang so my implementation/coding of these views may be a bit clumsy here and there. All comments on that is super helpful to me !
So, here are our views on the mechanics behind my PR. I want to make clear that I have no problem implementing it the way you've suggested. We were under the impression that having the metrics read from a data source (file,db) would be a short time solution anyway (hopefully metrics will become a part of the CS3 API's in the near future). I just want to make sure we understand each other. Hey, we can always do a videocall on this.
So please skip my comments and tell me to proceed with your suggestion if you like. No problemo!

@ishank011 @labkode : This drawing is what we understand of 'read metrics from file'. It might help to pinpoint where our views on this may diverge.

@ishank011 : created new PR #1118


fluxedredstone{<IP_ADDRESS>} crashes dedicated Forge 1.10.2 build 2026 on startup

Tesla updated to 1.10.2-<IP_ADDRESS> and now your mod doesn't recognize it and complains that it can't find it.
crash report:
https://gist.github.com/35a896cbb831dd57b22c13efca1ca6fc
fml server latest:
https://gist.github.com/0b3d620b9cdc468a8d1b3d4ddd887598

Ok I have talked to the tesla dev and somethings have changed. This version of tesla is only an alpha so I will provide you with the jenkins link to a compatible version.

You can get a compatible version here: http://modmuss50.me:8080/job/FluxedRedstone/


Change rbd example to use v1

Moved some files out of a v1beta3 sub directory and converted them.
#9286
@caesarxuchao @bgrant0607 @nikhiljindal

LGTM


Jar size and MC 1.17.

You should update the Gradle to 7.0.1 to be able to build with Java 16, which is needed for MC 1.17.
Compress assets. You can use lossless compression to cut ~25% of the file size or go with lossy and make it 1/4 of the original size (the quality difference shouldn't be noticeable even when you use rough compression on the particles/splashes).
There is a GitHub bot that should do this for you https://imgbot.net/

Tried using lossy on yellow_splash_1.png - 8,0 KiB >> 105 B so like 99% smaller file size. So even if you do this only for splashes you should easily cut in half the jar size.

Updated to 1.17.
Had to temporarly remove colored lighting support since the mod that allows that isn't on 1.17 yet, will be re-added when possibile :)
I will reduce size as soon as i have time!


Idea: don't set default tag for Heading component, restrict allowed tags, add more examples

Headings elements carry semantic meaning in conveying page hierarchy but it is often used mistakenly to convey stylistic meaning. Incorrect usage of headings can be confusing for screen reader users and detrimental to accessibility.
Proposed changes:

Don't have any default tag and require the consumer to always pass it in.

Currently the tag defaults to :h1. However, the heading level is largely dependent on the page/context it is to be used so it may not make sense to have a default. By not offering a fallback, we encourage the consumer to be intentional about heading levels.

Restrict the possible tags to <h1> ~ <h6> tags for this component

Currently there are no restrictions.

Add examples that dissociate heading levels from aesthetics.

Often times, people using heading levels for stylistic purposes (e.g. they want a large text and so they use <h1>), but this should be discouraged. We should have examples in the documentation that disassociate heading levels from aesthetics (maybe examples using font_size:). We can also redirect the user to use the Text component with relevant typography arguments as an alternative.

Fantastic! I am 👍🏻 to all of these suggestions. I think something like HeadingComponent.new(tag: :h3, font_size: 1) is the kind of example we need in the docs to emphasize this concern.


[Feature Request] Allow reference to declaration files with import and export keywords

Search Terms
d.ts reference declaration import export
Suggestion
Allow reference to declaration files (.d.ts) with import and export keywords
Use Cases
I want to use it for type extends and reuse，Currently, it can only be import as a module.
Examples
example.d.ts
import { Action, AnyAction, Reducer } from "redux";

interface Model<S = any, A extends Action = AnyAction> {
    namespace: string;
    state: S,
    reducers: Record<string, Reducer<S, A>>
}

example.ts
/// <reference path="./example.d.ts" />

const User : Model = {
  namespace: 'user',
  state: {},
  reducers: {

  }
}

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Duplicate of #36812.


Wireless Config update

I added a command that display the IP address of the device we want to connect to. This will helps a lot users from going to Settings -> Wifi to get that informations.

Thank you :+1:
I applied some formatting changes: https://github.com/Genymobile/scrcpy/commit/pr1944
Rendered: https://github.com/Genymobile/scrcpy/blob/pr1944/README.md#wireless
Please review, I'll merge it soon :)

Merged 47c897126758ed016881bdf51b2eb643c4a5fe07


enabled core cell delete for IW and undo operations

https://github.com/microsoft/vscode-jupyter/issues/7756

Check if an operation should be placed on the undo stack, rather than checking if undo is available at all

The interactive window is a partially read-only notebook, and delete cell is an exception since it can be performed by the user. We want to enable undo for that operation

deleting a cell while having focus elsewhere requires first focusing on the history, but doesn't always work depending on which component was actually given focus (out of scope issue)
Also stopped some maintenance code from being placed on the undo stack (reset)

It would be great if @jrieken can help take a look at the change in how applyEdits is changed.

@jrieken - any feedback on the changes here?


Hue Gradient LEDStrip (per zone control/ambilight)

Feature request

What problem does this feature solve?
Hue Gradient LEDStrip can't be controled to feature more then 1 color at the time.
Since a couple of days it's possible in the APP to controle 3 of the 7 zones on a Hue Gradient LEDStrip.
Within the API it's possible to address all 7 zones. These zones are in fixed places within the strip size (55" / 65" / 75")
What does the proposed API look like?
REST Api
https://developers.meethue.com/develop/get-started-2/
How should this be implemented in your opinion?
When setting up a light source that is Hue and one starts the entertainment control, there should be a check
for the gradient LEDstrip. When detected the size should be confirmed by the user.
Are you willing to work on this yourself?
Absolutely! :D
Although my experience in developing is zero...

I'd love this feature too! The Hue HDMI Sync Box left a bad taste in my mouth, but hyperion has been simple and amazing!
Except for the one issue that my Hue gradient lightstrip will on do single colors. Would truly love this implemented!

++ from me,
this would make it perfect for me as my LG CX WebOS just got rooted and PicCap sends the image successfuly to Hyperion.
But I can only use the gradient as a single bulb yet

Hello; is this feature possible to impement? I'd love to help but I would like to know if it's theoretically possible :)

Hello; is this feature possible to impement? I'd love to help but I would like to know if it's theoretically possible :)

this is doable yes

Just gonna add a +1 from me 👍 If this could be implemented I would be very happy :)

I just built a container with https://github.com/jdmg94/Hue-Sync
and output via file from hyperion to it.
hyperion now works like a charm with the hue gradient strip, no sync box needed

Cool, but.. can you integrate it to Hyperion?

I just built a container with https://github.com/jdmg94/Hue-Sync and output via file from hyperion to it.

@gotschi Anyway you could provide a simple tutorial on what you've done there?
I've got your same setup (Hue Gradient, rooted LG OLED, PicCap, Hyperion, and a pi) but am limited to following instructions and tinkering. I don't really know where to start to follow your approach.

Yes a tutorial will be good 👍

@gotschi Anyway you could provide a simple tutorial on what you've done there?

This is the script:
import HueSync from "hue-sync";
const fs = require('fs')
const path = require('path');

(async function () {
	const bridge = await HueSync.discover();

        // read fifo
	const fd = fs.openSync('/tmp/fifo', 'r+')
	const stream = fs.createReadStream(null, {fd})

        // login
	const credentials = {username:'username',clientkey:'clientkey'}
	const hueBridge = new HueSync({ credentials, id: 'id', url: 'ip' });

        // select entertainment area
	const selectedArea = await hueBridge.getEntertainmentAreas();
	await hueBridge.start(selectedArea[x]);

	stream.on('data',async (d) => {
                 // read matrix to array
		var data = d.toString().replace(/}/g, ',').replace(/[^0-9&,]/g,'').split(",");
		var data = data.map(function (x) {
			return parseInt(x, 10);
		});

                 // send colors
		await hueBridge.transition([[data[0],data[1],data[2]],[data[3],data[4],data[5]],[data[6],data[7],data[8]],[data[9],data[10],data[11]],[data[12],data[13],data[14]],[data[15],data[16],data[17]],[data[18],data[19],data[20]]]);
	})

})()

You generally need npm/hue-sync for this (I would not recommend trying this on your tv, use a raspberry pi or something), I used a docker container:
FROM node:alpine
COPY app /app
WORKDIR /app
RUN npm install hue-sync
RUN npm install -g ts-node
CMD ['TS_NODE_FILES=true', 'TS_NODE_TRANSPILE_ONLY=true', 'NODE_TLS_REJECT_UNAUTHORIZED=0', 'ts-node', 'index.ts']

then you just need to modify the script to get the user/token from your hub:
https://github.com/jdmg94/Hue-Sync#usage
and use these in my script afterwards.
use this layout in hyperion: https://imgur.com/a/59gzAi8
make hyperion output to a fifo (mkfifo huesync) with the file LED Output controller and share this fifo with the huesync container like:
version: '3'
services:
  hyperion:
    network_mode: host
    container_name: hyperion
    image: registry.gitlab.com/gotschi/hyperion
    volumes:
      - ./config:/root/.hyperion:rw
      - ./fifo:/tmp/fifo

  huesync:
    container_name: huesync
    image: huesync
    volumes:
      - ./fifo:/tmp/fifo

maybe I will make a standalone container image out of it when I get time

I just built a container with https://github.com/jdmg94/Hue-Sync and output via file from hyperion to it.

@gotschi Anyway you could provide a simple tutorial on what you've done there?
I've got your same setup (Hue Gradient, rooted LG OLED, PicCap, Hyperion, and a pi) but am limited to following instructions and tinkering. I don't really know where to start to follow your approach.

finished the container,
use
docker run --network host -v /home/youruser/huesync/config_huesync:/app/config -ti registry.gitlab.com/gotschi/hue-sync
then do a mkfifo fifo in the folder
and follow the instructions.
then use the docker-compose I use from above with the persistent config directory you created:
version: '3'
services:
  hyperion:
    network_mode: host
    container_name: hyperion
    image: hyperion
    volumes:
      - ./config:/root/.hyperion:rw
      - ./fifo:/tmp/fifo

  huesync:
    container_name: huesync
    image: huesync
    volumes:
      - ./config_huesync:/app/config
      - ./fifo:/tmp/fifo

lmk if it works for you

Bumping into an error locating the fifo file.
hue-sync   | Error: ENOENT: no such file or directory, open '/tmp/fifo'
hue-sync   |     at Object.openSync (node:fs:594:3)
hue-sync   |     at /app/index.ts:13:16
hue-sync   |     at step (/app/index.ts:33:23)
hue-sync   |     at Object.next (/app/index.ts:14:53)
hue-sync   |     at fulfilled (/app/index.ts:5:58)
hue-sync   |     at processTicksAndRejections (node:internal/process/task_queues:96:5) {
hue-sync   |   errno: -2,
hue-sync   |   syscall: 'open',
hue-sync   |   code: 'ENOENT',
hue-sync   |   path: '/tmp/fifo'
hue-sync   | }
hue-sync exited with code 1

For grins, I tried changing the   - ./fifo:/tmp/fifo  line in the compose file to an alternate file name (e.g., - ./fifo:/tmp/fifo1), yet I still get the same error line Error: ENOENT: no such file or directory, open '/tmp/fifo'.
I get the same error if I run just docker run --network host -v $(pwd)/huesync/config_huesync:/app/config -ti registry.gitlab.com/gotschi/hue-sync
Other Notes:

The fifo file exists and is updating from Hyperion
The username, key, ip, id, etc. seem to be working
If it matters, I'm also running Hyperion directly on the pi, outside of a docker container

Got it running, but the hue-sync container exits with code 0 after 5-10 seconds.  I can see in my Hue app the sync starts, then drops off within a few seconds after the exit.
Starting hue-sync ... done
Attaching to hue-sync
hue-sync   | {
hue-sync   |   username: 'REMOVED',
hue-sync   |   clientkey: 'REMOVED',
hue-sync   |   area: '0',
hue-sync   |   ip: 'REMOVED',
hue-sync   |   id: 'REMOVED'
hue-sync   | }
hue-sync   | (node:8) Warning: Setting the NODE_TLS_REJECT_UNAUTHORIZED environment variable to '0' makes TLS connections and HTTPS requests insecure by disabling certificate verification.
hue-sync   | (Use `node --trace-warnings ...` to show where the warning was created)
hue-sync exited with code 0

Followed the layout you mentioned above (https://imgur.com/a/59gzAi8).  Any other critical settings for the file output?
For me, Hyperion is running on the pi, outside of a docker container.
Hyperion seems to write to the file just fine.

hyperion running bare metal and this running in docker should work just fine.
just make sure to map your hyperion output file (wherever it is stored on your rpi) to /tmp/fifo inside the container.
e.g. if your ooutput file is: /home/clayton/hyperion/fifo
you have to start the container with:
```docker run -v /home/clayton/hyperion/fifo:/tmp/fifo  -v $(pwd)/huesync/config_huesync:/app/config -ti registry.gitlab.com/gotschi/hue-sync ````
(the first -v maps the fifo, the second provides the config to huesync)

It works!  I had some permissions issues with the fifo, but when I moved the fifo file to the /tmp folder, chmod'ed 777 the fifo file, things lit right up!
FYI, my docker command was
docker run -v /tmp/fifo:/tmp/fifo -v /home/pi/huesync/config_huesync:/app/config registry.gitlab.com/gotschi/hue-sync
My hyperion setup is

Thanks @gotschi !

Ich würde mir das mit dem dauerhaften /tmp/fifo schreiben noch mal überlegen. 😸
sd karte viele schreibzugriffe
Solltet auf eine RamDisk umsteigen. Oder besser noch, schreibt ein neues Hyperion LEDDevice.
JM2C

you can easily use a tmpfs in docker or create a ramdisk on the host for this, yes.
as mi pi is pxe booted I don‘t really care about sd cards anymore

Just to be sure, can I run this directly on Home Assistant OS ?
I'm not experienced with docker.

Yes, home assistant OS runs as a whole on docker like this and you can get this running on OS but you will enter unsupported territory by using this unless you make a addon out of it.
So you should not attempt to run this on HAOS without docker experience.

As a status update...
I started to look into this and established the UI configuration and doing justice to the gradient config on the layout.
Now, it requires to be supported by the backend which is a bigger rework, as gradient comes with API v2 need,
Here some screenshots, in case you already have feedback.
Selecting the Entertainment Group and assign positions on screen

Layout generated doing justice to gradient segments and their length

just to make sure, this is how the gradient looks:
https://www.allround-pc.com/wp-content/uploads/2020/10/Philips-Hue-Play-Gradient-Lightstrip-9.jpg
so it wraps around without the bottom

I'm not sure if i can follow but I used this:
While you want to solve your Gradient problem, I am looking to solve other devices with segments too (and best that I do not need to differentiate on model-numbers).
Given that other devices have a different number of segments and layout, I would need to further think about it.

Unfortunately, I do not have a true Gradient Strip at hand (and nobody sent me one :) ).
I try to test with a simulated one , which is why I need your support and feedback...

I'm not sure if i can follow but I used this:

While you want to solve your Gradient problem, I am looking to solve other devices with segments too (and best that I do not need to differentiate on model-numbers). Given that other devices have a different number of segments and layout, I would need to further think about it.
Unfortunately, I do not have a true Gradient Strip at hand (and nobody sent me one :) ). I try to test with a simulated one , which is why I need your support and feedback...

thats what I imagine with "multiple layers" - if one wants to add lights to the gradient they would be either only on the bottom or in addition to the already lit gradient areas

That is not an issue. The picture, I share withed before already covers a scenario with two Iris devices and one gradient. Hyperion can handle the fact that multiple devices map the same screen area.

If you need any help to test things, I will gladly try my best :)

Hi there. I am not a programmer. My english is not good at technical stuff.
Is this thread about, Philips TV Ambilight+Hue does not support gradient function of Hue Play Gradient Light Tube and Gradiend Light Strip? The gradient light works as a regular one light. Because Philips TV identifes the gradient tube and strip as a bulb. Thanks in advance.

@gotschi Hi,
thanks for your guide, however i've no skills in programming, i can just do ctrl-C ctrl-V in terminal :-P
I have a rooted Lg tv with piccap and hyperion.ng, then i have a philips hue gradient and a spare raspberry pi 3 with raspbian 64bit.
First of all: the 1st step to reproduce is to install your docker file right? I installded docker on raspberry and then used your command:
docker run --network host -v $(pwd)/huesync/config_huesync:/app/config -ti registry.gitlab.com/gotschi/hue-sync
but i have an error:
Error: Cannot find module 'cross-fetch/polyfill'
Require stack:

/app/node_modules/hue-sync/build/hue.js
/app/node_modules/hue-sync/build/index.js
/app/setup.ts
at Function.Module._resolveFilename (node:internal/modules/cjs/loader:956:15)
at Function.Module._resolveFilename.sharedData.moduleResolveFilenameHook.installedValue [as _resolveFilename] (/usr/local/lib/node_modules/ts-node/node_modules/@cspotcode/source-map-support/source-map-support.js:811:30)
at Function.Module._load (node:internal/modules/cjs/loader:804:27)
at Module.require (node:internal/modules/cjs/loader:1028:19)
at require (node:internal/modules/cjs/helpers:102:18)
at Object. (/app/node_modules/hue-sync/build/hue.js:1:308)
at Module._compile (node:internal/modules/cjs/loader:1126:14)
at Module._extensions..js (node:internal/modules/cjs/loader:1180:10)
at Object.require.extensions. [as .js] (/usr/local/lib/node_modules/ts-node/src/index.ts:1608:43)
at Module.load (node:internal/modules/cjs/loader:1004:32) {
code: 'MODULE_NOT_FOUND',
requireStack: [
'/app/node_modules/hue-sync/build/hue.js',
'/app/node_modules/hue-sync/build/index.js',
'/app/setup.ts'
]
}

I'm stucked here. If you could please be more accurate as you can for a noob like me for all the steps (and prerequisites)  i'll appriciate, i'd love to replicate and use the gradient light with 7 zones.
Thank you very much and sorry for my english

any news ?

I have some building block working, but have not have enough time packaging all up.
I need to find some time focussing on this...

any news ?

hyper-hdr has it integrated

Any updates on this matter? When play gradient lightstrip is used as source, the hardware led count is static to 1.

Unfortunately, the development took longer than expected (due to some long term illness and other timely constraints)....
Nevertheless, there is now a working version ready for you to test and provide further feedback.
See the description of the PR1637 for details what was added , changed and removed.
You should be able to do testing in a quite simple way on Linux due to a new script....
In case there is interest, I can describe how to test on windows without effecting the current installation, too.
Attention: Its is always recommended to export Hyperion's configuration or do backup of the configuration database before start testing a newer version. The script install a version next to your installation, but just in case...
Testing

As you are already a GitHub user ,create a fine grained personal access token for your user.
Go to https://github.com/settings/tokens?type=beta and do a "Generate new token":
a) Give the token a name
b) Select "All Repositories"
c) Under Permissions/Repository permissions
Define Read-Only Access to "Actions"
d) Generate token

Copy the new generated token

Log on to your Linux system, e.g. ssh into the HyperBian rpi

Execute the install_pr script replacing the token with your token copied in 2.
wget -qO- https://raw.githubusercontent.com/hyperion-project/hyperion.ng/master/bin/scripts/install_pr.sh | bash -s -- -t token 1637

run the execution script to start Hyperion under the current user
~/hyperion_pr1637/1637.sh

Configure your Hue device via the updated wizard and test the new API environment, streaming etc.

Remove  installed version for the given PR

Note:
The script will stop the current hyperion service before execution.
After doing your tests, you should start the service again or just reboot to start the service automatically.
Replace USER with the username, Hyperion runs with.
systemctl start<EMAIL_ADDRESS>Note 2:
In case you would like to test with a current configuration, provide the configuration location via the "-c" option.
The configuration will be copied to avoid any side effects on your current configuration,
i.e. any updates to the configuration when testing the PR will also not be reflected in the current setup.
Sample
Current user
wget -qO- https://raw.githubusercontent.com/hyperion-project/hyperion.ng/master/bin/scripts/install_pr.sh | bash -s -- -t token -c ~/.hyperion 1637

Let me try it this Wednesday :)
I'll have everything sorted then

Some feedback from a Windows installation of the 2.0.16-beta.1+PR1637
The Hue Wizard CSS seems broken.
On 1920x1080 resolution I dont see the port option until I zoom to 175% in Chrome:
100% Zoom:

175% Zoom:

Also the option to create a user is gone on beta, it was showing in stable like this:

Hi @sorano, just to prove that I am the one that accidently chatted to you on Discord and my account was not compromised.


Disables SELinux for all platforms.

/cc @acaiafa @jerenkrantz

Please fix the build

/cc @acaiafa


Illegal syntax in an active pattern produces an internal compiler error

Description: A specific kind of incorrect syntax in an active pattern definition causes an internal compiler error rather than a message specific to the error.
Repro Steps: Add the following code:
open System.Text.RegularExpressions

let (|USZipPlus4Code|_|) s =
    let m = Regex.Match(s, @"^(\d{5})\-(\d{4})$")
    if m.Success then
        USZipPlus4Code(x=m.Groups.[1].Value,
                       y=m.Groups.[2].Value)
        |> Some
    else
        None

Expected: An error along the lines of:
'Active pattern bindings do not support field names.'
Actual:
'error FS0073: internal error: impossible (Failure)'
Severity: minor.
Version: F# 4.5
Workaround: Use correct active pattern syntax. In this case, omit the x= and y=.

From F# 4.0:
exit status 1

/home/runner/main.fs(6,9): error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

/home/runner/main.fs(6,9): error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

/home/runner/main.fs(5,5): error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

/home/runner/main.fs(4,5): error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

/home/runner/main.fs(4,5): error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

/home/runner/main.fs(3,1): error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

/home/runner/main.fs(1,1): error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

/home/runner/main.fs(1,1): error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

error FS0193: internal error: The index was outside the range of elements in the list.
Parameter name: index

error FS0073: internal error: The index was outside the range of elements in the list.
Parameter name: index (ArgumentException)

error FS0193: internal error: The index was outside the range of elements in the list.
Parameter name: index

Unhandled Exception:
System.ArgumentException: The index was outside the range of elements in the list.
Parameter name: index
  at Microsoft.FSharp.Compiler.ErrorLogger+ErrorLoggerExtensions.ReraiseIfWatsonable (System.Exception exn) <0x40ad6fe0 + 0x00173> in <filename unknown>:0
  at Microsoft.FSharp.Compiler.ErrorLogger+ErrorLoggerExtensions.ErrorLogger.ErrorRecovery (Microsoft.FSharp.Compiler.ErrorLogger x, System.Exception exn, range m) <0x40ac22e0 + 0x002e7> in <filename unknown>:0
[ERROR] FATAL UNHANDLED EXCEPTION: System.ArgumentException: The index was outside the range of elements in the list.
Parameter name: index
  at Microsoft.FSharp.Compiler.ErrorLogger+ErrorLoggerExtensions.ReraiseIfWatsonable (System.Exception exn) <0x40ad6fe0 + 0x00173> in <filename unknown>:0
  at Microsoft.FSharp.Compiler.ErrorLogger+ErrorLoggerExtensions.ErrorLogger.ErrorRecovery (Microsoft.FSharp.Compiler.ErrorLogger x, System.Exception exn, range m) <0x40ac22e0 + 0x002e7> in <filename unknown>:0

So the error message is different, but it's still an internal error.

would this make more sense? it says UnionCase USZipPlus4Code doesn't have field y.

I assume we should replace the "UnionCase" with "ActivePattern". But would this be helpful?


Crashes if gnome-settings-daemon is not running

If gnome-settings-daemon is not running, fluxgui crashes with the following:
$ fluxgui
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/fluxgui/fluxgui.py", line 365, in <module>
    app = Fluxgui()
  File "/usr/lib/pymodules/python2.6/fluxgui/fluxgui.py", line 17, in __init__
    self.indicator = Indicator(self)
  File "/usr/lib/pymodules/python2.6/fluxgui/fluxgui.py", line 134, in __init__
    self.setup_indicator()
  File "/usr/lib/pymodules/python2.6/fluxgui/fluxgui.py", line 145, in setup_indicator
    'gtk-icon-theme-name')
ValueError: could not find setting

This can be a problem in KDE, which obviously doesn't run gnome-settings-daemon by default.

The git version still doesn't seem to be working for me, am I doing something wrong here?
f.lux-indicator-applet$ fluxgui
Traceback (most recent call last):
File "/usr/lib/pymodules/python2.7/fluxgui/fluxgui.py", line 381, in
app = Fluxgui()
File "/usr/lib/pymodules/python2.7/fluxgui/fluxgui.py", line 17, in init
self.indicator = Indicator(self)
File "/usr/lib/pymodules/python2.7/fluxgui/fluxgui.py", line 149, in init
self.setup_indicator()
File "/usr/lib/pymodules/python2.7/fluxgui/fluxgui.py", line 162, in setup_indicator
'gtk-icon-theme-name')
ValueError: could not find setting

what OS are you running? There are currently problems with running F.lux on oneiric.

Yup, same problem here:
$ fluxgui
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/fluxgui/fluxgui.py", line 381, in <module>
    app = Fluxgui()
  File "/usr/lib/pymodules/python2.7/fluxgui/fluxgui.py", line 17, in __init__
    self.indicator = Indicator(self)
  File "/usr/lib/pymodules/python2.7/fluxgui/fluxgui.py", line 149, in __init__
    self.setup_indicator()
  File "/usr/lib/pymodules/python2.7/fluxgui/fluxgui.py", line 162, in setup_indicator
    'gtk-icon-theme-name')
ValueError: could not find setting

system:
$ uname -mr
3.0.0-14-generic x86_64 Kubuntu 11.10

$ dpkg -s kdm | grep Version
Version: 4:4.7.3a-0ubuntu0.1

fluxgui:
$ dpkg -s fluxgui | grep Version
Version: 1.1.8

python:
$ dpkg -s python | grep Version
Version: 2.7.2-7ubuntu2

after every run to be able to rerun I have to do:
rm -f /tmp/fluxgui_*.pid

# you may replace * with your username or:
rm -f /tmp/fluxgui_`whoami`.pid # this should work for current user

redshift seems to have no gnome dependence, so it works nicely if you are on something like kde.

I tried redshift too, and it requires gnome geoclue, which seems even more difficult to fix than f.lux gtk-icon-theme-name problem. I was hoping someone will sit and try to fix it in finite time, but I guess I'm going to inspect it myself soon :)

You can input the location data manually in the command prompt, thereby bypassing the need for geoclue. So run the command like: redshift -l XX:-XX -m vidmode

I was having the same problem and found that the workaround from @sricks did the trick!
I'm on Linux Mint 17 Qiana (Ubuntu 14.04.1) running the KDE desktop

Thanks for nice fix @sricks  :thumbsup:


Support to run locally

Hi Aarya Chopkar,
My name is Daniel, and I'm trying to run the application locally on my machine. However, after starting the server, I only see a blank page instead of the application.
Could you please provide some instructions or guidance on how to properly run the application on localhost:8080? Any help would be greatly appreciated.
Thank you in advance for your assistance!
Best regards,
Daniel Casgrande

I realized I forgot to run the client side of the application. After starting both the server and the client, everything is working perfectly. Thanks for the assistance!
Steps I followed to get it running:
Clone the repository:
sh
Copiar código
git clone https://github.com/accodes21/tshirt3d
cd tshirt3d
Server setup:
Navigate to the server directory:
sh
Copiar código
cd server
Install server dependencies:
sh
Copiar código
npm install
Start the server:
sh
Copiar código
npm start
Client setup:
Navigate to the client directory:
sh
Copiar código
cd ../client
Install client dependencies:
sh
Copiar código
npm install
Start the client:
sh
Copiar código
npm run dev
Make sure to start the server before running the client. Once both are running, you should see the application working in your browser.
Thanks again for the help!


(storage): impl some function

added some implementation for todo! function in mock.rs in storage

some confusion here, going to re open a pr


[sitecore-jss-react] [template/nextjs-sxa] Use FEAAS.ExternalComponent for BYOCWrapper

In order to reduce the responsibilities of BYOC wrapper and to ensure extensibility across apps, the BYOCRenderer which is now BYOCCompoent uses <FEAAS.ExternalComponent  /> wrapper from @sitecore-feaas/clientside 0.3.12 to render the external registered component.

Re-named BYOCRenderer to BYOCComponent since rendering of registered component is now handled by FEAAS.ExternalComponent.
The FEAAS.ExternalComponent  takes a fallback prop which would now handle the missing component error cases.

Refactored BYOCComponent from a class based component to a functional component.

[x] This PR follows the Contribution Guide

[x] Changelog updated

Description / Motivation

Testing Details

[x] Unit Test Added
[ ] Manual Test/Other (Please elaborate)

Types of changes

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

Re-named BYOCRenderer to BYOCComponent since rendering of registered component is now handled by FEAAS.ExternalComponent.

This is not needed -  and is actually harmful since DevEx would need to modify their rednering for this in Sitecore once again.

Re-named BYOCRenderer to BYOCComponent since rendering of registered component is now handled by FEAAS.ExternalComponent.

This is not needed - and is actually harmful since DevEx would need to modify their rendering for this in Sitecore once again.

@art-alexeyenko This is true but might be worth doing if we can before release (the name makes more sense and is in line with FEAASComponent/Wrapper).


Refactor unit tests to use libvmi for resource creation in softreboot

What this PR does
Before this PR:

Unit tests directly created and modified VM and VMI resources by manipulating their fields.
After this PR:
Unit tests have been refactored to use the libvmi package for creating and modifying VM and VMI resources, promoting readability and standardization.

Partially address: https://github.com/kubevirt/kubevirt/issues/12059
Why we need it and why it was done in this way

Using the libvmi package simplifies the unit tests and makes them more readable.
This approach ensures a standard way of creating VM and VMI resources across tests.

The following tradeoffs were made:

Direct manipulation of resources was replaced with utility functions, which may introduce minor overhead but enhances readability and maintainability.

The following alternatives were considered:

Continuing with direct manipulation of resources in tests, which was deemed less maintainable.
Links to places where the discussion took place: Issue discussion: #12059

Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[x] PR: The PR description is expressive enough and will help future contributors
[x] Code: Write code that humans can understand and Keep it simple
[x] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/cc

/approve


cmd/exec: adding CLONE_NEWPID to Unshareflags breaks execs

What version of Go are you using (go version)?
any version up to and including Go 1.8. It's not necessarily a Go issue, it's the sometimes
non-intuitive behavior or NEWPID namespaces with unshare(2).
What operating system and processor architecture are you using (go env)?
Ubuntu 16.06 LTS; LInux 4.4.0; amd64
What did you do?
Here's a simple program, which when run as root on Linux, will fail in a surprising way. I've left out error checking for space reasons. You can't test this on the playground so I put it here.
package main
import (
"os"
"os/exec"
"syscall"
)
func main() {
c := exec.Command(os.Args[1], os.Args[2:]...)
c.Stdin, c.Stdout, c.Stderr = os.Stdin, os.Stdout, os.Stderr
c.SysProcAttr = &syscall.SysProcAttr{Unshareflags: syscall.CLONE_NEWPID}
c.Run()
}
Here's what happens when you run it with itself as the first thing to run. I'm running it with itself as the first argument to show the issues with starting a Go program in a CLONE_NEWPID namespace.
If you simply run /bin/bash as the first argument it works fine. But what if, as in u-root, we have an init process written in Go and we wish to test it via private pid namespaces?  We can't.
rminnich@akaros:~/tmp/unsharefail$ sudo ./unsharefail  ./unsharefail /bin/bash
runtime: failed to create new OS thread (have 2 already; errno=22)
fatal error: newosproc
runtime stack:
runtime.throw(0x4be7ae, 0x9)
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/panic.go:596 +0x95
runtime.newosproc(0xc420026000, 0xc420036000)
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/os_linux.go:163 +0x185
runtime.newm(0x4c4b70, 0x0)
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/proc.go:1614 +0x12d
runtime.main.func1()
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/proc.go:126 +0x36
runtime.systemstack(0x53bb00)
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/asm_amd64.s:332 +0x79
runtime.mstart()
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/proc.go:1115
goroutine 1 [running]:
runtime.systemstack_switch()
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/asm_amd64.s:286 fp=0xc420022788 sp=0xc420022780
runtime.main()
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/proc.go:125 +0x6c fp=0xc4200227e0 sp=0xc420022788
runtime.goexit()
/home/rminnich/projects/u-root/go1.6dynamic/src/runtime/asm_amd64.s:2152 +0x1 fp=0xc4200227e8 sp=0xc4200227e0
rminnich@akaros:~/tmp/unsharefail$
Script done on Sat 25 Mar 2017 12:06:26 PM PDT
What did you expect to see?
A bash prompt.
What did you see instead?
A failure in the Go runtime. The reason is the rather strange rules of CLONE_NEWPID in unshare: the child only gets one fork, so you generally immediately need to do another one; subsequent ones return ENOMEM (true!). Since the Go runtime forks several times at startup, it fails, after the first fork. It would probably be best, when CLONE_NEWPID is set in Unshareflags, to instead move it to cloneflags.
A further problem is the behavior of proc. The intent of a NEWPID namespace is that the spawned process have PID 1. But it does not until /proc is remounted. The weirdness here is that mounts of /proc were always idempotent. But in a CLONE_NEWPID namespace, the first mount of /proc after unshare results in a proc with the private PIDs.
More fun is that if you do the mount of /proc in the CLONE_NEWPID namespace, without having done a CLONE_NEWNS, you'll break /proc, and hence ps and many other things, outside the NEWPID namespace.
You can try this yourself, but be sure you have another window with su running to see the effects or you'll need to reboot on some distros, as you can't even su when proc goes wrong:
window 1:
sudo unshare -p /bin/bash
/bin/bash # so you can run more than one more command
ps
mount -t proc proc /proc
ps
In the other window:
ps # note it doesn't work out well
in window 1:
exit
in the other window, depending on your distro, there's no proc mounted any more. Fun!
Hence, we probably, when faced with CLONE_NEWPID in unshare flags, add CLONE_NEWNS to the flags, and while we're at it just go ahead and mount /proc for the child, so that the NEWPID namespace functions in the way it was intended to.
I'm working on a fix.

I made a patch to update the Unshareflags docs to define this behavior.

@jessfraz - I see that the CL was abandoned with the reason as "unneeded". Should this issue be closed too ?

From @bradfitz on the CL:

In that case, let's drop this. Not only is it time consuming (and an uphill battle) to mirror the Linux manpages into the Go docs, but it detracts from Go-specific notes we want to document.

Closing in favor of this.


Indexer

Is your feature request related to a problem? Please describe.
We need a indexer which indexes the documents stored in the list data structure.
Think of search bar in the email application.
Describe the solution you'd like
A trie/skip list for every indexed field should be built to help search.

KV PR has indexer too


Adds support for tailLines in get_pod_log

Why is this PR needed
From https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.10/#read-log:

If set, the number of lines from the end of the logs to show. If not specified, logs are shown from the creation of the container or sinceSeconds or sinceTime

This will allow us to fetch only the tail of the log.

I do not think the pipeline failures are related to these changes.

Travis error for ruby 2.5 LoadError: cannot load such file -- bundler/dep_proxy was known bundler issue.
Travis now fixed this by upgrading their rubygems: https://github.com/travis-ci/travis-ci/issues/8969#issuecomment-390330259 , I restarted the 2.5 builds and they passed.

@cben I see no evidence either on the API or by testing it on my local env that they are exclusive. And it does not make much sense that they are exclusive because this scenario should be possible:
"Retrieve x log lines since TIME"
Either way I tested it on my local environment and everything worked accordingly.

Ah cool, I got confused with "Only one of sinceSeconds or sinceTime may be specified" (we don't do sinceSeconds presently).


Causes build failure when used as nested addon

I have an addon, say my-addon which has ember-simple-auth as a dependency. When I try running an app that uses my-addon, I get an error message from the ember-cli build:
version: 2.4.2
app.import is not a function
TypeError: app.import is not a function
    at Class.module.exports.included (C:\Projects\sc-pp\pixelpipe-show-control-player\node_modules\ember-simple-auth\index.js:15:15)

This appears to be related to ember-cli/ember-cli#5747 - I noticed there's already a check for one level of "nestedness" in index.js - perhaps that could be replaced with a loop to account for deeper nesting.

This should be fixed in 1.2.0 by the use of ember-cli-import-polyfill - see #1162


Host can toggle grants on or off at Collective and Project level

OCF has recently launched its grantmaking policy, which involves holding grants in Projects and paying them out via grant expenses. We need a way to turn on and off the grant expenses option at the Project level.

I suggest this setting could be added on the 'hosted collectives' tab on the dashboard, if we're ok adding Projects there, or maybe in the Project's action menu, for hosts only?

I think this setting should be independent of the parent-child relationship, i..e just because a Collective has grants on doesn't mean its Projects should, and vice versa. For OCF it's important that only host-approved Projects have grant expenses active.

In an ideal world we'd also be able to turn off the other expense types but we can work with it if that's not possible.

We know that hosts sometimes want to edit certain settings for their collectives. Having a modal for that here could be nice.
The second button already has a "Settings" icon, so maybe we should just upgrade the "Host fee structure" modal to be a "Account settings" modal and add it there? @opencollective/design wdyt?

Yeah that sounds fine - any design that allows host admins to change the setting will work. If it's hard to design, put it in root admin dashboard for now.

@SudharakaP what is the status for this issue?

@SudharakaP what is the status for this issue?

Was discussing about deploying this issue with Benjamin yesterday and how to make sure we enable it for the two collectives. It's clarified now and will deploy it today. 😃


[Wishlist] Mounting the Storage

It would be nice, if i could mount the storage, instead of an ISO-file.
MTP is pretty slow.

YES @MartinX3 , some Cyanogenmod versions ago it was possible but unstable to mount your phone storage as "mass storage device" for writing and reading.
The MTP support under LINUX is a very unstable and writing to the phone storage is not possible, only reading seems to be possible.
Therefore mounting the complete phone storage directly would be very pleasant.

MTP is stable in Linux Mint 18.1, but slow.
The advantage is, that you can use the SDCard, while mounting on your pc.
But i can live with a device on standby, while mounting the SDCard.

This is impossible to do without shutting Android down entirely, I can't do anything about it. SD cards are feasible but I don't have a device with an SD slot so I can't test it.

But the old sdcard mode worked very fine, even on phones without a sdcard.

That has nothing to do with file systems. Android uses ext4 on most devices, f2fs on some, and they are free to use for anything.
Old devices handled mounting to the PC by having a separate partition that was treated like an SD card, and unmounting it as necessary. Newer devices have a unified internal storage partition to which all your applications are installed, and all other settings Android relies on to run. The only way to unmount it is to stop the framework and take over, which is a huge undertaking and out of scope for this application.

Thank you


Users/vivasa/symboltask/node10warning

Task name:  SymbolPublishTask
Description: Adding warning to stop supporting node10 task till 07/31/2024
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) N
Attached related issue: (Y/N) N
Checklist:

[x] Task version was bumped - please check [instruction](https://github.com/microsoft/azure-pipelines tasks/tree/master/docs/taskversionbumping.md) how to do it
[x] Checked that applied changes work as expected -- Done

Please rebuild the task locally as some changes are not included in _generated folder(e.g. README file and changes related to console.log(nodeVersion) )

Please rebuild the task locally as some changes are not included in _generated folder(e.g. README file and changes related to console.log(nodeVersion) )

Files updated

/azp run


Question about normalization calculations

Hello,
First of all I really enjoyed reading your paper. It was an excellent analysis that gave me a lot of insight into the various tools. I've been doing a similar analysis so I was interested in looking at your evaluation functions. I'm trying to figure out what this line of code is doing though:
normFacts <- paste("NF", normFacts, sep = ".")  NFs = get_variable(physeq, normFacts)     NFs = NFs/exp(mean(log(NFs)))
I understand that you are getting the normalization factors from the various methods (TMM, etc.) using the calcNormFactors, however, I don't understand why these factors are then further changed in the last part the of code! I haven't seen this done in other edgeR tutorials so any guidance would be really appreciated.
Thanks, Jacob Nearing

Hi Jacob,
I'm glad to know you enjoyed the reading and I thank you for the kind words
of appreciation!
You have a sharp eye 😉, I took that specific code from a great work of
Hawinkel et al. (code:
https://users.ugent.be/~shawinke/ABrokenPromise/03_diffAbundDetect.html).
Indeed, they established a really inspiring benchmarking framework for
differential abundance detection methods, that's why I decided to start
from there instead of starting from scratch. The exp(mean(log(NFs))) is the
geometric mean of the normalization factors. The results are not affected
by this further change (because it is a simple "multiplication by a
constant") however the authors suggest to use it in order to facilitate
fitting (It is mentioned as a comment under the function negBinTestDESeq2).
I have not verified this last statement by stressing the normalization
process with and without that further change, I simply kept it.
Don't hesitate to contact me if you have any doubts or other questions.
Thank you for your question.
Best regards,
Matteo Calgaro
PhD Student,
Dept. of Biotechnologies,
University of Verona
Il giorno mer 2 set 2020 alle ore 20:38 nearinj<EMAIL_ADDRESS>ha scritto:

Hello,
First of all I really enjoyed reading your paper. It was an excellent
analysis that gave me a lot of insight into the various tools. I've been
doing a similar analysis so I was interested in looking at your evaluation
functions. I'm trying to figure out what this line of code is doing though:
normFacts <- paste("NF", normFacts, sep = ".") NFs = get_variable(physeq,
normFacts) NFs = NFs/exp(mean(log(NFs)))
I understand that you are getting the normalization factors from the
various methods (TMM, etc.) using the calcNormFactors, however, I don't
understand why these factors are then further changed in the last part the
of code! I haven't seen this done in other edgeR tutorials so any guidance
would be really appreciated.
Thanks, Jacob Nearing
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mcalgaro93/sc2meta/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKLV5ZWJZE3NL4QSRD4HSDDSD2GKHANCNFSM4QTQOBXA
.

Hi Jacob,
I'm glad to know you enjoyed the reading and I thank you for the kind words
of appreciation!
You have a sharp eye 😉, I took that specific code from a great work of
Hawinkel et al. (code:
https://users.ugent.be/~shawinke/ABrokenPromise/03_diffAbundDetect.html).
Indeed, they established a really inspiring benchmarking framework for
differential abundance detection methods, that's why I decided to start
from there instead of starting from scratch. The exp(mean(log(NFs))) is the
geometric mean of the normalization factors. The results are not affected
by this further change (because it is a simple "multiplication by a
constant") however the authors suggest to use it in order to facilitate
fitting (it is mentioned as a comment under the function negBinTestDESeq2).
I have not verified this last statement by stressing the normalization
process with and without that further change, I simply kept it.
Don't hesitate to contact me if you have any doubts or other questions.
Thank you for your question.
Best regards,
Matteo Calgaro

Thanks for getting back to me so quick. I'll take a look at the link above. Something I have notice while testing out limma_voom_TMM is that the normalization step (calcNormFactors) doesn't seem to be deterministic when the sampling order is shuffled.
Me and a colleague have noticed that if you take a count table and shuffle the column order the normalization factors can be drastically different for the same sample. Furthermore, we noticed that this can have an impact on the downstream number of significant hits found be the tool. We have tested this on two different 16S datasets and received similar results. Though we are not sure if this is due to the sparsity/distrubition of 16S data.
Below is a boxplot showing the correlation between the normalization factors calculated from 100 count tables with randomly shuffled column orders. The first shows the correlation between the 100 shuffles and the original order and the second shows the correlation between all 100 random shuffles. Overall you can see that while there is a moderate correlation these differences do seem to impact the downstream results.

Have you come across this before?

That's quite strange. I tried to build a toy example which compares shuffled results. I didn't notice such behaviours. Have you shuffled both the otu_table and sam_data slots of the phyloseq object?
# Load functions
source("./eval_functions.R")
# Load data
load("./data/16Sdatasets_for_replicability_filtered.RData")
# Example dataset
ps <- ps_list_16S$Subgingival_Supragingival

# Relabelling the comparison variable
ps@sam_data$grp <- factor(ps@sam_data$HMP_BODY_SUBSITE,
                          levels = c("Subgingival Plaque","Supragingival Plaque"),
                          labels = c("grp1","grp2"))

# edgeR TMM normalization on unshuffled data
physeq_unshuffled <- normEdgeR(ps,"TMM")

# Random shuffling
set.seed(123)
shuffle <- sample(1:nsamples(ps),size = nsamples(ps),replace = FALSE)
# Create the shuffled phyloseq object
ps_shuffled <- ps
# Remember to shuffle both otu_table and sample_data slots
ps_shuffled@otu_table <- ps_shuffled@otu_table[,shuffle]
ps_shuffled@sam_data <- ps_shuffled@sam_data[shuffle,]

# edgeR TMM normalization on shuffled data
physeq_shuffled <- normEdgeR(ps_shuffled,"TMM")

# Check if the Normalization Factors are equal
head(data.frame("Unshuffled_NFs" =<EMAIL_ADDRESS>                "Shuffled_NFs" = physeq_shuffled@sam_data$NF.TMM[names(physeq_unshuffled@sam_data$NF.TMM)]))

# Let's try the entire pipeline

unshuffled_DA <- limma_voom(physeq_unshuffled,normFacts = "TMM")
unshuffled_DA_names <- rownames(unshuffled_DA$pValMat[unshuffled_DA$pValMat[,2]<0.05,])

shuffled_DA <- limma_voom(physeq_shuffled,normFacts = "TMM")
shuffled_DA_names <- rownames(shuffled_DA$pValMat[shuffled_DA$pValMat[,2]<0.05,])

# Count the differences
sum(shuffled_DA_names != unshuffled_DA_names)

Hi Matteo,
I spent some time troubleshooting the odd results I have been getting and think have found the issue. I tested nine separate datasets (the 3 16S datasets in this repo along with 6 previously published datasets). I have found that the three datasets in this repo do not suffer from the reproducible issue, however, other 16S datasets do.
I found that after 100 random prefumations of the column orders of these nine datasets four other them give consist normalization factors from the normedgeR function but five of them do not.

Investigating the differences in these datasets I found that the datasets that failed to give reproducible normalization factors also had the highest sparsity.

To investigate this further I took three datasets that didn't work (Biscuit, Chemerin, CRC_baxter) and filtered the ASV table based on the prevalence of features. Prevalence filtering at 10%, 5% and 3% revealed that as the matrix because less sparse the results become reproducible.

This indicates to me that TMM normalization should not preformed on matrix's that have sparsity greater than >0.78 as the normalization factors may not be reproducible. I would love to hear your thoughts on this and whether you find this in your own hands. I have included the datasets along with an rmarkdown file contain all the code for the analysis that I did.
Thanks for looking into this.
Jacob Nearing
Dalhousie University
Langille Lab

Dear Jacob,
thank you for your troubleshooting. I tried your code and now I understand your results.
In the following link you can find the original calcNormFactors code: https://rdrr.io/bioc/edgeR/src/R/calcNormFactors.R . I tried it directly on your datasets and the problem persists. As you said, it is probably due to the sparsity but if you want to specifically identify what is causing the problem I suggest you to debug the original edgeR code or ask to the edgeR authors.
I've never experienced it in my data because for differential abundance analysis I usually use the filtered count table obtained by the following function:filter_taxa(ps,function(x) sum(x>10)>1,1). In this way I am sure that all the taxa are present in more than 1 sample and with more than 10 counts. Sparsity is also reduced by doing so. I also use the filter by prevalence but usually the first filter is enough. The phyloseq objects in ./data/16Sdatasets_for_replicability_filtered.RData were already filtered, that's why the issue didn't arise.
The filtering choice is arbitrary, however you have to think about what the data are intended for. In my case, they served to make the counts comparable between samples and then a negative binomial should fit each taxon count distribution. For these reasons I needed more than 1 positive counts for each taxon (the more, the better for the model estimation). Taxa with very low values, together with a sparse presence in the samples, only bring unwanted variability. The models hardly handle those taxa and the results on them are not trustworthy. However, I don't think this is an excuse for normalization factors not to be reproducible. In the next months I will insert some checks in the code to prevent too sparse count tables from being handled. Thank you for noticing it.
I remain at your disposal for any clarification.

Hi Matteo,
If your interested in how this was resolved you can check out my post on bioconductor.
https://support.bioconductor.org/p/133921/#133922
Thanks for taking the time with this issue and best of luck in your future work.


GOBBLIN-973: Increase timeout for copying Gobblin workunits to the wo…

…rkunit state store in GobblinHelixJobLauncher
Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-973

Description

[x] Here are some details about my PR, including screenshots (if applicable):
The default value of 10 seconds used by the ParallelRunner to copy workunits may not be sufficient in cases where the destination system is slow or when the number of work units is large.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Tested with an end-to-end run to ensure no regression.

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
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"

Codecov Report

Merging #2821 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2821      +/-   ##
============================================
+ Coverage     45.63%   45.63%   +<.01%
+ Complexity     8966     8963       -3
============================================
  Files          1900     1900
  Lines         71222    71222
  Branches       7857     7857
============================================
+ Hits          32499    32500       +1
+ Misses        35734    35730       -4
- Partials       2989     2992       +3

Impacted Files
Coverage Δ
Complexity Δ

...pache/gobblin/cluster/GobblinHelixJobLauncher.java
82% <100%> (ø)
27 <0> (ø)
:arrow_down:

...a/org/apache/gobblin/util/limiter/NoopLimiter.java
40% <0%> (-20%)
2% <0%> (-1%)

...lin/util/filesystem/FileSystemInstrumentation.java
85.71% <0%> (-14.29%)
3% <0%> (-1%)

.../org/apache/gobblin/cluster/GobblinTaskRunner.java
63.42% <0%> (-1.39%)
27% <0%> (-1%)

...e/gobblin/runtime/locks/ZookeeperBasedJobLock.java
63.33% <0%> (-1.12%)
15% <0%> (-1%)

...main/java/org/apache/gobblin/util/HadoopUtils.java
30.2% <0%> (-0.68%)
24% <0%> (-1%)

.../apache/gobblin/runtime/api/JobExecutionState.java
80.37% <0%> (+0.93%)
24% <0%> (ø)
:arrow_down:

...lin/restli/throttling/ZookeeperLeaderElection.java
72.22% <0%> (+2.22%)
13% <0%> (ø)
:arrow_down:

...in/java/org/apache/gobblin/cluster/HelixUtils.java
39.25% <0%> (+6.54%)
13% <0%> (+2%)
:arrow_up:

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 e41fd94...3331f43. Read the comment docs.


Summary.Generation_total not updating

Hi Gaspode,
since the update from Alpha, the names Object isnt updating any longer.
No entry in the logfile or any other error/ issue is shown.
Best
br41n

the names Object

?
Have you tried to restart the adapter?

Restarted/ reinstalled… 😕
Summary.Generation_total
Gaspode69 @.***> schrieb am Do. 24. Okt. 2024 um 08:09:

the names Object
?
Have you tried to restart the adapter?
—
Reply to this email directly, view it on GitHub
https://github.com/Gaspode69/ioBroker.alpha-ess/issues/137#issuecomment-2434381096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMIXOIWKULVYJQESTYN54P3Z5CFKFAVCNFSM6AAAAABQQFMJ52VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZUGM4DCMBZGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

What is the timestamp of the value? Is it really not updated at all or updated with old values?
But anyway, there are still problems with the Open API by Alpha-ESS, so please open a ticket there an describe the problem.
Use something like "For API call getSumDataForCustomer the attribute epvtotal is not delivered properly" and describe exactly what's going on (if the timestamp is actual, then you can assume that the API deliveres the same old value over and over again)
Sorry, that's all I can do.

Hey no worries, thanks for the hint. Mit freundlichen Grüßen - Yours sincerely Michael Brück Diese E-Mail samt Anlagen ist vertraulich und nur für den genannten Empfänger bestimmt. Sollten Sie diese E-Mail-Nachricht versehentlich erhalten haben, so unterrichten Sie bitte hierüber den Absender und löschen Sie bitte endgültig alle erhaltenen Informationen. Sie sind nicht befugt, unbeabsichtigt erhaltene Informationen Dritten zu offenbaren, sie zu kopieren, weiterzuleiten oder in sonstiger Weise zu verwenden.Am 24.10.2024 um 08:48 schrieb Gaspode69 @.***>:
What is the timestamp of the value? Is it really not updated at all or updated with old values?
But anyway, there are still problems with the Open API by Alpha-ESS, so please open a ticket there an describe the problem.
Use something like "For API call getSumDataForCustomer the attribute epvtotal is not delivered properly" and describe exactly what's going on (if the timestamp is actual, then you can assume that the API deliveres the same old value over and over again)
Sorry, that's all I can do.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

See also here


The @help macro does not show the docstrings for macros

Example: @help @info displays just "#= =#" in the pager, not the docstring for @info.
I could track the issue down to the macro help(f) definition in helpers.jl, line 24. When calling @help @info in the REPL, the variable f_str is set to something like #= REPL[21]:1 =# @info.
I tried
local f_str = chopprefix(string(f), r"#= .+ =# ")

which is not elegant but it works fine.
I could make a PR for that or maybe you have a better solution?

Hi @piechologist !
Sorry for the delay.
I took a look at the issue and I think your proposed solution is a great idea! If you could submit this PR, that would be awesome!
Thanks!

Works as intended in v0.6.0. Thanks for merging!

Thanks for the fix @piechologist !


Set UmbraPaymaster trusted forwarder & redeploy Rinkeby

Can you run prettier on the JSON files? Otherwise looks good! cc @wildmolasses for more monorepo prettier config

https://github.com/ScopeLift/umbra-protocol/blob/master/.lintstagedrc.js is the file to adjust!

Thanks Ed, just pushed a commit that should fix it!


Remove client

I've moved the client script to a separate repository to maintain modularity. This pull request removes the client script from this repo.
Server tests appear to be failing. Not sure why though.

We're doing a server rewrite, so I think we can solve the failing tests while that's going on.
LGTM. Merging now.


Test failure: pip freeze no longer protects setuptools, distribute or wheel on Python 3.12+

Some CI tests are failing due to this change in pip to no longer protect setuptools, distribute or wheel by default on Python 3.12+: https://github.com/pypa/pip/commit/5dc65eabb75f89d4f4749b6c764042c227f6870a
That has been in the pip dev branch for a while, and was published in pip 24.1 in the past few days.
This change affects the pip-tools test suite by way of the PACKAGES_TO_IGNORE value, which reads the pip freeze internal API via https://github.com/jazzband/pip-tools/blob/53309647980e2a4981db54c0033f98c61142de0b/piptools/_compat/pip_compat.py#L101
Simplest CI fix would likely be to drop those packages from the affected test cases, since their handling depends on the versions of other components.

The test_compile_recursive_extras_build_targets failures feels like it might be due to https://github.com/pypa/pip/pull/12449 (pip used to inject wheel as an implicit build dependency for every setup.py project)

The test_compile_recursive_extras_build_targets failure feels like it might be due to pypa/pip#12449 (pip used to inject wheel as an implicit build dependency for every setup.py project)

Could you remove this line, then?
https://github.com/jazzband/pip-tools/blob/53309647980e2a4981db54c0033f98c61142de0b/tests/test_cli_compile.py#L3458

Some CI tests are failing due to this change in pip to no longer protect setuptools, distribute or wheel by default on Python 3.12+: pypa/pip@5dc65ea

We don't test on 3.12+ yet, though...

The test_compile_recursive_extras_build_targets failure feels like it might be due to pypa/pip#12449 (pip used to inject wheel as an implicit build dependency for every setup.py project)

True, I've opened a PR, but there are still the failing Windows tests... Could you look into them?

We don't test on 3.12+ yet, though...

I hit this one locally when trying to reproduce the other CI failures. I didn't check if the GitHub CI was running 3.12.

The test_compile_recursive_extras_build_targets failure feels like it might be due to pypa/pip#12449 (pip used to inject wheel as an implicit build dependency for every setup.py project)

@ncoghlan no, apparently, this is because @agronholm moved wheel into setuptools during the sprints at PyCon US this spring. That failure actually led me to discovering a bug in the feature the failing test checks: https://github.com/jazzband/pip-tools/pull/1681#issuecomment-2212541289 / https://github.com/jazzband/pip-tools/pull/1681/files#r1667748889.

Fixed the issue title to reflect that this isn't a CI issue (yet)

From a windows testing point of view, I just ran tox -e piplatest locally, and the only reported failures were this (which I fixed by switching to my PR branch) and the wheel related failure that #2106 attempted to resolve.

From a windows testing point of view, I just ran tox -e piplatest locally, and the only reported failures were this (which I fixed by switching to my PR branch) and the wheel related failure that #2106 attempted to resolve.

#2106 has been merged. We need to re-evaluate if this is still a problem.


0.3.3

Making a PR instead of updating the master branch here, which will hopefully trigger the proper events

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


feat(file preview): add response from @web to file previews.

for this one: https://github.com/orgs/smallcloudai/projects/5/views/1?pane=issue&itemId=77058507

Is it still relevant? @web tool appears to be working in actual alpha


Major reward bug

Hi @yzexeter,
We found a major bug in reward fuction and reset episode.
We fix that in our last commit, We invite you to update gym_ma_toy.
Best regards,
Adil

@AdilZouitine Thank you very much for reminding me of it.


[Translation] Add more live tests

Adds more tests especially to verify error behavior (Fixes https://github.com/Azure/azure-sdk-for-net/issues/19560)
Uses UtcNow instead of Now so it doesn't use the local computer time when setting the expiration date of the SAS tokens.
Adds ClientDiagnostics implementation so when a method (outside of LRO context) throws, we can encapsulate the information in a DocumentTranslationError.
Refactoring of the tests to avoid creating input documents in multiple places

Uses UtcNow instead of Now so it doesn't use the local computer time when setting the expiration date of the SAS tokens.

How is the DateTimeOffset passed? Regardless of how it was created, we should always being writing out UTC if that is what the service expects. At the very least, the offset if not in UTC should be reflected.
That said, I hadn't started reviewing the code, but this caught my eye. It's basically dictating a user must pass in the DateTimeOffset a certain way, which could be problematic. DateTimeOffset was created, in part, to fix many of the problems of DateTime had so now the offset is always known, meaning we can and should handle it for users automatically.

/azp run net - documenttranslation - ci


support mem typed arg to store any size of memory

https://github.com/pikasTech/pikascript/commit/f23dac8a8ab8ea011d061b9cac1242c356c18035


Backward-compatible --version command line option?

Is your feature request related to a problem? Please describe.
Thanks for creating gitleaks!  It's a handy tool, for sure.  The recent command line changes do simplify the options and code quite a bit, but I'd like to propose a backward-compatible consistent way to retrieve the version number so any tooling like pre-commit hooks can be built to be version-aware in a straightforward way for projects with multiple devs.
Describe the solution you'd like
Support a --version command line option in version 8 and going forward
Describe alternatives you've considered
Something like gitleaks --help | grep -q "\-\-version" && gitleaks --version || gitleaks version works fine too, but the --version and/or -v idioms are quite common across *nix command line tools.
Thanks for your consideration and your efforts in maintaining a helpful tool!  I'll work on a PR too, but I wanted to submit the feature request to get the thoughts of the maintainers here as well.
cc @zricethezav

@joealba here ya go! https://github.com/gitleaks/gitleaks/commit/26f34692fac6e9daec13c770421b4ed990d1c321
better late than never ;)


update packages

:white_check_mark: Build TheIdServer 7.3.1-fix-dependencies1-0033 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/71e3fb994d by @invalid-email-address)


Update checks for inputs that start advertising

This is fine for some kind of quick and dirty debug mode, but this led to our team struggling trying to figure out why the console was spamming with 'Socket Exception : Only one socket use is permitted'. Please disable this feature or only enable it for testing.

@AldeRoberge Ah, yes. I will be removing them in the upcoming release.

@AldeRoberge this has been in the latest release.


Improve luacheck accuracy by moving the Expression 2 base components into E2Lib and adding some E2 globals to luacheckrc

@Divran @Nebual @AbigailBuccaneer any ramifications?

If you want to find some more globals, you can just run luacheck and scan the output for relevant warnings
From wire folder run: luacheck lua <additional parameters> | grep -o 'global variable.*' | sed -e 's/^global variable \(.*\)$/\1/' | sort | uniq

or shorter: luacheck lua <additional parameters> | sed -nr 's/^.*global variable (.*)$/\1/' | sort -u

rebased onto current master
@AbigailBuccaneer if you think it's good and think your review was thorough enough, merge it :)

@AbigailBuccaneer @TomyLobo why hasn't this been merged yet

This commit breaks E2, as some places are not updated, like inside RunString here:
https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_expression2/base/compiler.lua#L654

oops, nice spot

hnng, @Divran could you stop using "squash" already?

@TomyLobo but you told me to use squash
also I didn't merge this

oh, right, you didn't, my bad.
but, no I didn't tell you to squash this PR.
...or any PR in the last 2 years or something.
I changed my mind on the subject a while ago.
What needs to happen is that if the history we're handed is messy, we need to ask the author to clean it up before we merge.


Local build does not create artifacts in bin folder

Describe the bug
The instructions to build the module locally do not work for me in PowerShell or Windows PowerShell. Running .\build\Install-Dependencies.ps1 -ModuleName Entra installs of the module dependencies and creates the bin folder, but does not build the module files.
To Reproduce
Steps to reproduce the behavior:

Clone or download and extract the repo into the folder .\entra-powershell
cd entra-powershell
.\build\Install-Dependencies.ps1 -ModuleName Entra

Expected behavior
Build the module including its manifest and file structure. (Does this depend on a build task being configured in the .vscode folder?)
Debug Output
❯ .\build\Install-Dependencies.ps1 -ModuleName EntraBeta -Debug
❯ .\build\Install-Dependencies.ps1 -ModuleName EntraBeta -Verbose
VERBOSE: Installing Module AzureADPreview
VERBOSE: Installing Module Microsoft.Graph.Beta.DirectoryObjects
VERBOSE: Installing Module Microsoft.Graph.Beta.Users
VERBOSE: Installing Module Microsoft.Graph.Beta.Users.Actions
VERBOSE: Installing Module Microsoft.Graph.Beta.Users.Functions
VERBOSE: Installing Module Microsoft.Graph.Beta.Groups
VERBOSE: Installing Module Microsoft.Graph.Beta.Identity.DirectoryManagement
VERBOSE: Installing Module Microsoft.Graph.Beta.Identity.Governance
VERBOSE: Installing Module Microsoft.Graph.Beta.Identity.SignIns
VERBOSE: Installing Module Microsoft.Graph.Beta.Applications
VERBOSE: Installing Module Microsoft.Graph.Beta.Reports

Module Version
❯ Get-Module -Name 'Microsoft.Graph.Entra'

Name                  Version PreRelease ExportedCommands
----                  ------- ---------- ----------------
Microsoft.Graph.Entra 0.10.0             {Add-EntraAdministrativeUnitMember, Add-EntraApplicationO…

Environment Data
❯ $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.4.2
PSEdition                      Core
GitCommitId                    7.4.2
OS                             Microsoft Windows 10.0.22631
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Screenshots

@SamErde Thanks for raising the issue, we are looking into it.

Hi @SamErde,
The command .\build\Install-Dependencies.ps1 -ModuleName Entra installs the necessary dependencies but does not build the module files.
To build the module files and help files, follow these steps after running .\build\Install-Dependencies.ps1 -ModuleName Entra :

This process will generate the help files and module files. Once you import the module, you will be able to use it.
For any queries or additional information, please refer to the BUILD.md file inside the build folder.

Thanks. I should have been more clear in my description: I was following the instructions on BUILD.md and still not successful. I'll update the description to note that I ran:
cd .\Public\entra-powershell
.\build\Install-Dependencies.ps1 -ModuleName Entra
Install-Module -Name PlatyPS
. .\build\Common-functions.ps1
Create-ModuleHelp -Module Entra

The local build script works now. Thank you!

Thanks for the update!


EML parser

The problem the component solves
Mailbag needs to be able to read data from a directory of EML files to create mailbags. This parser needs to be interchangeable with other format parsers, so it should output data into an abstracted Input class
Relevant part of mailbag spec?
...
Type of component

[ ] Core
[X] Input
[ ] Attachments
[ ] Derivatives conversion
[ ] Reporting/Exporting
[ ] GUI
[ ] Distribution

Expected contribution

[X] Pull Request
[ ] Comment with proposed solution

Major challenges or things to keep in mind
Email exports can be messy. We'll need a good about of sample data and to document edge cases.

@haritgarg for the EML parser, you should be able to use the email library, particularly email.message_from_file. Also, the parser should read a directory structure and find all the emls with that folder and subfolders. You can take a look at the msg parser for an example.
Finally, you should spend some time experimenting with the email object that you get using the email library. Try to get the email headers, the content of the email, and attachments and experiment with the data types you get. We can compare that to other libraries to see what should go into the email model.

@haritgarg I merged the branch to get the parser in develop and so you can make a new branch that includes more recent updates. We'll talk more on the next call but this now needs code ported over from the other parsers.
First, the model now has 4 additional fields that hold the message content. You should be able to use code from the mbox parser and port it over.
Second, you can use the moveWithDirectoryStructure helper at the end of the parser. Code from the MSG parser should work.

Addressed with #79


No Check for File Type

There's no check on the input files for file type. Only .txt files should be able to be passed (for now). You could add a check for this in generateFromFile().

Thanks for reminding me. I have added file type validation with this commit 1edac1f98d08606c4f9ed66f473ae90e182423b2


Can you add support to sort for hasMany relation

Can you add support to sort for hasMany relation :
for example user hasMany posts
i need to sort user posts

+1

+1

+1

Hey guys! HasMany sorting has been added in version 1.4.0, though it comes with one drawback: you can only sort on either the Index view OR the HasMany view, but not both. Both would be too difficult to implement at this point. Let me know if this works for you. Good luck!


make pywps python 3.7 compatible

Description
Python 3.7 is available and it comes with some changes. For example:

W606  'async' and 'await' are reserved keywords starting with Python 3.7

INFO:conda_build.build:Packaging pywps-4.1.14-py37_0
compiling .pyc files...
  File "lib/python3.7/site-packages/pywps/app/Process.py", line 99
    self.async = False
             ^
SyntaxError: invalid syntax

Environment
Steps to Reproduce
Additional Information
See also ticket in OWSLib: https://github.com/geopython/OWSLib/issues/455

Ran into this problem today. Glad that this is already being looked into. :)

we probably need to fix owslib first:
https://github.com/geopython/OWSLib/issues/515

Same issue. Python 3.7.

we might want to change the naming to mode to preserve consistency with owslib,


Daily Data loads with active receiver settings

Problem statement
Currently, the receiver service dropdown on Daily Data populates with the first of the array received from the database. This may not always be an active setting in our database. We should ensure we are setting the dropdown with an active receiver setting for the organization.
What you need to know
Evidence of a need for this can be found in #7471
Receiver (and Sender) settings utilize a customerStatus property that is expected to be active for any active receiver settings in production.
We can't further define any "preferred" first choice if more than one setting is active, and that is considered out of scope for this.
This is where we're setting receivers[0] currently
Acceptance criteria

[ ] Ensure the setting chosen by default for the dropdown is an active setting
[ ] Update O&O on this change, and ensure they know its impact as it relates to settings changes

To do

[ ] ...

@chris-kuryak This is the change for the helpdesk ticket raised by CA -- should merge in Tuesday, Jan 17!


Compatibility with HoloViews bokeh plotting API

This PR updates the bokeh plot classes to be compatible with https://github.com/ioam/holoviews/pull/1984 in holoviews.

Going to merge this since the corresponding PR has been merged into HoloViews.


Missing Mode information

Hi, I have just started using this node and its working fine however I have one big problem as follows ... I set up a profile and initiate it by setting topic to 'setProfile' & payload to 'office' and I can see under the node that profile / mode is set to 'office' have tested it and it works great,
If I setTarget manually that overrides the profile and I can see under the node that 'manual' is set.
But I cannot find a way to get that current mode into my flow like 'office' (auto) or 'manual' (override) without this mode information I cannot tell on my dashboard if the thermostat is following the profile or if the target temperature has been set manually and the thermostat is not following the profile, is it possible? am I missing something? or if not could I suggest this fundamental requirement be added ?

Hi,
You can use the following (undocumented) topic, payload:
msg.topic: getProfile
msg.payload: actual

Output3 sends the msg.payload.name (profile name or manual).
Please, install version 0.8.2.

Perfect thanks


Implement Hash#merge! with block

In ruby world

Thanks!
Next time please use a separate "it" block for this because it's a different method.
There's also a slight thing I will fix: if the values are nil then self[key]? will return nil even though the key is present, so it's safer to use has_key?.

OK thanks for your review!


Explaining Ramda to the average guy, fixes #711

I tried to write an introduction that should be understandable by average javascript programmers, without functional programming knowledge. See issue #711. The "Philosophy" section is still intact for the advanced mind :smile:
I explicitly like to compare LoDash and Underscore directly, since they are what most folks know. Remember that LoDash compared itself vs Underscore and that helped tremendously to understand it's Raison d'être.
While
A practical functional library for Javascript programmers. is a good explanation, it doesn't explain the difference of Ramda vs LoDash or Underscore in one sentence.
Feel free to edit as pleased :+1:

also, one thing that distinguishes ramda is that it will not mutate your objects. We should make a point of highlighting that fact.

ok added a line about mutation as well.

Why is the title "Project Ramda" ? Sound like an experiment, guess this can be changed ?

Probably there from the beginning.  Change away!

copied some from this guy :smile:
http://buzzdecafe.github.io/code/2014/05/16/introducing-ramda/

copied some from this guy :smile:

that guy has no idea what he is talking about

:+1:

Is it a conscious decision to not use monospace for function names?

The idea is that, if foldl has this signature:

instead of

The idea is that, if foldl has this signature:

:+1:

:+1: on the function names in monospace.

Ramda does pretty well in this http://jsperf.com/lodash-underscore-ramda/2
should we link to a jsPerf on the note to "ramda performance" ?

no, 'cuz i don't wanna be constrained by that test

I agree that we don't want to tie to specific tests.
And I'd rather not have a section on performance.  I know we mention it, in the context of how this library differs from Eweda, but I wouldn't want to sell too hard on that front, honestly.  It's important, important enough that fixing performance issues at one time involved a significant overhaul, but the clean, useful, functional API is our main point.  As long as the performance is reasonable, I won't be terribly focused on it, myself.
Other than that, I'm almost ready to merge this one.  (I know it's been a long slog.)  There are only two things that I would still like to see.  One is that the list of features under What's Different? should probably be a bulleted list rather than a numbered one.
The other is that the section called Structure, which has nothing but information on the currying, should simply be removed.  That was originally planned as the beginning of a longer discussion, which never got written.
Oh, and I guess one third thing: It would probably be best to squash all these commits.

Because i edited the readme online for checking formatting, i have no history of commits to squash in git. I could make a new PR instead, with a single commit ? Any other options ? :smile:

Of course you do. It's all in your master branch, fyyyyy/ramda.

when i do rebase -i masteri get
noop

# Rebase d99a6c2..d99a6c2 onto d99a6c2
#
# Commands:
#  p, pick = use commit
#  r, reword = use commit, but edit the commit message
#  e, edit = use commit, but stop for amending
#  s, squash = use commit, but meld into previous commit
#  f, fixup = like "squash", but discard this commit's log message
#  x, exec = run command (the rest of the line) using shell
#
# These lines can be re-ordered; they are executed from top to bottom.
#
# If you remove a line here THAT COMMIT WILL BE LOST.
#
# However, if you remove everything, the rebase will be aborted.
#
# Note that empty commits are commented out

@fyyyyy
assuming origin/master points to ramda master
git checkout update-readme
git rebase origin/master
git rebase -i origin/master # squash all to the first one

Like this: https://gist.github.com/70228b83cb434c170fef

uhm, not sure this went well. i had to merge upstream master or otherwise there were no commits to squash. strange stuff i will not use online edit in the future :smile:

looks like its 30 commits now -- you included the doc changes :smiley_cat:

how about i simply make a new pr and squash this one :smile:

maybe simplest

Closing, see #716


Enable serialization of source locations for AstSerdeOptions

I believe this will serialize note scripts without source locations. Serializing with source locations now is a bit cumbersome - so, maybe we should just create two issues for now:

First issue would be in Miden VM to enable serialization of source locations by simply setting the appropriate property for AstSerdeOptions. For example, here, it should work something like this:
let note_script_bytes = note_script_ast.to_bytes(AstSerdeOptions {
    serialize_imports: true,
    serialize_source_locations: true,
});

The second issue would be here so that we don't forget to come back and fix this once the issue in Miden VM is fixed.
Originally posted by @bobbinth in https://github.com/0xPolygonMiden/miden-base/pull/339#discussion_r1421689757

This should be done for ModuleAst as well. The following tests currently fails because source locations are not being serialized correctly:
    let account = account::mock_account(Some(account_id.into()), Felt::ZERO, None, &assembler);

    let account_module = account.code().module();
    let account_module_bytes = account_module.to_bytes(AstSerdeOptions{serialize_imports:true});

    let reconstructed_ast = ModuleAst::from_bytes(&account_module_bytes).unwrap();

    assert_eq!(*account_module, reconstructed_ast)


Edit invoice

Notion card
https://www.notion.so/saeloun/Edit-Invoice-05b8cca4cf234559a51c3d49f7c39573
Summary
Added Edit invoice page
Preview

Type of change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Test cases for APIs were already in place.
Checklist:

[x] I have manually tested all workflows
[x] I have performed a self-review of my own code
[ ] I have added automated tests for my code

@apoorv1316 Please test this PR.

@apoorv1316 Fixed bugs, let me know if I am missing something.

@sudeeptarlekar Please resolve the conflicts on this PR.

@sudeeptarlekar This still has bugs on editing discount. Please see the loom video https://www.loom.com/share/006958646bdf4a6eb3e1e980baadd316


Issue 53

Try to address #53 to remove the sub-classes from BFO.
Things to consider:

Probably the most important, how can I open the edit-owl file in Protege from a branch? I wasn't able to check if the changes were made.
I follow the Compile Instructions wiki with some chages:

I think the second command that runs the 'robot templates' needs to be check, there is an error in "/modulesintermediate/"
I removed object properties from BFO with the following command: "robot remove --input imports/bfo_import.owl --axioms logical annotate --ontology-iri "http://purl.obolibrary.org/obo/cdno/modules/intermediate/bfo_import_axioms_removed.owl" --version-iri "http://purl.obolibrary.org/obo/cdno/modules/intermediate/bfo_import_axioms_removed.owl" --output modules/intermediate/bfo_import_axioms_removed.owl"
And in robot merge I added an extra module/intermediate as follows: --input modules/intermediate/bfo_import_axioms_removed.owl

Please let me know what you think @kaiiam.

I think wasn't the way to proceed, we'll try it again in a new PR.
Good catch with http://purl.obolibrary.org/obo/cdno/modulesintermediate/nutritional_components_framework.owl we'll need to fix that while were at it.

Ok, thanks. I'm closing this now.

Great I'm going to close this as it was taking care of in #59


GraphARM. Use of initial diffusion trajectories for whole learning step

I am getting an error when starting the diffusion ordering network training part. You can take a look.

I had no troubles running the the training on your branch. Could you post here what you're getting?

Oh, that's a surprise. This is my outcome.

Maybe checking the versions of your packages, specially torch_geometric, what I have here is 2.3.1. But also, looks like that would be solved adding a retain_graph=True in the .backwards() function.

Yes, it worked but I was not sure it would be a good solution. Anyways, it seems that the second fix is completed. Only one left!


Storymap Link 404

The link to the Intro to text mining StoryMap no longer works. Perhaps you mean this one instead: https://arcg.is/1j84jz

Nice catch, @spswanz , thanks for the corrected link. README updated in 94302c5 and Jupyter Notebooks updated in 451e97e.