[improvement]Use std::iota to set values of _block_rowids in SegmentIterator::_read_columns_by_index

Proposed changes
Issue Number: close #xxx
Problem Summary:
std::iota is faster than for loop.
ssb-flat 100g q2.3  reduced from 1.5s to 1.2s
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

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

Hi @mrhhsg , the underlying implementation of std::iota is also a loop. Why does this modification take effect?
template <class _ForwardIterator, class _Tp>
inline _LIBCPP_INLINE_VISIBILITY
void
iota(_ForwardIterator __first, _ForwardIterator __last, _Tp __value_)
{
    for (; __first != __last; ++__first, (void) ++__value_)
        *__first = __value_;
}

Hi @mrhhsg , the underlying implementation of std::iota is also a loop. Why does this modification take effect?
template <class _ForwardIterator, class _Tp>
inline _LIBCPP_INLINE_VISIBILITY
void
iota(_ForwardIterator __first, _ForwardIterator __last, _Tp __value_)
{
    for (; __first != __last; ++__first, (void) ++__value_)
        *__first = __value_;
}

Maybe the original for-loop cannot be vectorized.

Hi @mrhhsg , the underlying implementation of std::iota is also a loop. Why does this modification take effect?
template <class _ForwardIterator, class _Tp>
inline _LIBCPP_INLINE_VISIBILITY
void
iota(_ForwardIterator __first, _ForwardIterator __last, _Tp __value_)
{
    for (; __first != __last; ++__first, (void) ++__value_)
        *__first = __value_;
}

Maybe the original for-loop cannot be vectorized.

I think it is better to provide some evidences to prove it.

https://github.com/apache/doris/pull/9111
How about test this pr?  Let's see which is faster.

#9111 How about test this pr? Let's see which is faster.

@wangbo Great!  It is the same with std::iota


Scatterplot Widget

This PR introduces a scatterplot widget to napari-spatialdata, which is able to visualise points in 2D coordinates.
Related issue: https://github.com/scverse/napari-spatialdata/issues/3
Contributors: @rahulbshrestha @tothmarcella @ganesh25

@rahulbshrestha there is also a test for the logger failing, cool that it's catching this!

Codecov Report
Base: 78.00% // Head: 66.34% // Decreases project coverage by -11.65% :warning:

Coverage data is based on head (1d74237) compared to base (c1f598a).
Patch coverage: 26.49% of modified lines in pull request are covered.

:exclamation: Current head 1d74237 differs from pull request most recent head c84b2a9. Consider uploading reports for the commit c84b2a9 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #26       +/-   ##
===========================================
- Coverage   78.00%   66.34%   -11.66%
===========================================
  Files           8        8
  Lines         823     1049      +226
===========================================
+ Hits          642      696       +54
- Misses        181      353      +172

Impacted Files
Coverage Δ

src/napari_spatialdata/_utils.py
68.22% <ø> (-1.15%)
:arrow_down:

src/napari_spatialdata/_view.py
46.21% <10.74%> (-33.34%)
:arrow_down:

src/napari_spatialdata/_widgets.py
69.78% <43.36%> (-11.59%)
:arrow_down:

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

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

I'd like to quickly try it out but can't rn, will do after meeting. After that, I'll merge!

trying it out, super nice, color renders really nicely.
One thing, when categorical is chosen, the colormap is not updated (should be removed) and the legend is not visible:

with continuous colors though works well!
one thing, it'd be nice to put the color variables (e.g. obs: row_num here) in the title of the plot, so that the info for the variable used to color is also present (the x and y axis labels looks great btw)

another funny thing I noticed is that as I continue changing variables for axis and colors, the scatterplot keeps getting narrower to a point were the dots can't be distinguished anymore. Really weird not sure what could cause it

finally, not sure I mentioned before but this bit for getting the colors that reassign the vec value to anndata should be changed
https://github.com/scverse/napari-spatialdata/blob/aec224238a3a9e9df93f7fb977f6a838262bedd2/src/napari_spatialdata/_utils.py#L86-L89
my understanding is that it is needed in order to then assign the categorical color in un in the following function call.
If that's the case, then couldn't just the vector be passed to that function and the colormap could be then just generated from it? In other words, existing anndata entries should never be modified in place (or even as copy) by the widgets. Let's discuss today alternatives to current behaviour.

@rahulbshrestha @tothmarcella I'll go ahead and merge this. There is a fair bit of refactoring in handling categoricals that we will have to do but for another PR.
Great work!


Feature add solidity method run

I need more args to run my contract's method or deploy my contract.

I add a solidity method selector component to implement method selection in solidity.
I add constructor info when compiling the contract.  so that we can add more args in deploying the contract.

Other features or Bug fixes:

Fix call value not work in deploying the contract. Because we need to update the worker's onmessage method when the call value changed.
Add opcode id in opcode window. when the opcode is jump or jump, we need to know where the code will run in the next step.

Some Component Preview:

Thank you for contributing we will look into this PR shortly!

peroket

Yes, I think your point is right. I think we may need more discussion to solve these problems.

The run need to be split into two methods, compiler`` and run```.
The arguments format is args1,args2,args3, this format is the same as the remix.
Several contracts support.

May you give me some thought about these three problems?
These bugs I will fix:

run method is not work correctly.

Next, I will split the opcode id to another pr.

peroket

Yes, I think your points are right. I think we may need more discussion to solve these problems.

The run need to be split into two methods,  compiler and  ``run```.
The arguments format is args1,args2,args3, this format is the same as the remix.
Several contracts support.

May you give me some thought about these three problems?
These bugs I will fix:

run method is not work correctly.

Next, I will split the opcode id to another pr.
Thank you.

I'm not really good at UI, but I don't think we can just add buttons in the current one. Each windows are already pretty small, and adding buttons is just taking more space. So probably to add all that the UI would need to be revamped, like for example having a different UI for writing code and for running a transaction. That would make the first point more logical as well, and third easier maybe.
For the arguments, do you have a link? That should be available from the UI as well. The types supported, the way to write the data...
Of course I'm talking ideally what I would like to have :-)

We can use ethereumjs-abi to convert the method's signature and the arguments as follows:
When we have a contract with a method inc that needs two arguments which type are both uint256.  So we can use abi.simpleEncode("inc(uint256, uint256)", 12, 13) to encode this data to 0xe8a7201f000000000000000000000000000000000000000000000000000000000000000c000000000000000000000000000000000000000000000000000000000000000d.
In SolidityMethodSelector component, you can select inc() method. Then fill 12,13 into arguments input. The component use , to split the user's input. The pageshot as follows.

The ethereumjs-abi  support most of solidity argument types, such as: addresss, bool, uintN, intN and [] etc.

I agree with your point that the window is small enough. we can't add more buttons to this page.
How about adding some Dialog or Modalon this page, then we can let the users select contract, select method, change argument, and change value?
If we add a new playground, users may feel confused about the design.
Now, we can follow the step to run a solidity method:

select solidity code type.
Then, run the contract
If the contract constructor has some arguments, we need to add the argument. And click the run button again.
Next, click the  step into button or continue execution button. It will deploy the contract.
Next, we can choose a contract method, and put the argument or the value
Next, click run method button
Finally, click the  step into button to debug your method.

After deploying the contract, the transaction will return the opcodes about the contract methods. The opcodes can be used to call ```loadInstructions``.

I think these are so complex that I have no idea about cutting down some steps. Do you have some ideas?

Yeah, I guess one way would be to add an "Advanced" option that opens up a modal/dialog box with these additional settings, another option is to have that advanced settings expand over the console and then minimize when you run/click minimize.
A VERY UGLY illustrative sketch, that way you have more room without another dialogue box/modal, I don't think the console is that useful anyway before you run anything.

also @thevaizman might have ideas here

This is a good idea, I will do it
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Dor Levi @.>
发送时间: Saturday, June 18, 2022 11:35:50 PM
收件人: comitylabs/evm.codes @.>
抄送: jonahcui @.>; Author @.>
主题: Re: [comitylabs/evm.codes] Feature add solidity method run (PR #148)
Yeah, I guess one way would be to add an "Advanced" option that opens up a modal/dialog box with these additional settings, another option is to have that advanced settings expand over the console and then minimize when you run/click minimize.
A VERY UGLY illustrative sketch, that way you have more room without another dialogue box/modal, I don't think the console is that useful anyway before you run anything.
[Screen Shot 2022-06-18 at 11 32 17 AM]https://user-images.githubusercontent.com/1457950/174445611-6fb12a79-83c1-4e77-87dd-0b09783ebfbc.png
also @thevaizmanhttps://github.com/thevaizman might have ideas here
―
Reply to this email directly, view it on GitHubhttps://github.com/comitylabs/evm.codes/pull/148#issuecomment-1159484783, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYIQURRRGFL6YXPWRZMALFTVPXUFNANCNFSM5Y5765MQ.
You are receiving this because you authored the thread.Message ID: @.***>

@peroket @dorlevi
Hello, everyone. I had made a solution through our discussion. I agree with @peroket that we don't need more complex buttons in the window. So I make three scenarios, each scenario has one master button in the window.
Fist, Simple Mode:

When I haven't compiler my contract, the window will show the run button as same as current.
Second, when the user compiles a contract whose constructor has more arguments, I show an error in the console window. As follows:

Then, when I click the advance mode  button, I will hide the run and show the  run method bar.

the button text is Deploy because the contract had not been deployed. I will select the constructor for user automatic.

After deploying the contract, the user can choose another method to run or change the mode to Simple Mode,

@peroket @dorlevi
Hello, everyone. I had made a solution through our discussion. I agree with @peroket that we don't need more complex buttons in the window. So I make three scenarios, each scenario has one master button in the window.
Fist, Simple Mode:  When I haven't compiler my contract, the window will show the run button as same as current.
Second, when the user compiles a contract whose constructor has more arguments, I show an error in the console window. As follows:
Then, when the user clicks the advance mode  button, The run bar will be hidden, and the  run method will be shown.

the button text is Deploy because the contract had not been deployed. I will select the constructor for user automatic.

After deploying the contract, the user can choose another method to run or change the mode to Simple Mode,
I will push my code after our discussion. Looking forward to your reply.

This is already a big improvement thanks :-)
However I don't see anything that deals with several contracts deployed in your solution. Do we explicitely accept to have only the last deployed that can be interacted with?
Also, I don't see how it solves the limited space problem, can you explain further? I don't really like the idea of modal windows, because they become cumbersome really quickly.
Feel free to push your code so that we can play with it :-)

I'm sorry, I didn't solve the several contract problem. I think several contracts are not a common scenario. But I can add a contract select component in the method select row. If we need to solve it.
I had added the advanced mode which is different from the current run mode. The advance mode is hidden in the run bar and creates new three rows in the editor window.

First row used to be select method, then the user can click deploy or run method
The second row used to be setting values like the input in the run bar.
The other rows used to be setting the argument by the input that shows the argument type
The bar only shows three rows once, if the contract has more arguments, the scroll bar will be shown.
The bar's height is 52 * 3 (run bar's height is 52)

I pushed my code yesterday.

Hello, all, I had added a contract select. like this:

If we don't like the scrollbar in Advance Mode, I think we can change the arguments input to a single input, like this:

arguments joins with ,
Would you like to give me some thoughts about the solution?

This is looking better again thanks :-)
My first comment would be that I find it weird that I can't use advanced mode before I click on "run". I know If my contract needs arguments, so I should probably be able to go to advance mode directly.
The logs also give me errors when I deploy in advance mode, even though I put a value and it launched the transaction. And it printed the same error 3 times for 1 click ("The constructor of the contract need arguments...").
And running the transaction to the end again prints the error countless time, even though it did deploy the contract.
Then I ran the "count" method, which ran fine, but after that all the methods are grayed out and I cannot run anything anymore.
I then modified the contract to add two constructor parameters, and changed the name of the contract. I was able to deploy it even though I didn't give any parameter to the contructor, the ui did not update to ask me two parameters, and the new contract did not appear in the contract list. Plus again a lot of error printings.
So I'm sorry, but it looks like it is nowhere near ready to be deployed. Interface wise though, What I could test works well, and I don't think we need anything more for a simple tool like ours anyway, so nothing to do there. Would be nice to have similar features for the other mode (yul...), except there it would simply be select contract address and enter the calldata manually. Also when I am in advanced mode and swith to another language, it does not reset the UI.
I know it's a lot of efforts to do all of that, so don't worry if you can't/dont't want to finish it :-)

I think the multiple contract support and contract method arguments make the UI more complex. And I will complete this feature with test all scenarios. But I don't think the feature is not necessary, because we need to show the users how EVM works with arguments when the users are similarly with solidity.
And I think if we add a  page that shows more code examples like Solidity by example or other code example pages. The evm.codes might be the best app for evm beginners.
I will fix all those bugs and think of a better solution to cut down the confusion for users.

deploy contract need to click the deploy button and then click the continue execution button, because some users may like to see how does evm deploy a contract. Maybe we need a better solution to do it.  But if I just want to run a method of the contract, it's too complex. Any thoughts on it?

I will try it, but if you have other ideas that need to be implemented, I'll be there. I can do it.

For the mutliple contract agreed, I'm fine explicitely supporting only the last deployed contract. I was not asking to support several, just to make a choice that's what we do.

OK, I will fix all the bugs and change some logic to make it more simple.

I'm sorry that I hadn't found an excellent way to make users deploy contracts simpler. So I won't close this pr, if some day we find a perfect way to solve the problem, we can try it again.

If you want to run a contract method, you need to deploy the contract and click the continue execute, then the contract will be deployed successfully. Next, you can run the method that you need.
The process is so confusing. But I didn't know how to make it simpler.

Relax, I have fixed some bugs, and I will fix the other bugs tomorrow. I will push the branch after all the bugs are fixed. Then we can test it together.

But I want to solve the problem of how to deploy a contract.

@peroket @dorlevi

I found a method to make users don't feel confusing. I add a compile option in method selection. And if you change to the advance mode when the user has not compiled the contract before, I will autoselect compile option for the user.
The Simple Mode:

If the user clicks run, I will compile the contract first, then deploy it.
When the user contract has arguments, the console will show revert after the user debug the transaction.

From The Simple Mode To The Advanced Mode:

If the user has not compiled the contract, I will autoselect the compile option for the user.
If the user compiles the contract before, I will autoselect the constructor for the user.
If the user has deployed the contract before. I will autoselect the constructor for the user. We can talk about whether to help users choose the first method.

The Advanced Mode:

The users can choose the compile option to compile the code.
The users can choose the constructor option to deploy the code.
The users can choose a method to run. But users must deploy the code first.

@peroket @dorlevi

I found a method to make users don't feel confusing. I add a compile option in method selection. And if you change to the advance mode when the user has not compiled the contract before, I will autoselect compile option for the user.
The Simple Mode:

If the user clicks run, I will compile the contract first, then deploy it.
When the user contract has arguments, the console will show revert after the user debug the transaction.

From The Simple Mode To The Advanced Mode:

If the user has not compiled the contract, I will autoselect the compile option for the user.
If the user compiles the contract before, I will autoselect the constructor for the user.
If the user has deployed the contract before. I will autoselect the constructor for the user. We can talk about whether to help users choose the first method.

The Advanced Mode:

The users can choose the compile option to compile the code.
The users can choose the constructor option to deploy the code.
The users can choose a method to run. But users must deploy the code first.

@peroket @dorlevi
I found a method to make users don't feel confusing. I add a compile option in method selection. And if you change to the advance mode when the user has not compiled the contract before, I will autoselect compile option for the user.

The Simple Mode:

If the user clicks run, I will compile the contract first, then deploy it.
When the user contract has arguments, the console will show revert after the user debug the transaction.

From The Simple Mode To The Advanced Mode:

If the user has not compiled the contract, I will autoselect the compile option for the user.
If the user compiles the contract before, I will autoselect the constructor for the user.
If the user has deployed the contract before. I will autoselect the constructor for the user. We can talk about whether to help users choose the first method.

The Advanced Mode:

The users can choose the compile option to compile the code.
The users can choose the constructor option to deploy the code.
The users can choose a method to run. But users must deploy the code first.

Thanks, I have finished these bugs:

Remove the default constructor in the example of solidity.
Can not click run when returning from advance mode.
The  run method with tips has been changed to three tips:

if the method has arguments and the user inputs arguments, then the console will show run method with arguments.
if the method has no arguments and the user inputs arguments, then the console will show Ignore arguments
if the method has no arguments and the user doesn't input arguments, then print run method.

Add width for inputs. And add a title for the arguments line.
Change text-transform to 'none' for the method name in select.
Add "Share permalink" button after method selected.

Some question:

Whether we need to scroll to the bottom for the console after pushing a log?

Scrolling to the bottom would be very convenient yes, if you can do that I would be grateful :-)

Thanks @jonahcui as a user looks great :-)
Will get a technical review soon as well.

I try to add a  running state to implement the UI suggestion about button feedback. Buut the run time is so short that the user can't see the button style changed.
Finally, I added a active:opacity-50 to the button's class name.

I have fixed some bugs, and I will fix the other bugs tomorrow. I will push the branch after all the bugs are fixed. Then we can test it together.
But I want to solve the problem of how to deploy a contract.
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Dor Levi @.>
发送时间: Thursday, June 30, 2022 6:58:29 PM
收件人: comitylabs/evm.codes @.>
抄送: JonahCui @.>; Mention @.>
主题: Re: [comitylabs/evm.codes] Feature add solidity method run (PR #148)
@jonahcuihttps://github.com/jonahcui : We appreciate the work here and understand you want to merge it (or do you mean abandoning and closing it?) We can punt on the ease of use of deploying a contract, but can you take care of some of the bugs mentioned earlier by @perokethttps://github.com/peroket ? at this stage, I don't believe it's not ready to review/merge; it's a slippery slope to merge stuff that is not entirely up to the standard of experience we would want for the users of evm.codes _and as the branch is automatically deployed by Vercel so if someone wants to use this functionality before it's merged they can do that easily so we rather not merge it until some of the larger bugs are addressed.
If you can't address the bugs, that is fine, you've done a lot already -- we are soon onboarding additional FE engineers to our team (evm.codes is not our main product) and have them take it from here when they start in a couple of weeks.
―
Reply to this email directly, view it on GitHubhttps://github.com/comitylabs/evm.codes/pull/148#issuecomment-1171074338, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYIQURXM3VYLLHILTEU3PH3VRV4VLANCNFSM5Y5765MQ.
You are receiving this because you were mentioned.Message ID: @.***>

Got an argument list as follows:
solution(uint256[2][3],uint256[2][3])

Can't figure out the correct way to supply the arguments. What works with Remix - [[1, 1], [1, 1], [1, 1]], [[2, 6], [0, 10], [3, 3]] - doesn''t work with EVM.codes Playground


[serverless-1.29.0] devconsole nodejs pipeline hack

hack to bypass double sha256 error on pipeline. Definitive expected fix on 1.30

This change fixes the following error on dev console pipeline (create serverless function page) at func-deploy task execution
Error: knative deployer failed to deploy the Knative Service: admission webhook "validation.webhook.serving.knative.dev" denied the request: validation failed: Failed to parse image reference: spec.template.spec.containers[0].image
image: "image-registry.openshift-image-registry.svc:5000/test488/git-hello-node-func-git@sha256@sha256:20235eb3183b2ae5c64d4c4333dbd1e90d88dc2355be64d8e7a0456ecad12c1f", error: could not parse reference: image-registry.openshift-image-registry.svc:5000/test488/git-hello-node-func-git@sha256@sha256:20235eb3183b2ae5c64d4c4333dbd1e90d88dc2355be64d8e7a0456ecad12c1f

/lgtm


panic when MaxSessions is 1

If MaxSessions is set to 1, the following panic occurs:
panic: invalid/out of range capacity

goroutine 1 [running]:
github.com/thales-e-security/pool.NewResourcePool(0xc00629c030, 0x0, 0x0, 0x0, 0x0, 0x0)
	/Users/cbroglie/.../vendor/github.com/thales-e-security/pool/resource_pool.go:87 +0x481
github.com/ThalesIgnite/crypto11.Configure(0xc0000caba0, 0x0, 0x0, 0x0)
	/Users/cbroglie/.../vendor/github.com/ThalesIgnite/crypto11/crypto11.go:327 +0x4bd

This is due to:
// We will use one session to keep state alive, so the pool gets maxSessions - 1
instance.pool = pool.NewResourcePool(instance.resourcePoolFactoryFunc, maxSessions-1, maxSessions-1, 0, 0)

Thanks for flagging this.
A straight-forward fix for this is to document that the actual number of sessions opened is one greater than MaxSessions. Then we can drop the -1 from the offending code line.
@cbroglie  Do you have any use cases where only 1 session is available? If so, we may need to more thoroughly redesign this part of the library to handle that case.

No, we haven't come across any HSMs which support exactly 1 session. Simply documenting the behavior would be fine for us.

Documentation updated. The library will also return an error rather than panicking now. Release 1.2.1 contains this fix.


Show message when input value is not what expected

When the user enters a non valid value in the account forms inputs the interface should show a message to inform about the errors.
Examples to fix:

[ ] Twitter username accepts any string (pattern should be "@" + username).
[ ] At the account form, when user erases the email value and saves the changes a message like "You can't leave empty your email" should be displayed. Currently an "everything's ok" message is displayed and the email doesn't change.

CC/ @javierarce @xavijam

Seems like the error validating email comes from EmailAddressValidator in https://github.com/CartoDB/cartodb/blob/master/config/initializers/rfc822.rb.

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


fix: ci

#2  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @GoodM4ven and the rest of your teammates on  Graphite


flex: 1 changes the height of the View

Is this a bug report?
No

Have you read the Contributing Guidelines?
Yes
Environment

Steps to Reproduce

flex:1 changes the height of the View container (I am using onLayout prop on the View)

Expected Behavior

The onLayout prop of the View had to be returned the actual height of the mobile (namely it should be the same as Dimensions.get('window').height)
Actual Behavior
At first it returns Dimensions.get('window').height and then returns Dimensions.get('window').height minus 25px

I want to get the mobile height using onLayout prop of container View. The returned data is 25px less than the original height of the mobile (saying original I meant SCREEN_HEIGHT = Dimensions.get('window').height). Actually I need onLayout prop to calculate children View height, but the investigation showed that I receive the issue as described above. Please help to solve the problem.
I also attach the log of 'SCREEN_HEIGHT' and 'event.nativeEvent.layout': https://prnt.sc/gofryl
Reproducible Demo

https://gist.github.com/anonymous/349ce590d6a7a9a1da607fa3932792db

Short Answer:
There is no problem here, you only did not notice NavBar.
Complete Answer:
The purpose of flex:1  is to:
To specify the length of the item, relative to the rest of the flexible items inside the same container.
Consequently, when you use flex:1 on a View, the expected behaviour is to fill the remaining area of the page proportionately.
For example, if you have two Views in the page,
1- header component with specefic height (say, 25px)
2- the View of the page, with flex:1,
The View is going to fill all the page below header, Therefore when you use onLayout to get dimension of the View, you expect:

the height to to be 25 px less than  Dimensions.get('window').height,
and the width to be equal to  Dimensions.get('window').width

which is what happens in your case.
If  you delete the NavBar component from your page (and its View) by writing flex:1 for the other view the result will be equal to  Dimensions.get('window').height.

Thanks for your answer. But unfortunately NavBar height is not problem here. For example, if even I take simple View (not contatining any items) and gives {flex: 1} style to this View, then onLayout prop at first returns 592px height, and after several milliseconds it returns 567px (namely 592px - 25px). This data was taken for my mobile. I also had tested on genymotion mobile, and in this case again I receive 2 heights: one is the same as Dimensions.get('window').height, and the second one is less than that value (but in this case it is less with 24px). So I think, these calculations are based on mobile real height.

Yes you are right, the onLayout should give total the hight of specefic View (i.e. total height of page minus the hight of NavBar), but I don't receive this. Namely, if Dimensions.get('window').height = 592px, and NavBar height is 70px, so onLayout prop at first returns 522px (592px-70px) AND after this it returns 497px (522px-25px). To note, my container View contains only 2 Views: one for NavBar and second View, for which I want to apply onLayout prop


Build success with invalid credentials

If I don't set the user and key for plugin, the build is still marked as successful, and nothing is uploaded to bintray.

Just ran into the same issue and spent quite a while to figure that out. Please fix.


Fix tests and update examples (do not merge yet)

I am going through and fixing tests, mostly in nei.py and eigenvaluetable.py.  I am also about to make a few changes to how things are set up for Travis CI, so I'll need to keep this pull request open to make sure the test settings are correct.

Hello @namurphy! Thanks for submitting the PR.

In the file nei/classes/eigenvaluetable.py, following are the PEP8 issues :

Line 281:5: E303 too many blank lines (2)

In the file nei/classes/nei.py, following are the PEP8 issues :

Line 91:18: E251 unexpected spaces around keyword / parameter equals
Line 91:20: E251 unexpected spaces around keyword / parameter equals
Line 92:20: E251 unexpected spaces around keyword / parameter equals
Line 92:22: E251 unexpected spaces around keyword / parameter equals
Line 186:80: E501 line too long (88 > 79 characters)
Line 187:80: E501 line too long (90 > 79 characters)
Line 381:80: E501 line too long (104 > 79 characters)
Line 452:20: E251 unexpected spaces around keyword / parameter equals
Line 452:22: E251 unexpected spaces around keyword / parameter equals
Line 470:80: E501 line too long (85 > 79 characters)
Line 630:39: E225 missing whitespace around operator
Line 913:80: E501 line too long (82 > 79 characters)
Line 969:80: E501 line too long (89 > 79 characters)
Line 981:5: E303 too many blank lines (2)
Line 1031:80: E501 line too long (84 > 79 characters)
Line 1035:80: E501 line too long (86 > 79 characters)
Line 1077:5: E303 too many blank lines (2)
Line 1187:80: E501 line too long (83 > 79 characters)
Line 1191:57: E225 missing whitespace around operator
Line 1192:9: E122 continuation line missing indentation or outdented
Line 1193:9: E122 continuation line missing indentation or outdented
Line 1197:1: E302 expected 2 blank lines, found 1
Line 1214:80: E501 line too long (82 > 79 characters)
Line 1220:9: E265 block comment should start with '# '
Line 1224:9: E265 block comment should start with '# '
Line 1231:9: E265 block comment should start with '# '
Line 1231:9: E303 too many blank lines (2)
Line 1236:9: E303 too many blank lines (3)
Line 1238:74: E231 missing whitespace after ','
Line 1238:80: E501 line too long (81 > 79 characters)
Line 1239:36: E231 missing whitespace after ','
Line 1239:62: E228 missing whitespace around modulo operator
Line 1239:80: E501 line too long (85 > 79 characters)
Line 1242:80: E501 line too long (80 > 79 characters)
Line 1246:70: E231 missing whitespace after ','
Line 1247:32: E231 missing whitespace after ','
Line 1250:63: E228 missing whitespace around modulo operator
Line 1250:77: E231 missing whitespace after ','
Line 1250:80: E501 line too long (82 > 79 characters)
Line 1255:80: E501 line too long (80 > 79 characters)
Line 1262:80: E501 line too long (90 > 79 characters)
Line 1273:14: E225 missing whitespace around operator
Line 1279:13: E303 too many blank lines (2)
Line 1282:73: E231 missing whitespace after ','
Line 1282:80: E501 line too long (91 > 79 characters)
Line 1282:91: E502 the backslash is redundant between brackets
Line 1283:25: E127 continuation line over-indented for visual indent
Line 1293:76: E231 missing whitespace after ','
Line 1293:80: E501 line too long (103 > 79 characters)
Line 1311:1: W391 blank line at end of file

In the file nei/classes/tests/test_eigenvaluetable.py, following are the PEP8 issues :

Line 8:1: E265 block comment should start with '# '
Line 8:80: E501 line too long (80 > 79 characters)
Line 10:1: E265 block comment should start with '# '
Line 10:80: E501 line too long (80 > 79 characters)
Line 14:1: E265 block comment should start with '# '
Line 16:1: E265 block comment should start with '# '
Line 17:1: E302 expected 2 blank lines, found 1
Line 23:54: E261 at least two spaces before inline comment
Line 23:80: E501 line too long (94 > 79 characters)
Line 30:25: E231 missing whitespace after ','
Line 46:1: E265 block comment should start with '# '
Line 47:1: E302 expected 2 blank lines, found 1
Line 73:1: E302 expected 2 blank lines, found 1
Line 110:1: E302 expected 2 blank lines, found 1
Line 123:17: E261 at least two spaces before inline comment
Line 124:17: E261 at least two spaces before inline comment
Line 137:18: E261 at least two spaces before inline comment
Line 152:1: E302 expected 2 blank lines, found 1

In the file nei/data/read_save_ir_rate.py, following are the PEP8 issues :

Line 13:1: E265 block comment should start with '# '
Line 50:23: E231 missing whitespace after ','
Line 54:9: E265 block comment should start with '# '
Line 54:80: E501 line too long (80 > 79 characters)
Line 55:61: E231 missing whitespace after ','
Line 59:9: E265 block comment should start with '# '
Line 60:9: E265 block comment should start with '# '
Line 71:23: E231 missing whitespace after ','
Line 75:9: E265 block comment should start with '# '
Line 75:80: E501 line too long (80 > 79 characters)
Line 76:61: E231 missing whitespace after ','
Line 80:9: E265 block comment should start with '# '
Line 81:9: E265 block comment should start with '# '
Line 98:19: E231 missing whitespace after ','
Line 98:44: E231 missing whitespace after ','
Line 99:19: E231 missing whitespace after ','
Line 99:44: E231 missing whitespace after ','
Line 101:8: E225 missing whitespace around operator
Line 105:1: E265 block comment should start with '# '
Line 107:1: E265 block comment should start with '# '
Line 108:1: E265 block comment should start with '# '

I finally got the tests to pass again on Travis CI, though I'm still having some problems with test_nei_class.py and test_eigenvaluetable.py.  The main problem is that the tests time out when the tests include heavier elements like iron.  The heavier elements work fine when running the tests on my workstation locally, but for some reason take a lot longer on Travis.  I had started to think it may have been memory limitations, but the data files are each ~5 MB so I'm not sure.

Closes #32.  Closes #29.   <-- These statements tell GitHub to close those issues once the current issue is merged.


Fix for using jobber_mobile repo with dependabot

We were using the wrong repo when getting dependabot warnings for jobber-mobile.
This PR also removes the hack put in place to work around a github API bug.

@naarok Was the Jobber Mobile repo typo the main reason for the sheet not getting updated regularly?
Also, are we only looking at Jobber Online and Jobber Mobile at the moment?


token-cli: Feature Request: Wrap SOL as Associated Token Account in One Command

If I already have an associated token account in my wallet, I have to run spl-token wrap 1 --create-aux-account and then spl-token gc, two commands.
It would be nice if the CLI can provide one command for this.
Thanks! :)

A way to do this in one transaction:

transfer SOL into the account, ie solana pay 1 <ASSOCIATED_TOKEN_ADDRESS>
sync the native tokens to the account, ie spl-token sync-native
Perhaps the CLI can allow transferring from a SOL wallet into a wrapped account with a flag.

I would like to work on this !

Finally, I'm not sure if this is an important use-case to make a separate CLI instruction, considering it can easily be done with solana transfer and spl-token sync-native.
Otherwise, we could consider adding a top-up-native <AMOUNT> <DESTINATION> --from <SIGNER> command, but it seems too niche to me.


Testing authentication protections

Currently, Playwright e2e has been implemented for App Authentication Route which requires some configurations:
In client folder create folder client/playwright/.auth
Upload files into client/playwright/.auth from 1password
OBPS FE Auth Session cas_analyst
OBPS FE Auth Session cas_pending

Copy file client/e2e/.env.local.example to client/e2e/.env.local
Set client/e2e/.env.local values as detailed in file client/e2e/.env.local.example
Ensure the the example id used for "new user" in the e2e test is NOT created in the user table:
cd bc_obps && psql

 \c registration
registration=# DELETE FROM erc.user where user_guid='c3cd84a8-e261-4814-93e1-5bd4a7ccd638';

From terminal command, start the app development server:
cd client && yarn dev
From terminal command, run Playwright e2e tests:
cd client && yarn e2e
All tests should pass (24)

We have some (hopefully all) endpoint tests re auth

Blocked by #28

As part of this ticket we should also be checking to ensure that external users aren't able to change the status of their operator, operation, or prime admin access request themselves by posting a request using Postman, Thunder Client, or the like (i.e., bypassing the UI).
I haven't investigated this yet, but off the top of my head my concern is that we accept values for the "Status" field in the request body on PUT endpoints. We might need to add logic that removes any updates to the "Status" field for operators, operations, or prime admin when the request is coming from an external user.

No longer needed. Closing. CC @shon-button @andrea-williams


Update DeepFreeze

Add DeepFreeze Recommended mods to include optional mods that enhance
upcoming DeepFreeze update.
Added USI-LS to recommended life support mods.

You need to rebase in order to clear up the merge conflict (specifically with the unnecessary netkan epoch)


Corrects handling of tabs.onUpdated event

Updates the handling of the tabs.onUpdated event to eliminate duplicate notifications.
Fixes #93

Tested and working as expected on:

OSX in Chrome and FireFox
Windows in Chrome and FireFox


Bluetooth

Recognition doesn't work with Bluetooth headset

Why not,"Recognition doesn't work with Bluetooth headset" this explains a
lot of some of my frustration with this software I bought with a life long
ownership subscription, and FYI  I haven't be able to enjoy my lab top
version of product due to not reaction to my Laptop's mic's for speech
recognition? can u explain?
Is there going to be a fix to remedy this so i can keep program running and
still use my bluetooth
If not you should
Also who or where and how do I submit support tickets to my AI API
assistant that i purchased the life long package  to request improvement or
errors in programming so that the AI part can feel more not bullshit.  I
was promised a life long use of.  this program with unlimited request
allowed to be submitted i just never got the detail on where or how  can u
tell me please
I Have a few other issues with my assistant that need to be addressed and
fixed or i'm going to put a refund request for "product not as describe"
alternatively i would like this problems and non compatibility issues i'm
experiencing with my AI API assistant fixed so i may keep and enjoy the
benefits of intent.
Thank you,
Noah Patton<PHONE_NUMBER>
<EMAIL_ADDRESS>Contact me by cell @<PHONE_NUMBER> or email me at **<EMAIL_ADDRESS><EMAIL_ADDRESS>On Fri, Feb 6, 2015 at 10:43 PM, Danil Skachkov<EMAIL_ADDRESS>wrote:

Recognition doesn't work with Bluetooth headset
—
Reply to this email directly or view it on GitHub
https://github.com/api-ai/api-ai-android-sdk/issues/3.

Hi, @nsp4real!
This section is only intended to the discussions about Android API.AI SDK.
All questions related to Speaktoit Assistant please send to the support<EMAIL_ADDRESS>Thanks!


Python/Throwing Knife Price Tweak

About the PR

I changed the Python's price to 6TC and the Throwing Knives to 7
Why / Balance

Python isn't worth 8TC and Throwing Knives have AP alongside being relatively quiet and reusable. They should be sidegrades, but I view the knives as a little bit better.
Technical details

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: Python 8 -> 6TC
tweak: Throwing Knives 6 -> 7TC

Some folk have suggested tweaking Python ammo refill prices down too. Any thoughts?

Throwing knifes even with AP aren't worth more than 4-5 TC, making them 7 would kill them. They are a high skill item and I think we should aim to not nerf said items. If you made them 7 why would anyone ever get them over viper + 4 spare mags, which would vastly outclass the throwing knives. They also have sooo many downsides to actually trying to use them and are useless if you're against more than one person and very easy to lose.

How is an AP automatic revolver less value than knives?

I dont think throwing knives should be touched considering their required skill floor.

After I PR'd the buff, throwing knives are now at the perfect balance. They have A LOT of downsides and are generally not a great purchase unless in certain situations.

I like this change.

I like this change.

No but it would be way slower and gives more time for backup to arrive
Getting off-topic but if sec had instastuns (requiring reaction time faster than ping allows) we'd be seeing massive discourse over sec being too OP - thankfully we don't have soap or lube because that would break balance
Getting back on-topic, knives are okay for 6 tc - unlike a python where you can just instantly teleport more ammo at low price, you'd need to repurchase the knives for full cost to match its speed, so making it more expensive seems ofd

don't personally see the reasoning here, agree with the dissenting comments


Update filtered_stream.py

fixed the typo in the comment
Fixes: #63

I have read the CLA Document and I hereby sign the CLA

I have done the needful


Not able to remove the delay from flux script

Updating task by setting offset field to 0s or to and empty string (not sure if it is supported) will result in error:
cannot update task without content

I am assuming that setting offset to 0s should result in removing the delay option from flux script.

Create a task with delay option.

Request body:

{
  "flux": "option task = {name:\"task_urdfc\", cron: \"* * * * *\", delay: 15s} from(bucket:\"read_from_b_query_params\") |> range(start: -1h)",
  "orgID": "0376c42cfa80c000",
  "status": "active"
}

Update created above task with offset = 0s

Request body

{"offset": "0s"}

Response:

{
  "code": "internal error",
  "error": "cannot update task without content",
  "message": "failed to update task"
}

@gshif Per http://docs.internal/api/swagger/#tag/Tasks/paths/~1tasks~1{taskID}/get, taskid, name, and flux are all required fields.

This will be fixed by https://github.com/influxdata/influxdb/issues/12288.

@gshif I think this is already done.

I can see that offset is removed from the flux script, but offset is still being returned by the response object.

Create task:

requests.post(http://localhost:9999/api/v2/tasks, data={"orgID":"03c7daed2e6cb000", "flux":"option task = {name:\"task_urdfc\", cron: \"* * * * *\", offset: 15s} from(bucket:\"FIoDnyQuI7e0\") |> range(start: -1h)", "status":"active"}, headers={'Authorization': u'Token mHeEEfFCbIZ6mzNh-FFDJ2PrmUnj1w1W-L58yY-ycqUQJV1Qt18CAJC6Q_ViwMEJxWSFPeTAB7RYlU1-0drM0w=='})

Update task (remove offset from cron):

requests.post(http://localhost:9999/api/v2/tasks/03c7daed462cb000, data={"offset": "0s"}, headers={'Authorization': u'Token mHeEEfFCbIZ6mzNh-FFDJ2PrmUnj1w1W-L58yY-ycqUQJV1Qt18CAJC6Q_ViwMEJxWSFPeTAB7RYlU1-0drM0w=='})

It returns the following object
u'status': u'active',
u'latestCompleted': u'2019-04-30T18:07:41Z',
u'name': u'task_urdfc',
u'links': {u'runs': u'/api/v2/tasks/03c7daed462cb000/runs',
               u'logs': u'/api/v2/tasks/03c7daed462cb000/logs',
               u'self': u'/api/v2/tasks/03c7daed462cb000',
               u'labels': u'/api/v2/tasks/03c7daed462cb000/labels',
               u'owners': u'/api/v2/tasks/03c7daed462cb000/owners',
               u'members': u'/api/v2/tasks/03c7daed462cb000/members'},
u'labels': [],
u'cron': u'* * * * *',
u'flux': u'option task = {name: "task_urdfc", cron: "* * * * *"}\n\nfrom(bucket: "FIoDnyQuI7e0")\n\t|> range(start: -1h)', u'orgID': u'03c7daed2e6cb000',
u'offset': u'15s',
u'updatedAt': u'2019-04-30T18:07:41Z',
u'org': u'ZyFJ5LE0CJ0n',
u'authorizationID': u'03c7daed246cb001',
u'id': u'03c7daed462cb000',
u'createdAt': u'2019-04-30T18:07:41Z'

Thanks, I think I see what is happening.  Looks like a separate bug.  I will self-assign.

Investigate if this is still an issue

fixed in #14961

I am seeing the error failed to decode request: cannot update task without content when removing offset.

Create task with offset. Request body :

{"orgID":"512d2f5e662f3544", "flux":"option task = {name:\"task_urdfe\", every: 1m, offset: 15s} from(bucket:\"read_from_b_query_params\") |> range(start: -1h)", "status":"active"}'

Update the created task. Request body:

{"offset": "0s"}

Returned error:
{u'message': u'failed to decode request: cannot update task without content', u'code': u'invalid'}

I'm still seeing this issue:
I am seeing the error failed to decode request: cannot update task without content when removing offset.

Create task with offset. Request body :

{"orgID":"512d2f5e662f3544", "flux":"option task = {name:\"task_urdfe\", every: 1m, offset: 15s} from(bucket:\"read_from_b_query_params\") |> range(start: -1h)", "status":"active"}'

Update the created task. Request body:

{"offset": "0s"}

Returned error:
{u'message': u'failed to decode request: cannot update task without content', u'code': u'invalid'}


Install Windows 95

[x] Uninstall Wes2019
[ ] Desktop background
[ ] Icons
[ ] Start menu
[ ] Clippy
[ ] Notepad
[ ] Calculator
[ ] Paint
[ ] Internet Explorer 6
[ ] Minesweeper
[ ] Sounds
[ ] Boot

Closes #29

Using project and the branch instead.


Add tempest API tests for L7 content switching.

@pjbreaux
What's this change do?
Adds tempest API tests for L7 switching. Also, adds additional validation methods
to BigIpClient validator.
Where should the reviewer start?
test_l7policy_update.py
Any background context?

Looks good. We just need to have this all run in automation.


v1.3.6 locks

I maintain the cointop port on FreeBSD and I upgraded it to the latest version 1.3.6 released so far, on my system. After opening cointop, it seems to freeze after the data is shown on screen. Can not hit Enter, go up or down, or even quit.

Thanks for reporting the issue and maintaining the FreeBSD port! That's a strange bug, maybe something is causing an endless loop. I'll look into it when time is available

The issue is with the TUI library cointop is using (gocui). Planning on migrating to the TUI library tcell at some point since a simpler library and has no issues with keybindings.

The keybinding issue on freebsd was fixed on the dependency package. Will create a new release soon.


How to add sqlite3??

I tried to manually compile sqlite3 and add in in the project using nw-gyp as described in the sqlite3 installation instructions for node-webkit from here.
Many of the tutorial say that to use sqlite3, nw and angular together I have to approach like below
Angular -> Express -> sqlite3
Where Angular handles the UI part, using express I create an API and the api endpoints performs the database read write operations.
I created an express server in the project and started it by adding "node-main":"path/to/server-startup.js" in the package.json. Everything works fine until I add sqlite3. The server starts, I can call the api end points from Angular using HttpClient and many other stuff. But as soon as I add sqlite3 in my express app using nw-gyp, i get this error.

changes in package.json
"node-main": "api/index.js",
"dependencies": {
    "express": "^4.17.1",
    "sqlite3": "^4.2.0"
  }
"devDependencies:{
    "nw": "0.46.0-sdk",
}

Minimal code for api/index.js
const sqlite3 = require("sqlite3");

sqlite3.Database("../customeronly1.database", (err) => {
  if (err) {
    console.log(err);
  } else {
    console.log("connected");
  }
});

api folder is at the same level as package.json
Any help would be appreciated. Thanks

Closing, unrelated to this repo. You will need to do your own research.

https://github.com/nwjs/nw.js/wiki/Save-persistent-data-in-app
https://github.com/nwjs/nw.js/issues?q=is%3Aissue+sqlite3

Thanks for the links. However, I found a better-sqlite3 and used it in the project and everything went well. Posting this as a future reference for someone stuck on the same road.


fix: add missing fields to $$Props in FloatingLableInput

📑 Description
The $$Props in multiple components are missing some fields that are used by the component.
If a component uses a prop by the $$prop syntax it will work when rendering but it will produce typescript errors.
This issue exists for multiple components, not just this one.
✅ Checks

[x ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] I have checked the page with https://validator.unl.edu/
[ ] All the tests have passed (Idk, i barely got the tests to run because of missing env-variables and timeouts)
[x ] My pull request is based on the latest commit (not the npm version).

Summary by CodeRabbit

New Features

Enhanced the FloatingLabelInput component with new optional properties for custom class names (classDiv, classInput, classLabel), allowing for improved styling flexibility.

Documentation

Updated JSON configuration for the FloatingLabelInput component for better readability and included new properties in the documentation.

If you find more, please create another PR. Thank you for your contribution.


richColors works in dev but not in prod

Describe the bug
My Toaster looks like this:
<Toaster expand={true} richColors closeButton position="top-center" />
When I run npm run dev the richColors are shown, but when I use npm run build followed by npm run preview it shows the toasts without richColors.
EDIT: it seems the close button is in richColor:

Reproduction
Create Toaster component e.g. <Toaster expand={true} richColors closeButton position="top-center" /> build app and run in preview mode.
Logs
No response
System Info
System:
    OS: Linux 5.15 Debian GNU/Linux 12 (bookworm) 12 (bookworm)
    CPU: (12) x64 13th Gen Intel(R) Core(TM) i5-1335U
    Memory: 4.34 GB / 7.60 GB
    Container: Yes
    Shell: 5.2.15 - /bin/bash
  Binaries:
    Node: 21.6.1 - ~/.nvm/versions/node/v21.6.1/bin/node
    npm: 10.2.4 - ~/.nvm/versions/node/v21.6.1/bin/npm
  npmPackages:
    @sveltejs/kit: ^2.0.0 => 2.5.0
    svelte: ^4.2.7 => 4.2.10
    svelte-sonner: ^0.3.16 => 0.3.16

Severity
annoyance

Tried it on a fresh SK project and richColors in prod works as expected. Try to update your version!

Same issue, using 0.3.17

@Scooter1337  Ah, might need to update shadcn-svelte version. Would love if you can open a PR to shadcn-svelte updating sonner version!

Same issue, using 0.3.17
@LickABrick are you using shadcn-svelte? If I switch to the vanilla Toaster, the richColors work.

Ah yes I do, completely forgot it uses it's own Toaster component.

For some reason, in my Development build, the styles given by shadcn-svelte (they make you install 0.3.17) are overwriting the default styles.

While the exact same code (or so it seems) on shadcn-svelte's website does not. (they use 0.3.9, which I also tried, same problem on my side.) Very weird.

I added '!important' to all richColor styles to fix it temporarily. Any ideas? @wobsoriano

For some reason, in my Development build, the styles given by shadcn-svelte (they make you install 0.3.17) are overwriting the default styles.

While the exact same code (or so it seems) on shadcn-svelte's website does not. (they use 0.3.9, which I also tried, same problem on my side.) Very weird.

I added '!important' to all richColor styles to fix it temporarily. Any ideas? @wobsoriano

I think we need to move this to the Shadcn-Svelte repo as it seems to be an issue in their Toaster component.


Support for Companion Internal Variables

Describe the feature
Support custom variables in the cell and value fields.
Usecases
Dynamically change the cell or content that is being edited based on custom variables provided by companion.
One specific use case is to set a checkbox on each row to TRUE based on the number of times that the button has been pressed.
Another use case is to write dynamic content to a cell based on internal variables from companion.
Here's an example of what I would expect to work (it doesn't currently):

@Terranesaurus beat me to it!
I would like a similar function – for instance,  if "$(internal:time_hms)" was in the Value field, that the actual timestamp would be put into the Spreadsheet, such as 12:32:14.
Also, it would be helpful to have an Action that adds data to a new row, and also an Action that would generate a new sheet within a workbook.
For instance, I want to automate YouTube livestream chapter markers after a show. Each button press on Stream Deck during a show would create a new row below the most recent data and timestamp a cell (ie hence converting $(internal:time_hms)" to "12:32:14"). Ideally I can import dynamic Stream Deck button Text in an adjacent cell that would contain the description.

Support for variables is something that will definitely be added.
As for an action that adds data to a new row, the trouble I can see with this is if someone has a sheet with 10 rows of data, and then 10 empty rows, should the data be added to row 11, or should a row 21 be created and used?  What if on row 20 there was a single cell with data in some column  but the column for the timestamp is empty? It sort of gets complicated and so it may be easier to use custom variables to target a specific row/column and then increment the variable used to target a row.

I'm thinking of how Google Forms will add new submissions to a row at the bottom of a spreadsheet. Not sure how that works on the back end. Either way, some ability to press the same button in Companion and it adds the data to a new line, rather than overwriting the same field.

We would love to see the internal variable function to be implemented. We wanted to create an easy way to mark a specific timecode into an spreadsheet for the editor. Currently we just write it down on a piece of paper. The program we use for recording sends out a timecode to companion which we could then write into a custom variable and then write that variable to a new field in google sheets.
@davidjoshuaford one way you could write such a time into a new field is having a custom variable which increases by 1 everytime you press the button. The companion instance does need to accept internal variables but hopefully we'll see that option soon.

I would also love to have internal variables working both for value and cell! Also the function "add sheet" (with support for internal variable within the sheet name) could be nice to have.

Just wanted to chime in to say thank you for the Google Sheets support!
I would absolutely benefit from being able to pass a Companion internal variable to a cell in Google Sheets, essentially completing the link between Companion and Sheets. Any more functionality would obviously be welcome, but at a base level the "value" field should support custom internal variables. Thank you!

Would also love to push a variable to a cell in gsheet!

v1.1.1 of the Google Sheets module now includes support for variables as both the cell, (eg sheet5!A2), and the value.
The next Companion build will include these changes.

I've personally not been a fan of variable hints for performance reasons when there are lots of variables, so have not implemented it in any of my modules. It has been quite a few updates of the web ui since I last checked performance though so I'll have to look in to it again.


Weird interactions between Form#onChanged and FormState#reset

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Open and run the example at https://dartpad.dev/?id=fc62719dac640d4bc2e92d61becbeb92
Write some text in the form field and submit the form

Expected results
Form is cleared as expected after the submit.
Actual results
Form is "cleared" with the value inputted by the user.
Using a debugger, I could see that the widget.initialValue used here is actually the text the user entered and not an empty string or null.
https://github.com/flutter/flutter/blob/164fd993d70c635b8f6adced275859a7de8566cc/packages/flutter/lib/src/material/text_form_field.dart#L366-L372
Code sample
Code sample
View on Dartpad: https://dartpad.dev/?id=fc62719dac640d4bc2e92d61becbeb92

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.1, on macOS 13.3.1 22E772610a darwin-x64, locale en-IT)
    • Flutter version 3.10.1 on channel stable at /usr/local/Caskroom/flutter/3.10.0/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d3d8effc68 (2 weeks ago), 2023-05-16 17:59:05 -0700
    • Engine revision b4fb11214d
    • Dart version 3.0.1
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/<redacted>/Library/Android/sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.78.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] Connected device (3 available)
    • iPad (10th generation) (mobile) • <redacted> • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)                 • macos                                • darwin-x64     • macOS 13.3.1 22E772610a darwin-x64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 113.0.5672.92

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Might be related to #113327

Might be related to #113327

Hi @niktekusho
I see there's an open issue addressing the case you described https://github.com/flutter/flutter/issues/123009 that seems to be similar. Can you check if it is?

Hi @niktekusho I see there's an open issue addressing the case you described #123009 that seems to be similar. Can you check if it is?

Hi @huycozy, it could probably be related but in my example, the Form.onChanged is regularly called and the TextFormField is wired with a TextEditingController.

Yes, I notice Form's onChanged is invoked when calling reset()whilstTextFormField` is not (this makes a difference with https://github.com/flutter/flutter/issues/123009).
But if you remove Form's onChanged callback, it works as expected. Going further with this clue, you are calling _formKey.currentState!.validate() inside the callback, it does rebuild in the underlying methods. This is intended behavior. Also, if you want to validate the form on each change, your current AutovalidateMode.onUserInteraction is enough for that purpose.

As I said though, I'm confused by the inspection I did with the debugger where widget.initialValue is not an empty string (or null) but it's exactly what the user wrote submitting the field.

I'm not sure I can get this point if it's related to the issue.  When you call reset, the initialValue is putted back to the field if you declared it on parameter.

I'm facing the same issue, TextFormField with a custom controller won't reset to empty when I call reset() of formState. Because if initialValue is null, it will be set by the current value of the controller in constructor of the widget initialValue: controller != null ? controller.text : (initialValue ?? ''),. The problem is TextEditingController was created in parent scope, so the value doesn't change when we rebuild TextFormField before calling reset(). So, I think if not rebuild the widget, reset() will work.


Add helpers to read installconfig from file/cluster config

Lifts code from https://github.com/openshift/machine-config-operator/pull/47 to provide a generalized approach for operators to access the cluster's installconfig from the cluster configmap file available on the system.
Will be helpful for fixing https://github.com/openshift/cluster-kube-apiserver-operator/issues/83, for example

@damemi I think we can close this, right?


Enable cache for all steps that have explicit enable_cache=True

Describe changes
This PR changes the precedence of enable_cache configs from run > pipeline > step to run > step > pipeline.
Human Readable Explanation
Previously, if enable_cache=False was set on pipeline level, caching would always be disabled, no matter what was set at step level.
Now, if enable_cache=True is explicitly set for a step, caching is enabled for that step even if enable_cache=False is set for the pipeline in general.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[x] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

@strickvl requesting review again since I added additional logic with corresponding test that ensures that enable_cache is also overridden for steps if set at run level.


Support for Terraform 0.7.0 data sources

I know it's brand new, just thought I'd raise an issue for the new data sources feature: https://www.terraform.io/upgrade-guides/0-7.html

P.S.  This plugin is great, keep up the good work.

Implemented would be available in next plugin release (0.5.5). Hopefully it would be released this week.

I've just released plugin version 0.5.0, check it out via regular 'Check for Updates' action in your IDE.

Works for me.
Thanks.


Update view-only-meeting-experience.md

Removing extra information

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@cichur PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Closing bug in brew update formula rule (changed error message)

Brew changed the output of the error message so that the rule no longer works. They have changed the apostrophe to a backtick
from

Use 'brew upgrade foo' instead.

changed to

Use `brew upgrade foo` instead.

Change in brew: https://github.com/Homebrew/brew/commit/2f7c3afeb8b6cb76b35860ac195e3b35d18132f6#diff-c234c527e0e187e0743c519ab0e415afdc1e41e33e014cd78152930b11885057
The regex in the code change will handle both scenarios.
This PR fixes or closes #1290 support brew update <package name>

In brew_reinstall.py the backtick is hardcoded. Actually you don't need to check both cases, that's why no regex is needed. The test has been aligned.


rewording on readme

Duplicate of #8

oh, thanks @mmwtsn on the heads up!

No problem. Thanks for opening the pull request!


Why no longer live?

Hello, may i ask why it is no longer live?

Google has discontinued the Panaramio API on which this app worked.


terragrunt: 0.28.2 -> 0.28.3

Motivation for this change
Bump terragrunt to 0.28.3
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 112663 run on x86_64-linux 1

1 package built:

terragrunt

Result of nixpkgs-review pr 112663 at 2a5b91dd run on aarch64-linux 1

1 package built:

terragrunt


getUserFromClient is not defined

grant.js line 293
  return this.model.getUserFromClient(clientId, clientSecret,
      function (err, user) {
    if (err) return done(error('server_error', false, err));
    if (!user) {
      return done(error('invalid_grant', 'Client credentials are invalid'));
    }

    self.user = user;
    done();
  });

This method returns undefined when I try to get access_token with client_credentials. I check the code but I couldnt find the implementation of it.

This is one of the model functions you have to implement yourself.
See 2.4.1 README.
For example, if you don't care about the user, or just to try it out:
app.oauth = oauthserver({
  model: {
    // ...
    getUserFromClient: function(clientId, clientSecret) {
        return {};
    }
  },
  grants: ['password'],
  debug: true
});

@maxtruxa You mentioned password grant type in your example. The only usage for this function is client_credentials.
@barisvelioglu I have implemented without caring about user and its looks like:
this.getUserFromClient = (clientId, clientSecret, callback) => {         callback(null, true);     };

@AlbertHambardzumyan You're right. IIRC I just copied the basic example from the 2.x README (that's where password came from) and added the handler.


fix - correct typeFullName for member with generic type, issue 105

@fabsx00 Have raised a new commit which persists the generic information in typeFullName

I haven't had much time to look into this, but also check this thread: https://github.com/joernio/joern/pull/2655

Can you elaborate? What is the effect of this change now that typeFullName stays the same, or does it not?

@DavidBakerEffendi Thanks for the thread, I went over it. To frame my understanding, two points which are related to this PR.
1st - Creating a data structure which handles the generics, for which you have proposed a few approaches
2nd - There is an argument here which talks about not adding generics in typeFullName.
Regarding 2nd point, earlier this PR was made based on a similar understanding of not including generics in typeFullName, but as per a comment by @fabsx00, I have modified the PR to include the generics in typeFullName as well.
Open to further discussion

Can you elaborate? What is the effect of this change now that typeFullName stays the same, or does it not?

Yes, In earlier version of this PR the typeFullName for
import org.apache.kafka.clients.consumer.Consumer;
private Consumer<String, Integer> consumer;

was org.apache.kafka.clients.consumer.Consumer
now with further change as per your comment it is
org.apache.kafka.clients.consumer.Consumer<String,Integer>
The reason for the PR was the <unresolvedNamespace>.poll methodFullName for the call node poll, wanted to have a resolved namespace for the member consumer and the callNode poll

Observed some fluke with a jssrc test here, but this one doesn't touch jssrc, so, that seems unrelated.


fix-vimWebSessionPaloAltoCEF

Required items, please complete
Change(s):

Fix ASimMatchingIpAddr

Reason for Change(s):

ASimMatchingIpAddr returned wrong value

Testing Completed:

Testing from Fortinet parser

Checked that the validations are passing and have addressed any issues that are present:

Yes

requested changes. details in the previous comment

changes should be there

Hi @KustoKing, Thank you for the updates. We will provide you with an update as soon as possible. Thanks


Bugfix on detail endpoints for ajax CRUDs

Found a bug on AJAX CRUDs view for detail endpoint: instance referenced variable doesn't exists in that function. Replacing it with self.data solves the issue:

@developerpe Please take a look on this. :)

Ohh thanks to pull request and the report bug, Thank u for fix bug!
Pull request acepted!


Wasn't able to parse friends index

Hello, this is my first time using your app. I was trying to get the friends list but, the next error message appers when i execute the next command:
$ python make.py --index
Loaded credentials from .env file.
Opening saved friends list...
Wasn't able to parse friends index. This probably means that Facebook updated their template.
Please raise issue on Github and I will try to update the script.
Or if you can code, please submit a pull request instead :)
I tried to use the command --download and -- parse, but it return a empty list in json.
Thank you

Hi @Laura2797,
For the first run, try running just python make.py without any of the parameters. Currently that's the only time that it will download the friend list. I'll clarify that in the readme for the next update.
If that doesn't work, try commenting out this line: options.add_argument('--headless'), so that it's #options.add_argument('--headless'). This should make Chrome visible so you can see what is happening and where it might be getting stuck.
I just pushed an update (15 minutes after you posted) with a few fixes. Though I don't know if it'll fix this particular issue, can you download the latest one just to be sure?

Hello @jcontini, thank you for you very fast answer!
I tried run "python make.py" and it create a new file index.html. It has a html code, but the index.json and profiles.json was empty.
Later I tried comment the line options.add_argument('--headless'), and i saw that the chrome open a new window and then go to the friend tab, but dont get anymore information. In the terminal show the same message than before.
And finally I update the repository with your fixes, and creates a file "friend_list.html" and data.db. The .db is empty too, but I'm not sure.
In conclusion, the .html have information, but the .json and .db are empty.
Anyways, I'm interested that the information returns in a .json format is that possible?
Thank you, have a good day!

Hi @Laura2797,
If you open the friend_list.html file on your computer, can you see the list of your friends?
Ah it's good to know there's an appetite for the JSON format after all. I'll update the script later to add a feature for JSON export.
Either way, would you be comfortable sending me the file<EMAIL_ADDRESS>so I can see if it parses on my computer and try to troubleshoot? Or if you prefer, we can do a screenshare session instead via Zoom/Google Meet.

Hi @jcontini, I just send you the email to<EMAIL_ADDRESS>notice me if you dont recieve the email.

Hi @Laura2797, thanks - helping you out there. Also, I've just added a feature for JSON export. To use it, just run python make.py --json. This will create 3 json files (friend_list, profiles, locations) in the db/ folder. Let me know if it works for you, or if you're looking for different data in the files.


Use Guix and direnv

raygui.h doesn't need the patch as it's possible to set as a build flag
merges all patches for raylib to a single patch
build raylib and raygui using guix
integrate using direnv

This might be a bit controversial...
If you have Guix and direnv installed, it's enough to navigate to the claylib folder, and you'll have raylib and raygui built and installed in a deterministic fashion. For other systems (didn't detect running in direnv), the prebuilt libraries will still be used.
If you've never used Guix or Nix, I understand this might be a bit strange, but for us Guix users, it's great as we don't have to patch the libraries -- I actually had to build them myself.
I set the PR as draft as we discuss and review it.

I'm going to get back to you on this. We're pondering some changes to the build process. #1 priority is making sure the build is straightforward on standard Linux and on Windows (the latter we don't have yet). I'm a little iffy on committing changes that only apply to Guix but I don't really know the norms there.

It's quite normal for projects to have nix-shell support, but Guix is not as popular although it's nice to use Guile Scheme rather than the homegrown Nix language together with shell scripts.
If you're on Linux, you should install both Guix and Nix (the latter because it has more libraries packaged if you need that).
I can maintain this from the outside although it's not as ergonomic because I have to maintain a commit that I never push (.envrc) upstream. The setup is quite nice though! You don't have to include libraries for Linux as those will be built by the user, and the libraries will work for all distributions -- a key feature of Nix and Guix. And because of direnv you don't have to do anything manually at all. Visiting the directory is enough to get a reproducible build which is the same for all operating systems and works for all (all dependencies are included down to glibc), which means there will be no "works on my system".
For Windows it's probably best to include all dependencies in the project though.

I can probably solve this with much less interference. I can create a new repo and have claylib as a subrepo. The only change I need would be to skip using claylibs libraries, maybe through an environment variable CLAYLIB_USE_SYSTEM_RAYLIB_LIBRARIES or similar. I'll come back with another PR.


Radiation not working for Zurich case

There is a memory error, it might have to do with the size of the case study, specially the district. we will check this into the code.
@chenkianwee is there a way to limit the resolution of the district.shp (context buildings) to a wider sensor grid, but keep that one of the Zone.shp smaller?

Hi Jimeno, memory from where? From PythonOCC? It is definitely possible to vary the resolution.

140 mb is ok I think in terms of geometry, but I have not run a solar simulation on such a big file before.  Ya sure, 4pm today ?

sure, see you here!
Jimeno

From: chenkianwee<EMAIL_ADDRESS>Sent: Wednesday, May 17, 2017 9:44 AM
To: architecture-building-systems/CEAforArcGIS
Cc: Jimeno Fonseca; Assign
Subject: Re: [architecture-building-systems/CEAforArcGIS] Radiation not working for Zurich case (#648)
140 mb is ok I think in terms of geometry, but I have not run a solar simulation on such a big file before. Ya sure, 4pm today ?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/architecture-building-systems/CEAforArcGIS/issues/648#issuecomment-301961791, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIjrguDC2Wb07uV2sJh9VARFrpK44wgDks5r6lELgaJpZM4NdRf6.

I get this: Process finished with exit code -1073741819 (0xC0000005)
And it is happening when the next text is printed:
('Error status:', '- Ok')
('Can work?', True)
('Error status:', '- Ok')
('Can work?', True)
('Error status:', '- Ok')
('Can work?', True)
('Error status:', '- Ok')
('Can work?', True)
('Error status:', '- Ok')
('Can work?', True)
('Error status:', '- Ok')
('Can work?', True)
('Error status:', '- Ok')
('Can work?', True)
('Error status:', '- Ok')
In addition i opend  the cityGML file and it shows nothing. so there might be an error on how the file itself is being created right? @chenkianwee

hmnnn Jimeno I am not sure what is happening. I need to look at the error in detail.

i got the same error again :-((

@chenkianwee so this is the profiler of the City GML creator create_gml.py file. Which is quite slow for several buildings. the bottle neck is definitely creating building windows and the intersect_shape_with_pt function... 50% of all the time!

so i was able to correct that function and the bottle neck now is  th emake shell:

Ok the radiation script is working for the Zurich case, however it is quite slow. I think it needs some parallel computing. It did my best in refactoring but i will need an expert to jump in :-) @daren-thomas would you mind to giving me a hand on this? Here is a run of the Profiler.

@JIMENOFONSECA I've added it to my list, but for this sprint, I don't think I have time for improving speed - we can do that afterwards, as long as it runs!


fix(login): update api url

Proposed changes
update api url. as shown below, can't resolve the domain.
$ host app.pulumi.acmecorp.com
Host app.pulumi.acmecorp.com not found: 3(NXDOMAIN)

This example is auto-generated from: https://github.com/pulumi/pulumi/blob/master/pkg/cmd/pulumi/login.go#L53-L55, so we'd need to pull the fix there.
However, this is correct. It's supposed to be an example if you wanted to run a self-hosted service on your own domain, eg: acmecorp.com, then this is how you'd login.


Fixed password input width on smaller screen sizes

In the auth/login component the password input is not sizing its width correctly on smaller screens. See image below.

Adding the w-full class to the styleClass property of the p-password. Fixes the issue.
<p-password id="password1" [(ngModel)]="password" placeholder="Password" [toggleMask]="true" styleClass="w-full mb-5" inputStyleClass="w-full p-3 md:w-30rem"></p-password>

This PR adds the class.

OK


No licence

Hi !
I am investigating using your library for binary signing. Unfortunately I can't really do so unless you provide some kind of licence by which I can use the code.

Oops, thanks for pointing that out. I've added an MIT license to the project.

Thanks ! I'm working on making it work with pesign (which uses the nss hell hole). Slowly getting there, I'll send a PR when I"m done


Update detect-mobile-browsers.mdx

The client sends a user-agent string, not the server.

Thank you, @millette 👍
The updated post is available at https://phuoc.ng/collection/html-dom/detect-mobile-browsers/
I also included your GitHub to the top of the article.


Create a popup.html that will be use to turn off/on the extension

So there are few websites which we want to support so we allow add blocker so instead of going in your extension add a popup.html that can have a toggle button.

hey can i work on this issue

hey can i work on this issue

You can start working on it no need to get it assigned to you

any update @Sauhard74

I am done working on the issue .. will be raising the pr tommorow or today evening

I just wanted to let you know that I’ve made some updates related to this issue in my fork of the repository. I’ve implemented a fix for the ad blocking functionality. You can find the details in my pull request here.

@Sauhard74 you can work in the css of new popup.html

Any progress with the issue ?? need any help


Unblock strawberry upgrade to 0.243.1

Strawberry reorganized a lot of its types, so this is just a no-op that changes a bunch of import paths.
Issues Fixed or Closed by This PR

Closes #189

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Summary by Sourcery
Unblock the upgrade to Strawberry 0.243.1 by updating import paths and adapting to changes in type definitions.
Enhancements:

Update import paths to align with the reorganization of types in Strawberry 0.243.1.

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)

Hi @mattalbr, I noticed that this PR seems to duplicate the work in #187. It wasn’t immediately clear because the first PR was opened before the issue, so the issue came up without a linked PR.
Since you have more experience with this project, what do you think is the best course of action? Should we prioritize the later PR?

Hi @mattalbr, I noticed that this PR seems to duplicate the work in #187. It wasn’t immediately clear because the first PR was opened before the issue, so the issue came up without a linked PR.
Since you have more experience with this project, what do you think is the best course of action? Should we prioritize the later PR?

Amazing, I missed #187! Assuming the author of #187 is responsive to the few tweaks we need for the latest version, I would definitely prefer we go with their PR! It never feels good to have your PR swooped :)
Closing for now, I'll reopen if we stall on #187.


[Fix]add set thread num config for wg flush pool

Proposed changes
calculate workload group's mem table flush pool 's thread num by cpu and disk num,  other wise it may cause mem table flush cost more memory than not enable workload group.

run buildall

@liaoxin01  please add merge branch label for this pr.

run buildall

run buildall

run buildall


Bad LowerCase/UpperCase file loading in recent file window

Describe the bug
When you loading a project with Recent Project window or menu, the project file is loading in lowercase instead of real case name.
When you generate files, the project name is used with the loaded case name instead real case name
Please also mention any information which could help others to understand
the problem you're facing:

GuiX Studio version : <IP_ADDRESS>
Environment : Windows 10

To Reproduce
Steps to reproduce the behavior:

Create a project with different lower/upper case like "GuixProject.gxp"
Open It  with a double click on the file in file explorer : File is correctly loaded (GuixProject.gxp)
Generate files : All reference of project name are "GuixProject_****"
Close Guix Studio
Open Guix Studio, the Recent Projects window will list all project .... In lower case !
Open with Recent Projects Window your "GuiProject.gxp" : File is lowercase loaded (guixproject.gxp)
Generate files : All reference of project name are "guixproject_****"

Impact
Compilation with error because of bad variable/constant name (C is case sensitive)

Hi xNiux, I'm able to reproduce the issue, and we'll fix it.

Hi xNiux, we have fixed the issue, and will be included in the next release.

GUIX Studio version <IP_ADDRESS> has been released on the Microsoft Store, and it includes the fix.

GUIX Studio version <IP_ADDRESS> has been released to Microsoft Store, and it includes the fix.


Fix Hotwire posts in RSS feed

Context
Hotwire newsletters are not visible from the RSS feed.
Issues

Hotwire newsletter posts are missing the date in their file name
They must be inside a _posts folder
They must be in hotwire directory to be considered as a category and target as such in hotwire.html

Solution

Fixed 2022 dates to 2021
Added dates in Hotwire posts file names
Moved posts from _hotwire to hotwire/_posts

Thanks for the contribution! Especially those date fixes.
I didn't know that the feed was generated from _posts, TIL. The only problem with this approach is the home page now shows the newsletters as blog articles. Is there any way to get around that and have it look like it was before?

Hello, sorry for the late reply.
Indeed, Hotwire posts are now visible from the homepage, as it's retrieving the 5 latest posts:
{% for post in site.posts limit:5 %}

TBH I don't know how to find the best solution, as:

Hotwire posts are currently absent from the RSS feed because they are not seen as posts but pages
As long as they are seen as posts, they are included in site.posts

Hmm. What if we include them as posts but ignore them on the homepage? I think they have a special front matter key or attribute we could switch on.

I thought it f that, but I thought you wouldn't be inclined to have something like "every post but this kind of posts" on the homepage as it doesn't look like the best thing if your blow grows with multiple new categories.

I think that for now that is preferred. The exclusion is only because the Hotwire stuff is technically a newsletter, not a specific type of post.

@joemasilotti Here is another proposal, which works as you want to.
This is not the best code I have ever seen, especially with the counter thing, but it is the only way I found to ensure hotwire posts were not displayed on the homepage, and only 5 "not hotwire" posts were displayed.
What do you think?

Would something like this work?
{% assign posts = site.posts | where: "layout", "post" %}
{% for post in posts limit:5 %}

Unfortunately I did not find how to use where to query for posts which category does not contain "hotwire".
I tried the following:
{% assign posts = site.posts | where: "item", "item.categories not contain 'hotwire'" %}
But it is ignored

I don't think it's possible. That's why I'm filtering by layout of "post" instead in the snippet.

I have no idea how where: "layout", "post" works but indeed, it works.
Well, for a first PR here, it was quite laborious :facepalm:

I think it's matching the layout key in the front matter of each post. And since the Hotwire newsletters use the hotwire layout they are excluded.
Regardless, thanks so much for the PR! this was sneakily tricky. 😆


Add more sdk and psdk

Add more sdk and psdk. For example, add the latest instruments by major and minor versions.

Add a flag for this

Change size list version to 4 and add flug --show-all. Version: 2.4.8


Change Role Name

By adding the unique prefix "EPAC" to the existing role name, we can help to prevent collisions.

Complete, PR submitted


SignalR Server Ready Event

I notice that every call to the server creates a new instance of the hub.
What I'm looking for is something similar to a page ready event, i.e. one where the Clients and Context are initialized when a call is made. I've looked, but can't find anything like that except for a hub method, and I don't want to write something for each method if I don't have to.
Does this event exist?

Yeah, I know there is no concept of a page, it was just an analogy, likely a bad one. OnConnectedAsync is fired only once as far as I can determine, however, if I add a constructor, it is hit every time a request come in to the server.
Unfortunately, nothing is initialized at that point.

So why doesn't OnConnectedAsync work for you?

Because it fires only once, even though the hub is created every time any hub method is called. I wanted something that would fire any time a hub method is called, so that I can do something before the method exits.
The only other way is for me to place a call to a function to preprocess my task in every hub method, which is what I'll do if this event doesn't exist.
I'm doing some authentication of the user(a token with additional info), which is why I want this.

Oh I see that wasn't clear to me earlier, seems like you wanted a "page ready which would only happen once, not per hub invocation". Anyways we don't have anything fo cross cutting concerns yet but we do have something for auth. Why not write an authorization policy?

You're right, I just saw a thread on that, and it fits what I want exactly. Thanks for talking me down off the ledge, lol.
I'm closing this one.