Mbusd Docker Container

Is there details of how to create an mbusd docker container please ?  I want to be able to share the /dev/ttyUSB0 port with the sunsynk-multi, node-red and homeassistant docker containers, and it seems like the way to go
Should I be making a sort of bash script file and putting the contents of the Dockerfile into it ?  I tried to run that but it didn't work, so I am clearly going about it the wrong way.

Try the instructions https://kellerza.github.io/sunsynk/guide/standalone-deployment#mbusd

Ok, I'm doing something wrong
My docker-compose.yaml file contains the following
[code]
mbusd:
container_name: mbusd
restart: unless-stopped
image: ghcr.io/kellerza/hass-addon-sunsynk-multi/amd64:stable
devices:
- /dev/ttyUSB0:/dev/ttyUSB0
volumes:
- /dev/ttyUSB0:/dev/ttyUSB0
[/code]
I got the following errors after running docker compose up mbusd
[code]
root@Solar-M93:/opt# docker compose up mbusd
[+] Running 1/0
✔ Container mbusd  Created                                                                                0.0s
Attaching to mbusd
mbusd  | s6-rc: info: service s6rc-oneshot-runner: starting
mbusd  | s6-rc: info: service s6rc-oneshot-runner successfully started
mbusd  | s6-rc: info: service fix-attrs: starting
mbusd  | s6-rc: info: service fix-attrs successfully started
mbusd  | s6-rc: info: service legacy-cont-init: starting
mbusd  | s6-rc: info: service legacy-cont-init successfully started
mbusd  | s6-rc: info: service legacy-services: starting
mbusd  | services-up: info: copying legacy longrun sunsynk (no readiness notification)
mbusd  | s6-rc: info: service legacy-services successfully started
mbusd  | 09:39:53 ERROR   No configuration file found
mbusd  | 09:39:53 INFO    sunsynk library version: 0.6.4
mbusd  | 09:39:53 INFO    Schedules:
mbusd  | +-----------+-----+------+--------+-----------+----------+------------+
mbusd  | |    Key    | src | Read | Report | Change by | Change % | Change any |
mbusd  | +-----------+-----+------+--------+-----------+----------+------------+
mbusd  | | date_time |     |  60  |   60   |           |          |    True    |
mbusd  | |     rw    |     |  5   |  300   |           |          |    True    |
mbusd  | |     w     |     |  5   |   60   |     80    |          |            |
mbusd  | |    kwh    |     | 300  |  300   |           |          |            |
mbusd  | |  any_unit |     |  15  |  300   |           |          |            |
mbusd  | |  no_unit  |     |  15  |  300   |           |          |    True    |
mbusd  | +-----------+-----+------+--------+-----------+----------+------------+
mbusd  | 09:39:53 INFO    Using Single phase sensor definitions.
mbusd  | 09:39:53 INFO    Added hidden sensors as other sensors depend on it: Rated power, Serial
mbusd  | Traceback (most recent call last):
mbusd  |   File "", line 198, in _run_module_as_main
mbusd  |   File "", line 88, in _run_code
mbusd  |   File "/usr/local/lib/python3.11/site-packages/ha_addon_sunsynk_multi/main.py", line 79, in
mbusd  |     main()
mbusd  |   File "/usr/local/lib/python3.11/site-packages/ha_addon_sunsynk_multi/main.py", line 74, in main
mbusd  |     loop.run_until_complete(main_loop(loop))
mbusd  |   File "/usr/local/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
mbusd  |     return future.result()
mbusd  |            ^^^^^^^^^^^^^^^
mbusd  |   File "/usr/local/lib/python3.11/site-packages/ha_addon_sunsynk_multi/main.py", line 31, in main_loop
mbusd  |     MQTT.availability_topic = f"{SS_TOPIC}/{OPT.inverters[0].serial_nr}/availability"
mbusd  |                                             ~~~~~~~~~~~~~^^^
mbusd  | IndexError: list index out of range
mbusd  | [09:39:53] WARNING: Halt add-on with exit code 1
mbusd  | s6-rc: info: service legacy-services: stopping
mbusd  | s6-rc: info: service legacy-services successfully stopped
mbusd  | s6-rc: info: service legacy-cont-init: stopping
mbusd  | s6-rc: info: service legacy-cont-init successfully stopped
mbusd  | s6-rc: info: service fix-attrs: stopping
mbusd  | s6-rc: info: service fix-attrs successfully stopped
mbusd  | s6-rc: info: service s6rc-oneshot-runner: stopping
mbusd  | s6-rc: info: service s6rc-oneshot-runner successfully stopped
mbusd exited with code 0
[/code]

Ok, I see my first problem
What is the correct image to use
I try
https://ghcr.io/v2/kellerza/hass-addon-mbusd/amd64:stable
But this does not work

Ok, something not right
All looks OK with mbusd
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
But from Sunsynk-Multi
No response on the Modbus interface tcp://mbusd:502

What is <IP_ADDRESS>? The host address?
What do you use in the addon?

If you move away from docker name resolution (tcp://mbusd:502) - you probably want to expose the ports


Export Responsive pages to their respective platform folder

Change Proposal
If a given screen has support for multiple platforms, parabeac-core shall export each of these platforms to its respective folder to keep the structure organized.
Module and Current Solution
As an example, let's say we have a screen HomeScreen that has support for Desktop and Mobile. Parabeac-core should call upon the FileStructureStrategy to add these responsive pages to a folder with a directory structure similar to this one:
lib/
 |___ screens/

     |___ home_screen/

         |___ desktop/

             |___ home_screen_desktop.dart

         |___ mobile/

             |___ home_screen_mobile.dart

        |____ home_screen_layout_builder.dart

Benefits of the Change
The benefits of this change is that each screen will be organized according to platform, making it easy to locate, import, and expand platforms for any given screen.

This would require for us to create a command to export to mobile, desktop, tablet, etc.


CodePlex support should be removed

The CodePlex support on this page should be removed. CodePlex is no longer active.
Bottom of the page, in the section Integration and migration.
Christian

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a1dc9a4a-7828-f40f-4cf0-e0b1f5476ff2
Version Independent ID: 278aed60-558e-3beb-2ed4-c2517c2cd05d
Content: Git and TFVC version control - Azure Repos
Content Source: docs/repos/tfvc/comparison-git-tfvc.md
Product: devops
Technology: devops-code-tfvc
GitHub Login: @apawast
Microsoft Alias: sdanie

I'll get this updated.


[FALCON] [ERROR] GET / => Traceback (most recent call last):

[@ ~]$ python3<EMAIL_ADDRESS>/volume1/@appstore/monitorrent/monitorrent/upgrade_manager.py:35: UserWarning: 'connection' argument to configure() is expected to be a sqlalchemy.engine.Connection instance, got <sqlalchemy.orm.session.ContextSession object at 0x41fa3bc8>
migration_context = MigrationContext.configure(session)
/volume1/@appstore/monitorrent/monitorrent/rest/init.py:156: DeprecatedWarning: Call to deprecated function init(...). API class may be removed in a future release, use falcon.App instead.
return falcon.API(request_type=MonitorrentRequest, response_type=MonitorrentResponse, middleware=middleware)
Server started on <IP_ADDRESS>:6687
2020-10-18 15:04:45 [FALCON] [ERROR] GET / => Traceback (most recent call last):
File "/var/packages/py3k/target/usr/local/lib/python3.8/site-packages/falcon/app.py", line 321, in call
responder(req, resp, **params)
File<EMAIL_ADDRESS>line 56, in on_get
resp.stream_len = os.path.getsize(file_path)
File "/var/packages/py3k/target/usr/local/lib/python3.8/site-packages/falcon/response.py", line 220, in stream_len
raise AttributeError(_STREAM_LEN_REMOVED_MSG)
AttributeError: The deprecated stream_len property was removed in Falcon 3.0. Please use Response.set_stream() or Response.content_length instead.
2020-10-18 15:04:45 [FALCON] [ERROR] GET / => Traceback (most recent call last):
File "/var/packages/py3k/target/usr/local/lib/python3.8/site-packages/falcon/app.py", line 331, in call
process_response(req, resp, resource, req_succeeded)
TypeError: process_response() takes 4 positional arguments but 5 were given
Chrome:

500 Internal Server Error

Ставил по стандарту как здесь
Python 3.x and pip
Download latest build: https://github.com/werwolfby/monitorrent/releases/download/1.1.12/monitorrent-1.1.12.zip Extract into monitorent folder
pip install -r requirements.txt
python server.py

Почти 3 года спустя :) пофикшену указанием четкой версии в requirements.txt


Redefine Coin in terms of Natural and remove the Bounded instance.

Issue Number
ADP-1283
Summary
This PR:

Redefines Coin in terms of Natural (rather than Word64).
Removes the Bounded instance for Coin.
Adds txOut{Min,Max}Coin constants to Primitive.Types.Tx, in the same style as the existing txOut{Min,Max}TokenQuantity constants.
Pushes validation checks for coins that must be within a certain range to the places where those checks are actually required.

Implementation Notes
Where possible, this PR uses intCast and intCastMaybe (instead of fromIntegral) to perform statically-checked integral conversations.
Motivation
The wallet uses the Coin type to represent an amount of lovelace.
It's currently defined in terms of Word64, and has a Bounded instance that limits the range to [0, 45_000_000_000_000_000].
However, this approach is problematic, for several reasons:

Boundary checks for Coin values make sense in contexts where there are boundaries.  However, for pure Coin values in the absence of any particular context, there isn't one particular upper bound that makes sense.
The current choice of Bounded specifically defines the limits of what can be included in a transaction output. However, encoding values that appear in transaction outputs is not the only use of the Coin type.
For example, it's quite reasonable to use Coin for other purposes, like finding the total volume of ada transacted over some period of time. For such usages, there is no obvious choice of upper limit that we could consider “ideal”.
Right now, we often use awkward workarounds such as casting Coin values to Natural values before computing summations, thus losing the descriptive value of using the Coin type to mark that “this is a quantity of lovelace”.
Using the existing Semigroup instance, we can combine valid Coin values into values that are invalid.  💣

bors r+

bors r-

bors r+

bors r-

Due to problems with hydra, I plan to merge this PR manually. 🔧
I have confirmed that the following test suites pass when run locally (for commit a8d68a3371287a44843799e0828fdbb0e3e0427a):

cardano-wallet-core:test:unit
cardano-wallet:test:unit
cardano-wallet:test:integration

I have run each test suite twice to guard against spurious test failures.


ngx_lua: support ngx_lua custom coroutine's fun

Hello
@mpeterv
Now the newest luacheck 0.19.0 redefines the ngx_lua standard and it will make the CI failed because of accessing undefined field _yield of global coroutine in lua-resty-core
OpenResty inject the custom API to coroutine table as the following:
    lua_getglobal(L, "coroutine");

    /* set running to the old one */
    lua_getfield(L, -1, "running");
    lua_setfield(L, -3, "running");

    lua_getfield(L, -1, "create");
    lua_setfield(L, -3, "_create");

    lua_getfield(L, -1, "resume");
    lua_setfield(L, -3, "_resume");

    lua_getfield(L, -1, "yield");
    lua_setfield(L, -3, "_yield");

    lua_getfield(L, -1, "status");
    lua_setfield(L, -3, "_status");

    /* pop the old coroutine */
    lua_pop(L, 1);

    lua_pushcfunction(L, ngx_http_lua_coroutine_create);
    lua_setfield(L, -2, "__create");

    lua_pushcfunction(L, ngx_http_lua_coroutine_resume);
    lua_setfield(L, -2, "__resume");

    lua_pushcfunction(L, ngx_http_lua_coroutine_yield);
    lua_setfield(L, -2, "__yield");

    lua_pushcfunction(L, ngx_http_lua_coroutine_status);
    lua_setfield(L, -2, "__status");

    lua_setglobal(L, "coroutine");

So add the coroutine fields to pass the CI.

Hello @detailyang, are these extensions documented? The field list could be added to the standard so that typos like coroutine._yieldd are detected. But if there are not documented maybe the field list is not stable.

@mpeterv
As far as I know, these are not official document in OpenResty :(. But you know the code maybe is the best document although it could not stable.
Wish @agentzh can help us:)

@detailyang Those were internal API functions and are undocumented by intention :) Use of those API functions by normal users are discouraged. I think maybe you should just whitelist those specifically in your own luacheck configuration file, if user whitelisting is supported in luacheck?

gotcha


Memory leak  in Select2 plugin

Hi,
Events not properly detached. Its eating lot of memory. Try to Fix it.

I know that there are problems with memory management because it is generally difficult to to bind between a complex jQuery component and an ember component without leaking any memory but keeping all the data binding benefits.
I currently don't have any time for this problem and this project is discontinued in favour of ember-power-select anyways, but if you need this fix, please try to achieve it yourself and issue a pull-request.


Revamp the test system

The plan is as follows:

There will be a master test runner.
The test runner executes every test script with a fresh instance of PhantomJS.
The exit code of every script defines the success or failure.
If necessary, the output (stdout, stderr) can be compared against a defined baseline.

This was previously discussed in the mailing-list.

Looks good for now. Further refinements will be filed as a separate ticket.


Loading a graph asset trips an ensure in BlueprintCompilationManager.cpp

Line 1715: ensure(QueuedRequests.Num() == 0); is being tripped frequently, when dependencies are loaded for heart graphs.
Why? Aren't assets allowed to load other assets? What is different about this one.

I believe this was fixed accidentally either as a result of upgrading to 5.2 or by changing stuff in the registry subsystems


[SPARK-40707][CONNECT] Add groupby to connect DSL and test more than one grouping expressions

What changes were proposed in this pull request?

Add groupby to connect DSL and test more than one grouping expressions
Pass limited data types through connect proto for LocalRelation's attributes.
Cleanup unused Trait in the testing code.

Why are the changes needed?

Enhance connect's support for GROUP BY.
Does this PR introduce any user-facing change?

No
How was this patch tested?

UT

R: @cloud-fan

Can one of the admins verify this patch?

@cloud-fan PR updated. PLAT,

thanks, merging to master!


[Windows] Add SQL OLEDB Driver to win19 and win22

Description
Add SQL OLEDB Driver to win19 and win22
Related issue: https://github.com/actions/virtual-environments/issues/4881
Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows 2019, windows 2022

@al-cheb can u run the failed ones by using /azp, command

Something went wrong


docs(stepper): use mat-icon for custom icons example

Switches the custom icon example in the stepper to use mat-icon instead of the non-existing custom-icon. The idea with custom-icon was to show that consumers aren't limited to using mat-icon, however somebody that is copying from the examples would get a compilation error.
Fixes #10283.

This makes it much more understandable, thanks.


Configure an ESLint rule for prefer export default

When there is only a single export from a module, prefer using default export over named export.
// bad.js

// There is only a single module export and it's a named export.
export const foo = 'foo';

// good1.js

export const foo = 'foo';

const bar = 'bar';

export default 'bar'; // There is a default export.

For your reference or inspiration:
https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/prefer-default-export.md

@robinv85 should we move this to closed?

@manualzuru it's not fixed yet, we have ESLint configuration but if we change the rules then we have to update all the code so this is probably not a good time right now when a lot of development is happening, it will cause a lot of merge conflicts

I'm going to close this. It's kind of stale & we need to fix #786 first putting effort into adding more rules.


Fix for NPM usage #78

There is a proposal to solve the issue #78.
I have made the least possible changes to keep the file structure and the project the same.
The webm-writer library requires the "fs" module on Node and it causes some issues using webpack, it can be fixed on the webpack config, but i think we can delete the dependency if we target browsers.

Hi @Unsigno
I checked out your PR and set target: 'node' in my webpack config. But I'm still getting require related errors. Should they be analysed at build time, and the required code inlined? The required files no longer exist after the build step is performed.
ERROR in ./node_modules/ccapture.js/build/CCapture.all.min.js
Module not found: Error: Can't resolve './download' in '/Users/simonfrost/Code/threejs-tapered-hair/node_modules/ccapture.js/build'
 @ ./node_modules/ccapture.js/build/CCapture.all.min.js 1:24146-24167
 @ ./js/app.js
 @ multi (webpack)-dev-server/client?http://localhost:8080 ./js/app.js

ERROR in ./node_modules/ccapture.js/build/CCapture.all.min.js
Module not found: Error: Can't resolve './gif' in '/Users/simonfrost/Code/threejs-tapered-hair/node_modules/ccapture.js/build'
 @ ./node_modules/ccapture.js/build/CCapture.all.min.js 1:24188-24204
 @ ./js/app.js
 @ multi (webpack)-dev-server/client?http://localhost:8080 ./js/app.js

ERROR in ./node_modules/ccapture.js/build/CCapture.all.min.js
Module not found: Error: Can't resolve './tar' in '/Users/simonfrost/Code/threejs-tapered-hair/node_modules/ccapture.js/build'
 @ ./node_modules/ccapture.js/build/CCapture.all.min.js 1:24114-24130
 @ ./js/app.js
 @ multi (webpack)-dev-server/client?http://localhost:8080 ./js/app.js

ERROR in ./node_modules/ccapture.js/build/CCapture.all.min.js
Module not found: Error: Can't resolve './webm-writer-0.2.0' in '/Users/simonfrost/Code/threejs-tapered-hair/node_modules/ccapture.js/build'
 @ ./node_modules/ccapture.js/build/CCapture.all.min.js 1:24224-24254
 @ ./js/app.js
 @ multi (webpack)-dev-server/client?http://localhost:8080 ./js/app.js

Hi @SimonHFrost , thanks for your report , i was pointing directly the source so i don't noted the issue .
I updated the entry point of the NPM module , so it should works now. For browser you can link the single files ( i keep the UMD pattern ) or use the "build" source.
I tried to touch as little code as possible , but if there are still problems I could add some transpiler to build the code .
Anyway using 'node' as target will throw error , since "webm-writer" is using "fs", you can try "node-async" or find a solution for your needs. But some of the modules used are designed to work in browsers.
Best regards Unsigno.

Does this work? It would be great to get this fix published.

I tested this one and it works well!

maybe someone could merge and build?  pretty please!


[Feature] Add project id/workflow definition id/instance id to the built-in variables

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
These Ids are useful things that can be used as part of path (or something like that) to record temporary result generated by each modules
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@SbloodyS  i will fix it, plz assign this issue to me

@haibingtown I've assigned this issue to you. If you add other variables like workflow definition name and task name, that would be great. By the way the variable of task instance id already exists. You can take a look at the class of TaskConstants.

@calvinjiang  Thank you very much, the information you provided is very useful, I will submit the PR as soon as possible, and then help me review it

@TCGOGOGO

@TCGOGOGO hi，based on the information you provided, do you mean we need to put the informations ( project Code 、workflow definintion id and instance id ) into this path ?

@TCGOGOGO
@TCGOGOGO hi，based on the information you provided, do you mean we need to put the informations ( project Code 、workflow definintion id and instance id ) into this path ?

No. You can just provide some more built-in variables like 'workflow definition name' and 'task name' . Because these variables will be very common for others.

@TCGOGOGO
@TCGOGOGO hi，based on the information you provided, do you mean we need to put the informations ( project Code 、workflow definintion id and instance id ) into this path ?

Hi Haibing, just like calvin said, hope these IDs could be something like the ${system.biz.date} as built-in variables :-)

@TCGOGOGO @calvinjiang  Got it! I have submitted pr, please help me to review it => https://github.com/apache/dolphinscheduler/pull/14099 , Thx！


Adapt to new auth scheme

This adds support for JWT's we've issued ourselves with the new auth scheme, not relying on JWKS. It's non-destructive so that we can run the two schemes in parallel.

Whoops, wrong origin! Sorry!


Pod names are used instead of service names

tl;dr
Since version 1.8.2 kubefwd uses the pod names instead of the service names as hostnames

I used to use kubefwd 1.8.0 a lot for local development and embedded the service names in configs, e.g. the redis service oy my app acme for the feature featxy was forwarded according to the service name featxy-acme-redis. Since upgrading to 1.8.4 (actually since 1.8.2 and later) /etc/hosts is populated with the pod names instead (e.g. featxy-acme-redis-57cbcdbd98) which forces me to change all the configs if the pod gets a new identifier. Is this a bug or a feature?

If I remember correcly it was the result of a PR to fix headless services. I will look into this.

Thanks for the quick resolution!

@cjimti Sorry for digging this up again, but this issue seems to have reappeared. I'm using 1.9.7 and kubefwd only forwards the pod names (including the suffix) as host names again.

@Gerrit-K thanks for bringing this up. @calmkart, any chance you can help with this regression in #93 ? Thanks
I am at KubeCon this week so limited in my availability and then on vacation next week.

@Gerrit-K @cjimti
From my logic, the headless service should not be represented only by the first pod. So the service name accessing the headless service should not be forwarded to his first pod.
The current logic is:

The normal service forwards its first pod to the service name (nginx), and we can access the first pod by accessing the service name (nginx-deployment-5cdfb5fc49-b44j2)

Headless service forwards all pods (similar to this: nginx-deployment-5cdfb5fc49-b44j2.nginx1; nginx-deployment-5cdfb5fc49-dfhvs.nginx1) but does not forward the service name (nginx1), we access the headless service name (nginx1) Unable to access any pods.

Is it correct and necessary to automatically forward its first pod as a service name for a headless service? (Accessing nginx1 means accessing nginx-deployment-5cdfb5fc49-b44j2.nginx1)
I will make changes if necessary. Looking forward to the reply.

@calmkart yes to support 12-factor style development. When developing software locally the application may wish to point to a headless service (such as cassandra-client:9180,) since /etc/hosts is not DNS we need to point any headless service to the first Pod it finds.
However, we also need to continue forwarding the individual Pods backing a headless service to be as flexible as possible.

@cjimti
fixed in commit
for pr #95

@Gerrit-K I have reverted back to 1.9.6 and released it again as 1.10.0. This solved the problem for me as well as keeping @calmkart's performance updates.

Sorry for the late response and thanks to both of you for revisiting and fixing this!


honggfuzz: 2.3.1 -> 2.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/google/honggfuzz/releases.
meta.description for honggfuzz is: "A security oriented, feedback-driven, evolutionary, easy-to-use fuzzer"
meta.homepage for honggfuzz is: "https://honggfuzz.dev/"
meta.changelog for honggfuzz is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.4 with grep in /nix/store/kyvk29gqvisadbbjzpm5fimma8kmb7j5-honggfuzz-2.4

directory tree listing: https://gist.github.com/ba360602bb95c4b8792e4d78133cc3aa

du listing: https://gist.github.com/83e61108c0f3cf5263f732a839d113f4

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
honggfuzz

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kyvk29gqvisadbbjzpm5fimma8kmb7j5-honggfuzz-2.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A honggfuzz https://github.com/r-ryantm/nixpkgs/archive/1d61041fdfbf4ab98044b58561b08edf98fcb6fb.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kyvk29gqvisadbbjzpm5fimma8kmb7j5-honggfuzz-2.4
ls -la /nix/store/kyvk29gqvisadbbjzpm5fimma8kmb7j5-honggfuzz-2.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

honggfuzz

Maintainer pings
cc @cpu for testing.

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

1 package built:

honggfuzz


Add initial "Managed debugging with Unity IL2CPP" guide

This guide was requested at https://github.com/microsoft/MixedRealityToolkit-Unity/issues/4037.

Docs Build status updates of commit f843ebc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-attach.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-build.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-capabilities.png
:white_check_mark:Succeeded
View

mixed-reality-docs/managed-debugging-with-unity-il2cpp.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 006b54c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-attach.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-build.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-capabilities.png
:white_check_mark:Succeeded
View

mixed-reality-docs/managed-debugging-with-unity-il2cpp.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 006b54c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-attach.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-build.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-capabilities.png
:white_check_mark:Succeeded
View

mixed-reality-docs/managed-debugging-with-unity-il2cpp.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 1843566:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-attach.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-build.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-capabilities.png
:white_check_mark:Succeeded
View

mixed-reality-docs/managed-debugging-with-unity-il2cpp.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 1843566:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-attach.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-build.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-capabilities.png
:white_check_mark:Succeeded
View

mixed-reality-docs/managed-debugging-with-unity-il2cpp.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit a20be56:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-attach.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-build.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-capabilities.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-machines.png
:white_check_mark:Succeeded
View

mixed-reality-docs/managed-debugging-with-unity-il2cpp.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 7b18947:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-attach.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-build.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-capabilities.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-machines.png
:white_check_mark:Succeeded
View

mixed-reality-docs/managed-debugging-with-unity-il2cpp.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 7b18947:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-attach.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-build.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-capabilities.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/il2cpp-debugging-machines.png
:white_check_mark:Succeeded
View

mixed-reality-docs/managed-debugging-with-unity-il2cpp.md
:white_check_mark:Succeeded
View

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

Excellent! Thank you @keveleigh!

Is it possible to add a warning, that the App will not show up if the network does not support Multicast. Sadly I don't know any other way to know the port, which is opened on HoloLens by Unity. So without Multicast we can't connect VS to the Unity App on HoloLens :(


Caching does not work properly with multiple schemas

Describe the bug
When you register multiple schemas and endpoints the two schemas seem to interfere.
I have a reproducible here: https://github.com/SebastianStehle/dynamic-schemas/blob/main/WebApplication1/Startup.cs
Please ignore the other stuff inside the #if`, as it is for another test.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://localhost:5001/graphql/foo/'
Run the following query

query {
  both {
    value
  }
}

Go to 'https://localhost:5001/graphql/bar/'
Run the same query again.
See error

{
  "errors": [
    {
      "message": "Unexpected Execution Error",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "both"
      ],
      "extensions": {
        "message": "The parent cannot be casted to WebApplication1.Startup+QueryBar.",
        "stackTrace": "   at HotChocolate.Execution.Processing.MiddlewareContext.Source[T]()\r\n   at HotChocolate.Execution.Processing.MiddlewareContext.Parent[T]()\r\n   at lambda_method46(Closure , IResolverContext )\r\n   at HotChocolate.Types.FieldMiddlewareCompiler.<>c__DisplayClass3_0.<<CreateResolverMiddleware>b__0>d.MoveNext()\r\n--- End of stack trace from previous location ---\r\n   at HotChocolate.Execution.Processing.ResolverTask.ExecuteResolverPipelineAsync(CancellationToken cancellationToken)\r\n   at HotChocolate.Execution.Processing.ResolverTask.TryExecuteAsync(CancellationToken cancellationToken)"
      }
    }
  ],
  "data": {
    "both": null
  }
}

Expected behavior
The 2 schemas should be strictly separated, but something gets cached.
Desktop (please complete the following information):

OS: [e.g. iOS]
Version 11.0.9

Additional context
Add any other context about the problem here.

I have extended my test with a code first approach
If you go to
https://localhost:5001/graphql/fooType/ and then https://localhost:5001/graphql/barType/ then the value from the first used schema is returned.

I have extended my test with a code first approach
If you go to
https://localhost:5001/graphql/fooType/ and then https://localhost:5001/graphql/barType/ then the value from the first used schema is returned.

The issue is with the schema interceptor. I will close this issue since this is not a bug. We will have a look at your approach together on Friday.

Why is it with the schema interceptor?
I have disabled them for this test, so I have only registerd two schemas
services.AddGraphQLServer("myFoo")
    .AddQueryType<QueryFoo>();
services.AddGraphQLServer("myBar")
    .AddQueryType<QueryBar>();
services.AddGraphQLServer("myFooType")
    .AddQueryType<QueryFooType>();
services.AddGraphQLServer("myBarType")
    .AddQueryType<QueryBarType>();

and a few endpoints
app.UseEndpoints(endpoints =>
{
    endpoints.MapControllers();

    endpoints.MapGraphQL("/graphql/foo", "myFoo");
    endpoints.MapGraphQL("/graphql/bar", "myBar");
    endpoints.MapGraphQL("/graphql/fooType", "myFooType");
    endpoints.MapGraphQL("/graphql/barType", "myBarType");
});

Is this supported or not? If it is not supported, why can I define the schema name?

I simplified my example in this branch: https://github.com/SebastianStehle/dynamic-schemas/tree/simple
I also checked all the caches and when I turn off the IPreparedOperationCache it works as expected. I think it does not has anything to do with my special use case.

@SebastianStehle you are right. Just checked it. The issue is if the schemas essentially have an overlap so that the same queries is used with all of the schemas. In this case the query has is the same. I will issue a fix for this.


Fix corner case in MPT key encoding

See conflux-rust#1693 and conflux-rust#1695.

This change is

It looks pretty good. We should discuss and accept this CIP in a fast track way.


Switch default hotkey modifiers from ctrl alt cmd to just alt cmd

This facilitates using ctrl as an additional modifier for reverse cycle width and reverse cycle height window actions. Cmd alt conflicts with fewer important default MacOS keyboard shortcuts than ctrl cmd or ctrl option. Cmd alt also exists on the left and right side of the space bar.

@ambirdsall FYI


vr_slider doesnt scroll if item already selected

Before:
https://user-images.githubusercontent.com/216870/115997565-80956180-a5b1-11eb-96e7-9c8c2eff250a.mp4
After:
https://user-images.githubusercontent.com/216870/115997574-8ab76000-a5b1-11eb-8c9b-b451ea12c665.mp4

Reopening PR after I rebased the branch to only include fix commit


Added another dummy test

opening PR should activate test.yml

merging should hopefully activate release.yml (?)


Talgud neljapäeviti kell neli / Talgud on every Thursday @ 16:00

Place - Tallinn Univeristy A-428
Possibilities to contribute:

Estonian Open Data Issue Tracker
Post and review, build an API or shoot some ideas!
https://github.com/infokujur/Estonian-Open-Data-Issue-Tracker/wiki/About

Contribute to #Mydata2017 org team
http://mydata2017.org/2017/02/08/mydata-talkoot-what-when-why/

#7

Today we start again Tallinn Univeristy A-428 @ 16:00:

#MyData2017 team in Slack https://okfi.slack.com
If you already know that you wish to participate on Slack, please send an email to<EMAIL_ADDRESS>and Salla will sort sign up issues with you!

Working (contribute, review) with Estonian Open Data Issue tracker - https://github.com/okestonia/Estonian-Open-Data-Issue-Tracker/issues
The 5 Stars guideline for linked open data is a way to measure how well data is integrated into the Web.
About 1-3 stars - https://www.europeandataportal.eu/elearning/en/module10/#/id/co-01
About 3-5 stars - https://www.europeandataportal.eu/elearning/en/module13/#/id/co-01

Today again:
16 - 18, TLU, room A-428.
Mainly two topics:

MyData conference organizing
Info here: https://www.facebook.com/events/259934777781913/
Virtually join Open Knowledge Finland Slack team https://okfi.slack.com
Estonian Open Data Issue Tracker
Info here: https://github.com/…/Estonian-Open-Data-Issue-Tra…/issues/23
Virtually join Open Knowledge Estonia Slack team
https://okestonia.slack.com

As MyData conference is happening in a week, then in autumn to get together every Thursday is not that much needed (if interest is arising, then it can be reconsidered).
I'll open another topic with new regular get-together routine.


Add recalibrated benchmark ECMWF real-time forecast 2020

along the same lines as #2:

Will ECMWF provide this recalibrated real-time forecasts on EWC?
If so, would the code for that be also shared somewhere in this repo?

Yes, ECMWF can host the observation data and the recalibrated data, (and the tercile data) at the same place (S3 bucket wit https access).

The code to build the data should not be shipped to the end-user. It cannot got to the climetlab-s2s-ai-challenge/ directory in the present repos.
But some part of it may something you want to give them (tools to compute a tercile, tricky monthly average with leap year, etc.). IMHO, this code should go in the climetlab-s2s-ai-challenge/ directory.
I see to proper locations for the databuilding code :

Another repos. Perhaps in https://github.com/ecmwf-lab/dev-climetlab-s2s-ai-challenge/ or the one in renkulab ? This new repos should have the present one as a dependency.
Another folder in this repos which will not be zipped when releasing to pypi. For instance ./building_data/ ?

The code to build the data should not be shipped to the end-user.

Why shouldn’t it?

The code could also just be in the renku repo. But if it’s hosted here, why not have it as an explanation here?
My question furthermore is about how to compute this recalibration. A simple way https://renkulab.io/gitlab/aaron.spring/s2s-ai-competition-bootstrap/-/blob/master/notebooks/bias_reduction_forecast.ipynb from me but probably you at ECMWF have better codes for that.

We discussed with Frederic, he will recompute it and I will upload the data (same GRIB+netcdf, maybe zarr) formats.
Since we have two benchmarks now, I suggest we name them as:

forecast-benchmark-ml : the benchmark using the quick ML model trained by Aaron (was previously forecast-benchmark)
forecast-benchmark-recalibrated : this recalibrated data.

Latter on, we will open-source the appropriate code (likely to by python) to compute this recalibrated data.

I would skip that:

forecast-benchmark-ml : the benchmark using the quick ML model trained by Aaron (was previously forecast-benchmark)

I was just playing around with the data and keras, and the time alignment was very slopping and actually only really computed for one step.
I think forecast-benchmark as recalibrated ECMWF is enough.

ok

Closing as issues are tracked at :
https://renkulab.io/gitlab/aaron.spring/s2s-ai-challenge/-/issues  (general)
https://renkulab.io/gitlab/aaron.spring/s2s-ai-competition-bootstrap/-/issues (internal to S2S challenge organisers)


Motivational quotes might be confusing

Carrying on from #46429:
Some of the compliments in our quotes object, such as They're in another castle, may not make sense to campers.
Some of these are pop-culture references, like "Legen - wait for it - dary!", and "Moon Prism Power!". As someone who is familiar with those two particular phrases, seeing them pop up in my learning journey was a nice treat. However, campers who are not familiar with the references might not understand the meaning here.
Do we want to go through and clean up the ones that don't directly compliment the camper?
Original Conversation: https://github.com/freeCodeCamp/freeCodeCamp/pull/46429#issuecomment-1581267853

I vote to leave them. I didn't get some of them - but the ones I did, were nice to see.

Ya, I'm fine with leaving them. My main concern is that we should also make it clear to screen reader users that they have passed the step by unhiding the check mark icon from them and giving it an accessible name like "Step completed". Then they would hear "Step completed", and then the quote.

Thank you for creating a separate issue to address the quotes.
How about keeping the success message. And having the quote in the row of its own so there is no confusion?
In future iterations we could make the success message shorted and merge both if necessary.

How about this for the success state so there is no confusion.

So…is this issue solved or???

Yes, the related pr will be modified accordingly.


Adding orbit camera to extras

Adding the orbit camera from the Model Viewer to the extras folder

It only add orbit camera, and not control for it though.

Would we want the controls for the camera in the extras though? Were you envisioning a folder for the orbit camera with all the input components?

Currently there is Fly Camera, which has controls in it. So it is simply plug'n'play script. With orbit, it depends on controller script - input files. So I believe it would be good to have them, similar to ones we use in Model Starter Kit, that way dev can simply plug in files, and have orbit camera working.


Enable the "visitor" mode

Allow other users to connect the manager throught a visitor mode.
The means to add some sort of authentication.

Well there is two things in you issue.

Enable authentification. This is either with the Django session/auth or either a webserver auth (like with htacess). The first one can be enabled easily, but i planned to totally remove the session/auth to totally avoid database usage, to enlight cpu/memory usage. The other is for "real" webserver, like Apache/lhttpd/nginx/etc.. but actually we don't have one (again to save memory/cpu usage).

Visitor mode. Depend on choice for the previous part.

Frankly, for the first release i thought we don't really have to bother on these things.

I was not thinking about the first release. Just an idea
Le lun. 21 sept. 2015 à 13:39, David THENON<EMAIL_ADDRESS>a
écrit :

Well there is two things in you issue.

Enable authentification. This is either with the Django session/auth
or either a webserver auth (like with htacess). The first one can be
enabled easily, but i planned to totally remove the session/auth to totally
avoid database usage, to enlight cpu/memory usage. The other is for "real"
webserver, like Apache/lhttpd/nginx/etc.. but actually we don't have one
(again to save memory/cpu usage).
Visitor mode. Depend on choice for the previous part.
Frankly, for the first release i thought we don't really have to bother on
these things.
—
Reply to this email directly or view it on GitHub
https://github.com/sveetch/recalbox-manager/issues/23#issuecomment-141948915
.

Adding some sort of authentification should be a requirement if you are not alone on your LAN, or if, for some reason, you want to access your recalbox from the internet.
So it would be good if there was a variable somewhere in the recalbox to :

either enabling/disabling the whole interface
or enabling/disabling authentification (and setting the password)

If we "secure" the manager interface it wont be a switchable feature, because the machinery need for authentication is too complex to implement it as a conditional feature and will require too many time to build and manage.
I'm agree about authentication requirement for non private LAN, it's just on raspberry we are somewhat restrained on ressources, but enabling will take some more ressources in the django instance.
The recalbox team should validate than the manager can take some more ressources on system, so i will be more confident about enabling authentication. Itself the authentication is pretty easy to enable.


Using locate in packaging/osx/bayeslite/build_dmg.py is probably wrong

We use locate to find libboost_atomic-mt.dylib, but (a) macs don't always have locate service running, there is a more mac friendly service mdfind, mdfind -name libboost_atomic-mt.dylib and more importantly (b) I don't think locate is required to respond with the best file, just a file with a matching name, so this could get weird on a machine with multiple boost installs.
Better to use whatever library resolution scheme crosscat is using, if we need boost.

This should move to the packaging repo?
On the other hand, hacks are lower priority when they're only required for distribution, rather than for use.

Move to packaging repo: Yes, that's the right place.
Now, if this is just for our own builds, we could hard-code BOOST_ROOT=/usr/local/brew/wotsit/doodazzle/quagga/hoist/boost...

(Still a better option would be to nix boost altogether.)

725e378


Fix pipeline build break

Previous commit breaks the build with:
Targeting .NET 6.0 in Visual Studio 2019 is not supported.
Try retarget to VS 2022

/azp run

/azp run

/azp run


Rewrite SECURITY.md with technical writing

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/MyMethod
Description of issue (what needs changing):
Clear description
For example, why should someone use this method? How is it useful?
Correct links
Is the link to the source code correct?
Parameters defined
Are all parameters defined and formatted correctly?
Returns defined
Are return values defined?
Raises listed and defined
Are the errors defined? For example,
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/feature_column/categorical_column_with_vocabulary_file#raises
Usage example
Is there a usage example?
See the API guide: https://www.tensorflow.org/community/contribute/docs_ref
on how to write testable usage examples.
Request visuals, if applicable
Are there currently visuals? If not, will it clarify the content?
Submit a pull request?
Are you planning to also submit a pull request to fix the issue? See the docs
contributor guide: https://www.tensorflow.org/community/contribute/docs,
docs API guide: https://www.tensorflow.org/community/contribute/docs_ref and the
docs style guide: https://www.tensorflow.org/community/contribute/docs_style
Rewriting SECURITY.md with technical writing for better understanding.

I don't think SECURITY.md needs rewriting at this moment. Moreover, there is a separate internal process to handle this and some related changes.


Analytics page should report the host.

Analytics changed to using named arguments in hopes of avoiding any confusions.

LGTM.
HOST is defined in the BasePageHandlerTest in the refactored test code in apprc-t, should we be doing this work in the branch instead of main to take advantage of the refactoring we did there?

lgtm

@chuckhays probably we need to backport that change but I will do that when I get a chance in another CL.


Prompt Users for Feedback on The Ticket Raising Process

What would you like to change?
Continuous feedback and improvement is an important aspect of the BAU lifecycle. To reflect this, ask users who raise a ticket for their feedback on how the process went whenever their ticket is closed. Feedback should be anonymous, obtained via direct message from SlackBot and posted to a private slack channel.

Sending direct messages means this needs to be enabled on the app


Your system:
Windows 10 Pro 64 bits 10.0.19044 21H2
How to reproduce the bug:

Configure the profile.ps1 at WindowsPowerShell user's directory, following the reference guide.
Enter some wrong command, then enter with the alias configured to activate thefuck.
PowerShell recognizes the alias…
But always display the output message No fucks given as an answer.

Anything else you think is relevant:

I have tested on my Raspberry Pi, also installed using pipx and running awesome.
Only on my Windows PC that I'm having this issue.

Screenshot:

我也遇到了这个问题，无法解决

我也遇到了这个问题，无法解决

I hope the developers can fix it for the community.

 Check
the spelling of the name, or if a path was included, verify that the path is correct and try again.


Detect imported extvars and TLAs via sentinel import path

When a Jsonnet consults an Importer for a path nominated by an external variable or top-level argument file-based command-line flag, the "importedFrom" parameter passed to the Importer.Import method is not a valid URL, but is instead a sentinel placeholder value—namely, an empty string. Detect these sentinel values and use the current working directory as the base URL instead when resolving relative import paths.
NB: This patch is speculative for now, based on a change to the Jsonnet Importer.Import method's contract proposed by @sbarzowski (https://github.com/bitnami/kubecfg/pull/266#issuecomment-535090890). See google/go-jsonnet#329 for a request for establishing this contract.

See google/go-jsonnet#447. With that, we can proceed here. @mkmik, take note.


Where is the autoencoder mentioned in the paper.

Hi, ashual.
An autoencoder for generation is adopted in your paper, but I can't find it in this repo. Actually, many GANs is seems to be responsible for generation, but GANs are generally different with autoencoder. So how can I understand it?

hi Wang Penghui,
the term auto-encoder is used only once in the paper when referring to the network R.
This is a mistake, the right term is encoder-decoder and not auto-encoder since the input and output are different.
When referring to R in other places in the paper, the right term encoder-decoder is used.
Thanks for telling me


feat(meta): build info to telemetry

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Telemetry reports git sha and branch.
Example:
{
.......
        "build_info": {
                "build_date": "2023-05-25",
                "git_branch": "feature/telemetry-version",
                "git_sha": "b80004956"
        },
}

🤖 Generated by Copilot at b800049
Summary
🛠📜📡

The meta crate now includes git and build information in the telemetry data of the risingwave node. This is done by using a build script and the vergen crate to generate and access these data at compile time.

BuildInfo struct
sends git and build data
telemetry in spring

Walkthrough

Create a build script for the meta crate that generates environment variables for the git and build information using the vergen crate (link, link, link)
Define a BuildInfo struct that holds the build date, the git branch, and the git sha as strings and can be created from the environment variables (link)
Add a build_info field to the Telemetry struct that contains the BuildInfo struct and represents the git and build information of the node (link)
Initialize the build_info field of the Telemetry struct with a new instance of BuildInfo from the environment variables in the src/meta/src/telemetry.rs file (link)

Checklist For Contributors

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[x] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[x] My PR DOES NOT contain user-facing changes.

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

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

Release note

Hey, any updates?


Switching between faster-whisper or openai-whisper via env seems to be broken #edit: most likely not, just not as fast as I hoped for

I transcribed a 22 minute mp3 file with - ASR_ENGINE=openai_whisper via the webui, timed it and it took 2:15 min. I then changed the env to - ASR_ENGINE=faster_whisper, recreated the container and it took approximately the same 2:16 min. Tried another file with the same outcome. Switched to :debug, also the same outcome. Vram consumption is also much higher than expected for faster-whisper, but in line with openai-whisper.
services:
  whisper-asr-webservice:
    #image: onerahmet/openai-whisper-asr-webservice:debug-gpu
    image: onerahmet/openai-whisper-asr-webservice:latest-gpu
    deploy:
      resources:
        reservations:
          devices:
            - driver: nvidia
              capabilities: [gpu]
    container_name: Whisper-ASR
    environment:
      - ASR_ENGINE=openai_whisper
#      - ASR_ENGINE=faster_whisper
      - ASR_MODEL=small
    ports:
      - 9007:9000
    restart: unless-stopped

Would be glad if you could take a look at it @ahmetoner, as it seems to be broken.
Also described by other people https://github.com/morpheus65535/bazarr/issues/2144
I don't know much, but it seems possible that the downloaded openai-whisper model isn't converted to the CTranslate2 model format. If that is the case, wouldn't it be possible to just download the model directly from https://huggingface.co/guillaumekln/faster-whisper-small (as an example)?
Or could it be that the wrong model is selected? Instead of loading the converted model, the openai-whisper model is loaded?
Another possibility could be that the settings differ between OpenAI-whisper and faster-whisper: Depending on the gpu fp16 could be a lot faster than fp32 https://github.com/openai/whisper/discussions/391 . Also beam_size https://github.com/guillaumekln/faster-whisper/issues/9 https://github.com/guillaumekln/faster-whisper/issues/172 and temperature https://github.com/guillaumekln/faster-whisper/issues/172
If these options are the reason for "underperforming" would it be possible to expose them as env variables for the docker container? (@ayancey)

Okay, I tried some things. I modified the docker image and basically cut openai-whisper out, so that only the faster-whisper implementation was running. I than modified the core.py and utils.py to also deactivated the converter and than mounted a downloaded model from https://huggingface.co/guillaumekln/faster-whisper-small. And yeah, same performance.
I then played a little bit with the settings, modified compute type (float32 actually had the best performance for my gpu) added os.environ["OMP_NUM_THREADS"] = "12" (which most likely helps only with cpu) and also change beam_size=5 to 1 and added best_of=1
segment_generator, info = model.transcribe(audio, beam_size=1, best_of=1, **options_dict)
all of that brought the time down to 1:30 min from 2:15 for the same 21 min mp3 on my rtx 3060. The small models vram usage was in line with faster-whisper float32 with 1430mb. But the quality has most like degraded (Beam_size=5 is the recommendation), though I haven't noticed so far.
So yeah, Idk, maybe it's as fast as it can be. I'm not a coder, I just poked around. Maybe it's as slow/fast as the openai implementation because it was the other way around and only faster-whisper is used. Maybe the openai-whisper has worse and faster-whisper higher quality settings by default (Though from what I understood they where the same). The beam_size setting certainly added vram usage when upped.
This one https://github.com/m-bain/whisperX with the same whisper-asr-webservice ui would most likely be chefs kiss though 😅

I'll take a look, but no promises 😅

I've heard whisper-x does a better job with subtitle time stamps so I'd love to see that get added!

@Deathproof76 if you have input on this would love to hear it: https://github.com/ahmetoner/whisper-asr-webservice/pull/125


Some drivers for imx225 and JX-H42

imx225: solved gain/explosure problems (may be not for you... :), new resolution 960p.
JS-H42: working driver, tested

Source files were commited to sersors repo. Eveyone can test driver, I will be happy!


MERGING NEW CODE EDITOR

[LEVEL]/[COMPONENT]
TASK: MERGING NEW CODE EDITOR
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Only
[x] Refactor

Description, Motivation and Context
Moved Component Docs to New Editor
User Doc Added
Image Doc Added
Screenshots - Animations

Codecov Report

Merging #340 (4ba2f89) into main (35f392b) will decrease coverage by 0.58%.
The diff coverage is 95.91%.

@@            Coverage Diff             @@
##             main     #340      +/-   ##
==========================================
- Coverage   75.66%   75.07%   -0.59%
==========================================
  Files         140      162      +22
  Lines        2272     2568     +296
  Branches      679      755      +76
==========================================
+ Hits         1719     1928     +209
- Misses        549      631      +82
- Partials        4        9       +5

Impacted Files
Coverage Δ

packages/react/src/checkbox/checkbox.styles.ts
100.00% <ø> (ø)

packages/react/src/checkbox/index.ts
100.00% <ø> (ø)

packages/react/src/grid/grid-item.tsx
100.00% <ø> (ø)

packages/react/src/grid/grid.tsx
100.00% <ø> (ø)

packages/react/src/input/input.tsx
93.54% <ø> (ø)

packages/react/src/checkbox/checkbox.tsx
96.22% <95.65%> (+0.57%)
:arrow_up:

packages/react/src/grid/grid-container.tsx
100.00% <100.00%> (ø)

packages/react/src/use-keyboard/helper.ts
77.77% <0.00%> (-22.23%)
:arrow_down:

...ages/react/src/pagination/pagination-highlight.tsx
68.42% <0.00%> (-19.08%)
:arrow_down:

packages/react/src/use-keyboard/use-keyboard.ts
89.18% <0.00%> (-10.82%)
:arrow_down:

... and 35 more

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 35f392b...4ba2f89. Read the comment docs.


RW does not seem to work anymore

I had been successfully using RW in the past. Today, after some time I've not been updating any mission, I tried to update some missions of mine and noticed RW does not update them.
More precisely, every mission, after being processed by RW, has no cloud preset and has cloud base, thickness and density to minimum values. Wind is instead correctly updated.
I'm attaching a zip contaning:
a) one of the many "original" mission files I tried RW with (mission.miz);
b) the resulting miz file after being processed by RW (realweather.zip);
c) RW logfile (logfile.log);
d) my config.json (of course, I'm going to remove the API key and replace it with a series of 'x' for safety reason).
Thank
RW issue.zip
s.

Thanks for making a new issue. I'm looking at your logfile, and I'm unsure if this is actually a bug.
Here's the METAR that the utility is receiving from checkwx:
URSS 052000Z 04005MPS 9999 NSC 15/09 Q1022 R02/010070 R06/010070 NOSIG RMK R06/00000MPS QFE765
And the METAR the utility is generating after translating to closest DCS match:
URSS 052000Z 04010KT 6SM CLR 15/09 A3018 NOSIG
Basically the METAR you're getting says there are clear skies so the realweather utility is replicating that. You can confirm this by using an ICAO that has some weather other than clear skies and ensure that the weather is updated accordingly.

I'll try other airbases, with not so clear skies, and let you know. Thanks for feedback.

RW rightly updated the miz file with a METAR with broken clouds and rain. Closing issue, thx.


Raytracing and pointsprites do not work well together

Describe the bug
Raytracing and pointsprites do not work well together and eventually hangs F3D
To Reproduce
Steps to reproduce the behavior:

Open the file using f3d --dry-run cow.vtp -ro
Rendering is incorrect, should show point sprites in raytracing, only show raytracing

Expected behavior
Correct rendering at all steps

@Meakk do you know if this could ever work ? It does in ParaView at least.

We could I guess by just using the regular polydata mapper instead of point gaussian with the "render point only" option?

Not great imo. Lets wontfix this for now.


Better support for git log --graph

Info

Operating System: Arch Linux

Shell: bash

Terminal: kitty

nvim --version: 0.9.1

fzf --version: 0.42.0

[ ] The issue is reproducible with mini.sh

fzf-lua configuration
require('fzf-lua').setup({
  git = {
    commits = {
      cmd = "git log --graph --all --color --format='%C(auto)%h%d %s %C(black)%C(bold)%cr'",
    }
  }
})

Description
This will cause yanking, checkout, edit and split actions to fail because the plugin fails to find the commit hash. git log --graph output starts with special characters at the beginning of the line, which is what the plugin was matching. Is it possible to change this line in actions.lua from
local commit_hash = selected[1]:match("[^ ]+")

to something like this instead
local commit_hash = selected[1]:match("[a-z0-9]+")

I am aware this doesn't cover all cases but it's the quick and easiest to implement while still allow for some level of flexibility.

I’ll do some testing and see if I can find a better regex that captures more use cases.

Thank you! Lua's "regex" is relatively limiting so that's the best I can come up with. I suppose another solution is to let the users define the pattern themselves via config?

Thank you! Lua's "regex" is relatively limiting so that's the best I can come up with. I suppose another solution is to let the users define the pattern themselves via config?

Good idea, try this (can also be defined via setup):
require("fzf-lua").git_commits({
  fn_match_commit_hash = function(line, _)
    return line:match("[a-z0-9]+")
  end
})


DefaultMinVersion="1.10" for outlook-set-signature

Describe the bug
[outlook-set-signature] sample use the signature API which is available from version 1.10, so manifest should reflect that.
To Reproduce
The Add-ins isn't compatible with version prior to 1.10
Expected behavior
Add-ins shouldn't be load if version 1.10 is not available
Environment
Example : Outlook 2016 shouldn't load the Add-ins
Additional context
https://docs.microsoft.com/en-us/javascript/api/requirement-sets/outlook/requirement-set-1.10/outlook-requirement-set-1.10?view=common-js-preview

Thanks for reporting this issue @maxCSA.
@samantharamon, could you plese investigate?

Hi @maxCSA,
Thank you again for catching this. The change is pending review in PR #299. Should you have additional feedback, please do not hesitate to let us know.


Angular directive ng-hide in cell template is not binding on change in grid data

Angular directive ng-hide in cell template is not binding with latest data on change in grid data. Please check plunkr for more details :- http://plnkr.co/edit/rj0PrfyzeyVeV8Q8RWtG?p=preview
In plunker template field is not updating with change in data.

I was trying something wrong.
ng-hide should be like
..ng-hide=row.entity.isActive=="Y"..
not like this interpolation,
.. ng-hide={{row.entity.isActive=="Y"}}..


:bookmark: release 1.18.1

Description
Release fix #851 and closes #850.
Checklist

[x] My changes follow the contributing guidelines of this project.
[x] I have performed a self-review of my code.
[ ] ~I have made corresponding changes to the documentation.~
[ ] ~I have added tests that prove my fix is effective or that my changes work.~
[x] New and existing unit tests pass locally with my changes.

Coverage: 94.954%. Remained the same when pulling 7363dc99ce5981d524cf44a34a0269e1ae23338e on release/1.18.1 into 7bf7eea359e5232e18ce032bd4fbac46bb2a8ce3 on main.


impl of cmd create test report

add first version of cmd create-test-report, including some core modules impls and unit test cases

cli cmd, group command and sub command

worksheet manager to duplicate, update test report

advisory manager to get jira issues (flaw bugs are on bugzilla)

jira manager to get/update/create jira issues, util funcs to check issue is critical/cve/blocker

logging init, disable debug logging for some internal loggers.

declared some exceptions, raise from app level, can be handled by higher level app

declared constants for frequently used str

cli running result:

# help
cli --help
Usage: cli [OPTIONS] COMMAND [ARGS]...

Options:
  -r, --release TEXT  z-stream releaes version  [required]
  -v, --debug         enable debug logging
  --help              Show this message and exit.

Commands:
  create-test-report  Create test report for z-stream release

# required option is missed
cli create-test-report
Usage: cli [OPTIONS] COMMAND [ARGS]...
Try 'cli --help' for help.

Error: Missing option '-r' / '--release'.

# w/o debug flag
cli -r 4.12.16 create-test-report
2023-05-05T16:38:07Z: INFO: local config is loaded
2023-05-05T16:41:47Z: INFO: ocp build data of 4.12.16 is downloaded
2023-05-05T16:41:51Z: INFO: build info is updated
2023-05-05T16:41:52Z: INFO: advisory info is updated
2023-05-05T16:41:53Z: INFO: jira info is updated
2023-05-05T16:43:21Z: INFO: bugs to be verified are updated
2023-05-05T16:43:21Z: INFO: new test report is created: https://docs.google.com/spreadsheets/d/1Xv8qfYUwp61lOOMQaXNO7wnlo9b8mXACgug49cZHCTM#gid=1040444075

# w/ debug flag
cli -r 4.12.16 -v create-test-report
2023-05-05T16:51:15Z: INFO: local config is loaded
2023-05-05T16:51:16Z: INFO: ocp build data of 4.12.16 is downloaded
2023-05-05T16:51:19Z: INFO: build info is updated
2023-05-05T16:51:19Z: DEBUG: build info:
aarch64: 4.12.0-0.nightly-arm64-2023-05-03-200750
ppc64le: 4.12.0-0.nightly-ppc64le-2023-05-03-200749
s390x: 4.12.0-0.nightly-s390x-2023-05-03-200748
x86_64: 4.12.0-0.nightly-2023-05-04-010834

2023-05-05T16:51:20Z: INFO: advisory info is updated
2023-05-05T16:51:20Z: DEBUG: advisory info:
extras: 113789
image: 113788
metadata: 113790
rpm: 113787
microshift: 113791

2023-05-05T16:51:20Z: INFO: jira info is updated
2023-05-05T16:51:20Z: DEBUG: jira info:
ART-6731
2023-05-05T16:52:32Z: DEBUG: updating jira issue OCPBUGS-6572 ...
2023-05-05T16:52:32Z: DEBUG: jira issue OCPBUGS-6572 status is Verified, skipping
2023-05-05T16:52:33Z: DEBUG: updating jira issue OCPBUGS-11559 ...
2023-05-05T16:52:33Z: DEBUG: jira issue OCPBUGS-11559 is ON_QA, updating
2023-05-05T16:52:34Z: DEBUG: updating jira issue OCPBUGS-11844 ...
2023-05-05T16:52:34Z: DEBUG: jira issue OCPBUGS-11844 is ON_QA, updating
2023-05-05T16:52:35Z: DEBUG: updating jira issue OCPBUGS-11972 ...
2023-05-05T16:52:35Z: DEBUG: jira issue OCPBUGS-11972 status is Verified, skipping
2023-05-05T16:52:36Z: DEBUG: updating jira issue OCPBUGS-11993 ...
2023-05-05T16:52:36Z: DEBUG: jira issue OCPBUGS-11993 status is Verified, skipping
2023-05-05T16:52:36Z: DEBUG: updating jira issue OCPBUGS-12159 ...
2023-05-05T16:52:36Z: DEBUG: jira issue OCPBUGS-12159 status is Verified, skipping
2023-05-05T16:52:37Z: DEBUG: updating jira issue OCPBUGS-12199 ...
2023-05-05T16:52:37Z: DEBUG: jira issue OCPBUGS-12199 status is Verified, skipping
2023-05-05T16:52:38Z: DEBUG: updating jira issue OCPBUGS-12265 ...
2023-05-05T16:52:38Z: DEBUG: jira issue OCPBUGS-12265 status is Verified, skipping
2023-05-05T16:52:39Z: DEBUG: updating jira issue OCPBUGS-12361 ...
2023-05-05T16:52:39Z: DEBUG: jira issue OCPBUGS-12361 is ON_QA, updating
2023-05-05T16:52:40Z: DEBUG: updating jira issue OCPBUGS-12440 ...
2023-05-05T16:52:40Z: DEBUG: jira issue OCPBUGS-12440 status is Verified, skipping
2023-05-05T16:52:41Z: DEBUG: updating jira issue OCPBUGS-12473 ...
2023-05-05T16:52:41Z: DEBUG: jira issue OCPBUGS-12473 status is Verified, skipping
2023-05-05T16:52:42Z: DEBUG: updating jira issue OCPBUGS-12476 ...
2023-05-05T16:52:42Z: DEBUG: jira issue OCPBUGS-12476 status is Verified, skipping
2023-05-05T16:52:42Z: DEBUG: updating jira issue OCPBUGS-12477 ...
2023-05-05T16:52:42Z: DEBUG: jira issue OCPBUGS-12477 status is Verified, skipping
2023-05-05T16:52:43Z: DEBUG: updating jira issue OCPBUGS-12688 ...
2023-05-05T16:52:43Z: DEBUG: jira issue OCPBUGS-12688 is ON_QA, updating
2023-05-05T16:52:44Z: DEBUG: updating jira issue OCPBUGS-12768 ...
2023-05-05T16:52:44Z: DEBUG: jira issue OCPBUGS-12768 status is Verified, skipping
2023-05-05T16:52:45Z: DEBUG: updating jira issue OCPBUGS-12993 ...
2023-05-05T16:52:45Z: DEBUG: jira issue OCPBUGS-12993 is ON_QA, updating
2023-05-05T16:52:46Z: DEBUG: updating jira issue OCPBUGS-1753 ...
2023-05-05T16:52:46Z: DEBUG: jira issue OCPBUGS-1753 status is Verified, skipping
2023-05-05T16:52:47Z: DEBUG: updating jira issue OCPBUGS-4276 ...
2023-05-05T16:52:47Z: DEBUG: jira issue OCPBUGS-4276 status is Verified, skipping
2023-05-05T16:52:48Z: DEBUG: updating jira issue OCPBUGS-6888 ...
2023-05-05T16:52:48Z: DEBUG: jira issue OCPBUGS-6888 status is Verified, skipping
2023-05-05T16:52:48Z: DEBUG: updating jira issue OCPBUGS-7400 ...
2023-05-05T16:52:48Z: DEBUG: jira issue OCPBUGS-7400 is ON_QA, updating
2023-05-05T16:52:49Z: DEBUG: updating jira issue OCPBUGS-13087 ...
2023-05-05T16:52:49Z: DEBUG: jira issue OCPBUGS-13087 status is Verified, skipping
2023-05-05T16:52:50Z: DEBUG: updating jira issue OCPBUGS-13088 ...
2023-05-05T16:52:50Z: DEBUG: jira issue OCPBUGS-13088 status is Verified, skipping
2023-05-05T16:52:51Z: DEBUG: updating jira issue OCPBUGS-13089 ...
2023-05-05T16:52:51Z: DEBUG: jira issue OCPBUGS-13089 status is Verified, skipping
2023-05-05T16:52:51Z: INFO: bugs to be verified are updated
2023-05-05T16:52:51Z: INFO: new test report is created: https://docs.google.com/spreadsheets/d/1Xv8qfYUwp61lOOMQaXNO7wnlo9b8mXACgug49cZHCTM#gid=1040444075

unittest result

Running tests (unittest): /Users/rio.liu/coderepo/release-tests/tests/test_advisory_mgr.py
Running tests: /Users/rio.liu/coderepo/release-tests/tests/test_advisory_mgr.py::TestAdvisoryManager::test_get_jira_issues
/Users/rio.liu/coderepo/release-tests/tests/test_advisory_mgr.py::TestAdvisoryManager::test_init
./test_advisory_mgr.py::TestAdvisoryManager::test_get_jira_issues Passed

./test_advisory_mgr.py::TestAdvisoryManager::test_init Passed

Running tests (unittest): /Users/rio.liu/coderepo/release-tests/tests/test_config_store.py
Running tests: /Users/rio.liu/coderepo/release-tests/tests/test_config_store.py::TestConfigStore::test_get_advisories
/Users/rio.liu/coderepo/release-tests/tests/test_config_store.py::TestConfigStore::test_get_cadidate_builds
/Users/rio.liu/coderepo/release-tests/tests/test_config_store.py::TestConfigStore::test_get_email_contact
/Users/rio.liu/coderepo/release-tests/tests/test_config_store.py::TestConfigStore::test_get_jira
/Users/rio.liu/coderepo/release-tests/tests/test_config_store.py::TestConfigStore::test_get_owner
/Users/rio.liu/coderepo/release-tests/tests/test_config_store.py::TestConfigStore::test_get_report_template
/Users/rio.liu/coderepo/release-tests/tests/test_config_store.py::TestConfigStore::test_get_slack_contact
/Users/rio.liu/coderepo/release-tests/tests/test_config_store.py::TestConfigStore::test_init
./test_config_store.py::TestConfigStore::test_get_advisories Passed

./test_config_store.py::TestConfigStore::test_get_cadidate_builds Passed

./test_config_store.py::TestConfigStore::test_get_email_contact Passed

./test_config_store.py::TestConfigStore::test_get_jira Passed

./test_config_store.py::TestConfigStore::test_get_owner Passed

./test_config_store.py::TestConfigStore::test_get_report_template Passed

./test_config_store.py::TestConfigStore::test_get_slack_contact Passed

./test_config_store.py::TestConfigStore::test_init Passed

Running tests (unittest): /Users/rio.liu/coderepo/release-tests/tests/test_jira_mgr.py::TestJiraManager
Running tests: /Users/rio.liu/coderepo/release-tests/tests/test_jira_mgr.py::TestJiraManager::test_create_issue
/Users/rio.liu/coderepo/release-tests/tests/test_jira_mgr.py::TestJiraManager::test_get_issue_fields
/Users/rio.liu/coderepo/release-tests/tests/test_jira_mgr.py::TestJiraManager::test_init
/Users/rio.liu/coderepo/release-tests/tests/test_jira_mgr.py::TestJiraManager::test_invalid_token
/Users/rio.liu/coderepo/release-tests/tests/test_jira_mgr.py::TestJiraManager::test_update_issue
./test_jira_mgr.py::TestJiraManager::test_create_issue Skipped: null

./test_jira_mgr.py::TestJiraManager::test_get_issue_fields Passed

./test_jira_mgr.py::TestJiraManager::test_init Skipped: null

./test_jira_mgr.py::TestJiraManager::test_invalid_token Skipped: null

./test_jira_mgr.py::TestJiraManager::test_update_issue Passed

Running tests (unittest): /Users/rio.liu/coderepo/release-tests/tests/test_worksheet_mgr.py::TestWorksheetManager
Running tests: /Users/rio.liu/coderepo/release-tests/tests/test_worksheet_mgr.py::TestWorksheetManager::test_init
/Users/rio.liu/coderepo/release-tests/tests/test_worksheet_mgr.py::TestWorksheetManager::test_report
./test_worksheet_mgr.py::TestWorksheetManager::test_init Passed

./test_worksheet_mgr.py::TestWorksheetManager::test_report Passed

/assign @wewang58 @kuiwang02 @Xia-Zhao-rh @jianzhangbjz

@kuiwang02 checked-in the fixes of error handling logic

/lgtm

/retitle OCPQE-17513 impl of cmd create test report


next-transpile-modules is deprecated and I believe it's slowing down the site

Hi, @nandorojo, thanks for the amazing lib!
Recently I've been playing with it and exploring my options for a next project.
Can you please go through some of my questions?
If it's the case I'll maybe be able to open a PR and contribute a little.
First of all, I tried adding a lib that required SSR ( Server Side Rendering ) and had some troubles with that.
Later I found you were in the same situation as I in this discussion.
Then it clicked, I had to add the lib to the group of next-transpile-modules and it worked great.
But as you said, that's kind of a necessary evil, as it doesn't help with performance.
Question 1
I didn't understand why the app is being transpiled
https://github.com/nandorojo/solito/blob/ff1497c4b40242f415f1e6c7591807a3a212d5ea/example-monorepos/blank/apps/next/next.config.js#L5-L11
To test my hypothesis, I removed it and it works perfectly. It even squeezed some performance points in the Chrome's Lighthouse reports that I did.
Is there a reason it's there that I'm not seeing? Can I remove it and open a PR?
I believe it's just a matter of removing it from the 4 examples.... right?
Question 2
The README.md file has a section about adding dependencies.
Should we add an observation about this transpilation workaround for SSR? I can do that.
It's pretty likely someone will try adding a dependency ( probably UI related ) and stumble on the same problem in the future.
Question 3
Are you aware that this lib (next-transpile-modules) was deprecated?
Here's a cute good bye post from the lib maintainer: https://github.com/martpie/next-transpile-modules/releases
From now on that's a native feature from Next.JS
Unfortunately that doesn't exist in the current version of next we're using, so an upgrade is necessary.
Is there any special reason we're in such version of next? If not, I could try upgrading it, let's hope things don't break that easily, it's Javascript we're talking about here 🙃

I didn't understand why the app is being transpiled

I don't see how this will work if it's not. It's a third-party package that ships in typescript files as far as Next.js is concerned. You're saying it works for you without that?

Should we add an observation about this transpilation workaround for SSR? I can do that.

I wouldn't want to do this without better understanding what's going on here. Transpilation shouldn't have an impact on SSR. What in particular about transpiling affects the output? It might just be that transpilePackages from Next.js is better optimized for Next than next-transpile-modules was.

Are you aware that this lib (next-transpile-modules) was deprecated?

Yes, just waiting to make sure the community confirms Next.js 13.1 is satisfactory for this.

I didn't understand why the app is being transpiled

I don't see how this will work if it's not. It's a third-party package that ships in typescript files as far as Next.js is concerned. You're saying it works for you without that?

Yes, it worked for me without the app  in the next-transpile-module list.

Maybe removing it adds problems I didn't observe because of my quick observation (in a very simple project)?
Maybe it was a cache thing on my end that made it work?

I don't know, but I've just educated myself a little better about the need of the next-tranpile-module and you're right.
We should keep the app in the list at least following a theoretical standpoint of what the next-transpile-module is supposed to do. It's a third party package outside of the next root, we have to bring it ( with the tranpilation ).
Sorry for bothering if I did.
I'm closing the issue then 🙂


Socket.io not calling connection

I'm trying the following in a NodeJS application:
var io = require('socket.io')();

io.on('connection', function (socket) {
    console.log('connected');
    socket.emit('news', { hello: 'world' });
    socket.on('my other event', function (data) {
        console.log(data);
    });
});

io.listen(8004);

var socket = require('socket.io/node_modules/socket.io-client')('http://localhost');

socket.on('news', function (data) {
    console.log(data);
    socket.emit('my other event', { my: 'data' });
});

But I don't see any of the console logs.
Am I doing something wrong? I don't even see the connected log and don't get any errors.

If I console out io I get:
{ nsps:
   { '/':
      { name: '/',
        server: [Circular],
        sockets: [],
        connected: {},
        fns: [],
        ids: 0,
        acks: {},
        adapter: [Object] } },
  _path: '/socket.io',
  _serveClient: true,
  _adapter: [Function: Adapter],
  _origins: '*:*',
  sockets:
   { name: '/',
     server: [Circular],
     sockets: [],
     connected: {},
     fns: [],
     ids: 0,
     acks: {},
     adapter: { nsp: [Circular], rooms: {}, sids: {}, encoder: {} } } }

So the module is being loaded. It just doesn't seem to be connecting.

The url in var socket = .... should be http://localhost:8004

Ah right, and the connection event only fires when both client and server have connected rather than just the server end. Thanks.


Allow Profile Sample to be set using at the decimal point

In the 2 location where we can set profile sample we should be able to set a decimal sample

completed with https://github.com/open-metadata/OpenMetadata/pull/7485


Update brew cli on macOS

removed tap, since it is not under any taps, simply run 'brew install minikube'

Looks like the issue has been fixed by PR#17296.

Hi @uhlhosting  Thank you for your contribution. This has already been addressed by  https://github.com/kubernetes/website/pull/17296  so I will close this.
Please make sure to sign the CLA if you would like to make future contributions.
/assign
/close


Private the types in the bindings by default

This means the user has to opt-in to using rely on internal implementation details!

I'm not sure about this so gonna leave closed for now.


affected/package: flag

What version of Go are you using (go version)?

$ go version
go version go1.19 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?

If am using the follwing flag type to create a flag
flag.StringVar(&outputTypes, "o", "", "Outputs desired output formats.")
flag.StringVar(&outputTypes, "output", "", "Outputs desired output formats.")
flag.Parse()
fmt.Println(outputTypes)

to create a string to check different output conditons for.
One of my possible output conditions shall be indicated by a "#" character, but if I handle over something like
myapp -o #abc outputTypes does not contain "#" after parsing. What you can do though is myapp -o "#abc" or myapp -o \#abcor myapp -o abc\#abc which gives me "#abc", respectively "abc#abc" like intended.
It seems that I cannot use a "#" character as an argument. That a pity. I hoped that in an UTF-8 based language I could use even some runes in strings, but flag.Parse() strips it out if it the first (unescaped) character in the String followed to my flag.

see above.
What did you expect to see?
That the "#" character is usable as an argument
What did you see instead?
it is removed from my argument after calling flag.Parse()

that would appear to be an issue with your shell rather than flag: https://go.dev/play/p/5jTx1rOVsyK
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


cmake error when compiling in visual studio 2022

Hi Guys :D
I tryning to install zxing with cmake and vcpkg, and i have this error :

If someone can help me I appreciate ^^
Thxs !

OK, so ...
I simply add find_package(Threads) before zxing and it compile


v1.13 Backports 2023-04-12

[x] #24788 (@jrajahalme)

:warning: @jrajahalme Had to fix a conflict in the unit tests around RulesOrigin vs DerivedFromRules

[x] #24815 (@aditighag)

Once this PR is merged, you can update the PR labels via:
for pr in 24788 24815; do contrib/backporting/set-labels.py $pr done 1.13; done

or with
make add-labels BRANCH=v1.13 ISSUES=24788,24815

/test-backport-1.13

/test-1.23-4.9

Roses are red
CI is green
Reviews are in
Let's merge


How to get file path of selected file from HTML form

Hello, I have to upload files from the server side of my project, so I'm using addFile() method, but I don't know how to get the file path of the selected file from file input in my HTML form. Please can you help with this?

Hello @tobitech ,

Files added via .addFile() accessible within .find() or .findOne() as any other uploaded file
I didn't get this one:  but I don't know how to get the file path of the selected file from file input in my HTML form, please speak code
Please can you help with this? Sure just explain what do you want to accomplish

Okay. Want to upload file that a user selects from an HTML 5 input file type.
<input type="file" id="productImage">

Creating a database of products that have images as one of their parameters. so in my Template events I would get the file with var file = $('#productImage').get(0).files[0];.

I can just use insert file from the client side and would add it to an Assets Collection of type FilesCollection. But I want the insert to be made on the server side by calling a Meteor method, so I have to use addFile() or writeFile based on your docs. But addFile() needs a path to the file to be inserted.

I have tried this from the client
reader.onload = function(e) {              filePath = e.target.result;              console.log(filePath);          };          reader.readAsDataURL(file);         Meteor.call('insert method', filePath);
and this also
pathURL = URL.createObjectURL(event.target.files[0]);.
but addFile() keeps throwing an error that “No such file or directory”.

So that's the full explanation on what i'm trying to do. I hope you clearly understand

@tobitech thank you for more info.

.addFile(), .load() and  .write() are server only methods
As I see you want to upload files from HTML5 form, you should use .insert() method
Returned result from .insert() method can be added to the product, for example as image field
When user submits the form you will need to .insert() selected image as a first step, then send all other form data via Meteor's method to the server
Upload on form submit code sample
Another option is to upload file right away after it's selected
Upload on select code sample
.insert() method docs

Let me know if this helped you.

That was going to be my last try. I just felt it would be more secured to insert files into collections (including FilesCollection) from the server. Thank. So if I may ask. What kind of files then can one upload with .addFile(), .write()? files in a directory already on the server? What's a sample use case?

Security is widely discussed topic, and this is one of our top priorities. You can find out more in a tickets marked with SECURITY label. TL;TR; This package is secure, and provides a lot of tools (as hooks) to control upload and download flow. But don't take my word for it, ask on Meteor's forum
.write() method - Use it when you have a file as a Buffer
.addFile() method - Use it when you have a file on the local file system and you know its absolute path.

@tobitech any news? Is this one solved?

Yes

Please, support this project by:

Star on GitHub
Star on Atmosphere
Check other packages we have made


Incorrect path in main index definition file

I noticed that there is an error in the following file: https://github.com/ManuCutillas/ng2-responsive/blob/master/src/index.d.ts
The import on line 8 should reads:
import { ResponsiveWindow } from './responsive-css/responsive-window';
but it should be:
import { ResponsiveWindow } from './responsive-window/responsive-window';

This is listed correcting in https://github.com/ManuCutillas/ng2-responsive/blob/master/src/index.ts

Thanks! ; )
Fixed in github. Go to update npm package.

Fixed npm package and update to 0.3.2.


Add Environment Variable for default http/https ports

Hi,
i would love to see if the default ports (8080 / 4443) can be defined via an env variable (e.g. to 80 / 443).
Having that, it would simplify things when running the container on its own IP via Macvlan.
Currently, Docker does not support port re-mapping/exposing of specific ports when driven by a Macvlan.
Setting default to 80/443 would allow me to add A-Records on my DNS server which reflect back to reverse proxy directly instead to be exposed to the internet before. But for this, the NPM needs to listen on 80/443 as i can't define ports on my DNS server (Pi-hole). Please correct me if i'm wrong.
This setup works already fine with the JC21 container but for some reason i got quite a lot of CPU load with this container which i therefore prefer yours.
Bets regards,
Christian

This is discussed here: https://github.com/jlesage/docker-nginx-proxy-manager/issues/117