'Mixins should come before declarations' cross 2 levels of import

What version of Sass Lint are you using?
1.9.1
Please include any relevant parts of your configuration
folder

--main.scss => @import helpers.scss then @import parts.scss
 |__helper.scss => @mixin a
 |__parts
        |__ parts.scss => @import part-a.scss
        |__part-a.scss => @include a

What did you do? Please include the actual source code causing the issue.
@include a{
height: $height;
}
What did you expect to happen?
no error
What actually happened? Please include any error messages given to you by Sass Lint.
'Mixins should come before declarations'
If you're using a IDE plugin have you tried the CLI too?
Atom

Sorry im not quite sure what you're saying here, can you explain it a little more please.

OK, let me try i again :)
in main.scss i @import helpers first, then @import parts.scss. => so @mixin a is declared before parts.scss. => the part-a.scss (@import from parts) should be able to use @include a with out error
but, in part-a.scss, i have a @mixin c using @include a , and i got this error.

Found the issue. it's my set not the sass lint


Android Not Showing Image!

Hi, local images successfull showing but server images not show android.
Im add
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
Problem still is continue...

<uses-permission android:name="android.permission.INTERNET" /> ??
If you resolved it already, and I assume you did, please close the issue.

Hello I have the same problem!! but... if I use your image it works ('https://i.redd.it/rc29s4bz61uz.png'), overwise if I use my server image it not works ('http://web3drendering.eu/wp-content/uploads/2015/03/cropped-fincantieri_marco_polo_102.jpg')
is there any reason that I can't see?

I don't understand... suddenly it works!!!
I only managed better the connection with event listener... mah
but another question...
if I have many pages with image, but I want download all at first connection and in a second time, in offline mode see them, How can I do?

mmm.... compiling on another phone... it not visualize images... as the first time... why? I don't understand...

Any update on this?

Maybe this help https://github.com/facebook/react-native/issues/13600


English docs perhaps?

Expected behavior 期望表现
{type something here...}
It's natural to compare JanusGraph and HugeGraph.
When I surf over JanusGraph community (groups), people seems to have much interests about HugeGraph. But, the documentation seems to be a barrier to them.
I suggest HugeGraph communty to put some efforts into welcoming more contributors all around the world. We could start, for example, from translating the docs into English.
Best,
Actual behavior 实际表现
{type something here...}
Only English docs provided.

@suesunss Yes, this is a good idea. We also hope that everyone can participate in this work.

@javeme Sure, very glad to be part of it.

Hi guys,
It's great that you've put so much efforts into this project and that you've been developing it so actively!
However, the online (and international) community has adopted English as its main language, as it's more commonly used across the globe and more people are likely to understand it among developers. I appreciate that the project is being developed mainly by Chinese-speaking people, but perhaps it would be more successful and beneficial for the rest of the world, if you get more documentation translated to English.
It's also peculiar that all of the Github issue/pull templates have English section titles, but then everything else is in Chinese.
Any plans on translating your docs and perhaps switching to English? :)
Kind regards,
Martin

We plan to complete this task in 2022, as HugeGraph has joined the Apache Foundation Incubation.
There is a lot of work and some help may be needed, if anyone would like to contribute, we are very welcome and grateful.


DH_KEY_TOO_SMALL

I have set up a docker image with this Dockerfile:
FROM debian:latest
apt-get update
yes | apt-get upgrade
yes | apt-get install python
yes | apt-get install curl
curl https://bootstrap.pypa.io/get-pip.py -o get-pip.py
python get-pip.py
pip install oandapyV20
yes | apt-get install git
git clone https://github.com/hootnot/oandapyV20-examples
cd oandapyV20-examples/
pip install urwid
pip install -r requirements.txt
CMD tail -f /dev/null

It sets everything up nicely - but when i run
root@ed209233d1bb:/oandapyV20-examples# python src/instruments_list.py
i get:
Traceback (most recent call last):
  File "src/instruments_list.py", line 13, in <module>
    rv = client.request(r)
  File "/usr/local/lib/python2.7/dist-packages/oandapyV20/oandapyV20.py", line 306, in request
    request_args, headers=headers)
  File "/usr/local/lib/python2.7/dist-packages/oandapyV20/oandapyV20.py", line 234, in __request
    raise err
requests.exceptions.SSLError: HTTPSConnectionPool(host='api-fxpractice.oanda.com', port=443): Max retries exceeded with url: /v3/accounts/thomasschindler/instruments (Caused by SSLError(SSLError(1, u'[SSL: DH_KEY_TOO_SMALL] dh key too small (_ssl.c:727)'),))

All i can find on this is that the server must be operating on an outdated TLS version.
Do you have any experience with that?

Hi,
I have not encountered this error before.
I don't find any setup of the token and accountID in your Dockerfile.
You need these.
The URL shows:  /v3/accounts/thomasschindler/instruments
that means you use thomasschindler as account ID. That is not going to work.
The you need a V20-account number and the token. Check the README in the repo for details.
When you are sure that you have a correct token and account, you always can use curl to check if things are OK, see examples: http://developer.oanda.com/rest-live-v20/account-ep/

Thank you for your answer! I will use the accountID - my bad!


Create a page that allows users to choose which system setup route they want to take

Is your feature request related to a specific problem?
In the config UI, we need a page that allows users to decide which method of system setup they'd like to choose: AWS/Okta/Manual
Describe the solution you'd like
see: step 3 here
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Points - unknown -- QUESTION: Will this require actual integration with AWS or Okta, or has this already been handled?

@LKCSmith we've re-scoped this to 1.8, but to answer your question all of the backend work for this ticket is getting done in 1.7


Listview does not highlight the selected item

When i select a item from the listview, it should highlight the background color or border color, but here when i place the cursor in the list item, i am able to see the grey color background, after moving the cursor, it is gone.

As part of this PR https://github.com/trimble-oss/modus-mobile-maui-components/pull/367. This issue got fixed


Lined up currencies

Very minor but it was annoying me
Old:

New:

Feel free to reject if it's not an undesired change.

Just looking at the preview, I have two thoughts:

Can you right-align the currency values?  Then the decimal places would line up and it would be easier to compare values.
Moving the values off to the right makes it a little harder to see the value of an individual currency, specifically when not hovering over the currency with the mouse (which is essentially all the time, for me).  Can you add some UI feature to make it easier for the eye to move between the two columns (e.g., keep the mouse-over effect permanently, or add grid lines)?

I had pondered about right aligning the currency values, I will do it.
Agreed on the on hover effect, I'll add grid lines in.


Reformat only changed code (only if VCS configured) ignores new created files

Reformat only changed code (only if VCS configured) seems to ignore new created files, that are not automatically added into git.
It would be great if these files also get formatted.

Sadly this can't be changed easily as this functionality is provided by AbstractLayoutCodeProcessor which is inside the IntellIJ platform


Fix code scanning alert - Inclusion of functionality from an untrusted source

Tracking issue for:

[ ] https://github.com/open-risk/equinox/security/code-scanning/105

done


Route for each clock

/clock/digital for Digital Clock
/clock/analog for Analog Clock
/clock/flipclock for Flip Clock

https://github.com/angular-ui/ui-router
https://github.com/angular-ui/ui-router/wiki/URL-Routing

http://clock.phuongb.go1.com.vn/#/clock/analog


feat(v2): supports string type for the Layout's keywords props

Motivation
According to the W3C's keywords meta tag spec, it accepts a string type content. Most developers are already familiar with the string type content. So for better DX, we can provide both string and array types for the value of the Layout's keywords props.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan

Add the feature to packages/docusaurus-theme-classic/src/theme/LayoutHead/index.tsx.
cd packages/docusaurus-theme-classic then yarn link
cd website, link the package then yarn start
Ensure both string and array values are work for the Layout's keywords props

@slorber Thank you for the reminder, I have updated the .d.ts file.

@slorber Thank you for the reminder, I have updated the .d.ts file.


Enhancement: setting to disable rust-analyzer sever path override

Sometimes rust-analyzer stops working when the server path is overridden by flatpak-vscode.
One cause is this issue, where the only solution that I know of is to disable rust-analyzer completely to stop it from consuming resources, or to disable flatpak-vscode and delete that override so that rust-analyzer can run on the host ( where it does not hang for me)
But there is also the issue that sometimes rust-analyzer does not start by itself, the only solution I have found that I need to build or run the flatpak at least once and restart vscode.
I think a setting to disable the server path override would be very useful, so that rust-analyzer could run on the host if wanted.
I already have the implementation ready, but I think somebody who actually knows typescript should take a second look at it.
Tell me if this sounds reasonable and I would open a PR.
Thanks!

Do you mind opening a PR? I will have a look at your changes and see if we can easily integrate it to the rest of the extension

I think I fixed the issue of rust-analyzer stopping working, do you mind removing your .flatpak, updating the extension & trying again?

I think the issue of rust analyzer getting stuck in the dir is solved. I do have some other issues with flatpak (e.g. document portal not starting after reboot), so rust analyzer still fails sometimes when having set the path override. I would still like to update the PR to the lastest version and make it ready for merging (if I am able to and you are willing to merge), but the code on github is outdated (v0.15 instead of v0.16). Could you push to v0.16? Thanks!

Oh sorry, I thought i pushed! Done. Yes I would still like to merge your PR, i was just wondering how to properly fix the root of the issue instead of working around it


detailed route spends 30 iteration on fixing trivial pin access.

Describe the bug
Fallout from https://github.com/The-OpenROAD-Project/OpenROAD/issues/5300
All but V1 cut spacing violations were solved after the 4th iteration.
It took detailed routing 30 iterations to put all V1 on the metal1/metal2 track intersection.
Why aren't the via placed at this violation free location to start with ?
Is this an issue with pin access computation ?
Expected Behavior
Put V1 on violation free metal1/metal2 track intersection.
Environment
M1 Mac
OpenROAD v2.0-14343-g6dfa697a1  (Jun 29 2024)
Features included (+) or not (-): +Charts +GPU +GUI +Python

To Reproduce
use risc2.noram.def.gz from https://github.com/The-OpenROAD-Project/OpenROAD/issues/5300
Relevant log output
No response
Screenshots

Additional Context
No response

Could you please provide all the steps to reproduce alongside the files? I see you've attached the def file but not the guides file resulting from global routing? Also I see unsupported via rules by DRT like:
VIARULE TURNmetal1 GENERATE
    LAYER metal1 ;
        DIRECTION vertical ;

    LAYER metal1 ;
        DIRECTION horizontal ;
END TURNmetal1

DRT errors when reading such a rule so it doesn't even start routing, but I assume you have a different version of the lef since you've passed that step.

I'm sorry, my bad. I forgot that I had to comment out the TURN rules in lef.
risc2.lef.gz
I ran the script mentioned in #5300, which doesn't generate a guide file by default.
read_lef lef/risc2.lef.gz
read_def def/risc2.noram.def.gz
global_placement
detailed_placement -disallow_one_site_gap
global_route -verbose
detailed_route -drc_report_iter_step 1

I just tried to generate a guide file by using "global_route -guide_file guide", but this didn't write a file.
Maybe because it errors out ?
global_route -guide_file guide
[WARNING GRT-0300] Timing is not available, setting critical nets percentage to 0.
[ERROR GRT-0118] Routing congestion too high. Check the congestion heatmap in the GUI.
[ERROR GUI-0070] GRT-0118

I have to start detailed_route "by hand" because OpenRoad stops after the global_route error.

Then that means you're using the detailed router's global routing. We are not working currently on supporting TritonRoute's WXL global router. So I would recommend fixing the issues with the global routing step before starting detailed routing.
I will close this issue now. After https://github.com/The-OpenROAD-Project/OpenROAD/issues/5300 is resolved and you are able to run global routing correctly, try detailed routing. If the issue still persists, please feel free to reopen.

@osamahammad21 he mentions that he is running global_route  in his script so the guides will be in the db I think.

@maliberty Does GRT writes the guides to the db even if it produces an error?

No, if grt errors out then this isn't a drt issue.  -guide_file guide shouldn't cause grt to fail if it passes otherwise.  @stefanottili are you able to pass grt in any case?

@maliberty @stefanottili @osamahammad21 You can use the -allow_congestion flag to generate route guides even when the groute ends with congestion. The error message will not happen. Instead, a warning message about the congestion is generated.

Thanks for the hint, I was not aware that when global route errors out I would be running detailed route with its "internal" global router.
global_route -verbose -guide_file guide -allow_congestion

keeps the script running and continues from global route to detailed route.
I would argue that -allow_congestion should be the default behavior.
I've seen detail routers resolve groute overflow issues, so I always run detailed route after groute.
a) previously I had 2 of these drt warnings, now there are 305 of them
[WARNING DRT-0225] CE1_SEL_E_R 1 pin not visited, fall back to feedthrough mode.
[WARNING DRT-0225] JTAG_TMS 1 pin not visited, fall back to feedthrough mode.
[WARNING DRT-0225] RESET_D1_R_N 23 pin not visited, fall back to feedthrough mode.
[WARNING DRT-0225] BUSCLKF 2 pin not visited, fall back to feedthrough mode.

b) after the 4th detailed route iteration, there are mostly V1 cut spacing violations left, more then before.
 834 [INFO DRT-0195] Start 4th optimization iteration.
...
 856 Viol/Layer      metal1     V1 metal2     V2 metal3 metal4
 857 Cut Spacing          0    191      0      1      0      0
 858 Metal Spacing        1      0      4      0      0      0
 859 Short                0      0     21      0      6      1

After the 36th iteration, all pin access errors are solved by putting the V1 on the pin's metal1/metal2 track intersection.
So this issue didn't change by going from global route guides to "internal detailed router global route".
The error markers from the 5th iteration show that the V1 didn't start on pin access points.
It seems to me putting them there to start with could save a lot of lengthy routing iterations.

c) the groute congestion report and the groute map display with a centered die area are completely wrong.
This design is 100% routable. (It was routable even with macros in 2004)
d) don't assume die area left/bottom to be 0,0, it could be negative or positive
Having a chip centered was a tapeout requirement and it was default behavior of "init floorpan" to center die area.
I've also done blocks with the left/bottom corner with a positive offset because we were "pressing down" full chip power and pin's doing floorpans for hierarchical chips. We just used global coordinates in the hierarchy below.
I would argue that assuming that the left/bottom of the die area is 0,0 is a bug.

By the looks of the error markers, it seems as if the V1 for pin access would be initially placed into the "center" of the pin, not on the pin access point.
In all cases I checked, getting rid of the V1 cut spacing errors was eventually achieved by putting them on the pin access point that the gui displays a X
Please reopen this case.

0,0 is not a bug and every design taped out with OR uses it.  We should allow negative coordinates (though not a high priority).
Many designs will never finish in drt if they fail in grt so making it the default is not desirable.  It just slows down the process of isolating the issue.
This design may have been routable in 2004 (though we have no existence proof) but that doesn't mean the issue is in drt.  It could well be that it had a more routable placement.  It is necessary to identify where the failure is rooted.

drt routes this design despite the fact that groute map display/congestion warnings seems to have a bug.
By the looks of it either the guides are correct or the detailed router groute put's things in to place anyways.
This bug report is about the 30 drt iterations that are wasted runtime because the V1 put on pin's seems to be on the center of the pin initially. Eventually drc puts them at the metal1/metal2 track intersection that's marked as pin access point.

The option is there if you wish to use it.  grt isn't a perfect predictor of drt but expecting it to be isn't realistic.
I'll leave the question of V1 to @osamahammad21

If drt would be quick in "fixing" the V1 cut spacing error, there wouldn't be a bug report.
But it takes only a couple of minutes of routing time to get to the 4th iteration with 191 VI cut spacing errors left and then another 30 painfully slow iterations to fix these.
There is clearly room for a significant runtime improvement.

c) the groute congestion report and the groute map display with a centered die area are completely wrong.
@stefanottili Could you detail what is completely wrong with the congestion report?

@eder-matheus
please have a look at the last two pics in
https://github.com/The-OpenROAD-Project/OpenROAD/issues/5300
groute claims an unroutable design with 206.58% routing resource usage,
rudy show everything is fine and
detail route finishes without any violations.
It takes ~4min of runtime to get past groute using the lef/def attached to this test case.


Switching tabs by swiping on the touchscreen

it would be awesome if Tocas UI supports switching tabs by swiping on the touchscreen owo//

Sounds like Material design. I do not think that this applies to a framework that tries to avoid using JavaScript.

This cannot be done based on the current tab structure.


[Fix-11583][UI] Fix the API usage of gantt graph

Purpose of the pull request
Fixes https://github.com/apache/dolphinscheduler/issues/11583, the backend passed wrong data that cannot be parsed by the frontend.
(The import items were changed because of the spotless check)
Verify this pull request
This change added tests and can be verified as follows: Manually verified the change by testing locally.

Please review the backend @SbloodyS

@caishunfeng Please review the backend

Codecov Report

Merging #13778 (dfb5836) into dev (69e7449) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

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

@@             Coverage Diff              @@
##                dev   #13778      +/-   ##
============================================
- Coverage     39.02%   38.97%   -0.05%
+ Complexity     4450     4442       -8
============================================
  Files          1133     1133
  Lines         42353    42316      -37
  Branches       4789     4769      -20
============================================
- Hits          16528    16494      -34
+ Misses        23999    23998       -1
+ Partials       1826     1824       -2

Impacted Files
Coverage Δ

...r/api/service/impl/ProcessInstanceServiceImpl.java
59.92% <0.00%> (ø)

... and 33 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


add pod distribution budget information collector.

This implementation is to collect the information which usually help folks for identification of Pod disruption budget related glitches.
This PR should take care of #134 , cc: @SriHarsha001 as he is aware of these logic essentially this is just raw collection of data which could help in identification of PDB related issues.
Thanks. 🙏

Codecov Report

Merging #135 (f600431) into master (f55868b) will increase coverage by 0.83%.
The diff coverage is 53.12%.

@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
+ Coverage   38.62%   39.45%   +0.83%
==========================================
  Files          12       13       +1
  Lines         523      555      +32
==========================================
+ Hits          202      219      +17
- Misses        304      314      +10
- Partials       17       22       +5

Impacted Files
Coverage Δ

pkg/collector/pdb_collector.go
53.12% <53.12%> (ø)

Continue to review full report at Codecov.

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


Swapd refact2

Codecov Report

Merging #561 (b61aab7) into main (0f80924) will decrease coverage by 0.1%.
The diff coverage is 0.0%.

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

@@           Coverage Diff           @@
##            main    #561     +/-   ##
=======================================
- Coverage   12.0%   12.0%   -0.1%
=======================================
  Files         34      34
  Lines      10601   10660     +59
=======================================
  Hits        1277    1277
- Misses      9324    9383     +59

Impacted Files
Coverage Δ

src/swapd/runtime.rs
0.0% <0.0%> (ø)

Continue to review full report at Codecov.

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


Add max slippage option in the settings

There's a problem with users executing transactions with huge slippage without noticing and losing lots of funds in the process.
In order to circumvent it, we'd like to add a new setting - "Max Slippage", default at 5%.
When the user attempts to execute such a transaction, the app should prevent it and show a popup instead that says:
"This swap slippage will exceed the maximum slippage set. To execute it anyway, set a higher maximum slippage in the app settings."

Discussed in meeting with Asaf, @ashachaf to engage you on strategy with this one, placing on hold for now.

Ok, Asaf is working on a new design for this. Let's keep this one here and update it with the new design once it's ready.

as a user, when slippage increase i would like to be aware of this.
currently, when slippage is > 3%, we color the slippage section in red.

mockup:
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=3268%3A0
update the flow in the following way:
0-3% slippage, do nothing.
3%-5% slippage, color the slippage value red (as we do today)
new warning levels:
5%-10% slippage, update the CTA text
10%+ slippage, update the CTA text+color


Introduced input trigger for date-picker

Description
Introduced input trigger for date-picker. In general it works but there are still some doubts. Also I need to update documentation, handle button trigger deprecation in better way and add tests.
Old button trigger is still used by default for backward compatibility. Hope we will replace it in nect major release.
Motivation and Context
Button trigger is inaccessible for blind users and not handy for keyboard only users. My hypothesis is that input trigger will resolve these issues.
How has this been tested?
I have added more vo-tests
Screenshots:

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 not work as expected)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Lena described all the main points 👌 I think, we need to synchronize with designers about the size and jumps of the font.
I also wanted to add that the up and down arrows should work.


Bump version to 0.8.0.dev1

Bumps the version to a dev version. This is a pre-release, so pip will ignore it by default (https://pip.pypa.io/en/stable/cli/pip_install/#pre-release-versions).

LGTM! Do you want to publish a release instead of installing from a tag though?
https://stackoverflow.com/questions/13685920/install-specific-git-commit-with-pip

LGTM! Do you want to publish a release instead of installing from a tag though?
https://stackoverflow.com/questions/13685920/install-specific-git-commit-with-pip

Resolved offline!


How to make it detect live person only not photos

We need this library for an attendance app where user must scan their actual face.
The library scans both actual face as well as printed image of the same face.
How can we restrict it to live people only?

@Alecckie did you find any solution?

@Renatinaveen Unfortunately no. Inform me if you get any

I started using flutter_face_api_beta: ^0.0.67. But this sometimes doesn't give response when their server is down.


Support Webpack aliases

I work in a large codebase and this plugin could be extremely valuable to us but for one issue: we use webpack aliases for imports and currently only real imports are supported.
Example:
@import "~styles/constants/spacing.less";
@import "../constants/colors.less";

The first import is ignored, the second one resolves correctly and provides IntelliSense for all variables from colors.less.
I believe that it should be possible to provide the user an option to provide the path to his webpack configuration and the plugin could read the aliases from there and correctly resolve the imports. Parsing the aliases can be achieved pretty easily since the webpack configuration is standard.
I'd be happy to add this change myself but I'd love it if someone could point me in the right direction in regards to how sane this proposal is and how doable it is considering the existing codebase.

+1

maybe you can change the way you import that stylesheet with. so, extension can understand that.

I'd settle for just hard coding a list of files to read in the extension config.
Ex:
less.scannerInclude: ["./root/constants/colors.less"];

Is that possible?

Related: Jump to definition (ctrl+click) for import statements that do not have a file extension. Common convention with Webpack for import URLs to not have extension, resolves automatically via Webpack resolve extension settings.


textField.observe has issues with cell reuse

If I have multiple text fields in a table observing changes to a model property, then as the cells are reused they start observing multiple properties. This can be fixed by checking that the observable value returned matches the model property, but it would be better to clear observables when a cell is reused (and as far as I know you can only add obervables).
For the example below, changing pickerTwo value will sometimes also update rowOne as it has previously been observing rowTwo (under cell reuse rules)
case .rowOne:
    cell.textField.observe(for: vModel.propertyOne) { (prop1) in
        //hack way of ensuring observable is right one (as cannot remove "observe" for cell reuse)
        if prop1 == vModel.propertyOne.value {
            cell.textField.text = getPropOneValue()
        }
    }
    cell.textField.text = vModel.getPropOneValue()
    cell.textField.inputView = pickerOne
case .rowTwo:
    cell.textField.observe(for: vModel.propertyTwo) { (prop2) in
        if prop2 == vModel.propertyTwo.value {
            cell.textField.text = getPropTwoValue()
        }
    }
    cell.textField.text = vModel.getPropTwoValue()
    cell.textField.inputView = pickerTwo

Actually that hack way doesn't help.....

#4 Related, but for bind rather than observe

The only way I can see to fix is to either observe everything (rather than bind) and compare index paths, or to override "prepareForReuse" and entirely replace the text field.
Neither is ideal. When I have more time I can look at a "clear all bindings" method that can be applied at start of "cell for index path"


Use Ol 3.2 and Cesium 1.6

Migrate to new stable versions of Ol3 and Cesium.

Looks good, please merge.

Thanks for the review @ahocevar.


Updated for Swift 3?

I'm interested in using this but it doesn't seem to be compatible for Swift 3 at the moment, will it be updated?

Hi!
Yep, i'll update the view!
I've added it to my TODO list 👍

Great! I'm glad this project is active 👍 😄

hope project support swift3

@holzleitner @lou-lan  @flamingfusion
I have updated the framework to Swift 3. See #request for more.

how can i update to swift 3

You can use my fork, which is compatible with Swift 3.
https://github.com/sanjeetsuhag/ISTimeline

Have a look at the release ISTimeline v0.0.4 🤙


Add Connector info to Overview - Millennium

JIRA issue created by: rcook
Originally opened: 2012-07-09 02:59 PM
We have an over view section at http://code.google.com/p/xcncip2toolkit/wiki/ConnectorDetails but it needs some work.
We should add the other connectors (Symphony & Millennium). I think the Voyager connector info requires updating, as it mentions two versions (ODBC, VXWS) and I think we now have just one (using both ODBC & VXWS).

JIRA User: rcook
JIRA Timestamp: 2012-07-09 03:01 PM
Added Bach as a developer so I could assign this to him.

JIRA User<EMAIL_ADDRESS>JIRA Timestamp: 2012-07-25 11:15 AM
Done - Updated all documents


Add cabal check CI

Description
If cabal files are malformed, it can cause downstream projects to not be able to use this project.  Adding the CI check will ensure the cabal files are never malformed so downstream projects can update their dependencies freely.

Fix malformed cabal files
Bring cabal files up to standard (syntactically) for publishing
Add cabal file check so that new and existing cabal files do not regress below standard

Checklist

Branch

[ ] Commit sequence broadly makes sense
[ ] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] If this branch changes Consensus and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[ ] Self-reviewed the diff
[ ] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[ ] Reviewer requested

How's this PR coming a long? It'd be nice to be able to compile cardano-node against most recent ouroboros-network master branch :)


MudTable: Finish (commit/cancel) inline-edit using code

Feature request type
Enhance component
Component name
MudTr
Is your feature request related to a problem?
I have the following requirement regarding a MudTable
"As a user I want to finish inline-edit mode by pressing the enter key so that I can complete the edit operation"
I tried to commit the changes using the following code without success:
// Set the item value to cancel edit mode
_table.SetEditingItem(null);

// Trigger the commit event
_table.RowEditCommit?.Invoke(Item);

// Force/indicate a refresh on the component to remove the edition mode for the row
StateHasChanged();

With this code the inline-edit does commit but I am not able to open the inline-edit mode of the current row again. I guess this is because of the hasBeenClickedFirstTime variable in MudTr.razor.cs class.
Describe the solution you'd like
Make CommitEdit or CancelEdit public methods of class MudTr.razor.cs to allow the user to finish inline-edits using code.
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[x] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Here is a workaround  that I found that is related to this and might help others.
I needed to have a table where you can add a new row, then exit commit by pressing enter and creating a new row underneath :
<MudTable @ref="tableRef" Items="@NoSerieList" Dense="true" Hover="true" SortLabel="Sort By" Elevation="2"
          EditTrigger="TableEditTrigger.RowClick">
    <ToolBarContent>
        <MudButton Variant="Variant.Filled" Color="Color.Primary" OnClick="AddSerialNumber" Class="mb-2">Add Serial Number</MudButton>
    </ToolBarContent>
    <HeaderContent>
        <MudTh>
            <MudTableSortLabel SortBy="new Func<SerialNumberDTO, object>(x=>x.Number)">Serial Number</MudTableSortLabel>
        </MudTh>
    </HeaderContent>
    <RowTemplate>
        <MudTd DataLabel="Serial Number">
            <MudTextField<EMAIL_ADDRESS>Required />
        </MudTd>
    </RowTemplate>
    <RowEditingTemplate>
        <MudTd DataLabel="Nr">
            <MudTextField Immediate="true" T="string" AutoFocus="true" OnKeyDown="HandleKeyDow"<EMAIL_ADDRESS>        </MudTd>
    </RowEditingTemplate>
    <PagerContent>
        <MudTablePager />
    </PagerContent>
    <EditButtonContent Context="button">
        <MudIconButton<EMAIL_ADDRESS><EMAIL_ADDRESS>Class="pa-0" />
    </EditButtonContent>
</MudTable>

@code {
    [Parameter]
    public List<SerialNumberDTO> NoSerieList { get; set; } = new();
    private MudTable<SerialNumberDTO> tableRef;

    private async Task HandleKeyDow(KeyboardEventArgs e)
    {
        if (e.Key == "Enter")
        {
            tableRef.SetEditingItem(null);
            await AddSerialNumber();
        }
    }

    private async Task AddSerialNumber()
    {
        tableRef.SetEditingItem(null);
        var newLine = new SerialNumberDTO();
        NoSerieList.Add(newLine);
        await Task.Delay(25);
        tableRef.SetEditingItem(newLine);
        StateHasChanged();
    }

    public class SerialNumberDTO{
        public string Number {get;set;}
    }
}

Three key points here :

First, the "Immediate" attribute set to true on the MudTextField.  With this, the new item on the object list bind to the table that you create by pressing enter or the "Add" button is updated as you type.
Second, the "tableRef.SetEditingItem(null);" that unselect the current row. Since your object is updated as you type, you can exit safely and the value will still be updated.
Finally, the "await Task.Delay(25);" is a workaround for a known issue that you can found here : #3279

Hope that help, you can find the live example here : https://try.mudblazor.com/snippet/QYmokfvzdwlupabg


feat(docs): optimize illustrations for documents in Zone Management

close: #2024
What this PR does / why we need it:
There are errors in the image of the Zone Management chapter, this pr is going to correct it.

@Victor1319 @bboyCH4 please review this.

thanks for your contribution

:smile: good job，the new diagram expresses much more clearly～


Add PR title guidelines to CONTRIBUTING.md

I had difficulty deciding on a suitable name when opening a pull request recently. I couldn't find any guidance on naming PRs in CONTRIBUTING.md despite seeing some of them be renamed before merging (e.g., #9166, #9174, #8995, #8036).
I think adding some naming guidelines could be helpful for both contributors and maintainers. Just to share my experience, I spent around 10 minutes going through recently merged PRs to try and perfect the name of mine. 😄

Hello @trag1c  i'm frontend developer, i would like tto work on this issue

Hah, yeah, this would be great. I end up doing the names either before merging or when creating the changelog.


[cli] Implement Display for BalanceChange type

Description
Implements the Display trait for the BalanceChange type. This is part of a larger effort (and multiple PRs) in implementing a nice format output for SuiTransactionBlockResponse in the CLI, following @laura-makdah's excellent formatting in SuiTransactionBlockEffects.
Test Plan

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[x] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes
Implements the Display trait for the BalanceChange type as part of a larger effort in implementing a good-looking format output for SuiTransactionBlockResponse in the CLI.

(For next time, but the release notes should be in terms of the externally visible change -- i.e. the format of balance changes in the CLI changed).

I see. I did not want to add any release notes here, and was thinking only to do that for the final PR that combines everything together (e.g., when the Display trait for SuiTransactionBlockResponse is implemented which will format nicely these transaction responses).

Agreed with your line of thoughts here!


Update dependencies to v8 internal

FYI I am not authorized to merge to this branch, hoping you can please do it @scottn12 ?

FYI I am not authorized to merge to this branch, hoping you can please do it @scottn12 ?

I think unfortunately only @tylerbutler has authorization right now


Empty squares on the right should mean it's unchanged

When you put a range square on the left, and then leave the corresponding square on the right blank - no square at all, it does some interesting behaviour.
I think it should leave that cell alone, keeping the colour it was, but at the moment it pretends the range was there, and converts it to a random colour in that range.

good idea! I think I'll make a new instruction on the backend that's "DO_NOTHING" or something! Currently it does the "RECOLOUR" instruction by default.
this also shows another bug -> the colour of numbers in a range should be maintained. It's on my list of issues to fix this! (range identity should be maintained)

ahhh right, yeah the only range i used in this was one that spans from black to a dark red and it resulted in all manner of colours

This is the issue: #21

No it's ok this is another thing :) It does recolour stuff, which it probably shouldn't

So, empty squares are now automatically set to three diamonds: 🔴🟢🔵
So I reckon that should keep cells the same??? But maybe it has some weird behaviour (rightly or wrongly) due to auto-stamping or stamping??? I think a new "do nothing" instruction would still be a good idea to make sure there is no weird behaviour going on.


TE: Add test cases for PqlUtils between clause

Added test cases for date ranges aligned and misaligned with respect to data buckets, as well as cases with and without defined date formatters. Also removed unnecessary 'collection' argument.

LGTM


Python inner loop support

[kind/Feature]
Which functionality do you think we should add?
Leveraging an official image from community and Red Hat to be able to do inner loop development using Python.
Why is this needed?
Python is growing in popularity due to its usage in AI/ML based applications and robotics/automation.
Scope

TODO

"...documenting better what inner loop is and what images we actually test/support with it?
https://docs.openshift.com/container-platform/4.2/cli_reference/openshift_developer_cli/understanding-odo.html#key-features
I find what we have to be a bit vague
For example "Official full support of Node.js and Java components." I assume that isn't any possible Java image, like tomcat or eap or quarkus."

"...documenting better what inner loop is and what images we actually test/support with it?
https://docs.openshift.com/container-platform/4.2/cli_reference/openshift_developer_cli/understanding-odo.html#key-features
I find what we have to be a bit vague
For example "Official full support of Node.js and Java components." I assume that isn't any possible Java image, like tomcat or eap or quarkus."

@Preeticp I believe this was a slack auto-add of a comment or what did you have in mind with this comment as it relates to Python support?

I was trying to capture the feedback so that we could improve the docs. But you are right, I will create another docs issue for the above suggestion.

Before we start adding full support for more languages and runtimes we should solve the problem with s2i or figure out an alternative approach for the inner loop builds
The current approach where we try to hack s2i to play nice with inner loop and odo is not a good way forward.

This should be based on devfile support

Leveraging an official image from community and Red Hat to be able to do inner loop development using Python.
@sspeiche as mentioned we would have two images - one set of dev files created by odo team and other productised version created by language team. Am I assuming this correctly?

I'd assume you just use/update what is being used with community Che https://github.com/eclipse/che-devfile-registry/tree/master/devfiles/python and downstream with CodeReady Workspaces (I don't know the link off hand)

As per a recent discussion - this issue was deemed as good-to-have for devfile v2

https://github.com/eclipse/che-devfile-registry/blob/master/devfiles/python-django/devfile.yaml is probably a better starting point as it does also cover debug.

we can create a python devfile based on what gorkem has mentioned and add that to the registry to make this easy to use

/assign

added in https://github.com/odo-devfiles/registry/pull/36

done in https://github.com/odo-devfiles/registry/pull/36
/close


802.11n packets incorrectly parsed [was Payload limit for MacOS 10.15 devices (at least) - large payloads unseen]

For my MacBook MacOS 10.15 (at least) I don't see large packets addressed to its MAC address - I only see payloads of 28 bytes in length - however, it does seem to work properly for iOS devices. Other packets that may be asscoiated with this target device and have larger payloads are parsed with different MAC addresses, so this may be a problem with the parsing or (more likely) a misunderstaning of how MacOS addresses packets. This is an issue for both the ESP8266 and ESP32.

This is a problem with 802.11n packets - for which rx_ctrl.sig_mode is set to 1. The struct wifi_mgmt_hdr is no longer correct and means that the MAC addresses are inccorretly parsed, it also appears that the type is not correctly set for these packets.

Interesting my Google Home is now exclusively using 11n packets? Part of a firmware update I guess

Interesting my Google Home seem to now be exclusively using 11n packets - part of a firmware update I guess?

That doesn't seem to be true - at least not now


adding the export installations method

Coverage increased (+0.01%) to 99.539% when pulling d7577970164d21e7366be0224b9876de544b8d73 on lholmquist:export into 078112a874c8ef0bb8ed1b6c2eaf0f47759c6c6f on bucharest-gold:master.


Browse buttons in Configure environment should start from prefix path

When prefix path has been set we know roughly where the rest of the fields will be, so we should start browsing from there.

The new add environment dialog does this.


Remove-SFTPItem Should accept more than on file in Path Parameter

Remove-SFTPItem Should accept more than on file in Path Parameter

Added a -Force parameter for none empty directories to Release_v2. Still considering if I should add the ability to pass multiple locations as an array since it will mean a change from terminating error to none terminating and will impact existing scripts by users,

Hi
Please let me know if this gets implemented anytime soon. 'm fairly new to this and I need to delete multiple files from a location before pushing new files. I keep getting this error:
bin/./. is not an empty directory.

Next week

Thank you very much for the update. :)
I'm assuming the fix would include something on the lines of an -rf that can be used with Remove-Directory or in this case Remove-SFTPItem.
Also, I wanted to understand why each directory appears as a subdirectory inside a directory named '.' or '..'
My debugger took me to this part of the code in sftp.ps1
function Get-SFTPDirectoryRecursive
{
param($Path,$SFTPSession)
        $total = $Sess.Session.ListDirectory($Path)

        #List Files
        $total | ? {$_.IsDirectory -eq $false}

        #List non filtered directories
        $total | ? {$_.IsDirectory -eq $true -and @('.','..') -contains $_.name}

        #Get items in a path
        $total | ? {$_.IsDirectory -eq $true -and @('.','..') -notcontains $_.Name } |
        % {Get-SFTPDirectoryRecursive -Path $_.FullName -SFTPSession $sess}

    }

. and .. represent current folder and the one above the structure, They should not be shown in my opinion, will remove them from the output when recursively listing directories in V2.0
rom: ManasviBhargava
Sent: Thursday, August 3, 2017 6:34 AM
To: darkoperator/Posh-SSH
Cc: Carlos Perez; Assign
Subject: Re: [darkoperator/Posh-SSH] Remove-SFTPItem Should accept more thanon file in Path Parameter (#24)
Thank you very much for the update. :)
I'm assuming the fix would include something on the lines of an -rf that can be used with Remove-Directory or in this case Remove-SFTPItem.
Also, I wanted to understand why each directory appears as a subdirectory inside a directory named '.' or '..'
My debugger took me to this part of the code in sftp.ps1
function Get-SFTPDirectoryRecursive
{
param($Path,$SFTPSession)
$total = $Sess.Session.ListDirectory($Path)
    #List Files
    $total | ? {$_.IsDirectory -eq $false}

    #List non filtered directories
    $total | ? {$_.IsDirectory -eq $true -and @('.','..') -contains $_.name}

    #Get items in a path
    $total | ? {$_.IsDirectory -eq $true -and @('.','..') -notcontains $_.Name } |
    % {Get-SFTPDirectoryRecursive -Path $_.FullName -SFTPSession $sess}

}

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

Forgot to mention the Fix is the -Force paramter
From: ManasviBhargava
Sent: Thursday, August 3, 2017 6:34 AM
To: darkoperator/Posh-SSH
Cc: Carlos Perez; Assign
Subject: Re: [darkoperator/Posh-SSH] Remove-SFTPItem Should accept more thanon file in Path Parameter (#24)
Thank you very much for the update. :)
I'm assuming the fix would include something on the lines of an -rf that can be used with Remove-Directory or in this case Remove-SFTPItem.
Also, I wanted to understand why each directory appears as a subdirectory inside a directory named '.' or '..'
My debugger took me to this part of the code in sftp.ps1
function Get-SFTPDirectoryRecursive
{
param($Path,$SFTPSession)
$total = $Sess.Session.ListDirectory($Path)
    #List Files
    $total | ? {$_.IsDirectory -eq $false}

    #List non filtered directories
    $total | ? {$_.IsDirectory -eq $true -and @('.','..') -contains $_.name}

    #Get items in a path
    $total | ? {$_.IsDirectory -eq $true -and @('.','..') -notcontains $_.Name } |
    % {Get-SFTPDirectoryRecursive -Path $_.FullName -SFTPSession $sess}

}

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

-Force parameter it is. I am really in need of this to fix something at work. Please let me know when the release is available. Thank you very much!
All that I'm trying to do is remove everything from a directory called bin which consists of some folders and files. Each folder has a few files in it too. I am able to delete all files by using conditions like
if(file.IsRegularFile)
{
Remove-SFTPItem -path $file.FullName
}
But, it wouldn't read empty directories as empty as it considers each directory within '.' and '..' as a subdirectory in each, every time it reads a directory. :(

Hi - any update on the new release? :)

documenting stuff to be able to release it

Hi. Thanks for the new release.
The Get-SFTPChildItem cmdlet is not returning folders inside the target directory. I have a bin folder which contains some folders and files. The Get-SFTPChildItem only returns files in bin and files inside the subfolders but does not get the subfolders.
Also the force parameter doesn't allow for recursive delete.
Error : (Tried to delete bin with a few blank folders inside)
Method invocation failed because [SSH.SshModHelper] doesn't contain a method named 'DeleteDirectoryRecursive'.
At C:\Program Files\WindowsPowerShell\Modules\Posh-SSH\Sftp.ps1:367 char:25

                    [SSH.SshModHelper]::DeleteDirectoryRecursive($Path, $ses ...

                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : MethodNotFound

Can you give me some solution for this or maybe help me solve this problem another way? :(

Will push fixes tonight

On Aug 14, 2017, at 1:35 PM, ManasviBhargava<EMAIL_ADDRESS>wrote:
Hi. Thanks for the new release.
The Get-SFTPChildItem cmdlet is not returning folders inside the target directory. I have a bin folder which contains some folders and files. The Get-SFTPChildItem only returns files in bin and files inside the subfolders but does not get the subfolders.
Also the force parameter doesn't allow for recursive delete.
Error : (Tried to delete bin with a few blank folders inside)
Method invocation failed because [SSH.SshModHelper] doesn't contain a method named 'DeleteDirectoryRecursive'.
At C:\Program Files\WindowsPowerShell\Modules\Posh-SSH\Sftp.ps1:367 char:25
                [SSH.SshModHelper]::DeleteDirectoryRecursive($Path, $ses ...
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : MethodNotFound
Can you give me some solution for this or maybe help me solve this problem another way? :(
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/darkoperator/Posh-SSH/issues/24#issuecomment-322255855, or mute the thread https://github.com/notifications/unsubscribe-auth/AAf0HhUrlU8G5mTyVkxqgDVGPPdq66Jtks5sYIVLgaJpZM4Dw0Bu.

Fixes pushed to PS Gallery


Implicitly set merge-contribution:true on first template component

The problem
Currently both a contribution and the attribute controller.devfile.io/merge-contribution: true are required to properly "inject" an IDE in workspace Pod:
kind: DevWorkspace
(...)
spec:
  contributions:
    - name: editor
      kubernetes:
        name: che-code
  template:
    components:
      - name: tooling-container
        attributes:
          controller.devfile.io/merge-contribution: true

Considered that:

most of the time the template only has one container component.
the merge-contribution attribute makes sense in a Che scenario but not in odo or dev console scenarios (yet) so we should not specify it in registry.devfile.io.
we want the devworkspace template to match the original devfile so it's easier to

we should define a convention to apply contributions when that's not explicitly specified in the template/devfile.
A proposal

opt-in to contribution is implicit: when no component has a merge-contribution attribute, the contribution should be applied to the first container component.
out-out to contribution is explicit: the author or a devfile should explicitly set merge-contribution to false to avoid contributions to that component.
a devfile with multiple merge-contributions: true is invalid: the devworkspace controller should return an error if multiple components have merge-contribution explicitly set to true.

One grey area with implicitly including the merge-contributions: true attribute is for devfiles that do not explicitly include a container. What will happen in this case is that DWO will attempt to inject into the first container in the resolved (flattened) devfile.
For example, in our default "theia-next" sample, DWO would attempt to inject into the theia-ide component, which may not work as expected (the behavior of DWO as it is now would be to run the theia-ide-contributions container using the default image specified there -- i.e. UDI).
Currently, I believe DWO internally flattens contributions + plugin components into one list of components to start, so this behavior would be present with the theia-next-contributions sample
One of the reasons we defaulted to explicit opt-in, implicit opt-out is to be sure a workspace can be started (assuming the default image for unmerged components is suitable for running the injected component). With implicit injection, adding e.g. a busybox container to the workspace will cause startup to likely fail -- though this may not be a use case we care to support.
There are two solutions I can see here:

Make sure we explicitly opt out of merging for plugin components (i.e. set merge-contribution: false on the theia-ide component, etc.)
Try to work out a system where DWO only attempts to inject into components (and not contributions) -- this would be hard due to how contributions are treated internally and could break some (deprecated but still possible) use cases.

I am not 100% sure I have understood the problem so apologies if what I am saying doesn't make sense. My understanding:

The "theia-next" sample doesn't have any contribution so there is no problem.
The "theia-next-contributions" sample has a contribution and doesn't have a component. In this case the contribution will be used as component but it may not have been thought to be used like this so it may not work.

From my point of view, DevWorkspaces with no components should be considered invalid. For the Devfile we may want to be more permissive and allow component-less: Che, or any other tool that generates the DW, would be responsible to add the default component (this case is similar to a recent customer case).

From my point of view, DevWorkspaces with no components should be considered invalid

That's a valid option; currently devfiles with no components whatsoever are valid in order to allow for the flow "create a devfile, add components after creation". Additionally, from a "resolved workspace" perspective, the theia-next samples (including contributions) do technically have components, they're just imported from the Theia DevWorkspaceTemplate. We can imagine use cases where this set up is valuable -- for example a plugin/contribution that references a bootstrapping component (that clones the repo and sets up the devfile, in order to support non-github/gitlab/bitbucket repositories).
Regarding

The "theia-next" sample doesn't have any contribution so there is no problem.

In order to support a smoother transition to using contributions instead of plugin components, the container-contribution attribute is respected for both plugin components and contributions, so this workspace will be treated the same way as the contributions sample internally. We can (and perhaps should) change that.


Remove outdated "Note about deploying to Heroku" from docs

Heroku no longer does any special processing for the cryptography library, as it now provides wheels: https://github.com/heroku/heroku-buildpack-python/commit/c373e80c1285260e5adcbc855f54bbeb6999005c

Coverage remained the same at 100.0% when pulling 2659a12b7ec6d02fed9b0a94a8b23d767ebb8abe on brianhelba:patch-1 into 8f8cadb8a9637e3f5777c98b9a9dd7db1b044374 on orcasgit:master.

+1 to this - I'm using this on Heroku in prod without the workaround described


feat:Theme customiser

What feature?
I want to add a theme customizer that allows the user to switch the theme o the website according to their will.
Add screenshots
NA
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

we have decided to use only black, & white theme for our website.


Update composer.json for Laravel 5.5 release

Will be getting to this later this week or this weekend. thanks for sending in the PR. :)

Waiting on this as well 👍

Done, use version 0.7.0 for Laravel 5.5. :)


[SPARK-21622][ML][SparkR] Support offset in SparkR GLM

What changes were proposed in this pull request?
Support offset in SparkR GLM #16699

Test build #80194 has started for PR 18831 at commit 6ec068e.

Test build #80194 has finished for PR 18831 at commit 6ec068e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80194/
Test FAILed.

Jenkins, retest this please

Test build #80213 has started for PR 18831 at commit 6ec068e.

Test build #80213 has finished for PR 18831 at commit 6ec068e.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80213/
Test FAILed.

Merged build finished. Test FAILed.

Test build #80218 has started for PR 18831 at commit dc8ccbc.

Test build #80218 has finished for PR 18831 at commit dc8ccbc.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80218/
Test FAILed.

Test build #80219 has started for PR 18831 at commit 3c4ebf9.

Test build #80219 has finished for PR 18831 at commit 3c4ebf9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80219/
Test PASSed.

Thanks for your comments, Felix.
Addressed all issues.
@yanboliang Could you take a quick look?

To be clear, I'm not suggesting to rename the parameter. I'm suggest we should support the type being passed in as column like df$myoffset in addition to it being a string. This will be more R like

@felixcheung Sorry for misunderstand, I agree we can support df$myoffset as well, the requirement make sense for R users. Let's create a separate JIRA to track it and do this change for other similar arguments like weightCol as well. Thanks.

Thanks both for the comments. Yes, I think it's be to keep this PR on offset and we can address the other improvements later.

merged to master


[QUESTION] How to authenticate requests with SSL client certificates

First check

[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.

Description
How can I authenticate requests using SSL client certificates?
The only resource I found that could be helpful is https://www.osso.nl/blog/checking-client-ssl-certificate-from-python/, that shows some Python code to authenticate requests against their provided certificate. It uses NginX, which passes the certificate in a HTTP_X_CLIENT_CERT header if I understand correctly.
How would it work with FastAPI / uvicorn (without NginX)?
Does anyone have experience or working code on that matter? Does anyone have ideas or hints on how to implement such authentication method in FastAPI?

I read more about SSL certificates (both server and client) and think I understand how to do it.
I will not use NginX at all, but rather Gunicorn (with Uvicorn workers, but whatever). Gunicorn will receive server and client certificates, along with the certificates authority to encrypt to HTTPS but also authenticate clients. I'll also use the VerifyMode.CERT_OPTIONAL mode (1 for Gunicorn: https://docs.gunicorn.org/en/stable/settings.html#cert-reqs), allowing clients to still send requests without a certificate so I can send back an error message or authenticate them with other methods (with a cookie for example).
I just need to make sure that Gunicorn will verify that the client certificate was signed with the CA, and not just verify its validity (expiry date, etc.). If not, I'll have to use code similar to what's in the blog post I mentioned to verify this in the API code myself :confused:
I will report back later once I tried all this!

In the end I'll completely drop the client certificates, as they will cause more problems than they solve in the architecture we have. Instead, we'll use something like API keys or similar.
If you want to read about SSL anyway, here are the resources that helped me understand how it works:

https://medium.com/@sevcsik/authentication-using-https-client-certificates-3c9d270e8326
https://www.osso.nl/blog/checking-client-ssl-certificate-from-python/
https://tech-habit.info/posts/https-cert-based-auth-with-flask-and-gunicorn/
https://docs.gunicorn.org/en/stable/settings.html#ssl
https://docs.python.org/3/library/ssl.html?highlight=verifymode#ssl.CERT_OPTIONAL

Closing as my previous comment contains a partial, if not full, solution to the issue.

Thanks for reporting back and closing the issue :+1:


refactor(consumer-tlv): refactor manifest name filtering

This PR applies some refactoring to the ConsumerTlv class, moving the filtering of NDN names of ndn::Data objects to a separate method. This also gets rid of a complaint by cppcheck regarding the for loop that was being used before.

I now also refactored the iteration over the manifest names, making the code a bit more readable.

@tanim-ics Thank you for your approval! :) I will merge the PR as soon as I have figured out a compilation problem on macOS.


Powershell connect-Exchangeonline

I want to connect to exchange online in order to set permissions to an account or office365 for send and receive emails without need verification... Form daemon applications.
I'm comfigure this with Powershell and i cannot continue because i can not connect with Exchange online.
When i put the command:

Connect-ExchangeOnline -UserPrincipalName<EMAIL_ADDRESS>
Raise this error:

Connect-ExchangeOnline: A parameter cannot be found that matches parameter name 'UserPrincipalName'.

What appens. Can you help me ?
Thanks

Hi @josepmmadd , thank you for filing this bug and bringing this issue to our attention.  Does this work for you locally outside of Cloud Shell?


[Removal]: Please Remove Short Clay Shell

Hotspot Name
Short Clay Shell
Hotspot b58 Address
/hotspots/112dD11Y6wjqt72dFHKkLvCedm5XCfTWVbJ4zPMVTXABx5jGSxVU
Discord Handle
No response
Hotspot Manufacturer
Sensecap
Removal Reason
It is located where it says it is in Grosvenor road
Modifications
Yes
Extra forwarders
No
Extra antennas
No
Additional Information
I have added an external antenna from Mcgill.
Also attach is an images of our utility bill for this address.
A picture of the Sensecap miner with google maps showing the location next to it.

Hey, I'm sure someone will get around to this issue. However, I'd remove that bill as you're giving everyone your personal information.


"Save All" and "Find All" stuck at 50%

This happens every time when I try to export or find words in a bigger (20 MB) jar file.

I'm sorry I waited an hour and it completed.

If anything I've personally had an obfuscated 10 Mb jar take a couple mins but not an hour. I'm going to reclose this as I don't believe there is an issue. I think under normal circumstances, with a fairly common machine, in any environment, anyone can get the same results.


Return only the number of auto-correctable and safe offenses

Is your feature request related to a problem? Please describe.
We introduced Rubocop to a fairly large codebase and wanted to enforce only safe, auto-correctable cops while we worked on the non-auto-correctable offences. After reading through everything I could find I couldn't find out how to answer "How many auto-correctable offences do I have in my codebase" easily through the Rubocop CLI. It seems like the return count ALWAYS reports on every offence (I.e. only passes if both auto-correctable and non-auto-correctable are clear).
Describe the solution you'd like
I would like to be able to specify that Rubocop only return the number of auto-correctable and safe offenses that are in the codebase.
Describe alternatives you've considered
Our current workaround just lists all of the non-auto-correctable cops we have yet to correct in our rubocop.yml and fully omits them from the scan.
Additional context
N/A
If this is achievable I would like to know where to find the answer.. I've combed through a lot of documentation to no avail.

Yea anything to help out with this would help with my project too! Would appreciate any eyes taking a look at this,

This would be super helpful for my app!


Create Apple Books extension

Description

A simple extension to search Apple Books.
Screencast

https://user-images.githubusercontent.com/36263538/226165914-43a92480-1402-4662-95d6-32772bdb9732.mp4
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer done!

I still get the same error with the search term ee

I'm looking forward to testing this extension again 🔥
Request a new review when you are ready. Feel free to contact me here or at Slack if you have any questions.

Hi @pernielsentikaer!
I don't know why is it failing to you. Also, the error you showed seems to be React related since it is complaining that no object is being passed to the component.
https://user-images.githubusercontent.com/36263538/232302428-4f0d438c-a12d-42ed-920a-789a11b1694e.mp4


Scrollable notebook output

Refs: https://github.com/microsoft/vscode-jupyter/issues/4406

[ ] anyOS
[ ] anyOS
[ ] anyOS

Complexity: 4
create issue

There are a few known issues already, but we are looking for any feedback on the look and feel of this feature before introducing it more broadly.
known issues:
Scrollable notebook output does not reveal find results correctly
notebook Error output is not scrollable
maximum limit message should not be within the scrollable region
scrollable output region keeps resetting scroll position

set "notebook.experimental.outputScrolling": true
Create a new notebook file
Run a few cells with large outputs
Play around with different combinations of related settings (might require reloading vs code): notebook.output.textLineLimit, notebook.output.wordWrap, notebook.outputLineHeight

Try on insiders.vscode.dev as well by either:
re-opening that same noteboook with output
or
Start up a jupyter server from a command prompt to provide kernels
1. python -m pip install jupyter
2. python -m jupyter notebook --no-browser --NotebookApp.allow_origin=*
3. copy the URL that the previous command provides and
4. open a notebook -> select kernel -> existing Jupyter Server -> paste the URL from step 3

great that this is being implemented! One feature I'm missing from jupyterlab is being able to toggle scrolling no/off for a specific cell using a keyboard shortcut
Specifically, for jupyterlab I have the following in the keyboard shortcuts json:
        {
            "args": {},
            "command": "notebook:enable-output-scrolling",
            "keys": [
                "S"
            ],
            "selector": ".jp-Notebook:focus"
        },
        {
            "args": {},
            "command": "notebook:disable-output-scrolling",
            "keys": [
                "Alt S"
            ],
            "selector": ".jp-Notebook:focus"
        },

example when using it:

Nice job. Feels like we are moving in the right direction. I would be happy to try it out again once we address some of the bugs :) Thanks


the standalone controlplane need to support deployment

Cause the controlplane need to run the OCM hub, the depoyment rousources need to be supported. Otherwise, the following error will occur when joining a cluster to the controlplane hub.
$ clusteradm  join --hub-token $token --hub-apiserver ***
I1129 16:06:56.653006 2591989 recorder_in_memory.go:80] &Event{ObjectMeta:{dummy.172c19e1268f3014  dummy    0 0001-01-01 00:00:00 +0000 UTC <nil> <nil> map[] map[] [] []  []},InvolvedObject:ObjectReference{Kind:Pod,Namespace:dummy,Name:dummy,UID:,APIVersion:v1,ResourceVersion:,FieldPath:,},Reason:DeploymentCreateFailed,Message:Failed to create Deployment.apps/klusterlet -n open-cluster-management: the server could not find the requested resource (post deployments.apps),Source:EventSource{Component:clusteradm,Host:,},FirstTimestamp:2022-11-29 16:06:56.652865556 +0000 UTC m=+0.072841870,LastTimestamp:2022-11-29 16:06:56.652865556 +0000 UTC m=+0.072841870,Count:1,Type:Warning,EventTime:0001-01-01 00:00:00 +0000 UTC,Series:nil,Action:,Related:nil,ReportingController:,ReportingInstance:,}
Error: "join/operator.yaml" (*v1.Deployment): the server could not find the requested resource (post deployments.apps)

The controlplane doesn't need deployment, Cause the OCM hub is running as a binary. Thanks @ycyaoxdu !


Doesn't show result in console

I wrote 1 + 3, the console should say 4, but instead of 4 it only shows this
[Running] node "c:\Users\User\Desktop\1 +g 2.js"
[Done] exited with code=0 in 0.105 seconds

Based on the JS code your screenshot shows I think you need to take a basic introductory course in JS programming.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Support for Photo linking

It would be nice if iD could support Photo linking:
https://wiki.openstreetmap.org/wiki/Photo_linking
However, it shouldn't show the image directly in the editor, because this can introduce unwanted data traffic (and the image would probably be to small anyways).
Rather the editor should provide a link with the complete URL, which upon clicking would open in a new tab.
Ideally this link is directly available in the fields section or should be visible after clicking the i next to the tag.

Thanks @fmao - have you tried iD's Mapillary / Streetside / OpenStreetCam integrations?

No, so far I encountered only links which referred to wikimedia commons...

I'd suggest this for OpenStreetCam and Bing Streetside too but people don't seem to be linking those with tags right now.
(from @quincylvania in #7064)

"right now" as the coverage is not as good and as the editors were not as tuned for those sources. If you improve the usability, you'll be rewarded by the growth of usage.

"right now" as the coverage is not as good and as the editors were not as tuned for those sources. If you improve the usability, you'll be rewarded by the growth of usage.

@Bibi56 I agree. When we add photo linking I don't see why it wouldn't work the same way for all four photo overlay sources.

Re-reading this issue, I see it just specifies the sort of field functionality added for #7064. I've been considering "adding support for photo linking" to involve the actual linking of images from iD's street level viewer to OSM features with a UI element, so I'd say let's keep this open until we add that.

I've been considering "adding support for photo linking" to involve the actual linking of images from iD's street level viewer to OSM features with a UI element, so I'd say let's keep this open until we add that.

I talked to @cbeddow about this by mail and wanted to add a few thoughts.
Possible usecases:
a. Case: OSM object selected, does not have a mapillary-key, street level viewer shows image
=> Button on street level viewer to add the current image to the object as mapillary=

b. Case: OSM object selected, mapillary-key field was added manually via "add field", street level viewer shows image
=> Button on the key-form-element to copy the ID

c. Case: OSM object selected, mapillary-key is present
=> Button to reveal the current image in the street level viewer

Further cases would be, to handle situations where a key is already present. In this case, (a) + (b) could just overwrite the existing key; but maybe with a confirmation first?

I'm really supportive of what Tobias has suggested. Another idea is to have the button inside the viewer window itself, maybe in the corner, with "Tag Currently Selected Object w/Mapillary image ID".
As a note, it's preferred to use mapillary:image=* and not just mapillary=*.


adata.X = None raises “incompatible shape” error

If we only want to support del adata.X, we need to change the pbmc tutorial.

Do you want to allow del adata.X?

If adata.X = None should be legal, del adata.X should be too, right?
But yeah, right now there’s no way to do what the tutorial wants to …

I forget why this came up. I don't think we should allow del adata.X or adata.X = None.
Can we close this?

I stumbled on the same thing while running your 3k PBMC tutorial. Are you planning to allow del adata.X or adata.X = None? Or is this functionality gone?

I would say the tutorial is out of date. I don't think we currently support having no value for X, since it's pretty central to how AnnData objects work.


Infrastructure landscape: Remove RedBPF from list of libraries

The GitHub repository for RedBPF now displays the following banner:
This repository has been archived by the owner on Jul 11, 2023.
It is now read-only.

Therefore, the project no longer qualifies with the requirements for being listed:
The project must be actively maintained.

This commit removes it from the landscape.

Preview


Using in Xamarin apps

Is there a way to use Bugsnag in Xamarin projects (shared and Android/iOS)
I used something called Toggl.Bugsnag which is a very old implementation that was updated in 2015.

Hey @apoorvadixit-ttc, we don't currently support Xamarin, but it's on our product roadmap and is something we hope to add support for in future. Will keep you posted with any updates in this thread.


Speed rates

What feature do you want to get added on the base engine?
I would like changeable speed on songs to be added to Psych Engine.

@dum-du-m, this is already a feature. In Lua, you can use setProperty('SONG.speed',speed) to set the scroll speed.

Duplicate of #317

@dum-du-m, this is already a feature. In Lua, you can use setProperty('SONG.speed',speed) to set the scroll speed.

Duplicate of #317

not what i meant, like this https://youtu.be/HooSd3Z2ltE


Make QubitizationWalkOperator expect a BlockEncoding instead of Select and Prepare

A bit uneasy about the controlled aspect of this, forcing block encodings to be SpecializedSingleQubitControlGats seems wrong, or do we only expect singly controlled walk operators?

I think this breaks the current decomp of HamiltonianSimulationByGQSP's, which uses prepare, prepare^dagger at either end (which shouldn't be neccessary anymore I think)

I think this breaks the current decomp of HamiltonianSimulationByGQSP, which uses prepare, prepare^dagger at either end (which shouldn't be neccessary anymore I think)

Is it not necessary? (I'm just getting back to this)

I think this breaks the current decomp of HamiltonianSimulationByGQSP, which uses prepare, prepare^dagger at either end (which shouldn't be neccessary anymore I think)

Is it not necessary? (I'm just getting back to this)

So the current walk operator is: PREP . R_0 . PREP^\dagger . SELECT, but the upgraded one using block-encodings is R_0 . (PREP^\dagger . SELECT . PREP) (term in brackets is the block-enc). So the new one automatically handles the basis change as the PREP is in front instead of at the end.

GQSP assumes the block encoding is of the LCU form right? If we force the QWO to be built from a block encoding we'll need to handle this.

GQSP actually doesn't assume anything, it just applies the map
$$U \mapsto \begin{bmatrix} P(U) & \cdot \\ \cdot & \cdot \end{bmatrix}$$
Only Hamiltonian simulation assumes $U$ is a walk-operator for some hamiltonian $H$. The paper does assume an LCU form, but the algorithm works for any arbitrary walk operator. This is because the PREP / PREP^\dagger are never controlled, so you can always re-group them the new way to merge into a block-enc.
I'm not sure why they restricted the paper to a  specific form. Possibly for a simpler exposition.

Some issues:

It seemed a bit wrong (but maybe not) to make all bloq encodings GateWithRegisters just to derive from the SingleQubitControlledGate so I just duplicated the get_ctrl_system logic.
The diagrams may look a little more cluttered as the signature of the block encoding includes junk registers whereas before only Selects signature was used (which excludes the junk registers from Prepare)
I pushed alpha out of the constructor of the LCU block encodings and just grab this from Prepare
To avoid GQSP issues I expose a prepare property for the LCU block encodings so that still works without any more changes.
Make block encoding epsilon optional as it's currently not really used anywhere

It seemed a bit wrong (but maybe not) to make all bloq encodings GateWithRegisters just to derive from the SingleQubitControlledGate so I just duplicated the get_ctrl_system logic.

Maybe it's worth making SingleQubitControlledGate not derive from GateWithRegisters and instead just derive from the Bloq class and avoid the duplication here? This change can be handled as part of a separate PR (we can just open an issue for now), especially if we already have multiple call sites that derive from SingleQubitControlledGate but don't necessarily need it to be a GateWithRegisters ?

It seemed a bit wrong (but maybe not) to make all bloq encodings GateWithRegisters just to derive from the SingleQubitControlledGate so I just duplicated the get_ctrl_system logic.

Maybe it's worth making SingleQubitControlledGate not derive from GateWithRegisters and instead just derive from the Bloq class and avoid the duplication here? This change can be handled as part of a separate PR (we can just open an issue for now), especially if we already have multiple call sites that derive from SingleQubitControlledGate but don't necessarily need it to be a GateWithRegisters ?

I'll do this.

@tanujkhattar I integrated those 2 PRs PTAL when you get a chance

@tanujkhattar / @anurudhp PTAL, setting the ancilla to zero for LCUBlockEncoding fixed the OOM issue (back to the old diagrams)

@charlesyuan314 do you want to take a look at this

Latest changes look good from a block encoding perspective!


Remove kwargs from set connection defaults

#1605  👈 (View in Graphite)
main

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


Destroy memberships on User and Repo destroy

https://trello.com/c/kaZhpv02

What would you think about adding foreign keys for these relationships?  https://github.com/thoughtbot/guides/blob/master/best-practices/README.md#rails

I think we need to run this as a rake task as we don't know how much time it's going to take on production.
Should we split this into two PR's and first cleanup memberships and then add fk's ?

I think that's a good approach @arunagw

Or maybe we don't need to split the PR up? We can just make sure to run the rake task before deploying master to staging and then production


[BUG]: Table already exists error during migrations when migrations are run the first time

Report hasn't been filed before.

[X] I have verified that the bug I'm about to report hasn't been filed before.

What version of drizzle-orm are you using?
0.36.4
What version of drizzle-kit are you using?
0.27.2
Other packages
No response
Describe the Bug
Environment

Runtime: Hermes (React Native)
Driver: OP SQLite

Steps to reproduce
Setup

Create a React Native app with OP SQLite that applies a database migration (contents of migration is not significant).

Reproduction

Start the app, ensuring there is no existing database (the app has been freshly installed or the app data has been cleared).

Expected behaviour
Migrations are applied successfully.
Actual behaviour
Migrations error with message saying the first table that appears in the migration already exists.

Investigation notes
Factor: migrations start in useEffect hook
Migrations are applied in React Native using the useMigrations hook from the relevant driver (documentation for OP SQLite). For OP SQLite, the hook looks like this: https://github.com/drizzle-team/drizzle-orm/blob/10d22305b062dd0f4b0ff1594a6ae722487b7886/drizzle-orm/src/op-sqlite/migrator.ts#L41-L99
What is significant here is the migrate function, which calls SQLite core driver's migrate function (at db.dialect.migrate), is run inside the useEffect hook. When React is run in strict mode, effects will be run twice (documentation). This means the migration will be run twice, a factor in causing the error saying a table already exists.
Factor: asynchronous APIs
Another point of significance is OP SQLite having an asynchronous API. Expo SQLite's useMigration hook is similar in that it runs the migrate function inside a useEffect hook. However, as the Expo SQLite driver uses Expo SQLite's synchronous API (source), the second migration is guaranteed to run after the first one has completed. This means the checks that are run to determine if the migration should be applied have the correct state of the database: https://github.com/drizzle-team/drizzle-orm/blob/10d22305b062dd0f4b0ff1594a6ae722487b7886/drizzle-orm/src/sqlite-core/dialect.ts#L823-L835
For asynchronous APIs, the check is similar: https://github.com/drizzle-team/drizzle-orm/blob/10d22305b062dd0f4b0ff1594a6ae722487b7886/drizzle-orm/src/sqlite-core/dialect.ts#L875-L886
However, as the second migration begins before the first migration has been completed, the database read for the last migration (L875–L877) doesn't return anything so the check (L833) does not prevent the migration from being applied a second time, another factor in causing the error saying a table already exists.
Factor: broken transactions
As seen above, the statements that apply the migration run inside a transaction. In SQLite, only 1 simultaneous write transaction is allowed (SQLite documentation). As a result, it would be expected that the application is informed the database is busy. However, asynchronous transactions are ended prematurely. The logger shows something like this:
begin
<migration 1, statement 1>
commit
begin
<migration 2, statement 1>
commit
<migration 1, statement n>
// migration 2 no longer continues because of the duplicate table error

This issue is being tracked in https://github.com/drizzle-team/drizzle-orm/issues/2275.

Fixing broken transactions by awaiting the statements that build the transaction and the transaction itself doesn't actually cause the database to inform the application the database is busy. As the statements that make up the transaction are sent individually, something like this happens:
begin
begin

This causes an SQLite error saying transactions cannot be nested.

Example patch

diff --git a/node_modules/drizzle-orm/op-sqlite/session.js b/node_modules/drizzle-orm/op-sqlite/session.js
index ff84604..14cf731 100644
--- a/node_modules/drizzle-orm/op-sqlite/session.js
+++ b/node_modules/drizzle-orm/op-sqlite/session.js
@@ -27,31 +27,31 @@ class OPSQLiteSession extends SQLiteSession {
       customResultMapper
     );
   }
-  transaction(transaction, config = {}) {
+  async transaction(transaction, config = {}) {
     const tx = new OPSQLiteTransaction("async", this.dialect, this, this.schema);
-    this.run(sql.raw(`begin${config?.behavior ? " " + config.behavior : ""}`));
+    await this.run(sql.raw(`begin${config?.behavior ? " " + config.behavior : ""}`));
     try {
-      const result = transaction(tx);
-      this.run(sql`commit`);
+      const result = await transaction(tx);
+      await await this.run(sql`commit`);
       return result;
     } catch (err) {
-      this.run(sql`rollback`);
+      await this.run(sql`rollback`);
       throw err;
     }
   }
 }
 class OPSQLiteTransaction extends SQLiteTransaction {
   static [entityKind] = "OPSQLiteTransaction";
-  transaction(transaction) {
+  async transaction(transaction) {
     const savepointName = `sp${this.nestedIndex}`;
     const tx = new OPSQLiteTransaction("async", this.dialect, this.session, this.schema, this.nestedIndex + 1);
-    this.session.run(sql.raw(`savepoint ${savepointName}`));
+    await this.session.run(sql.raw(`savepoint ${savepointName}`));
     try {
       const result = transaction(tx);
-      this.session.run(sql.raw(`release savepoint ${savepointName}`));
+      await this.session.run(sql.raw(`release savepoint ${savepointName}`));
       return result;
     } catch (err) {
-      this.session.run(sql.raw(`rollback to savepoint ${savepointName}`));
+      await this.session.run(sql.raw(`rollback to savepoint ${savepointName}`));
       throw err;
     }
   }

Here's a example patch of making the useMigrations hook resilient to being called multiple times:
diff --git a/node_modules/drizzle-orm/op-sqlite/migrator.js b/node_modules/drizzle-orm/op-sqlite/migrator.js
index 02354ae..28f71a9 100644
--- a/node_modules/drizzle-orm/op-sqlite/migrator.js
+++ b/node_modules/drizzle-orm/op-sqlite/migrator.js
@@ -1,4 +1,4 @@
-import { useEffect, useReducer } from "react";
+import { useEffect, useReducer, useState } from "react";
 async function readMigrationFiles({ journal, migrations }) {
   const migrationQueries = [];
   for await (const journalEntry of journal.entries) {
@@ -48,14 +48,34 @@ const useMigrations = (db, migrations) => {
     }
   };
   const [state, dispatch] = useReducer(fetchReducer, initialState);
-  useEffect(() => {
-    dispatch({ type: "migrating" });
-    migrate(db, migrations).then(() => {
+  const [migrationQueue, setMigrationQueue] = useState([])
+
+  const runMigrations = async () => {
+    try {
+      dispatch({ type: 'migrating' })
+      await migrationQueue[0](db, migrations)
       dispatch({ type: "migrated", payload: true });
-    }).catch((error) => {
+    } catch (error) {
       dispatch({ type: "error", payload: error });
-    });
+    } finally {
+      setMigrationQueue((currentMigrations) => currentMigrations.slice(1))
+    }
+  }
+
+  useEffect(() => {
+    setMigrationQueue((currentMigrations) => [...currentMigrations, migrate])
+
+    return () => {
+      setMigrationQueue((currentMigrations) => currentMigrations.slice(0, -1))
+    }
   }, []);
+
+  useEffect(() => {
+    if (migrationQueue.length) {
+      runMigrations()
+    }
+  }, [migrationQueue.length])
+
   return state;
 };
 export {

How it works:

Migrations are pushed to a queue via an effect. This effect's cleanup function removes the pushed migration from the queue.
An effect reacts to the queue length. When it changes, it runs the first migration then removes it from the queue. As the queue length changes due to a migration being removed upon completion, the effect will keep running until no migrations are left.

[!WARNING]
This doesn't solve useMigrations being called in multiple places (not sure why you would). It only solves useMigrations being called in a single place multiple times.


Change references of Shared Secret to Signature Secret

It's possible that the API Secret and Shared Secret could be confused. Signature Secret defines exactly what that secret is used for; for signing and for checking signatures.
We could call is request signature secret to be event clearer?

Docs call it 'security_secret', so maybe that?
I think I used shared secret because that's the generic term, I think it's called that in the dashboard, but probably just the admin dashboard.
In the code, you'd do either:
$client = new Client(new Basic('key', 'secret'));

or
$client = new Client(new SharedSecret('key', 'secret'));

Perhaps Basic should change to be more specific. ApiCredentials?

Docs call it 'security_secret', so maybe that?

They do. But when I spoke to @reuvensh and we did some analysis (available on the wiki) we agreed that it was confusing to have an API Secret and a Shared Secret.
The "Shared" doesn't indicate what it's to be used for or who it's shared with. "Signature" makes it clear that it's used with signatures so I'd suggest we update the documentation to match that too. @iain-cox do you agree?
We did look at other potential names (token, key etc.), but "Signature Secret" seemed to fit best.
Since the docs are the only place where this is presently mentioned (it's not in the dashboard yet) it's just a simple update to make this change.


Add version to permalink string

Fixes #424

while this will work for the previews, will this work for the main site, where we do want https://dandiarchive.org and not gui.dandiarchive.org ?

@satra I've updated this PR, I was mistaken originally.

thanks @AlmightyYakob


Crash when installing nsp with goldleaf (atmosphere w/kosmos) 0x4a8

Error: when installing hbl.nsp using goldleaf, switch crashes when in applet mode, and app crash when using it on normal mode
it used to work just fine in my prior installation of atmosphere without kosmos, but when i did a fresh install of atmos w/kosmos, it crashes upon install

hbl.nsp is not an "installable" (aka warez) nsp. It is an ExeFS image; that is used with Atmosphère's first class hbloader support. Do not use goldleaf or any similar installer apps; they are completely unnecessary for running homebrew and only entail a ban risk.


Real-world example: https://github.com/Azure/azure-functions-durable-extension/blob/master/src/WebJobs.Extensions.DurableTask/DurableOrchestrationContextBase.cs.

Repro steps
Create the following class in a C# project:
public abstract class AbstractClass
{
    public int ConcreteInternalSet { get; internal set; }
    public virtual int VirtualPrivateSet { get; private set; }
    public virtual int VirtualProtectedSet { get; protected set; }
    public virtual int VirtualInternalSet { get; internal set; }
    public abstract int AbstractReadOnly { get; }
}

Create this test in an F# test project:
[<Fact>]
let abstractClassTest() =
    Mock<AbstractClass>().Create() |> ignore

Expected behavior
Test should pass.
Actual behavior
Fails with this error:

System.TypeLoadException : Method 'set_VirtualInternalSet' on type 'Mock.AbstractClassefa0bfaa-f64d-4c8b-b1b2-7eef47cfe38a' from assembly 'Foq.Dynamic, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' is overriding a method that is not visible from that assembly.

Stack Trace:
   at System.Reflection.Emit.TypeBuilder.TermCreateClass(RuntimeModule module, Int32 tk, ObjectHandleOnStack type)
   at System.Reflection.Emit.TypeBuilder.CreateTypeNoLock()
   at System.Reflection.Emit.TypeBuilder.CreateTypeInfo()
   at Foq.Emit.mock(MockMode mode, Type abstractType, FSharpList`1 otherTypes, FSharpList`1 calls, Object[] args, FSharpOption`1 returnStrategy)
   at Foq.Mock`1.Create()

If you remove the VirtualInternalSet property, the test passes.
Related information

Windows 10
.NET Core 2.1
Foq 1.8.0
XUnit 2.4.0

Have just hit this when trying to use Foq to mock the Microsoft.Azure.WebjobsDurableOrchestrationContextBase type  - I was attempting port the sample here https://docs.microsoft.com/en-us/azure/azure-functions/durable/durable-functions-unit-testing#unit-testing-orchestrator-functions to use Foq
It should be noted that I can use Moq in f# to re-create that sample and mock the type without issue but I really like the use of quotations in Foq!

Any progress on this issue? I just ran into it after upgrading to latest Azure.Storage.Blobs library (12.11.0). The Azure.Response type can no longer be mocked:
let resp = Foq.Mock<Azure.Response>().Create()

Fails at runtime with System.TypeLoadException : Method 'set_IsError' on type 'Mock.Responseda15e24f-4b36-4ebb-a375-a1c2b3ef1ec8' from assembly 'Foq.Dynamic, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' is  overriding a method that is not visible from that assembly.


make command - Unexpected token < in JSON

make all is failing with the following error -
SyntaxError: Unexpected token < in JSON at position 0
On further investigation - the error is coming from .env.json which is starting with -

<<EOT

The extra characters are generated by the commands -

terraform output api_env_json

and

terraform output vue_env_dev

inside Makefile
Note:
The following command avoids the extra characters -

terraform output -raw api_env_json

terraform output -raw vue_env_dev

This is because of a change in behavior in terraform 0.14.x for outputting multi-line strings in output variables.
I am dusting-off DAC and will patch to meet the new behavior.

This is because of a change in behavior in terraform 0.14.x for outputting multi-line strings in output variables.
I am dusting-off DAC and will patch to meet the new behavior.


Stricter rules for .include files

This is the first step in implementing issue #69.   It changes the checking of .include files to prohibit arbitrary references.   It is still possible to refer to the directory hosting the .include, the directory lib off the project root, and any subdirectories of those directories.   All other references are illegal.

The major version will increment as part of this PR since this is a breaking change.


Update startup-config behavior for vr-sros node.

Implement updates of startup-config behavior for vr-sros node.

Do not create and use startup config file if config file at //tftpboot/config.txt exists,
Fix issue: #1685.
Do not apply partial config if config file at //tftpboot/config.txt exists.

thanks @gusman !


The   "model_type": "genet" is not implemented

generating task from 0 to 10000
[14:29:02] * Starting Classification for 10000 Tasks...
0%|                                                  | 0/1000 [00:05<?, ?it/s]
Traceback (most recent call last):
File "/home/lly/PythonProjects/REDL/code_fsl/main.py", line 309, in
main(proced_config_dict)
File "/home/lly/PythonProjects/REDL/code_fsl/main.py", line 197, in main
raise NotImplementedError
NotImplementedError

---- train classifier
        if model_type == 'evnet':
            classifier = train_iedl(support_data, support_label, loss_type, act_type, fisher_coeff,
                                    max_iter=lbfgs_iters, verbose=False, use_wandb=use_wandb, n_ep=ii)
        elif model_type == 'menet':
            classifier = train_medl(support_data, support_label, loss_type, act_type, lamb1, lamb2, fisher_coeff,
                                    max_iter=lbfgs_iters, verbose=False, use_wandb=use_wandb, n_ep=ii)
        else:
            raise NotImplementedError

Sorry for the mistake. "genet" is the old name for "menet." Please replace all instances of "genet" with "menet." I have also corrected the configuration files.


Feature Request: Binary executable for multiple OS

Hi,
It would be fantastic to have ready-to-go binaries to use an different OS-Platforms.
Sometimes there are systems were it is not possible/wanted/allowed to install a full python-stack to run tools like yasha.
Nice to have would be:
-linux32
-linux64
-linuxARM(hf)
-windows32
-windows64
Unfortunately I don't have any experience with creating these binaries, but as a reference for releases for multiple OS, see https://github.com/borgbackup/borg/releases

Pyinstaller could work, https://pyinstaller.readthedocs.io/en/v3.3.1/operating-mode.html.