Active Power Setpoints in DID704 - Wrong Data Type?

The active power setpoints are specified as int32. Shouldn't they be int16?

WSet
WSetRvrt
WSetPct
WSetPctRvrt

Actually, let's extend that to the other setpoints in 704 as well. Do we really need 32 bits for VarSet, etc? Doesn't seem like we do.

Resolved.


[$500] Web - IOU - I can't navigate using the "Tab" key

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open FAB > Request money
Select manual, enter a value and click "Next"
Select a workspace chat
Press the "Tab" key

Expected Result:
I should be able to use it
Actual Result:
I can't navigate using the "Tab" key. It works in task request flow
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/535b5b04-4e69-4387-bca4-b81707f25393
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0144e80c4837a63c84
Upwork Job ID:<PHONE_NUMBER>532207616
Last Price Increase: 2024-02-14

We think that this bug might be related to #vip-vsp
CC @quinthar

Proposal
Please re-state the problem that we are trying to solve in this issue.
Can't navigate using the TAB key on the money request page.
What is the root cause of that problem?
In https://github.com/Expensify/App/pull/35154, we intentionally disable the TAB to "fix" https://github.com/Expensify/App/issues/26326.
https://github.com/Expensify/App/blob/6914b1344ed1f8bd69037232a67e34ffea2581e4/src/pages/iou/request/IOURequestStartPage.js#L91-L102
But this affected all money request page, including participant and confirmation page.
What changes do you think we should make in order to solve the problem?

We should unregister the tab listener when the screen is out focus. We can use useFocusEffect to achieve it.
useFocusEffect(
    useCallback(() => {
        const handler = (event) => {
            if (event.code !== CONST.KEYBOARD_SHORTCUTS.TAB.shortcutKey) {
                return;
            }
            event.preventDefault();
            event.stopPropagation();
        };
        KeyDownPressListener.addKeyDownPressListener(handler);

        return () => KeyDownPressListener.removeKeyDownPressListener(handler);
}, []));

What alternative solutions did you explore? (Optional)
This isn't the best solution, but just in case. We can store the navigation focus state to a ref and return early if the page is not focused anymore.

@bernhardoj
Just to confirm, does your solution keep https://github.com/Expensify/App/issues/26326 fixed?

Proposal
Please re-state the problem that we are trying to solve in this issue.
Can not navigate by tab key in request money page.
What is the root cause of that problem?
Since #35154 to fix #26326
https://github.com/Expensify/App/blob/6914b1344ed1f8bd69037232a67e34ffea2581e4/src/pages/iou/request/IOURequestStartPage.js#L91-L102
This change disable all tab actions in request money page (which wrapped by IOURequestStartPage component)
What changes do you think we should make in order to solve the problem?

Revert above code to get back normal behaviors.
Set css attribute: visibility: hidden to not-active TopTab.Screen, all focusable child of that element will not tab(able) anymore.

<TopTab.Screen name={CONST.TAB_REQUEST.MANUAL}>
  {({navigation}) => {
    const state = navigation.getState();
    if (state?.index === 0) return <IOURequestStepAmount route={route} />;

    return (
        <View style={visibility.hidden}>
           <IOURequestStepAmount route={route} />;
         </View>
     );
}}

What alternative solutions did you explore? (Optional)
Above solution need to change all TopTab.Screen implementation. I think about more generic solution which make changes in OnyxTabNavigator's children:
{React.Children.map(children, (child, i) => {
  if (!React.isValidElement(child) || child.type !== TopTab.Screen || !isFunction(child.props.children)) {
    return child;
  }

  return React.cloneElement(child, {
    children: (props) => {
      const state = props?.navigation?.getState();
      if (state.index === i) {
        return child.props.children(props);
      }
      return <View style={visibility.hidden}>{child.props.children(props)}</View>;
    },
  });
})}

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~012cb272a7b7771f06

@cubuspl42 yes

@damadoko I'm confused by the behavior of the discussed tab component (TopTab.Navigator). Is the content of non-active tabs actually rendered to the DOM tree?

@cubuspl42 yes, it actually render non-active tabs to the DOM tree as  https://github.com/Expensify/App/issues/26326#issuecomment-1700290699 mentioned. This bug only happens with TopTab.Navigator.
The original https://github.com/Expensify/App/issues/26326 included 2 different issues:

When press tab, focus is moved, the whole tab navigator elements move too.
When press tab, focus can move out of RHP. (Seem like we try to address it here https://github.com/Expensify/App/issues/36476)

I decided to approve the proposal by @bernhardoj.
Both proposals suggested to keep the useEffect block introduced by https://github.com/Expensify/App/pull/35154. I believe it's cleaner to improve its behavior than to introduce another solution for dealing with the "Tab" key (namely CSS visibility property).
C+ reviewed 🎀 👀 🎀

PR is ready
cc: @cubuspl42

Update: The PR is waiting for @mountiny's review.

Awesome, looks like automation kicked in here.

Heads up, I will be offline until Tuesday, March 5, 2024. I will not be actively watching over this GitHub during that period. If anything urgent is needed here, please ask for help in the Open Source Slack Room-- thanks!

Alright, I've completed the payment summary here:
https://github.com/Expensify/App/issues/36513#issuecomment-1972142123
For this Upwork Job:
https://www.upwork.com/jobs/~0144e80c4837a63c84

Closing - Everyone has been paid via Upwork.


feat(backend): add metric alertmanager_up

If last api call failed, the metric is 0, otherwise 1.
This makes writing alerting rules clearer, rather than rate(karma_alertmanager_errors_total[5m]) or similar.

Thanks!


Apply coupon button is not responsive if we enter any random data in coupon code field, it should throw an error message "Invalid coupon code".

Framework version: 0.1.6

The boolean false returned by applyCouponAbleRule($code) is not a valid return value.

@stefano73it The Discount module is not yet released and we are in mid of the process to comply features that need to be deployed in next release.

Framework Version: 0.1.6

Fixed, now showing "Cannot apply coupon"


Issues/15: Introduce workspaces into the project

:tada: This PR is included in version @joberstein12/commit-history-validator-v1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @joberstein12/commitlint-config-v1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature request: provide a way to view the Code With Us terms

User Stories
As a proponent, I would like to see what the Code With Us terms are before deciding to apply on an opportunity.
As a government administrator, I need a way to send the Code With Us terms to my finance staff so that they have a copy of "the contract" and can set up a purchase order to pay invoices against it.

Hi @mark-a-wilson ,
The Terms and Conditions for both Code With Us and Sprint With Us can be viewed from your User Profile -> Accepted Policies, Terms & Agreements, if you have already accepted them.  You will also be presented with the Terms and Conditions when first signing into your User Profile as a new user.
Would this meeting the requirements for your feature request?

@mark-a-wilson my apologies, it does appear that the CWU specific T&C are missing.  We have an issue in our backlog to address this.

Thanks for the fast reply! ⚡
I was just thinking it would be helpful for a user to able to read the terms while browsing the opportunity.
And for internal gov admin purposes, it's necessary to be able to screen print or download the terms as a document as we have to submit "the contract" to Finance via email w/ attachments, as well as file a copy on a secure LAN for records management requirements.
P.S. I am keeping https://github.com/bcdevexchange/code-with-us up-to-date with just the Terms in isolation. Have just updated that now.
Cheers 🍻

This has been completed and deployed to production. The CWU terms are presented to vendors when they try to submit their CWU proposals (must check a box to accept before submitting). The terms are also published at this URL: https://digital.gov.bc.ca/marketplace/content/code-with-us-terms-and-conditions


Update monitoring-installer.sh. Improve idempotency of script

Add -p flag for mkdir even if 1-level directory created. This ensures script doesn't exit fail fast if directory already exists
Add check if prometheus user already exists. If not script won't try to create one and fail fast because it already exists

Requested changes are not problematic, but the script itself is not intended (nor tested) to be idempotent. As it is, I think it may be better to fail early if the app is already installed than to proceed and possibly mess up the already present installation.
For example, step 3 blindly adds avalanchego and avalanchego-machine jobs to prometheus config file. Running that twice will surely break prometheus.
If you would be willing to test the script and harden it against such situations, that would be great. Even just failing with a correct message would be an improvement.


Segmentation fault creating basic blocks

There's a segmentation fault creating the basic blocks for the method android.support.v4.os.ResultReceiver->writeToParcel(android.os.Parcel,int), probably some strange structure makes the program to crash. The next code represents the smali from the method:
void android.support.v4.os.ResultReceiver->writeToParcel(android.os.Parcel,int)
00000000  1d 00                   monitor-enter v0
00000002  54 02 df 00             iget-object v2, v0, Landroid/support/v4/os/ResultReceiver;->mReceiver Landroid/support/v4/os/IResultReceiver; (223)
00000006  39 02 09 00             if-nez v2, 9
0000000a  22 02 6c 01             new-instance v2, android.support.v4.os.ResultReceiver$MyResultReceiver (364)
0000000e  70 20 e1 07 02 00       invoke-direct {v2, v0}, void android.support.v4.os.ResultReceiver$MyResultReceiver-><init>(android.support.v4.os.ResultReceiver)
00000014  5b 02 df 00             iput-object v2, v0, Landroid/support/v4/os/ResultReceiver;->mReceiver Landroid/support/v4/os/IResultReceiver; (223)
00000018  54 02 df 00             iget-object v2, v0, Landroid/support/v4/os/ResultReceiver;->mReceiver Landroid/support/v4/os/IResultReceiver; (223)
0000001c  72 10 da 07 02 00       invoke-interface {v2}, android.os.IBinder android.support.v4.os.IResultReceiver->asBinder()
00000022  0c 02                   move-result-object v2
00000024  6e 20 40 07 21 00       invoke-virtual {v1, v2}, void android.os.Parcel->writeStrongBinder(android.os.IBinder)
0000002a  1e 00                   monitor-exit v0
0000002c  0e 00                   return-void
0000002e  0d 01                   move-exception v1
00000030  1e 00                   monitor-exit v0
00000032  27 01                   throw v1

Probably generation of basic blocks crashes due to the catch block that surrounds the whole function.

The problem comes from the throw instruction, this instruction is considered as a conditional jump, and when the function determine_next is called, it returns the by default next instruction (the fallthrough), but since the instruction is at the last address of the method, it returns an incorrect address. The current commit is a work around that makes the code not to break: https://github.com/Fare9/KUNAI-static-analyzer/commit/114c8556c199e056472e908814f3722f145bd774, maybe a more beautiful solution would be to remove throw from conditional jumps, since this instruction target is calculated through exceptions.
If you have any comment @sunniAngela just comment here, and maybe we can do a better solution.


Report missing referenced resources

Is your feature request related to a problem? Please describe.
Profile/ClusterProfile can reference ConfigMap/Secret. Because of a type it can happen that Sveltos ends up referencing a non existing ConfigMap/Secret.
Currently Sveltos reports no issues in such situation.
Describe the solution you'd like
When a referenced ConfigMap/Secret does not exist, report it in the Status. So it is easier to debug.
Describe alternatives you've considered
None.
Additional context
None.

@egrosdou01


Screenings Theme breaking after the latest update

Hi Sacha,
I've been playing around with the screenings theme, however after i took the update for telescope (using meteor update) the left nav is breaking, all the links are breaking.
Also, i think you moved the code for screenings and made it to just the theme? - Can you please point me to that repo, so i can keep the workspace in sync. i hope it's still public :+1:
Once again, thank you! i am learning and will love to contribute to the overall telescope project.

Sure, here's the theme: https://github.com/TelescopeJS/Zeiss
Maybe try using that, and then make sure you're running Telescope's devel branch?

Got you! thanks... and for anyone who is looking to know more about the theme, there is a video series.
Closing the issue.
https://www.youtube.com/playlist?list=PLBoa_Q6hVeSzw6cGRaKInzNc3saxWcH4M


How to interpret dNdS ratio from parsed results table?

Hello,
In the output results, there is omega1, omega2, and omega3.
Omega 3 is all NaN in my results. Omega1 ranges from 0 to 1.
How do i calculate a dNdS score for each line in the parsed output table?
Thank you.

Hi,
For a full description of the different omega values, please visit this
paper  https://doi.org/10.1093/molbev/msv022.
Basically, the aBSREL software tests for episodic positive selection at a
significant percentage of sites. It does sound by allowing multiple rates
of evolution at each branch. The omega1-3 in the output table is equivalent
to w1-3 as described in the paper. So if all of your branches in the
phylogenetic alignment have w1 <= 1 it means that all sites tested were
under negative or neutral selection. Omega3 or w3 > 1 means that there were
a significant portion of sites under positive selection.
To get one dN/dS value for a whole branch or for one site over an alignment
you would use different tests from the Hyphy suite. If I remember
correctly, MEME or BUSTED tests for selection over a whole branch. Hope
this helps.
Best,
Taiya Jarva
On Thu, Nov 28, 2024, 9:13 AM Michael Winter @.***>
wrote:

Hello,
In the output results, there is omega1, omega2, and omega3.
Omega 3 is all NaN in my results. Omega1 ranges from 0 to 1.
How do i calculate a dNdS score for each line in the parsed output table?
Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/stupornova33/parse_absREL_json/issues/2, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APAWHXFJLRMITLDQK3XQCJT2C4QJRAVCNFSM6AAAAABSVHPH7SVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YDEMRZGE3DSNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Preparatory GitHub Actions

This runs existing actions and then echoes hello

To validate this pull request against other sbt modules, please visit this link.


Enable DNS Add On

It would be useful for the DNS Add On to work out of the box, especially as some of the examples depend on it.
I really like the project and think it is going to be a really useful tool in helping less ops inclined devs to know "Just Enough" about k8s to be dangerous.
I may well have a quick bash at getting this to work at some point over the weekend.

@errm sure DNS Add on wold be nice to have there. I'm a bit busy with terraforming big k8s cluster setup.
If you have some spare time and can contribute, it would be awesome :)
My small OS X coreos apps are big helpers for me to test things locally.


Fix bug with deletePage() not properly updating pagesContext()

deletePage() was not updating pagesContext[], causing havoc, particularly with documents where the pages were not all the same dimension or orientation.

Here's a demo of the bug: https://risacher.org/deletePageBug.html

Thank you


if picture is too big ,the theme can`t make picture small

in the index,if a picture is too big , the picture will be outof the screen.please fix it.
thx

Can you fix it and provide pull request ?
Or provide screenshot and source photo
thanks

I found a method to fix it. And I am trying to add ''out of ''

Ok let me know how it goes or I will close this. thx

I hope to add some function like duoshuo(comments function in China),the quantity of reading,tagcloud,sticky posts,share function. So many function I need to redevelop because I`m in China.
Please tell me which function has been developed in this theme, I can use directly.

Most are not here so you need to add it yourself.
Please submit a pull request when done so that others can enjoy as well.

On 30 Oct 2016, at 12:51 PM, litttlefisher<EMAIL_ADDRESS>wrote:
I hope to add some function like duoshuo(comments function in China),the quantity of reading,tagcloud,sticky posts,share function. So many function I need to redevelop because I`m in China.
Please tell me which function has been developed in this theme, I can use directly.
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Keep Watir browser running on the backend?

Meta -
Watir Version:
Selenium Version:
Browser Version:
Chrome
Browser Driver Version:
OS Version:
Windows 10
Expected Behavior -
Each time when use .goto or open a browser tab, it jumps on the front end
Actual Behavior -
Expect the browser window keeps running on backed so it will not interrupt other works
Steps to reproduce -

Then use headless browser.
On Tue, 9 Apr, 2019, 1:07 PM yenanqu<EMAIL_ADDRESS>wrote:

Meta -
Watir Version:
Selenium Version:
Browser Version:
Chrome
Browser Driver Version:
OS Version:
Windows 10
Expected Behavior -
Each time when use .goto or open a browser tab, it jumps on the front end
Actual Behavior -
Expect the browser window keeps running on backed so it will not interrupt
other works
Steps to reproduce -
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/watir/watir/issues/862, or mute the thread
https://github.com/notifications/unsubscribe-auth/AeOm7qoj6vJdw19xD1GeaXs4eGR2TPxFks5vfENBgaJpZM4cj0E7
.

Hi Can you please give an example on this?
Thanks

Then use headless browser.
…
On Tue, 9 Apr, 2019, 1:07 PM yenanqu, @.***> wrote: Meta - Watir Version: Selenium Version: Browser Version: Chrome Browser Driver Version: OS Version: Windows 10 Expected Behavior - Each time when use .goto or open a browser tab, it jumps on the front end Actual Behavior - Expect the browser window keeps running on backed so it will not interrupt other works Steps to reproduce - — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#862>, or mute the thread https://github.com/notifications/unsubscribe-auth/AeOm7qoj6vJdw19xD1GeaXs4eGR2TPxFks5vfENBgaJpZM4cj0E7 .

http://watir.com/guides/firefox/#headless
http://watir.com/guides/chrome/#headless
On Tue, 9 Apr, 2019, 1:15 PM yenanqu<EMAIL_ADDRESS>wrote:

Hi Can you please give an example on this?
Thanks
Then use headless browser.
… <#m_7813694022823339508_>
On Tue, 9 Apr, 2019, 1:07 PM yenanqu, @.***> wrote: Meta - Watir
Version: Selenium Version: Browser Version: Chrome Browser Driver Version:
OS Version: Windows 10 Expected Behavior - Each time when use .goto or open
a browser tab, it jumps on the front end Actual Behavior - Expect the
browser window keeps running on backed so it will not interrupt other works
Steps to reproduce - — You are receiving this because you are subscribed to
this thread. Reply to this email directly, view it on GitHub <#862
https://github.com/watir/watir/issues/862>, or mute the thread
https://github.com/notifications/unsubscribe-auth/AeOm7qoj6vJdw19xD1GeaXs4eGR2TPxFks5vfENBgaJpZM4cj0E7
.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/watir/watir/issues/862#issuecomment-481140944, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeOm7j4P7bbQGHxPq8L6C2ENV6L1IOnBks5vfEU0gaJpZM4cj0E7
.

http://watir.com/guides/firefox/#headless http://watir.com/guides/chrome/#headless
…
On Tue, 9 Apr, 2019, 1:15 PM yenanqu, @.> wrote: Hi Can you please give an example on this? Thanks Then use headless browser. … <#m_7813694022823339508_> On Tue, 9 Apr, 2019, 1:07 PM yenanqu, @.> wrote: Meta - Watir Version: Selenium Version: Browser Version: Chrome Browser Driver Version: OS Version: Windows 10 Expected Behavior - Each time when use .goto or open a browser tab, it jumps on the front end Actual Behavior - Expect the browser window keeps running on backed so it will not interrupt other works Steps to reproduce - — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#862 <#862>>, or mute the thread https://github.com/notifications/unsubscribe-auth/AeOm7qoj6vJdw19xD1GeaXs4eGR2TPxFks5vfENBgaJpZM4cj0E7 . — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#862 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AeOm7j4P7bbQGHxPq8L6C2ENV6L1IOnBks5vfEU0gaJpZM4cj0E7 .

Seems it does not work in my case. I am trying to fill in a form and search, but it times out when fill in
[0409/155248.812:ERROR:ssl_client_socket_impl.cc(962)] handshake failed; returned -1, SSL error code 1, net_error -101
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/watir-6.12.0/lib/watir/elements/element.rb:669:in `rescue in element_call': timed out after 30 seconds, waiting for #<Watir::Input: located: false; {:id=>"pt1:r1:0:pt1:cBodFDC:r1:0:search1:dc_i1:1:dc_id3::content", :tag_name=>"input"}> to be located (Watir::Exception::UnknownObjectException)

Seems it does not work in my case. I am trying to fill in a form and search, but it times out when fill in. There is no issue with normal browser..  seems headless browser is not working well for complicate browsers
[0409/155248.812:ERROR:ssl_client_socket_impl.cc(962)] handshake failed; returned -1, SSL error code 1, net_error -101
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/watir-6.12.0/lib/watir/elements/element.rb:669:in `rescue in element_call': timed out after 30 seconds, waiting for #<Watir::Input: located: false; {:id=>"pt1:r1:0:pt1:cBodFDC:r1:0:search1:dc_i1:1:dc_id3::content", :tag_name=>"input"}> to be located (Watir::Exception::UnknownObjectException)

Then what you are asking not only possible in normal browser and also it is
not even in the control of water, its in the control of driver.
On Tue, 9 Apr, 2019, 1:27 PM yenanqu<EMAIL_ADDRESS>wrote:

Seems it does not work in my case. I am trying to fill in a form and
search, but it times out when fill in. There is no issue with normal
browser.. seems headless browser is not working well for complicate browsers
[0409/155248.812:ERROR:ssl_client_socket_impl.cc(962)] handshake failed;
returned -1, SSL error code 1, net_error -101
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/watir-6.12.0/lib/watir/elements/element.rb:669:in
`rescue in element_call': timed out after 30 seconds, waiting for
#<Watir::Input: located: false;
{:id=>"pt1:r1:0:pt1:cBodFDC:r1:0:search1:dc_i1:1:dc_id3::content",
:tag_name=>"input"}> to be located
(Watir::Exception::UnknownObjectException)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/watir/watir/issues/862#issuecomment-481144630, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeOm7kRUQJ_nfq_oHJbS_2Ht-5illQdEks5vfEfngaJpZM4cj0E7
.

Then what you are asking not only possible in normal browser and also it is not even in the control of water, its in the control of driver.
…
On Tue, 9 Apr, 2019, 1:27 PM yenanqu, @.***> wrote: Seems it does not work in my case. I am trying to fill in a form and search, but it times out when fill in. There is no issue with normal browser.. seems headless browser is not working well for complicate browsers [0409/155248.812:ERROR:ssl_client_socket_impl.cc(962)] handshake failed; returned -1, SSL error code 1, net_error -101 C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/watir-6.12.0/lib/watir/elements/element.rb:669:in `rescue in element_call': timed out after 30 seconds, waiting for #<Watir::Input: located: false; {:id=>"pt1:r1:0:pt1:cBodFDC:r1:0:search1:dc_i1:1:dc_id3::content", :tag_name=>"input"}> to be located (Watir::Exception::UnknownObjectException) — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#862 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AeOm7kRUQJ_nfq_oHJbS_2Ht-5illQdEks5vfEfngaJpZM4cj0E7 .
Ok.. Thanks.. appears headless set the browser invisible.. Is there any way to keep the browser run at the backend? -- E.g. After opening a watir browser, I focus in other browser. and the  watir browser just run on the background no matter what happens on it...

Headless mode should be equivalent to normal execution. Take a screenshot and see what the issue is.
On the off chance you are using Linux, you could use the headless gem.
Regardless this request is limited to the driver implementation, and there's nothing that watir can  do about it.


add network cleanup to exit callback to fix infinite exit loop

if you don't cleanup the driver before you exit it will be stuck on the exit screen.

If only libc exited all threads like it's supposed to :P

this only occurs on psplink. still annoying tho

It appears that network cleanup will still not happen if you exit via reset - I'll still add a check


[EAK-509] Make sure irrelevant content is cleaned up the process of build

As an Etoolbox Authoring Kit developer, I need to make sure that obsolete and irrelevant content is properly cleaned up from the build-related folders (**/target** folders of every model plus jcr:root/etc/packagesfolder within theall` module)

Scheduled for release in v.2.5.1


python312Packages.reolink-aio: 0.11.3 -> 0.11.4

Diff: https://github.com/starkillerOG/reolink_aio/compare/refs/tags/0.11.3...0.11.4
Changelog: https://github.com/starkillerOG/reolink_aio/releases/tag/0.11.4

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 360448

x86_64-linux

:white_check_mark: 5 packages built:

home-assistant-component-tests.reolink
python311Packages.reolink-aio
python311Packages.reolink-aio.dist
python312Packages.reolink-aio
python312Packages.reolink-aio.dist


Orbiter sync slower than before

The backend catch.js seems much slower these days to sync up the DB once there are more transactions happened on chain. So the frontend seems vastly dragged.  We can reduce the interval between writing each record, but probably it can cause DB panic we can't predict, will try to experiment and improve.

if need .. i can admin and run orbiter on my servers ... let me know .. i am owner of some pools and i want contribute more with musicoin

Thanks, maybe we need a backup service. I will share some details here later.

@immartian  as we talked earlier on Slack, I'm done installing Orbiter (https://orbiter.reidocoin.com.br/).
I guess it will take up ages to sync it all. catch.js is running for some time now (+3h), and it's barely on block 2100.
Feel free to contact me if you need testing on a modified version of catch.js.
Cheers!

It's seems quite abnormal for the speed of syncing. Are you sure you are running only one catch.js instance.

This time it stops at 1062386 for long time and lagged 14 hours.  Obviously Parity runs into panic and returns nothing to node modules.
root@orbiter7762959-1:~# tail /root/.forever/hfzx.log -n 25
error: Script restart attempt #1
No config file found. Using default configuration (will download all blocks starting from latest)
Using configuration:
{ gethPort: 8545,
  output: '.',
  blocks: [ { start: 0, end: 'latest' } ] }
/musicoin/orbiter/node_modules/web3/lib/web3/httpprovider.js:83
        throw errors.InvalidResponse(request.responseText);
        ^
Error: Invalid JSON RPC response: undefined
    at Object.module.exports.InvalidResponse (/musicoin/orbiter/node_modules/web3/lib/web3/errors.js:35:16)
    at HttpProvider.send (/musicoin/orbiter/node_modules/web3/lib/web3/httpprovider.js:83:22)
    at RequestManager.send (/musicoin/orbiter/node_modules/web3/lib/web3/requestmanager.js:58:32)
    at Eth.get [as blockNumber] (/musicoin/orbiter/node_modules/web3/lib/web3/property.js:107:62)
    at patchBlocks (/musicoin/orbiter/tools/catch.js:307:29)
    at Object.<anonymous> (/musicoin/orbiter/tools/catch.js:371:1)
    at Module._compile (module.js:410:26)
    at Object.Module._extensions..js (module.js:417:10)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Function.Module.runMain (module.js:442:10)
    at startup (node.js:136:18)
    at node.js:966:3
error: Forever detected script exited with code: 1
root@orbiter7762959-1:~# ps aux|grep parity
root     12537 28.6  4.1 1673168 639444 ?      Sl   Sep15   4:39 parity --chain mc.json --tracing on```

^ is that parity or rmc?

Found several overlapped nodejs processes twisted, resolved by killing redundant ones.


kHz in SI is lower case k for kilo-

@boltomli : Thanks for your contribution! The author(s) have been notified to review your proposed change. @wsturman

@boltomli : Thanks for your contribution! The author(s) have been notified to review your proposed change. @wsturman

@boltomli : Thanks for your contribution! The author(s) have been notified to review your proposed change. @wsturman

#sign-off


Better error for disabling runner v2 with cross language pipelines.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

noting that tests passed, but status not updated.


Completion for kubectl plugins

FEATURE REQUEST

Kubernetes version (use kubectl version): 1.13
Environment:

Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release): macOS 10.14
Kernel (e.g. uname -a): Darwin Kernel Version 18.2.0: Mon Nov 12 20:24:46 PST 2018; root:xnu-4903.231.4~2/RELEASE_X86_64

What happened:
Having installed a kubectl plugin kubectl-test:
$ kubectl t[tab][tab]
taint top

You can't complete plugin commands as you can complete built-in commands.
What you expected to happen:
$ kubectl t[tab][tab]
taint test top

$ kubectl te[tab]  # Completes to 'kubectl test'

Anything else we need to know:
Completion for plugin names could be implemented in the kubectl completion script. But completion for arguments of plugins are plugin-dependent and can't be implemented in the kubectl completion script. In this case, plugin creators could provide completion scripts for their plugins which can then be invoked by the kubectl completion script when it detects that the user is using a plugin command.

/sig cli
/area kubectl
/kind feature

I try to implement completion for kubectl plugins.

/remove-lifecycle stale

/remove-lifecycle stale


start-tor-browser.desktop exists but isn't found

 ~ $ torbrowser-launcher
Tor Browser Launcher
By Micah Lee, licensed under MIT
version 0.3.6
https://github.com/micahflee/torbrowser-launcher
Launching Tor Browser.
Running /home/lutra/.local/share/torbrowser/tbb/x86_64/tor-browser/start-tor-browser.desktop
Traceback (most recent call last):
  File "/usr/bin/torbrowser-launcher", line 30, in <module>
    torbrowser_launcher.main()
  File "/usr/lib/python3.10/site-packages/torbrowser_launcher/__init__.py", line 87, in main
    gui = Launcher(common, app, url_list)
  File "/usr/lib/python3.10/site-packages/torbrowser_launcher/launcher.py", line 167, in __init__
    self.update()
  File "/usr/lib/python3.10/site-packages/torbrowser_launcher/launcher.py", line 214, in update
    self.start(None)
  File "/usr/lib/python3.10/site-packages/torbrowser_launcher/launcher.py", line 231, in start
    self.run_task()
  File "/usr/lib/python3.10/site-packages/torbrowser_launcher/launcher.py", line 299, in run_task
    self.run()
  File "/usr/lib/python3.10/site-packages/torbrowser_launcher/launcher.py", line 479, in run
    subprocess.call(
  File "/usr/lib/python3.10/subprocess.py", line 345, in call
    with Popen(*popenargs, **kwargs) as p:
  File "/usr/lib/python3.10/subprocess.py", line 971, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib/python3.10/subprocess.py", line 1847, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: '/home/lutra/.local/share/torbrowser/tbb/x86_64/tor-browser/start-tor-browser.desktop'

 ~ $ file /home/lutra/.local/share/torbrowser/tbb/x86_64/tor-browser/start-tor-browser.desktop
/home/lutra/.local/share/torbrowser/tbb/x86_64/tor-browser/start-tor-browser.desktop: a /usr/bin/env ./Browser/execdesktop script, Unicode text, UTF-8 text executable

How?

How did you install torbrowser-launcher
And which distribution are running it on?

I am on EndeavourOS with Arch Linux repos.
I installed torbrowser-launcher with yay -S torbrowser-launcher

You might want to delete /home/lutra/.local/share/torbrowser/ and try again.
I just checked on a fresh install, but cannot confirm your problem.
Maybe check file permissions of /home/lutra/.local/share/torbrowser/tbb/x86_64/tor-browser/start-tor-browser.desktop first.

File permissions are rwxrwx--- and owner is me.

Same here on Arch..  ../share/torbrowser removed; torbrowser-launcher and tor package removed and reinstalled. It started downloading then dropped the same No such File or directory error

Hey, I'm in the exact same scenario as you. I solved it by installing it from the website: https://www.torproject.org/download/
Removing the previous tor files:
rm -rf ~/.local/share/torbrowser/
Exctract the archive using:
tar xvf ./tor-browser-linux-x86_64-13.0.tar.xz
And then run the binary directly
./tor-browser/Browser/start-tor-browser


CB-11478 Parse --archs option consistently

This PR aligns arguments parsing logic with build command where passing --archs either as CLI argument (cordova build --archs) or platform arg, behind -- (cordova build -- --archs) doesn't make any difference.
See also original issue: CB-11478

@daserge, @sgrebnov, please review

Current coverage is 74.21% (diff: 100%)

Merging #190 into master will increase coverage by 0.02%

@@             master       #190   diff @@
==========================================
  Files            14         14
  Lines          1937       1939     +2
  Methods         360        361     +1
  Messages          0          0
  Branches        390        391     +1
==========================================
+ Hits           1437       1439     +2
  Misses          500        500
  Partials          0          0

Powered by Codecov. Last update 43b101a...0289fcc


Decouple Empty View from backgroundView, make it move with PTR

Changes in this pull request

The "Empty View" and CV background view are different things. People can now differentiate.
The Empty View isn't created until it is needed.
The Empty View moves with Refresh Controls.

Closes #458

Pull request checklist

[x] All tests pass. Demo project builds and runs.
[x] I added tests, an experiment, or detailed why my change isn't tested.
[x] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[x] I have reviewed the contributing guide

Closes #458

Hi,
I am curious that this feature is already implemented in 3.x.x?
I just tracked the source code from 3.1.1, this enhancement is not merged into master branch.

@ananfang see #924


"New Pad" should select the DART tab

If you're looking at the HTML or CSS tab when you click New Pad..., it stays on that tab. Which means that you see no code, since the initial sample is Dart only.
Instead, you should see the Dart code.

Done! This will be in the next release.

Excellent!


remove egg-info from excludes (Fixes #1601)

Description

Fixes   #1601.
Leaves the egg-info files to the package, since these might be needed during deployment depending on the platform the deployment is run on.
GitHub Issues

Coverage increased (+0.06%) to 73.933% when pulling 3fcf48fe1ede8b7b7521787e5ce811d452666b43 on 4lph4-Ph4un:master into 12ac82e71deeffcb03dbb6549be4f6f60d537382 on Miserlou:master.

For reference, this was added in https://github.com/Miserlou/Zappa/commit/eb4ddbcb44754e1f15bec24a7be467d89480aab4


Cilium monitor -t drop doesn't show L7 drops

Title
Cilium monitor -t drop doesn't show L7 drops
General Information

Cilium version (run cilium version)
root@minikube:~# cilium version
Client: 1.2.0 23fde20 2018-08-15T17:30:44+02:00 go version go1.10.3 linux/amd64
Daemon: 1.2.0 23fde20 2018-08-15T17:30:44+02:00 go version go1.10.3 linux/amd64

Kernel version (run uname -a)

minikube v0.28.0

Orchestration system version in use (e.g. kubectl version, Mesos, ...)

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.1", GitCommit:"b1b29978270dc22fecc592ac55d903350454310a", GitTreeState:"clean", BuildDate:"2018-07-18T11:36:43Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.0", GitCommit:"fc32d2f3698e36b93322a3465f63a14e9f0eaead", GitTreeState:"clean", BuildDate:"2018-03-26T16:44:10Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Link to relevant artifacts (policies, deployments scripts, ...)
http://docs.cilium.io/en/v1.2/gettingstarted/minikube/

How to reproduce the issue

Deploy SW app
Implement L7 policy from GSG
Run cilium monitor -t drop
Run kubectl exec tiefighter -- curl -s -XPUT deathstar.default.svc.cluster.local/v1/exhaust-port. Policy will deny traffic and return "Access denied" but no entry shows in cilium monitor output.

cilium-sysdump-20180823-083758.zip

They are not drops. You need to run with -t l7.


Add FileData and FileText

Add Wrapper for Files management functions (LoadFileData, LoadFileText, ...)

Great idea! 😉

With these additions, we'll be able to overload the other file methods too. Mind if we get a couple tests in place for them? Just something simple.

Should be working I moved them from my other Project :^) ... I also added some tests.
I would recommend using a test framework like Catch2 or google-test, like I did in my fork.

Should be working I moved them from my other Project :^) ... I also added some tests.
I would recommend using a test framework like Catch2 or google-test, like I did in my fork.

Was considering catch2 a while ago, but wanted to keep the syntax similar to raylib formatting. And just needed some simple assertions anyway 😉


Update Cask entry with this repo's releases and update related out-of-date tools

Homebrew Cask should be updated to point to this repo now that v0.8.1 was released.
Also, users of Spotify Notifications and, of course, Spotify Now Playing should be made aware of this since those projects are very out-of-date.

See the documentation on this topic. Excerpt:
Forks must have the vendor’s name as a prefix on the Cask’s file name and token. If the original software is discontinued, forks still need to follow this rule so as to not be surprising to the user. There are two exceptions which allow the fork to replace the main cask:

- The original discontinued software recommends that fork.
- The fork is so overwhelmingly popular that it surpasses the original and is now the de facto project when people think of the name.

The first thing is not likely going to happen for a very long time. The second point might happen, but I can't say that's going to be the case for sure. Either way, we can't take over the spotify-now-playing cask because this repo is not popular enough.
According to this documentation page we fail the following requirement:
The Cask is for an app that is too obscure. Examples:

- An app from a code repository that is not notable enough (under 30 forks, 30 watchers, 75 stars).

My options are limited; the old repository is outdated software and is grounds for removal from the casks repository, meaning forking it doesn't seem to make much sense. Creating a new formula seems to be impossible until we have > 75 stars.
As such, I'm afraid I need to close this issue until we gain popularity. I will still work on this project, though I will need to focus on marketing it a bit.

The following is the temporary PR description I wrote:

Information
This app is a hard fork. The repo obscurity currently is due to the new maintainer. The old repo (https://github.com/davicorreiajr/spotify-now-playing) has 182 stars as of writing.
Aside from the maintainers being more than 2 years absent on GitHub, I have attempted to:

Raise issues and submit PRs for bugs and feature requests in the original repository (>12 days no response).
Email the maintainer directly (>10 days no response).

In terms of my commitment to create a compelling reason to change the maintainer and repo:

I have contributed a fairly significant amount of code as change requests to the project.
Followed common configuration management practices and maintained a history of all old releases on my repository.
I have already created a plan for upcoming releases of the software.
Have gone through Apple's involved process of code signing and notarisation through the Apple Developer Program.

The rename is for legal/trademark and usability reasons.
Pull Request Information
Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.
In the following questions <cask> is the token of the cask you're submitting.
After making all changes to a cask, verify:

[x] The submission is for a stable version or documented exception.
[x] brew audit --cask <cask> is error-free.
[x] brew style --fix <cask> reports no offenses.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] Checked the cask was not already refused.
[ ] Checked the cask is submitted to the correct repo.
[x] brew audit --new-cask <cask> worked successfully.
[x] brew install --cask <cask> worked successfully.
[x] brew uninstall --cask <cask> worked successfully.

Outputs

brew install

$ HOMEBREW_NO_AUTO_UPDATE=1 brew install now-playing-for-spotify
==> Downloading https://github.com/teaminkling/mac-spotify-np/releases/download/v0.8.1/Now.Playing.for.Spotify-0.8.1.dmg
Already downloaded: /Users/thomas/Library/Caches/Homebrew/downloads/f047fde722df63982c87ac969edd0e0ba866d2b5293a24b96285739776b8e39c--Now.Playing.for.Spotify-0.8.1.dmg
==> Installing Cask now-playing-for-spotify
==> Moving App 'Now Playing for Spotify.app' to '/Applications/Now Playing for Spotify.app'
🍺  now-playing-for-spotify was successfully installed!

brew uninstall

$ brew uninstall now-playing-for-spotify
==> Uninstalling Cask now-playing-for-spotify
==> Backing App 'Now Playing for Spotify.app' up to '/usr/local/Caskroom/now-playing-for-spotify/0.8.1/Now Playing for Spotify.app'
==> Removing App '/Applications/Now Playing for Spotify.app'
==> Purging files for version 0.8.1 of Cask now-playing-for-spotify

brew audit --cask

$ brew audit --cask now-playing-for-spotify
audit for now-playing-for-spotify: passed

brew audit --new-cask

$ brew audit --new-cask now-playing-for-spotify
audit for now-playing-for-spotify: passed

brew style --fix now-playing-for-spotify

$ brew style --fix now-playing-for-spotify

1 file inspected, no offenses detected

I will still make some changes to make integration easier in the future:

Add the .rb formula file into git.
Change the README with info to why it's not in Cask.


keybinding ctrl-shift-w close Atom window

hello i have some trouble to set  your package (i'm novice).
When i use the ctrl-shift-w, it close the window (Atom).

Hi, It suppose to be At+Shift+w
Regards,
Sasha

Can be closed I guess


Avoid jumpiness in sort indicator

Presently when sorting a column, the column title text is offset to accommodate the indicator.
Ideally the indicator has opacity of 0 while the column is not under sort. This should avoid the jumpiness.

Ideally support user-configurable images, like the validation indicator in SwiftDetailer.


[FR] expose Preconditions in the set call for the admin API

(the Firestore Node Admin API folks sent me over here from https://github.com/firebase/firebase-admin-node/issues/1597)
Describe the problem
The Admin API exposes an update method which includes Preconditions. This allows us to specify that a document update call should fail if the DB document was modified after the precondition.updateTime. This is a nice way to make sure that REST API callers don't overwrite documents that were simultaneously modified.
Unfortunately, the set method does not expose a way to specify a similar Precondition, which means we can only specify Preconditions when updating an existing document, and not when we want to replace it wholesale.
Relevant Code:
// Setup
const ref = db.doc('/some/path');
const v1 = {foo: 'bar'};
await ref.set(v1);
const t1 = await ref.get().then((d) => d.updateTime);

// I can use preconditions with update
await ref.update({bim: 'baz'}, {lastUpdateTime: t1}); // This will succeed
await ref.update({bim: 'boing'}, {lastUpdateTime: t1}); // This will then fail due to the precondition

// But the result is a merged document. I want to *replace* the document as this is a REST API
await ref.get().then((d) => d.data());  // --> {foo: 'bar', bim: 'baz'} but I want just {bim: 'baz'}

// So we try to use `set` instead
await ref.set({bim: 'baz'}); // but there is no way to specify the precondition above

Workarounds:

I can manually read the existing document and throw an error myself if the precondition isn't met, but that makes for awkward transactions (which expect to do all reads before any writes).
I could enumerate top-level fields and spread across them with a FieldValue.delete() overridden by the new document (something like ref.update({...{field1: delete(), field2: delete()}, ...theNewDoc}), but this is also really awkward. While I know my top-level types, enumerating them is hard in typescript (where types are erased at runtime).

(ported over from https://github.com/firebase/firebase-js-sdk/issues/6058 and https://github.com/firebase/firebase-admin-node/issues/1597)

@jakebiesinger-storyhealth Thank you for making this feature request. Your use case sounds reasonable so I'll look into it. I hope to have a reasonable response within 7 days from now.

@jakebiesinger-storyhealth We discussed this as a team today and we're going to try to find a solution for this. The current idea is to add a lastUpdateTime member to the SetOptions type, and I'll prototype that change. Note that we have other priorities so it could take some time for this to get done, and I cannot promise any particular timeline.
Another workaround for you would be to use a transaction (i.e. Firestore.runTransaction()) which will lock the document and ensure that any writes are performed on the document that was read.


createDocument inputs

Hi,
I'm still a user of your great library ;)
I have a question about the inputs used for the "createDocument" operation.
According to the documentation, there is 2 required parameters : repositoryId & properties (array).
There is also some optionals like "folderId", "contentStream" and so on...
When I read your "createDocument" method, I can see you set "options.objectId" with the "parentId" parameter.
Is this a mistake ? I think we should have "options.folderId" instead of "options.objectId".
If you are agree with that, I can suggest you a PR.
BR.
Wiser.

I don't remember why the library uses objectId instead of folderId, probably beacuse the Alfresco implementation when I developed the library only worked with objectId as parameter..
Any PR is welcome, tank you!

Hi,
As I suggested to you, I prepared a PR.
But with my modifications, the 'createDocument' test do not pass anymore because of an error :
{
    "exception" : "contentAlreadyExists",
    "message"  : "An object with this name already exists: Existing file or folder test.txt already exists"
}

Obviously, the Alfresco repository does not recognize the folderId parameter of the createDocument method because it tries to create the "test.txt" file on the root folder instead of the previously created random folder...
Of course, I don't want to break your compatibility with Alfresco, so I guess you can close this issue, I will adapt my CMIS server implementation to accept the objectId parameter if folderId is not sent...
Wiser.

Ok, thank you


Automate adding protocol versions to the Change Log

SUMMARY
Currently, a maintainer needs to manually add a table with the current versions of the Oasis Core protocols to the Change Log as described in https://github.com/oasisprotocol/oasis-core/blob/master/docs/release-process.md#prepare-the-change-log.
Find a way to automatically extract this info from https://github.com/oasisprotocol/oasis-core/blob/master/go/common/version/version.go file and add this table to the Change Log.
ISSUE TYPE

Feature Idea

Currently, a maintainer needs to manually add a table with the current versions of the Oasis Core protocols to the Change Log as described in https://github.com/oasisprotocol/oasis-core/blob/master/docs/release-process.md#prepare-the-change-log.

The part that places the versions table to the Change Log was implemented in #3891. A maintainer now only needs to replace the <VERSION> placeholders with appropriate versions.


[Feature] Figure out best way for Semantic Model -> Relationships to be established in dbt DAG

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[ ] I am requesting a straightforward change to existing dbt-semantic-interfaces functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Right now everything in dbt land is represented as a node in the DAG. These relationships are established with things like ref('some_model').
Once we introduce the semantic constructs (metric & semantic model) we'll want those relationships to hold true. IE I should be able to see that model A feeds into semantic model B and Metric C.
🤔
Describe alternatives you've considered
Hiding?
Who will this benefit?
Anyone who uses the semantic layer
Are you interested in contributing this feature?
Yarp
Anything else?
No response

This is being handled in core by tickets focused on populating the MetricNode.depends_on property


Nest can't resolve dependencies of the MikroOrmCoreModule (Symbol(mikro-orm-module-options), ?). Please make sure that the argument ModuleRef at index [1] is available in the MikroOrmCoreModule context.

Describe the bug
I've tried several configurations, but I keep getting this error.
Stack trace

[Nest] 69475  - 07/17/2021, 4:01:41 PM     LOG [NestFactory] Starting Nest application...
[Nest] 69475  - 07/17/2021, 4:01:41 PM     LOG [InstanceLoader] ConfigModule dependencies initialized +188ms
[Nest] 69475  - 07/17/2021, 4:01:41 PM     LOG [InstanceLoader] DomainModule dependencies initialized +1ms
[Nest] 69475  - 07/17/2021, 4:01:41 PM     LOG [InstanceLoader] MikroOrmModule dependencies initialized +0ms
[Nest] 69475  - 07/17/2021, 4:01:41 PM   ERROR [ExceptionHandler] Nest can't resolve dependencies of the MikroOrmCoreModule (Symbol(mikro-orm-module-options), ?). Please make sure that the argument ModuleRef at index [1] is available in the MikroOrmCoreModule context.

Potential solutions:
- If ModuleRef is a provider, is it part of the current MikroOrmCoreModule?
- If ModuleRef is exported from a separate @Module, is that module imported within MikroOrmCoreModule?
  @Module({
    imports: [ /* the Module containing ModuleRef */ ]
  })

Error: Nest can't resolve dependencies of the MikroOrmCoreModule (Symbol(mikro-orm-module-options), ?). Please make sure that the argument ModuleRef at index [1] is available in the MikroOrmCoreModule context.

Potential solutions:
- If ModuleRef is a provider, is it part of the current MikroOrmCoreModule?
- If ModuleRef is exported from a separate @Module, is that module imported within MikroOrmCoreModule?
  @Module({
    imports: [ /* the Module containing ModuleRef */ ]
  })

    at Injector.lookupComponentInParentModules (/App/node_modules/@my-org/api/node_modules/@nestjs/core/injector/injector.js:189:19)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at Injector.resolveComponentInstance (/App/node_modules/@my-org/api/node_modules/@nestjs/core/injector/injector.js:145:33)
    at resolveParam (/App/node_modules/@my-org/api/node_modules/@nestjs/core/injector/injector.js:99:38)
    at async Promise.all (index 1)
    at Injector.resolveConstructorParams (/App/node_modules/@my-org/api/node_modules/@nestjs/core/injector/injector.js:114:27)
    at Injector.loadInstance (/App/node_modules/@my-org/api/node_modules/@nestjs/core/injector/injector.js:47:9)
    at Injector.loadProvider (/App/node_modules/@my-org/api/node_modules/@nestjs/core/injector/injector.js:69:9)
    at async Promise.all (index 0)
    at InstanceLoader.createInstancesOfProviders (/App/node_modules/@my-org/api/node_modules/@nestjs/core/injector/instance-loader.js:44:9)

To Reproduce
import { Module } from '@nestjs/common';
import { GraphQLModule } from '@nestjs/graphql';
import { ConfigModule, DatabaseModule } from '@my-org/infra';
import { ApiModule, ApiController, ApiService } from '@my-org/api';
import { Config } from '@my-org/config';
import { MikroOrmModule } from '@mikro-orm/nestjs';
import { getRootPath } from '@my-org/utils';
import { DomainModule } from '@project/domain';

import { ProjectAppController } from './app.controller';
import { ProjectAppService } from './app.service';

@Module({
  imports: [
    ConfigModule,
    DomainModule,
    GraphQLModule.forRoot({
      debug: false,
      playground: true,
      autoSchemaFile: getRootPath('apps/project-api/graphql-schema.gql'),
      path: Config.serviceGraphqlApiEndpoint,
    }),
    MikroOrmModule.forRoot(
      Config.getMikroOrmConfig({
        migrations: {
          migrationsList: [],
        },
      })
    ),
    DatabaseModule,
  ],
  controllers: [ProjectAppController, ApiController],
  providers: [ProjectAppService, ApiService],
})
export class AppModule extends ApiModule {}

DomainModule imports ResourceModule
import { Module } from '@nestjs/common';
import { ResourceService } from './resource-manifest.service';
import { ResourceResolver } from './resource-manifest.resolver';
import { MikroOrmModule } from '@mikro-orm/nestjs';
import { Resource } from './entities/resource-manifest.entity';

@Module({
  providers: [ResourceResolver, ResourceService],
  imports: [MikroOrmModule.forFeature([Resource])],
})
export class ResourceModule {}

Expected behavior
To see dependencies resolved
Additional context
Add any other context about the problem here.
Versions

Dependency
Version

node
v12.18.3

typescript
4.3.5

    "@mikro-orm/core": "^4.5.7",
    "@mikro-orm/migrations": "^4.5.7",
    "@mikro-orm/nestjs": "^4.2.0",
    "@mikro-orm/postgresql": "^4.5.7",

I console logged out the options I passed into the MikroOrmModule static methods I used:
forFeature [ [class ResourceManifest extends BaseEntity] ]
forRoot {
  dbName: 'project',
  user: 'project',
  host: 'localhost',
  port: 5432,
  password: undefined,
  type: 'postgresql',
  autoLoadEntities: true,
  migrations: { migrationsList: [] }
}

See: https://github.com/mikro-orm/nestjs/issues/27

It's a strange thing, but I was able to overcome this issue by starting my feature branch fresh. There was probably some implemenation quirk I ran into.
I'm happy to close this issue (although I may open it back up if I come across it again).

If you are using nest 8 then it is a dup of #27. If not, provide full repro.

@hcharley what do you mean by "starting fresh" ? What exactly you did to make it work? I'm having the same issue and upgrading to v4.3 doesn't  solve it.

Preface
I encountered this error and was stuck for a while, but recently figured it out and wanted to post my issue+solution in case someone else stumbles on it.
The issue
The core issue was that I had multiple installations of nestjs, although they had the same version, but just in different folders. And then the mikro-orm packages were accessing a different installation of NestJS than my application. I think this led to the error that ModuleRef couldn't be found, because the ModuleRef that the mikro-orm modules wanted to use is from a different NestJS installation than the one that my application was actually providing.
The reason it was so hard to figure out is that I am using yarn v1 workspaces and my application is just a single package within the workspace. And my package within the workspace had (for some unknown reason) its own installation of NestJS, whereas mikro-orm was properly hoisted (hoisting means the module is installed in the root of the workspace instead of just the individual package) and installed in the root of my workspace, in addition to the aforementioned separate installation of NestJS that was also hoisted and installed in the root of my workspace.
So the structure looked like this:
/                     (root of my yarn workspace)
  node_modules/       (hoisted node modules)
    @nestjs/...       (hoisted version of @nestjs)
    @mikro-orm/...    (hoisted version of @mikro-orm packages, they access the
                       hoisted @nestjs packages from this same directory)
  packages/
    my-subpackage/    (this is my own package)
      node_modules/
        @nestjs/...   (this is a different installation of the @nestjs packages,
                       that are actually used by my application code)
      src/
        ...           (my application code)
      package.json    (the package.json for this my-subpackage)
  package.json        (the package.json for the whole yarn workspace)
  yarn.lock           (the yarn lockfile for the whole workspace)

And the confusing part was that the yarn.lock indicated that only a single version of the @nestjs/... packages were installed. Only after manually checking the directories I figured this out.
How I solved it
I could finally solve it by setting the nohoist attribute in the package.json within my-subpackage, to force yarn to not hoist the nestjs and mikro-orm packages.
So the package.json inside of my-subpackage finally looked like this:
{
  "name": "my-subpackage",
  ...
  "workspaces": {
    "nohoist": ["@mikro-orm/**", "@nestjs/**"]
  }
}

Afterwards, it was important to remove the node_modules directories and remove the existing yarn.lock file, so that yarn actually applies the nohoist rules.
Additionally, in case you encounter this error but aren't using yarn workspaces, my advice is just to go through the node_modules folders and figure out if there are any multiple versions of @nestjs/... or @mikro-orm/... packages installed, and if so, you'll have to figure out why and how to force NPM/Yarn (or whichever package manager of your choice) to only install a single version of these packages.
Good luck!

for pnpm in the .npmrc need to add
public-hoist-pattern[]=@mikro-orm/*


New nt sim context

What does the code in this PR do / what does it improve?
In this PR we add a new "offfline" simulation context to straxen. The context queries correction values from CMT and pass them to straxen as constant values. This allows to freez-in a certain setting given by a cmt_run_id for simulations.
Can you briefly describe how it works?
First we create the context and register all plugins. Then we check all Plugins for CMT options, query the corresponding correction values and overwrite the CMT options with "constant" options. There was one issue though. Some corrections also need in addition the name of our default reconstruction algorithm. I added therefore the _depends_on_pos_algo function but would prefer to replace it with something less hardcoded. I am open for suggestions.

Thank you @WenzDaniel, for the PR and for the private written and ZOOM discussions on it :)
Just a few conclusive points if we move forward, following up on the discussion we had on Tuesday on this topic (cc @ershockley @terliuk @petergaemers @zhut19):

We conclude that we always need a cmt_run_id for this simulations context, so no option for cmt_run_id=0 to ignore CMT configs.
Instead, we now have the flag overwrite_from_fax_file, which points to the fax_config* file variables that match CMT parameter name. This requires renaming the relevant fax_config* parameters and propagating this renaming to WFSim.
In this case, we risk to have, e.g., an elife definition in fax_config* different than that from CMT. Regardless of the overwrite_from_fax_file value, WFSim will always use the one defined in fax_config*. We can allow this only when setting _allow_diverging_settings=False, so there's a check for it 👌
In order to be able to load raw_records with any context (we all agreed on this), we could either

hardcode the gain model via context when producing them in WFSim; or
untrack the gain model from raw_records, although this requires to properly document how these were produced (we will store the context in mc_chain). -> I prefer this option. Do you all agree?

one consideration which might be worth changing is putting all the CMT work arounds in the CMT module?

Sure I can move the corresponding code.

I think we can merge and start testing.
We don't need to resolve this before merging, but better sooner than later: do we all agree with what I wrote in the bullet point above? This is, can we untrack gain model from the raw_records lineage from now on?

I would try and not do this whenever possible. You want to know if channels aren't producing data because of gains set to 0. So we should track this in my opinion.

I think we can merge and start testing.
We don't need to resolve this before merging, but better sooner than later: do we all agree with what I wrote in the bullet point above? This is, can we untrack gain model from the raw_records lineage from now on?

I would try and not do this whenever possible. You want to know if channels aren't producing data because of gains set to 0. So we should track this in my opinion.

Hi @jorana. I think this was the compromised we reached in the dedicated meeting with @WenzDaniel @ershockley @zhut19 @petergaemers et al. We will document the gains used for that in the mc_chain, for example (context and configs will be tracked in the corresponding database entry), but we want to be able to load raw_records with any context, and having the gains define the lineage would not allow us to do that. I would appreciate if tagged people can share their thoughts here.

I would try and not do this whenever possible. You want to know if channels aren't producing data because of gains set to 0. So we should track this in my opinion.
It can be saved, but strax(en) should not know/care about generation gains, similarly as it doesn't know about "true" gains in data. We can keep it for information somwhere, but it should not affect behavior of further steps of processing in any way. Ideally, we want to ensure that there is one copy of raw_records, as different generation gains should always mean different MC run number/ID, as it is easier to track it such way.

I would try and not do this whenever possible. You want to know if channels aren't producing data because of gains set to 0. So we should track this in my opinion.

It can be saved, but strax(en) should not know/care about generation gains, similarly as it doesn't know about "true" gains in data. We can keep it for information somwhere, but it should not affect behavior of further steps of processing in any way. Ideally, we want to ensure that there is only one copy of raw_records, as different generation gains should always mean different MC run number/ID, as it is easier to track it such way.

We absolteu

whoops wrong button

It can be saved, but ~strax(en)~ WFSim should not know/care about generation gains, similarly as it doesn't know about "true" gains in data.

This isn't strictly true, unless you mean to run gain calibrations on simulated data 😉 and use those gains. We at least know that the gains we are processing real data with are the best representation of the gains we currently have for that channel. If you untrack the gains at raw-records level and just start doing something completely different later in raw records, you would end up with very strange results and an uncalibrated simulation context right? If the raw-records instruct gains of 1e6  everywhere and you later process some channels with 5e6 and other with 1e5, doesn't that create problems?
Anyway, I don't want to interfere too much if this has been al decided upon before. If you untrack the gains, it only makes sense to process data with the exact same gains as the raw-records were generated (perhaps only dropping x channels, I see the benefit there). So either you track this in code or rely on the user not to do something by accident.

This isn't strictly true, unless you mean to run gain calibrations on simulated data wink and use those gains. We at least know that the gains we are processing real data with are the best representation of the gains we currently have for that channel. If you untrack the gains at raw-records level and just start doing something completely different later in raw records, you would end up with very strange results and an uncalibrated simulation context right? If the raw-records instruct gains of 1e6 everywhere and you later process some channels with 5e6 and other with 1e5, doesn't that create problems?
Anyway, I don't want to interfere too much if this has been al decided upon before. If you untrack the gains, it only makes sense to process data with the exact same gains as the raw-records were generated (perhaps only dropping x channels, I see the benefit there). So either you track this in code or rely on the user not to do something by accident.

The main reason why we want to have different gains at simulation and later analysis/peaklet level is exactly to check the effect of gain mismatch in data/truth and our processing. We might have problem if we have very different gains at wfsim simulation level and then straxen processing, but it will be the same as we have in data, as we know gains only with finite precision. And using the same gains for both makes assumption that we know gains precisely, so limiting possibility to study effects of gain uncertainties. If we do not track it, we can introduce some fluctuations to our "simulation" gain model (or any other simulation parameter) and the process it with mismatched one to evaluate the impact.

Right, for such dedicated studies I can see the point. But I think for such very specific checks, there are other options, like just ignoring the gains when you are performing this detailed check using fuzzy for: https://strax.readthedocs.io/en/latest/advanced/fuzzy_for.html
Again, I don't want to stall progress on things already agreed on.

Based on previous discussions we said we would untrack the gains in raw_records. The main point is what Diego said: like we have for real TPC data, any simulated raw data should be able to be loaded using any context (even if the gain model is wrong). I guess using 'fuzzy' is an option too. I don't really know which one would be better


How does cookie code work

I'm having trouble setting up jwtauthenticator to authenticate my user. We simply want to allow the user to log into our web app, then be able to hit a link that takes them to Jupyterhub without having to log into there.
I've asked some questions on security.stackexchange asking what I need to do, but I don't seem to have any responses that help. For reference, here is that conversation: Understanding JWT and SSO
In a nutshell, here's what we've tried:

Submitting Authorization header with JWT token... Works in PostMan, but we can't open the Jupyter link in a new tab, as we can't submit headers.

GET request with JWT token as query param. This works, but I believe there are security issues.

POST equivalent of GET. I believe this would be secure, as the POST variables are encrypted when using HTTPS. However, it doesn't appear jwtauthenticator allows POST.

Set XSRF-TOKEN cookie with the value being the JWT token string. Both web app and Jupyterhub are subdomains of the same domain. Domain is set on the cookie, with path "/". Setting the cookie seems like the way to make this work, but it doesn't seem to be reading the cookie. DevTools shows the cookie is there when at the Jupyterhub site.

I was hoping this would be easier than it seems to be. All we are trying to do is provide our users a link to Jupyterhub that doesn't require login. Can someone tell me if I'm on the right track to implement that with the cookie option? And if so, what could be wrong. Thanks.

Never mind. I wasn't the one setting up the server and apparently whoever did had a very old version of jwtauthenticator. Odd, since we only recently went to set it up, so I wouldn't have expected an old version. Not sure where this old version came from. But it seems to work, so I'm happy.

Hi @uler3161, I exactly have the same question as you. I am using query parameters but it is not a good idea for security reasons.
Would you mind to share how you finally solve it?
Many thanks!

We first made sure we had the latest version of jwtauthenticator (which was our main issue).
Our code basically calls an endpoint which will set the proper cookie before returning. In our case, we also return the url to our Jupyter server which the browser will then navigate to instead of hard-coding it into the web app.
`
var cookie = new CookieHeaderValue("XSRF-TOKEN", $"{BuildJWTToken()}")
{
HttpOnly = true,
Secure = true,
Domain = GetJupyterDomain(),
Path = "/"
};
        var resp = new HttpResponseMessage
        {
            StatusCode = HttpStatusCode.OK,
            Content = new StringContent(GetJupyterUrl())
        };
        resp.Headers.AddCookies(new[] { cookie });
        return resp;

`
And we use This Library to generate a JWT token string (which we do in the BuildJWTToken method).
I think we also had to have our web app and our Jupyter server mapped to the same Domain for this to work. Basically whatever you can set up in order for the XSRF-TOKEN Cookie to be seen in both the web app and in Jupyter.

Many thanks for the sharing. I forgot to use the same domain. It works fine.
BTW, have you handled the logout case?
When I access to Jupyter with another token in XSRF-TOKEN cookie without logging out the previous account, it shows 403 Forbidden error OR the project of the previous account.

We are having some logout issues, specifically seeing projects from the previous account. It seemed to work fine for me to use the logout link in Jupyter. However, another person on my team says the logout link seems to not work reliably. We are planning to find a way to force a logout before the next login, but we haven't worked on it yet.

Thank you so much for the sharing!


feat: add optional starting step to VTour

Hey there, I've just merged #24 which does that already, I'll cherry pick your 2nd commit if that's good for you.

Excellent! thanks!


Removing a remote project folder does not remove the lint errors from the pane

Admittedly, remote project folders are Nuclide-specific (see http://nuclide.io/docs/remote/), but when I remove one, I expect the lints associated with the files from such a folder to be removed from the linter's Project pane.

The messages from linters with a scope of file are removed as soon as the person closes that text editor, but project scoped messages aren't removed unless by the provider.
We could use atom.project.onDidChangePaths https://atom.io/docs/api/v1.0.0/Project#instance-onDidChangePaths

@bolinfest: Just to confirm, removing a project folder triggers that event right

I just saw this when I removed a local folder, as well. I even reloaded the window and the errors from that folder are still displayed in the UI.
I think this qualifies as a "bug" rather than an "enhancement."

Also, I've been working on putting together a set of instructions to make it easier to test remote server integrations locally on OS X: https://github.com/bolinfest/hhvm-vagrant.
What's there now works, but there's no troubleshooting section or anything else like that.

Cool. Thanks.

"Project" doesn't actually always mean "Project". It often means "All of your open tabs". When you remove one of these remote folders does it also close the tabs you have open?

@AsaAyers but the messages from Project scoped Providers remain even if you close all the tabs.

It needs to be made far more clear what "Project" means. As I understand it:
If you're using a file based linter like linter-coffeelint then Project refers to all open tabs and the messages will get removed when you close the tabs.
If you're using a project based linter Project actually refers to your whole project and the messages remain even as the tabs close.

@AsaAyers You understand it correctly.

A fix is up in #641


Fix code generation for clang

Code generation with clang set resource names to _<name> unlike for GCC where it is <name>, while exporting the symbol <name> thus resulting in this kind of linking error (sample from ImHex build):
[ 92%] Linking CXX executable imhex
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource2'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource3_size'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource1_size'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource4'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource0_size'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource2_size'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource0'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource4_size'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource3'
/usr/bin/ld: plugins/libimhex/libimhex.so: undefined reference to `resource1'
clang-13: error: linker command failed with exit code 1 (use -v to see invocation)

This patch fixes that by simply removing the prefixed _ character

Thank you! Sorry I didn't see this PR until now


Konflux update test-component-pac-sfxloi

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-sfxloi-on-pull-request has successfully validated your commit.
Namespace: build-e2e-bkwe-tenant
PipelineRun: test-component-pac-sfxloi-on-pull-request-hshgz

Task Statuses:

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
35 seconds
deprecated-base-image-check

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
38 seconds
ecosystem-cert-preflight-checks

✅ Succeeded
33 seconds
sbom-json-check

✅ Succeeded
8 seconds
push-dockerfile

✅ Succeeded
18 seconds
apply-tags

✅ Succeeded
30 seconds
sast-snyk-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
10 seconds
show-summary

✅ Succeeded
6 seconds
show-sbom

Pipelines as Code CI/test-component-pac-sfxloi-on-pull-request has successfully validated your commit.


Update misc webpack related packages

As part of the effort to update to webpack5 (#9568) this updates a few webpack related packages, hopefully reducing the number of issues when updating webpack itself.
The most significant change is migrating from duplicate-package-checker-webpack-plugin to its fork @cerner/duplicate-package-checker-webpack-plugin because the original has not merged support for webpack5, see https://github.com/darrenscerri/duplicate-package-checker-webpack-plugin/pull/46

Found some broken links!

I checked, and these links now work, so I think this was a temporary external issue, and not due to web packing the docs differently.

debugger seems to be failing to find the "fs" module, which should be provided by the "@types/node" that is has.
For some reason (which I have no idea about), it did not need  "types": [ "node" ] in the tsconfig before, but needs it now.
This is now fixed.

Found some broken links!

I checked, and these links now work, so I think this was a temporary external issue, and not due to web packing the docs differently.

Yeah, I've seen these failures intermittently before. You can ignore them.


editor-styles not loaded on production.

Hi Chris,
Im having a issue with the backend styles on my production site. I know i need to run the build command and i do so. Im having a staging and a production site. Both are on the same server. Staging is just a subdomain of the production version. The weird thing is that on my staging site all works as expected, but on my production site it doestn seem to load the editor styles.
Any idea? I tried a lot but no succes
Maybe it has anything to do with the following: My subdomains are:
/var/www/vhosts/domain.nl/test.domain.nl/wp-content/themes/timberland
My main has the httpdocs:
/var/www/vhosts/domain.nl/httpdocs/wp-content/themes/timberland

Hi, can you confirm that all the expected files are in the assets build directory? Also, ensure that the file names in theme/assets/dist/.vite/manifest.json match the asset filenames.

Yes they ar all there. It look like the add_editor_styles function just doesn't run/work.

What versions of WordPress are your staging and production sites running? I'm working on a new project using the latest version of WordPress and noticed that block styles in the editor are being overridden.


Armor and Weapons for GeckoAnimtatedModels

Hi, can you tell me how to make my GeckoAnimtatedModel could carry weapons and armor

This is not a place to ask for support but to report plugin issues.


def-package! in config file

After reading the documentation I'm a little confused about how to include packages in my ~/.doom.d/config.el file. I would suggest putting adding a complete example in the Wiki.
I'm trying to switch back to highlight-indent-guides mode because I prefer the character-based guides. I've added the following to my config:
(def-package! highlight-indent-guides
  :commands highlight-indent-guides-mode
  :hook (prog-mode . highlight-indent-guides-mode)
  :config
  (setq highlight-indent-guides-method 'character
        highlight-indent-guides-character ?\•
        highlight-indent-guides-delay 0.01
        highlight-indent-guides-responsive 'top
        highlight-indent-guides-auto-enabled nil))

but I get an error: Cannot open load file: No such file or directory, highlight-indent-guides. Am I missing something?
Using doom develop and emacs-plus --HEAD from homebrew.

I apologize for the underdeveloped documentation. It is something I am currently working on.
The trick to adding a new package is to add a package! statement to your private ~/.doom.d/packages.el file. Like so:
;; -*- no-byte-compile: t; -*-
;;; ~/.doom.d/packages.el

(package! highlight-indent-guides)

Then run bin/doom refresh (or make; they are equivalent).

I had tried adding that file as well without any luck, and the final piece I was missing was to run doom refresh. I was only running doom clean && doom compile :core. All good now!
So it seems like the .doom.d folder is basically a private module?
No need to apologize, I appreciate you answering my questions, and you've already gone above and beyond with this project. Impressive work!

doom clean && doom compile :core

It is unnecessary to call doom clean yourself. All doom compile commands will do it for you before compiling.

So it seems like the .doom.d folder is basically a private module?

That is correct.

Impressive work!

Thank you! I'm glad Doom suits you.


ISPN-14816 Replacing Protobuf schema does not report validation errors

https://issues.redhat.com/browse/ISPN-14816

Merged, thanks

Can we have a backport please?

Can we have a backport please?

Sure. Here he is https://github.com/infinispan/infinispan/pull/10913.
Shall I clone a JDG issue from ISPN-14816?


Event Metrics and Advanced Metrics API

What: An API proposal for event metrics and advanced metrics.
View the proposed changes using the redocly API viewer
Sample Convergence Test Case
import pytest

@pytest.fixture
def primary_routes_name():
    return 'Rx Primary Routes'

@pytest.fixture
def secondary_routes_name():
    return 'Rx Secondary Routes'

@pytest.fixture
def convergence_config(api, primary_routes_name, secondary_routes_name):
    """A skeleton configuration suitable for testing convergence.
    Sets up TX routes and identical RX primary/secondary routes.
    """
    config = api.config()
    p_tx, p_rx_pri, p_rx_sec = config.ports \
        .port(name='Tx Port') \
        .port(name='Primary Rx Port') \
        .port(name='Secondary Rx Port')
    d_tx, d_rx_pri, d_rx_sec = config.devices \
        .device(name='Tx Device', container_name=p_tx.name) \
        .device(name='Primary Rx Device', container_name=p_rx_pri.name) \
        .device(name='Secondary Rx Device', container_name=p_rx_sec.name)
    tx_rr, = d_tx.ethernet.ipv4.bgpv4.bgpv4_routes.bgpv4route(name='Tx Routes')
    rx_prr, = d_rx_pri.ethernet.ipv4.bgpv4.bgpv4_routes.bgpv4route(name=primary_routes_name)
    rx_srr, = d_rx_sec.ethernet.ipv4.bgpv4.bgpv4_routes.bgpv4route(name=secondary_routes_name)
    flw, = config.flows.flow(name='Tx Ipv4 Device -> Rx Primary/Secondary Routes')
    flw.tx_rx.device.mode = flw.tx_rx.device.ONE_TO_ONE
    flw.tx_rx.device.tx_names = [tx_rr.name]
    flw.tx_rx.device.rx_names = [rx_prr.name, rx_srr.name]
    return config

def test_data_plane_convergence(api, convergence_config, primary_routes_name, secondary_routes_name):
    """Test data plane convergence using advanced metrics

    Configure convergence setup
    Configure advanced convergence metrics
    Create a convergence event
    Retrieve convergence metrics
    """
    # set the traffic generator configuration
    api.set_config(convergence_config)

    # build the configuration for advanced convergence metrics
    adv_metrics = api.advancedmetrics_config()
    adv_metrics.convergence.enable = True
    adv_metrics.convergence.rx_rate_threshold = 90
    api.set_advanced_metrics(adv_metrics)

    # withdraw primary routes using the primary_routes_name
    route_state = api.route_state()
    route_state.state = route_state.WITHDRAW
    route_state.names = [primary_routes_name]
    api.set_route_state(route_state)

    # get advanced data plane convergence metrics
    adv_metrics_request = api.advancedmetrics_request()
    adv_metrics_request.convergence.flow_names = [
        convergence_config.flows[0].name
    ]
    adv_metrics_request.convergence.event_sources = [
        secondary_routes_name
    ]
    while True:
        adv_metrics = api.get_advanced_metrics(adv_metrics_request)
        if all([m.data_plane_convergence_ns is not None for m in adv_metrics.convergence_metrics]):
            break

    # output the convergence metrics
    print(adv_metrics.convergence_metrics)

    # fail the test if data plane convergence takes longer than 500ms
    assert(all([m.data_plane_convergence_ns < 500000000 for m in adv_metrics.convergence_metrics]))

if __name__ == '__main__':
    pytest.main(['-vv', '-s', __file__])

Will it be feasible to facilitate a way to fetch tx/rx stats (frame count, loss, etc.) and latency stats for a given flow at same instance of time ?
Maybe we need to understand how important this use case is, but currently this is not achievable.

There are multiple places we are using *_timestamp attributes. It would be nice if we could have one common description of what is the format of timestamp values. (unix timstamps ?)

@ashutshkumr : sample test as discussed is updated here https://github.com/open-traffic-generator/snappi-tests/pull/15/files

@ashutshkumr : sample test as discussed is updated here https://github.com/open-traffic-generator/snappi-tests/pull/15/files

@ankur-sheth @ajbalogh @winstonliu1111

@ByReaL @selldinesh did you get a chance to review the data model ? These changes affect how we'll end up writing community test cases for CP/DP.

removing in favor of pr #108


Show "Trezor firmware not supported" popover if DEVICE_FAILED is fired

MultiBit Hardware will fire DEVICE_FAILED containing a valid Features if the firmware is not supported. The MainController should detect this and react accordingly - typically with a delayed popover similar to the security alerts.

This change requires the latest MultiBit Hardware develop-SNAPSHOT.

Added support for security popovers and alert bars for unsupported firmware throughout. The rules are:

either an alert bar or a popover will be shown once for each DEVICE_FAILED
popovers will be displayed if a wizard is showing on a secure screen (password, PIN etc)
alert bars will be displayed if no wizard is showing (unlocked wallet)
alert bars will remain (and stack) until dismissed
switching between a good and bad device on credentials should be painless

Ready for review and close.

It's almost perfect but I managed to confuse it by doing the following:

Start up a soft wallet
Plug in a 1.2.0 firmware Trezor. Saw alert correctly. Dismissed alert.
Unplugged trezor
Did a Switch wallet. I then saw a Security alert saying the attached wallet firmware was out of date. But I had just unplugged the trezor.

Other than that it all workes exactly as expected.

Ah yes, it's a race condition. Edit MainController.onSecurityEvent() to have Uninterruptibles.sleep... be 250ms.

Ok, pushed the change. Ready for review and close.

Nope sorry, it is still the same.
I've checked the MainController has the sleep at line 469 so I am definitely working with your change

I'm AFK for significant changes so I'll need you to take this one on.
Could you remove the Uninterruptibles completely, then in MainController.onHardwareWalletEvent look for SHOW_DEVICE_DETACHED and add a check for CoreServices.getApplicationEventService.getLatestsecurityEvent(), verify it is an UNSUPPORTED_FIRMWARE event and if so to remove it through onSecurityEvent(null).
This will ensure that if the user detaches the device that the security alert will be cleared. If they don't then each secure operation (e.g. enter PIN, password etc) will nag them.

Push changes as per your suggestion above. Works fine on my machine.
Awaiting review and closing by @gary-rowe

Thanks, Jim. I'll check it tonight.

Minor adjustment and verified working.
Closing.


[metal-cpp] add new port

Describe the pull request

What does your PR fix?

no, I need add a new port.

Which triplets are supported/not supported? Have you updated the CI baseline?

osx, Yes

Does your PR follow the maintainer guide?

Yes
If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@justforlxz Thanks for posting this PR, please sign CLA

@justforlxz BTW, the precautions for adding new ports must all meet the conditions. 🦾
New port checklist:

Check the name against https://repology.org/
Check the name against Bing/Google. (If the first results aren't C++ related, try "name C++")
Check the source code for optional find_packages
Check that the versioning scheme and license match what upstream says
Check that the source code comes from the upstream project's authoritative source.
Check that the generated usage is accurate.

@justforlxz Are you still continuing this PR?

@justforlxz Are you still continuing this PR?

Sorry, will update soon.

Pinging @justforlxz for response. Is work still being done for this PR?

Pinging @justforlxz for response. Is work still being done for this PR?

Sorry for waiting so long, I have finished editing.

Log form x64-osx:
Mismatching number of debug and release binaries. Found 0 for debug but 1 for release.
Debug binaries

Release binaries

    /Users/vagrant/Data/packages/metal-cpp_x64-osx/lib/libmetal.a

Debug binaries were not found

Found 1 post-build check problem(s). To submit these ports to curated catalogs, please first correct the portfile:
    /Users/vagrant/Data/work/1/s/ports/metal-cpp/portfile.cmake
-- Performing post-build validation done
error: building metal-cpp:x64-osx failed with: POST_BUILD_CHECKS_FAILED

@justforlxz

Pinging @justforlxz for response. Is work still being done for this PR?

Draft this PR since it seems that no progress is being made

Pinging @justforlxz  for response. Is work still being done for this PR?

This PR has been inactive for a long time, please reopen it if there is any progress.


Messages from discord not parsed correctly

Something seems to be going wrong when the mod converts the discord message -> MC chat message.
server log shows the message correctly:

but in-game it gets rendered as:

Using this in the config:
ingame_discordMessage = "§6[§5DISCORD§6]§r<%user%> %msg%"

Was using the latest "release" (2.4.7), but also tried the latest "beta" (2.5.0) to the same result.

This happens when a mod (like FTB Teams) messes with chat -- see #5

closed as duplicate of #5


[REQ] bloxstrap icons

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
where you can use all bloxstrap icons like v2.8.0 bloxstrap icon (old bloxstrap logo is already usable)

the icons are located in appearance

 the icons are located in appearance

doesnt have new one.

which mean do you mean

I've tried adding Bloxstrap icon. I ran into rendering issues. I'll try readding it next update.

I've tried adding Bloxstrap icon. I ran into rendering issues. I'll try readding it next update.

oh

file for ico bloxstrap icon
https://www.mediafire.com/folder/99bd1doa4rike/Photos

file for ico bloxstrap icon https://www.mediafire.com/file/fdsem9eslys7eqc/Bloxstrap.ico/file

I'm going to directly get it from Bloxstrap repository.

file for ico bloxstrap icon https://www.mediafire.com/file/fdsem9eslys7eqc/Bloxstrap.ico/file

I'm going to directly get it from Bloxstrap repository.

i did

https://raw.githubusercontent.com/bloxstraplabs/bloxstrap/refs/heads/main/Bloxstrap/Bloxstrap.ico

Yea I know thanks for contributing.

woahh

fishstrap dont like my  physics fps to be 0

How did you manage to get 0 physics fps?

fflags.
{
"DFIntMaxMissedWorldStepsRemembered": "-1000"
}

Please don't post "exploit" fflags here. Also setting physics fps to 0 is going to mess up some stuff.

Please don't post "exploit" fflags here. Also setting physics fps to 0 is going to mess up some stuff.

love the "exploit" bit there, kinda tells you everything abt fflags

if you don't have discord disable the feature


Creating a gallery view

Contributing.md needs some work, but so far the base is running off parcel
the view uses Fancybox and has several online images posted inside.
Click once to select the photo to view, and click it again to bring it to full size.
You can also drag the photos around when selected.
The navbar will also follow when scrolling down.
The username is used as a placeholder for now nothing there yet
the About page is not there yet
test website

Which version of fancybox is this site talking about? because I know versions 1, 2, and 3 are now discontinued, but nothing is ever said about version 4.

https://www.npmjs.com/package/@fancyapps/fancybox hasn't been updated in 4 years, I'd say it's dead.  Also, it's based on jQuery, which isn't a good choice in 2022.

I just changed it to photoswide and parcel should be 2.0 and above now. Its now 100% html and file has been renamed to index.html

@LostButton do you want to wait to push until you're done?  I'm getting a lot of notifications on this PR.

@LostButton do you want to wait to push until you're done? I'm getting a lot of notifications on this PR.

the thing is its working perfectly when running locally, I'm having a hard time running it on github pages

What does "running it on GitHub Pages" mean? You don't run code on GitHub Pages.

it should be working on github pages as intended now

I left some comments on your PR.
When I click on "test sample," it redirects me to https://lostbutton.github.io/my-photohub/gallery, which renders a 404 error.
When I navigate to https://lostbutton.github.io/my-photohub, it seems to work fine. Maybe you are deploying the website to the root path instead of /gallery?

No that's not the case, I commented previously that when it was changed to index that https://lostbutton.github.io/my-photohub would be the link its deployed to. But I will look into your suggestions

Lastly, on mobile devices tapping on a single image, shows two in preview. See below:

Thanks for getting us started on this! I love that you did a demo site to show how it runs.
My first initial feedback:

fancybox doesn't seem to be maintained any more.  Looking at https://npmtrends.com/fancybox-vs-lightgallery-vs-lightgallery.js-vs-photoswipe-vs-swipebox it seems that https://photoswipe.com/ is the one that more people use, or https://github.com/sachinchoolur/lightGallery.  I think we should use one of these.
Add this library to the dependencies of package.json vs. asking the user to install it manually
We won't have accounts or search, so you can remove both of those from the header
Blue text on black background is hard to read
I'd probably rename gallery.html to index.html so it resolves better
Use a newer version of Parcel (2.x)
Can we get rid of the JS in gallery.js and make it 100% static HTML?

Prof, for number 7 what do you mean get rid of js to make it 100% static HTML?

Prof, for number 7 what do you mean get rid of js to make it 100% static HTML? I know its index now but the removing of JS still confuses me.

Ideally we should produce a static HTML page that has everything rendered in our build step vs. having to build the DOM dynamically on load. It will be way faster if we do.

For number 7, I can't remove the JS because the photoswipe relies on the modules inside. Unless you want me to write the script within the HTML file?

I don't mean remove all JavaScript.  I mean let's do as much in HTML as possible vs. having to set everything up at runtime with JS.  It's fine to augment that with JS afterward.

I don't mean remove all JavaScript.  I mean let's do as much in HTML as possible vs. having to set everything up at runtime with JS.  It's fine to augment that with JS afterward.

So would a better option would be to script what i have within the html file?