private type UploadThroughputCheckFuture in public interface

Error found today in trying to build, after a cargo update updated use of this library from... I want to say 1.19.x or 1.20.0, to 1.23.0.
Using Rust 1.73.0.
Cargo.toml uses

aws-config = "1.1"
aws-sdk-s3 = "1.20"
and Cargo.lock specifies aws-config 1.2.0 and aws-smithy-runtime 1.3.0.

As all the background AWS libraries and dependencies have changed and are maintaining these two versions regardless of a rollback in Cargo.toml, it's unclear what I need to roll back to, although it seems like this could be rectified with an appropriately-placed pub?
Compiling aws-smithy-runtime v1.3.0
error[E0446]: private type `UploadThroughputCheckFuture` in public interface
   --> /home/imann/.cargo/registry/src/index.crates.io-6f17d22bba15001f/aws-smithy-runtime-1.3.0/src/client/http/body/minimum_throughput.rs:333:1
    |
213 | / pin_project_lite::pin_project! {
214 | |     /// Future that pairs with [`UploadThroughput`] to add a minimum throughput
215 | |     /// requirement to a request upload stream.
216 | |     struct UploadThroughputCheckFuture {
...   |
231 | |     }
232 | | }
    | |_- `UploadThroughputCheckFuture` declared as private
...
333 | / pin_project_lite::pin_project! {
334 | |     #[project = EnumProj]
335 | |     pub(crate) enum MaybeUploadThroughputCheckFuture {
336 | |         Direct { #[pin] future: HttpConnectorFuture },
337 | |         Checked { #[pin] future: UploadThroughputCheckFuture },
338 | |     }
339 | | }
    | |_^ can't leak private type
    |
    = note: this error originates in the macro `$crate::__pin_project_reconstruct` which comes from the expansion of the macro `pin_project_lite::pin_project` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0446]: private type `UploadThroughputCheckFuture` in public interface
   --> /home/imann/.cargo/registry/src/index.crates.io-6f17d22bba15001f/aws-smithy-runtime-1.3.0/src/client/http/body/minimum_throughput.rs:333:1
    |
213 | / pin_project_lite::pin_project! {
214 | |     /// Future that pairs with [`UploadThroughput`] to add a minimum throughput
215 | |     /// requirement to a request upload stream.
216 | |     struct UploadThroughputCheckFuture {
...   |
231 | |     }
232 | | }
    | |_- `UploadThroughputCheckFuture` declared as private
...
333 | / pin_project_lite::pin_project! {
334 | |     #[project = EnumProj]
335 | |     pub(crate) enum MaybeUploadThroughputCheckFuture {
336 | |         Direct { #[pin] future: HttpConnectorFuture },
337 | |         Checked { #[pin] future: UploadThroughputCheckFuture },
338 | |     }
339 | | }
    | |_^ can't leak private type
    |
    = note: this error originates in the macro `$crate::__pin_project_make_proj_ty` which comes from the expansion of the macro `pin_project_lite::pin_project` (in Nightly builds, run with -Z macro-backtrace for more info)

For more information about this error, try `rustc --explain E0446`.
error: could not compile `aws-smithy-runtime` (lib) due to 2 previous errors

Hey @i-mann, thanks for submitting this issue. We'll look at this today.

Hi @i-mann, can you share with us the output of running the following?
rustc --version

Hi @ysaito1001 - thank you  - I was on 1.73 and upgrading to 1.77 looks to have resolved it. Do you have any backwards compatibility guarantee that you look to uphold? Just aware that Rust is releasing a minor version almost monthly...!
Thanks for the fast response too @Velfi

We do. It's stated in the README: https://github.com/awslabs/aws-sdk-rust?tab=readme-ov-file#supported-rust-versions-msrv
Of course, now that I say that and read that section, I realize it's still mentioning "alpha" 😅 It's still correct, however. We currently support two Rust releases prior to the latest (the Rust patch versions are excluded, so if Rust releases 1.80.0, 1.80.1, and 1.80.2, that won't make the minimum supported version 1.80.0).

:facepalm: of course.
Saw your PR - thanks for responding quickly and updating the readme! As background it was a little difficult digging into this as this was a transitive dependency, and I haven't touched Rust and my Rust env for 6 months...! Good to close.
Thanks again.

I also ran into this now. Would it be possible to add the MSRV into the Cargo.toml so this information is directly available instead of having to check if it actually works?

As a matter of fact, it is: https://github.com/smithy-lang/smithy-rs/pull/3601
Thanks for the suggestion.


Fix test cases

If the resident idp is updated without properly setting idp properties, it will override the expected properties in the resident idp. Without these properties, authentication flow will not work properly.
There should be a array size check for the below code, as there can be ArrayIndexOutOfBoundsExceptions.
https://github.com/wso2-extensions/identity-governance/blob/v0.1.15/components/org.wso2.carbon.identity.recovery/src/main/java/org/wso2/carbon/identity/recovery/util/Utils.java#L286

Thanks @madurangasiriwardena for the contribution. Much appreciated..!


Star wildcard works only in last "path" segment

It seems that * wildcard works only in the end of path and when it used with some path-parameter.
For example url /post/123/comments matches to route /post/:id*. And matches.id == "123/comments"
But if we use * in the middle of query it does not work:
const App = () => (

    <Router>
      <Home path="/*/" />
      <Profile path="/*/profile/:user?" />
      <Error type="404" default />
    </Router>

);
const Header = () => (

      Home
      Profile
      John
      Error

);

This config always show "404" page. See JSFiddle

I think it is because of exec() function in utils.js:
export function exec(url, route, opts=EMPTY) {
  for (let i=0; i<max; i++) {
    if (route[i] && route[i].charAt(0)===':') {
      ... // star wildcard handled only here
    }
    else if (route[i]!==url[i]) {
      // but not here
      ret = false;
      break;
    }
  }
...

Right now * is just a "splat" modifier, the standalone asterisk isn't supported.  I'd love to add support for it though!  It seems like the second else if there could just be changed to:
else if (route[i]!=='*' && route[i]!==url[i]) {
  ret = false;
  break;}


ardupilot real-time ros wrapper

Hello,
Do you have a version of the file "realtime_ros_wrapper_infogain_ardupilot.py" that can be used to fly with sevae-ORACLE? Because in this file is not consider at all the latent space that should be generated from the autoencoder.
Thank you.

Sorry for the late reply, you could make a subscriber for the latent vector in "realtime_ros_wrapper_infogain_ardupilot.py" like this:
sevae_realtime_ros_wrapper_infogain_ardupilot.zip

Thank you for the support!


2020-05-21 tc131 & tc133 M1SS + tc1514 M2SS "tau < orig_time..." execution error with actual time introduced with 786bf37

Issue
When running tc131, tc133 M1SS and tc1514 M2SS using actual time with commit 786bf37. This error did not occur for these testcases with the previous commit.
> results[[11]]
$Test
[1] "test_base.actual.tc133.M1.SS"
$Computation
[1] "Error in value[[3L]](cond) : \n  Error in if (tau < orig_time[length(orig_time)]) {: missing value where TRUE/FALSE needed\nFor SDEID 21421443\n"

Execution error still occurs with commit e7df513, however with a new error message. Error message provided below for tc1514.
Error in value[[3L]](cond) :
  Error in isTRUE(time_min_range <= lastttime): object 'lastttime' not found
For SDEID 50896291
In addition: Warning messages:
1: In run_computation(data = d, map = mct, flag = flags, parameterset = parameterset) :
  'TAUs and TOLDs values are not provided via 'flag'! Using values provided via 'data' instead (if present)!
2: In run_M2_SS_computation(data = merged_data, map = map_data, method = method,  :

This issue no longer occurs with the latest commit cca452d. Closing this issue.


Allow navigation through resources with critical signals

A new progress bar showing data on resources with critical signals is introduced in #1029.
This progress bar should be made clickable such that each click brings the user to resources with critical signals and no attributions (high criticality first).
Part of #1012.

completed in #1272


Labeling with initial caps - header/footer, elsewhere

We are planning on employing consistency across the site, in terms of what gets caps. In titles we ALWAYS want initial caps on every word in the title. There's places where we need your help to do this.
Header:
Contribute To BSSw
Footer:
Follow Our RSS Feed
Home:
Resources For Developer Productivity And Software Sustainability
Blog:
Subscribe To Our Email Digest
Category pages (Better Planning, Better Development, Better Performance, etc.)
For topics > See All XX Resources

Fixed!

In Header, can we lower case the "w" in: Contribute To BSSw
and
Category pages (Better Planning, Better Development, Better Performance, etc.)
in topic links I'm still seeing initial lower case, should be: See All XX Resources

Should be fixed!


s do not replace symbol on the cursor

Observed behavior
pressing s do not replace symbol on the cursor - it's pasting >2 to the command line
Expected behavior
I expected to replace symbol on the cursor and change to insert mode
Steps to reproduce

Select some line of code
Press s

This is intentional. s and S have been replaced by evil-snipe (which provides 2-character versions of the f/F motion keys, ala vim-seek or vim-sneak).
It was done because the original substitution behavior of s and S can already be preformed with cl and cc respectively.
If you want to restore the old behavior, simply disable evil-snipe-mode:
;; in ~/.doom.d/config.el
(after! evil-snipe
  (evil-snipe-mode -1))

Otherwise, I suggest using cl and cc instead.

@hlissner it's nice to know. What do you think if we'll add it to the wiki?

That's a good idea. I will be publishing some new documentation soon (including a new FAQ with common issues), I will be sure to add this to it. Thanks! In the meantime, I'll consider this resolved.


New release?

I'm keen to get the changes in HEAD (particularly the fixes to font: inherit on form elements) through into the normalize-rails gem. Our policy is that we don't release a new gem until those changes are tagged in a release.
Would you mind releasing a new version? :)

Thanks for your patience. 5.0.0 has been released!

Thanks for that! :D


Update privileged-access-strategy.md

Fixed some typos

Thanks for your contribution! Resolved in the linked PR that should publish this afternoon.
#please-close


Can I have two vaults of ASR with a same vCenter?

I'm planning deploy two vaults of ASR, the first with a Standard Storage account and the second one with a Premium Storage account. This is for the performance required of some virtual machines in my vCenter, I have only one vCenter. Can I have two vaults of ASR protecting virtual machines from an only one vCenter?.

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

ID: 279a9fc5-6bcb-957f-a398-e675650eace8
Version Independent ID: 896f6892-2b3d-6181-c244-a22d6803b7b2
Content: Common questions about VMware to Azure disaster recovery with Azure Site Recovery
Content Source: articles/site-recovery/vmware-azure-common-questions.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

@JeissonL vCenter is not connected to Vault, rather vCenter is added to Configuration Server(CS) and CS is registered to Vault. With that said, you should be able to have 1 CS per Vault and add the vCenter in both the CS servers and should be able to achieve what you're planning. But Honestly I would suggest to avoid and go with the below option.
You do not have to have two vaults for the purpose of Standard vs Premium storage. Storage type is chosen at the replicated item level setting when enabling replication.
If your requirement is to replicate VMs of high performance to premium storage and the other VMs to a standard storage, you can do so while enabling replication and defining the storage account in the target.
Hope that clarifies!


Update on Mirror & Coding Guidelines

Decimal values are replaced with Hex values in Cabin_Climate_topics proto.
Same will applied to rest of the project. once its accepted.

Updated Climate for coding guidelines
Updated Mirror with additional services

@ravurinaresh why (hex in lieu of decimal)?

I had similar question.
Also which coding guideline being updated ?

Hello,I couldn’t find coding guidelines for COVESA. Can you please point to me? If doesn’t exist. May be better develop coding guidelines by adopting from AUTOSAR?Usage of HEX avoid confusion and make it clear while debugging and also avoid mistakes while coding like usage of 08 considers as Octal number rather than decimal number in some occasions.How to stop a compiler from assuming octal?stackoverflow.comRegards,Naresh RavuriSent from my iPhoneOn Mar 25, 2024, at 9:30 AM, halimragab @.***> wrote:
I had similar question.
Also which coding guideline being updated ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Hi @ravurinaresh let us discuss in a call when we can and we can decide what to do with this PR

Sure. Are there any weekly calls Scheduled?
Regards,
Naresh Ravuri
On Tue, Apr 9, 2024 at 1:03 PM halimragab @.***> wrote:

Hi @ravurinaresh https://github.com/ravurinaresh let us discuss in a
call when we can and we can decide what to do with this PR
—
Reply to this email directly, view it on GitHub
https://github.com/COVESA/uservices/pull/7#issuecomment-2045688412, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AT4BLOSDT3MDHN3K2XH5ESLY4QNPRAVCNFSM6AAAAABFF54D22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBVGY4DQNBRGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Naresh Ravuri.


Test Case Road Test Failed

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
Test Case Details:

Expected behavior
The distance between the front of the ego and the set target after reaching the modified goal is less than D2[-2,2].
Actual behavior
The distance between the front of the ego and the set target after reaching the  modified goal  is greater than D2[-2,2], which is approximately 4 meters.
test video:
https://github.com/autowarefoundation/autoware.universe/assets/104069308/fa7eca34-be3f-40be-81db-4c3ec6d99590
Steps to reproduce
please refer to test case details.
Versions

OS: Ubuntu20.04 x86-amd64
ROS2: Humble
Autoware: release/2023.09

Possible causes

Is the range setting of D2 parameter reasonable in this test case?
How to adjust Autoware parameters to make the modified goal meet the test requirements?

Additional context
No response

good afternoon，mist-san can you assign an engineer to take a look at this one @mitsudome-r

@cyn-liu
Could you please share the scenario and the map file?

@mehmetdogru
this is our test scenario:

map files: download link

Thanks @cyn-liu
Could you please share as well the scenario file created with scenario editor if possible so that issue can be reproducible using ssv2 with your parameters specified in the scenario?

Thanks @cyn-liu
Could you please share as well the scenario file created with scenario editor if possible so that issue can be reproducible using ssv2 with your parameters specified in the scenario?

Thanks for your replay, but we don't have scenario file.
we use real vehicles for testing on the road.

I have checked the original scenario definition.
I think the expectation is that you manually set the goal near the target vehicle within [-2 m , 2 m] in distance, which would be too close for Autoware vehicle to park without collision. Then, the Autoware should automatically modify the given goal by appropriately finding an open space in road shoulder and park to the modified goal.

I have checked the original scenario definition. I think the expectation is that you manually set the goal near the target vehicle within [-2 m , 2 m] in distance, which would be too close for Autoware vehicle to park without collision. Then, the Autoware should automatically modify the given goal by appropriately finding an open space in road shoulder and park to the modified goal.

Thanks for your clarification, If the scenario definition is as you explained, then our road test  is successful.

I have checked the original scenario definition. I think the expectation is that you manually set the goal near the target vehicle within [-2 m , 2 m] in distance, which would be too close for Autoware vehicle to park without collision. Then, the Autoware should automatically modify the given goal by appropriately finding an open space in road shoulder and park to the modified goal.

Thanks for your clarification, If the scenario definition is as you explained, then our road test is successful.

Hello @cyn-liu. In this case, can we close this task?

I have checked the original scenario definition. I think the expectation is that you manually set the goal near the target vehicle within [-2 m , 2 m] in distance, which would be too close for Autoware vehicle to park without collision. Then, the Autoware should automatically modify the given goal by appropriately finding an open space in road shoulder and park to the modified goal.

Thanks for your clarification, If the scenario definition is as you explained, then our road test is successful.


Improve Sub-Navigation Styling

This is a small PR to try to improve the sub-navigation styling. Currently, it looks a bit "clunky". This looks to bring it in line with the Filament styling.
BEFORE

AFTER

@zepfietje made this new PR to target 3.1.
FYI, I'm also looking to add the option to align the sub-navi to the right. I'll probably add to this PR if that's OK?

@glennjacobs, it still targets 3.x currently. 🤣
Feel free to add the new right-aligned API, though a new PR would be easier to review while already merging in the styling changes.

@glennjacobs, it still targets 3.x currently. 🤣
Feel free to add the new right-aligned API, though a new PR would be easier to review while already merging in the styling changes.

Oh, £$%£$% haha!
I'll sort it later... was sure I branched off 3.1...


ci: add publish-image workflow

This PR adds an action that will publish the ghcr.io/bonial-international-gmbh/pod-image-swap-webhook docker image.
The following tags are published automatically:

latest on push to the main branch
v${VERSION} when a v* tag is pushed
pr-${PR_NUMBER} when a PR is created/updated

Here is the link to the image that resulted from this PR: https://github.com/Bonial-International-GmbH/pod-image-swap-webhook/pkgs/container/pod-image-swap-webhook/17399805?tag=pr-2

Looks good but maybe embracing more security in this step is not a bad idea, something like :
https://github.com/marketplace/actions/container-image-scan


[admin] Removing errant code page

I think this page was accidentally added to the wrong repo.

Docs Build status updates of commit 1eb020c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

javascript/api/custom-functions-runtime/customfunctions.cancelableinvocation
:white_check_mark:Succeeded

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


pygame compiled widget ValueError

2.0.5
Commercial: None
Python: 3.11.7 (tags/v3.11.7:fa7a6f2, Dec  4 2023, 19:24:49) [MSC v.1937 64 bit (AMD64)]
Flavor: CPython Official
Executable: C:\Users\Divine\Desktop\OSS contributions\nuitka-toxemon\venv\Scripts\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.39.33519\bin\Hostx64\x64\cl.exe (cl 14.3).

standalone build
this is what i managed to boil down the code to
import pygame
import pygame_menu
from pygame_menu import themes

pygame.init()
surface = pygame.display.set_mode((600, 400))

def start_the_game():
    mainmenu._open(loading)
    pygame.time.set_timer(update_loading, 30)

mainmenu = pygame_menu.Menu("Welcome", 600, 400, theme=themes.THEME_SOLARIZED)
mainmenu.add.text_input("Name: ", default="username")
mainmenu.add.button("Play", start_the_game)

loading = pygame_menu.Menu("Loading the Game...", 600, 400, theme=themes.THEME_DARK)
loading.add.progress_bar(
    "Progress",
    progressbar_id="1",
    default=0,
    width=200,
)

arrow = pygame_menu.widgets.LeftArrowSelection(arrow_size=(10, 15))

update_loading = pygame.USEREVENT + 0

while True:
    events = pygame.event.get()
    for event in events:
        if event.type == update_loading:
            progress = loading.get_widget("1")
            progress.set_value(progress.get_value() + 1)
            if progress.get_value() == 100:
                pygame.time.set_timer(update_loading, 0)
        if event.type == pygame.QUIT:
            exit()

    if mainmenu.is_enabled():
        mainmenu.update(events)
        mainmenu.draw(surface)
        if mainmenu.get_current().get_selected_widget():
            arrow.draw(surface, mainmenu.get_current().get_selected_widget())

    pygame.display.update()

issue will be raise once you click on the "play button" on the screen, it results in the dollowing error
Traceback (most recent call last):
  File "C:\Users\Divine\Desktop\OSSCON~1\NU00F8~1\MAIN~1.DIS\main.py", line 43, in <module>
  File "C:\Users\Divine\Desktop\OSSCON~1\NU00F8~1\MAIN~1.DIS\pygame_menu\menu.py", line 2566, in update
  File "C:\Users\Divine\Desktop\OSSCON~1\NU00F8~1\MAIN~1.DIS\pygame_menu\widgets\core\widget.py", line 2762, in update_menu
  File "C:\Users\Divine\Desktop\OSSCON~1\NU00F8~1\MAIN~1.DIS\pygame_menu\widgets\widget\button.py", line 226, in update
  File "C:\Users\Divine\Desktop\OSSCON~1\NU00F8~1\MAIN~1.DIS\pygame_menu\widgets\core\widget.py", line 1260, in apply
  File "C:\Users\Divine\Desktop\OSSCON~1\NU00F8~1\MAIN~1.DIS\pygame_menu\widgets\core\widget.py", line 1588, in get_value
ValueError: Button<"40aa7dcd-4a59-4f5e"> does not accept value

For me it's crashing on fonts missing, I am including package data. Also please always include PyPI package names to use.

For me it's crashing on fonts missing, I am including package data. Also please always include PyPI package names to use.

ah apologies, i meant to make a PR with the pygame_menu package data, you probably figured it out already

- module-name: "pygame_menu"
  data-files:
    dirs:
      - 'resources'```

Figure, I managed to reproduce it, but the function there is a static raise, question is what is there in standard Python. What kind of UI is that, it seems not Qt, is this homebrew of pygame via say SDL?

Figure, I managed to reproduce it, but the function there is a static raise, question is what is there in standard Python. What kind of UI is that, it seems not Qt, is this homebrew of pygame via say SDL?

are you asking if pygame_menu is a homebrew of pygame? as far as i know no, it's built on top of normal pygame to add extra functionality.

For rendering the widgets. It seems it constructs internally when compiled something that will not work, probably due to something missing. I don't know what renders the button and why it would have an on click that statically raises, there must have been something that fails to be present.

Kevin, can you figure out, what kind of object there is in pure Python in pygame_menu there to receive the click event and not complain about it statically. I cannot do that kind of research really well, as I have like zero clue of what objects it constructs. From past experience, it might be using some sort of backends, and depending on availability construct different object trees, must be that kind of thing somehow.

Hello, chiming in from a referral to this issue, it seems the issue might be related to exception handling somewhere in, I presume, nuitka. Looking at the pygame-menu source code, it appears it is meant to raise a ValueError, however, that happens inside a try/except ValueError block.
So the call comes down to this bit right here, the try/except block
https://github.com/ppizarror/pygame-menu/blob/b00b6ce186f01ccc39fb407c80ac479816c45c3d/pygame_menu/widgets/core/widget.py#L1259-L1262
So, then that calls self.get_value which, unless otherwise implemented, simply raises a ValueError which is apparently expected behaviour (it is being caught in CPython)
https://github.com/ppizarror/pygame-menu/blob/b00b6ce186f01ccc39fb407c80ac479816c45c3d/pygame_menu/widgets/core/widget.py#L1577-L1588

Thanks for your analysis @Matiiss that is great.
I think I know now, what's going on. Nuitka is recognizing the type to be a list, and optimizes into a list.insert special node, and thinks somehow that it cannot raise.
This contains a lot of false assumptions in class ExpressionListOperationInsert right there:
    def mayRaiseException(self, exception_type):
        return self.subnode_list_arg.mayRaiseException(exception_type)

    def mayRaiseExceptionOperation(self):
        # TODO: We do not yet have isIndexable from the type shape.
        return (
            self.subnode_item.isExpressionConstantRef()
            and self.subnode_item.isIndexConstant()
        )

That is a grave bug. First of all the subnode_item is the second argument, and not the index, which probably is very good at hiding its bug, but that should have been subnode_index. Then of course, subnode_item and subnode_index are neither considered in mayRaiseException which makes this totally embarrassing.
I am absolutely to blame here of apparently copy&pasting code for insert and not properly adapting it. This will be fixed in 2.1.1 for sure.

Also excuse me for misreading your code, I thought I saw a static raise there, that doesn't exist. That's only explained by Nuitka usually not being caught with errors other than not including something required.

Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback if this is working is very welcome, just please do not share plans of doing it, but rather confirmations or denials of it working.

This is part of the 2.1.1 hotfix release I made


add words from current buffer to completion list

I think this should be behind a flag, and I don't this this should affect people using the completion via Omni completion

I think this should be behind a flag, and I don't this this should affect people using the completion via Omni completion

make optional in https://github.com/yegappan/lsp/pull/284/commits/f2c8255cd0bb7a3be344b8341256f36f88f757dc

I am including matches from current buffer in my plugin completion-buzz
I used function from vim-CompleteHelper in the beginning but removed dependency on it by including library function in the plugin itself and removing code not needed.
That library functtion is fast. I do not get lag for more than 15000 lines of code for current buffer match.
But I am including only matches rather than collecting every word.

I am including matches from current buffer in my plugin completion-buzz
I used function from vim-CompleteHelper in the beginning but removed dependency on it by including library function in the plugin itself and removing code not needed.
That library functtion is fast. I do not get lag for more than 15000 lines of code for current buffer match.
But I am including only matches rather than collecting every word.

thanks, I'm currently looking for acceleration options and will definitely consider the solution from your plugin


Stopped Updating

Since this afternoon (27 Aug) my Alpha ESS sensors stopped updating. I tried to uninstall and reinstall the integration, and now I get the following after entering the correct login credentials:

Anybody else experiencing problems?

My AlphaESS is also showing as Unavailable. The AlphaESS app is working fine.
Get Outlook for Android

From: KortgatGH @.>
Sent: Saturday, August 27, 2022 3:26:56 PM
To: CharlesGillanders/homeassistant-alphaESS @.>
Cc: Subscribed @.***>
Subject: [CharlesGillanders/homeassistant-alphaESS] Stopped Updating (Issue #28)
Since this afternoon (27 Aug) my Alpha ESS sensors stopped updating. I tried to uninstall and reinstall the integration, and now I get the following after entering the correct login credentials:
[image]https://user-images.githubusercontent.com/49630727/187034521-205fc87e-e0d8-4532-9f35-9fc310740def.png
Anybody else experiencing problems?
—
Reply to this email directly, view it on GitHubhttps://github.com/CharlesGillanders/homeassistant-alphaESS/issues/28, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AK7ZPIPGRLB2KHNNWHOHVCDV3IQTBANCNFSM57ZNBIXA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

If only we could interface directly to our inverters and not rely on the Alpha server. I wonder if Alpha made a small change that broke the Home Assistant integration without affecting their own mobile and web apps.

I’m awY this weekend, I’ll take a look at whatever changed as soon as I can.

HA logs show the following:
2022-08-27 15:33:58.273 WARNING (MainThread) [homeassistant.config_entries] Config entry<EMAIL_ADDRESS>for alphaess integration not ready yet; Retrying in background
(Email address changed)

Same issue here, however the values are still being returned by DasLetzteEinhorn's add in:
https://github.com/DasLetzteEinhorn/AlphaESS_Monitor_Hass
So something odd has happened with Charles values and not DasLetzteEinhorn

@Spookster same for me, Alpha ESS to MQTT add-on by MarioCake still returning values (DasLetzteEinhorn addon).
Thanks for your work @CharlesGillanders, in your own time :)

I use a bit of a failsafe in using both integrations
eg:
   - name: "Const Solar Battery SOC"       unit_of_measurement: "%"       device_class: battery       state: >         {%if states('sensor.alxxxxxxxxxxxxxxxx_battery_soc') == 'unavailable' %}         {{ states('sensor.solar_battery_soc') |int   }}         {%else%}         {{ states('sensor.alxxxxxxxxxxxxxxxx_battery_soc') |int   }}         {%endif%}
If both are unavailable, Im screwed :p

thanks to @matt-oneill I have found the error.  Date format change on the part of AlphaEss.  I'll get a fix built today.

Thanks @CharlesGillanders, what would us non-programmer types do without you gurus!?

Thanks @CharlesGillanders, what would us non-programmer types do without you gurus!?

Seconded!

I have updated the AlphaEss library to fix the date format issue and I've tried to code it to catch future similar changes.  Time will tell.....
If you are using the HACS method of installing the integration then you will need to go into HACS and update the integration (if HACS hasn't updated the version yet you can force it to be updated using the hamburger menu on the integration in HACS)
If you installed manually you'll need to download the updated integration code from github and restart home assistant,

I have installed the update via HACS (it had appeared) and everything is now working fine. Excellent work Charles and thank you very much for your quick attention to this.

Just me or are more people getting "unavailable" responses? Looks like it started around 09:00 UTC today

hmm, I just "updated" via HACS and now have a broken version because of the changed date. Shouldnt the latest update fix it in any case? I will try to completly delete and reinstall with HACS, maybe that fixes it. Just wanted to let you know that the update process isn't that smooth as hoped.

@Tupsi - updating to the latest version in HACS will fix the integration but I need to actually write the fix first.  I'll update the open GitHub issue when I have done so.

and now I remember that you do some stuff with that external lib stuff and that needs an update as well, so I will run and get that by hand installed in the hope it fixes it.

ok, can someone please clarify if it is currently possible to install a working version or not, because that is still confusing for me to read here, as DavidqStokes above wrote he reinstalled via HACS the latest version and it now works. I have tried that and are now stuck getting my credentials back in (because I removed it completly before the fresh install). If it needs some manuall poking I can do that as I have to issues with going into touch with the command line, just need some pointers where to do/replace what please.

@Tupsi - Charles is still working on the Fix, he has not yet released the working update.

@Tupsi - Charles is still working on the Fix, he has not yet released the working update.

thanks for the clarification, then I will wait patiently. Is that maybe another/new problem with the dates? Because I remember we already had that a few weeks ago (and fixed). Because as I said, it was running fine here until me stupid pressed update this morning :-)

I am not sure, when people started reporting the issue my instance was working fine until I updated to home assistant Core 2022.10.2, and then I got the same error as everyone else.

yeah, the v10 broke a few things here as well. now it is whining about me having portainer installed and refuses to update. Silly.

I've just pushed the latest fix which should hopefully prevent future date related API issues....

Thanks again for the quick response @CharlesGillanders, you're a star!

Thank you!


Question : system_stats > cpu value ?

Hello, I just wanna know what values report "cpu" in system_stats
{
  "callid": "5b3b511655589",
  "action": "system_stats",
  "message": "ok",
  "data": {
    "stats": {
      "cpu": "20770421|20494981",

Thanks for your wonderful API

Good question, this comes from the pfSense cpu_usage() function itself that you can see here:-
https://github.com/pfsense/pfsense/blob/master/src/usr/local/www/includes/functions.inc.php#L84
The comment there states:-
"""
Returns the current total ticks and user ticks. The dashboard widget calculates the load from that
"""
Reading the pfSense code is seems that the first value is the sum of the CPU usage and the second value is the CPU idle time


RangeError: Maximum call stack size exceeded at recursiveTypeRelatedTo after upgrading to 3.0

import { Model, RelationMappings, RelationMappingsThunk } from 'objection';

export class BaseModel extends Model {
  id: number;

  static get modelPaths() {
    return [__dirname];
  }
}

export class Product extends BaseModel {
  static tableName = 'product';
  price: number;
  user?: User;

  static relationMappings: RelationMappings | RelationMappingsThunk = {
    productList: {
      relation: Model.BelongsToOneRelation,
      modelClass: 'User',
      join: {
        from: 'product.userId',
        to: 'user.id',
      },
    },
  };
}

export class User extends BaseModel {
  static tableName = 'user';
  name: string;
  // productList?: Product;

  static relationMappings: RelationMappings | RelationMappingsThunk = {
    productList: {
      relation: Model.HasManyRelation,
      modelClass: 'Product',
      join: {
        from: 'user.id',
        to: 'product.userId',
      },
    },
  };
}

If I uncomment the line "productList?: Product", I got the following error:
RangeError: Maximum call stack size exceeded
at recursiveTypeRelatedTo (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:63793:44)
at isRelatedTo (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:63388:30)
at typeRelatedToSomeType (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:63654:35)
at structuredTypeRelatedToWorker (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:63926:32)
at structuredTypeRelatedTo (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:63908:30)
at isRelatedTo (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:63385:30)
at checkTypeRelatedTo (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:63003:26)
at isTypeRelatedTo (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:62954:24)
at isTypeAssignableTo (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:62160:20)
at getConditionalType (/Applications/MAMP/htdocs/test-obj3/node_modules/typescript/lib/typescript.js:60943:133)
This only happens after upgrading to<EMAIL_ADDRESS>This is fine in 2.x.
Already set tsconfig to
"strict": true,
Seems related to
https://github.com/microsoft/TypeScript/issues/33460

setting "strictNullChecks": false can fix temporarily

Also using strict: true fixes this.

Already set tsconfig to
"strict": true,

Are you sure? Your example won't compile in strict mode. id: number; or name: string; won't work in strict mode.

Here is a test project to reproduce the issue.
test.zip
npm test fails with "RangeError: Maximum call stack size exceeded"
Adding "strictNullChecks": true to tsconfig.json fixes the issue for me (the test still fails but that's because I removed the non-relevant code).

Running npm i objection@2 <EMAIL_ADDRESS>and updating the imports fixes the issue.

Replacing QueryBuilder<ImageModel> with QueryBuilder<Model> also fixes the issue :-)

I don't have time to debug or fix this for you. You'll have to do it yourself. I'll merge the PR.

@hkjeffchan
I copied your example, with "strict: true" I am not getting any error and inference works fine.

I find the "strict: true" thing almost to be the opposite.
I've copied the example and at<EMAIL_ADDRESS>"strict: true": No error, inference works but performance is very slow; it takes ~5 seconds for any kind of inference to pop up. "strict: false": No error, inference works and everything is very fast.

Already set tsconfig to
"strict": true,

Are you sure? Your example won't compile in strict mode. id: number; and name: string; won't work in strict mode.

sorry, forgot to mention. The following is set:
"strictPropertyInitialization": false

Another found:
Downgraded typescript to 4.3 works like a charm even with "strictNullChecks": true

"typescript": "<4.5.3" works.

Linking my fix without downgrading TS: https://github.com/Vincit/objection.js/issues/2132#issuecomment-1001968056

Thank you @Linksku! Awesome work!

@Linksku may have found the fix here. Their fix is implemented in 3.0.1. Please try it out and report if it fixes all your issues.

@Linksku may have found the fix here. Their fix is implemented in 3.0.1. Please try it out and report if it fixes your issues.

Would you push 3.0.1 to npm package? Thanks.

3.0.1 + TS 4.5.4: Still RangeError: Maximum call stack
3.0.0 + TS 4.5.2: No error

do we still need to add the "strict": true for objection 3.0.1 ? when I set "strict": false, I am still getting the RangeError

Thanks @hkjeffchan. So the change didn't seem to be enough. I don't know why, but I'm unable to reproduce any of these typescript bugs. Could somebody please create a standalone reproduction for these?

Objection: 3.0.1
Typescript: 4.5.4
This generates the errors
import { Model, QueryBuilder } from "objection";

export class Base<T> extends Model {
    id?:T;
}

export class NumberModel extends Base<number>{}

interface IExample {
    scopeQuery<T extends NumberModel>(where: QueryBuilder<T>):void
}

export class Example implements IExample{

   public scopeQuery<T extends NumberModel>(where: QueryBuilder<T>){
       return null;
   }
}

Sometimes I can get it to spit this error out instead of crashing
Excessive stack depth comparing types 'UpsertGraphMethod<?>' and 'UpsertGraphMethod<?>'.

A note on strict, it crashes before strict gets enforced. i.e. remove the constructor so id is uninitialised, it still crashes

Doesn't generate the error when I run it.

Doesn't generate the error when I run it. By a reproduction I mean the whole project, tsconfig and everything.
@koskimas

https://github.com/hkjeffchan/obj3rangeError
change typescript to "<4.5.3" runs perfectly

In our case, the bug seems to be caused by a cyclical type definition. Here's the reproduction repo:
https://github.com/jmeyers91/objection-3-tsc-stack-error
import { Model } from "objection";

class SoftDeleteQueryBuilder<
  M extends SoftDeleteModel,
  R = M[]
> extends Model.QueryBuilder<M, R> {
  SingleQueryBuilderType!: SoftDeleteQueryBuilder<M, M>;
}

export default class SoftDeleteModel extends Model {
  QueryBuilderType!: SoftDeleteQueryBuilder<this, this[]>;
}

/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:94444
                throw e;
                ^

RangeError: Maximum call stack size exceeded
    at indexSignaturesRelatedTo (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:54056:34)
    at structuredTypeRelatedToWorker (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:53466:47)
    at structuredTypeRelatedTo (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:53084:30)
    at recursiveTypeRelatedTo (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:53055:53)
    at isRelatedTo (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:52648:34)
    at isPropertySymbolTypeRelated (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:53641:24)
    at propertyRelatedTo (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:53673:31)
    at propertiesRelatedTo (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:53861:43)
    at structuredTypeRelatedToWorker (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:53460:34)
    at structuredTypeRelatedTo (/Users/james/projects/objection-3-tsc-stack-error/node_modules/typescript/lib/tsc.js:53084:30)
    ```

Initial problem
Objection.js: v3.0.1
TS: v4.5.4
We're also having this issue. I can't provide a repo with the error, but I can talk through what I was seeing which will hopefully help debugging. But it's very strange as the repo has no TypeScript errors and none of the other issues people in this thread have referenced.
In the following code sample, uncommenting the doSomething function caused the Maximum call stack size exceeded error in ts-node. Nothing is calling that function. The name of the function didn't impact anything (neither did changing the return type and method body). yarn tsc --noEmit also succeeds in both cases (commented or uncommented). So only ts-node is failing when uncommented.
Commenting it out again made ts-node run without a problem. We have a very vanilla use of Objection.js so far.
export class AnswerModel extends BaseModel {
	static tableName = 'answers'

	// Default
	id?: string
	text?: string
	imageUrl?: string

	// doSomething() {
	// 	console.log('Test')
	// }
}

Attempted solution 1
Objection.js: v3.0.1
TS: v4.5.2
We downgraded the TS version as some had recommended. And it fixed the error in ts-node. We kept developing for another week and the error popped up again (again, unrelated to any type errors).
Attempted solution 2
Objection.js: v2.2.18 (latest 2.x version)
TS: v4.5.2
We downgraded Objection (and kept the downgraded TS) and the error seems to have gone away again. But we'd love to be on 3.0.x. I will update this comment if we see it again.

I can confirm this happens on latest versions of Objection and TypeScript when strictNullChecks is set to true. I definitely want to use strictNullChecks though so I help this will be fixed soon.

Here's another reproduction.  Tested with objection 3.0.1 and typescript 4.5.x
Run:
tsc --strictNullChecks index.ts

// index.ts

import { Model } from 'objection';

class ModelA extends Model {
  id!: number;
}

class ModelB extends Model {
  id!: number;
  static relationMappings = {
    model: {
      relation: Model.HasManyRelation,
      modelClass: ModelA,
      join: { to: "", from: "" },
    },
  };
}

Outputs:
RangeError: Maximum call stack size exceeded

I should note that splitting ModelA and ModelB into separate files with modelClass: require('./ModelA') like in the examples fixes the error.  It's not a circular reference though, so I don't think that should be necessary in this case.

Another way to reproduce it is to upgrade the examples/koa-ts repo to TypeScript 4.5.3 (oddly 4.5.2 works in this case) and add one of the following to tsconfig.json:
{
  "compilerOptions": {
    "skipLibCheck": true
  }
}

or
{
  "compilerOptions": {
    "strictFunctionTypes": false
  }
}

Compiling the project will reproduce the error:
> npx tsc

RangeError: Maximum call stack size exceeded


Tel v2 schema for app-started payload requires 'configuration'

What does this PR do?
Fixes app-started telemetry messages that don't include the "configuration" key. This key is required by the schema.
Motivation
Failing system tests: https://github.com/DataDog/system-tests/pull/2785
Additional Notes
System tests started failing once https://github.com/DataDog/dd-trace-php/pull/2757 was merged and telemetry messages were actually sent.

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

Project coverage is 70.45%. Comparing base (4805ac0) to head (e53f4da).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #543      +/-   ##
==========================================
- Coverage   70.50%   70.45%   -0.05%
==========================================
  Files         208      208
  Lines       28397    28414      +17
==========================================
  Hits        20020    20020
- Misses       8377     8394      +17

Components
Coverage Δ

crashtracker
25.53% <ø> (ø)

datadog-alloc
98.73% <ø> (ø)

data-pipeline
50.00% <ø> (ø)

data-pipeline-ffi
0.00% <ø> (ø)

ddcommon
86.43% <ø> (ø)

ddcommon-ffi
75.29% <ø> (ø)

ddtelemetry
58.95% <ø> (ø)

ipc
84.13% <ø> (ø)

profiling
78.09% <ø> (ø)

profiling-ffi
56.76% <ø> (ø)

serverless
0.00% <ø> (ø)

sidecar
35.42% <ø> (ø)

sidecar-ffi
0.00% <ø> (ø)

spawn-worker
54.98% <ø> (ø)

trace-mini-agent
71.14% <ø> (ø)

trace-normalization
98.24% <ø> (ø)

trace-obfuscation
95.73% <ø> (ø)

trace-protobuf
77.16% <ø> (ø)

trace-utils
91.03% <ø> (-0.60%)
:arrow_down:


Features/ethan ucs 79 beast milestone5

Reworked room.service to accommodate instructor search.
Worked on add-class component and service to communicate with the backend to attempt to add a class.

Have to give you the crown I guess!


How to export ABI to JSON file?

I was using the following script to export contract's ABI as JSON:
const contract = await ethers.getContractFactory("Test")
const token = await contract.deploy(
  "Test"
)

const data = {
  address: nft.address,
  abi: JSON.parse(nft.interface.format('json'))
};

fs.writeFileSync('output.json', JSON.stringify(data, null, 2));

I couldn't find much information whether ABI is in JSON format or arbitrary:
export interface Deployment {
  address: Address;
  abi: ABI;
  ...
}

// ABI

// eslint-disable-next-line @typescript-eslint/no-explicit-any
export type ABI = any[]; // TODO abi

How can I achieve the same method using hardhat-deploy plugin?

hardhat-deploy have builtin facility for that : https://github.com/wighawag/hardhat-deploy#6-hardhat-export

Thanks! So exporting a JSON file in the deploy.ts does not make sense since there is an export command?

you can still call the task "export" in your deploy but yes, feel like you do not need it at all


Weird Spacing in Kindle Edition

Thank you for taking the time to write this book. I am learning a lot from going through it.
Initially I had some problems with the code because I am not super familiar with swift. I was getting errors in my code when I thought I added the code just like in the book. The problem turns out to be some of the way the code is displayed in the kindle reader app on mac (I also noticed the spacing issue when using the kindle app on my iPhone).
For example...
Page 16 in the app reads like this...
//  MARK: -  Init
required   init   ?(coder      aDecoder:      NSCoder  )  {
super     .    init   (coder:      aDecoder)
}
In reality it should be more like...
// MARK: - Init
required init?(coder aDecoder: NSCoder) {
super.init(coder: aDecoder)
}
Since I am new to swift and the syntax and spacing, I was getting errors because I was putting a space between init and ?. There are other places where the spacing caused problems as well when updating the sceneLabel on page 24. I was able to quickly fix it and learn from the provided code for the chapter, but it would be nice if whatever you are using to publish the book would not try and format the code differently than it should be written. Maybe it doesn't do this on an actual Kindle but it does in the app.
Sorry if this seems nitpicky, you may not have any control over the output. It is just slightly frustrating for us newbies.
Again, thanks for taking the time to teach what you know and write this book. I think I will buy your other book as well.

Hi djaxutah,
I will double check the book and see if there is something I can do about the formatting that leaves that space in there. I agree, that is frustrating and annoying to deal with, especially if you aren't sure if the space should be there or not.
Not nitpicky at all. The code is pasted right out of Xcode as I type it. The intent is for it to look just like it does in Xcode using the default theme . I really appreciate you telling me about this, I wasn't aware that the Kindle App has this issue.

Jeremy

Jeremy,
Thanks for responding. It looks like github removed some of the extra spaces I put in my post. Just so you can see how bad the formatting is I took a screencap from my iphone and am attaching it here.

Hi djaxutah,
I am uploading a new version to see if that resolves the formatting issues. It looks pretty good in the Kindle Previewer, hopefully it will resolve the spacing and formatting issues you reported.
Thank you again for pointing this out.

I haven't seen a new version posted yet from Amazon. Their listing still shows 9/21/2016 as the latest version.
https://www.amazon.com/Epic-SpriteKit-Tutorial-Swift-Programming-ebook/dp/B01BB2U0PA/ref=sr_1_1?ie=UTF8&qid=1478890578&sr=8-1&keywords=epic+spritekit
Hopefully the kindle app will automatically update the book to the latest version.


Bump max version to 53

Just a version bump, @dkearns.

Works for me too with Firefox 53.0.3. Judging by the lack of activity it makes me worried that fewer and fewer people are interested in Pentadactyl development :(

Works for me too with Firefox 53.0.3. Judging by the lack of activity it makes me worried that fewer and fewer people are interested in Pentadactyl development :(

I seems that @dkearns is the one that usually merges the PRs, but he/she has not public activity in GitHub since 2017-03-08.

@rbasso: Yeah, I know that and it makes me very worried. I just can't imagine using Firefox with a mouse without pentadactyl. There is a discussion going on somewhere here about pentadactyl future but the general conclusion is that simply there will be no pentadactyl for future Firefox. One can consider switching to Palemoon or keep using old versions of Firefox but it's not a long-term solution.
It doesn't look good.


Create a separate repo of the mavenized BoneJ1 code

The mavenized BoneJ1 code should be uploaded to a different repo  for future safekeeping from where it can easily be cloned. The mavenization branch in this repository will eventually be merged into the master branch, and the development will continue. The mavenized BoneJ1 could be used as a basis for the automatic builds for different platforms mentioned in the forum.

This not how the platform specific versions would be built after all, but I still believe having the mavenized BoneJ1 code around would be helpful.

When creating this repo, make sure that you make in issue about not being able to solve the issue with enforcing Java 6 across the board (org.scijava java3d requires 1.7).

Create from commit https://github.com/bonej-org/BoneJ2/commit/fea4138fcbd9348fd1c179f71883b59e9ca5d746

No need for this anymore as there's no plans to release independent, mavenized BoneJ1 anymore.


Is it possible to count docs in an alias?

Is it possible to count docs in an alias?

You should be able to join the elasticsearch_indices_aliases metric to the elasticsearch_indices_docs_primary on the index label to understand the number of docs for an alias.


Bug? - broken gauges

Hi,
After a few minutes of render several gauges with Firefox, suddenly all the gauges become broken.

Information you provided is not enough to identify a problem...

Dear Mikhus, same occurs here.
I attach the webpage. (i just hacked the update function to provide dummy values)
Leonardo
broken_gauges.zip

Dear Mikhus, same occurs here.
I attach the webpage. (i just hacked the update function to provide dummy values)
Leonardo
broken_gauges.zip

hi Mikhus,
i have the same error in the visualisation from iobroker.

i am using iobroker on an laptop with win11. the error occured under firefox 103.0.2 (64 bit) after a while (minute or so) without other acticity on the laptop.
i have already done:  clear cache, private tab.
The visualization under windows edge is OK.
Thomas


Hardcode available_locales to english and spanish, for now

To prevent other language failing before we have translations for them.

@guitsaru I think it'll only redirect to a subdomain that is:

in this list AND
is set by your browser as your preferred language

@veganstraightedge right, isn't that the case for pt-BR that we're trying to fix?  I just want to make sure that doesn't keep happenning.

should be ok since browsers send actual locale code (e.g. pt-BR whereas TCE is in a word)

@astronaut-wannabe gotcha, I thought that might be the case, but wanted to make sure.


[Feature Request] highlight current prayer and add countdown timer to next prayer

It would be nice to have the current prayer time highlighted in the app similar to most prayer times apps.
Also, a countdown showing hh:mm:ss until next prayer would be helpful to see remaining time at a quick glance.

I added a simple countdown timer on the complication in my fork, but didn't make a pull request since it's more tailored to my needs/may not have proper logic.

I added a simple countdown timer on the complication in my fork, but didn't make a pull request since it's more tailored to my needs/may not have proper logic.

There is a PR from another contributor to have this as a separate complication, I will merge it as soon as I can find time to test it.

Please check the latest version and let me know if this is what you were referring to, my watch got bricked so I currently have no device but you can create a PR if your idea is different and I will try to merge the two features.

Please check the latest version and let me know if this is what you were referring to, my watch got bricked so I currently have no device but you can create a PR if your idea is different and I will try to merge the two features.

I'm sorry to hear about your watch. Was it the buggy firmware update for Galaxy Watch 4 that did that?
Assuming this is the top widget you are referring to, it seems halfway to what I was looking for by utilizing range widget to represent remaining time. The other half would be similar to what I have in my fork which shows the actual remaining time. However, my fork also has issues where the time doesn't update every minute (sometimes up to 15-30 min delay) so still some work to be done there. I'm not sure if I'll have time to get around to this anytime soon.

OK got it, this widget counts 'UP' to the next prayer and showing the time when current prayer time started. And you are looking for something which counts 'DOWN' to the start time of the next prayer and show hours and minutes remaining rather than an absolute time.
Yes unfortunately my watch was a victim to the firmware update. I had switched to the apple ecosystem so wasn't using or charging it normally anyway. And ironically in this country local Samsung does not honor international known issues or recalls, etc. I have acquired an old used watch to work on this soon and add another complication inspired from your code, hence made a PR but not merged.


Different Scrollbar margins

Hi,
the margin of a horizontal scrollbar is about 8px, where the margin of a vertical scrollbar is about 4px

also, the margin of the vertical scrollbar changes if the horizontal scrollbar is pulled completely to the right:

Repo with code to reproduce:
https://github.com/njust/table_grow_issue

There are two problems here:

Your application has nested ScrollAreas, which is almost never what you want.
egui_extras::Table does not support horizontal scrolling at all.

Adding horizontal scrolling support to egui_extras::Table is something that needs to be done here, but you will also have to remove the ScrollArea from your application.

Hi, thanks for your reply. You are right, the Table does not support horizontal scrolling out of the box. That's why I added the wrapping ScrollArea. #2369 mentions "Fix bugs when putting Table inside of a horizontal ScrollArea" so I guess it's somehow supported?

I'm not convinced that it should even be implied to be supported. 😆 Nested ScrollAreas have always behaved poorly, and I don't think there is a good reason to ever do this. Even if the behavior is somehow "fixed", the user experience will still be quite bad.


Use ganache for testing

Issue link
https://github.com/tasitlabs/TasitSDK/issues/38
Auto-close the issue?
Closes #38
Types of changes

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected)
[ ] Docs (modified documentation or code comments)
[ ] Other (please describe below if you select "Other")

Observations

tasit-act remains empty (Is it to be done in this PR or another?);
ethers.test.js still complete at tasit-account project (Its splitting is to be done in this PR or another?);
I'm not sure if the way that I've written scripts chain is the better one, what do you think @pcowgill?
I've needed downgrade ganache-cli from 6.2.3 to 6.1.8 (https://github.com/ethers-io/ethers.js/issues/275#issuecomment-440427518);
Contract event testcase remains slow (even using ganache), Will be necessary tryingout another way (e.g.: https://kalis.me/check-events-solidity-smart-contract-test-truffle/)

I'm thinking here, maybe will be better if ganache lives at root repo because not only tasit-contracts do need it.

tasit-act emptiness
I think this can come in a separate PR for this issue https://github.com/tasitlabs/TasitSDK/issues/40

ethers.test.js splitting
I'm a fan of small PRs, so let's create an issue for splitting these tests.
I've done that here https://github.com/tasitlabs/TasitSDK/issues/63

scripts chaining
I'll comment on the scripts chain approach while reviewing the PR itself

Downgrading ganache
Sounds good. Good to know. Thanks.

Contract event testcase
Which part is the slow part right now?
Rosco's approach from that blog post sounds good.
Here is another useful utility related to this topic https://github.com/OpenZeppelin/openzeppelin-solidity/blob/master/test/helpers/expectEvent.js

Ganache in root repo.
Sounds good. We can use lerna hoisting for this https://github.com/lerna/lerna/blob/master/doc/hoist.md
https://github.com/lerna/lerna/blob/master/doc/hoist.md

Event listening is slow, we are currently using the same function that ethers.js uses in their tests, I'll check out those alternatives. Thanks!
I've forgotten to comment about ENS tests, they use the mainnet (I don't found how to test they using ganache) and maybe should move into an integration test suite.

If it's simpler we can skip the ENS tests for now.
Down the road if they are inherently long-running, moving them to an integration test suite would make sense. But I'm not sure from first principles why those tests would need to be especially long-running.

FYI I'm planning to review this PR first thing tomorrow CST!

@marcelomorgado The way you wrote the scripts chain makes sense to me. I added a clarifying question and some thoughts on that in my inline comments. I also think the diagram I suggested making in a testing section of the README will help us reason about other ways we might chain that. But I think it's great as it is!

I'm working on solving that test case with an event that is slow.

I've figured out what was happening, ethers.js default pooling interval is 4000ms.
https://docs.ethers.io/ethers.js/html/api-providers.html#properties

I'm working on solving that test case with an event that is slow.

I've figured out what was happening, ethers.js default pooling interval is 4000ms.
https://docs.ethers.io/ethers.js/html/api-providers.html#properties

Ah that explains it! Nice find.


Atom IO build not working or Bootstrap/script

script/bootstrap

Node:   v8.11.2
Npm:    v5.6.0
Python: v2.7.10
Installing script dependencies
npm ERR! Unexpected end of JSON input while parsing near '..."_hasShrinkwrap":fals'

npm ERR!      C:\Users\Jono\AppData\Roaming\npm-cache_logs\2018-07-16T08_00_29_037Z-debug.log
child_process.js:624
throw err;
^

Error: Command failed: npm.cmd --loglevel=error install
npm ERR! Unexpected end of JSON input while parsing near '..."_hasShrinkwrap":fals'

npm ERR!      C:\Users\Jono\AppData\Roaming\npm-cache_logs\2018-07-16T08_00_29_037Z-debug.log

at checkExecSyncError (child_process.js:601:13)
at Object.execFileSync (child_process.js:621:13)
at module.exports (C:\eleclearn\atom\atom\script\lib\install-script-dependencies.js:9:16)
at Object. (C:\eleclearn\atom\atom\script\bootstrap:28:1)
at Module._compile (module.js:652:30)
at Object.Module._extensions..js (module.js:663:10)
at Module.load (module.js:565:32)
at tryModuleLoad (module.js:505:12)
at Function.Module._load (module.js:497:3)
at Function.Module.runMain (module.js:693:10)

2018-07-16T08_00_29_037Z-debug.txt

Thank you for contributing!
We require the issue template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests. Please open a new issue and fill out the template for us.
Thanks for understanding and meeting us half way :grinning:


[Sonarr] match with CF

Pull request
Purpose
Match with CF
Approach
If this Pull Request is created to solve a issue explain how does this change address the problem?
Open Questions and Pre-Merge TODOs
Check all boxes as they are completed

[ ] Use github checklists. When solved, check the box and explain the answer.

Learning
If you're adding a new Custom Format make sure you follow the Radarr/Sonarr Custom Format (JSON) Guidelines.
Requirements
Check all boxes as they are completed

[x ] These changes meet the standards for contributing.
[ x] I have read the code of conduct.

i think we should use [ ] and not use space on L10+L25+L32

Basically just copied over: https://github.com/TRaSH-/Guides/blob/7aae5ecd12a30ffd3e10561d6ef4736546707baf/docs/json/sonarr/cf/atvp.json#L13

Need to add the regex also to the guide,
the Release Profile regex isn't automatically updated when changing the JSON.
Where with the Custom Formats it does.
Will try to do it this weekend.

Need to add the regex also to the guide, the Release Profile regex isn't automatically updated when changing the JSON. Where with the Custom Formats it does. Will try to do it this weekend.

Updated the PR with the needed changes to the Sonarr guide.


Proxy server not able to authenticate using cookies

I am using a proxy to redirect calls during development to our application server.
I need to login using a login form submit and a returned cookie to be sent on every next request to the server.
I had to adjust the serve.js file and add
opts.rejectUnauthorized = !(proxy.rejectUnauthorized === false); // create setting in ionic.project
opts.cookieRewrite = proxy.proxyUrl; // my application server needing authentication

for every proxy specified otherwise the cookie is not set in the browser and thus not redirected thru the proxy. The ionic.project configuration:
"proxies": [
    {
      "path": "/api",
      "proxyUrl": "https://app.ontwikkeling.webbergen.local/api",
      "rejectUnauthorized": false
    }
]

@RonB this fixes it for you entirely? I'll patch it in, mind testing a beta version?

You also should enable cookieRewrite on the http-proxy options to make sure that authentication can be enabled. Next to that an option to configure the local hostname is something we would appreciate. It can be achieved by using the --address param but specifying this in the ionic.project seems like a good idea.
I will test the beta if you want.
Thanks.

@jbavari Can you shine a light on this issue as I cannot find the previous code anymore where you create the proxies from the ionic.project config?

Same issue here? What exactly is the fix? When will this be merged in?

I can't find the code referenced in the posts above, is there a way to implement this at the moment?

@josephmeiring
I am puzzled just like you. The place where the proxy server was created (used to be in serve.js) is no longer there  in the sourcecode.
This is the code I' ve adjusted to get it working but now itseems that this code has disappeared in the entire project:
Serve.setupProxy = function setupProxy(options, app) {
if(options.useProxy) {
for (var x=0; x < options.proxies.length; x++) {
var proxy = options.proxies[x];
var opts = url.parse(proxy.proxyUrl);
if (proxy.proxyNoAgent)
opts.agent = false;
opts.rejectUnauthorized = !(proxy.rejectUnauthorized === false);
// adjustment InProces to rewrite cookies
console.log('InProces Adjustment authentication enabled for proxy ' + proxy.proxyUrl);
opts.cookieRewrite = proxy.proxyUrl;
// end adjustment
app.use(proxy.path, proxyMiddleware(opts));
events.emit('log', 'Proxy added:'.green.bold, proxy.path + " => " + url.format(proxy.proxyUrl));
}
}
};

@RonB they moved a lot of stuff to ionic-app-lib
https://github.com/driftyco/ionic-app-lib/blob/b24b47c8a95a0dd7881b6cfc7a91492d5e5d0070/lib/serve.js#L341

@blackjid
Thanks for pointing me toward that library.
@jbavari
Could you add the cookieRewrite property to the proxy options?
I've tested this with my local application and it works like a charm. Thank you.

I have this same issue.  The solution to use the following also fixed my issue.  Could this be added permanently?
opts.cookieRewrite = proxy.proxyUrl; // my application server needing authentication

Thanks for opening the issue. It seems that issue has been resolved. I am going to close this issue but please feel free to reopen another issue if this occurs again.


Calendar using VanillaJs

Hi there, myself Soham Das, wants to add a new simple project, called “Calendar”, using HTML, CSS and VanillaJs.
Description:
Calendar is a system of organizing days. This is done giving names to period of times, days, weeks, months and years. As we all know how days are valuable for us, so if we add calendar, we will navigate our progress of each day and also good for setting short- or long-term goals.
Therefore, I would like to add a simple project “Calendar”
Would be delighted if you consider my submission for this        issue @Vishal-raj-1 @urvashi-code1255 @991rajat.
Dear project maintainers, please assigned me to this issue, so that I can work on it, as early as possible.
Thank You

@Soham2020 Calender is already present in the project. If you want to improve it then I can assign it to you.

I'm closing this. If you want to improve upon this project create a new issue for improvement.


Upgrading causes :tools biblio to raise a void-variable citar-bibliography error at startup.

What did you expect to happen?
Enable :tools biblio
What actually happened?
Upgrading (with https://github.com/hlissner/doom-emacs/commit/5656a96a1bca6afb1842e08dcb58f91289150404) causes :tools biblio to raise a void-variable citar-bibliography error at startup.
Warning (initialization): An error occurred while loading ‘/home/abdelhak/.emacs.d/init.el’:

Error in a Doom module: modules/tools/biblio/config.el, (void-variable citar-bibliography)

To ensure normal operation, you should investigate and remove the
cause of the error in your initialization file.  Start Emacs with
the ‘--debug-init’ option to view a complete error backtrace.

Describe your attempts to resolve the issue
No response
Steps to reproduce

Upgrade to https://github.com/hlissner/doom-emacs/commit/5656a96a1bca6afb1842e08dcb58f91289150404
Enable :tools biblio
Run doom sync
Restart Emacs

System Information
https://pastebin.com/x4cYAsmX

Can confirm I ran into this as well. Uncommented the module until this is resolved.

This issue should now be resolved. Let me know if that isn't the case and I'll reopen it. Thanks for bringing it to my attention!


improving the code to not generate a triangle when detecting an object

Hi @KePeng1019,
This pull request that I am sending is to improve detection and also to prevent a triangle from being generated when detecting an object.
I know that the triangle generation fix was closed in issue 12, but what it seemed when running the app is that this problem still remained.
Thanks for the shared content!

great job.


Optimized how the ElementBatcher calculates renderbounds for larger lists

This PR contains:

[x] Changelog
[-] Documentation
[-] Tests

wrong origin


ESP32-2432S024 not work display

Will there be support for 2.4 inch screens or how can I make the screen work?

I'll have mine ready soon but this fork should get you there if you can't wait
https://github.com/lsdlsd88/ESP32-Marauder-Cheap-Yellow-Display

Fr4nkFletcher thanks for the tip. But no matter how much time I spend I can't get the project to compile trivially because of various errors. I realized that I need to change parameters in User_Setup.h from TFT_eSPI library. But at the moment of compilation everything is very tight. Please make different variations of screens in your CYD-Flasher. Thanks before.


Food test

Hello giancarlo!
The milestone in this pull request is add testing for food model, recipe model and integration test for food index and food new
Thanks!

Approved!
Nice job!
Nothing else to say


Adjust colours and hierarchy in the table component

A suggestion have been made. A review will be taken when we color all the new components in 2.0. The new tasks will be added and we will take a review on them then.


FFCV accrues massive overhead with heavy transform pipelines

Hi,
I've noticed that FFCV can deliver nice speedups for individual steps. For context:

I have lots of images of shape 256 x 256 x 3
I use many augmentations, some even repeated as I'm using an SSL method which relies on this for performance
My baseline __getitem__ call:

read a JPEG image from a NFS
decode JPEG
Perform augmentations

Noticing with FFCV:

Dataloading on its own is 3-4x faster (reading from disk + JPEG decoding)
Transforms on their own are 3-4x faster after optimization on my end (due to using numba or other tricks)
E2E dataloading with FFCV is 2-3x slower than my baseline.

I profiled it quite a bit and it seems to be due to FFCV overhead. I can see in htop that even when using something like 16 dataloader workers, most cpu cores are actually idle

Has anybody else experienced something similar, or would have any tips on debugging/addressing?
Thank you!

Thanks for opening this issue. Can you define what you mean by "lots of images" or "heavy transform" pipelines ? Is it 1B images and 20 transforms or just 1M images with 4 transforms ? In addition, it would be easier if you could share a snippet of your code to see if this is reproducible on our side.


Custom dataset in projecting lidar points face trouble!Any suggestions

Hello,I try to custom some dataset,But When i try to project and calculate the points in different instances.It always ruturn a dict()
Are there any suggestions?
i use the debug option in driving_dataset.py and output some ply,but it looke strange.And it was delete In the function  delete_invisible_pts()
the output ply pic is here

But i think Intrinsics is correct ,Because the program can train ,but it train like normal 3dgs because the rigid-gs,smpl-gs are removed,but it's strange that the deformable gs is still work


Getting "Unexpected error" after a username change

I am following a user that have change his name from nayaser88 to j_blow.
I am getting an unexpected error when launching TL as it is the default search on this user.
It took me a while to understand the issue, it will be nice if I go a better error message like "The user nayaser88 doesn't exist,...".
Thank you for this very useful application.
I wish you the best for the new year.
Here is the full log file:
System.AggregateException: Une ou plusieurs erreurs se sont produites. ---> System.Net.WebException: Le serveur distant a retourné une erreur : (404) Introuvable.    à System.Net.WebClient.DownloadDataInternal(Uri address, WebRequest& request)    à System.Net.WebClient.DownloadString(Uri address)    à TwitchLeecher.Services.Services.TwitchService.SearchChannel(String channel, VideoType videoType, LoadLimitType loadLimit, DateTime loadFrom, DateTime loadTo, Int32 loadLastVods)    à TwitchLeecher.Services.Services.TwitchService.Search(SearchParameters searchParams)    à System.Threading.Tasks.Task.Execute()    --- Fin de la trace de la pile d'exception interne --- ---> (Exception interne #0) System.Net.WebException: Le serveur distant a retourné une erreur : (404) Introuvable.    à System.Net.WebClient.DownloadDataInternal(Uri address, WebRequest& request)    à System.Net.WebClient.DownloadString(Uri address)    à TwitchLeecher.Services.Services.TwitchService.SearchChannel(String channel, VideoType videoType, LoadLimitType loadLimit, DateTime loadFrom, DateTime loadTo, Int32 loadLastVods)    à TwitchLeecher.Services.Services.TwitchService.Search(SearchParameters searchParams)    à System.Threading.Tasks.Task.Execute()<---

This is actually a bug. Some time ago the "channel name" ComboBox was a Textbox (without the dropdown option).
If there is an error, the textbox should get a red background (not only a red border) and a tooltip with the error message.
I'll fix that in a future update.

This is actually a bug. Some time ago the "channel name" ComboBox was a Textbox (without the dropdown option).
If there is an error, the textbox should get a red background (not only a red border) and a tooltip with the error message.
I'll fix that in a future update.

Ok, thank you

Ok, thank you


Adding biologist Christiane Nüsslein Volhard and AI pioneer Marvin Mi…

…nsky
Signed-off-by: Anuj Bahuguna<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:anujbahuguna/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Man, Minsky still makes me sad. Sign off on the commit and you'll be good to go.

@anujbahuguna also commented on your old PR, but I think the sign-off is not recognized because it's missing < and > around your e-mail address.

@thaJeztah thanks for the help. (had moved to different development machine where config user.name and user.email was not setup up). Fixed it now

Thanks @anujbahuguna!
LGTM

LGTM 🐼

Janky's are having issues after the GitHub outage, so moved to "merge"

Works on my machine™.


Can't run / eject from expo

Hi
I found this project on reddit and would like to use it on a personal project for study purpose, but can't get it running on expo. Also, I would like to eject from expo to work with it as pure react native project. Tried to first update expo to properly eject with expo eject but got some errors. Could you help me get it running on ubuntu 18.04? Better if you can eject it from expo.
Sry for english mistakes if you find. Non-native here.
You did an awesome job! Thanks in advance.

@betodasilva ejection from expo is not an issue with this project

@betodasilva not that it's related to this package, because it's not, but does this help?


Tabs: changes to mouseover / tooltips

Taxonomy:
The "Taxonomy" tab provides taxonomy subtree for the current taxon level.
Phylogeny:
The "Phylogeny" tab provides order or genus level phylogenetic tree, constructed using core protein families.
Genomes:
The "Genomes" tab contains a list of all genomes associated with the current view and their metadata"
Sequence Tab:
The "Sequences" tab contains a list of genomic sequences (e.g. chromosomes, plasmids, contigs) for genomes associated with the current view
Specialty Genes:
The "Specialty Genes" tab contains a list of specialty genes (e.g., antimicrobial resistance genes, virulence factors, drug targets, and human homologs) for genomes associated with the current view
Browser:
The "Browser" tab shows genome sequence and genomic features using linear genome browser
Circular Viewer:
The "Circular Viewer" tab provides circular overview of the entire genome and its genomic features
Transcriptmics: [gene level]
The "Transcriptomics" tab shows gene expression data available for the current gene
Correlated Genes:
The "Correlated Genes" tab shows list of genes from the same genome with correlated expression profiles

Fixed.


Collection data access APIs --> core

The collections can now (optionally) expose an api.js file in their src dir and it will be required in by core and add it's endpoints (GET only right now). This is intended to enable collection processes to turn themselves off more often (only need to run for data processing) and (hopefully) enabled better process priority control (nice down processing, while leaving core at normal priority).
This also add Common/node/collectionDataStore, which abstracts several common data store patterns in collections. Next step is to make some of the API methods common, but I'd rather merge (and deploy) this first.
This passes all tests, but they're aren't really a ton of collection tests. I'd be happy to write more, but if we are moving away from vows it feels like a waste (tell me I'm wrong!). I did auth all connectors to ensure data processing works properly and hit all API endpoints to make sure they work.

I need at least one person to test this:

on master, clean me dir, connect all services
once all data is in, hit /state on all collections, record the result (mostly count)
repeat process from clean me dir on this branch

again, check /state, compare results -- should be the same as master!
check the logs closely for for any errors.
hit all collection API endpoints, listed below.

###Contacts
/Me/contacts
/Me/contacts/
/Me/contacts?limit=10&offset=25
/Me/contacts/state
/Me/contacts/since?id=
/Me/contacts/id/:someid
/Me/contacts/update (and check logs again for errors!)
###Photos
/Me/photos
/Me/photos/
/Me/photos?limit=10&offset=25
/Me/photos/state
/Me/photos/since?id=
/Me/photos/id/:someid
/Me/photo/update (and check logs again for errors!)
/Me/photos/image/:someid
/Me/photos/thumbnail/:someid
###Places
/Me/places
/Me/places/
/Me/places?limit=10&offset=25
/Me/places/state
/Me/places/since?id=
/Me/places/id/:someid
/Me/places/update (and check logs again for errors!)
/Me/places/from/:network/:from (eg /from/foursquare/someones4sqIdNum)
/Me/places/from/geo/:network (eg /geo/foursquare) (and check logs again for errors!)
###Links
/Me/links
/Me/links/
/Me/links?limit=10&offset=25
/Me/links/state
/Me/links/since?id=
/Me/links/id/:someid
/Me/links/encounters/:id
/Me/links/update (and check logs again for errors!)
/Me/links/search?q=singly
/Me/links/embed?url=http://www.youtube.com/watch?v=1bJxYRtGUM4

Dude, kudos, this was quite the undertaking, moving a lot of logic around, thanks! It feels like a solid step towards some generalized collection functionality too, a pattern that could start to be applied to the data processing as well (I've had some ideas here, making this look more like synclets).
One caveat is this will increase the ram in the core process, and any bugs will bring the whole ship down, but the work @temas is doing with synclets using vm contexts as isolation might work perfectly here too.
Now to do some testing :)

I ran this with every app, and I also tested the memory impact on my mac between the two for lockerd node process. By including the apis in the main process it only bumped up RSS by 2MB, not bad.


support dynamic cdn prefix

What version of Remix are you using?
1.14.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
Unable to pass dynamic injection publicPath.
Expected Behavior
My resource path address is returned dynamically, so I hope can dynamically inject paths on static resources,just Like the webpack_public_path in the webpack.
I don't know how many people want to do this.Of course, I can submit a pr for Remix if necessary.
Actual Behavior
Dynamic prefixes are not supported

I'm not sure if anyone is concerned about this problem, or that there are other official solutions.

#6075

Hi @19Qingfeng, thanks for bringing this up. This is definitely something we'd like to support eventually, though it isn't a priority for us right now. Leaving this open for some future point at which we will tackle this problem.

Hi @19Qingfeng, thanks for bringing this up. This is definitely something we'd like to support eventually, though it isn't a priority for us right now. Leaving this open for some future point at which we will tackle this problem.

Okay, thank you very much. Maybe in my spare time recently, I can come up with some ideas to solve this problem and contribute to my PR.

+1 on this.
We deploy our Remix server to AWS ECS, but we specifically deploy the built frontend assets to S3 + Cloudfront for better performance in serving the frontend assets.
We want to deploy each version of our frontend assets to a different, dynamically and uniquely prefixed (eg, git hash) subfolder on S3. This allows us to quickly and easily roll-back the actual server, and still have old frontend assets be reachable. Otherwise, we have to redeploy the frontend assets and server rather than just rolling back.
Are there any workarounds available to dynamically modify publicPath?

+1 on this.
We deploy our Remix server to AWS ECS, but we specifically deploy the built frontend assets to S3 + Cloudfront for better performance in serving the frontend assets.
We want to deploy each version of our frontend assets to a different, dynamically and uniquely prefixed (eg, git hash) subfolder on S3. This allows us to quickly and easily roll-back the actual server, and still have old frontend assets be reachable. Otherwise, we have to redeploy the frontend assets and server rather than just rolling back.
Are there any workarounds available to dynamically modify publicPath?

It seems that remix cannot implement this function at present, but I have modified the  component locally and the  component supports the splicing of host link , which can be satisfied for the time being, but that is why I cannot upgrade remix version.

Any news on this ? I think we can provide cdnPrefix to Script ?

You can do this by customizing the publicPath option in `remix.config file https://sergiodxa.com/tutorials/use-a-cdn-for-your-static-assets-in-remix, I imagine a similar thing can be achieved using Vite with some specific Vite option.

@sergiodxa That is helpful if your CDN path is predetermined (known at build time). But for dynamically hosted applications, it would be great to be able to set this at runtime. Hoping something like this can be implemented in the future.


moto_server S3 HTTP PUT/GET

using moto 1.3.16
Not sure if moto actually supports this and I'm just being daft, or if this isn't supported to begin with.
I've tried searching through documentation and other issues/PRs with no luck.
I am running a standalone moto_server s3 as I need to proxy NGINX HTTP GET/PUT requests to it, but for the purposes of getting-it-working I have cut out the NGINX middleman for now.
I also have some pytests which use @mock_s3 for creating a test bucket.
I have a couple of issues:

My pytest tests seem to work, successfully creating a bucket and PUTing a file in the correct bucket, however with no contents according to http://localhost:5000/moto-api, failing the last stage of one of my tests:

    data_input = b'some data 123 321'
    req = urllib.request.Request(url='localhost:5000/mybucket/test.txt', data=data_input, method='PUT')

    with urllib.request.urlopen(req) as f:
        assert f.status == 200  # PASSED

    req = urllib.request.Request(url='localhost:5000/mybucket/test.txt', method='GET')
    with urllib.request.urlopen(req) as f:
        assert f.status == 200   # PASSED

        data_response = f.read()
        assert data_response == data_input  # <- FAILURE: b'' != b'some data 123 321'

If I manually PUT a file using curl I again get no contents when using curl -X PUT localhost:5000/mybucket/test.txt -d 'test123', however when using curl -X PUT localhost:5000/mybucket/test.txt --upload-file ./test.txt it works, creating a file with actual contents for a change. The problem is if I try to access the file via HTTP GET (either through curl or my browser) I get a 403 error. As part of the bucket creation (using @mock_s3) I have tried attaching a bucket policy allowing read-only permission for anonymous users, however this has not made any difference.

TL;DR: I want to be able to PUT and GET files (and their contents) unauthenticated with REST/without boto3.
Just to clarify, all my tests work when proxying NGINX to an actual live S3 bucket (with v4 authentication).
Apologies if I haven't been too clear, I'm writing this after a day of tinkering not having achieved much.
On a completely unrelated note, I thought I'd use this opportunity to ask if moto_server has a daemon/background mode.
Update
Turns out adding req.add_header("Content-Type", "text/plain") to the PUT request makes it all work.
In which case, this issue could now be a question about whether or not moto_server should accept/store data without a content-type. S3 REST allows it therefore I believe (granted, as a new user of moto in general) moto_server should accommodate this.

Hi @notmeta, thanks for raising this, and providing a workaround! You're right, if it works in S3 it should work in moto as well.
Going to mark this as a bug.
Validating/parseing a bucket policy is a feature request, but hasn't been implemented yet: https://github.com/spulec/moto/issues/2892
moto_server does not have a daemon mode build in - I typically just use moto_server &
Some notes for whoever picks this up:
This if-statement needs an additional elif:
https://github.com/spulec/moto/blob/master/moto/s3/responses.py#L1101
The data comes in as a form, so we'd need to parse request.form as well.
(Similar to how this is done in a POST request: https://github.com/spulec/moto/blob/master/moto/s3/responses.py#L837)

Thanks for the response, glad I found something valid!

moto_server does not have a daemon mode build in - I typically just use moto_server &

Ah okay - that's what I was going to do anyway in the event that such a thing didn't exist


Idea: resign_responders method in screen module

I often find myself wanting to resign the active keyboard responder and thought it would be nice to have a quick method in screenmodule that allows me to do this without having to remember the weird line of code:
self.view.endEditing(true)

[self.view endEditing:YES];

Thoughts on adding this to screenmodule?
def resign_responders
  self.view.endEditing(true)
end

:+1: but in the PM::Support module and with an alias of hide_keyboard. :)

If it's in PM::Support it'll need to be a little smarter about what view is active, though.

I like it!

+1

On second thought, this seems like an RMQ thing. What do you think. @twerth ?

what would the API be?
rmq.app.hide_keyboard?   RedPotion app.hide_keyboard
my $0.02
not a huge fan of resign_responders over end_editing
Would be an easy add to RMQ.

Yes, rmq.app.hide_keyboard.

:+1: https://github.com/infinitered/rmq/issues/233

https://github.com/infinitered/rmq/issues/233


"culture metadata" not mentioned in the tables (as opposed to Fig 4)

Hey,
The fields microbial ID, microbial isolate, culture substrate, are mentioned under "site metadata" in all the biological/environmental tables (e.g. marine ).
But in the overview table, we have them listed under the term "culture metadata, if applicable" (as shown in the overview table Fig 4 here: https://github.com/NFDI4Microbiota/MetadataStandards/blob/main/images/BioEnvMetadata9Aug2023.jpg)
Should this discrepancy be clarified in the explanations or adapted in the tables? Or would you keep it as is?

Maybe I'll just add a separate sub-division of the tables, to be listed as "Culture metadata", as I don't see that it perfectly fits in either "site metadata" of "host-metadata"


including tests using manifest so they are distributed via sdist

this pr replaces #306, test code after this pr will be distributed via the source tarball. This won't require a version bump

Coverage decreased (-0.007%) to 99.782% when pulling 9d672a96714d006acbbeda29d12ae355602fbbc8 on pr_include_tests_manifest_source into 01e5a4245fa03fa6237bf6010624b91f056731c8 on master.

Coverage decreased (-0.007%) to 99.782% when pulling 9d672a96714d006acbbeda29d12ae355602fbbc8 on pr_include_tests_manifest_source into 01e5a4245fa03fa6237bf6010624b91f056731c8 on master.


Is there any way to disable Auto-updating when installing a package?

As we all know, China users face the problem of G.F.W. It's really slow for auto updating, I wonder is there is a way to disable this feature.
Thanks for your great work.

Yes, you can set HOMEBREW_NO_AUTO_UPDATE in your environment.

May be this should be documented in somewhere, I really did some googling before I open this issue.

It is: man brew.


Write a query for following.

This issue is a duplicate because at first, I think I need to add GET following and GET followers separately.


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2024-02-09T17:25:30+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test unit-tests-arm64

lint_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test unit-tests-arm64

lint_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test lint

integ-ambient_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test unit-tests-arm64

integ-ambient_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test integ-ambient

lint_istio_release-1.19
b1dcca5364bb9a75e7a9b254f589447307946b35
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Superseded by https://github.com/istio/istio/pull/49327 which contains the same make update-common but adds a manual cherry-pick of #49137 to fix the lint error.

/retest


Not able to call EndOfDayAsync();

Hey there,
first of all, thank you very much for offering this open source.
Everything is working fine except EndOfDayAsync() mehtod.
When i'm calling it, i receive following LOG output.
Can you maybe tell me what i'm doing wrong?
Thank you very much!

Which payment service provider do they use? Can they provide the source codes of their call? The EndOfDay works for us.
Can you also provide the complete communication log?
Do they use the Library or the ControlPanel?

Hey, thx for your quick replay.
We tried to use your library via serial port to connect to our terminal.
The ControlPanel only supports lan connections.
The terminal i tried to use is following:
https://www.cardcomplete.com/fuer-unternehmen/bezahlloesungen/terminaluebersicht/unattended-nfc-all-one/
every other command is working perfectly and as expected, also the payment with receipt as return is throwing no errors at all.
Only the EndOfDay part is throwing an unknown error as you see above.
I saw no other errors than this, i can try to look for further responses or errors tomorrow morning :)

This terminal also support ethernet i have the same here

i know :) we first tried to test a other terminal but this was blocked by the provider i later recognised ^^
tomorrow we will check this terminal via ethernet if there's any difference :) with the tester tool everything worked fine, thats why i opend this issue. Maybe it's my fault. I'll check this tomorrow

basically a little source code at issue is not bad :) blind debugging is not my expertise

i'm sorry :) for sure i'll send you my tests tomorrow morning :)

Okey i really don't know what i'm doing wrong now ^^ yesterday the payment worked perfectely today every payment failed.
I'm using .Net Core 3.1 on Windows 11
Now here's my testing source atm:
Startup:
private async void Form1_Load(object sender, EventArgs e)
        {
            AllocConsole();

            myDevice = new SerialPortDeviceCommunication("COM4");
            if (!await myDevice.ConnectAsync())
            {
                return;
            }

            using var loggerFactory = LoggerFactory.Create(builder =>
            {
                builder.AddConsole().SetMinimumLevel(LogLevel.Debug);
            });

            var deviceCommunicationLogger = loggerFactory.CreateLogger<TcpNetworkDeviceCommunication>();
            var zvtClientLogger = loggerFactory.CreateLogger<ZvtClient>();

            var clientConfig = new ZvtClientConfig
            {
                Language = Language.German,
                Password = 0101
            };

            myZvtClient = new ZvtClient(myDevice, zvtClientLogger, clientConfig);

            myZvtClient.ReceiptReceived += (receipt) => Console.WriteLine(receipt.Content);
            myZvtClient.IntermediateStatusInformationReceived += (intermediateStatusInformation) => Console.WriteLine(intermediateStatusInformation);
            myZvtClient.StatusInformationReceived += (statusInformation) => Console.WriteLine(statusInformation.ErrorMessage);
        }

Device Registration:
   private async void registerDevice_Click(object sender, EventArgs e)
        {

            var regConfig = new RegistrationConfig
            {
                ReceiptPrintoutForPaymentFunctionsViaPaymentTerminal = false,
                ReceiptPrintoutForAdministrationFunctionsViaPaymentTerminal = false,
                SendIntermediateStatusInformation = true,
            };

             var response = await myZvtClient.RegistrationAsync(regConfig);
             Console.WriteLine(response.State);
        }

Make Payment:
        private async void makePayment_Click(object sender, EventArgs e)
        {
            string amount = paymentAmount.Text;
            var convertDecimal = Decimal.Parse(amount,NumberStyles.AllowDecimalPoint, CultureInfo.InvariantCulture);
            var response = await myZvtClient.PaymentAsync(convertDecimal);
            Console.WriteLine(response.State);
        }

EndOfDay:
        private async void endOfDayAction_Click(object sender, EventArgs e)
        {
            var response = await myZvtClient.EndOfDayAsync();
            Console.WriteLine(response.State);
            Console.WriteLine(response.ErrorMessage);
        }

Please download the ControlPanel project and replace in MainWindow.xaml.cs
the tcp code
var loggerCommunication = this._loggerFactory.CreateLogger<SerialPortDeviceCommunication>();
this._deviceCommunication = new SerialPortDeviceCommunication(
    comPort: "COM9",
    baudRate: 9600,
    parity: Parity.None,
    dataBits: 8,
    stopBits: StopBits.Two,
    logger: loggerCommunication);

Where can i find the donwload for this project, i only see the .exe download.

oh crap.. my bad i'm sorry, didn't see that everything is combined in the repo ^^ sure one sec i'll try :)

still get "Tagesende nicht erfolgreich" when i'm trying to exec EndOfDay:
Console output:
EndOfDay (06 50) sent
info: Portalum.Zvt.ZvtClient[0]
      EndOfDayAsync - Execute
dbug: Portalum.Zvt.ZvtClient[0]
      SendCommandAsync - Send command to PT
Please wait...
PT is sending post-bookings
Please wait...
Please wait...
dbug: Portalum.Zvt.ZvtClient[0]
      ProcessApdu - 'Completion' received

i've copied every function of your code into my forms project cause on my machine i've only installed default minimal .NET Core and .Net Framework.

You must call CardComplete and have the terminal changed over by support

okey i'll try that :)

i do not know exactly why, but today it's working without any problems.
The only thing i recognised is that the Terminal have to be logged off for the EndOfDay function.
i can't tell why this happend, maybe something on the Terminal side.
The funny part is, that the test tool named "T-Manager" worked perfectly the whole day.
Nevermind, thanks a lot for your effort!