Support Retry-After headers for GitRepository objects using HTTP(S)

Git Servers may throttle operations when a given threshold (operations/time) is met.
If the controller continues to attempt cloning whilst blocked, it will impair the controller's ability to self-heal as some servers will keep extending the ban window.
The ban window length varies per service provider, in public GitLab for example this is set to 60 minutes.
Considering that the current default --max-retry-delay is set to 15min, in case of error caused by being banned, the controller will continue to try cloning at most 4 times per hour, and the server would reject all those attempts, and keep extending the ban window indefinitely. To avoid this problem users have to increase the --max-retry-delay to be higher than the ban window used by the Git servers being used.
In the long term, the controller could observe the header Retry-After RFC-6585 header, which indicates the amount of seconds remaining on the current ban. This can be used in conjunction with Kubernetes' RateLimiter to define a per item retry policy.

Example of throttling from Gitlab: https://docs.gitlab.com/ee/security/rate_limits.html


Misaligned function declaration if return value is of type struct timeval

Describe the bug
There is no way to format correctly a function declaration in .h file if return value of a function is of type struct timeval.
If I try to shift all other functions, they are marked as misaligned.
Erroneous code
struct timeval	time_get_now(void);
size_t	time_get_millis(struct timeval time);

Error:
Error: MISALIGNED_FUNC_DECL (line:  89, col:  17):      Misaligned function declaration
Additional infos

OS: Linux 5.17.3-artix1-1
python --version: Python 3.10.4
norminette -v: norminette 3.3.51

Additional context
Add any other context about the problem here.

Fixed.


[TLP-5003] - Migrate to Drone

Description
This PR migrates the CI pipeline from CircleCI -> Drone.

✅ SAST Report
No issues found. Good job! 💪


fix(tokens): DLT-2135 fix remaining iOS issues

fix(tokens): DLT-2135 fix remaining iOS issues

Obligatory GIF (super important!)

:hammer_and_wrench: Type Of Change

These types will increment the version number on release:

[x] Fix

:book: Jira Ticket
https://dialpad.atlassian.net/browse/DLT-2135
:book: Description

Removed linear gradient values from android / ios
Changed value from .8 to 0.8 for iOS compatibility
Fixed iOS font stack transform that needed to be transitive
Fixed untransformed shadow color that must have been missed in francis' prior PR.

:bulb: Context
Couple of follow up fixes to https://dialpad.atlassian.net/browse/DLT-2135 to get the iOS tokens syntax correct again.
:pencil: Checklist

For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.
[x] I have added all relevant documentation.
[x] I have considered the performance impact of my change.

:crystal_ball: Next Steps
Let Trevor know the fix is in, and to try it out.
:link: Sources
https://styledictionary.com/

Hi @braddialpad which tokens are affected by this and how can I test it?

Need to run nx build within packages/dialtone-tokens and check the output of the file packages/dialtone-tokens/dist/ios/tokens-base-light.swift and tokens-dp-light.swift.
Should no longer be any syntax errors which previously were

usage of .8 rather than 0.8.
linear-gradient() which is not valid in swift.
The expressive font-family was previously Archivo, UIFont.systemFont(ofSize: 15), it should now just be UIFont.systemFont(ofSize: 15)


Window layout issues on some scaling settings

I cannot view most of the right half of the the TQVault (Inventory, stash, etc.) as they are partially outside of the window. Changing the window size does nothing.

same problem here, i cant see my 2nd weapon and rings too
and i have the latest TQVault version but it say <IP_ADDRESS>
help

Yes, I have the same problem. I thought it could be related to the screen settings, but not.
I have the last version TQVault 3.0

@Nagaira @ridderzwaard @DarkLance69 Thanks for reporting.
Could any of you provide us with your resolution, windows version and windows scaling option?

This problem has also been reported by other users from other forums. It should be a priority to anyone willing to fix bugs.

Can't reproduce the problem. Maybe two screens?

my monitor reso: 1366 768 and win 10
when I installed tqvault version 3 and try to open it everything looked normal
but after that i cant see my 2nd weapon ,rings, 3 extra backpack , and minimize/maximize X button
double click top of the window for full screen :

@Nagaira @ridderzwaard @DarkLance69 Thanks for reporting. What happens when you manually resize the window or use the full-size button at the top right (next to minimize and the red x)?
Could any of you provide us with your resolution, windows version and windows scaling option?

resizing the screen doesnt help as the character and inventory sections are still not in place.
My monitor's resolution is 1920 by 1080, windows 10, with a scale of 150 percent. maybe i installed it wrong?

Hi, my screen resolution is 1920x1080, scale (recomended) 125%. I have changed the scale, but nothing happens. And I'm using Windows 10.
I'm having the same problem as @ridderzwaard. I see the vaults but not the right side nor the bottom.
I cannot resize manually, as I just see only the first button, which only minimize the window.

This is caused by the windows DPI scaling (found in the Display Settings under Scale and Layout as "Change the size of text, apps and other items" in the neweset version, it was specifically called DPI scaling in earlier versions and I think was in a menu screen of its own?).
A fast fix would be tell windows not to DPI scale the application usually (through the compatibility settings) but in my case this just creates a worse mess, and I presume the problem is easy enough to fix once located.
Now, I'm not really a coder these days so I won't even attempt it lest I break all in the process, and someone who knows the code already can probably find what's up much faster. My thinking is the scaling applied to different elements is somehow using a different, well, scale, thus having the whole scaled up by the Windows scaling has varying results on different items.
I don't know if it's the width and height of of the main vault and the player inventory forms or the location of the two, but whichever decides where those two elements go is getting multiplied needlessly (see the 125% vs 150% screenshots above have different layouts) and thus sends the player inventory and the equipped intems/stash panels out of frame.
So, if it indeed is easy to get to, whatever scaling factor is used for the elements themselves should also be used for setting the position of the player inventory and the equipped items/stash, whether it is a position of the panel or the space the other panels take that scales here. I would suspect it is the former since the "Character:" label is also on an adventure, but the main vault panel seems to be aligned correctly, as does the "Vault:" label. Either these two have a non-scalable position (0, 0?) within their parent or their position scaling is actually working.
But as I said, not really a coder so takes me a good while to find this in the code, should I ever do.

Thanks a look for the feedback

I changed the scaling to 125% and I get it cropped just like @Nagaira ...

Finally got it working properly. The fix is on #116

How do i download the fix?

You have to wait for the owner to properly review and accept the changes, then make a new release. Sorry.

It'll get in eventually, no worries!

thanks for your hard work guys

Yes, thanks for your efforts and great job.
By the way, not sure what I settings have changed, but now, I can almost the complete vault.

Just to narrow down the issue, the main program window is 1350 pixels wide and 910 pixels height. If your resolution is lower than that or the DPI scaling (in the windows 10: display settings > Scale and layout) makes the initial window size doesn't fit in your screen, the initial resize mess things up. Will be fully fixed in the next update if it gets approved.

@DarkLance69 @Nagaira @clomaka @EchoC4 @AvKy @ridderzwaard You guys might be interested in the new stable release, which should fix your problem!

works well on my laptop; thanks!


Websocket connections are blocked by this plugin!

I have an issue where websocket connections are blocked even if there is an API_KEY specified!!! Regular https calls work normally!

Same issue here


Add enchanted love by linear ring for the level name is enchanted love

Song Name
enchanted love
Artist Name
linear ring
Source
Youtube
Youtube Link
https://youtu.be/qE6mruRB1PA?si=esXV71oTTtxS7ctd
Direct File Link
No response
Song ID
949936
Start Offset [ms]
No response
Extra
Used in the level "enchanted love" (id: 80398571), and.. may be used in a level I will make for it!

Accpet plsss

accept


Amazing！！

Hai! Could you tell me whether the detail of this project have a introduction?

in fact, i want to know the meaning of 'pval'.

Hi, you can check this paper: https://www.researchgate.net/publication/318391632_An_Algorithm_to_Extract_and_Judge_the_Main_Text_Based_on_the_Law_of_Total_Probability

Thank you for your generous offer!


diamond blastx subworkflow

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Small test works: nextflow run . -profile test,docker --outdir <OUTDIR>, and files to run the full test were also added

When I run the test profile, I get the following warning.
WARN: Found unexpected parameters:
* --uniprot_blastx: /lustre/scratch123/tol/teams/tolit/users/ps22/pipelines/blobtoolkit/assets/test/mCerEla1.1.buscoregions.dmnd
- Ignore this warning: params.schema_ignore_params = "uniprot_blastx"

I think modifying the schema and config should fix it.

When I run the test profile, I get the following warning.
WARN: Found unexpected parameters:
* --uniprot_blastx: /lustre/scratch123/tol/teams/tolit/users/ps22/pipelines/blobtoolkit/assets/test/mCerEla1.1.buscoregions.dmnd
- Ignore this warning: params.schema_ignore_params = "uniprot_blastx"

I think modifying the schema and config should fix it.

Done, the warning should not appear anymore.

@priyanka-surana Just want to confirm if this can be merged, the requested change should be resolved in this commit: 8f3f719. If there are no other changes it can be merged.

Closing and opening a fresh PR. Give a couple days.


missing dash in release version retrieved

at this repo
https://github.com/Aircoookie/WLED/releases
notice the latest is 0.13.0-b2
that's dash -b2
lastversion is returning pre-release version without that dash, which sadly breaks the downloads etc. without me doing some extra sed manipulation
> lastversion --pre Aircoookie/WLED
> 0.13.0b2

installed lastversion via pip, 3.02

for now I just added a sed
ver=$(lastversion --pre Aircoookie/WLED)
ver=$(sed -e s/b/-b/ <<< "$ver")

lastversion normalizes versions according to PEP 440 Version Identification and Dependency Specification. So 0.13.0b2 is a more or less canonical version (while others can put it as -b2 or -beta2 or whatever fancy format which lacks any consistency).
To get an unmodified release tag, you can do:
[user@pc ~]# lastversion --pre Aircoookie/WLED --format json | jq -r .tag_name
v0.13.0-b2

breaks the downloads etc.

Or better yet, get the desired download URLs from lastversion itself:

[user@pc ~]# lastversion --pre Aircoookie/WLED --format source
https://github.com/Aircoookie/WLED/archive/v0.13.0-b2/WLED-v0.13.0-b2.tar.gz

[user@pc ~]# lastversion --pre Aircoookie/WLED --format assets
https://github.com/Aircoookie/WLED/releases/download/v0.13.0-b2/esp32_bootloader_v2.bin
https://github.com/Aircoookie/WLED/releases/download/v0.13.0-b2/WLED_0.13.0-b2_ESP01.bin
https://github.com/Aircoookie/WLED/releases/download/v0.13.0-b2/WLED_0.13.0-b2_ESP32.bin
https://github.com/Aircoookie/WLED/releases/download/v0.13.0-b2/WLED_0.13.0-b2_ESP32_Ethernet.bin
https://github.com/Aircoookie/WLED/releases/download/v0.13.0-b2/WLED_0.13.0-b2_ESP8266.bin

Lastly, lastversion can do downloads for you:
lastversion download --pre Aircoookie/WLED

etc.

I forgot that for the bare tag name, you can use --format tag, e.g.:

lastversion --pre Aircoookie/WLED --format tag

But still, I highly recommend checking the download command of lastversion. You don't need un-normalized versions.

all good info thx!
my script downloads a particular asset so download option not useful in my case, unless there is a way to tell lastversion to download a particular asset of the set instead of the source?

well yes it can get the asset url which avoids my issue
but can it download that instead of the source?
lastversion --pre Aircoookie/WLED --format assets --filter ESP32.bin

https://github.com/Aircoookie/WLED/releases/download/v0.13.0-b2/WLED_0.13.0-b2_ESP32.bin

@dkebler looks like it can, if invoked like this:

lastversion download Aircoookie/WLED --pre --format assets --filter ESP32.bin

ok ha it can
lastversion --pre Aircoookie/WLED --format assets --filter ESP32.bin -d ESP32.bin
I think you should add this example to your readme.    Many times it's a particular asset that needs to be downloaded (like a binary for particular platform)

Just ending this by thanking you again.  I have all kinds of bash and nodejs scripts I've written to get repo release assets from different projects but with lastversion I can dump/simplify all those.  This is a small BUT very useful project.


Code Block Indentation Issue

Description
In some chapters, such as here, the code blocks might have indentation issues (copy-pasted from the link):
// Code snippet

#[async_std::main]
async fn main() -> Result<()>{
     // Read the database environment from the `.env` file
+ 	 let env_database_url = include_str!("../.env").trim();
     // Split the env url
+    let split_url: Vec<&str> = env_database_url.split("=").collect();
     // Get item with the format `database_backend://username:password@localhost/database`
+    let database_url = split_url[1];
+
+    let db = Database::connect(database_url).await?;
+
+    let builder = db.get_database_backend();
+    let schema = Schema::new(builder);
+
+	 let create_table_op =  db.execute(builder.build(&schema.create_table_from_entity(Fruits))).await;
+    println!("`CREATE TABLE fruits` {:?}",
+        match create_table_op {
+    		Ok(_) => "Operation Successful".to_owned(),
+    		Err(e) => format!("Unsuccessful - Error {:?}", e),
+    	 }
+    );
    Ok(())
}

Steps to Reproduce
Click the link provided above and observe the code block after the sentence Add code to perform execution.
Expected Behavior
The indentation should be aligned.
Actual Behavior
Some indentation in the code block is misaligned.
Reproduces How Often
Yes.
Versions
N/A
Additional Information
N/A

AH right thanks for pointing that out. I think it might be caused by original tab width of 8 being displayed in 4. PR would be appreciated!

@tyt2y3 Good to know.
Another issues I've observed: In the first tutorial there was no mentioning of creating the .env file but it is used to retrieve DB URL with include_str macro. I'd be happy to help out with editing the docs.

Um... I think the first chapter was written at a very early stage :/


fix: removed error regarding warnings due to missing generic type closes #3572

Summary

The error: JSX.Element' is not assignable to parameter of type 'ComponentType<{}>' was coming up due to missing generic
Test plan

yarn test passed 66 of 68 total tests and skipped 2 as configured with a few warnings unrelated to this commit.

@erezrokah Do I need to find one more reviewer?

uhh, not sure if this is correct, lads.
Where does that P come from? In React that's the name of the generic parameter, but when you use it, you have to provide a type.
...or another name (can be P), but you have to redefine it locally.
created a follow up PR : #3638


Icon Request: Signal level

Name : Signal 1,2,3,4, no-signal, signal-research
Is it possible to have something like that :

Thank you for your help

This is a good idea, but I don't know how we could show the empty bars.
I see 2 main solutions:

Nothing, like in the battery icon

The problem here is that the signal-1 would be a single small bar, which isn't easily recognizable.
translucent bars, like in material

I think this is a bad idea because no other icon is like this and the SVG color attribute could be a problem

Good idea!
I see a 3rd solution: the empty bars can have only the border. Something like this :

What do you think?

I think this is impossible because - if you take a look at other icons you will notice it - we don't fill any shape so we keep the icon pack consistent, therefore active bars will either have to be empty rectangles or simple bars

I think they just should be empty - don't we already have these icons?

don't we already have these icons
bar-chart can be used for full signal.

yes, we just need someone to make the other ones - but who?

I saw something like that :

What do you think?

oh yes why not
the empty bars will be dots

I like this one, it is similar to the battery icon and pretty clear to understand :)

What do you think of this?

This one is also pretty clear to understand but I still prefer the other one personally.

@LM1LC3N7 What do you exactly mean with the other one?

This once : https://github.com/lucide-icons/lucide/issues/283#issuecomment-891565089

@LM1LC3N7 But that was not a design I think. It was some inspiration.
Here a variant with dots:

Ok I didn't know!
I like the version with always dots, I found it pretty clear regarding the signal strength.
If I see two bar and then 3 dots I know for sure how the signal is received (for example), 2/5.

I think we could get rid of the first dot? I think the icon looks too 'wide'. Also the first dot will always be a dot, no matter the signal intensity, so I don't think it's very useful. In my opinion

We want to keep the first dot, for the "zero" value. Also keeps consistency with the wifi icon.
I will add the signal icons without the dots, with the idea from #477 you can create the empty signal lines.
Let me know if you aren't ok with it, we can always make some changes.


UX review & fixes for site

When the time is right, @abeck000 should do a review for UX polish on the site, and file bugs (along with associated severity: P1 (launch blocking), P2 ("should" do), P3 (nice to have), so @camelburrito can make fixes before launch.

i think we're ready for this @ericlindley-g

oops this comment was meant for another issue.

This can be closed now we have other bugs


[BEAM-1530] Cleanup BigQueryIO, and allow for value-based tables

BigQueryIO.Write takes in a user type, which the user can convert to a TableRow.
Table functions now take a ValueInSingleWindow, instead of just the window.

Self-mention @jkff  to put this in my mentioned PRs

Coverage decreased (-0.01%) to 70.153% when pulling bae611bb7f5a51e8c593ab026c5604a5316c4fe4 on reuvenlax:new_bigquery_api into 806c53c18793b43e7ccd28f99662ee73a97237b4 on apache:master.

Coverage decreased (-0.03%) to 70.14% when pulling d423c05b3ec7ab6b6499d869f16bb6df1ce43dc4 on reuvenlax:new_bigquery_api into 806c53c18793b43e7ccd28f99662ee73a97237b4 on apache:master.

Coverage increased (+0.02%) to 70.185% when pulling 87fa7ae80b59c18e4dbcacf76ac58f62312e6344 on reuvenlax:new_bigquery_api into 806c53c18793b43e7ccd28f99662ee73a97237b4 on apache:master.

Coverage decreased (-0.009%) to 70.184% when pulling 34c4e24b215cf6437d45635b8af5ba15fcfcc4c8 on reuvenlax:new_bigquery_api into a0b8a854df6129e2dbbbd4bad683e53d975f3146 on apache:master.

@tgroh does this look good to you? If so, could you help with the merge?

Coverage decreased (-0.05%) to 70.143% when pulling 21e9476c2b81c5801f85e223b594cbe3a3d577c0 on reuvenlax:new_bigquery_api into a0b8a854df6129e2dbbbd4bad683e53d975f3146 on apache:master.

LGTM. Would you like to rebase, or should I?

Rebased against origin/master.
On Fri, Mar 17, 2017 at 8:51 AM, Thomas Groh<EMAIL_ADDRESS>wrote:

LGTM. Would you like to rebase, or should I?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/2238#issuecomment-287392592, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUGE1T1zsmNFCTYCHAeuv0J4Clj1a-d9ks5rmqv_gaJpZM4Mb5rA
.

Can you squash things into more self-contained commits (I suppose 2 commits: 1) move ValueInSingleWindow 2) the BigQueryIO changes per se)?

Coverage increased (+0.01%) to 70.127% when pulling 5785a53687aedad7157864b5e501cf135d28645b on reuvenlax:new_bigquery_api into b672cde111b2f11d185479e1d208e63ef6ff322e on apache:master.

@jkff commits are squashed

Coverage increased (+0.001%) to 70.117% when pulling e1d28c04edb14c5b4728864de6cd949c6233808d on reuvenlax:new_bigquery_api into b672cde111b2f11d185479e1d208e63ef6ff322e on apache:master.

Coverage increased (+0.01%) to 70.13% when pulling e1d28c04edb14c5b4728864de6cd949c6233808d on reuvenlax:new_bigquery_api into b672cde111b2f11d185479e1d208e63ef6ff322e on apache:master.

Coverage decreased (-0.002%) to 70.114% when pulling e1d28c04edb14c5b4728864de6cd949c6233808d on reuvenlax:new_bigquery_api into b672cde111b2f11d185479e1d208e63ef6ff322e on apache:master.

Coverage increased (+0.01%) to 70.127% when pulling e1d28c04edb14c5b4728864de6cd949c6233808d on reuvenlax:new_bigquery_api into b672cde111b2f11d185479e1d208e63ef6ff322e on apache:master.

Coverage increased (+0.008%) to 70.124% when pulling e1d28c04edb14c5b4728864de6cd949c6233808d on reuvenlax:new_bigquery_api into b672cde111b2f11d185479e1d208e63ef6ff322e on apache:master.

Coverage increased (+0.01%) to 70.127% when pulling e1d28c04edb14c5b4728864de6cd949c6233808d on reuvenlax:new_bigquery_api into b672cde111b2f11d185479e1d208e63ef6ff322e on apache:master.


Training code

Is the training code going to be released in near future ?

already included in this repo. please check.


Lazy way of setting pauses

Hi, I've recenly discovered presenterm and I like it :)
However I'd like that there I would like adding pauses between list items to be less cumbersome, somewhat like:
intrementalmode: true
in global configuration, per presentation or per slide. Let's assume that you have some slides with 4 or five items, it's a bit annoying I to have to put a tag for each item in a list.
What do you think?

I like the idea! I'll implement it soon, it shouldn't be too complicated. Would doing this per slide only be enough for you?

I just pushed this in branch automatic-list-pauses. Could you try it out and see if it works for you? This is per slide and you can enable it by using a comment command like <!-- incremental_lists: true -->.

@mfontanini Thanks! It works like a charm. It would be great if this feature could be incorporated in the the master branch.
Ideally, it would be great to set a default value for all presentations, whether true or false, and then customize the behavior for each slide individually within the file.
Greetings

Great! I'll add something to make this work globally as well but for now at least this simplifies your life a bit.

Alright the changes I just merged to master from #109 allow setting this globally. See the example in that PR for more info.


issues 5 and 6

issue 5 - fix for tslint path issue on Mac.
issue 6 - initial circleci setup (demo build not yet included).

closing as mixed commits and will request under a different PR.


fix: shut down operator when failing to start

fix: start timer as daemon to avoid prolonging the app's life
fix: exit operator on exception when starting

Thanks 🎉 !


git: Filename completion with directory symlinks

Current Behavior
When there's a symlink anywhere in the directory hierarchy to the current directory, filename completion will create a relative path to the symlink target, effectively breaking filename completion.
Expected Behavior
Completion should work for the current directory regardless of whether there's a symlink anywhere in the directory hierarchy.
Steps To Reproduce
With the default config:
mkdir /tmp/a
ln -s /tmp/a /tmp/b
cd /tmp/b
git clone <some repo> test
cd test
touch some_file
git add som<Tab> # will not offer completion
git add <Tab> # will complete to ../../a/test/some_file

Version
1.0.1
OS

[X] Linux
[ ] OSX
[ ] Windows

Shell

[ ] Bash
[ ] Elvish
[ ] Fish
[ ] Nushell
[ ] Oil
[ ] Powershell
[ ] Xonsh
[X] Zsh

Anything else?
No response

I can imaging that there's still something amiss with symlinks.
But following the given example it does complete some_file for me for both cases.

Thanks for the feedback, maybe it's something in my configuration / environment that's the issue, I'll try to see if I can identify it.

Same issue here, I thought to be the shell, but it was carapace. It works with fish autocompleter though.

Urks, so I can see something missing with the directory symlinks.
But regarding git add it doesn't look like it accepts anything beyond a directory symlink:
moep on  master [?]
esh ❯ ls -T
.
├── one
│  ├── a
│  ├── b
│  └── c
└── two -> one
error: no candidates
error: no candidates
moep on  master [?]
esh ❯ git add two/b
fatal: pathspec 'two/b' is beyond a symbolic link
Exception: git exited with 128

In this case probably due to the real folder being on a different mounted disk.

That's the case for me as well. Sorry, didn't think to include that information.

The information was enough, I must have testet it in the wrong shell :man_facepalming:.
The completion is generated using git status and determining the relative path from current work directory to ${GIT_ROOT}/${CHANGE}.
Seems there is a difference beween zsh and elvish regarding symlinks.
When changing to /tmp/b/... in zsh the workdir uses the symlink path. Hence the ../../ relative path to the git root in/tmp/a/.
But in elvish the workdir becomes the actual dir (/tmp/a/...) so the issue doesn't appear there.


Updates with code changes

These updates of dependencies need code changes.
Every module update is in own commit.
The Netty update showed a problem with some of the unit tests that depended a bit too much on the internals of Netty - so first changed the unit tests to be more resilient and then I updated the dependency so even middle commit states should be fine.

LGTM


Updating settings to disable Helth Provider

Description:

Linked issues:

Screenshots:

How to test:

@inteljudajocu @iocedano Este es ell PR de ocular el heath provider please merge if everything i ok I had to make another PR so we dont change the environment files now THANKS


Audible and/or Haptic Feedback?

As a potential quality of life feature, it would be great if this app had some sort of feedback when either players hits the clock, this could be in the form of a sound clip that plays when tapped, (such as a "click" sound) and potentially a haptic tap that the player might feel when they tap the clock.
Otherwise, it's hard to tell sometimes if one of the players has already tapped the clock without looking at the clock.

Thanks a lot for the suggestion. This should be implemented in release v1.9.0.


Remote debugging Gauge Java tests

Hi, is there a way if remote debugging Gauge Java tests, i.e. connecting to the Gauge JVM from a remote debugger? Many thanks.
Gauge version
Gauge version: 1.0.5
Commit Hash: 562f036

Plugins
-------
html-report (4.0.8)
java (0.7.2)
js (2.3.5)
screenshot (0.0.1)

@jphillips-bp3 Gauge Vscode and Gauge Intellij plugins has good support for debugging. If you you are looking for some thing else, please tell us your use case.

@jphillips-bp3 You can set GAUGE_DEBUG_OPTS to remotely debug the Gauge JVM
Example: GAUGE_DEBUG_OPTS=<port-number>  gauge run

@jphillips-bp3 You can set GAUGE_DEBUG_OPTS to remotely debug the Gauge JVM
Example: GAUGE_DEBUG_OPTS=<port-number> gauge run

Thanks @negiDharmendra , I will give it a go in a few days.


Updated helm template to write to one file and check its valid

What is this PR About?
Simplifies helm by writing stdout to a file, which means anything using it (i.e.: conftest) just needs to look at 1 file. Added some safety checks as well for when helm might fail.
Fixes: https://github.com/redhat-cop/pelorus/pull/149
How do we test this?
Green action
cc: @redhat-cop/bats-library

/lgtm


Allow MemGPT to read/write text files + make HTTP requests

Addresses #26

TODO

[x] Allow toggling of preset from CLI (right now it's hardcoded)

Example with MemGPT launching HTTP requests

Example with MemGPT reading / writing from a shared .txt file

$ cat notes.txt
I will tell you a secret

Modify the file, then check if MemGPT can read the updated file:
$ echo "I like dogs" >> notes.txt
$ cat notes.txt
I will tell you a secret
I like dogs

MemGPT can successfully read from the updated file (after some prodding to read more lines):

@vivi do we have to do anything special to support toggling the preset in the CLI?
Or can we simply add another arg here (and up the stack): https://github.com/cpacker/MemGPT/blob/main/memgpt/main.py#L165

Still need to add preset selection, put it separate PR


Upgrade to Bevy 0.14

Fixes #59
Note: dolly submodule is pointing at my fork. Just needed to bump some version numbers over there.
There were some color-related deprecations, but otherwise this one is super easy.

I'm back from holiday as of now, as luck would have it, dolly just updated to version 0.6 an hour ago. But it would be better ergonomics to avoid using mint and keep it as is.


Added cli and yml parser

Reference Issues/PRs

What does this implement/fix? Explain your changes

Adds a simple starter cli to read a pdb, convert it into a graph, and save it as a pickle. New features can be added in the future.
Implements a yaml parser so that config objects can be written as yaml and will be parsed and read as Pydantic Objects. The yaml parser allows the specification of classes, functions and partial functions
Because Python doesn't have a proper eq operator to compare that two partial functions are the same, I added an eq overwrite operator for ProteinGraphConfig which uses deepdiff to check whether two Pydantic objects are equal

What testing did you do to verify the changes in this PR?

A test that confirms that the cli produces valid outputs.
A test that confirms that a Pydantic object parsed from a yaml file is the same as a Pydantic object that was initiated within python

Codecov Report

Merging #113 (32533a4) into master (8123f42) will increase coverage by 2.24%.
The diff coverage is 43.99%.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
+ Coverage   40.27%   42.51%   +2.24%
==========================================
  Files          48       51       +3
  Lines        2811     3206     +395
==========================================
+ Hits         1132     1363     +231
- Misses       1679     1843     +164

Impacted Files
Coverage Δ

graphein/protein/analysis.py
0.00% <0.00%> (ø)

graphein/protein/visualisation.py
12.93% <9.67%> (-3.29%)
:arrow_down:

graphein/protein/features/nodes/amino_acid.py
80.43% <50.00%> (-3.29%)
:arrow_down:

graphein/protein/graphs.py
74.21% <66.66%> (+2.51%)
:arrow_up:

graphein/protein/utils.py
68.00% <73.33%> (+16.78%)
:arrow_up:

graphein/protein/subgraphs.py
73.64% <73.64%> (ø)

graphein/cli.py
79.16% <79.16%> (ø)

graphein/protein/config.py
96.15% <89.47%> (-3.85%)
:arrow_down:

graphein/__init__.py
100.00% <100.00%> (ø)

graphein/protein/__init__.py
84.61% <100.00%> (+1.28%)
:arrow_up:

... and 12 more

Continue to review full report at Codecov.

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


Questions about the algorithm speed.

Hi, thanks for your good work.
I notice that the paper does not compare the algorithm speed. I would like to know the speed comparison of NAT vs swin-Transformer, and CNN model.
Thanks！

Hello and thank you for your interest.
We did not discuss that in this version of the paper, because the implementation (the CUDA kernel) is still a work in progress. Because both training and inference speeds are largely dependent on the implementation, it would not make sense to compare them at this stage.
However, with the current version of the kernel that is publicly released, we found that NAT-Mini is faster in ImageNet classification inference than Swin-Tiny, and NAT-Tiny is as fast as Swin-Tiny. However, we found that NAT-Small and Base are slower than their Swin counterparts, due to the inefficiency still present in our kernel, which we are working on.
Note that these speeds are all subject to input sizes, therefore with bigger models (more channels), or larger inputs (larger resolution), they will be different.
I hope this answers your question, but please let me know if you need further clarification.

Ok, thanks for your reply!


[Breadth Coverage] Onboard Datashare Commands

Spec Link
https://microsoft.sharepoint.com/:w:/t/AzureManagementExperience/EQ-hQ0oBQv9Hsplk-sfJ97kBWBXiZR4D9_FEcX3pGTJ5MA?e=LLw4Go
ADO Work Item Link
https://dev.azure.com/azure-mgmt-ex/Onboarding/_boards/board/t/Onboarding Team/Features/?workitem=152
Resource Provider
What is the Azure resource provider your feature is part of?
Microsoft.DataShare
Description of Feature or Work Requested
Support Azure CLI commands for Datashare
Minimum API Version Required
Microsoft.DataSagre 2019-11-01
Swagger Link
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/datashare/resource-manager/Microsoft.DataShare/stable/2019-11-01/DataShare.json

Datashare


Installation error

When using the FBI to install the game, encounter running the installation package immediately prompts an error. Try to replace the game a number of different installation packages, or the same error.

And? It already tells you that you have a bad .cia file. How are you launching FBI? HBL? Without a9lh?
http://3ds.guide

我的3DS 使用的是B9S。下载了多个不同的CIA 都是这个提示。如果是CIA的错误，我自己在尝试一下吧。多谢你的回复。

My 3DS is using B9S. Download a number of different CIA are the tips. If it is CIA wrong, I try it now. Thank you for your reply.

It's probably either that the CIAs are bad or there's something wrong with your SD card.


AMQP-595: Fix Auto-Startup for @RabbitListener

JIRA: https://jira.spring.io/browse/AMQP-595
Ignore container's auto-startup once the context is refreshed.
cherry-pick to 1.5.x

And cherry-picked as https://github.com/spring-projects/spring-amqp/commit/26d02f6216d912cd1acc677cc6e4b1c9f1b8663b


Public message overflows off card in collective "About" section

Describe the bug
A per-collective personal message from team members/contributors can be added to a card. The character limit for this card is 140. It renders fine in one place, but overflows under edges in another.
To Reproduce
Steps to reproduce the behavior:

Create a mock collective.
Add yourself as team member
On your collective page, clock your card to "add a personal message"
View the message in the team "About" section down below.

Expected behavior
Would expect that up to 140 character limit would render fine in all places its displayed.
Screenshots
Overflow in team About section

Find in Contributors section

Desktop (please complete the following information):

OS: OSX Catalina 10.15.7
Browser Chrome
Version 88.0.4324.150 (Official Build) (x86_64)

Additional context
n/a

Thanks for reporting this issue @patcon! It should be resolved with the next deployment


Subscription Support

Hi!
What would be the required steps to add subscription support to this client?

@Yamashou I got it working by using a slightly modified websocket client: https://gist.github.com/maaft/ac197c72fe5d18eec5916f3f5a75943f
How to use with generated structs:
func main() {
	subscriptionClient := subscription.NewSubscriptionClient("ws://localhost:8080/graphql")

	defer subscriptionClient.Close()

	errChan := make(chan error)

	subscriptionID, err := subscriptionClient.Subscribe(generated.SubscribeProjectsQuery, nil, func(dataValue *json.RawMessage, errValue error) error {
		if errValue != nil {
			errChan <- errValue
		}

		data := generated.SubscribeProjects{}
		if err := graphqljson.UnmarshalData(*dataValue, &data); err != nil {
			errChan <- err
		}

		for _, p := range data.QueryProject {
			fmt.Println(p.Name)
		}

		return nil
	})

	if err != nil {
		panic(err)
	}

	go func() {
		err := <-errChan
		if err != nil {
			subscriptionClient.Unsubscribe(subscriptionID)
			panic(err)
		}
	}()

	err = subscriptionClient.Run()

	if err != nil {
		panic(err)
	}
}

Do you think you could integrated this properly into this project? I think it should be relatively easy.

Thanks for issue and a script. Show me your implementation and think about it

[Q] nhooyr.io/websocket is that you guess best ws client ?

If you have time, I want you to create PR for this issues.
my start point is reading your gist. So, It takes longer to implement this client than you do

Hi! I also just copy-pasted the client I found and it works fine so far. The only issue is 100% CPU after some time, but that should be fixable.
I see if I can spend some time on this on the upcoming year.

Just to document it for later:
The >100% CPU issue is related to this function here:
func (wh *websocketHandler) ReadJSON(v interface{}) error {
	ctx, cancel := context.WithTimeout(wh.ctx, wh.timeout)
	defer cancel()
	return wsjson.Read(ctx, wh.Conn, v)
}

The timeout is set to 1 minute, then the load starts. When I use it like this:
func (wh *websocketHandler) ReadJSON(v interface{}) error {
	return wsjson.Read(wh.ctx, wh.Conn, v)
}

the issue is gone. I'm not sure about the implications though. But for now it works for me.

Thanks for comment and response !
I try to create new client package for cache and option now. If I finished it when you create PR for subscription support, I start this issue.

@Yamashou I got it working by using a slightly modified websocket client from hasura: https://gist.github.com/maaft/ac197c72fe5d18eec5916f3f5a75943f
How to use with generated structs (generated.SubscribeProjectsQuery, generated.SubscribeProjects):
func main() {
	subscriptionClient := subscription.NewSubscriptionClient("ws://localhost:8080/graphql")

	defer subscriptionClient.Close()

	errChan := make(chan error)

	subscriptionID, err := subscriptionClient.Subscribe(generated.SubscribeProjectsQuery, nil, func(dataValue *json.RawMessage, errValue error) error {
		if errValue != nil {
			errChan <- errValue
		}

		data := generated.SubscribeProjects{}
		if err := graphqljson.UnmarshalData(*dataValue, &data); err != nil {
			errChan <- err
		}

		for _, p := range data.QueryProject {
			fmt.Println(p.Name)
		}

		return nil
	})

	if err != nil {
		panic(err)
	}

	go func() {
		err := <-errChan
		if err != nil {
			subscriptionClient.Unsubscribe(subscriptionID)
			panic(err)
		}
	}()

	err = subscriptionClient.Run()

	if err != nil {
		panic(err)
	}
}

Do you think you could integrated this properly into this project? I think it should be relatively easy.

If I try to run gqlgenc on gql files that contains a subscription I get
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x8a86a0]

goroutine 1 [running]:
github.com/Yamashou/gqlgenc/clientgen.(*SourceGenerator).NewResponseField(0xc000375d98, 0xa51d80, 0xc00067ca80, 0xc0002ca6b8)
        /home/delaney/.gvm/pkgsets/go1.16.2/global/pkg/mod/github.com/!yamashou/gqlgenc@v0.0.0-20210330020310-5eab2091f840/clientgen/source_generator.go:147 +0x480
github.com/Yamashou/gqlgenc/clientgen.(*SourceGenerator).NewResponseFields(0xc000375d98, 0xc0007abe00, 0x1, 0x1, 0x30, 0x7fd726ebcd28, 0x30)
        /home/delaney/.gvm/pkgsets/go1.16.2/global/pkg/mod/github.com/!yamashou/gqlgenc@v0.0.0-20210330020310-5eab2091f840/clientgen/source_generator.go:83 +0xc5
github.com/Yamashou/gqlgenc/clientgen.(*SourceGenerator).NewResponseField(0xc000375d98, 0xa51d80, 0xc00067ca00, 0xc0002ca6b0)
        /home/delaney/.gvm/pkgsets/go1.16.2/global/pkg/mod/github.com/!yamashou/gqlgenc@v0.0.0-20210330020310-5eab2091f840/clientgen/source_generator.go:142 +0xa5
github.com/Yamashou/gqlgenc/clientgen.(*SourceGenerator).NewResponseFields(0xc000375d98, 0xc0007abe10, 0x1, 0x1, 0x0, 0xd2cec0, 0xc000375b48)
        /home/delaney/.gvm/pkgsets/go1.16.2/global/pkg/mod/github.com/!yamashou/gqlgenc@v0.0.0-20210330020310-5eab2091f840/clientgen/source_generator.go:83 +0xc5
github.com/Yamashou/gqlgenc/clientgen.(*Source).OperationResponses(0xc000375d78, 0xd2cec0, 0x0, 0x0, 0x0, 0x0)
        /home/delaney/.gvm/pkgsets/go1.16.2/global/pkg/mod/github.com/!yamashou/gqlgenc@v0.0.0-20210330020310-5eab2091f840/clientgen/source.go:144 +0x125
github.com/Yamashou/gqlgenc/clientgen.(*Plugin).MutateConfig(0xc0000aab80, 0xc00018cf20, 0xc0000aab80, 0x7fd6fc6bbed8)
        /home/delaney/.gvm/pkgsets/go1.16.2/global/pkg/mod/github.com/!yamashou/gqlgenc@v0.0.0-20210330020310-5eab2091f840/clientgen/client.go:69 +0x67c
github.com/Yamashou/gqlgenc/generator.Generate(0xa582d0, 0xc0000260d0, 0xc0000de6c0, 0xc00018bf60, 0x1, 0x1, 0xc000084710, 0x4426ca)
        /home/delaney/.gvm/pkgsets/go1.16.2/global/pkg/mod/github.com/!yamashou/gqlgenc@v0.0.0-20210330020310-5eab2091f840/generator/generater.go:32 +0x2e2
main.main()
        /home/delaney/.gvm/pkgsets/go1.16.2/global/pkg/mod/github.com/!yamashou/gqlgenc@v0.0.0-20210330020310-5eab2091f840/main.go:30 +0x1cc

I'd like to try this way of supporting subscriptions but no luck on the initial type generation.  What am I doing wrong?

Facing the same issue, we ended up creating our own GQL client: https://github.com/infiotinc/gqlgenc

Thank you, looks good!
I'd advice you to honor the work of @Yamashou in your repository though as you obviously based your work on his. Currently you are violating the MIT License of this repo.

Thank you :)
The relevant directories that are direct copies (or at least started this way) from this repo contain the LICENSE file from Yamashou, and some files that have been modified contain a link to their origin. But i agree that adding an Acknowledgement section in the readme would only be fair!


Ensure data ready before rendering

Make data ready before rendering script list, script editor and perftest detail
Fix bug that cannot refresh on quickstart

LGTM


Extend the login process beyond the basic method

Right now, your component provides a single authentication method GetSignInResultAsync() by passing a login request object.
However, there are many cases where additional authentication methods are needed. For example, I could let user in from another application by using a mutually agreed token value. Or I could allow an admin to impersonate into another user's account by simply having the user's id (of course, I will manage permission myself).
These additional authentication process are not supported and I have to expand the default username/password login request model to include these fields and handle all these different scenarios in the single login method.
Maybe I am missing something, but please advice.
Thanks,
Frank

@franky1017
There are no "default request models", those provided in the sample are just example classes for the purpose of demonstrating the package's functionality. You are free to define any models you need.
The package's responsibility is to manage the user's JWT via browser cookies, and to make those work with Blazor's built-in authorization capabilities.
You are free to implement any flow that includes working with JWT in your application logic. But your application's logic is not the responsibility of Blazor.Auth.

@YuriyDurov
I understand there is no default request model, but there is only one single GetSignInResultAsync() function for the entire app's authentication method, no? How can I use different request models for different authentication workflows/processes?

@franky1017
You can see the Sign-In method as a method where some request data goes in, and the user gets some JWT in their cookies as a result.
The contract models are not limited here in any way, except for the fact that these models need to be serializable and deserializable for them to be able to get passed between the server and the client parts of the app.
For example, you could create your different processes' models as subclasses from some base class, and use that base class in the generic definition. Then you can check which subclass the model is in the logic of processing the Sign-In request. You could also have some enum indicating which process is currently taking place. You can do a number of different things, I am not able to advise you more on how your end application should look exactly, as this is outside of the scope of this package.
Just make sure to follow these guidelines:

Use the Sign-In method as something goes in, the user gets a JWT pair (that you provide) as a result;
The sign-in request payloads need to be serializable and deserializable;
Deserialization process is using the model type that you provide in the server-side IAuthenticationService's generic definition (something like JsonSerializer.Deserialize<MySignInModel>(json)).

For additional guidance, refer to the sequence diagrams provided in the docs.
You provide the data, Blazor.Auth simply passes it back and forth. You can use the Sign-In method to handle all these situations that you described here.
If all this is still not enough, you can provide your custom IAuthenticationService implementations when registering Blazor.Auth in your app (for both client and server).
Hope this helps.

Then you can check which subclass the model is in the logic of processing the Sign-In request.

This is exactly what I am doing. In the sign-in request, I am doing checks to see what type of request model it is and perform different sign in method accordingly.
I was just wishing there could be different sign in methods instead of the single entry point. But it works just fine this way.
Thanks a lot for your time and explanation.
Frank


lhs glm

Hi all, here is a first attempt at addressing glms (and so this is related to #1 ). The internal function modify_lhs_for_link()is, in this PR, called within create_eq(), if the class/one of the classes of the model is a glm. Then, it finds what the link function is, and replaces the lhs with the lhs modified by the link function.
Example here:
library(equatiomatic)

set.seed(8675309)
d <- data.frame(out = sample(0:1, 100, replace = TRUE),
                cat1 = rep(letters[1:3], 100),
                cat2 = rep(LETTERS[1:3], each = 100),
                cont1 = rnorm(300, 100, 1),
                cont2 = rnorm(300, 50, 5))
mod5 <- glm(out ~ ., data = d, family = binomial(link = "logit"))
extract_eq(mod5, wrap = TRUE)
#> $$
#>  \begin{aligned}
#> log\left[ \frac { text{out} }{ 1\quad -\quad text{out} } \right] =& \alpha + \beta_{1}(\text{cat1}_{\text{b}}) + \\
#> & \beta_{2}(\text{cat1}_{\text{c}}) + \beta_{3}(\text{cat2}_{\text{B}}) + \beta_{4}(\text{cat2}_{\text{C}}) + \\
#> & \beta_{5}(\text{cont1}) + \beta_{6}(\text{cont2}) + \epsilon
#> \end{aligned}
#> $$

Created on 2019-06-04 by the reprex package (v0.2.1)
There are (at least) two issues.

The text command that is supposed to wrap the lhs has the two backslashes removed when inserted into the link function; is this due to the gsub() (and do you have ideas for how to address)?
I didn't replace the rendered image, yet, as I wanted to check this first, and be consistent regarding how you generated the other images, second.

Hmm... I'm not able to install this branch. I got
Error in data.frame(link_name, link_formula, stringsAsFactors = FALSE) :
  arguments imply differing number of rows: 4, 3

Sorry about that, I pushed a commit right after making this comment about the identity function - one that omitted a line from the lookup table. Should work now.

Just pushed a small change to the gsub, adding fixed = TRUE, which should fix the issue with \\text. Take a look and see what you think. I think it's probably ready to merge.
As an FYI - I mentioned in a different thread that I'm going to try to address the S3 stuff tonight (hopefully, although it might have to wait until tomorrow). Some of this may change slightly with that update, but hopefully not too much.

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff          @@
##           master   #27   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files           2     2
  Lines          96   104    +8
====================================
- Misses         96   104    +8

Impacted Files
Coverage Δ

R/extract_eq.R
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks - and, oops re: forgetting about updating the documentation (was that what was causing the Travis fail)?
This looks good. I tested it with a logit and a log function and it worked fine. I also tested with italicized coefficients and that worked. I tried it with an inverse function (the third link function presently supported), but couldn't get a model to fit, so, I'll create an issue to dig into that more deeply, if this sounds good!

Great! I'll merge this and delete the branch. This should address #1, minus the minor issue you mention, which we can create a new branch/issue/PR for.


Control SyneTune logging verbosity

Is your feature request related to a problem? Please describe.
When using Renate with SyneTune the logs are cluttered with a large number of less-important information.
I did not find a way to control that such that only a minimal amount of information is logged (allowing users to focus on more important aspects logged by other libraries used in the training). If a solution already exists, it would be great to highlight it in the documentation.
Describe the solution you'd like
I would like to be able to log a minimal amount of information from Syne Tune.
I can provide a list of the the information I would like to log if needed.
Describe alternatives you've considered
Renate is currently bypassing the Syne Tune logging in favor of its own

You can specify the logging level, even for certain modules. You can also switch off status reporting (the big table).
One example for how to change the log level is in:
https://github.com/awslabs/syne-tune/blob/main/benchmarking/commons/hpo_main_common.py#L134

If you want to get information out of Syne Tune during an experiment, we have a callback which is quite powerful. See example here:
https://syne-tune.readthedocs.io/en/latest/faq.html#how-do-i-append-additional-information-to-the-results-which-are-stored


tidak bisa mendownload visual studio code

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Pleas provide a description of the issue in English.


Choose which upload port

ADSBx allows "custom feeds" by changing the default standard port from 30005 to another one which is free, and available from the Custom Feeds map.
Can you add an option during the install to allow the user to change the default port from 30005 to another of their choice?

Easy enough...
Ability to do so add a custom feed port added.
commit 2e5351371a283c882b4c7bc88999d685d2ff692c.


Use Python 3.6 in readthedocs build

Somehow type hint isn't recognized in readthedocs build with Python 3.5. This PR simply bumps up the Python version to 3.6.
https://github.com/databricks/koalas/commit/0470b24b895c7e80fa1dd300837c9d2d234252e4
It was tested in readthedocs.

Resolves #198

cc @rxin

LGTM if it works.
On Mon, Apr 29, 2019 at 10:34 PM Hyukjin Kwon<EMAIL_ADDRESS>wrote:

cc @rxin https://github.com/rxin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/databricks/koalas/pull/202#issuecomment-487825941,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACO6PGZY42CLHP3VXGBNHTPS7K5FANCNFSM4HJITTZA
.

Thanks. let me keep my eyes on master build


Three independant difficulty adjustment algorithms

This PR replaces the single difficulty adjustment algorithm that we used to have with three independent instances of the algorithm. One for each of the PoW seal types.

I've disabled clippy because I need this ready for HK. IDK why clippy won't pass in CI. It passes locally and leaves no error in CI. I've made a followup issue to fix it in #58


Function Capabilities & Zero Copy Arrays

Adds a capabilities property to our pg_proc.prosrc json structure and defines an enum of known function capabilities, which right now only contains ZeroCopyArrays.
If a function is re-compiled (really only via a pg_restore) then we'll use the set of old capabilities and adjust the wrapper code accordingly.
New functions get the ZeroCopyArrays capability by default, which uses pgrx::Array<'a, T> instead of Vec<T>, which old functions would have been using.
Also adds pgrx::Array and friends to plrust-trusted-pgrx.

Reviewed after merge 👍


use expect_none and unwrap_none where it makes sense

@bors r+

:pushpin: Commit 88c88530ec5788ada08bcf23d6c7b149337f0713 has been approved by RalfJung

:hourglass: Testing commit 88c88530ec5788ada08bcf23d6c7b149337f0713 with merge cfd95998d7e4a24672d0b16e71e6e8dd1fdf3dd7...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing cfd95998d7e4a24672d0b16e71e6e8dd1fdf3dd7 to master...


fix workspace persistence issue

What changes are proposed in this pull request?
Fixes some issues with loading and using workspaces in the app
How is this patch tested? If it is not, please explain why.
Using app and session
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[x] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

See above
What areas of FiftyOne does this PR affect?

[x] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Summary by CodeRabbit

New Features

Introduced a clearSpaces function to reset spaces in the MainSpace component.
Added a new enum QueueItemStatus to categorize request statuses.

Improvements

Enhanced performance by using the useMemo hook for filtering pending requests.
Improved code organization by moving QueueItemStatus to a separate constants file.

Bug Fixes

Fixed an issue where spaces could not be set to undefined in the state management.

@imanjra @benjaminpkane confirmed that workspace persistence works when navigating between datasets via the dataset selector in the App (although I believe the real test bed is Teams here, not OSS).
Similar to my comment https://github.com/voxel51/fiftyone/pull/4469#issuecomment-2156058282, I did notice that if I manually remove ?workspace=WORKSPACE from the URL bar and press ENTER, the workspace is added back to the URL.
However, unlike for saved views, if I navigate from http://localhost:5151/datasets/DATASET1?workspace=WORKSPACE directly to http://localhost:5151/datasets/DATASET2, it does correctly load the second dataset without carrying over the workspace.

Similar to https://github.com/voxel51/fiftyone/pull/4469#issuecomment-2156058282, I did notice that if I manually remove ?workspace=WORKSPACE from the URL bar and press ENTER, the workspace is added back to the URL.

This is intentional for both workspaces and saved views to recover app state based on the current session state. However, changing the workspace/view param in URL to some other workspace/view should trigger session to update along with the app.

Updated the base to merge/v0.24.1-to-main


feat: added kubesec

Closes #22

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Functional tests hang on MS Edge 14 on Saucelabs

It was ok on MS Edge 13

We've found only one issue locally: https://github.com/DevExpress/testcafe/issues/875 And the fix is on it's way. It seems like that Edge 14 is just unstable on Saucelabs for some reason. Let's try to turn it on a little bit later.

The problem is gone after migration to BrowserStack


spelling in (Sub-)title?

Un?itialized iterators ==> unINitialized iterators ? (or has a new word entered the language, quite possible Im out of touch)

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

ID: c200cd0a-85db-45ed-8e50-2a5f563d3156
Version Independent ID: d43ff7d9-9c13-7eb3-88e6-c0221a5e8a80
Content: Debug Iterator Support
Content Source: docs/standard-library/debug-iterator-support.md
Product: visual-cpp
Technology: cpp-standard-libraries
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Thanks, @dhoke4tdb. Fixed. How embarrassing :-) The fix will be live Monday.


refactor: move permission migration into executor

Description
Linked issue

Closes #{issue_number}

Benefits
we no longer depend on the ExecutorDataModel::permissions being correct.
It's just for users to query it the same way as ExecutorDataModel::custom_instruction is
Checklist

[ ] I've read CONTRIBUTING.md
[ ] I've used the standard signed-off commit format (or will squash just before merging)
[ ] All applicable CI checks pass (or I promised to make them pass later)
[ ] (optional) I've written unit tests for the code changes
[ ] I replied to all comments after code review, marking all implemented changes with thumbs up

I can't proceed with this PR atm due to #4743

Looks like executor_upgrade_should_run_migration and migration_should_cause_upgrade_event tests are failing

Looks like executor_upgrade_should_run_migration and migration_should_cause_upgrade_event tests are failing

dang. I'll fix


Name conflict

One of the more popular event dispatchers for Riot uses this name
https://github.com/jimsparkman/RiotControl
In a community this small, it's almost certain to cause confusion. I used npm install riot-control instead of npm install riotcontrol and was confused for about an hour before I realized that I hadn't even installed the package I was looking for.

Wow. Talk about a n00b mistake. My b.
Changed everything to RiotFlow. npm install riotflow. Thanks for letting me know! I'm loving Riot and I hope to keep contributing to the community. Apologies to the crew over at RiotControl.


⚠ Turnos 24x7 has degraded performance

In 2b26d88, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1077 ms

Resolved: Turnos 24x7 performance has improved in 9f7fe56.


[Feature][Connector-V2] Add hdfs file json support

Purpose of this pull request
#1946

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

@CalvinKirs pls review.


chore: update FIG chart to the latest released version 3.2.0

@redhatrises there wasn't a 0.4.0 version published in the helm repo for this change. The latest is still 0.3.0
The github action failed due to some unrelated tag already existing https://github.com/CrowdStrike/falcon-helm/actions/runs/9551134785

@redhatrises looks like it published after I posted that.
Testing the new version I found a problem with the way you templated out the config
    #confirm_instance = true
    {{- if .Values.push.aws_security_hub.confirm_instance }}
    confirm_instance = {{ .Values.push.aws_security_hub.confirm_instance }}
    {{- end }}

{{- if .Values.push.aws_security_hub.confirm_instance }} is only evaluated true if confirm_instance is set to true. So it makes it impossible to ever override the default true value which I found out when trying to set it to false. We want to check if the value is defined, not if it is true.
It would be simpler just to remove the conditional and let confirm_instance pick up default value of true if not overridden. This would apply for the other vars too.


(#455) updated mapping list UI

#455

Updated mapping list UI
Introduced page titles/description
Added mapped_at to Mapping model

Important: rake task oneoff:set_mapped_at_for_mappings need to be run after deployment to set up mapped_at date for existing mappings.

Looks good! Shouldn't the rest of the pages have titles, too? E.g. /forgot-password, /view-mappings, /new-mapping, and all the other mapping pages? Or will those be added later?

yes, will add as will work on those pages or related functionality. there is no degradation compared to the current page title, default will be in use.


files.associations stop accepting wildcard

Issue Type: Bug
Until today, using a files.associations with a wildcard, would include all files covered by it.
After updating to todays latest nightly release, that seems to fail.
"files.associations": {
    "Dockerfile-*": "dockerfile",
    "*.yml.*": "yaml",
    "*.tpl": "shellscript",
},

For example, a file called Dockerfile-nginx will be open as Plain Text
VS Code version: Code - Insiders 1.29.0-insider (4626bc0e1b679d555dd6f034ece35ffc394b3bb7, 2018-10-29T06:11:45.152Z)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2800)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.40GB free)

Process Argv
-psn_0_1524084

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

markdown-toc
Ala
1.5.6

spellright
ban
3.0.6

vscode-icontheme-nomo-dark
be5
1.3.6

gitlens
eam
8.5.6

mdhelper
jos
0.0.11

terraform
mau
1.3.7

prettify-json
moh
0.0.3

vscode-docker
Pet
0.3.1

material-icon-theme
PKi
3.6.0

vscode-icons
rob
7.27.0

vscode-nginx
sha
0.5.0

sort-lines
Tyr
1.7.0

@alexandrudima I can still repro this issue with the latest code-insiders release.
Version: 1.29.0-insider
Commit: 97ad72a02c351d6cde0fd32b41d9700a43e5964a
Date: 2018-10-31T06:13:36.475Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64


CON-4928: Added a Functional test for the main Connect Controller

Added the WebTestCaseTrait from the B2B suite and its infrastructure.
Implemented a ConnectControllerTest to check the most used actions.
Refactored all functional tests to use the WebTestCaseTrait

Coverage increased (+3.3%) to 42.374% when pulling 2e1c569656c07fa05461f97de7b18984fa3671ea on CON-4928 into 7f62eabbd29309e44338a17622d01654d0802d35 on master.

Coverage increased (+3.3%) to 42.374% when pulling 2e1c569656c07fa05461f97de7b18984fa3671ea on CON-4928 into 7f62eabbd29309e44338a17622d01654d0802d35 on master.

Coverage increased (+3.02%) to 42.969% when pulling e0c646753afafed73e190ce7f30a72983814988d on CON-4928 into b510aab4097635638067d6e0e1a7dbe43b82c9fb on master.

Coverage increased (+3.02%) to 42.969% when pulling 0aa461b03ce885940fc177e487b86dee2123da55 on CON-4928 into b510aab4097635638067d6e0e1a7dbe43b82c9fb on master.

Coverage increased (+3.02%) to 42.969% when pulling 3f2bd967201dd2965c181e32cfdc7056ddcb0b2f on CON-4928 into b510aab4097635638067d6e0e1a7dbe43b82c9fb on master.


Extended spo web set to control footer visibility #946

Extended 'spo web set' with controlling the footer's visibility #946
spo web set --footerEnabled [footerEnabled]

Coverage remained the same at 100.0% when pulling 8e6cf72164870a4b98e3296c1f8cdfefc5f16eac on rjesh-git:dev into 371caf7fef6d11341b74b6c084bf48c4f4cb1dba on pnp:dev.

Woohoo! That was a quick one! Thanks! We'll review it shortly 👏

Merged manually. Thank you! 👏

I run as below and can't get around footer on or off. Both resulted same which is footer invisible:
SPO web set --webUrl https://abc.sharepoint.com/sites/intranet --footerEnabled true
Site has custom design and custom template with communication. Any idea?

@Asfiti could you please create a separate issue with the exact repro steps that we can use to replicate the issue in our environment? Thank you


Bug 1887392: allow configuring authn/z caches TTLs

This should contribute to reducing traffic inside a cluster since not every metrics request to our operators (once every 30s per operator) will need to pass an authn/z check (tokenreview/SAR).
/assign @sttts
/hold
contains fake bump

/retest
google was out

depends on https://github.com/openshift/api/pull/832

depends on https://github.com/openshift/api/pull/832

/hold cancel
no longer needs API change, the timeout is hardcoded

/hold cancel
no longer needs API change, the timeout is hardcoded

/lgtm
/approve

/lgtm
/approve


Document OpenID Connect flow

We should document how to implement a simple OpenID Connect flow. This has already been demonstrated, so we should write down how to do it.

Consider using @HarryKodden's OIDC PAM module: https://gitlab.kodden.nl/harry/PAM-OIDC-Device-Flow
And the demonstration can be found in the TRiRODS talk from February 2024: https://youtu.be/heFIGvlHhck?si=8jdxT7jR-U74uk9d


Allow dashboards for different groups to have the same name

Fixes https://github.com/ManageIQ/manageiq/issues/18924
This PR should go together with Data Migration PR, which will copy owner_id to group_id

I expected to see a change to a validator in this code, but there isn't one.
I'm also curious if the data migration and most of this code is even needed if we can get away with group_id || owner_id.  I'm sure I'm overlooking something but that's the feeling I get from reading it.

I expected to see a change to a validator in this code, but there isn't one.
No need to change validation in  lib/extentions/ar_miq_sert.rb:

validates_uniqueness_of :name,
                        :scope => [:set_type, :userid, :group_id],
                        :if    => proc { |c| c.class.in_my_region.exists?(:name => c.name) }

the same logic still,  but instead of group_id been nil for dashboards assigned to group (and preventing different group to have the same name) it will be <group_id>
.
.

I'm also curious if the data migration and most of this code is even needed if we can get away with group_id || owner_id.

first  thought is: why to use group_id if owner_id already holds that data, BUT acts_as_miq_set is used in 8 models. It would be a massive refactoring to drop group_id column from miq_sets table and alter validation above - very intrusive changes for this small improvement

without data migration duplicated dashboards for group (the same name) can be created, since old dashboard will have group_id nil and new ones will have


update(integration): move /integration to contrib repo

Signed-off-by: Leonardo Grasso<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area examples

/area rules

/area integrations

/area tests

/area proposals

What this PR does / why we need it:
As discussed with the community we want to move /integrations to the contrib. Thus we have to clean it up here.
Which issue(s) this PR fixes:

Fixes #1145
Special notes for your reviewer:
:point_right: https://github.com/falcosecurity/contrib/pull/3
/hold
/milestone 0.23.0
Does this PR introduce a user-facing change?:

update(integration): move `/integrations` to `contrib` repo

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 67e076680a0565f0d1758d58e07e635e215ba263

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kris-nova
I will cancel "hold" as soon as https://github.com/falcosecurity/falco-website/pull/184 gets done

🧹🧹🧹🧹🧹🧹 Cleaning!

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

As per https://github.com/falcosecurity/falco/pull/1184

/hold cancel

Quick question: Did someone create a new repository for the Falco Puppet module? Did someone update the Puppet forge readme? 😄

Quick question: Did someone create a new repository for the Falco Puppet module? Did someone update the Puppet forge readme?

Hi @JPLachance
the Puppet module was moved here and is currently in sandbox level.
/cc @maxgio92
The Puppet forge readme seems to be outdated :)
Not sure who can help out with that :eyes:

For those who find themselves here trying to find things like puppet modules;
The 'contrib' repos mentioned all over the place are in fact now called 'Falcon Evolution' are found here
https://github.com/falcosecurity/evolution

For those who find themselves here trying to find things like puppet modules;
The 'contrib' repos mentioned all over the place are in fact now called 'Falcon Evolution' are found here
https://github.com/falcosecurity/evolution


Editor and Preview panes syncing

If I use some pictures and stuff that take one line in the editor but a good amount of space in the preview, the scroll easily gets out of sync between the tow panes, Is it possible to fix it?
Thanks

I've tried to fix that a few times but have not come up with a way to do it.

@mike-ward unfortunately I don't have any experience with C# to know what is going on in Markdown Edit. But in online implementations like in stackedit.io, it syncs with the line that it is in the middle of the current  page. Are you rendering the whole markdown using Pandoc into HTML right now? Can you sync with the bookmarks of H1, H2, ... in the page then?

Any solution would require scanning the generated document as you suggest. It's something I want to fix but just not a high priority at the moment.

I looked at StackEdit's sync algorithm briefly. It appears that there are extra class attributes used as anchor points. Sounds like you're familar with the algorithm. If you can explain it I'll take a shot at implementing it.

Honestly I'm not that familiar with the algorithm if you mean what makes it a smooth scroll to different anchors. Even if it jumps to different anchors as you move the cursor in the edit pane, that is great start. On my long documents, the preview pane is useless. It is always out of sync.

OK, I'll have to think about how to approach this issue. Not sure what I could use for anchors at this point.

I was referring to the algorithm to determine scroll position, not the smooth scroll.
I dug into how they do this and it appears they place anchors in both the source and generated output. This works for them because they gernerate a source view with HTML. MDE doesn't do this since it's a desktop editor and does not have to render source in HTML.
This makes it a much harder problem for MDE since there are no anchors to match against.
Something I'm going to try this weekend is to compute the height of the document including images. That should get very close. However, without some type of text anchor, getting spot on will be difficult. Suggestions on a strategy to deal with this are welcomed.

https://github.com/mike-ward/Markdown-Edit/commit/97de2756ba85c46acdc34c687134a22709dbc78d

@mike-ward Thanks for working on this. When is it going to be released compiled?

It's in version 1.21


Database Names with Hyphens cause the error "Can't connect to target database"

If there are hyphens (-) in the database name, then the error "Can't connect to target database" appears otherwise not. For example: ./dbdiff --output="./migrations/today.sql" server1.my-database1:server2.my-database2

I had the same issue and created a fix for this issue.
https://github.com/DBDiff/DBDiff/pull/80
The changes are visible in my fork:
https://github.com/pddm/DBDiff/blob/master/src/DB/Data/LocalTableData.php

There is an improved fix for this issue, which also escapes table names presenting similar issues:
fix(SQL): Escape Database Names
#92 opened on May 19, 2019 by janfa


Support for 3-phase electricity installations?

Currently, homebridge-p1 only reports voltage and electric current for phase 1 and (current) consumption for all phases combined.  This suffices for most consumers, like me, who have a single-phase electricity installation.
The smart meter actually supports 3-phase installations as well, reporting separate (current) consumption, voltage and electric current per phase.  Technically it would be possible to expose these to HomeKit.  It would have to be through a separate accessory, not in interfere with the history in Eve.  Alternatively, homebridge-p1 could report the average voltage and total electric current for the combined phases.
Please let me know your views on this.

Sorry for not having found this threat...
Separate 'devices' like temperature meters with e.g. PowerConsumption L1, PowerConsumption L2, PowerConsumption L3 would be awesome for me.
It would be interesting to have Apple Home be able to automatically switch devices on / off according to current power use. (read: Capacity).
( If power consumption L3 > 3800 AND Temperature < 5C, switch off Electrical Car Charging and switch on Electrical Heater )

Would you need consumption history per phase?  If so, I’ll have to expose three additional accessories, one for each phase, each with their own Eve history.  If not, I could expose a single extra accessory with a service per phase.  What version of DSMR does your meter expose?  Could you run homebridge -D and post the telegram homebridge-p1 logs?
Consumption is a numeric attribute, so you should be able to use it in HomeKit automations.  I haven’t tried this myself, though.  Not sure if Apple’s Home app will support this, but you can probably set up the automations in Eve.

Is this what you're asking for?
[P1] raw data: {"type":"ISK5\2M550T-1012","checksum":"58E5","version":"5.0","lastupdated":"2019-12-29T23:37:14","id":"E0044007118000000","consumption_low":4000.647,"consumption_normal":1154.948,"consumption_back_low":0,"consumption_back_normal":0,"tariff":"low","power":1765,"power_back":0,"failures_short":9,"failures_long":4,"log":{"2018-03-27T06:41:11":180,"2019-01-08T10:59:23":2603},"l1_sags":1,"l2_sags":1,"l3_sags":1,"l1_swells":1,"l2_swells":1,"l3_swells":1,"msg_text":null,"l1_voltage":236.9,"l2_voltage":235.1,"l3_voltage":234.9,"l1_current":0,"l2_current":0,"l3_current":6,"l1_power":132,"l2_power":37,"l3_power":1596,"l1_power_back":0,"l2_power_back":0,"l3_power_back":0}
[P1] data: {"type":"ISK5\2M550T-1012","version":"5.0","msg_text":null,"electricity":{"id":"E0044007118000000","lastupdated":"2019-12-29T23:37:14","tariff":"low","consumption":{"low":4000.647,"normal":1154.948},"power":1765,"failures":{"short":9,"long":4,"log":{"2018-03-27T06:41:11":180,"2019-01-08T10:59:23":2603}},"l1":{"voltage":236.9,"sags":1,"swells":1,"current":0,"power":132},"l2":{"voltage":235.1,"sags":1,"swells":1,"current":0,"power":37},"l3":{"voltage":234.9,"sags":1,"swells":1,"current":6,"power":1596}},"electricityBack":{"id":"E0044007118000000B","lastupdated":"2019-12-29T23:37:14","tariff":"low","consumption":{"low":0,"normal":0},"power":0,"l1":{"power":0},"l2":{"power":0},"l3":{"power":0}}}
[12/29/2019, 11:37:16 PM] [P1] ISK5\2M550T-1012 v5.0
[12/29/2019, 11:37:16 PM] [P1] Initializing platform accessory 'Electricity'...
[12/29/2019, 11:37:14 PM] [P1] listening on /dev/ttyUSB0
[12/29/2019, 11:37:16 PM] [P1] telegram:
/ISK5\2M550T-1012
1-3:0.2.8(50)
0-0:1.0.0(191229233714W)
0-0:96.1.1(4530303434303037313138353239333138)
1-0:1.8.1(004000.647kWh)
1-0:1.8.2(001154.948kWh)
1-0:2.8.1(000000.000kWh)
1-0:2.8.2(000000.000kWh)
0-0:96.14.0(0001)
1-0:1.7.0(01.765kW)
1-0:2.7.0(00.000kW)
0-0:96.7.21(00009)
0-0:96.7.9(00004)
1-0:99.97.0(2)(0-0:96.7.19)(180327064111S)(0000000180s)(190108105923W)(0000002603s)
1-0:32.32.0(00001)
1-0:52.32.0(00001)
1-0:72.32.0(00001)
1-0:32.36.0(00001)
1-0:52.36.0(00001)
1-0:72.36.0(00001)
0-0:96.13.0()
1-0:32.7.0(236.9V)
1-0:52.7.0(235.1V)
1-0:72.7.0(234.9V)
1-0:31.7.0(000A)
1-0:51.7.0(000A)
1-0:71.7.0(006A)
1-0:21.7.0(00.132kW)
1-0:41.7.0(00.037kW)
1-0:61.7.0(01.596kW)
1-0:22.7.0(00.000kW)
1-0:42.7.0(00.000kW)
1-0:62.7.0(00.000kW)
!58E5

Yes, thanks.

Oh FYI: I'm not using gas (stadsverwarming too here) and I'm not supplying power back to the net (no solar).

Yeah, I figured that from the meter data.  The DSMR standard seems to accommodate for city heating, but I have never seen that in real life.
The meter doesn’t report cumulative consumption per phase; I’m not sure how reliably I can compute this from the power over time.  You have a DSMR 5.0 meter, which should update the reading every second.

Yeah, I figured that from the meter data. The DSMR standard seems to accommodate for city heating, but I have never seen that in real life.
The meter doesn’t report cumulative consumption per phase; I’m not sure how reliably I can compute this from the power over time. You have a DSMR 5.0 meter, which should update the reading every second.

current use would be fine for me.

Could you try v0.2.30-6?  Install through:
$ sudo npm -g i homebridge-p1@beta --unsafe-perm

This version should expose three additional services to the Electricity accessory.  These services contain Current Consumption, Electric Current, and Voltage per phase.  The main service also contains Current Consumption, which should be the total consumption over the three phases, as reported by the meter.
Note: when the meter doesn't show three-phase electricity, Electric Current and Voltage are exposed under the main service and no additional services are exposed.

@macdivernl, can you please confirm that the latest version works with three-phase electricity?

I just received a P1 - USB cable, configured this plugin on an old Raspberry running HOOBS and it works with 3 phases, the log looks like this:
[Homebridge] [2/8/2020, 1:59:48 PM] [P1] Electricity: set homekit total consumption from 4672.602 kWh to 4672.603 kWh
[Homebridge] [2/8/2020, 1:59:48 PM] [P1] Electricity: set homekit total consumption low from 2398.165 kWh to 2398.166 kWh
[Homebridge] [2/8/2020, 1:59:49 PM] [P1] Electricity: set homekit current consumption from 799 W to 802 W
[Homebridge] [2/8/2020, 1:59:49 PM] [P1] Electricity L1: set homekit voltage from 231.4 V to 231.3 V
[Homebridge] [2/8/2020, 1:59:49 PM] [P1] Electricity L2: set homekit current consumption from 55 W to 54 W
[Homebridge] [2/8/2020, 1:59:49 PM] [P1] Electricity L2: set homekit voltage from 231.2 V to 231.1 V
[Homebridge] [2/8/2020, 1:59:49 PM] [P1] Electricity L3: set homekit current consumption from 45 W to 46 W
[Homebridge] [2/8/2020, 1:59:49 PM] [P1] Electricity L3: set homekit voltage from 231 V to 231.1 V
[Homebridge] [2/8/2020, 1:59:49 PM] [P1] Electricity L1: set homekit current consumption from 700 W to 703 W
[Homebridge] [2/8/2020, 1:59:50 PM] [P1] Electricity L1: set homekit voltage from 231.3 V to 231.2 V
[Homebridge] [2/8/2020, 1:59:50 PM] [P1] Electricity L2: set homekit current consumption from 54 W to 55 W
[Homebridge] [2/8/2020, 1:59:50 PM] [P1] Electricity L2: set homekit voltage from 231.1 V to 231.2 V
[Homebridge] [2/8/2020, 1:59:50 PM] [P1] Electricity L3: set homekit current consumption from 46 W to 43 W

In the Eve app, I get Electricity consumed in kWh and current usage (W, A & V) for L1, L2 and L3 separately. I don't see the current consumption (3rd line above) which for me would be the most useful info.

Could you please capture and attach a debug log file, see above?
There should be a Total Consumption and Consumption on the main service, and an additional Consumption on each phase service.  Did you try Edit in the accessory window, to see if there's any hidden characteristics?  If not, could you please post a screenshot of the accessory window

Could you please capture and attach a debug log file, see above?

Hmm, I'm using HOOBS that starts automatically, I can't seem to get it to run in debug mode. When I have some more time I can try to set up homebridge-p1 on my main Homebridge where I can examine better.

There should be a Total Consumption and Consumption on the main service, and an additional Consumption on each phase service. Did you try Edit in the accessory window, to see if there's any hidden characteristics? If not, could you please post a screenshot of the accessory window?

Ah, I see it now - when I check the room view in Eve app, I see 'Electricity' in kWH but when I select that, in the following screen shows a lot more detail, including 'Consumption' which is the total of L1, L2 and L3. I don't see how I can show this in the room screen itself though.

I don't see how I can show this in the room screen itself though.

That's up to Eve.  It only shows one line per service in the room view.  It shows what it consideres the main characteristic in a bug font, and sometimes some secondary characteristics in a smaller font.  I haven't found any way to influence this behaviour.
From the screenshot, it looks like you have solar panels?  Is the electricity delivered back indeed reported per phase?

From the screenshot, it looks like you have solar panels? Is the electricity delivered back indeed reported per phase?

Indeed, I have solar panels; will need to wait for better weather before I can check how it shows up in the app. Youless only reports the aggregate, not per phase.

@macdivernl, can you please confirm that the latest version works with three-phase electricity?

It does!!


Stop inputting so much junk

Inputs sent into recent_inputs tend to get cluttered a lot, and that's probably due to how duplicates are being dealt here:
https://github.com/PistolRcks/flfa-project/blob/ab3a88665f8e36dbe8fb87c00b6c3073ef9448c2/util/ComboController.gd#L67-L71
Inputs end up getting reinput when a direction gets released when an attack button is being pressed, or when an attack gets released when a direction is being pressed (or so I hope).

Probably have to redo the if statement (also it looks ugly anyways)

Now realizing I have no time to get this done before the end of Wednesday


After the recipient is successfully uploaded, click the Upload file button again to replace the uploaded file. The page crashes

上传收件人成功后，再次点击上传文件按钮，更换上传的文件，页面崩溃

取消选择本地文件时，去掉请选择文件提示框

fixed this pr


Fix x-ms-examples how to doc pointing to providerhub docs

All changed packages have been documented.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/358/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/358/


[Vue warn]: Invalid vnode type when creating vnode: undefined.

Environment

Operating System: Darwin
Node Version:     v16.13.0
Nuxt Version:     3.0.0-27286006.a08b435
Package Manager<EMAIL_ADDRESS>Bundler:          Vite
User Config:      build
Runtime Modules:  -
Build Modules:    -

Reproduction
branch bug_ts
https://github.com/productdevbook/nuxt3-bug/tree/bug_ts
Describe the bug
yarn dev
Additional context
No response
Logs
ERROR  [proxy] connect ENOENT /var/folders/7c/jdnkxg5n6f3fz8425m0cwbsr0000gp/T/nitro/worker-47129-1.sock                    22:24:22

  at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1161:16)

 ERROR  [proxy] connect ENOENT /var/folders/7c/jdnkxg5n6f3fz8425m0cwbsr0000gp/T/nitro/worker-47129-1.sock                    22:24:38

  at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1161:16)

It will happen with a new line in the string also
throw createError({ statusCode: 404, statusMessage: 'before\nafter' });

16:07:32
[Vue warn]: Unhandled error during execution of setup function                                                                                16:07:32
  at <App>
[nitro] [dev] [unhandledRejection] H3Error: before                                                                                16:07:32
after
    at Module.createError (file:///Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/h3/dist/index.mjs:128:15)
    at Module.createError (/Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/nuxt/dist/app/composables/error.js:37:38)
    at setup (/Users/iamandrewluca/Projects/jagaad-team/my-app/app.vue:11:33)
    at _sfc_main.setup (/Users/iamandrewluca/Projects/jagaad-team/my-app/app.vue:33:23)
    at callWithErrorHandling (/Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:171:22)
    at setupStatefulComponent (/Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:7194:29)
    at setupComponent (/Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:7149:11)
    at renderComponentVNode (/Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:628:17)
    at Module.ssrRenderComponent (/Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/@vue/server-renderer/dist/server-renderer.cjs.js:94:12)
    at default (/Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/nuxt/dist/app/components/nuxt-root.vue:67:37) {
  statusCode: 404,
  fatal: false,
  unhandled: false,
  statusMessage: 'before\nafter',
  __nuxt_error: true
}
[nitro] [dev] [unhandledRejection] TypeError [ERR_INVALID_CHAR]: Invalid character in statusMessage                                                        16:07:32
    at new NodeError (node:internal/errors:387:5)
    at ServerResponse.writeHead (node:_http_server:362:11)
    at ServerResponse._implicitHeader (node:_http_server:305:8)
    at write_ (node:_http_outgoing:867:9)
    at ServerResponse.end (node:_http_outgoing:977:5)
    at Object.errorhandler [as onError] (file:///Users/iamandrewluca/Projects/jagaad-team/my-app/.nuxt/dev/index.mjs:500:18)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Server.toNodeHandle (file:///Users/iamandrewluca/Projects/jagaad-team/my-app/node_modules/h3/dist/index.mjs:1294:9) {
  code: 'ERR_INVALID_CHAR'
}


Error flashes when sending visual selection

System
Iron: commit 746f2...
Nvim: commit b9ceac46... (HEAD as of 24 September 2020)
REPL: ipython
OS: macOS Catalina 10.15.6
Iron config (plugins.lua)
local iron = require('iron')

iron.core.set_config {
  preferred = {
    python = "ipython"
  }
}

Issue
The following message flashes when sending a visual selection using the default bindings:
E5108: Error executing lua /Users/garfield/.vim/plugged/iron.nvim/lua/iron/init.lua:377: Expected 4 arguments
Despite the error message the visual selection is succesfully sent to the REPL and executed.
Expected behaviour
The error message shouldn't flash.
Thanks!

Thanks for the fix. Closing :)


fix(live-13376): show NotEnoughBalanceSwap error

✅ Checklist

[x] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description

show NotEnoughBalanceSwap error

❓ Context

JIRA or GitHub link:
https://ledgerhq.atlassian.net/browse/LIVE-13376
https://ledgerhq.atlassian.net/browse/LIVE-13335

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

Not the correct fix


Regarding results

Hi,
I was wondering if you managed to get any results close to the paper with this repository?
I downloaded the code and seem to get way worst results from the paper. Is it because of the unreported hyperparameters?
Thanks!

@jacobwei
Nope. With this repo I got about there, 10% lower. Definitely nothing close to the range of 70s. In fact, it's worst than LSTMs.
I am trying to reimplement the model myself from scratch. And it also seems like the results are around 65-ish. Have tried various hyperparameters, including different optimizers, learn rates etc. Still the same.

@vanzytay
Thanks for your reply.
Could you please upload your model to your repo ?
I'm a newbie in Tensorflow, it will be a little difficult for me to implement this model myself from scratch.

@vanzytay
Hi vanzytay. Could you tell me why we feed the target information(the answer) to the graph in the test method(model.py line 242 and line 248)?

@vanzytay As I've mentioned in the README, I guess the difference in the results is because of the unreported hyper-parameters. But now that the original author has released his code, I've added this task of reaching the state-of-the-art performance to my todo list.

As I just glanced through the author's code, I see some serious differences:
a) Use Stanford Tokenizer to tokenize the text
b) Replace the aspect term in the original sentence with a placeholder (say '$T').
c) ...

Can you share where is the location of original author code?
Thank you

http://ir.hit.edu.cn/~dytang/paper/aspect_memnet/src.zip

Thanks @ganeshjawahar,
unfortunately it is written on Java :(

I tried to improve the performance by tweaking the model and parameters, and finally got accuracy of 0.69 for Laptop, and 0.77 for Restaurant, which is still not perfect, but much closer to those ofthe original paper (0.72 for Laptop, 0.809 for Resturant). You can checkout my fix here https://github.com/pcgreat/mem_absa if you are interested.
@ganeshjawahar thanks for your great implementation, especially for the attention part. I also want to take a look at original author code, but the link is invalid now. Would you please share it with me if the license permits?

The code of the original author can't be downloaded anymore, do you have a copy file of the code?
Could you please send it to me ?
Thans.

@pcgreat  In your code i noticed that you feed the target (the sentiment ground truth) to the graph in the test method (model.py), can y explicate me why please ?

@nkooli yeah, target is fed into the model to calculate the test loss. But don't worry, the trained model won't be changed by this, as long as the self.optim is not triggered (no back prorogation, no update to the variables). But if you are not assured, you are welcome to move the target out of model and compare the results

@ganeshjawahar The code from on http://ir.hit.edu.cn/~dytang/paper/aspect_memnet/src.zip cannot be downloaded，would you please share it with me , if  possible?

@pcgreat I downloaded the code and seem that the result is not perfect(0.7 for restaurant) from https://github.com/pcgreat/mem_absa. You have mentioned in the README, the model can improve the accuracy by 8%, the accuracy of restaurant is around 78%. Do you have any idea for me?


Drop sheet columns that can be added as part of post-processing step

We will be dropping (from the Sheets, not the final CSV) : latitude, longitude, geo_resolution, location, admin3, admin2, admin1, country_new, admin_id.
We will be dropping from both sheets and CSV: wuhan(0)_not_wuhan(1).
Let's think a bit more about data_moderator_initials (it’s mostly empty), if we want to give some feedback perhaps (or if all errors come from the same person), I don't think it belongs in the final csv though.

@tbrewer-healthmap question related to #21 : does ID has to be part of the csv output? If so why?

I think its' best to keep the ID as part of the CSV output, mostly because I think the final output should keep the same format in case anything is depending on that, also the map script keeps the IDs when generating a certain file that we send to an S3.
I also think its good to keep it to differentiate between the different sheets. That way if we see an issue there we know where to look.
That being said, if you think its causing too many problems I'm open to debating it.

I haven't dropped the columns from the sheets yet, doing this now.


Link to documentation about the deprecation in the deprecation message (or make the deprecation message itself clearer)

It would be nice if the deprecation message linked to documentation that actually provided details on the issue. The following is not clear to anyone who doesn't use toml files for config (I assume this is clear for someone who knows the toml syntax based on it being the message you chose).
mise WARN  deprecated [TOOL_VERSION_ALIASES]: tool version aliases should be `alias.<TOOL>.versions.<FROM> = <TO>`, not `alias.<TOOL>.<FROM> = <TO>`

This would be less of a problem if this was implemented but as is it is a frustration with a pretty simple fix as you already have a nice docsite.

it just means instead of this:
[alias.node]
lts = '20'

do this:
[alias.node.versions]
lts = '20'

I think it's clear

That's your choice as the maintainer but I'm giving you direct user feedback that it is not. take that as you will

Also, it would take almost no time to have simply had that example on a page of your documentation that the warning message links to. That example is exactly what is needed.


Jesse: Watch (?-i:47hlU7Y%C2%A0to)(?#bit.ly)

Jesse requests the watch of the watch_keyword (?-i:47hlU7Y%C2%A0to)(?#bit.ly). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:47hlU7Y%C2%A0to)(?#bit.ly) has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Rejected by cocomac in Charcoal HQ. Reason: 'Rejected per authors request due to mistake in URL'


Installing Simmate on Lenovo Yoga 6

Describe the bug
Hi,
When following the simmate tutorial: https://jacksund.github.io/simmate/getting_started/installation/create_your_env/, I ran into an issue with installing simmate.
To Reproduce
(my_first_env) PS C:\Users\rpowe\Desktop> conda install -c conda-forage -n my_first_env simmate
Collecting package metadata (current_repodata.json): failed
CondaSSLError: OpenSSL appears to be unavailable on this machine. OpenSSL is required to
download and install packages.
Exception: HTTPSConnectionPool(host='conda.anaconda.org', port=443): Max retries exceeded with url: /conda-forage/win-64/current_repodata.json (Caused by SSLError("Can't connect to HTTPS URL because the SSL module is not available."))
Error
Copy complete error message here.

Versions

Operating System: Windows
Python version: 3.10.6
Simmate version:

Additional details
No response

Hey! Sorry you ran into this.
It looks like there's a typo in your install command:
conda install -c conda-forage -n my_first_env simmate

You have conda-forage when it should be conda-forge.

Oh shoot, sorry for the mistake! Thank you!
On Mon, Oct 31, 2022 at 8:37 PM Jack Sundberg @.***>
wrote:

Hey! Sorry you ran into this.
It looks like there's a typo in your install command:
conda install -c conda-forage -n my_first_env simmate
You have conda-forage when it should be conda-forge.
—
Reply to this email directly, view it on GitHub
https://github.com/jacksund/simmate/issues/308#issuecomment-1297857842,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A35UDJ3S7S5TFRQLP7B3V5LWGBQ5LANCNFSM6AAAAAARTS2RMY
.
You are receiving this because you authored the thread.Message ID:
@.***>

No worries! If you run into anything else, feel free to post here again


Ability to read the transaction logs

Hello @jmrossy,
Thanks for this wrapper.
While using the wrapper I came across an issue: I'm unable to access the transaction logs.
In my case, an NFT is minted using this wrapper in our infra and there is no way to get the tokenID since that can be obtained from transaction logs.
Typically while using WAGMI and after the mint function in the transaction log I can find the tokenID something like this

txData.logs[0].topics[3]

    tokenContract: txData.to,
    tokenID: parseInt(txData.logs[0].topics[3] as string, 16).toString(),
    tokenMintingTx: txData.transactionHash,
    nftOwner: txData.from,

Is it possible to get the same outcome from the wrapper? if so how?
Thank you.

@Ekaanth This wrapper doesn't change logs in any way. If logs are not being included, it's likely because the RPC provider you're using has chosen not to include them. See CeloProvider for details about what this wrapper does exactly.


Parallel append

Currently the append method on the Array class obtains a lock on the array, which means all append operations are forced to be sequential. An alternative approach would be to implement append by locking the array in order to resize, but then locking only at the chunk level when setting data to be appended into the newly extended region. This would allow multiple append operations to be working in parallel, because they would be writing into non-overlapping regions of the array, while still maintaining the same append semantics.

Maybe have a method extend() which returns the new shape. If locking is handled within extend then there may be no need to keep the lock during the rest of append?