add more tests

Do not merge yet.

Codecov Report
Patch coverage has no change and project coverage change: +0.28 :tada:

Comparison is base (7de6318) 59.38% compared to head (56309d4) 59.67%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #164      +/-   ##
============================================
+ Coverage     59.38%   59.67%   +0.28%
- Complexity      530      532       +2
============================================
  Files           108      108
  Lines          2073     2073
  Branches        218      218
============================================
+ Hits           1231     1237       +6
+ Misses          759      755       -4
+ Partials         83       81       -2

see 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Do not merge yet.

converted your pull request to a draft pull request to prevent accidental approval and merging. Feel free to convert back to a normal pull request when all is good!


Error when loading project

I get following errors on v0.10.0
Error: Failed to require "chrome-js" from "noflo-noflo-image/components/GetHistorgram.js"
Error: Failed to require "chrome-js" from "noflo-noflo-image/components/ExtractSaturation.js"

I see this issue is already reported here #15, however that was a year ago.


ADD: PayPal commit

Summary
The main purpose of this Pull Request is to manage commit status of the transaction - https://developer.paypal.com/docs/checkout/reference/customize-sdk/#commit.
This will expand the possibilities and make possible the next feature - https://developer.paypal.com/docs/checkout/integration-features/confirmation-page/.

Hi @pabloai,
I did the same as the intent attribute.
Maybe I didn't fully understand the idea. According to the code - PayPal configuration this is the place where such information should be stored, it is also present in unit tests.
Tell me in this case, where is it better to store this attribute?
Thanks in advance

Hi again @dkatashev,
The intent property is a special case because we receive it from the backend in the response from the paymentMethods endpoint. On this case it should get to the component in the props so there's no need of any special handling.
That being said, if I'm not mistaken this commit property is used in the Express Checkout flow. Is that what you're aiming to achieve? Would you do the call to the /payments/details endpoint later in the flow?
Just trying to understand the use case. Thanks!

Hi @pabloai,
Thank you for your detailed explanation.
Yes, you're right. I want to make possible PayPal Express Checkout Flow.
I guess I'll be able to initiate a component with that option first. Then on the server get information from PayPal about the customer (payer) based on PayPal Order ID. Then, do the call to the /payments/details to confirm the order.
Do you see any problems to make it possible?
I'll be very grateful for your feedback.

Merging this PR.
Thanks for your collaboration @dkatashev !


word vector

sorry, but how can I get data/.vector_cache/glove.6B.100d.txt?

https://nlp.stanford.edu/projects/glove/


ockam project ticket should return a proper error message

Observed behavior
Reset your project
ockam reset -y

and try to create a project ticket
❯ ockam project ticket                                                                                OCK500

  × unknown project default
  help: Please report this issue, with a copy of your logs, to https://github.com/build-trust/ockam/issues

  Version 0.90.0, hash: 78760ecfc8a9fb5b49b785f5e1a5ce0560e3bce3

Steps to reproduce
An error 500 is return but the problem is simply that the current user must enroll with ockam enroll first.
Expected behavior
Return a user error when a project does not exist for an ockam project command and suggest to run ockam enroll first
Ockam Version
0.90.0

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

Hi @etorreborre,
I just want to clarify. Do you want a PR to handle any relevant project subcommand where the project name is not found, or just the ticket subcommand?
Thanks!

Hi @DerekStrickland I haven't tested the other project commands and it is indeed probable that they return the same error. So please address them all :-).

I want to work on this. Being a new contributor can anyone tell me what files to look into?

@guha-rahul Here are some good starting points:

https://github.com/build-trust/ockam/blob/develop/implementations/rust/ockam/ockam_command/src/project/ticket.rs
https://github.com/build-trust/ockam/blob/develop/implementations/rust/ockam/ockam_command/src/enroll/command.rs#L42

@guha-rahul I just assigned you this issue. Please let us know if you have any questions as you explore. You can also ask questions on the contributors discord https://discord.gg/RAbjRr3kds

Thanks , will look into them right away!


Catalog filter dropdown line

The current filter has the second line only extending half way down the filter:

it should extend the length of the filter with 32px $spacing-07:

At different breakpoints, add spacer elements as necessary.

Hey team! Please add your planning poker estimate with ZenHub @alisonjoseph @andreancardona @francinelucca @jdharvey-ibm


Dark mode

Creating this issue to write down everything we may need to tweak to enable dark mode:

[ ] BG grants logo
[ ] Hero image asset, may need changes?
[ ] Community grants Apply button BG color
[ ] Community grants shape assets, may need changes?
[ ] Community grants shape text

Hello!
Made a dark mode version in Figma with some of the above: the logo, darker icons, CTA section (some doubts about it) + general color scheme (also for buttons)- as a suggestion, it doesn't need to be exactly as this if it's too complex.
Gonna make a darker version for the hero image as well.
And about the CTA section, think I would do either one of these two things:

Will get back to you with the hero image, let me know if you have any doubts or if some parts don't make a lot of sense!

And this would be the hero image: (just darkened the whole thing with a dark blue mask, and left the ethereum bubble a bit brighter than the rest). File is in the Figma doc 👌🏻


print: false column option is not working

Expected Behavior
Table column should be hidden when printing if its print option is set to false (see docs)
Current Behavior
Setting print option to false has no effect, the column is visible when printing.
Steps to Reproduce (for bugs)
This bug can be reproduced on codesandbox where all examples are located.

Go to https://codesandbox.io/s/github/gregnb/mui-datatables mentioned in the README
Select "Hide Columns Print" example
See how the table has 4 columns: Modified Title Label, Location, Age, Salary
Click "Print" button
See all 4 columns present in the print view, while according to the example source both Location and Age columns should've been hidden.

Your Environment
I am pretty sure this is related to this issue: https://github.com/gregnb/mui-datatables/issues/1886#issuecomment-1104304846, and was introduced during the move to tss-react: the CSS responsible for hiding the columns isn't properly created, causing the bug.

Tech
Version

MUI-datatables
4.2.0

browser
Chrome

The problem still persists


LocalAudioTrack.AudioFrameReady event doesn't called.

I'm having trouble transferring audio.
At the Peerconnection on the receiving side, when checking the transferred data (frame.audioData) in the RemoteAudioTrack.AudioFrameReady event , its values are all zero.
At the PeerConnecrion on the sender side, I tried to check the status of the audio capture, LocalAudioTrack.AudioFrameReaqdy event never called.
I think I created a local audio track following the way of C# tutorial, but is there something wrong?
[Local Side]
private AudioTrackSource _localAudioSource = null;
private LocalAudioTrack _localAudioTrack = null;

private async void CreateLocalAudioTrack(){
	LocalAudioDeviceInitConfig devconf = new LocalAudioDeviceInitConfig
	{
		AutoGainControl = true
	};
	_localAudioSource = await DeviceAudioTrackSource.CreateAsync(devconf);

	LocalAudioTrackInitConfig trackconf = new LocalAudioTrackInitConfig
	{
	    trackName = "localTrack"
	};
	_localAudioTrack = LocalAudioTrack.CreateFromSource(_localAudioSource, trackconf);
	_localAudioTrack.AudioFrameReady += _localAudioTrack_AudioFrameReady;
}

private void _localAudioTrack_AudioFrameReady(AudioFrame frame)
{
	Console.WriteLine("_localAudioTrack_AudioFrameReady");　// <= never called
}

[RemoteSide]
private void OnRemoteAudioTrackAdded(RemoteAudioTrack track)
{
    track.AudioFrameReady += Track_AudioFrameReady;
}

private int counter = 0;
private void Track_AudioFrameReady(AudioFrame frame)
{
    ++counter;
    if (counter >= 100)
    {
        byte[] data = new byte[8];
        Marshal.Copy(frame.audioData, data, 0, 8);
        string dump = "";
        for (int i = 0; i < 4; i++)
        {
            dump += data[i].ToString("X2") + " ";
        }
        Console.WriteLine(dump);  // <= Always "00 00 00 00 00 00 00 00"

        counter = 0;
    }
}

Because the official WebRTC samples and NAudio library can capture audio data correctly, I think there is no problem in audio device environment.
I would appreciate your help.

LocalAudioTrack.AudioFrameReady event never called

It's by design, and we can't really do anything about it. The underlying Google code doesn't ever raise this event, therefore we cannot either.
Logged #600 because I can't find a reference in the docs about this; this should have been made prominently clear, sorry about that.

when checking the transferred data (frame.audioData) in the RemoteAudioTrack.AudioFrameReady event , its values are all zero.

I've heard of other users having similar issues, but never actually reproduced that myself. The fact the callback is invoked at all is already a good sign; it means the connection is established. I would double-check things like the transceiver direction (send/recv) when the callback is called. There's unfortunately no good diagnostic statistics for this kind of issue, so not sure what else to suggest. @fibann any idea maybe?

Thank you for your quick reply.

Transceiver Direction
I checked the transceiver direction at AudioFrameReady event of the receiver side .
[Tranceiver Direction Check (RemoteSide - Track_AudioFrameReady event)]

	int counter = 0;
	private void Track_AudioFrameReady(AudioFrame frame)
	{
	    ++counter;
	    if (counter == 100)
	    {
	        Console.WriteLine($"Transceiver Direction = {_remotetrack.Transceiver.NegotiatedDirection}");
	        counter = 0;
	    }
	}

**[Result]**
Transceiver Direction = ReceiveOnly
Transceiver Direction = ReceiveOnly
.........
.........

Aditional question
The AudioFrameReady event continues to be called on the receiver side, even if the sender side's PeerConnection is destroyed.
Is it right operation, or strange?

Please let me know if you have any other point to check. I'll try to check It with my lacked skills.

@koh1019 what version of MR-WebRTC are you using? On what platform are you capturing audio (Windows or other, UWP or Win32, Unity or standalone)?
Note that released versions (including 2.0.x) have known issue #192 which involves silent audio when capturing in a Win32 application. This seemed to be a problem with the legacy ADM and has been changed by https://github.com/microsoft/MixedReality-WebRTC/commit/e839cd78859aa21f858839732e0f4f40d7178b20 which sets ADM 2 as the default. OTOH there have been reports of ADM 2 not capturing any sound where the legacy ADM works (#573). So depending on what version you are using, you might fix your issue by either using a version past e839cd78859aa21f858839732e0f4f40d7178b20 or forcing the legacy ADM at startup (see https://github.com/microsoft/MixedReality-WebRTC/issues/573#issuecomment-690963138).

Thank you for your reply, and I'm sorry for the lack of information.
[My environment]
Windows10 Pro 64bit
Windows forms Application (.NET Frame work 4.8) (C #)
MR-WebRTC Version 2.0.2 (from Nuget package manager)
I tried to changed 'UseAudioDeviceModule' to 'Legacy', but it didn't show up that property in Intellisense.

By pressing F12 to show the metadata of Legacy class, I found the information below.
[VS2019 Library metadeta]
#region Assemply Microsoft.MixedReality.WebRTC, Version = <IP_ADDRESS>, Culture = neutral, PublicKeyToken = null
 C: \ develop \ projects \ RemoteDiagnosis \ RemoteDiagnosisSolution \ packages \ Microsoft.MixedReality.WebRTC.2.0.2 \ lib \ netstandard2.0 \ Microsoft.MixedReality.WebRTC.dll
#endregion

Is it not referenced correctly?

See previous comment - at the moment this feature is not in any of the released packages, but only on the master branch. You will have to checkout the master branch and build MR-WebRTC from source (guide here).

Thank you for your advice.
I built it from the source code of the master branch and checked the operation, and it worked fine.
I am currently trying to develop an SFU/MCU server.
I'm looking for a server side mechanism to transfer Audio Data received from client to other peers, and looking forward to ExternalAudioTrackSource (# 345).

Glad to know that worked!


Add python extra for mkdocstrings

mkdocstrings 0.19.0 requires that the python extra be explicitly installed.
Also, to make tests pass with this version, we needed to do #51 as well and drop support for Python 3.6.
Closes #55
Closes #51

Codecov Report

Merging #56 (e9b0e3b) into main (1b44bc5) will decrease coverage by 3.8%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            main     #56     +/-   ##
=======================================
- Coverage   97.6%   93.8%   -3.9%
=======================================
  Files         13      13
  Lines        514     517      +3
=======================================
- Hits         502     485     -17
- Misses        12      32     +20

Impacted Files
Coverage Δ

erdantic/version.py
80.0% <0.0%> (-20.0%)
:arrow_down:

erdantic/typing.py
78.8% <0.0%> (-18.9%)
:arrow_down:

erdantic/exceptions.py
90.6% <0.0%> (-7.0%)
:arrow_down:

erdantic/base.py
97.2% <0.0%> (+0.1%)
:arrow_up:


Can you consider supporting Chinese services?

Can you consider supporting Chinese services?

You would need to tell me the address of the login page that opens when you log into your MyVAILLANT account. For Germany, it looks like this: https://identity.vaillant-group.com/auth/realms/vaillant-germany-b2c/protocol/openid-connect/auth?.... I need the vaillant-germany-b2c part for Chinese services.

Hello, @signalkraft , i have got the below log in  for chinese services. hope this can make some help and you can help add china service.

本文件内容为 https://vicapi.vaillant.com.cn/auth/oauth/token?grant_type=password 的请求抓包详情，供您分析和定位问题。

请求内容 Request:

POST /auth/oauth/token?grant_type=password HTTP/1.1
Host: vicapi.vaillant.com.cn
Accept: application/json, text/plain, /
version: 06.00.001
Authorization: Basic YXBwLXc6aW90MlExWDNY
Accept-Language: zh-CN,zh;
Accept-Encoding: gzip, deflate, br
Content-Type: application/x-www-form-urlencoded
Content-Length: 93
User-Agent: VaillantPlus/230329288 CFNetwork/1490.0.4 Darwin/23.2.0
appkey: 9109fb3ea6c04cf1be34c71ff82a51ec
Connection: keep-alive
Cookie: acw_tc=0b68a82217289757409397890ef5890d16ac66b91cb333c455181baa3f05d4
username=&password=&appkey=9109fb3ea6c04cf1be34c71ff82a51ec&scope=server


Post preview failed when no changes are made.

Thanks @TylerBarnes on rewriting the WP post preview, it's now functional in most situations but notice it still failed on certain situation.

Generating a preview on draft post with no content changes will return with error in WordPress
Preview after "Save Draft" will display Gatsby cloud error message in WordPress
Generating a second preview on draft post right after a successful preview will return with error in WP.

Gatsby error message in WP:  "The Gatsby Preview webhook was not successfully called."
Post status: Draft post
Note: Suspect above three situations have similar condition where it return a preview, if there are no content changes to a previous preview built.
On latest version of gatsby-source-wordpress-experimental: 3.2.0 and WP-Gatsby: 0.6.6

Thanks @hazuremon , that's really helpful! Can you share your Gatsby Cloud dashboard URL with me so I can take a look at your logs? It will be private to anyone who tries to visit the link but I have magic Gatsby powers so I can get in 😆

https://www.gatsbyjs.com/dashboard/bdd12c57-3dec-416e-80f8-a407b740e012/sites/8fb53603-2435-4182-b550-257ee3e524bc/deploys

Thanks Tyler!
To provide a bit more information, these preview will build successfully in Gatsby Cloud, but WordPress will display Gatsby error message before the preview has finished building.

I have updated the issue, as it was a bit unclear after reading it again.

@hazuremon I've identified the problem and we'll be releasing a fix tomorrow. See https://github.com/gatsbyjs/wp-gatsby/pull/64 for more info :)

https://github.com/gatsbyjs/gatsby-source-wordpress-experimental/pull/323 improves the draft preview experience as well and should be released tomorrow.

A fix for this is published in<EMAIL_ADDRESS>Please let me know if you see any similar issues, or if your problem wasn't fixed and we can re-open this.

Actually, I guess the fix for this is in WPGatsby v0.6.7 :) but both new versions have improvements to new draft previews.

Thank Tyler, thanks for fixing it quickly. It fixes all the conditions above, thank you!

Woo :D that's great to hear! Let me know if you run into any other problems. I feel like preview is getting really solid so if you find anything else I want to squash it right away haha


Split components into multiple repositories

i have to chunk orm, odm and other components into separate repositories, since they are loosely coupled no big issues are expected

dup


Use gloria in default config

Also update source catalog to DR5.

Actually @mcoughlin, I updated the secret key and pointed the default config at gloria. Please merge once&if green.

Ah, I'll need to re-run the CI once EDR3 ingestion is over on G - will ping you @mcoughlin then.


[request]: CLI possibilities

Having a CLI utility for postwoman is likely to ease the workflow.
(NB: Even postman has got a CLI utility, newman)

Perhaps we should make this a separate package like the repository?

Cloing thi iue


The last rocker switch skin is an error?

According to entity inspector:
in models/bull/buttons/rocker_switch.mdl:
[1] bull/buttons/rocker_switch
[2] bull/buttons/rocker_switch_red
[3] bull/buttons/rocker_switch_blue
[4] bull/buttons/rocker_switch_green
[5] bull/buttons/rocker_switch_orange
[6] ___error
Selecting "Skin 5" ingame produces the missing texture graphic.

Where are the model sources?

I'm not sure that I fully understand the question?
I've downloaded the game and am loading up addons and getting back on meta construct #1.

Button input output?

No, the files used to compile the model

Hm, perhaps here ?

No, has to be decompiled

Could try out Crowbar (or Hooch's decompiler), I can't be bothered going over decompiling atm :/

Yeah, I know how to decompile. I've just been busy with my own modelling project

Looked in the mdl file and the missing material is rocker_switch_dark. I can just change the number of skins to 5 instead of 6.


View my sessions.

I can't see a scheduled game after I've joined it.
I'd like to know how to find them or Get a feature that will let me list the current games scheduled for a server.

If you're in the "My Games" page, you should see games you've created or signed up for.
If you're in "Upcoming Games", you'll see games that have not passed the scheduled game date.


Fix Issue 14

https://github.com/WtfJoke/rtc2git/issues/14
The script was throwing errors like "No repositories found based on path. Specify repository URI." and "Problem running 'discard': Unable to find workspace. Specify workspace."

@ljhaywar I was so rude to create a pull request with your changes on my own. Hope you dont mind :)

Not at all.  I'm new to Git and am still figuring out the terminology, so let me know if you need me to do anything.

No worries, Im still a git amateur as well :stuck_out_tongue:

@ljhaywar Could you please test if your use case still works with my current development-branch?


Split challenge generation & challenge verification

For ease of automation, please split the generation of a challenge & the verification call.
Workflow example.

manuale authorize --generate-only --method dns example.com >somefile
grep TXT somefile |  $custom_update_DNS --append example.com
manuale authorize --verify-only --method dns example.com

manuale as its name suggets... 😄
I use this shell function as a background job: https://github.com/szepeviktor/debian-server-tools/blob/master/security/cert-update-manuale-CN.sh#L31-L49

You may do something similar with your API-based DNS update.

Yes, I already have tooling to do the DNS update, but right now I need something that parses the manuale output and tells it to continue after the DNS change is live. Just want to cleanly split it up.

@szepeviktor since upstream seems to have stopped making any changes, interested in forking this fully?

I am content with the current state of manuale.

Hey,
As you might gather from the project name, I'm surprised to see that people are using this for automated issuance at all. My intention was to create an easy, guided interface for a real person who wants to do things by hand and/or is issuing a single one-off certificate. The only reason authorize and issue are separate commands at all is that a long time ago there wasn't a method to detect an existing authorization for a domain, and authorizations lived longer than certificates. Now that this isn't the case, I've been thinking of rolling the commands into one.
I'm not completely opposed to this idea, but you may be better served by another client designed for the purpose or a custom fork. For example, acme.sh has built-in DNS provider integrations and lets you drop in your own DNS authorization script if needed. Maybe once I get around to doing an ACMEv2 overhaul there could be a single high level guided issuance command and lower level commands for working with authorizations and orders that could be used in automated scripts.

Where I think it's also going to be more relevant is cases that request multiple validations (ACME-08 has renamed them from challenges) be completed, and upcoming validations like ip-01.
As for other clients, I tried many of them before winding up Manuale. My usage case is not fully automated, but just semi-automated by wrapping Manuale, to authorize & issue via DNS, then securely ship the key+cert to new hosts (they don't even exist yet when the issuance is done).
I'd agree with having a high-level command that does authorize & issue together, which really wraps around lower-level commands (and are also exposed).
Manuale is the closest there is to a good competing Python Library other than Certbot's ACME module. Many others either just wrap certbot's ACME, or provide a minimal implementation.

I am open to a PR :)

https://github.com/schors/perkele/ perkeLE is a fork of ManuaLE

New in perkeLE Support for HTTP validation. (In fact, that's the only validation method supported).
Authorization is separate from certificate issuance. Authorizations last for months on Let's Encrypt: there's no need to waste time validating the domain every time you renew the certificate.
New in perkeLE The authorization can be divided into two parts - get authorization, and check validation. You can distribute verification files manualy.

I like the name!

@f-andrey why was DNS validation dropped?


GPS needs more speed!

Some tweaks to the GPS info app:

Speed is shown on the screen
"Animation" on the "Waiting for GPS" screen
With 10px padding feels a bit nicer :)

Nice - thanks!


feat: configurable access to default pool

Fixes: #31

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

33 of 43 (76.74%)  changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 86.7%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

bases/renku_data_services/data_api/config.py
0
1
0.0%

components/renku_data_services/resource_pool_adapters/adapter.py
8
17
47.06%

Files with Coverage Reduction
New Missed Lines
%

components/renku_data_services/resource_pool_adapters/adapter.py
1
86.79%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
2399

Relevant Lines:
2767

💛  - Coveralls


Use go-log.v1

Btw. my PR (https://github.com/src-d/go-log/pull/8) which fixes filename and linenum for Errors is not merged, yet.


Remove datasource in procedure task

Purpose of the pull request
close #13197
Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #13198 (99396cc) into dev (43e6ac3) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #13198      +/-   ##
============================================
- Coverage     39.37%   39.34%   -0.03%
+ Complexity     4278     4272       -6
============================================
  Files          1066     1066
  Lines         40479    40450      -29
  Branches       4657     4642      -15
============================================
- Hits          15937    15916      -21
+ Misses        22755    22744      -11
- Partials       1787     1790       +3

Impacted Files
Coverage Δ

...source/api/plugin/DataSourceProcessorProvider.java
0.00% <0.00%> (ø)

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.00% <0.00%> (-2.78%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

.../dolphinscheduler/plugin/task/datax/DataxTask.java
36.62% <0.00%> (-0.26%)
:arrow_down:

... and 18 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ruanwenjun  To avoid using connections from thread pool? create a separated connection in each task?

@ruanwenjun To avoid using connections from thread pool? create a separated connection in each task?

Yes, you can see the issue description.

@ruanwenjun Can we add this pr as an improvement? Will this change affect the existing usage?

@ruanwenjun To avoid using connections from thread pool? create a separated connection in each task?

Yes, you can see the issue description.

Is it possible to create too many connections if there are lots of tasks?

Yes, this will need to create connection for each processdure task, but right now, I don't find other solution. Since the temporary table is bound to the session, if we don't close the session the temporary will always exist.

@ruanwenjun Can we add this pr as an improvement? Will this change affect the existing usage?

Ok, I add improvement table, this PR will not affect the current usage

@ruanwenjun To avoid using connections from thread pool? create a separated connection in each task?

Yes, you can see the issue description.

Is it possible to create too many connections if there are lots of tasks?

Yes, this will need to create connection for each processdure task, but right now, I don't find other solution. Since the temporary table is bound to the session, if we don't close the session the temporary will always exist.

If we can't find another solution, then current  is the best one. Too many connections for a DB is a disaster absolutely, dropping a temporary table is just a flaw.

@ruanwenjun To avoid using connections from thread pool? create a separated connection in each task?

Yes, you can see the issue description.

Is it possible to create too many connections if there are lots of tasks?

Yes, this will need to create connection for each processdure task, but right now, I don't find other solution. Since the temporary table is bound to the session, if we don't close the session the temporary will always exist.

If we can't find another solution, then current is the best one. Too many connections for a DB is a disaster absolutely, dropping a temporary table is just a flaw.

This is a bug, will cause the procedure task cannot use, the temporary table is shared by multiple task,  if you create a template task in the procedure task, you will get an error, since there might already exist a temporary table with the same name created by another task.

If user drop it manually, is it OK?

If user drop it manually, is it OK?

It's OK for the temporary table if user drop in the end of procedure task, but we cannot make sure all user will drop their temporary table in their procedure task.


compiler ok, no problem icon! green light!

Hi, when you make compiler or check for error, maibe is good to have some green light icon, that everything is good, or ok.
in my Terminal is all dark, so i need to see very close to know what terminal says.

the green icon below is added by photoshop to show an example

Addressed in 0.7.0 :)


update nftbase.md

add Use of IPFS, Filecoin and Libp2p
edit how to contribute

The license policy has been added and the source has been changed to public.

I'm merging these changes, but there's still no source code. Your project source code must be MIT licensed in order to qualify for the grant.


Refactor endpoint handling

Short Description

refactor endpoint handling by unifying WSDLEndpoints and unused REST endpoints
adapt repositories to use Spring JPA

Looks (almost) good to me

Solves https://github.com/OpenTOSCA/container/issues/148


VMware Content Library refresh

Required for https://github.com/ManageIQ/manageiq-providers-vmware/pull/444

@miq-bot add_labels enhancement, providers, providers/vmware/infra

@jameswnl if we're defining this at the core infra_manager can we also define the persister collection here https://github.com/ManageIQ/manageiq/blob/master/app/models/manageiq/providers/inventory/persister/builder/infra_manager.rb ?


Snapshots sometimes captured at smaller size

I'm sometimes seeing failed tests where the snapshots look to be the same, but at different resolutions.
Has anyone else experienced this?

index.html
    <canvas id="canvas" width="1920" height="1080" style="width: 500px;"></canvas>

test.spec.js
// ...  code to render image on canvas.
cy.get("#canvas").matchImageSnapshot()

I have experienced this issue intermittently when switching between cypress run and cypress open.  Were both these screenshots taken in the same mode?
https://github.com/cypress-io/cypress/issues/1309 might also be related.

They were taken with the different modes for me too.
I did update snapshots with run
then using run again passed
but using open results in the failure.
Have you tried any workarounds?

I haven't tried this myself, but this seems like a promising workaround: https://github.com/cypress-io/cypress/issues/495#issuecomment-472723434.

That looks great, shall try it out!

This is also happening to me too.
run and open produces different snapshots.

@crosscompile that "workaround" does not work for me. Did it work for you?

@rodoabad I haven't had a chance to try it out yet.

I tried it quickly and it didn't work for me

Did anyone come up with a solution to solve this? The workaround did not work for me either

This is hugely problematic for us. Can we discard that this is an issue with this plugin though?

Yep happy to close in favour of https://github.com/cypress-io/cypress/issues/33244 if this a cypress cy.screenshot issue.

This obviously still affects this plugin though. Feel free to re-open if this issue is a useful way to track progress.

@PTaylour are you sure that's the right issue? It's a 404.


[Links] Magento integrations with Adobe products

Not sure where it's better to place these projects but Adobe started to create a couple of connectors with their products and Magento2:

https://github.com/adobe/commerce-cif-magento-graphql
https://github.com/magento/aep-launch
https://github.com/magento/adobe-stock-integration

They might be useful for someone 😌

Makes sense! Feel free to add them!

Now there are several adobe products that are integrated with adobe.  For example Progressive Web App (PWA) for Magento 2 is one the adobe product provided by them.

Here is the link to Magento integrations with Adobe products. You will find multiple extensions for integrations.

@shopaalogics This is a link to a list of products of your company. This Repo is not for marketing purposes...


SQuAPP Undergoing Updates

I see this error when running the app both locally and through the link you shared. When will this app be updated?
Error:
We apologize for the inconvenience, but our application is currently undergoing updates.
Please check back later for new content.

If you want to initialize your local setup, change the "READY" variable to TRUE in the global.R file and run it on RStudio. However, I added this message because a few people reported bugs that broke the code. In addition to these bugs, one of the UI packages has been updated and no longer works with Shiny. Please try running with READY=TRUE and let me know if you encounter any of these issues.

The app has been updated and includes several bug fixes and error-handling messages. It should be more reliable than ever, but unforeseen issues may still arise. The online version of the app has been redirected to the updated version, so feel free to access it now. If you prefer to use the app locally, you can update your version to 0.3.
I am closing the issue for now, but if you experience any further problems, please open another issue so I can assist you.


try fixing build on msys2 gcc 14

https://github.com/xmake-io/xmake-repo/actions/runs/9031275001/job/24817204958?pr=4001
https://packages.msys2.org/package/mingw-w64-x86_64-gcc

thanks


Modify DG for comment function

Codecov Report
Base: 75.67% // Head: 75.67% // No change to project coverage :thumbsup:

Coverage data is based on head (606ab5c) compared to base (863d14b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #311   +/-   ##
=========================================
  Coverage     75.67%   75.67%
  Complexity      719      719
=========================================
  Files           108      108
  Lines          2064     2064
  Branches        255      255
=========================================
  Hits           1562     1562
  Misses          439      439
  Partials         63       63

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


KYLIN-4407 fix IT

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Pull Request Test Coverage Report for Build 5891

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 27.323%

Totals

Change from base Build 5890:
0.0%

Covered Lines:
24350

Relevant Lines:
89120

💛  - Coveralls


Cannot load Phi3 with latest (0.2.84) release

Prerequisites

[x] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[x] I carefully followed the README.md.
[x] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[x] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Expected Behavior
Attempted to load "Phi-3-mini-4k-instruct-q4.gguf" with latest (0.2.84) release. Expected the load to be successful
Current Behavior
Load fails:
<snip>
llama_model_loader: - type q5_K:   32 tensors
llama_model_loader: - type q6_K:   17 tensors
llama_model_load: error loading model: error loading model hyperparameters: key not found in model: phi3.attention.sliding_window
llama_load_model_from_file: failed to load model
Traceback (most recent call last):
  File "C:\Users\riedgar\source\repos\guidance\llama_load_error.py", line 10, in <module>
    llama_model = llama_cpp.Llama(downloaded_file, logits_all=True, verbose=True, n_ctx=4096)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\riedgar\AppData\Local\miniconda3\envs\guidance-312\Lib\site-packages\llama_cpp\llama.py", line 372, in __init__
    _LlamaModel(
  File "C:\Users\riedgar\AppData\Local\miniconda3\envs\guidance-312\Lib\site-packages\llama_cpp\_internals.py", line 55, in __init__
    raise ValueError(f"Failed to load model from file: {path_model}")
ValueError: Failed to load model from file: C:\Users\riedgar\.cache\huggingface\hub\models--microsoft--Phi-3-mini-4k-instruct-gguf\snapshots\999f761fe19e26cf1a339a5ec5f9f201301cbb83\Phi-3-mini-4k-instruct-q4.gguf
Exception ignored in: <function Llama.__del__ at 0x0000029C3603D4E0>
Traceback (most recent call last):
  File "C:\Users\riedgar\AppData\Local\miniconda3\envs\guidance-312\Lib\site-packages\llama_cpp\llama.py", line 2089, in __del__
    if self._lora_adapter is not None:
       ^^^^^^^^^^^^^^^^^^
AttributeError: 'Llama' object has no attribute '_lora_adapter'

This was working with previous llama-cpp-python versions.
Environment and Context

Physical (or virtual) hardware you are using, e.g. for Linux:

i7 13800-H (for Windows)
For Linux, using GitHub-provided Runner machines

Operating System, e.g. for Linux:

Windows and Linux

SDK version, e.g. for Linux:

Python 3.12.3

Failure Information (for bugs)
Please help provide information about the failure if this is a bug. If it is not a bug, please remove the rest of this template.
Steps to Reproduce
import llama_cpp

from huggingface_hub import hf_hub_download

repo_id = "microsoft/Phi-3-mini-4k-instruct-gguf"
filename = "Phi-3-mini-4k-instruct-q4.gguf"

downloaded_file = hf_hub_download(repo_id=repo_id, filename=filename)

llama_model = llama_cpp.Llama(downloaded_file, logits_all=True, verbose=True, n_ctx=4096)
assert llama_model is not None

This ends up with the stacktrace above.

The attribute error should be fixed here #1630 , but now the error loading model hyperparameters: key not found in model: phi3.attention.sliding_window persists for me with 0.2.83.
In this ollama issue it's argued that it's just the gguf Phi model needing to be updated, but not sure.

Phi3 now works with 0.2.85
Note : I did have to recreate gguf files since older versions did not have some metadata.
@ltoniazzi  @riedgar-ms maybe try downloading the gguf again
llama_model_loader: loaded meta data with 36 key-value pairs and 197 tensors from <snipped>/Phi-3-mini-128k-instruct.gguf (version GGUF V3 (latest))
llama_model_loader: Dumping metadata keys/values. Note: KV overrides do not apply in this output.
llama_model_loader: - kv   0:                       general.architecture str              = phi3
llama_model_loader: - kv   1:                               general.type str              = model
llama_model_loader: - kv   2:                               general.name str              = Phi 3 Mini 128k Instruct
llama_model_loader: - kv   3:                           general.finetune str              = 128k-instruct
llama_model_loader: - kv   4:                           general.basename str              = Phi-3
llama_model_loader: - kv   5:                         general.size_label str              = mini
llama_model_loader: - kv   6:                            general.license str              = mit
llama_model_loader: - kv   7:                       general.license.link str              = https://huggingface.co/microsoft/Phi-...
llama_model_loader: - kv   8:                               general.tags arr[str,3]       = ["nlp", "code", "text-generation"]
llama_model_loader: - kv   9:                          general.languages arr[str,1]       = ["en"]


Adding .png to the list of allowed extensions in the im2rec.py

Description
(Brief description on what this PR is about)
Checklist
Essentials

[x ] Passed code style checking (make lint)
[x ] Changes are complete (i.e. I finished coding on this PR)
[x ] All changes have test coverage
[x ] For user-facing API changes, API doc string has been updated.
[x ] To my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

Adding .png as allowed in default extensions in the im2rec.py argument parser. Tested by running im2rec on a set of png images and was able to generate rec files.

Comments

N/A

Closing due to moved to https://github.com/apache/incubator-mxnet/compare/master...piiswrong-patch-2?quick_pull=1


No .npy file found

I have installed all the libraries but it keeps giving me this:
Traceback (most recent call last):
File "train.py", line 160, in
if gs > hp.num_iterations: break
File "/usr/lib/python2.7/contextlib.py", line 35, in exit
self.gen.throw(type, value, traceback)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py", line 1014, in managed_session
self.stop(close_summary_writer=close_summary_writer)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py", line 839, in stop
ignore_live_threads=ignore_live_threads)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/coordinator.py", line 389, in join
six.reraise(*self._exc_info_to_raise)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/queue_runner_impl.py", line 257, in _run
enqueue_callable()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1279, in _single_operation_run
self._call_tf_sessionrun(None, {}, [], target_list, None)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1429, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.UnknownError: exceptions.IOError: [Errno 2] No such file or directory: 'mels/LJ024-0008.npy'
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/script_ops.py", line 209, in call
ret = func(*args)
File "/home/alansmithans/dc_tts/data_load.py", line 113, in _load_spectrograms
return fname, np.load(mel), np.load(mag)
File "/home/alansmithans/.local/lib/python2.7/site-packages/numpy/lib/npyio.py", line 422, in load
fid = open(os_fspath(file), "rb")
IOError: [Errno 2] No such file or directory: 'mels/LJ024-0008.npy'

Sorry I misread the prepo. I thought running train.py would first run prepo automatically.


Logdna-agent v3 fail to start with "thread 'main' panicked at 'called Option::unwrap() on a None value'" error

Hi LogDNA team, im seeing logdna-agent crash with the following error on bootup:
[2021-09-10T17:00:36Z INFO  fs::tail] initialized "regional-snapshot-86dd4dcf77-t6hjd" with offset 0
thread 'main' panicked at 'called Option::unwrap() on a None value', /opt/rust/cargo/git/checkouts/logdna-rust-da7bb1a0a2df3f84/d7adef1/src/request.rs:274:49
I am not sure if this is related, but further investigating, i find that one of the log file that logdna-agent tries to tail has a log line but in the logdna-agent log it seems to think it is empty:
[2021-09-13T02:15:01Z INFO  fs::tail] initialize event for file "image-rest-server-5c5b957fff-qsj46", target "/var/log/at/image-rest-server-5c5b957fff-qsj46"
[2021-09-13T02:15:01Z INFO  fs::tail] initialized "image-rest-server-5c5b957fff-qsj46" with offset 0

But looking at the size of the file
[logdna@logdna-agent-atracker-lsgwb at]$ wc -l  image-rest-server-5c5b957fff-qsj46
1 image-rest-server-5c5b957fff-qsj46

which indicates there is a single line
This is observed in logdna-agent v3.1 and later

@jaszxg We think we've identified where this panic was coming from. I don't know if you still have the reproduction environment available, but can you confirm that the LOGDNA_INGESTION_KEY environment variable was set and the LOGDNA_HOST variable was either set or completely unset (not set to empty) when you ran the agent?

Yes they were set, in fact the workaround we used was to add the extensionless files to the inclusion rules, which worked

Sorry I did not update with latest findings, but the issue turn out to be that after Logdna-agent v3.1, a regression was introduced such that it no longer supports extensionless files. This was acknolwedged by the logdna team to be a bug.


Apple kern tables are wrongly compiled

The Apple kern table (with uint32 version) has different subtable format than the OpenType one.  We ignore this when compiling kern subtables.
https://developer.apple.com/fonts/TrueType-Reference-Manual/RM06/Chap6kern.html
https://www.microsoft.com/typography/otspec/kern.htm

could this one also be related? https://github.com/fonttools/fonttools/issues/999

Yep.

it looks like in the decompile code we do set an subtable.apple = True attribute when kern table version is 1. We just ignore it while compiling the subtables. It shouldn't be difficult to fix.

Definitely not hard.  Just clumsy.  Note that tupleIndex is loaded from Apple tables but not stored in XML...  Also, coverage bits mean different things in OT versus Apple spec.  Moreover, the subtable version field of OpenType is bogus and unused.

yes, and for the Apple coverage field only the low byte is kept (coverage & 0xFF), i.e. the one wich defines the subtable format, the high byte is thrown away.
There seems to be a confusion between the subtable version number (only in OT kern) and the kern subtable format.
Also we set the "subtable.apple" attribute only when decompiling from binary, but not when reading kern table version="1.0" from XML.

I'm still confused about that "version" field in the OpenType kern subtable header (not the kern table version, which is always 0 in the OpenType spec, and 1.0 in Apple TrueType spec). As you know this subtable header "version" is no longer present in Apple's kern table version=1.0.
Then, in both the specs, the last 8 bits of the coverage (ushort) field is specified as the "the format of the subtable": 0 and 2 for OpenType spec, and 0, 1, 2, 3 for Apple TrueType spec.
Format 0 kern subtables are the only ones that will be properly interpreted by Windows, as the OT spec notes. Therefore all the kern version=0 tables out there that are meant to work on Windows have only format-0 subtables; these all unsurprisingly have subtable header "version" 0.
Now, the OT spec also has format-2 kern subtables (two-dimensional array of kerning values). Even if Windows does not support this, I am wondering would the subtable header "version" of a theoretical format-2 kern subtable (inside a kern version 0 table) also be 2 (like its "format"), or would it be 0 (like the bogus header version), if one wished to follow the OpenType kern spec?
My real question is, can't we just ignore such bogus "version" field which is only in the old kern spec and which will unlikely ever be updated again, and only read and use the subtable "format" from the coverage bits when we want to distinguish the various subtables?
The kern table version (be it 0 or 1.0) is still what distinguishes the two flavors of kern: one has ushorts for for version, nTables and subtable length, the other has ulong, etc.

also, shall we load in the subtable coverage attribute only the bits 0-7 (high byte) and in the format attribute the low byte containing the actual subtable format? Then upon compile we would bitwise-OR them into a single coverage USHORT.
We only support format 0 for now, so my suggestion above wouldn't change anything. But in the future it's conceivable we add support for other kern subtable formats (harbuzz is adding support for AAT kern too).


Include tc filter ... output in bugtool

We currently don't show any tc filter ... output which could be useful.
cc @joestringer to add additional context when we were digging into a sysdump where this information would have been useful.

FWIW I think that technically bpftool net show provides the same information from a pure BPF perspective, and is a bit more human-readable.
tc filter show ... is more specific to tc though and could possibly provide details that bpftool net show doesn't.

Agree with Joe here. Is there any specific information we are missing today?

Looks like there's some tags, filter preference/chain values, plausibly any non-bpf filters (though these are highly unlikely in any default Cilium install).
root@kind-control-plane:/home/cilium# tc filter show dev lxc5869b5857a3d ingress
filter protocol all pref 1 bpf chain 0
filter protocol all pref 1 bpf chain 0 handle 0x1 bpf_lxc.o:[from-container] direct-action not_in_hw id 15843 tag a7a920f725fab0ef jited
root@kind-control-plane:/home/cilium# tc filter show dev lxc5869b5857a3d egress
root@kind-control-plane:/home/cilium# bpftool net show
xdp:

tc:
cilium_net(2) clsact/ingress bpf_host_cilium_net.o:[to-host] id 15871
cilium_host(3) clsact/ingress bpf_host.o:[to-host] id 15859
cilium_host(3) clsact/egress bpf_host.o:[from-host] id 15865
lxc5869b5857a3d(18) clsact/ingress bpf_lxc.o:[from-container] id 15843
eth0(21) clsact/ingress bpf_netdev_eth0.o:[from-netdev] id 15877
eth0(21) clsact/egress bpf_netdev_eth0.o:[to-netdev] id 15883
lxc_health(23) clsact/ingress bpf_lxc.o:[from-container] id 15853

flow_dissector:

Hm, it may actually make sense if we ever run into a conflict in the tc configuration with some other software.

@joestringer @pchaigno Taking a look at this for my first issue.
Lacking the full context on this so gonna take my best guess at it.
In my draft PR added commands for:
tc -s qdisc
tc qdisc show

As well, for the filter/class/chain info, these need to be done per interface, so doing something like:
tc filter show dev ifacexxx0
tc chain show dev ifacexxx0
tc class show dev ifacexxx0
...

for every interface.

@christarazi as well.


请问支持代理地址吗？

就是自己搭建反向代理服务，用我的域名替换 api.openai.com

new OpenAiService(final OpenAiApi api, final ExecutorService executorService) use this constructor

https://github.com/TheoKanning/openai-java#adding-a-proxy

不是用proxy，是使用第三方的代理接口地址直接代替官方的接口地址，比如 https://api.openai-proxy.com

/**
 * Creates a new OpenAiService that wraps OpenAiApi.
 * The ExecutorService must be the one you get from the client you created the api with
 * otherwise shutdownExecutor() won't work.
 * <p>
 * Use this if you need more customization.
 *
 * @param api             OpenAiApi instance to use for all methods
 * @param executorService the ExecutorService from client.dispatcher().executorService()
 */
public OpenAiService(final OpenAiApi api, final ExecutorService executorService) {
    this.api = api;
    this.executorService = executorService;
}

源码中提供了这种构造方式，你通过这种构造方式创建的 OpenAiService 可以是自己的域名，而不是用默认的域名

new OpenAiService(final OpenAiApi api, final ExecutorService executorService) use this constructor

If you are not sure how to use it, you can refer to this constructor — public OpenAiService(final String token, final Duration timeout) or the example below.
ObjectMapper mapper = OpenAiService.defaultObjectMapper();
OkHttpClient client = OpenAiService.defaultClient("sr-xxxx", Duration.ofSeconds(10));
Retrofit retrofit = new Retrofit.Builder()
        .baseUrl("https://api.notopenai.com/")
        .client(client)
        .addConverterFactory(JacksonConverterFactory.create(mapper))
        .addCallAdapterFactory(RxJava2CallAdapterFactory.create())
        .build();
OpenAiService service = new OpenAiService(retrofit.create(OpenAiApi.class), client.dispatcher().executorService());


Usage of both Imperial system and Metric systems in displaying metric related information for configs which has label_options available

With the incorporation of Metric System to display timeseries of emission for charts in which config have label_options (Dynamic config) is available. This has led to case of confusion with few other metrics being displayed, which still shows the charts in Imperial System.
The emission timeseries charts like

"Timeseries of emissions per kilometer"
"Timeseries of emissions"

displays the distance/weight in Metric system (as the Laos config has dynamic config) i.e. kg/km
while the following other charts like:

"Trip count under 10 miles",
"Trip count under 10 miles (sensed) All data",
"Trip miles by mode All data Default",
"Trip miles by mode (sensed) All Data Default",
"Trip miles by land mode (sensed) All data Default",
"Average trip length All data Default"

still uses Imperial system (existing implementation), depicting distance as in miles
This could lead into confusion for the end user.
Possible solution:
Incorporate usage of useImperial tag from the config file to determine whether to display all the information related with different metrics in Imperial Mode or Metric System.

I'm picking this up as my introduction to working with the dashboard itself and the interaction with the config (work until this point has been limited to notebooks used to generate charts for a paper, and mostly from TSDC data).
I had the chance to chat with @iantei this afternoon regarding this issue, who was kind enough to give me a bit of a crash course in dashboard development. We identified the display_config.use_imperial field of the config files as the best criteria for choosing which paradigm we'll use for the whole dashboard.
In looking at the code, I noted that the chart names (in the html files) are hard-coded to include the use of "miles" in their titles. My first thought was that I would need to create a second set of html files, but that seems less than ideal, so I will instead work towards a more dynamic solution, where labels are set based on some variables.
Miles are also ingrained in the way that charts are generated, so I will need to come up with some way to handle this more dynamically as well. If I can find an elegant way to do so, I will expand the current chart generation rather than introduce doubling (code for imperial charts and code for metric charts) but would resort to the latter if need be.
First step is to set up a branch and development environment to get a workflow going, and then I will move forward with making the necessary changes so that imperial vs metric is in line with what is dictated by the configuration file.

There are many charts which display trips "less than 10 miles" in coming up with an appropriate replacement, I'm considering showing trips "less than 15 kilometers" to get a round number, but a more direct translation would be 16.1 km - this would let me keep the math and just need to change the labels.

For each of the notebooks, I have been combing through and ensuring that any distances can be handled in either km or miles. For "cutoff" charts, such as "trips under 10 miles" I only changed the text, so the same metrics are displayed regardless of the units, just with labels that make sense in context. For other charts, I leveraged the addition of a distance_kms column to expanded_ct in order to calculate and display distance metrics in kilometers.
The images below represent the first 3 notebooks that I have updated, run with the snapshot of laos data (the other data I have loaded is CO ebike data, and it is extremely slow to work with it on my computer) for both imperial and metric "modes", as controlled by the variable use_imperial
Generic Metrics

Imperial
Metric

Generic Metrics Sensed

Imperial
Metric

Generic Timeseries

Imperial
Metric

These images indicate that both modes are consistent with each other, and only the units are different. Next steps:

[ ] update the "program" notebooks, which contribute 7 more distance charts (this might require loading some different program data)
[ ] implement changes on the front end (I have started this, but struggled to test)
[ ] test changes on the front end

There are many charts which display trips "less than 10 miles" in coming up with an appropriate replacement, I'm considering showing trips "less than 15 kilometers" to get a round number, but a more direct translation would be 16.1 km - this would let me keep the math and just need to change the labels.

I've been discussing this a bit offline with @iantei and wanted to bring the discussion into the issue. The concern with choosing 16.1 km is that it does cater to the imperial system, but that it also maintains a consistent threshold across all the dashboards - I think it might lose some meaning (and would absolutely lose some trips) if we decided to make the cutoff 10km.
I think we should either stick to 16.1 km = 10 miles, or, if we wanted to be able to present a round number to metric system users, select 15km as a middle ground. @shankari what do you think about the "cutoff" distance for metric system charts?

That's a good point. I just have a slightly different view that the end-user of Metric system based dashboard would be different to Imperial system dashboard (Like different program/study from non-US based research group who uses Metric system exclusively). Therefore having 16.1 km as the cut-off doesn't sound intuitive for the Metric system users on the public dashboard.
But as @Abby-Wheelis mentioned earlier, if there is a research/study upon which we have selected 10 miles as the cut-off. We might as well stick with 16.1 km cut-off.

With the incorporation of Metric System to display timeseries of emission for charts in which config have label_options (Dynamic config) is available.

For the record, the imperial versus metric distinction has been around since the first iteration of the dynamic config in summer 2022.

I think we should either stick to 16.1 km = 10 miles, or, if we wanted to be able to present a round number to metric system users, select 15km as a middle ground. @shankari what do you think about the "cutoff" distance for metric system charts?

That's a good question. I agree that we are unlikely to have both miles and km displayed at the same time. The partners are going to pick one and the config only supports one. The 10 miles cutoff was fairly arbitrary, and a request from the CEO for the original statistics, based on statistics like this:
https://www.energy.gov/eere/vehicles/articles/fotw-1230-march-21-2022-more-half-all-daily-trips-were-less-three-miles-2021
I don't even know why 10 instead of 3, although I can speculate that it is a round number that is at the limit of what an e-bike would travel so it would capture the travel of interest. Given that rationale, I vote for 15 km. Round number, at around the distance where the results would be meaningful.
I would not be opposed to changing/including 3 miles = 5km either to be consistent with the "half of trips" theme.

From some prior work that we did at the CanBikeCO work, here's the distribution by trip distance in the last NHTS. 10 miles covers ~ 80% of trips. An alternative method that is more data driven and not as hard-coded would be to display trips below the 80% distance. That would work for both miles and km and would be justifiable because it covers most of the trips (80:20 rule) while excluding outlier (I'm driving cross-country) trips.

An alternative method that is more data driven and not as hard-coded would be to display trips below the 80% distance.

I like the idea of <80% mark, that seems to be the most principled to me. I wonder how it could play in to different contexts, like if there was a study in a really densely populated area (or in a much less densely populated area). In any case, we would accomplish the goal of excluding the outlier / "cross country" trips. I'll plan on this approach, I think it helps work towards a plan which is both context dependent and maintains consistent meaning.

I wonder how it could play in to different contexts, like if there was a study in a really densely populated area (or in a much less densely populated area).

I actually think that is a bonus: "And it would work even in rural areas where the median distance might be longer."
I would display what that 80% distance is (e.g. 9.5 miles). and potentially also display how we got that number, if you can figure out how to represent that correctly.

As it turns out, this Laos snapshot data is a good example of typically shorter trip lengths! FYI very easy to get the desired percentile of a column with pandas quantile -- if we compare to above, only ~60 trips labeled "motorcycle" were greater than 6.8 km but less than 16.1 km!

Still working off of the COebike data, which seems broken at this point (a lot of trips seem to be missing?), but since it is consistent, I can still use it to compare the way the notebooks handle imperial vs metric data
Energy Calculations

Imperial
Metric

1kg ~ 2.2lbs, which at a glance matches this chart, 1 -> 2.1 could be accounted for by rounding
Mode Specific Metrics

Imperial
Metric

1 mile ~ 1.6 km : 821.6 ~ 132 and 51.6 ~ 8, so this matches at a glance
Mode Specific Timeseries

Imperial
Metric

These charts are one of the main reasons I think that my data has something wrong with it... but the comparison between imperial and metric measurements are behaving as expected, with the charts being the same shapes, but having different y axis scales.
I've updated each of the notebooks now, so I'm going to move to working on the dashboard/frontend changes!

I've started with the html changes, which related to having the chart labels be adaptable, since many of them had hard-coded labels. I've updated all 4 html files now, and checked the lists by putting different configs in the url parameter, things look good so far, and I think I'm ready to move towards testing/debugging my changes.
There are 8 different scenarios that could happen on the dashboard now, from the combinations of:

program or study
dynamic or default labels
imperial or metric

Since my changes rely on the config, I need to check a few different scenarios. I currently have a snapshot of the laos-ev data, which is a study with dynamic labels configured to use imperial measurements. To keep coverage as high as possible, I think it would be helpful to also test with a dump of data from a program with default labels and metric measurements. stage-program fits this description. @shankari would is be possible for me to get a mongodump of a couple of months of stage-program data to test with? Or, now that I think about it, any program with default labels and e-bike as the mode of interest could be used against that config file to accomplish the same scenario.

Front - end testing (hosted locally) [DRAFT]:

Config characteristics:
Metric
Imperial
Custom Labels Config

Program Data (stage)
metric program
imperial program
custom program

Study Data (laos ev)
metric study
imperial study
usaid-laos-ev: [as expected] study no energy charts, miles

note to Monday Abby - make sure you set the env variable for the study config if needed!

I ran the notebooks from the command line with the staging snapshot and the stage-program configuration one more time to iron out any issues, but I'm now experiencing a bug with the energy notebook in particular where it says that there is no Replaced Mode - I'm unsure how this bug popped up. Resolving it should be the last step before this PR is ready!

Getting into this issue more and finding a few things:

I'm seeing 0 e-bike trips and therefore 0 trips with replaced mode for the staging data snapshot that I have, but I was able to generate charts with e-bike data in them for the screenshots of the frontend testing I had done, I have no idea what I could have done wrong, I have cleared and restored the database a couple of times with the same results

@shankari do you happen to know for sure if there should be e-bike trips with replaced modes in the data you sent me?
And @iantei could you share any recommendations for deleting/restoring data so I can ensure I have a clean database?

in mode_specific_timeseries where we have data preprocessing to create mode_counts and mode_counts_of_interest there was an issue where it was generating all user-day-mode combinations, and therefore rows were created even if there were no trips that met that, which led to things like "600 walk trips of 300 confirmed trips" as the quality text, I believe I have resolved this bug

Update from this being on staging:

now showing trips under 80th percentile instead of under 10 miles
set of charts available:

excludes energy for custom label cases
switches between miles and km based on use-imperial in the config
includes mode-of-interest charts for programs

made sure that charts were different month-to-month

One issue was found and fixed #121

This change is now on production and working! @iantei I think we can close this issue as completed now


Set the reconcile rate to low

Signed-off-by: Joshua Packer<EMAIL_ADDRESS>
In 2.3 @rokej introduced a reconcile rate. Using low reduces our check to hourly.

/cc @rokej @hanqiuzh

Isue Ref: https://github.com/open-cluster-management/backlog/issues/11814


RDKBDEV-2306: Insufficent RBus message header

Reason for change:
Sufficient size to be able to display Device.WiFi.DataElements paths

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

This patch doesn't solve the problem. rbus_regDataElements fails with duplicate name error, which is the result of element name check vs MAX_OBJECT_NAME_LENGTH in rbus_addElement. The value is equal to RTMSG_HEADER_MAX_TOPIC_LENGTH which is defined in rtMessageHeader.h. Increasing it will solve the problem.
Using alias instead of indexes may also cause overflow, but trace will be different than above.


Incorrect mouse position when being on clickable buttons on JetBrains IDE (HiDPI)

Hyprland Version
 System/Version info
Hyprland, built from branch HEAD at commit 84ab8d11e8951a6551d1e1bf87796a8589da6d47  (props: bump ver to 0.35.0).
Date: Mon Feb 5 01:59:02 2024
Tag: v0.35.0

flags: (if any)

System Information:
System name: Linux
Node name: ultraluxor
Release: 6.7.6-arch1-1
Version: #1 SMP PREEMPT_DYNAMIC Fri, 23 Feb 2024 16:31:48 +0000

GPU information:
0000:00:02.0 VGA compatible controller [0300]: Intel Corporation Raptor Lake-P [Iris Xe Graphics] [8086:a7a0] (rev 04) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo
plugins:

Bug or Regression?
Bug
Description
The mouse position is incorrect when trying to click on the IDE buttons. It makes you to lose focus on the window and makes it really difficult to click on them.
How to reproduce
Open any jetbrains IDE and try to touch any button (if they are on corners it is easier to reproduce)
I have the following settings and I am using a HiDPI Display:
env = QT_QPA_PLATFORM=wayland;xcb
env = QT_WAYLAND_DISABLE_WINDOWDECORATION=1
env = QT_QPA_PLATFORMTHEME=qt5c
env = GDK_BACKEND=wayland,x11
env = SDL_VIDEODRIVER=wayland
env = CLUTTER_BACKEND=wayland
env = XDG_CURRENT_DESKTOP=Hyprland
env = XDG_SESSION_TYPE=wayland
env = XDG_SESSION_DESKTOP=Hyprland
env = GDK_SCALE,2
env = QT_ENABLE_HIGHDPI_SCALING,1
env = QT_AUTO_SCREEN_SCALE_FACTOR=1
xwayland {
force_zero_scaling = true
}
Crash reports, logs, images, videos
https://github.com/hyprwm/Hyprland/assets/21333617/91d49066-e5d6-4077-8e56-a8466a2c1874

update hyrpland and hyprland portal, I had same issue before, it got fixed later, current git is stable install the latest and this problem should be solved

Try to remove those env setting, I am using goland and the only issue I have the search and replace windows close instantly, other than that everything is perfect

With no env settings it loses the resolution and the problem is still happening. I can reproduce it in my other setup without HiDPI, It happens less but I can still see it.

I also have the same issue but haven't found a fix so far

env = XCURSOR_SIZE,24
env = QT_QPA_PLATFORM,wayland
env = WLR_NO_HARDWARE_CURSORS,1
env = XDG_SESSION_TYPE,wayland
Try updating your environment; it is most likely your configuration. I am using jetbrain daily, and do have nvidia gpu, as far as I know not all app play well with hidpi but jetbrain run with no issue for me

I am using the latest version and I am still seeing it. I can send you my whole config but using only the envs you sent did not work for me. I am using integrated gpu

I had that issue using the right toolbar icons.
What fixed it for me was
windowrulev2 = stayfocused,class:^(jetbrains-.*)

Works perfectly now. Thank you.


Failed to extract cab files from msi, please provide a valid BizTalk MSI

Hi, I'm running the framework with BizTalk 2010 RTM and Visual Studio 2010 Pro; when I generate an MSI with "Build Server Deploy MSI" I cannot import this into BizTalk using the usual import mechanism. When I examine the MSI contents, it contains all artifacts in my solution, so basically everything is fine. When I deploy my solution into BizTalk via Visual Studio an export it as an MSI the contents of that file list a lot of cab files instead of the artifacts. So somehow the output of both deployment path is different. What am I missing here? Is there a switch that causes the framework to create an MSI with cab files instead of the individual artifacts?

Thanks! Johannes
This work item was migrated from CodePlex
CodePlex work item ID: '7325'
Vote count: '1'

[tfabraham@10/1/2010]
Hi Johannes -- our MSI is not intended to be imported into BizTalk.  Once you've got the MSI built, then you run it on each BizTalk server.  At the end of the installation you'll have a checkbox to fire up the actual deployment (which is also available through the Start menu).  Our deployment model is completely different from BizTalk's native deployment model.  I assume that in Visual Studio you used the Deployment Framework Deploy commands vs. the Visual Studio Deploy menu command?  We do not provide an option to produce "BizTalk native format" MSI's -- that would defeat the purpose of our framework and tooling.  Does that answer your questions?
Thanks,
Tom

[JohRest@10/4/2010]
Hi Tom,
in a sense, yes. I assumed that the purpose of this function was to produce "BizTalk" compatible msi files. One question I do have though: what do you mean by "fire up the actual deployment"; does that initiate a deployment via BTSTask through the installer code? Thanks Johannes

[tfabraham@10/4/2010]
If you've (hopefully) run some deployments with BTDF using the Visual Studio Tools/Deployment Framework for BizTalk menu, then you've seen the MSBuild output in the Output window during the deployment.  That same MSBuild process carries over to server deployments, so you can set up and test your deployment on your own machine and configure your .BTDFPROJ file as necessary (in Visual Studio), then build an MSI when you are ready for a server deployment.  The "actual deployment" step that I'm referring to is launching the MSBuild process after the MSI installs your project files.  BTSTask is one of many BizTalk and custom tools that we use to cover every aspect of a complex deployment.

[UnknownUser@10/11/2010]

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]

Issue closed by tfabraham with comment
Working as designed.


Remove fdcache, use netx.Listener throughout

This change adopts netx.Listener for all NDT servers: NDT5 raw and HTTP(S) and NDT7.
Because netx fully manages the underlying connection file pointer, this change removes:

fdcache package
listener.CachingTCPKeepAliveListener
tcplistener.RawListener

This change fixes https://github.com/m-lab/ndt-server/issues/235
This change improves https://github.com/m-lab/ndt-server/issues/255 (see issue for details)

This change is

https://github.com/m-lab/ndt-server/issues/178


Heroku deployment

Hi,
Thank you for initiating this project.
I tried to download this eve-demo project, and I was able to run it fine using python run.py command. But it behaves weird when I run it using foreman start (this is from https://devcenter.heroku.com/articles/getting-started-with-python#procfile). All the possible links return a 404 Not Found error.
This problem also happens when we do deployment on Heroku. I thought it was a problem on the mongodb connection. So I made sure that the mongodb info on the settings.py is correct, but it doesn't fix the 404.
May I know how you deploy to heroku? Do you do foreman start when running locally?

FWIW, I ran into the same issue. I know this is an old issue (closed more than 2 yrs ago, with no confirmation from @genkiro) But I'm adding this with the hope that someone finds this useful.
I am not using foreman but I get this error on "live" heroku deployment
If I replace SERVER_NAME in my config.py with SERVER_URL the problem is fixed.
i.e. SERVER_NAME is no more defined (So I guess some default value is used, which works in heroku setup?)
I can't remember why I used SERVER_URL instead.
I am unable to remember/find the source of why this "fixes" the issue.
It worked for me twice, with the gap of 6+ months. Unfortunately I didn't document my "source" the first time. Second time (May 2016) , I just looked at my "old" git log and applied the change (somewhat reluctantly) but it worked.
May be @nicolaiarocci can add some comments on why it works.


Config Priority in Startup.cs

hi   , when we have two Implemented class from an Interface , How can we control which one can be Run for us? ?  plz show me how can i config that in startup.cs ?       services.AddServicesWithAttributeOfType();  Has Error.
For example , i have InterfaceTest and Two class that implemented it , in InterfaceTest i have a TestMethod() , in BaseClass1 TestMethod do something , and in BaseClass2  Do Something Else ,  If I Use InterfaceTest.TestMethod on Controller , I have Which One ?

You can use the [IgnoreServiceRegistration] attribute to suppress one implementation.


[bug] filename not contain prefix of output.filename

My webpack js output config
output: {
	filename: 'js/[name].js'
},

and extracttext plugin
new ExtractTextPlugin({
        filename: 'css/[name].css'
})

but manifest.json
"main.css": "css/main.css",
  "main.js": "js/main.js",
  "manifest.js": "js/manifest.js",
  "vendor.js": "js/vendor.js"

the right manifest.json content should be
"css/main.css": "css/main.css",
  "js/main.js": "js/main.js",
  "js/manifest.js": "js/manifest.js",
  "js/vendor.js": "js/vendor.js"

what's wrong and how can solve it ,thx

Looks like a bug... I will try to have a look tomorrow
Which version are you using?
If you can provide a failing test that would be awesome

@mastilver version is ^1.3.2,my option
new ManifestPlugin({
        filter: chunk => chunk.isInitial
      })

solve it
manifestOpts.generate = (seed, files) => files.reduce(
			(manifest, file) => {
				let fileName = _.last(_.split(_.last(_.split(file.name,'/')),'.')) + '/' + file.name
				let filePath = _.replace(file.path, config.output.publicPath,'')
				manifest[fileName] = filePath
				return manifest;
			},
			seed
		)


Update the game over message

Update the game over message so people know how to play again!

lee pl


Add uniqueFromArray filter

A filter that returns an array containing only unique items

Thanks so much for this @chris-barrett-ddts! Were you wanting to add any more filters? In which case I’ll wait before creating a new release of the package.


[App] Molecules are not visualised in Tables.

Describe the bug
Table view does not work with Molecules -- the molecule object is rendered as the text "Molecule".
To Reproduce
Steps to reproduce the behavior:
Try and visualise a Molecule in a table.
Expected behavior
You can see the molecule.
Screenshots

Operating System

OS: Ubuntu 20.04
Browser: Firefox
Version: 95.0

Hi @latkins,
I was not able to reproduce this on my end, could you please share a link to the project where you see this behavior?
Thanks,
Ramit

Hi @latkins,
Just following up here, Is this a bug you are still facing?
Thanks,
Ramit

Closing this issue due to lack of activity. If you would like to re-open this conversation, please reach back out!
Thanks


fix: createDevice race condition

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

bug fix

What is the current behavior? (You can also link to an open issue here)

Creating a second device would terminate the second device correctly, then delete the old one from the map, causing it to be leaked and go missing

What is the new behavior (if this is a feature change)?

Don't delete the device in the map, and clarify the flow a little

Other information:

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

Comparison is base (c49055d) 34.82% compared to head (eb884fd) 100.00%.

Additional details and impacted files
@@               Coverage Diff               @@
##           release50      #296       +/-   ##
===============================================
+ Coverage      34.82%   100.00%   +65.17%
===============================================
  Files             99        42       -57
  Lines          10118       651     -9467
  Branches        2481        76     -2405
===============================================
- Hits            3524       651     -2873
+ Misses          6007         0     -6007
+ Partials         587         0      -587

see 57 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Messages not arriving

Good morning Jonathan,
Before anything i would like to thank you about the support that you give, the application is running more stable now!
The only problem that still needs to be fixed and to be honest I don't know if it's a socketcluster problem is...
We built a client side control that run on receive message event (socket.emit) and trigger a event on server that set the "read" status of the message to 1... We are getting a 40% messages not read and we cant figure out why this is happening, because we don't show offline users, so every message should be delivered and read... We suspecting about 2 things...
1º - The users are getting offline and the server dont get it right...
2º - The users are online and the message are not delivered...
Im guessing if have some way to check before emit the message through the global.publish if the channel have someone online OR some channel control to run time by time to check their users status...

If a user navigates away from the page before their client responds to an event, then you could miss some acknowledgements but 40% sounds extremely high.
Another possible scenario:

An error is being thrown on the client-side (not necessarily related to SC) which is preventing the SC client from sending the acknowledgement to the server (even though they did receive the message).

It's also possible (if your site/app has lots of mobile users), that if their mobile device goes to sleep while they are looking at your website/app, that SC may not realize that they are offline until the ping timeout happens and they could miss some messages from the server.
The SC client will automatically disconnect if the user navigates away from the current page but sleeps are a bit trickier to detect but the connections should 'ping timeout' and disconnect on their own in this case - The ping timeout could take a few seconds to kick in though.

We have a meeting with the infrastructure team and we have decided to move to a mini-farm of 2 computers running linux... So, lets forget about it for now.

Closing this... Feel free to open a new issue if something else comes up.


[IMPROVEMENTS] remove unnecessary useState

Description
we can see events components on line 87 of the component use tabs state variable to contains filters data, but never call setTabs state update function.
In general,  if we are using state to manage some aspect of our component's behavior, we should use the setter function returned by useState to update the state value as needed. If we find that we don't need to update the state value, we may want to consider whether we really need state at all, or if we can use a different mechanism to manage the behavior of our component.
Path
pages/events.js
Github UI here

Screenshots
No response
Additional information
Proposed Changes
instead of state variable we should use normal variable to contains that filters data

Good spot 👍 yes we could probably use a standard variable for it

Can you assign me this issue, please?

I will be happy to work on this issue

I would like to work on this issue. Can I be assigned to it?

I would like to work on this issue

Can you assign me this issue, please?

@omlondhe as you were the first to request this Issue when the labels changed to status: ready for dev I will assign this to you


Use edm-hardened-uefi image for initcontainer

This container contains hardened image that would be used
to provision the baremetal nodes.
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/273

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck


Serialize recommender model

Is there a way we can serialize recommender model? Tried joblib, pickle and both returned attribute error.
TypeError: self._base_ptr,self.thisptr cannot be converted to a Python object for pickling

Just call the save function on your model.
Let me know if you have any other questions.


Upgrade OpenRewrite Templating 1.16.3 -> 1.17.0

This PR contains the following updates:

Package
Type
Update
Change

OpenRewrite Templating

minor
1.16.3 -> 1.17.0

OpenRewrite Templating
compile
minor
1.16.3 -> 1.17.0

Release Notes

openrewrite/rewrite-templating (OpenRewrite Templating)
v1.17.0: 1.17.0
Compare Source
What's Changed

refactor: Annotate methods which may return null with @Nullable by @nielsdebruin in https://github.com/openrewrite/rewrite-templating/pull/115

New Contributors

@nielsdebruin made their first contribution in https://github.com/openrewrite/rewrite-templating/pull/115

Full Changelog: https://github.com/openrewrite/rewrite-templating/compare/v1.16.3...v1.17.0

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade OpenRewrite Templating 1.16.3 -> 1.17.0

See:
- https://github.com/openrewrite/rewrite-templating/releases/tag/v1.17.0
- https://github.com/openrewrite/rewrite-templating/compare/v1.16.3...v1.17.0


Create API

Create API

Created a rails project to be used as API for a front-end React project.
It generated a random greeting.

Thank you for your nice review😃

thanks :)


hiccup-css: question about how to use pseudo classes

Hi Karsten 👋
First off thanks for all the work on these packages! I'm finding them very useful for a lot of things at the moment.
One thing I just ran into though with the hiccup-css package, and perhaps it's an oversight on my part and I simply haven't come across the corresponding code/example,  but is there a way to use css pseudo classes with the package? I'd like to set a hover state for something but I don't see an obvious way to do so.
Thanks!

Gagh! Apologies. it's still too early - forget I asked, I didn't think of just writing out the pseudo class directly when writing the selector. I've used SCSS for too long  and was so used to writing out
h1{
  &:hover {}
}

It's slightly less ideal writing it out with the selector but I can make this work.
Thanks again!

Hi @sortofsleepy - firstly, thank you for the kind words & trying out the libs! 😄
I always thought I did document the use of pseudo-classes, but it's seems I didn't and it something I'll correct asap. For now please take a look at overall pattern in this test case here:
https://github.com/thi-ng/umbrella/blob/master/packages/hiccup-css/test/index.ts#L36
Pls let me know if this works for you!

@postspectacular
Ah ha! Yup, that worked.
I had essentially tried the code described in the test but for some reason it hadn't occurred to me to try square brackets and I was using curly braces instead. doh!
Thanks so much again!

Square brackets are used to delineate nesting levels, curly ones are always just used for attributes/properties...
Also, reopening this issue for my own purposes (as reminder to update readme)...

Gotcha, I'll keep that in mind. (and apologies for closing too early)


Ian Campbell: Watch habibcoder.com "Linked on spam from websiteslearn and found in known spammer's profile"

Ian Campbell requests the watch of the watch_keyword habibcoder\.com "Linked on spam from websiteslearn and found in known spammer's profile". See the MS search here and the Stack Exchange search in text, in URLs, and in code.

habibcoder\.com "Linked on spam from websiteslearn and found in known spammer's profile" has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Rejected by Ryan M in Charcoal HQ. Reason: 'invalid comment format; discussed in CHQ'


Migration from Json.Net to Unity's JsonUtility

As #17.
The completed commit.

did you test this?

I use this commit on my local project.
GetServerList() works great.
I don't have the load balance setup, so feature relative with load balance is not tested.

i will test with load balancer project soon then


Getlisting(), cant access the files in the directory

FTP OS: Unix / Windows / Embedded
I am building FTP client to access the contents of a directory, all the code goes fine but when I try to read the files names or the check if they exist I receive the message "Timed out trying to connect!"
When I use the directory.exists() it replies with true or false but when I use file.exists() it reply the same error as Getlisting(). I tried several options of listing but nothing worked !
N.B
I accessed the FTP server via different sources and downloaded all the files inside and everything was fine.
The system type: is UNIX Type: L8
The server type is Unknown
The OS type is UNIX
`
public static async Task<IActionResult> Run(HttpRequest req, ILogger log)
 {
      using (FtpClient conn = new FtpClient())
    {

        conn.Host = "ftplink";
        conn.Credentials = new NetworkCredential("username", "pass");
        conn.RetryAttempts = 5;
        conn.Connect();

        log.LogInformation(conn.GetWorkingDirectory());
        log.LogInformation(conn.DirectoryExists("/out/work").ToString());
        //log.LogInformation(conn.FileExists("/out/work/qwewe.csv").ToString()); // gives the same error
        log.LogInformation("1");

    foreach (FtpListItem item in conn.GetListing("/"))
            {
                log.LogInformation(item.FullName);
            }
        conn.Disconnect();
    }

  }

`

Please paste the full FluentFTP logs. See the FAQ how to get logs..

Closing until you reply.


Allow the ** operator

I'm getting an error when using the ** operator:
Expected an identifier and instead saw '*'

But it's perfectly fine to use it like that:
let x = 2 ** 10;

It's not disallowed by a setting, because then it would at least understand the syntax.
And browser support for it these days is excellent.
This is on the latest version, 2.9.4.

A quick search for "exponentiation" returns this issue:
https://github.com/jshint/jshint/pull/3060
Note that this has been implemented and will be available in JSHint 2.10.0: https://github.com/jshint/jshint/pull/3060


Add pypi badge and correct one api documentation.

How it looks like:
https://testpypi.python.org/pypi?:action=display&name=uniq&version=<IP_ADDRESS>

Please take a look. @ameyspatil


Can't click the button like "ok" after the camera is finished.

First, thanks to the resources provided by the author.I encountered the following problems when using:
Can't click the button like "ok" after the camera is finished.
It's true to open for the first time.but it won't be available at any time.
and I have a suggestion,I hoped rebuild the project can do performed in the new branch.
Thanks again to the author.

Use this https://github.com/CJT2325/CameraView/releases


Assert when designing bandpass

I've tried to use a bandpass based on the example in http://liquidsdr.org/doc/firdes/
Unfortunately, I get result:
src/filter/src/firdespm.c:711: firdespm_iext_search: Assertion `num_found < nmax' failed.

I'm not sure what it means exactly and what's the way to deal with this? I can't find anything related in the docs. I'm using the LIQUID_FIRDESPM_BANDPASS type and the definition:
    float bands[6] = {
        0.0f, 18.5e3/sample_rate,
        18.7e3/sample_rate, 19.3e3/sample_rate,
        19.5e3/sample_rate, 0.5f,
    };
    float des[3] = { 0.0f, 1.0f, 0.0f };
    float weights[3] = { 1.0f, 1.0f, 1.0f };

    liquid_firdespm_wtype wtype[3] = {
        LIQUID_FIRDESPM_EXPWEIGHT,
        LIQUID_FIRDESPM_EXPWEIGHT,
        LIQUID_FIRDESPM_EXPWEIGHT,
    };

Unfortunately this is an internal error. The firdespm method is an implementation of the Park-McClellan algorithm for minimax FIR filter design that iterates over the Remez exchange algorithm. Theoretically there are supposed to be a certain number of extremal frequencies, but sometimes (because of machine precision) more are found.
Right now I haver an assertion that the number is valid, but this will kick the program out rather than exiting gracefully. I should try to be more resilient and/or perhaps a bit nicer if it encounters such an error.

To be more specific, I didn't actually mind the assert (but sure - an error would be nicer). What I was confused about was - so what do I do about it?
Would it be possible to add some suggestions? I removed the wtype and replaced it with null like I saw in some example and it suddenly started working. But I'm left with "huh?" :-)

Well, there are a few things I can suggest for the time being. To begin, you define three regions without much guard in between them (originally I think you didn't have any guard band in between them); increasing that might help. The algorithm is pretty sensitive to the parameters of the filter design, but I might be able to improve it if I could reproduce your results. I've guessed at your parameters (sample_rate = 100e3f;) and I can verify on my end that I'm getting the same error.

As an aside, if you just want to create a simple band-pass filter without too much complexity, you can do so with a low-pass windowed-Kaiser and then frequency shifting it. I've attached an example to do just that. I know this isn't exactly what your original ticket was about (firdespm), but it might solve your larger problem.

#include <stdio.h>
#include <math.h>
#include <liquid/liquid.h>

int main() {
    // define filter length, type, number of bands
    unsigned int n           =  251;        // filter length
    float        freq_center =  20e3f;      // center frequency [Hz]
    float        bandwidth   =   2.0e3f;    // filter bandwidth [Hz]
    float        sample_rate = 100e3f;      // sample rate [Hz]
    float        As          =  40.0f;      // stop-band suppression [dB]

    // allocate memory for array and design low-pass prototype filter
    float h[n];
    liquid_firdes_kaiser(n, 0.5f*bandwidth/sample_rate, As, 0.0f, h);

    // shift frequency
    unsigned int i;
    for (i=0; i<n; i++)
        h[i] *= cosf(2*M_PI*((float)i - 0.5f*(float)n) * freq_center / sample_rate);

    // generate output file for plotting
    FILE * fid = fopen("firdeswk.m","w");
    fprintf(fid,"clear all; close all;\n");
    fprintf(fid,"nfft = 1200;\n");
    for (i=0; i<n; i++)
        fprintf(fid,"h(%3u) = %12.8f;\n", i+1, h[i]);
    fprintf(fid,"f = ([0:(nfft-1)]/nfft-0.5)*%12.4e;\n", sample_rate*1e-3f);
    fprintf(fid,"plot(f,20*log10(abs(fftshift(fft(h,nfft)))));\n");
    fprintf(fid,"grid on;\n");
    fprintf(fid,"xlabel('Frequency [kHz]');\n");
    fprintf(fid,"ylabel('Power Spectral Density [dB]');\n");
    fclose(fid);
}

You can shift low pass to band pass by multiplying taps? Whaaaaaaat? I learned something cool today, thanks!


Add csv archive support to signature scan invocation

This allows for passing a csv archive option to the signature scanner so that csv files can be created. Detect will begin doing this in offline mode so that users have access to the files.

Is --outputFormat customer facing?

Is --outputFormat customer facing?

It is in the sense that you could run the scan CLI manually and specify it. This work is so that Detect can essentially ask for it to be added once we add a new specific Detect property.

Will there be a Detect property that customer could set to enable this property? Or, is it going to be enabled by default in offline mode?

Will there be a Detect property that customer could set to enable this property? Or, is it going to be enabled by default in offline mode?

Yes, this is the prep work for that. Another PR will be made on the Detect side to add the property.


Image generation is not listening any events. It appears that a particular prompt is experiencing an issue.

It appears that a particular prompt is experiencing an issue. You can try the following prompt:

A professional setting featuring a diverse group of tech experts working collaboratively in a modern office. Emphasize the themes of innovation and agility in technology. Show elements of AI and Blockchain, like holographic displays, digital interfaces, and data analytics. Display a futuristic and dynamic atmosphere, showcasing teamwork and cutting-edge technology in North America. Ensure the setting is sleek and high-tech, with a focus on business transformation and innovation.

Although Discord is showing a generated image, the library is not listing any events; it seems.
Originally posted by @Prajval108 in https://github.com/UrielCh/midjourney-client/issues/38#issuecomment-2340595305

I don't see any strange characters in this message.
An I have juste test this prompt and I do not get any issue.


Question: Should this be parse error?

I was working on Prettier to make tests run both flow and babel-flow parser https://github.com/prettier/prettier/pull/14245, and found an edge case.
https://github.com/facebook/flow/blob/f87d5bf972f47f14e8e51f8ad26da82a96770175/tests/generics/generics.js#L40
flow can parse function as identifier, but babel can't, and I found other keywords like class/while can't be used there.
Should it be valid or invalid?

I found it surprising and weird that function (lowercase) can be used as a type. Support for this was added when support for the Function type was added, back in 2015, but I am not aware of anyone actually using this (or even being aware of this). I would be happy to remove support for this and make this an error.

Thanks for the explanation, feel free to close if it not worth to fix.

I have a diff in progress to make it an error, just needs to be reviewed (internally)