Support Debugger

Maybe a solution like https://github.com/avajs/ava/pull/929 but with --inspect instead.

This is now supported on the denali test command (denali test --debug). Just needs to be ported over to the serve command

Closing for now, since server and test support --debug. If you had something else in mind, feel free to reopen


How do I use vlc.dotnet  gapless playback of video files

How do I use vlc.dotnet   gapless playback of video files

I think, that therefor one has to implement the "media list" and the "medial list player" classes from LibVLC.

playlist feature is tracked by #193 . Reopen if something else is needed


Embedding model change

According to the article, If I want to change the embedding model from text-embedding-ada-002 to text-embedding-3-small. Which code shall I modify?
Your tutorial inspires my interests on RAG very much. Thank you!!
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [X] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

I just found it in the faq.md.
azd env set AZURE_OPENAI_API_EMBEDDINGS_MODEL text-embedding-3-large

Hi, @davidfungf . How are you?
It depends. If you want to use the Azure OpenAI Service, you can use AI Studio to create the embedding you want to use and change it with the command used.
Now, if you're using an LLM's Open Source you should change the file contanst.ts

Regarding the deployment, if changing the embedding model for the deployed version you need to delete the existing index in your AI Search instance in the portal:

This is needed as the vector length might differ.
Then you need to redeploy the app with azd up

Thank you for your support!
Suggestion: You may consider using Azure Document Intelligence Layout Model to generate the Markdown components to enhance the data accuracy. I looking forward to seeing your post soon.

Using Azure Document Intelligence is on our watchlist for a possible add-on, the initial idea for this one sample was to make it as simple as possible to understand.
We also have other more full-featured samples like this one, that uses Azure Document Intelligence for ingestion.


GSSoC: Auto Commenting Feat for Issues and PRs

This feature aims to address the problem of delayed and inconsistent communication following the creation of issues. By automatically commenting on issues and PRs as soon as they are created, it ensures that contributors receive immediate feedback and acknowledgment for their efforts. This fosters a positive and supportive environment for collaboration, encouraging continued participation and engagement from contributors.

@Swarnendu0123 please assign me  this issue under gssoc

Include #24 as rules in the comment.

@Asymtode712 give me the comments you are going to attach for issues and PRs.

I have modified the comments (included#24). Use the modified comments.

@Swarnendu0123 Sure

Can you automate one CI/CD which will check the pushed code is fine to merge or not?

@Swarnendu0123 Sure, should I open an issue for it?

Yes!


Cannot build

Hi, I've built Sodium from source, and now where do I put it to build Indium from source ? Because it searches sodium when building but doesn't find it...

This seems like a duplicate of my issue, how did you manage to build?


CSS issue when rendering on Ubuntu 22.10

UI is not rendered correctly on Ubuntu 22.10. It looks fine on Windows 11. I've attached a screen shot of what I'm seeing.

Using Awsome-UI V2.0.8

just fixed )
update to latest 2.0.9

Wow, that was freaky fast. It's working correctly now. Thanks.

actually i've tried to fix this fkng bug all day ))


Personabar Pages ellipsis and indent

The fact that page names are being truncated in the PersonaBar Pages module treeview can be considered to be a problem. Content editors and managers will in case of multiple titles with say more than 4-5 letters and a deeper page structure consisting of several levels of subpages find this UI interface difficult to navigate.
To avoid missing out on tabnames in the treview area would it be possible to integrate a more flexible layout in a coming release where a user would have the possibility to enlarge the treeview part of the PersonaBar iframe in the classical Windows fashion by dragging a column to the right or left.
A simpler alternative could consist in just adding a horizontal scrollbar at the bottom of the pages section.
DNN 9.3

[Version 87.0.4280.141 (Official Build) (64-bit) ] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer
[ Version 88.0.705.50 (Official build) (64-bit)] Edge

This has been resolved already, please upgrade.

This has been resolved already, please upgrade.

A tooltip also shows for even larger ones when hovered

A tooltip also shows for even larger ones when hovered

Thanks - much obliged.
I didn't check the DNN change log before making this request.

Thanks - much obliged.
I didn't check the DNN change log before making this request.


Feature: Favorites

Show the last 5 people you made transactions with in a separate section on top of the transfer screen.
Lightweight way of acknowledging the fact that we will likely send to the same people
Could also add a way to send to the same person from the transfers list - sort of like when we see a list of calls we have made, we can call the same person again by tapping on that call.
But definitely in the transfer screen! It should remember who my friends are - e.g. user accounts that appear in the transfer history.

@JGDoh I see it's tagged with v2. Is this ticket still of interest? Or good to close?

@n13 It is this still valid feature valid/required?


Kotsadm addon: standard type for TLS secret

Right now with the kotsadm addon, kurl.sh will create a secret called kotsadm-tls of type Opaque. It would be great if this could be of type kubernetes.io/tls so it can be re-used in Ingress specs and other K8s tools which process Secrets of this type.
(Per @areed this was done so we can store additional TLS-related fields in the secret)
I'd like to be able to re-use these certs in any applications being managed by kotsadm, specifcally to integrate with the countor addon in this case.
There's an open question around whether kotsadm should further manage this by propagating secrets to application namespaces, but I think I'd consider that out of scope for a v1 of this.

I think we could switch the type to be the proper kubernetes.io/tls type to make it more reusable. The additional data is small (it's a bit to indicate if the user uploaded the cert). We likely can move this bit over to an annotation in the secret for re-use.

annotation works for me


Fix duplicate paragraph, typo, add another link to API.md

Hi, the correct spelling is documentation. :smile:

Oh wow how did I miss this :D fixed


[PT Run] Search Improvement

Summary of the new feature/enhancement

So currently the search function as we open it and search for files everything seems to be doing right but the issue I have is this and I'm not sure if it's a bug or something that can be improved on. So what exactly is this issue is that every time I open the search and search a file/program let say and then till this point is good but then when I open up the search again whatever that I type previously is still around which makes me have to delete them and retype in what is the new file/program that I'm looking for and aside from that the text cursor is not active which makes me have to go an extra step to get my mouse to click on the search to have the cursor active and then delete the text previously keyed and then type in the new text to look for the file/program and I feel like this is something small that can be improved and help in the long run in things.
Proposed technical implementation details (optional)

What I proposed is this.

when we open up the search again after the initial what can be done is whatever that was previously keyed can stay put but make it highlighted and have the text cursor active so that with a click of delete we can straight away type in something new to move on to the next thing.
or
simply make the text cursor active instead of what is currently available which the text cursor is not active and many steps have to be taken to just retype something in the search to look for something new.

#4995 will enable the option to automatically clear the search box everytime you open Run. Would that solve your issue?

#4995 will enable the option to automatically clear the search box everytime you open Run. Would that solve your issue?

Hi there, I am having somewhat similiar issue, when I use search it's not clearing the recent thing/search you have type before so I think your solution is better to automatically clear the search box after a search has been made or after the search has been closed. Not just everytime you open run command but make it a global setting.
Thanks

#4995 will enable the option to automatically clear the search box everytime you open Run. Would that solve your issue?

yea I think that would be great as long as it makes the search so much easier to manage then I'm good and I believe everyone else would feel the same way as I do. Anyway the whole purpose of this function is to make searching at stuff as quick as possible so I think this implementation will be a plus to the service!

#4995 will enable the option to automatically clear the search box everytime you open Run. Would that solve your issue?

yea I think that would be great as long as it makes the search so much easier to manage then I'm good and I believe everyone else would feel the same way as I do. Anyway the whole purpose of this function is to make searching at stuff as quick as possible so I think this implementation will be a plus to the service!

Great! The setting will be available in the next update. Can this issue be closed for now :)?

#4995 will enable the option to automatically clear the search box everytime you open Run. Would that solve your issue?

yea I think that would be great as long as it makes the search so much easier to manage then I'm good and I believe everyone else would feel the same way as I do. Anyway the whole purpose of this function is to make searching at stuff as quick as possible so I think this implementation will be a plus to the service!

Great! The setting will be available in the next update. Can this issue be closed for now :)?

Alright can't wait for the next update and sure you can close the issue now and thank you!

Closing as duplicate!


QST: Has the finviz page changed since 28 sept 2023?

Have there been changes to the finviz.com tables since 28 sept 2023?

im having issues running finviz screeners today using the library.  Everything was working as of yesterday

AttributeError                            Traceback (most recent call last)
1 overview.set_filter(filters_dict={'Earnings Date': 'Today After Market Close'})
----> 2 overview.screener_view(order='Market Cap.')
278
279         table = soup.find("table", class_="table-light")

--> 280         rows = table.findAll("tr")
281         table_header = [i.text.strip() for i in rows[0].findAll("td")][1:]
282         num_col_index = [table_header.index(i) for i in table_header if i in NUMBER_COL]
AttributeError: 'NoneType' object has no attribute 'findAll'
I have the same error

Same here, look like sucked by Finviz.

any chance this issue will be solved soon? I have the same error since yesterday

For anyone looking for a workaround, I got this to monkeypatch the screener_view function.  But you would need to overwrite it in every class.  I also do not promise it is stable, but it works until the authors dig in:
from finvizfinance.screener.overview import Overview # THIS SHOULD BE THE PATH TO THE OVERVIEW YOU ARE USING
from finvizfinance.util import web_scrap

# It works but it's not pretty
def screener_view(self, group="Sector", order="Name"):
    """Get screener table.
    Args:
        group(str): choice of group option.
        order(str): sort the table by the choice of order.
    Returns:
        df(pandas.DataFrame): group information table.
    """
    if group not in self.group_dict:
        group_keys = list(self.group_dict.keys())
        raise ValueError(
            f"Invalid group parameter '{group}'. Possible parameter input: {group_keys}"
        )
    if order not in self.order_dict:
        order_keys = list(self.order_dict.keys())
        raise ValueError(
            f"Invalid order parameter '{order}'. Possible parameter input: {order_keys}"
        )
    self.url = (
        self.BASE_URL.format(group=self.group_dict[group], v_page=self.v_page)
        + "&"
        + self.order_dict[order]
    )

    soup = web_scrap(self.url)
    return pd.read_html(str(soup))[-2]

Overview.screener_view = screener_view

I'm having the same issue in overview.py line 295 296
table = soup.find("table", class_="table-light") rows = table.findAll("tr")
Exception caught: 'NoneType' object has no attribute 'findAll'
Any fix for this?
Here's how I'm using it
overview = Overview() overview.set_filter(filters_dict=filters_dict) df = overview.screener_view(order='Volume', ascend=False)
Thanks.

I will fix this. Thanks for reporting.

Version 0.14.7rc1 is published. Please check.

Thanks a ton!
Just checked, used a variety of filters and it worked.

Great to see this got fixed quickly after I posted this issue.
Finviz changed a few things, so I fixed some stuff and got it working(with the previous version) and continued working on my custom version.
I only changed it in the 'Custom' screener
Mainly changed this
# since 28 sep 2023
#table = soup.find("table", class_="table-light")
table = soup.find("table", class_="styled-table-new is-rounded is-tabular-nums w-full screener_table")
...
#table_header = [i.text.strip() for i in rows[0].findAll("td")][1:]
table_header = [i.text.strip() for i in rows[0].findAll("th")][1:]
I only changed it in the 'Custom' screener.
@lit26 Do you have an overview of what was changed?

@ljilekor I have updated most of the parsing in screener, group and others. The files changes is here. https://github.com/lit26/finvizfinance/pull/84/files#diff-55c34951e474479bab690402d7fc126b898dc2d640382dcef2c62a55e10c04c1


sap_install_media_detect: Recognize more file types with sapfile

We also want to detect maintenance planner files and certain saplanguage files without extension

Solved by https://github.com/sap-linuxlab/community.sap_install/pull/480 .


Redesign of Date/Time Fields on Test Card

WHY?
Kenny has proposed the following changes to be more efficient with the space on the test card
AS A user of SR
I WANT the test card date/time windows to be smaller
SO THAT that the layout of the test card is cleaner
Design
BEFORE

AFTER

Existing date component from "Add person" workflow

Figma
https://www.figma.com/file/NPSPM5SBzQ7Ra3mmr9M9cB/SimpleReport-Public-Site?node-id=5381%3A24437
Spec

https://www.figma.com/file/NPSPM5SBzQ7Ra3mmr9M9cB/SimpleReport-Public-Site?node-id=5381%3A24437
Design tasks:

[ ] Move DOB to the top
[ ] Move calendar icon inside the date field
[ ] Tighten spacing within date and time components
[ ] Condense headers to one title: 'Test date and time'
[ ] Remove subtext headers in gray
[ ] Change copy of checkbox text and header
[ ] Tighten spacing between date and time components
[ ] I recommend pairing with Kenny to review the frontend work together, once completed
[ ] Expand the width of dropdown for devices and swab types with longer horizontal widths

Translations

[ ] Translations are required
[ ] Translation request has been submitted
[ ] Translations have been received

@kenieh to add before/after designs

@zdeveloper heads up, I made a small update to the design in this ticket. For the device and swab type dropdowns—if possible—the widths of the dropdown should expand to fit the full horizontal width of long fields. In the screenshots I posted in this story—for example—the swab type drop-down expands to accommodate the long text: 'Nasal and throat swab combination'. The reasoning here is seeing the full width of the text is important context for the user to make better data decisions when submitting test results

Also added to the design tasks that the drop down field content should be sentence case rather than capitalizing each word, i.e. 'Nasal and throat swab combination' instead of 'Nasal and Throat Swab Combination'. This is per Rebecca's content guidelines


Compile str.lower()

Compile str.lower() into C++
For String: For 3-byte UTF-8 characters, which are 16 bits, adjust bytes per codepoint required from 4 to 2

Also, this branch is failing due to the linter complaining:
10.44s$ flake8 --show-source --select=E999,E722,W291,W292,W293,W391,F403
./nativepython/tests/string_compilation_test.py:190:1: W391 blank line at end of file
^
1     W391 blank line at end of file

you can see that by clicking into the details on travis and searching for "The build failed", where "the build" is a link.

Got it, thanks...
-Jeff

On Mar 15, 2019, at 7:47 AM, Braxton McKee<EMAIL_ADDRESS>wrote:
@braxtonmckee requested changes on this pull request.
looks good except for a couple of minor things to correct.
In nativepython/type_wrappers/string_wrapper.py:

@@ -137,3 +141,63 @@ def constant(self, context, s):
).cast(self.layoutType)
)
)

def convert_attribute(self, context, instance, attr):

   if attr in ("lower", "find"):

       return instance.changeType(BoundCompiledMethodWrapper(self, attr))

   return super().convert_attribute(context, instance, attr)

def convert_method_call(self, context, instance, methodname, args, kwargs):

   if kwargs:

       return super().convert_method_call(context, instance, methodname, args, kwargs)

   if methodname == "lower":

       if len(args) != 0:

               return

Here we should fall through to the super().convert_method_call(...). That generates code that throws an exception at runtime, which we want to do. As written, if code calls lower with more than one argument compilation will fail.
We should also be testing this behavior.
In typed_python/StringType.cpp:

@@ -136,7 +222,7 @@ int64_t String::bytesPerCodepointRequiredForUtf8(const uint8_t* utf8Str, int64_t
if (utf8Str[0] >> 4 == 0b1110) {
length -= 1;
utf8Str += 3;

       bytes_per_codepoint = std::max<int64_t>(4, bytes_per_codepoint);

       bytes_per_codepoint = std::max<int64_t>(2, bytes_per_codepoint);

this looks right, but can we add some tests to make sure we're covering this case correctly?
In typed_python/types_test.py:

@@ -171,7 +171,7 @@ def pickRandomly(self):

class NativeTypesTests(unittest.TestCase):

def check_expected_performance(self, elapsed, expected=1.0):

def check_expected_performance(self, elapsed, expected=1.5):
rather than widening this out, can you try setting the TRAVIS_CI environment variable and running tests that way? We have widened out all test run requirements for travis already so that should work for you everywhere..

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Braxton:
I've made a pull request for jeff-str-lower.  I have a separate branch for
jeff-str-find -- I will send that when I'm done with those test cases.
Thanks,
-Jeff
On Fri, Mar 15, 2019 at 9:32 AM Jeff Rubidge<EMAIL_ADDRESS>wrote:

Got it, thanks...
-Jeff
On Mar 15, 2019, at 7:47 AM, Braxton McKee<EMAIL_ADDRESS>wrote:
@braxtonmckee requested changes on this pull request.
looks good except for a couple of minor things to correct.
In nativepython/type_wrappers/string_wrapper.py
https://github.com/braxtonmckee/nativepython/pull/54#discussion_r265948177
:

@@ -137,3 +141,63 @@ def constant(self, context, s):
).cast(self.layoutType)
)
)

def convert_attribute(self, context, instance, attr):

   if attr in ("lower", "find"):

       return instance.changeType(BoundCompiledMethodWrapper(self, attr))

   return super().convert_attribute(context, instance, attr)

def convert_method_call(self, context, instance, methodname, args, kwargs):

   if kwargs:

       return super().convert_method_call(context, instance, methodname, args, kwargs)

   if methodname == "lower":

       if len(args) != 0:

               return

Here we should fall through to the super().convert_method_call(...). That
generates code that throws an exception at runtime, which we want to do. As
written, if code calls lower with more than one argument compilation will
fail.
We should also be testing this behavior.
In typed_python/StringType.cpp
https://github.com/braxtonmckee/nativepython/pull/54#discussion_r265948402
:

@@ -136,7 +222,7 @@ int64_t String::bytesPerCodepointRequiredForUtf8(const uint8_t* utf8Str, int64_t
if (utf8Str[0] >> 4 == 0b1110) {
length -= 1;
utf8Str += 3;

       bytes_per_codepoint = std::max<int64_t>(4, bytes_per_codepoint);

       bytes_per_codepoint = std::max<int64_t>(2, bytes_per_codepoint);

this looks right, but can we add some tests to make sure we're covering
this case correctly?
In typed_python/types_test.py
https://github.com/braxtonmckee/nativepython/pull/54#discussion_r265948677
:

@@ -171,7 +171,7 @@ def pickRandomly(self):

class NativeTypesTests(unittest.TestCase):

def check_expected_performance(self, elapsed, expected=1.0):

def check_expected_performance(self, elapsed, expected=1.5):

rather than widening this out, can you try setting the TRAVIS_CI
environment variable and running tests that way? We have widened out all
test run requirements for travis already so that should work for you
everywhere..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/braxtonmckee/nativepython/pull/54#pullrequestreview-214990767,
or mute the thread
https://github.com/notifications/unsubscribe-auth/At14i3_MI2jKpOJXPY1PKfP8u4ZdsGrQks5vW4hrgaJpZM4b0dZs
.

obviated by #56


Query: Add Sqlite implementation for DateDiff functions

Add translation for DateTime.Today
Combine translators for DateTime members in Sqlite
Part of #9585
Resolves #4108
Resolves #10468

cc: @bricelam - To review SQL generated for SQLite.

SQL looks hideous, but it'll be way better than filtering on the client. It's also as good as it can be given SQLite's limited DateTime support, so.. :shipit:

@maumar - Can you review this?


Documentation typo?

Look at this:
https://pyairtable.readthedocs.io/en/latest/orm.html

It says:
partner = fields.LinkField("Partner", "Contact", lazy=False)

The second parameter is a string. In the code itself this is documented as object:
https://github.com/gtalarico/pyairtable/blob/main/pyairtable/orm/fields.py#L43
I guess that must be corrected in one of the two.

Good catch.
Doc should show model instance not str.
I intended to support both but have not implemented.
The str option allows circular referencing (django and my support similar approaches where it loads object by name reference)

This was resolved in #277; see https://pyairtable.readthedocs.io/en/stable/orm.html#linked-records


Remove console.log

Noticed this on esnextb.in 😋

added you as a collab - feel free to merge and publish :D
On Fri, Dec 16, 2016 at 12:58 AM Jason Miller<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/yoshuawuyts/rooch/pull/5
Commit Summary

Remove console.log

File Changes

M html.js
https://github.com/yoshuawuyts/rooch/pull/5/files#diff-0 (1)

Patch Links:

https://github.com/yoshuawuyts/rooch/pull/5.patch
https://github.com/yoshuawuyts/rooch/pull/5.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yoshuawuyts/rooch/pull/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWlelKwGFqeqzhgvCWo30lrch2Y5g-vks5rIdRAgaJpZM4LOtVW
.


Using react-three-renderer with Typescript

This doesn't appear to be straightforward given the lack of @types for this lib. I have tried to force it in and ignore the typescript compiler warnings but get the following error:
Uncaught TypeError: Cannot read property 'Object3D' of undefined
    at eval (webpack:///./~/react-three-renderer/lib/Resources/ResourceContainer.js?:31:18)

Are there any plans to provide support for Typescript? Or are there any workarounds perhaps?
Thanks

My error was due to using wrong THREE version, so that is resolved.
I also managed to get TS compile errors to be reduced significantly by adding the following to my code:
declare global {
    namespace JSX {
        interface IntrinsicElements {
            scene: any;
            perspectiveCamera: any;
            ambientLight: any;
            axisHelper: any;
            // other elements can be added here
        }
    }
}

Note that Typescript treats component names in JSX with lowercase as 'intrinsic', ie. expects them to be HTML and not React components.
I still have an error on compiling about missing module "react-three-component" but Webpack is including it and my app is running.
Perhaps others can add Typescript specific help/workarounds to this thread. Would be good to see first class Typescript support at some point!

Regarding lower case components being html perhaps we could extend the
definitions or something to accept custom "native types", I think that
would be awesome for prop type verification on compile time!
Currently the main priority of the project is to have full feature
compatibility with threejs on latest react, but it would still be awesome
to see it working nicely with TS :) let's keep the issue open until then,
could be a fun one.
On Thu, Nov 17, 2016, 20:57 jugglingcats<EMAIL_ADDRESS>wrote:

My error was due to using wrong THREE version, so that is resolved.
I also managed to get TS compile errors to be reduced significantly by
adding the following to my code:
declare global {
namespace JSX {
interface IntrinsicElements {
scene: any;
perspectiveCamera: any;
ambientLight: any;
axisHelper: any;
// other elements can be added here
}
}
}
Note that Typescript treats component names in JSX with lowercase as
'intrinsic', ie. expects them to be HTML and not React components.
I still have an error on compiling about missing module
"react-three-component" but Webpack is including it and my app is running.
Perhaps others can add Typescript specific help/workarounds to this
thread. Would be good to see first class Typescript support at some point!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/toxicFork/react-three-renderer/issues/127#issuecomment-261367274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0iLY1tdbnYIMhkJMsdBeCvhjbIUWByks5q_L_HgaJpZM4K1o39
.

I agree it would be great to have prop/attribute validation and auto-complete in the IDE and this is one of the main benefits of using Typescript IMO.
I am not sure adding elements to JSX.IntrinsicElements is the way to go, because of possible conflicts with other libs (you would effectively be polluting a shared namespace). I think ultimately you might consider providing initial-caps versions of react-three-renderer components, ie. Scene, PerspectiveCamera and so on. These could either be used as <React3.Scene> or aliased during import.
Just my 2p

For info I already found a name clash: line is defined as type React.SVGProps.
It runs :) but compiler and IDE complain :(

I think things would need to be context sensitive somehow as new renderers emerge. Could do an annotation to a file or the render function for example to specify what sort of renderer will be used for a component.

I don't understand very much about renderers, but presumably a single component could use React3 and SVG, eg:
render() {
  return (<div>
   <svg><line ... /></svg>
   <React3>
   ...
     <line ... />
   ...
   </React3>
  </div>)
}

That's a bad component that should be split in half and deserves multiple
warnings ;)
On Fri, Nov 18, 2016, 11:48 jugglingcats<EMAIL_ADDRESS>wrote:

I don't understand very much about renderers, but presumably a single
component could use React3 and SVG, eg:
render() {
return (
<line ... />

...
<line ... />
...

)
}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/toxicFork/react-three-renderer/issues/127#issuecomment-261514142,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0iLVZVWYGeFkzGqZatPPn1qoP-lxm1ks5q_ZCfgaJpZM4K1o39
.

Heheh. I tend to agree!
I thought more about what you said and I think what you need is a way to say "The React3 component doesn't contain JSX.IntrinsicElements but instead contains React3.IntrinsicElements". That would be a nice solution.

I just came across this thread.  I came up with a slightly different approach that doesn't involve having to extend JSX.IntrinsicElements but gives you all the type safety.  It involves a bit of work creating wrapper elements.  But honestly, I think these are probably as much work as creating a .d.ts file.
First, I we have to import React3.  I was wrestling with TS2+'s augmentation stuff and couldn't get it to work.  So I just punted and did this at the top of my file:
let React3: React.StatelessComponent<React3Props> = require('react-three-renderer');

...and I defined React3Props as:
export interface React3Props {
    mainCamera: string;
    width: number;
    height: number;
    onAnimate?: () => void;
}

I'm sure there is more, but this is all I'm using so far.  Then you have the issue of getting that stuff out of IntrinsicElements.  I managed that by creating wrapper classes like this:
export interface PerspectiveCameraProps {
                    name: string;
                    fov: number;
                    aspect: number;
                    near: number;
                    far: number;
                    position: THREE.Vector;
}

export class PerspectiveCamera extends React.Component<PerspectiveCameraProps,void> {
    render() {
        return React.createElement("perspectiveCamera", { ...this.props });
    }
}

Now you've got a component that the TS compiler recognizes as a first class component. This means you can then do:
<PerspectiveCamera
     name="camera"
     fov={75}
     aspect={this.props.width / this.props.height}
     near={0.1}
     far={1000}
     position={this.cameraPosition}/>

Now this may seem like a lot of work.  But like I said, I'm not sure it is any more work than writing a normal .d.ts file.  A slight advantage that this has, I suppose, is that it doesn't require react-three-renderer to change anything about its conventions.  As far as I can tell, what the author was going for was a direct mapping between his components and the underlying THREE.* functions (which have lower case names).  So this is just layering on top of that things that are more like first class React (class-based) components that can be composed just like the react-three-renderer components but with increased type safety.
Thoughts?

Interesting approach...!
I assume it works fine when you have nested react-three-renderer components, even though you are introducing an intermediate element into the component tree?
There would be a small performance overhead but I don't see this being an issue.
I guess one disadvantage is that this would likely be a separate project and cannot easily become a first class citizen, ie. part of the react-three-renderer project in the way that .d.ts definitions could.
I just posted this question to SO as a last attempt to find a "pure" solution: http://stackoverflow.com/questions/43493059/react-typescript-custom-renderer-and-jsx-intrinsicelements.
If we adopt this approach, would you be willing to host the project...? Or maybe @toxicFork would be willing to make it a companion project to react-three-renderer.

The intermediate elements don't really seem to cause any problems with the exception of <scene>.  I'm not sure why, but that one doesn't want to be wrapped.  Perhaps there is some logic somewhere that insists it be a direct child of the root.  Not sure.  For the others, I've successfully created "intermediate" elements that are nice first class react components (with their own typed properties, etc).
I agree, there is a small overhead.  Note that, as far as I can tell, anytime you'd like to group things into reusable components, you'd introduce just about the same overhead.  I'm just forcing each leaf in the tree to be a reusable component.  There is both the overhead of the function call as well as the overhead of traversing a deeper tree.  Perhaps there are other concerns as well.
I'm not sure how the "mounting" vs. "rendering" stuff is handled internally in R3R.  My hope is that it is using the virtual DOM approach to optimize (i.e., minimize) manipulations of the 3D scene.  If so, that might help.  If not, then it would perhaps amplify any inefficiencies introduced here.
You are correct that it would probably be a separate project.  You could, of course, include all this in R3R itself.  But I'm guessing the author doesn't want that headache.  I would have preferred the .d.ts approach myself, to be clear.  But since that seems very difficult in this case because of the conventions followed by R3R and TS, this was the best I could come up with.
As for "hosting" it, I'd certainly be willing to make it publicly available.  I make no promises on completeness or maintenance.  I don't expect to be using this extensively in the future.  So I can't say I'd be able to do all the greens keeping some might expect.  Perhaps @toxicFork can chime in with his opinions before we really make any decisions here.

Cool solutions :)
I would prefer a comment / .d.ts based solution even if it takes more work because I am against overheads in general. However, there are ways to solve the overhead problem, for example we could (with some effort) write a babel (or whatever compiler) pre/post process plugin to convert these back into native components.
I think it would be good to have it as a separate project if possible (so you will have authority over it, issues for it won't be lost in normal r3r issues, and so on), then we can consider merging it into R3R when the time comes.

FYI, I actually gave up on my wrapper stuff.  It wasn't that it was technically unsound.  I just ended up opting for something that interacted directly with the THREE api rather than representing the data declaratively in JSX.


Image Resizer is using width for height... I think.

Microsoft PowerToys version
0.37.2
Running as admin

[ ] Yes

Area(s) with issue?
Image Resizer
Steps to reproduce
Pardon the vagueness of the title. If I tell Image Resizer to fit an image to: 750x1000 (750 wide by 1k high), and tell it to not make images any bigger, only smaller, I'm seeing it create an image where the width is 1000 and the height is some other number. It's like it used the height for width.
✔ Expected Behavior
To resize as specified.
❌ Actual Behavior
It seems to be flipping the dimensions maybe?
Other Software
No response

what sizing method are you using?  Possible to just create a screenshot of the resizer and then maybe an image?  I did a Fit with 1024 and no height and my image was width 1024

Original image was 750 x  350. New image is 600 x 300.

I'm seeing similar results. We should investigate this in the 0.45/0.47 release cycles.

Isn't this because of the check box..?

[ ] Ignore orientation of pictures

@cfjedimaster try un-checking it and let me know if that does the trick?
Couldn't find this specific setting on https://docs.microsoft.com/nl-nl/windows/powertoys/image-resizer

Just tested with most recent (w/o changing the box, just wanted to confirm). Image now comes out at 600x280. So slightly different, but still wrong imo as I had said it needed to fit in the width. Now testing with the checkbox off I get 500x233. That value seems right.
So maybe I don't understand the setting, Ignore the orientation. To me, if UNCHECKED, it means check and see if the orientation is wrong, like, it's portrait but when view it it's landscape, but you can tell it was meant to be rotated. By keeping it checked, my mind says to just treat it as is.
For me test, I went into paint.net, made a new image 750x350, and that's it. I didn't rotate, or heck, even add color. Just a white canvas.

It appears this is not intuitive at all. I remember an explanation like this: "if checked, the smallest number (in width/height) in the settings will be applied to the smallest dimension of the picture. Regardless if this is declared as width or height." The result is that a number of different photos with different orientations (like vacation photos) will still be the same size.
@dedavis6797 I'm thinking this checkbox could use either a label or a tooltip with more info.
Also, this info should be on the page that I linked earlier.

A manual work-around, with the same result, is to set width & height the same value, so that all images fit into a square of n by n pixels.
@crutkas @niels9001 Maybe this is a more intuitive solution that we can use when the UX is updated?

Ref: #2387

Hate to ask, but what's keeping this issue open? Maybe more changes to the actual UI..?

@dedavis6797 @Jay-o-Way Oh missed this one. I think it would be nice to somehow have a visual preview of what the new image would look like. If it becomes smaller, we can visualize that as well.

@dedavis6797 @Jay-o-Way @niels9001 What is the actual POR here for this.  It is slightly unclear from the thread.

I'm lost too

@crutkas Only UX tweak I could imagine is showing a preview of what the new aspect ratio would look like. But since you can select multiple images with multiple aspect ratios I think that will be difficult to do (only if we could somehow show a preview list). Therefore, I'd say this can be closed?

+1 for closing this


T - UX for users to opt-out of marketing emails

Task
UX for users to opt-out of marketing emails
Acceptance Criteria

[ ] Engineering sign-off
[ ] Product manager sign-off

If a user is already opted-in, they can opt-out by unchecking the box below their email address and clicking [Save] button at the bottom of their profile. This is not intuitive since modern forms/fields auto save. But this is the current functionality on MDN.

Added acceptance criteria for engineering (@tobinmori) and product manager (@atopal) sign off - thank you!

I'm good with this design though I should say that making the opt-in checkbox 'live' (vs saved on page submit) will require extra work (require an API call and some JS), so maybe we implement as 'save on submit' for phase 1 and wait for phase 2 to make this page more dynamic (to keep Yari1 moving forward).

Agreed, the functionality proposed is 'saved on page submit' as you said. Also, proposed revising the copy in the button from [publish] to [save] for this reason.

Thanks @malqinneh that looks good to me 👍  Can you make sure the implementing dev is aware that the button text is changing as well? I missed that at first glance.


Add planetary-computer

Facilitates creating signed urls for planetary computer datasets

@scottyhq I opened https://github.com/microsoft/PlanetaryComputer/issues/364, do you know an answer to that question?

Source code is at https://github.com/microsoft/planetary-computer-sdk-for-python. It's mentioned under https://github.com/microsoft/PlanetaryComputer/tree/2023.1.26?tab=readme-ov-file#open-source-repositories


make installcheck fails in local environment

Environment

OS: Rocky Linux 9
PostgreSQL: REL_16_STABLE (1b9dd6b05a27ad96a027a0b8ab1e5efd418605a2)
AGE: master (e370db34afb255236a77012882d17e3f396e7f5b)

Summary
make installcheck fails in the local environment.
GitHub Actions succeeds, so there seems to be something wrong with my environment.
SELECT create_vlabel('graph123', 'vlabel01234567890123456789012345678901234567890123456789012345678901234567890123456789012345678'); should result in an ERROR, but the command succeeds after the label name is truncated to 63 characters.
postgres=# SELECT create_graph('graph123');
NOTICE:  00000: graph "graph123" has been created
LOCATION:  create_graph, graph_commands.c:66
 create_graph
--------------

(1 row)

postgres=# SELECT create_vlabel('graph123', 'vlabel01234567890123456789012345678901234567890123456789012345678901234567890123456789012345678');
NOTICE:  00000: VLabel "vlabel012345678901234567890123456789012345678901234567890123456" has been created
LOCATION:  create_vlabel, label_commands.c:205
 create_vlabel
---------------

(1 row)

postgres=# SELECT create_vlabel('graph123', 'vlabel01234567890123456789012345678901234567890123456789012345678901234567890123456789012345678');
ERROR:  3F000: label "vlabel012345678901234567890123456789012345678901234567890123456" already exists
LOCATION:  create_vlabel, label_commands.c:193

Does anyone know the cause of this?
Log
pg_config
$ pg_config
BINDIR = /home/shinya/pgsql/16-age/bin
DOCDIR = /home/shinya/pgsql/16-age/share/doc
HTMLDIR = /home/shinya/pgsql/16-age/share/doc
INCLUDEDIR = /home/shinya/pgsql/16-age/include
PKGINCLUDEDIR = /home/shinya/pgsql/16-age/include
INCLUDEDIR-SERVER = /home/shinya/pgsql/16-age/include/server
LIBDIR = /home/shinya/pgsql/16-age/lib
PKGLIBDIR = /home/shinya/pgsql/16-age/lib
LOCALEDIR = /home/shinya/pgsql/16-age/share/locale
MANDIR = /home/shinya/pgsql/16-age/share/man
SHAREDIR = /home/shinya/pgsql/16-age/share
SYSCONFDIR = /home/shinya/pgsql/16-age/etc
PGXS = /home/shinya/pgsql/16-age/lib/pgxs/src/makefiles/pgxs.mk
CONFIGURE =  '--enable-debug' '--enable-cassert' '--enable-tap-tests' '--without-icu' '--with-pgport=5416' 'CFLAGS=-std=gnu99 -ggdb -O0' '--prefix=/home/shinya/pgsql/16-age'
CC = gcc
CPPFLAGS = -D_GNU_SOURCE
CFLAGS = -Wall -Wmissing-prototypes -Wpointer-arith -Wdeclaration-after-statement -Werror=vla -Wendif-labels -Wmissing-format-attribute -Wimplicit-fallthrough=3 -Wcast-function-type -Wshadow=compatible-local -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -Wno-format-truncation -Wno-stringop-truncation -g -std=gnu99 -ggdb -O0
CFLAGS_SL = -fPIC
LDFLAGS = -Wl,--as-needed -Wl,-rpath,'/home/shinya/pgsql/16-age/lib',--enable-new-dtags
LDFLAGS_EX =
LDFLAGS_SL =
LIBS = -lpgcommon -lpgport -lz -lreadline -lm
VERSION = PostgreSQL 16.4

regression.out
# using temp instance on port 61958 with PID 3722157
ok 1         - scan                                      258 ms
ok 2         - graphid                                    11 ms
ok 3         - agtype                                    141 ms
ok 4         - agtype_hash_cmp                             8 ms
ok 5         - catalog                                   132 ms
ok 6         - cypher                                     54 ms
ok 7         - expr                                      914 ms
ok 8         - cypher_create                             176 ms
ok 9         - cypher_match                              771 ms
ok 10        - cypher_unwind                              62 ms
ok 11        - cypher_set                                177 ms
ok 12        - cypher_remove                              97 ms
ok 13        - cypher_delete                             434 ms
ok 14        - cypher_with                                89 ms
ok 15        - cypher_vle                               1487 ms
ok 16        - cypher_union                               32 ms
ok 17        - cypher_call                                43 ms
ok 18        - cypher_merge                              476 ms
ok 19        - cypher_subquery                            99 ms
ok 20        - age_global_graph                          191 ms
ok 21        - age_load                                 2481 ms
ok 22        - index                                     113 ms
ok 23        - analyze                                    29 ms
ok 24        - graph_generation                           87 ms
not ok 25    - name_validation                           204 ms
ok 26        - jsonb_operators                           120 ms
ok 27        - list_comprehension                         90 ms
ok 28        - map_projection                             44 ms
ok 29        - drop                                      440 ms
1..29
# 1 of 29 tests failed.
# The differences that caused some tests to fail can be viewed in the file "/home/shinya/pgsql/16-age/postgresql/contrib/age/regress/regression.diffs".
# A copy of the test summary that you see above is saved in the file "/home/shinya/pgsql/16-age/postgresql/contrib/age/regress/regression.out".

regression.diffs
diff -U3 /home/shinya/pgsql/16-age/postgresql/contrib/age/regress/expected/name_validation.out /home/shinya/pgsql/16-age/postgresql/contrib/age/regress/results/name_validation.out
--- /home/shinya/pgsql/16-age/postgresql/contrib/age/regress/expected/name_validation.out	2024-08-15 12:11:32.974396664 +0900
+++ /home/shinya/pgsql/16-age/postgresql/contrib/age/regress/results/name_validation.out	2024-08-15 14:02:27.320278775 +0900
@@ -423,29 +423,27 @@
 --
 -- now check vlabel creation, should fail
 SELECT create_vlabel('graph123', 'vlabel01234567890123456789012345678901234567890123456789012345678901234567890123456789012345678');
-WARNING:  label name length not in range (1 <= length <= 63) length = 95
-ERROR:  label name is invalid
--- should be successful
-SELECT create_vlabel('graph123', 'vlabel012345678901234567890123456789012345678901234567890123456');
 NOTICE:  VLabel "vlabel012345678901234567890123456789012345678901234567890123456" has been created
  create_vlabel
 ---------------

 (1 row)

+-- should be successful
+SELECT create_vlabel('graph123', 'vlabel012345678901234567890123456789012345678901234567890123456');
+ERROR:  label "vlabel012345678901234567890123456789012345678901234567890123456" already exists
 --
 -- now check elabel creation, should fail
 SELECT create_elabel('graph123', 'elabel0123456789012345678901234567890123456789012345678901234567');
-WARNING:  label name length not in range (1 <= length <= 63) length = 64
-ERROR:  label name is invalid
--- should be okay
-SELECT create_elabel('graph123', 'elabel012345678901234567890123456789012345678901234567890123456');
 NOTICE:  ELabel "elabel012345678901234567890123456789012345678901234567890123456" has been created
  create_elabel
 ---------------

 (1 row)

+-- should be okay
+SELECT create_elabel('graph123', 'elabel012345678901234567890123456789012345678901234567890123456');
+ERROR:  label "elabel012345678901234567890123456789012345678901234567890123456" already exists
 -- clean up
 SELECT drop_graph('graph123', true);
 NOTICE:  drop cascades to 20 other objects

@shinyaaa Did you make a clean installation?
Please try make clean before make install.

@MuhammadTahaNaveed
Thank you very much! This issue has beel resolved.


feat: Blueprint to demonstrate installation of multiple nginx ingress controllers for internal and public traffic split

Description
This example demonstrates how to provision an Amazon EKS cluster and configure two nginx ingress controllers; one to expose applications publicly and the other to expose applications internally. Today, it is not always clear how to install duplicates of an EKS Blueprint addon to meet this use case. This blueprint also demonstrates how to leverage the new Security Group support for Network Load Balancers feature.
Motivation and Context
See above.
How was this change tested?

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[x] Yes, I have updated the docs for this feature
[ ] Yes, I ran pre-commit run -a with this PR

Additional Notes
N/A

Looks like the security group doesn't get associated with those LB

Looks like the security group doesn't get associated with the LB

Can you please open an issue with what you are seeing?


Allow twins caching in mass deployer

Is your feature request related to a problem? Please describe
rmb allows caching twins info for better performance
Describe the solution you'd like
add new option to use rmb twin cache to improve deployer performance

Work Completed:

added a new flag rmbInMemCache to allow memory or disk cache
increased default timeout from 10s to 60s
updated all packages to use in memory cache
updated mass deployer to use in disk cache

Verified:
tfrobot still uses in memory cache, no cache can be found under /tmp/rmb-cache after running tfrobot.
$ ls /tmp/rmb-cache
ls: cannot access '/tmp/rmb-cache': No such file or directory

Work completed:
twin cache is now added to tfrobot

Verified:
tfrobot now uses /tmp/rmb-cache to cache twins:
$ tree /tmp/rmb-cache
/tmp/rmb-cache
└── tfchain.dev.grid.tf:443
    └── 192

1 directory, 1 file


Broken inplace integrand for HCubatureJL

Hi,
I was writing a quadrature code with inplace integrands and I noticed that Integrals.jl has the same problem I noticed. Depending on how the quadrature library allocates arrays, it is possible that multiple integrand evaluations are made before they are summed together. When the result is overwritten to the same array this means some of the quadrature nodes are overwritten by the values of another. For example, integrating sin(x) on [-1,1] as below should give zero on the initial segment, but the inplace evaluator seems to integrate sin(abs(x)) on that segment.
julia> using Integrals

julia> prob = IntegralProblem((x,p) -> sin(p*x), -1, 1, 3.0)
IntegralProblem. In-place: false

julia> inplaceprob = IntegralProblem((y,x,p) -> y .= sin(p*x), -1.0, 1.0, 3.0)
IntegralProblem. In-place: true

julia> solve(prob, HCubatureJL(), maxiters=15)
u: 0.0

julia> solve(inplaceprob, HCubatureJL(), maxiters=15)
u: 1-element Vector{Float64}:
 1.3044979103237169

The problem is in the definition of the integrand wrapper
dx = zeros(eltype(lb), prob.nout)
f = x -> (prob.f(dx, x, prob.p); dx)

An issue with dx is that it may have the wrong type, since the user's function could return complex values at real nodes, for example. A fix is to allow the user to pass in the output array. The second issue in the example above can be fixed by returning a new array at each integrand evaluation, which can be done by returning one(eltype(lb))*dx. The downside to this last fix is that most of the array operations aren't inplace, but the only way to achieve this is if the library supports them (e.g. see quadgk! in QuadGK.jl for an implementation).

Yes, I think a breaking change is needed to fix this.
Making the user pass their own prototype container seems like the best solution to me also.
This would then also get rid of the nout keyword.

I think that's a good suggestion.

Now that the SciMLBase PR has merged, Integrals.jl only supports SciMLBase v1 right now because the integrand interface is the v2 breaking change. This is the issue that has to be solved for the bump to v2 to occur.

Fixed by new interface 🎉 🎉 🎉 🎉 🎉


Multi language

What is the best approach for making aqua a multi language website?

Thanks for opening an issue. There isn't a best approach to internationalization (i18n). I suggest you do some research and find an approach that works best for your project. I haven't yet had the need for i18n in Aqua myself yet, but here are some of the first results I've found by searching:

https://www.npmjs.com/package/i18n
https://www.npmjs.com/package/i18n-react
https://github.com/yahoo/react-intl
https://github.com/funktionswerk/hapi-i18n

Please update this thread with your findings.

Hi there. I'm watching Aqua for while now, but didn't find time for more. :(
Currently I'm working with https://github.com/yahoo/react-intl and I think it's strait forward. Okay, the documentation isn't the best but there're enough google results out there.

I managed to implement a solution with https://www.npmjs.com/package/hapi-locale-i18n. It works great and it is not so complicated to use.

However, I have problems with onClick functions when i try to change language in Navbar.
Here is my function I'm trying to call:
setLanguage(lang) {         console.log('HERE');         i18n.setLocale(lang);         return false;     }
Here is part from Navbar:

                                 {i18n.__("si")}

                                 {i18n.__("en")}

I also tried without bind, so like this:

                                 {i18n.__("si")}

                                 {i18n.__("en")}

If you have any ideas why?
Thank you

If you had an error message it would be easier to help. But at first I'd wonder how the strings get loaded client side.


ENYO-3030: Prevent reading disabled component

Issue
screen reader doesn't read disable Button's string, but it reads disabled IconButton.
Cause
Button is created with button tag, but IconButton is div tag. If form element has disabled attribute,
screen reader doesn't read string.
Fix
Prevent reading disabled component's string as removing tabindex.
https://jira2.lgsvl.com/browse/ENYO-3030
Enyo-DCO-1.1-Signed-off-by: Bongsub Kim<EMAIL_ADDRESS>

Close as outdated PR


v1.0.2

@Umang01-hash, @vipul-rawat - I think GofrVersion is missed to update and is still 1.0.0. Please take a look on that.


DESI target labelling: did some bits change?

From Eddie Schlafly:
Re viewer updates... here's the viewer and the viewer-dev for a patch of the sky:
http://legacysurvey.org/viewer-dev?ra=178.1051&dec=20.6939&zoom=16&layer=decals-dr7&targets-dr67
http://legacysurvey.org/viewer?ra=178.1051&dec=20.6939&zoom=16&layer=decals-dr7&targets-dr67
does the center source lose its label for some reason?

Hey @geordie666 , were there any changes to the targeting bits that might cause this?
"viewer" is using dr7.1-0.23.0
"viewer-dev" is using dr7.1-0.29.0

I think the changes that @ziyaointl made were purely CSS, so wouldn't affect whether a target had a label...?

This is caused by how desitarget resolves targets with a view towards DR8. The default is now to resolve targets (RESOLVE =                    T in the header of the target files).
For resolved targets, the primary bits ELG, LRG, QSO... are only set, for objects from DECaLS photometry, if they pass the "southern" (DECaLS-appropriate) targeting algorithms. The particular target you highlight passes the "northern" cuts but its provenance is from the "southern" imaging. It therefore has ELG_NORTH set but not ELG.
At some point (I'm guessing) displaying the NORTH/SOUTH labels in the viewer was deprecated to avoid label-clutter, so no label is included for this object.
Python dump:
from desitarget.io import read_targets_in_cap
from desitarget.targetmask import desi_mask
targ = read_targets_in_cap("0.29.0/main", [178.1049, 20.6961, 1./3600], columns=["DESI_TARGET"])
desi_mask.names(targ["DESI_TARGET"])
Out[]: ['ELG_NORTH']

Lots of context in https://github.com/desihub/desitarget/pull/426 with more discussion in https://github.com/desihub/fiberassign/pull/193. Mark (working with @sbailey) also recently independently discovered this (see [decam-data 3396]).

Thanks, @geordie666 !
@ziyaointl , would you be willing to change the server-side code that does the labeling -- in map/cats.py function desitarget_color_names?  I think what we want is to only set ELG_NORTH, ELG_SOUTH, etc, if ELG is not set?  Otherwise, every ELG will be labelled "ELG, ELG_NORTH", which is going to clutter things.

@dstndstn Sure! I will get started this week


Polling for incoming messages

Hello,
Sorry for the intrusion.
Just wondering what would be the best way to implement a polling mechanism that would trigger a callback whenever a new message is received.
So far I only found examples that would receive a single message and then disconnect from the broker.
I need to implement an endpoint that would maintain a connection with the broker and that would permanently subscribe to a specific topic and keep receiving messages on said topic.
Thanks in advance.
Elia

Hello!
Essentially, you want to set a new mqtt_client::async_receive callback after invoking the previous one. With callbacks, it would look something like this:
namespace asio = boost::asio;

using stream_type = asio::ip::tcp::socket;
using client_type = async_mqtt5::mqtt_client<stream_type>;

void receive_message(client_type& client) {
	client.async_receive([&client](
		async_mqtt5::error_code ec, std::string topic,
		std::string payload, async_mqtt5::publish_props
	) {
                // session and our subscriptions are lost
		if (ec == async_mqtt5::client::error::session_expired) {
			// resubscribe...
		} else if (ec) {
			// break...
		} else {
			// process message
			receive_message(client);
		}
	});
}

int main(int argc, char* argv[]) {
	asio::io_context ioc;

        // using signals to exit
	asio::signal_set signals(ioc, SIGINT, SIGTERM, SIGBREAK);
	signals.async_wait(
		[&ioc](boost::system::error_code ec, int sig) {
			ioc.stop();
		}
	);

	client_type client(ioc, "");

        // configure the client, subscribe and check if the subscription is successfully established

        receive_message(client);

        ioc.run();
}

However, if you are able to, I would recommend using coroutines for this case, as it leads to simpler and cleaner code. Our receiver  example would be slightly modified to achieve the desired behaviour.
#include <boost/asio/as_tuple.hpp>
#include <boost/asio/co_spawn.hpp>
#include <boost/asio/detached.hpp>
#include <boost/asio/io_context.hpp>
#include <boost/asio/signal_set.hpp>
#include <boost/asio/use_awaitable.hpp>

#include <boost/asio/ip/tcp.hpp>

#include <async_mqtt5.hpp>

namespace asio = boost::asio;
constexpr auto use_nothrow_awaitable = asio::as_tuple(asio::use_awaitable);

asio::awaitable<void> client_receiver(asio::io_context& ioc) {
	async_mqtt5::mqtt_client<asio::ip::tcp::socket> client(ioc, "");

        // configure client
	client.credentials("tester", "", "")
		.brokers("mqtt-broker", 1883)
		.run();

       // subscribe and check if the subscription is successful
	auto [ec, sub_codes, sub_props] = co_await client.async_subscribe(...);

       // if we failed to subscribe, co_return here

	for(;;) {
		auto [rec, topic, payload, publish_props] = co_await client.async_receive(use_nothrow_awaitable);

		if (rec == async_mqtt5::client::error::session_expired) {
			// resubscribe
		} else if (rec) {
			break;
		} else {
			// process message
		}
	}

	co_return;
}

int main() {
	asio::io_context ioc;

        // using signals to exit
	asio::signal_set signals(ioc, SIGINT, SIGTERM);
	signals.async_wait(
		[&ioc](boost::system::error_code ec, int sig) {
			ioc.stop();
		}
	);

	co_spawn(ioc, client_receiver(ioc), asio::detached);

	ioc.run();
}

If you have any further questions, feel free to ask!
I hope this helps,
Korina


Remove simd gen accounts command in favor of SDK command

Description

Wrap GenTxCmd, MigrateGenesisCmd, CollectGenTxsCmd, ValidateGenesisCmd, AddGenesisAccountCmd into genesisCommand
Remove file genaccount.go, using sdk AddGenesisAccountCmd
Update tests

closes: #2072
Commit Message / Changelog Entry
type: commit message

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

We need to approval workflow @colin-axner. Thanks

@hieuvubk Will you be able to implement the requested change? Otherwise, it's fine if you don't have time, but would it be possible for you to give me (temporarily) permissions to write on your fork? Then I can push the fix myself.

Hey @crodriguezvega  @colin-axner, I reverted the code and e2e-test failed due to useNewGenesisCommand = false . Can you take a look and give us solution. Thanks

We need to add e2e's as required tests sweat_smile Sorry for the inconveince @hieuvubk! Can you reopen your pr? We think the issue is that the new SDK genesis command is formatted differently. Instead of:
simd add-genesis-account it is simd genesis add-account
So our e2e's need to understand which to use based on the version it is testing

Yes, I thought so too


WIP feature/52_Support_Gzip_and_Deflate

Issue
#52
Details
Brief summary of PR purpose and code changes.
CheckList

[ ] PR starts with [#ISSUE_ID].
[ ] Has been tested (where required) before merge to main.

Hey @JayHelton are you looking to continue with this, or shall I close the PR for now?

I closed the PR. I am still interested, however I started a new job and have been hyper focused on that. I will check back in when things are sane for me and see how things look to continue this

No worries! Congrats and good luck with the new gig 😄


how redirect page

``public function checksession(){
$userId = session()->has('name');
if (!$userId) {
return redirect()->to(base_url());
}
}
public function index()
{
		$this->checksession();
		echo view('Header');
		echo view('Dashboard/Home');
		echo view('Dashboard/footer');

}

$this->load->helper('url');
redirect('somewhere');

u should read the docs.

I think the question is for CI4, this is CI3, by looking at the code difference,

Please ask for help on the forums.
The issues here are for bug tracking.


Frequency Problem

Hello! The RGB image of my camera can only be sent out at a maximum frequency of 20hz, if I acquire data and control the robotic arm at 50hz, will the lack of image acquisition frequency have a greater impact?

Hi, has it been resolved?


.ci: Remove Travis and add unit testing and coveralls to Jenkins

We want to rely only on Jenkins, that's why we need it to be able to
run unit testing and coveralls code coverage.
Fixes #124

lgtm

expecting travis to fail here now - note, we need to remove the travis hooks from this repo.
semi-force-merging.


Fix ipip test, skipping checking PANIC error messages in the log.

Felix logs a PANIC message if ipip config changes. Fix ipip test, skipping checking PANIC error messages in the log.

Should fix this the change introduced here: https://github.com/projectcalico/felix/commit/c9f3a5eeef311a2c051ca09da3badded4142632b#diff-f3d61126e079203a26f75ba0e4ce5da5R670
Looks fine to me. Assigning to @lwr20 to review.

@song-jiang I don't think we should be ignoring all panics.  I put up a PR here #1393 that should ignore the specific log

This PR will be closed because the fix #1393 is the right way to skipping error messages. We should add support for per test LOG_IGNORE_LIST.


Configuration Avoidance: publications.register(…) Configures Immediately

Expected Behavior
According to the docs, the publication registered with publications.register(…) “will be created and configured when it is required.” So, when a call to this method returns, the configuration for the publication shouldn’t have run, yet.
Current Behavior
Objects registered with publications.register(…) are created and configured immediately.
Context
In a Gradle plugin, I’m trying to create a publication which uses a revision that can be configured via a custom Gradle extension. As the plugin is applied at the beginning of the build, there’s no chance at that time that the revision was already configured via the extension. It’d thus be useful to avoid configuring the publication until it’s really needed and when the revision property on the extension has been set.
Steps to Reproduce
When I ./gradlew publish in the following sample project, the published revision is 42 instead of the expected 123:
plugins {
    id 'java'
    id 'ivy-publish'
}

group = 'com.example'

def myRevision = objects.property(String).convention('42')

publishing {
    repositories {
        ivy {
            url = 'file:///tmp/myRepo'
        }
    }
    publications.register('myPublicationName', IvyPublication) {
        revision = myRevision.get()
        from components.java
    }
}

myRevision.set('123')

Your Environment

tested with Gradle 6.8.2/7.4.2 (on OpenJDK 11)
see also #9381

Still relevant


Contact Us link not working

Contact Us link not working on Pet News page want to improve this.
Please assign this issue.

I am from GSSoC'23.
I would like to be assigned this task.


License

Interested in adding a license like ASN.1 Editor?

This is SSLVerifier project. But I will add the license.

License is added. All my public and open source projects are licensed under MS-PL license.


ggplot hist draft fix

Describe your changes
Fix vertical color breaks in histogram
Issue number
Closes #702
Checklist before requesting a review

[ ] Performed a self-review of my code
[ ] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[ ] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://jupysql--706.org.readthedocs.build/en/706/

This draft PR isn't ready for review yet. I'm adding this comment if you happened to check this draft PR. I will add this part again when I make an actual PR.
After doing some digging, I found out the reason for the bug. It's because the distance between consecutive bin starting points isn't the same after querying data.

As you can see from the result of a query inside plot.histogram function, the bin starting points aren't evenly apart. For this query especially, the last two bins' distance is twice the others.

Thus, when using bins[-1] - bins[-2] to get the width of each bin, we get the weird vertical color breaks.
Currently, this issue is solved for the notebook you provided, and I'm working on how the behavior changes on different histograms in the test files.


Quoting not passed to subcommand

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
❯ pixi run python -c 'import sys; print(sys.argv)' '*foo*'
glob: no matches found '.../*foo*'

Issue description
Quoting is not passed to subcommand
Expected behavior
Quoting should be passed to subcommand

Since Pixi uses Deno's task shell I tried it in Deno:
❯ deno eval 'console.log(Deno.args)' xxx '*foo*'
[ "xxx", "*foo*" ]

Also in Bun:
❯ bun --print 'process.argv' xxx '*foo*'
[ "/opt/homebrew/Cellar/bun/1.1.12/bin/bun", "/private/tmp/[eval]", "*foo*" ]

IMO the arguments passed after the task name should not be evaluated in the Deno shell, but passed literally

bump!


Uncaught TypeError: Cannot read property 'path' of undefined

[Enter steps to reproduce below:]

...
...

Atom Version: 1.7.4
System: Mac OS X 10.11.4
Thrown From: atom-lupa package, v0.5.48
Stack Trace
Uncaught TypeError: Cannot read property 'path' of undefined
At /Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/ReactErrorUtils.js:52

TypeError: Cannot read property 'path' of undefined
    at /Users/camiloazula/.atom/packages/atom-lupa/lib/lupaMiddleware.js:31:60
    at /Users/camiloazula/.atom/packages/atom-lupa/lib/middleware/domMiddleware.js:22:9
    at Object.dispatch (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/redux-thunk/lib/index.js:14:16)
    at _react2.default.createElement._react2.default.createElement.onClick (/Users/camiloazula/.atom/packages/atom-lupa/lib/components/Lupa.js:253:34)
    at Object.invokeGuardedCallback (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/ReactErrorUtils.js:26:12)
    at executeDispatch (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/EventPluginUtils.js:87:21)
    at Object.executeDispatchesInOrder (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/EventPluginUtils.js:110:5)
    at executeDispatchesAndRelease (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/EventPluginHub.js:42:22)
    at executeDispatchesAndReleaseTopLevel (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/EventPluginHub.js:53:10)
    at Array.forEach (native)
    at forEachAccumulated (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/forEachAccumulated.js:24:9)
    at Object.EventPluginHub.processEventQueue (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/EventPluginHub.js:215:7)
    at runEventQueueInBatch (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/ReactEventEmitterMixin.js:18:18)
    at Object.ReactEventEmitterMixin.handleTopLevel [as _handleTopLevel] (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/ReactEventEmitterMixin.js:29:5)
    at handleTopLevelImpl (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/ReactEventListener.js:73:24)
    at ReactDefaultBatchingStrategyTransaction.Mixin.perform (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/Transaction.js:136:20)
    at Object.ReactDefaultBatchingStrategy.batchedUpdates (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/ReactDefaultBatchingStrategy.js:63:19)
    at Object.batchedUpdates (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/ReactUpdates.js:98:20)
    at ReactEventListener.dispatchEvent (/Users/camiloazula/.atom/packages/atom-lupa/node_modules/react/lib/ReactEventListener.js:150:20)

Commands
     -0:20.2.0 atom-lupa:toggle (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -0:11 tree-view:toggle (input.lupa-search.native-key-bindings)

Config
{}

Installed Packages
# User
atom-lupa, v0.5.48
atom-ternjs, v0.14.1
autocomplete-modules, v1.4.1
autocomplete-paths, v1.0.2
language-babel, v2.23.2
language-postcss, v1.2.0
linter, v1.11.4
linter-eslint, v7.2.2
linter-jsonlint, v1.2.5
linter-stylelint, v2.10.0
local-history, v3.3.0
merge-conflicts, v1.4.3
standard-formatter, v2.0.0

# Dev
No dev packages

Have you had focused on some file before activating Lupa? Because it was bug which triggered when user was for example in Settings and then triggered Index Project (related to: https://github.com/hex13/atom-lupa/issues/14 )
Bug was fixed but it's sort of reggresion due to refactoring ( need for test: #19 )
TypeError: Cannot read property 'path' of undefined     at /Users/camiloazula/.atom/packages/atom-lupa/lib/lupaMiddleware.js:31:60
(Unfortunately Lupa in current version needs user to open some file before indexing. I know this is a little cumbersome. In future versions I will work to improve that. But for now this is how it works.
But nevertheless it shouldn't break or catch exceptions, so I've fixed for now this exception.

Commit autoclosed this but I've reopened this just in case. Maybe you would like to give some feedback or something.


Please add toggle left & right shoulder camera function.

There is a button in many games to toggle shoulder, e.g. GTA V, Starwars Battlefront II, Call of Duty, Alan Wake, Max Payne 3
Is it possible to modify camera offset to toggle shoulder camera in RE4?

Not planning on this.


Nested task completion counter

Nested items will now be created using a TodoItemGroup which listens to when subtasks are completed, allowing it to keep track of whether all of them have been completed, in which case, it too is considered completed.
Screenshot showing the completion counter for nested tasks when they have all been completed:

Sidenotes:

Fixed some minor bugs in TodoItem
Made TodoList#renderItems also create the surrounding <ul>...</ul>
Consolidated all the CSS for the Todo page into one todo.css file (I don't think we're going to have that much CSS)
Converted camelCase naming of classes to kebab-case

Before I merge, do you think we should rename TodoItem and items, etc, to TodoTask and tasks or do you think it's fine?

Before it's merged, do you think we should rename TodoItem and items, etc, to TodoTask and tasks or do you think it's fine?
I think its fine


Split "Adding Providers" in the MTV user guide into source providers and destination providers

MTV 2.2
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2050724
Splits the "Adding Providers" in Installing and using the Migration Toolkit for Virtualization into two subsections, Adding source providers and Adding destination providers.
Preview: https://deploy-preview-292--forklift-documentation.netlify.app/documentation/doc-migration_toolkit_for_virtualization/master/#adding-providers

@istein1 -- I made the change you requested. Please review the PR again.

@RichardHoch Does this apply ONLY to 2.2? Or also to 2.3?

@stoobie 2.2+


How do you deploy this on Fly.io?

Is it possible to deploy the "BLIP/base" Image classification model on a Fly.io free tier? or put in other words, can you provide a working example on how to deploy a model using Bumblebee on Fly.io?
What did I do?
I copied the Dockerfile from LiveBeats. I imagine it implements  the "cached from HF", meaning a local cache with the env var BUMBLEBEE_CACHE_DIR set.
However, when I run this line - from the LiveBeat Dockerfile - in the builder stage:
RUN mix run -e 'MyApp.Application.serve()' --no-start

it fails and I get:
[output clipped, log limit 100KiB/s reached]

I found another command, RUN /app/bin/my_app eval 'MyApp.Application.serve' in a Fly.io blog, but it fails ("can't find /app/bin/my_app" ???).

I am not sure if this is a Fly.io limitation or if it means something else.
So ok, don't do this: I omit running "Application.serve()" (see the Application module below).  I have git and curl installed in the builder. It seems that indeed my image inflated, reaching 575Mb (+ Git for uploading the model as it seems I can't preload and cache it).
However, it seems that: [info]Out of memory: Killed process
I can't run this.
defmodule MyApp.Application do
  use Application

def serve do
    model = "Salesforce/blip-image-captioning-base"
    {:ok, model_info} = Bumblebee.load_model({:hf, model})
    {:ok, featurizer} =
      Bumblebee.load_featurizer({:hf, model})
    {:ok, tokenizer} =
      Bumblebee.load_tokenizer({:hf, model})
    {:ok, generation_config} =
      Bumblebee.load_generation_config({:hf, model})
    generation_config = Bumblebee.configure(generation_config, max_new_tokens: 100)

    _serving =
      Bumblebee.Vision.image_to_text(model_info, featurizer, tokenizer, generation_config,
        compile: [batch_size: 1],
        defn_options: [compiler: EXLA],
        preallocate_params: true
      )
  end

def start(_type, _args) do
    children = [
      {Nx.Serving, serving: serve(), name: MyApp.Serving}
   ]
   opts = [strategy: :one_for_one, name: MyApp.Supervisor]
    Supervisor.start_link(children, opts)
  end
end

Hi @ndrean. Please use the ElixirForum for general questions/help. If you can share your Dockerfile and app in a GitHub repository, it will be easier.
About the log, does it show anything as it progresses? Maybe it is a download bar or something else posting too much information?

Other things to try out: try saving the model to disk and loading directly from disk.
In any case, if the fly instance has 256MB of memory, I doubt you will be able to run the model. If the model is 990MB compressed, I expect it will require more than when loaded into memory.

@ndrean can't find /app/bin/my_app sounds like an issue with your Dockerfile.
Building with cache should be fine, but the model is definitely to big for the free instance. You can look at this HuggingFace space for an example LV app deployment with Bumblebee stuff. There is a Dockerfile that also downloads the model at build time.

Sorry for polluting, indeed...
@jonatanklosko
Thks for the link for this dockerfile.
It uses a GPU I guess with EXLA->CUDA. Then the build stage does:
# Start the app to download models into the cache
RUN SECRET_KEY_BASE="$(mix phx.gen.secret)" mix app.start

Whilst LiveBeats with Whispers does as below, and my Dockerfile is just generated by Fly, so just a copy.
RUN mix run -e 'LiveBeats.Application.load_serving()' --no-start

I will observe the memory used.

Yeah, I did mix app.start because that requires no additional code and in that case starting the app works fine (there is no database to connect to, or anything like that in the supervision tree). Using mix run with a custom function does not start the app, which is probably preferable in the general case.


update needed.

latest update have added "frames when empty" and a ton more settings both for host and game settings.
don't know how either but the discord webhook stopped working, just now. was working earlier today when the game got an update.
an option for message when the server status is online and joinable. could be when the serverlog sends this message "Shutting down Asynchronous Streaming" i havn't see it anywere els. could be wrong. but you know what i mean :D
PS. Nice work, keep it up :D

Thank you!
I am about 75% done with a semi-big update, I'll have a look at all this as well. The webhook issue sounds strange but I might have an idea.
Should hopefully be done with the update this week.

oh. found out they get SUPER delayed. shut down server around 01:51 and a message came up around 04:50.
started the server this morning around 09:21 and havn't still got anything on the Discord.
before i got message almost instant, but after the update and update in 5 min message, it got super delayed. they came 1/6-23 11:50, don't know if it's because they both got send at the same time?

If you are still running a server, could you see if the latest release has this issue? Thanks. 😊

after a day or two the issue stopped, and messages came through as soon i pressed the button. so may have been a discord server issue.
testing new release now :D

Thank you.
I tried making it as easy as possible to use. If you have the time, please let me know what you think in comparison to the old version. If you are migrating your server to the new version, don't forget to create backups.
The remove server is currently bugged but I'll release a new version tomorrow fixing it.

when i try to update a server with any other name in folder it sends
Failed to load script file 'C:\V-Rising' (real server path--> 'C:\V-Rising Server') might not be able to read (space) might just need som "serverpath" quotes in code.
when failing updating server due to space in server path, it comes with error code, and is just stuck on updating. people with no logic to look in steamcmd window, is lost and will just wait for update, maybe even uninstall again.
could remove the server tab without issue. no idéa what is bugged?
no webhook edit messages, i need those to give the messages alittle spice. also to remove the ip.
should load game and host settings in server folder save. C:*\SaveData\Settings
so maybe option to save elswere, or just press save and it saves in that location.
bigger "hitbox" for settings button, where you have to hit is very small :D
RCON console is not available. it's activated in settings.
maybe add the "Add server" button up right at the server fan list. together with VSM settings makes more sense up there.
option for different webhook messages for each server. would be very handy.
thats everything my eye hit right away :D
look awesome with new overlay. maybe add an darkmode (love that)

Big thanks!
I will have a look at all this tomorrow. RCON is temporarily disabled but should be back soon. I'll re-add the discord messages again, it's already planned, and per server is a good idea. The "Add Server" thing is hard to implement, I've been thinking about it but haven't found a way to do it "elegantly".
Only thing I'm wondering about is the Settings button, do you mean VSM Settings? The hitbox should act normal so it sounds a bit weird.
Thank you again!

no the hitboxes in host settings and game settings files.
and elegant maybe like this, and the +moves with server fans :D

Ah, I understand!
If you mean the ones with digits specifically (with up and down arrows on the side) they are meant to be replaced in the future, I did notice myself that they were frustrating to click on. I haven't gotten that far yet.
I'll see what I can do with the main UI in regards to your image. Thank you again for taking the time, I appreciate the feedback. 😊

no problem 😄  it's nice to have a little "touch" on the work 😄  especially when i use it my self 😄 not much of a coder my self tho, only basic understanding 😅

I've fixed almost everything above now.

Path can now contains spaces
Dark mode works
Game and Save Settings Editors now asks if you wanna save to a server and will automatically create it at the right place
All webhook messages are now customizable
RCON has been added again (a bit ugly, will improve over time)
Hitboxes fixed, all number boxes now use another control which looks and "acts" better

To do:

Add "New Server" button on the tabcontrol (might push the release before I can add this, haven't found a good method)

Need a bit more time then I should be able to push the release. Sorry for the wait.

I pushed the new release now, you can download it here or use the auto-updater. Let me know if it's like you wanted. Still trying to figure out where to put "Settings" and the "New" button for servers.

awesome work :D love the result :D could you add auto load of host and game settings? and maybe make it depend on what server fan you are on? if not on the fan it's fine with the current dropdown menu when you save :D but at least so i don't have to look after the files them self.

Of course. I'll make it a setting that you toggle, if enabled it will autoload the selected server when you open the editor (if it exists). I'll have a look at it this weekend. Let me know if you have any other suggestions or find any bugs.

Auto loading has been added in the latest release today.

so nice to see :D hard to see now how it would be any better :D maybe option for modinstall and auto update on that part.
only thingis, only dev builds of mods is available atm, cause the gloomrot update smashed the mods.
maybe if you are up for it, create a mod that will pull the player count, maybe even playerlist. so you can see if the server is empty or have players onlne :D
somthing completly diffrent, but hey if you have the time and want to explore i would say go you it :D

I've made some progress and should be able to release a new version with mod management being included. Hopefully it'll be done by this weekend. 👍

looks awesome. but just encounted a bug. when the server auto updates it closes the server but the auto restart starts the server while updating the server, making the application come to either alot of booting messages and fail, somtimes even crash.
the auto restart server needs to have a 5-10 sec delay, check if steamCMD is running and still updating, then wait.

Hmm, I've thoroughly tested that and it shouldn't happen. I'll have a look at it.

my server does somthing els XD

the one from 16:08 is manuel reboot

I found the error, I will add it to the next release.

The new version is getting flagged by anti viruses again, haven't had this happen since last year. I'll have to revert the latest version and try to fix it...

Should be uploaded now, let me know if it works. I think the mod manager is triggering some false detections.

Looks good. If you could implement the tags from the store, that would be really helpful, cause many plugins is outdated, and some is only for clients and some for only servers. Newst plugins is tagged with gloomrot update tag, but i have seen some updated without the tag. Not easy to go around.

woop :D new feedback :D leave an check off for webhook message per server. having a test server running, don't need messages to be sent, and have custom messages set and ready for when i need them. so deleting them would be a hassel :D

Sure, I'll add that. :)

I'm gonna close this issue because of its length, if you find any bugs or new feature requests please let me know in a new one. And thank you again for all the support.


Move the footnote about using v-for with v-if to list.md

The footnote about using v-for with v-if shouldn't be present in conditional rendering docs, because list rendering has not been introduced yet.
Also, v-for with v-if is also described in the list rendering docs, which can mislead people.
It would be better if the whole information about it was present in the list rendering docs.

Thanks for the PR! However, I disagree with this change. This is a really important caveat and it should be introduced as soon as possible in the docs as it does create confusion. Furthermore, there's some detail missing in your refactor. Appreciated, though!


Change AudioTrack when onload Event fired

hi, i have a question
we have anyway to change audio track when onload event fired?
example:
check the available audio tracks on .m3u8 source and then decide to select which one or show a list of available tracks to user with interface and play user's choice

you should be able to set audio track value before starting the playback, but you need to know what is the 'prefered' audio track for user ...


Setting the RegEx based on the list of URLs

Currently it is necessary to change a part of the crawler code to properly compile a regex that captures the crawled web. This should be generalized.
We should construct a regex autonomously based on the seed url (maybe with the assumption for now that it is the homepage).

[ ] strip the path from the URL
[ ] transcribe base URL to RE
[ ] raise error for malformed URLs


Fixed Typo error in CONTRIBUTING.md

Typo error
Hello sorry for not following the bug report template  #I think this PR doesn't need that as it is a typo error I have fixed some typo's in the following lines
Fixes
##CONTRIBUTING.md

Line 166: Removed the and added that as the doesn't mean anything.

#42816

This was merged


Selenide could take a snapshot of page, allowing fast checks without calling webdriver

The problem
We want to speed up our tests.
Sometimes we know for sure that a page has no animation. It makes sence to save page HTML once, and then do following checks using this HTML, without calling webdriver every time.
Details
Some possible libraries that can parse and analyze HTML:

https://github.com/jOOQ/jOOX
https://jsoup.org/

Sounds interesting.
I believe, that a good idea is to save not whole page, but selected element (or element collection).
But I need a name for method, because, unfortunately, .snapshot() is used for collections already.
Is it possible to rename .snapshot() to .cache() for collections?
Or maybe add ability to provide some options to the .snapshot function?
:thinking:

Why not the whole page?
Anyway, the name snapshot seems a perfect name. Yes, currently collections already have method $$. snapshot(), but it's intention is exactly the same: the speed! And we will make it even faster.


Cut 2.2.0 release

To complete the 2.2.0 release, the following tasks must be completed:
#1449
#1462
#1464
apache/couchdb-fauxton#1104
apache/couchdb-fauxton#1105
apache/couchdb-fauxton#1106
apache/couchdb-fauxton#1108
apache/couchdb-documentation#292
apache/couchdb-documentation#268
and unfortunately not these: :
#1407 #wontfix
apache/couchdb-documentation#275 needs too much work
This ticket serves to track the completion of these across multiple repos, then to track the RC process, for those who are not subscribed to the<EMAIL_ADDRESS>mailing list.

We're down to a Fauxton tagged release and the release notes. Woo hoo!
Don't forget to check all the versions in master's rebar.config.script, @wohali.

We have a tagged Fauxton release, so #1464 has been added to the milestone. Once the release notes and #1462 are resolved one way or the other, I can tag couchdb-documentation as 2.2.0, bump rebar.config.script and cut our first 2.2.0 release candidate! :D

@janl you asked about my process for creating release notes - I wrote it up - maybe this will help you:

Get a list of things to care about:
git show-branch --sha1-name master X.X.X|more

use this to find common ancestor (very last thing in the output)
use this to make sure everything on X.X.X has been pulled fwd to master
use this to eliminate anything that "looks the same" on X.X.X branch and master

cannot use sha-1s because they will be different on each branch
research if there is a way to auto-validate cherry picked stuff?

For each remaining commit on master:

use GH web UI to find the commit's related PRs or issues and write up
something nice
sort by GH issue/PR number
any issues that merge without a useful GH/PR number should go either at the
very top or bottom of the list, depending on importance of issue
do not explicitly list out things like Jenkins/Travis/rebar.config changes
unless they are significant in some fashion

Now do the same thing as above, but for documentation and fauxton. Pull out only
the big-ticket items.

Ready to go - just bumping the docs dependency now, then I'll make the 2.2.x branch.

done

Dear community,
I would like to release Apache CouchDB 2.2.0-RC1.
Changes since last round:

https://github.com/apache/couchdb/compare/2.1.2...2.2.0-RC1

Release notes:

http://docs.couchdb.org/en/latest/whatsnew/2.2.html

This is a big release, so your attention to detail is very much appreciated.
We encourage the whole community to download and test these release artefacts so that any critical issues can be resolved before the release is made. Everyone is free to vote on this release, so dig right in!
The release artefacts we are voting on are available here:
https://dist.apache.org/repos/dist/dev/couchdb/source/2.2.0/rc.1/

There, you will find a tarball, a GPG signature, and SHA256/SHA512 checksums.
Please follow the test procedure here:
https://cwiki.apache.org/confluence/display/COUCHDB/Testing+a+Source+Release

Please remember that "RC1" is an annotation. If the vote passes, these artefacts will be released as Apache CouchDB 2.2.0.
Please cast your votes now.
Thanks,
Joan "release all the things!" Touzet
Apache CouchDB PMC

binaries are up for windows rc1
https://dist.apache.org/repos/dist/dev/couchdb/binary/win/2.2.0/rc.1/

and mac: https://dist.apache.org/repos/dist/dev/couchdb/binary/mac/2.2.0/rc.1/


Zuul default Rest Client seems not to work correctly under OpenShift

I’ve tried to deploy under OpenShift Container Platform a project in order to test Zuul under OpenShift.
Code can be found here: https://github.com/aleMessoMale/zuul-poc.git
It only tries to call a rest service exposed by another microservice (J2EE Spring Boot) whose code can be found here: https://github.com/aleMessoMale/gs-rest-service.git
Exposed REST API (/greeting) can be called, by calling the /test api of the deployed Zuul Service:
zuul:
routes:
users:
path: /test/**
#write here the route of the called Rest API, GET /greeting is the URL.
url: http://example.com/greeting
Despite of, connected in ssh in the pod, the Zuul service container was able to reach the /greeting API, once called through Zuul it seems not be able to resolve the url.
Please find error log in attachment.
Same url, with Unirest client was instead callable.
I’ve created a Filter – UniRestSimpleHostRoutingFilter which used behind a Unirest Client in order to make the call and in this way call is correctly forwared.
There’s a way in order to make it works without defining a new Filter, maybe some properties that by default make it not works correctly?
Thank you,
zuul-error.log

All I can say is that this URL http://greeting-github-jenkins.dxl-pre.example.com/greeting is not reachable from Zuul.  As to why that is the case, I can't really say based on the information you have provided.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Fix routing_missing_exception on delete with parent missing

Don't break when we're trying to delete a child record which either doesn't have a parent or doesn't exist at all. In our context, these would be course details of deactivated courses. Original commit.

:+1:


Video controls float in scrollview (Android)

Hey,
Issue

Occurs in all android versions and devices we've checked.
Reproduce
 <StackLayout>
        <ScrollView>
            <StackLayout>

                <VideoPlayer:Video id="nativeVideoPlayer"
                                   controls="true"
                                   loop="true" autoplay="true" height="280"
                                   src="https://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4" />

            </StackLayout>
        </ScrollView>
</StackLayout>

We use this video plugin in production, so we forked this repo and hacked our way around it. We accomplished it by exposing "hideMediaControls" and calling it on scroll, gross.
Would be nice if the controls stick to the video :)
Alternatives
Exoplayer
We've tried this and we faced some issues:

1.5mb extra on Android (not really an issue but we rather avoid the additional mb's)
Suspend/Resume breaks the video on lower Android API's. We're not sure if this is an Exoplayer issue or NativeScript-Exoplayer issue. We see a "less than API 23" check in the Exoplayer sdk that might be the culprit as it might conflict with the NativeScript implementation.

Logged issue:

can you show code sample of how you solved that?


Bluetooth not disabling when done via Control Center menu (iOS)

iPhone XR
Bluetooth settings is ON
Open Control Center, disable Bluetooth
return to the app, open menu, no Bluetooth warning message appears.
Expected behaviour is that disabling Bluetooth via control Center should trigger the message in the app.

Message can be seen in this screenshot:

This was triggered by turning Bluetooth off via the Settings.

this is as designed.
When you disable the Bluetooth via  the Control Center  and go in settings you will see that Bluetooth is still on, the Control Center only disconnects all Bluetooth devices, it actually doesn't shut off Bluetooth. So its not a bug


Filter trigger

Version
2.3
Reproduction link
https://jsbin.com/dawulukuji/1/edit?html,js,output
Steps to reproduce
Let's create a list of 5 elements. Apply to them a filter that will substitute the time of creation of this element. Let's add one more element.
What is expected?
The filter function must be called for the last element added.
What is actually happening?
The filter function is applied to all elements

This is not a bug, but expected behaviour.


Unicode handling

Error probably caused from using emotes in chat
File "main.py", line 24, in handle_msg
    print '<'+msg[...]
UnicodeEncodeError: 'cp932' codec can't encode character u'?xf8' in position x: illegal multibyte sequence

should be fixed with #2


compilation error in utils

Required information
Operating system:
Ubuntu 16.04.6 LTS
(as per ubiquity robotics image compiled on raspberryPi 3b)
Compiler version:
GCC 7.4.0 (for arm ->raspi)
Observed result or behaviour:
allocator.cpp:44:75: error: no matching function for call to ‘align(uintptr_t&, const uint64_t&)’
uintptr_t l_alignedPosition = cxx::align(l_currentAddress, f_alignment);
--> template class has only a shared parameter type for both parameters of cxx::align. Add a second template type or make second parameter (alignment) a static type e.g. uint64_t.
Expected result or behaviour:
Does compile.
Conditions where it occurred / Performed steps:
Using ubiquity robotics image compiled on raspberryPi 3b
additional to install instruction added GCC v7 as default.
(Compilation did finish after changing cxx::align 2nd parameter type (in helplets.hpp) to static type uint64_t.)

Thanks @Jekyll555 . Funny that this is no problem with gcc 7.4 on Intel. We also have ARM targets in our build pipeline but maybe not the combination ARM and gcc 7.4

Hi @Jekyll555 i can confirm this issue when i try to compile iceoryx on Raspbian with a Raspberry Pi 3B.
The reason for that is that Raspbian is based on 32-Bit but the hardware processor has 64-bit. The 32-bit GCC compiler can then not handle the alignment for the different datatypes.
I guess that the ubiquity robotics image is also based on 32-Bit. You can check that with uname -a on which platform the operating system is based. If it is "armv7" then it is 32 Bit.
I created already an internal proposal how to fix that issue. I need to review it with my colleague and would then create a Pull-Request with you as reviewer.

HI @dkroenke Confirming the raspbian being 32bit image:
Linux ubiquityrobot 4.14.98-v7+ #1200 SMP Tue Feb 12 20:27:48 GMT 2019 armv7l armv7l armv7l GNU/Linux
Go ahead, I will be available for e.g. retesting...

@elfenpiff, can this issue be closed?

Hi,
please wait until @Jekyll555 can confirm that he can compile it for his setup before you close this issue.

@elBoberido from my perspective we could close this issue

@dkroenke @elfenpiff Don't bother me.
But: confirm 844b316aa55b57ab92c8e1fd4b7c4a9423014850 solved  it on my 32bit-ish raspbian image. Nice work, Thanks!
(I take the freedom to close the issue now. ;-)


Add support of Github App Installation authentication

Closes #16

I guess the functionality related to this PR has to be re-built significantly, considering large update of supporting several configs


changes to feed of poster

if I open a thread the background feed changes to that of the poster, then I have to refresh two time to get back to my feed, but all of the tweets have changed.

I think this is posted elsewhere. IF you sign out - and then sign back in to Twitter, it should take care of the issue. I don't know how long the fix lasts. But it's worked for many people.

I think this is posted elsewhere. IF you sign out - and then sign back in to Twitter, it should take care of the issue. I don't know how long the fix lasts. But it's worked for many people.

Yes, worked for me. If you can't find the logout link, just add /logout to https://twitter.com/  i.e. https://twitter.com/logout


Core Data subscribe to Events from MessageBus

Per MessageBus ADR

Add Subscribe config to MessageQueue config

SubscribeEnabled
SubscribeTopic

Add subscribing logic to bootstrapping MessageBus client
Implement Event received via MessageBus

Validate DTO
Persist Event

This is a stretch goal for Ireland so is on hold until rest of ADR is implemented

@cloudxxx8 , @weichou1229 I have put this back on hold until I complete the Secure MessageBus changes since this will be dependent on those chnages.

@lenny-intel Got it, thank you!