Sub menu arrows are not visible when using a light theme

Issue Type: Bug

Start VS Code
Go to File > Preferences > Color Theme
Pick a light theme
Open the File menu (or any other with sub menus)

Observed: Preferences and Open Recent menu items have invisible arrows, unless the mouse is hovered over them
Expected: Preferences and Open Recent menu items have arrows that are always visible

dark theme
light theme

VS Code version: Code - Insiders 1.31.0-insider (b313db6c58f8c02181a33cd3b5b4b17c64a5b85d, 2019-02-05T10:16:27.989Z)
OS version: Windows_NT x64 6.1.7601

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E3-1241 v3 @ 3.50GHz (8 x 3500)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.93GB (7.96GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

I can reproduce:

@sbatten @misolori didn't we change the arrow icons in this release to be smaller?

@bpasero this actually reproduces in stable as well, and I think I know why

@bpasero worse, I tried to debug it from dev tools and now I can no longer repro

got it. as a workaround, just reload the VS Code window after switching to the default light themes.
When the menu gets styled at launch, the styler picks either the theme's menu fg color or falls  back to the default fg color. since the built-in dark theme has a menu fg color, this is what is set. Later when you switch to the light theme, there is no menu fg color so it gets set to null. It needs to fallback here.


issues with reading .config/kion/config.yml

I am trying to authenticate with the scout accounts. I am able to use the CLI for all the other scout accounts dev, prod, impl but I run into a 400 error with test. What's strange is that the credentials command when supplied the values for account id and role name

working command

kion credentials --account-id<PHONE_NUMBER>87 --cloud-access-role scout-application-admin

but when I try to have kion read from the .config/kion/config.yml by just giving it the command

not working

kion credentials

I get this error message
kion: account not found (400)
and to show what values it's reading. Here is the contents of my .config/kion/config.yml
app-api-key-duration: 168h0m0s
host: cloudtamer.cms.gov
idms: 2
region: us-east-1
rotate-app-api-keys: false
session-duration: 1h0m0s
username: MYYG
account-id:<PHONE_NUMBER>87
cloud-access-role: scout-application-admin

The same method works for all other scout account environments. Is there some store of accounts that needs updated?

Nope

The file ~/.config/kion/config.yml should not contain account-id. Did you manually add it? In the README:
Read Setup again. There is no account-id.
Then read Config and kion.yml
Then read AWS CLI Credential Process.
You can set you account-id on the command line, kion.yml, and in a profile.

@CodaBool As Leslie said, you shouldn't put account-id in the global config file. Instead, specify it on the command line or in a local kion.yml.
What you've done would probably work, however, if you put double quotes around the account ID: account-id: "001265600887". Otherwise, it's interpreted as a number, and the leading zeroes are lost.

Ok good to know about the string double quotes I can see if that fixes something. I have a wrapper cli around yours that I've been using.
Wds Ops has 24 V4 accounts to manage and it seems writing to the shared global config was the easiest (although hacky) way to auth.
https://github.com/oddballteam/fast-auth

@CodaBool We also manage a bunch of accounts. The way we handle this is to create a directory for each account (a Terraform workspace) and put a kion.yml in each directory. When we do kion credentials or kion console in one of these directories, the Kion Tool automatically picks up the right account ID and Cloud Access Role.
Also, I noticed that your code has a message about logging out of the AWS console. In the latest version of the Kion Tool, you can do kion console --logout to end the current session before opening a new console.

The double quotes solves my main issue. :smile:
The console --logout seems to have issues for me.
I'm on Ubuntu with deb installed firefox. The default Ubuntu is installed with snap firefox which is probably even more restrictive about its file mounting.

firefox doesn't have access to all files on /tmp
After moving the html to a folder it could access I get a CSP block on inline script-src

But I can create a separate issue for that if you're interested.


FLAKE DETECTION PR - DO NOT MERGE

PR meant to add additional logging to provide info on https://github.com/flutter/flutter/issues/43763 where a LineMetrics test is flaking on Cirrus.
Do not review, do not merge.

Specifically, framework_tests-libraries-linux seems to be the task that flakes.

Ok, This has been modified to print data when the test would have failed. This will still continue to flake, but we will be able to see data this time.

Once this info dump is not necessary, this should be reverted.


Add doi for citation

It would be great to be able to cite your work on this!

I just enabled it! So we should get a DOI when I open release a new version (basically a new DOI with each new version release). I'm not sure if anyone will cite this though... let's see :)
I'll confirm that the DOI works in the next release and then close this issue if that's the case.


Code has stopped responding.

I am going to need help to actually get you useful information to debug this. As it stands my only data is "versions 0.10.0 and up have a bug where it randomly locks up, tells me it locked up, asks if i want to wait, and if i say yes wait, locks so hard windows itself cannot even close it without killing the process"
0.9.0 did not do this.
it will be fine running all weekend alone doing nothing, but then within minutes of actually using it it just gives up. i can constantly cause it by just using it, always happens in mid-stride while typing a string. i'm working in a php file almost every time it happened. and it happens fairly quickly (in under 10 minutes).
it affects both 0.10.0 and 0.10.1.
i'm running windows 10 on the fast insider ring, code on the insider ring, um. i got nothing else.

Can you let us know what languages are you using - for instance we have a pretty nasty PHP issue we are fixing up.  If you are using PHP that will cause this issue.

PHP, HTML, and Javascript. I can't say for sure, but I am /fairly/ certain that it almost always happens while in the middle of working on a double quoted string. Like yesterday I had `echo ""; and it happened while I was adding text inside the paragraph. I want to say it feels like it has almost always while doing that.

Nah, so it just happened while typing some CSS in a file that is running the PHP language syntax. a scoped style tag. So it's not just double quoted strings or anything like that.

@bobmajdakjr , looks like it does not matter what language you are using in .php files. Read #160 , maybe you will find out answers.

Very likely #160.

agreed, sounds like that much lower bug number.


don't double join to exp.Data

(aside from the perf issue) there are many column in exp.Data we are not exposing such as modified, which I believe I've heard requested.

We're offering a join to exp.data via the DataId column so users can always get the data's Created, CreatedBy, etc.
There's a separate request to track those columns at the individual assay result row too (which will be redundant with the exp.data copy in most cases). There's a proposal for how to implement this here:
https://www.labkey.org/home/Developer/issues/issues-details.view?issueId=38286


add more helpful --help messages

Expected results

airfoil version --help should print a helpful message for the airfoil version command
airfoil --help should print short descriptions of what each command does

Observed results

airfoil version --help resulted in an error (see screenshot below)
airfoil --help has no description for each command

Screenshots

Steps to reproduce

run airfoil version --help
run airfoil --help

Notes

According to the Gluegun docs, adding a description for each command should resolve the airfoil --help issue
The --help flag could be intercepted by each command via toolbox.parameters.option and handled accordingly. The command would need to simply implement an early return to achieve this functionality.

Added in #38


ListViewItem defaults to blue-white selection when focusable is disabled

Describe the bug
If you disable the focusable element of the listview, the items you hover over have the default windows blue-white selection bar..
To Reproduce
Steps to reproduce the behavior:
1)Add the focusable element to an existing ListView code, as shown below:
<ListView ItemsSource="{Binding GameCollection}" VerticalAlignment="Stretch" MinHeight="150" ScrollViewer.VerticalScrollBarVisibility="Visible">
                    <ListView.ItemContainerStyle>
                        <Style TargetType="ListViewItem">
                            <Setter Property="Focusable" Value="false"/>
                        </Style>
                    </ListView.ItemContainerStyle>
                    <ListView.View>
                        <GridView>
                            <GridViewColumn DisplayMemberBinding="{Binding Path=ExePath}" Header="Exe Path"/>
                            <GridViewColumn DisplayMemberBinding="{Binding Path=IconPath}" Header="Icon Path"/>
                            <GridViewColumn DisplayMemberBinding="{Binding Path=ArgumentsString}" Header="Arguments"/>
                        </GridView>
                    </ListView.View>
                </ListView>

Expected behavior
The ListViewItems would not show a change on hover or click
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

I've reproduced this bug,
You're replacing the style of Adonis with your own style, which is based on DefaultListViewItem  style.
try adding BasedOn="{StaticResource** {x:Type ListViewItem}}" , this works for me.
<ListView.ItemContainerStyle>
     <Style TargetType="ListViewItem" BasedOn="{StaticResource** {x:Type ListViewItem}}">
          <Setter Property="Focusable" Value="false" />
     </Style>
</ListView.ItemContainerStyle>

Thank you for clearing this up @rasyidf.
Indeed, when you use your own style at any point of an application, you have to specify explicitly that it should be derived from the default style of Adonis UI. Unfortunately this is necessary. If the BasedOn statement is missing the style will be based on WPF's default style which has no idea of Adonis UI's theming and thus looks weird.
PS: There is a little typo in the code block at {StaticResource** {x:Type ListViewItem}}. The asterisks have to be removed in order to compile.


Fix flaky Daemon and MimeType tracker tests

For #12567, #12637

Use a more popular package for a fake clock.
Added more comments in one of the damon pool tests.

Codecov Report

Merging #12646 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12646      +/-   ##
==========================================
+ Coverage   60.04%   60.06%   +0.01%
==========================================
  Files         657      657
  Lines       35792    35792
  Branches     5051     5051
==========================================
+ Hits        21493    21498       +5
+ Misses      13238    13221      -17
- Partials     1061     1073      +12

Impacted Files
Coverage Δ

src/client/common/utils/platform.ts
64.70% <0.00%> (-5.89%)
:arrow_down:

src/client/logging/levels.ts
61.11% <0.00%> (ø)

src/client/logging/logger.ts
62.74% <0.00%> (ø)

src/client/logging/formatters.ts
53.57% <0.00%> (ø)

src/client/logging/transports.ts
65.11% <0.00%> (ø)

src/client/common/process/pythonDaemonPool.ts
68.51% <0.00%> (+1.85%)
:arrow_up:

src/client/common/process/pythonDaemonFactory.ts
75.86% <0.00%> (+6.89%)
:arrow_up:

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 a42b026...3cf9f0f. Read the comment docs.


Default values for env vars

Providing default values for Environment Variables
Additions

Set ES_HOST/ES_PORT/DJANGO_ENV/SHEER_ELASTICSEARCH_INDEX default values

Checklist

[x ] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x ] Passes all existing automated tests

:+1: Jimmy's got a great template here that can be used for PRs. I keep it bookmarked and copy-paste into the PR body to fill in the relevant parts.

@anselmbradford cool thanks, i'll make the change


Advice on technical writing style for Google Cloud

I'm editing a training course (Architecting Hybrid and Multi-Cloud Infrastructure with Anthos v2.0) that uses many GKE terms, but I can't find an authoritative source for issues like capitalization and initialisms. For example, I'm seeing "Multi-cluster Gateways (MCG)" and "Multi-cluster Services (MCS)."
Any information would be appreciated.

You might have seen these already, but:

https://cloud.google.com/kubernetes-engine/docs/concepts/multi-cluster-services
https://cloud.google.com/kubernetes-engine/docs/how-to/multi-cluster-services
https://cloud.google.com/kubernetes-engine/docs/how-to/enabling-multi-cluster-gateways

@mafaulkner can you elaborate what specifically is the yield. Are you looking for a format or some documents around the topics you mentioned ?

/kind support
/retitle Advice on technical writing style for Google Cloud

This is the Kubernetes project. Google Cloud documentation, and the related style guide, are not part of Kubernetes.

Ji-hoon Seo's comment was helpful. We want to match the style of any product/service we're writing about. Thank you.


Load profile.json exception

Hello experts,
I followed the protocol example to build the reference server. The server generated the presigned URL when table/query endpoint is called.
Assumed that my table_url is profile.json#share.schema.table.
By using df = delta_sharing.load_as_pandas(table_url, limit=3) it loads the data well. But it has failed if I use load_as_spark.
Following code:
from pyspark.sql import SparkSession
spark = SparkSession.builder.appName("Delta Share Demo") \
    .config('spark.jars', 'packages/haddop-azure-3.3.6.jar,packages/delta-sharing-spark_2.12-0.6.4.jar') \
    .getOrCreate()

...

import delta_sharing
df = delta_sharing.load_as_spark(table_url)
df.limit(2).select("path").show()

In the error, it shows:
java.lang.RuntimeException: delta-sharing:/profile.json%23share.schema.table/123/25169076 is not a Parquet file. Expected magic number at tail, but found [0, 20, 14, 55]

Have you seen the error before?

@cic1988 looks like the file you are ingesting isn't a valid Parquet file.

@cic1988 sorry haven't seen it before.
Is this still happening?
Do you have a full stack trace?


Yangster doesn't build with @theia/next

It looks like the introduction of React in Theia breaks our typings for TSX. I get lots of errors like these:
../node_modules/@types/react/index.d.ts(2288,19): error TS2300: Duplicate identifier 'Element'.
../node_modules/@types/react/index.d.ts(2302,13): error TS2717: Subsequent property declarations must have the same type.  Property 'a' must be of type 'VNodeData', but here has type 'DetailedHTMLProps<AnchorHTMLAttributes<HTMLAnchorElement>, HTMLAnchorElement>'.

Fixed.
In sprotty: Use local JSX factories
In yang-sprotty: Avoid importing global React JSX typings
Here: Fixed a lot of minor issues with latest changes in Theia


g2o optimize all the time even there is no other constrained edge

hi, @koide3
I disable the floor detection, and use a rosbag without any loop closure to run hdl_graph_slam. The purpose of my experiment is to figure out what will g2o do while there is no other constrained edge except odom from the scan_matching_nodelet.
The results show like follows:
nodes: 144   edges: 143
iterations: 512 / 512
chi2: (before)24.9365 -> (after)3.64978e-29
time: 0.436[sec]
[ INFO] [1607912580.610194682,<PHONE_NUMBER>.959502665]: num??? 80 iterations: 33748 time: 12.2224
nodes: 146   edges: 145
iterations: 512 / 512
chi2: (before)47.0893 -> (after)3.51605e-29
time: 0.414[sec]
[ INFO] [1607912582.055616888,<PHONE_NUMBER>.396942136]: num??? 81 iterations: 34260 time: 12.6369
nodes: 148   edges: 147
iterations: 512 / 512
chi2: (before)3.51605e-29 -> (after)5.64421e-29
time: 0.485[sec]
[ INFO] [1607912583.642410340,<PHONE_NUMBER>.988057454]: num??? 82 iterations: 34772 time: 13.1221
nodes: 150   edges: 149
iterations: 35 / 512
chi2: (before)7.54993e-07 -> (after)5.54168e-29
time: 0.024[sec]
[ INFO] [1607912584.693144666,<PHONE_NUMBER>.040456984]: num??? 83 iterations: 34807 time: 13.1465
nodes: 151   edges: 150
iterations: 512 / 512
chi2: (before)27.6246 -> (after)7.58819e-29
time: 0.481[sec]
[ INFO] [1607912586.619230363,<PHONE_NUMBER>.965360703]: num??? 84 iterations: 35319 time: 13.6275

where NODELET_INFO is my extra code to accumlate the time that g2o have spent.
I also dump the inside data:
stamp<PHONE_NUMBER> 846135000
estimate
  0.161431   0.986767 -0.0151997   -21.7165
 -0.986837   0.161254 -0.0122047    15.7058
-0.0095922  0.0169699    0.99981 -0.0959998
         0          0          0          1
odom
  0.161431   0.986767 -0.0151997   -21.7164
 -0.986837   0.161254 -0.0122047    15.7058
-0.0095922  0.0169699    0.99981 -0.0959997
         0          0          0          1
accum_distance 141.422
id 135

In theory, if there are only inter frame constraint (obtained from odom), it's no need for g2o to optimize anymore as the origin pose is the best solution.However, in practice, g2o runs out of iterations(512/512) to get an estimation value which is equal to the origin pose.
Did I get something wrong? Any suggestions or discussions are appreciated!
Ps: g2o not always runs 512 iterations —— my sum of iterations is 39138 while the number of run time is 98, which means the average number of iterations is 399.

I guess you set g2o_solver_type to gn_var_cholmod, right? GN-based solvers in g2o doesn't check the convergence of the optimization while LM-based ones terminate when some convergence criteria is satisfied. Probably, by changing the optimizer to LM-based one, the optimization stops in a few steps.

hdl_indoor_2_local_new.TXT
I have check my launch file, and upload a copy here.Its solver type is lm_var_cholmod:
    <!-- typical solvers: gn_var, gn_fix6_3, gn_var_cholmod, lm_var, lm_fix6_3, lm_var_cholmod, ... -->
    <param name="g2o_solver_type" value="lm_var_cholmod" />
    <param name="g2o_solver_num_iterations" value="512" />

Maybe not this reason.

Hmm, it's strange. I reproduced the problem on my PC. I tried to save the pose graph and optimize it on g2o_viewer, and it also didn't converge until ran out of iterations. I'm not sure, but I guess g2o changed something relating to the convergence criteria. I'll take a look at recent changes on g2o to find the cause of the problem.

Hmm, it's strange. I reproduced the problem on my PC. I tried to save the pose graph and optimize it on g2o_viewer, and it also didn't converge until ran out of iterations. I'm not sure, but I guess g2o changed something relating to the convergence criteria. I'll take a look at recent changes on g2o to find the cause of the problem.

I am having the same issue as well, have you figured out why?

Hmm, it's strange. I reproduced the problem on my PC. I tried to save the pose graph and optimize it on g2o_viewer, and it also didn't converge until ran out of iterations. I'm not sure, but I guess g2o changed something relating to the convergence criteria. I'll take a look at recent changes on g2o to find the cause of the problem.

I am having the same issue as well, have you figured out why?

I found that this is caused by the graph structure. It seems lm_var optimizer causes some oscillation when a graph that has a structure like v1(fixed) -> v0 -> v2 -> v3 .... The oscillation disappeared when I swapped the vertex IDs such that it becomes v0(fixed) -> v1 -> v2 .... I'm not really sure why this happens, anyways I'll fix it later.

I found that this is caused by the graph structure. It seems lm_var optimizer causes some oscillation when a graph that has a structure like v1(fixed) -> v0 -> v2 -> v3 .... The oscillation disappeared when I swapped the vertex IDs such that it becomes v0(fixed) -> v1 -> v2 .... I'm not really sure why this happens, anyways I'll fix it later.


How to swipe cosmos to leave a rating inside a cell without swiping the cell

Please consider submitting the following information (if relevant):
• Library setup method: CocoaPods
• Version of the library. pod 'Cosmos', '~> 22.1'
• Xcode version. 11.5
• OS version. Example: iOS 13.5
I'm using a collectionView cell and inside of it I have the cosmos component. How can I swipe the cosmos to get a rating without worrying about swiping the cell?
For example if I try to swipe the cosmos to leave a rating, the cell swipes, I have no choice but to tap the cosmos. Tapping works fine but I like the swipe rating feature and don't want to lose it. I also don't want to lose the ability to swipe the cell meaning swiping the cell itself has nothing to do with swiping the cosmos to leave a rating.
My code is programmatic, I don't use storyboard at all.

answer is here, just replace slider with cosmosView
https://stackoverflow.com/a/62925429/4833705


Separate Users / Sites

I'm looking at headless CMS platforms which have the ability to have separate sites with different users without needing a new installation of cockpit. Similar to how contentfull has "Spaces"
Can cockpit be configured to account for this?

Considering that cockpit takes a php file for its config, there shouldn't be anything stopping you from swapping in a different db name depending on the route you come in on. This would give you a completely different data set with the same codebase. For example:
<?php
return [
    "database"    => [
        "server"  => "mongodb://localhost:27017",
        "options" => $_SERVER['REQUEST_URI'] === 'site-a') ? ["db" => "cockpitdb-a"] :  ["db" => "cockpitdb-b"]
    ]
];
?>

I have not tested this so not 100% sure this exact code will work out of the box

@gryphonmyers this is a nice solution for selecting the database. Thanks.
The next thing I'll need to tackle however is the user side of things.
I'd want them to be able to switch between "Sites" based on the ones they have access too.
Ideally we'd have this as a navbar drop down - showing the sites they are attributed too - click a site will load the relevant site in cockpit.
Can we customise the navbar safely?

Hm that is more complicated because now it sounds like you do want to share users between instances. Users are stored in the database like everything else, so using my suggestion, you'd actually have completely different user bases for the different "sites."
Based on what you're describing now, I'd question whether you actually need a separate "site." Cockpit is headless and is really just a place to organize and store data. A single instance of Cockpit can drive any number of "sites" via the REST API. You already have the ability to give collection-level permissions based on user group, so why not make your "sites" collections or groups of collections, and give access as needed based on user group?


frequent redirects on Chrome and Edge

Experiencing frequent redirects when logging in to Pathfinder UAT version. Behaviour observed on both Chrome and Edge browsers. Also causes tremendous processing speed up on laptops. Please troubleshoot what is the source of this slow down
URL used: https://redhat-pathfinder.6923.rh-us-east-1.openshiftapps.com/

Occuring for me also.


export-ps - a PowerShell variation of export

This is just a PowerShell version of the export command that creates PowerShell environment variable definitions that you can then feed into Invoke-Expression to yield the same effect at the shell version.

I went for a super-simple implementation given this is such a small but useful enhancement for PowerShell users. It's a pure extension without touching/refactoring any existing code so that it's a no-brainer to accept.

Thank you for the contribution!


Would it be realistic to expect to use blueprint with next?

Still learning about the react ecosystem, nextjs, and how all this works... So am confused by something basic.
Was hoping I'd be able to easily add npm components to my new next app - such as blueprint
import React from 'react'
import * as Blueprint from "@blueprintjs/core" // this line creates error below
export default () => (
  ....
)

I get this in the browser...
ReferenceError: window is not defined
    at Object.<anonymous> (/Users/danielnewman/Documents/next/camillo_client/node_modules/dom4/build/dom4.max.js:760:3)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/src/components/index.ts:8:1)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)

Am I expecting to be able to do something that is not really feasible?
Thanks for any insight...

I hope you getting this on the server. That means blueprint is not ready for SSR.

I am getting similar issues with Phenomic (a project that create a static website from React components - I am thinking about rewriting it using next) where people don't stop complaining about "window is not defined", "document is not defined" etc.
I was thinking about doing an experimental thing: injecting a jsdom instance in the global node scope so window, document are always defined. I guess most stuff will works (maybe sometimes without the proper values that they would expect from a real browser).
For now I am using this snippet for testing components with jest https://github.com/MoOx/phenomic/blob/bab74b8647ccfc20491f381303fdc340ae27ee24/src/_utils/jsdom/index.js but I was thinking it could be a good idea to inject it during SSR and see how it works. Probably via an explicit/documented experimental helper, or being a flag - as I expect "some" failures with "some" components in "some" cases :)

@arunoda - Yea the error is on server - you're correct.
Has anyone been able to use any similar UI component library like Blueprint with Next? Or is this a likely problem for any framework that has been in SSR?

@MoOx - thanks for sharing your investigations and thinking on this... Next has such a low barrier entry, yet seems to hit all the right notes, but I am definitely looking for scenario where I can focus on building an app and not having to do a lot of workaround-ing. Nothing against Next as a project - just my own needs at this stage (looking for the best, fastest way to build React app and leverage open source components and libs).

@MoOx thanks for sharing your experience. It'd be cool to provide a better developer experience by capturing the errors that are typically associated with lack of SSR (document, window, etc), and then pointing people to the wiki!

Has anyone been able to use any similar UI component library like Blueprint with Next?

I am using React-MD (Material UI) with no problems at all and I love it.


RulesVerifier should ensure that at least one of Recommendation or Description is filled in a Rule

Github's sarif upload feature requires the Text property of the Help property of each Rule to be populated. This field is a plaintext representation of the message shown to a user when an issue is found. Some rules have neither of these fields and thus result in an empty text property which fails to be uploaded to github, see #530.
The rulesverifier should require that at least one of the description or the recommendation is set, and the standard rules should all have descriptions/recommendations.

See https://github.com/microsoft/ApplicationInspector/pull/544 for a PR to require the Description field. Recommendation is a DevSkim only field and would require additional work to implement in the DevSkim verifier, but as long as there is a Description field it should resolve the original issue in #530 more cleanly than the backup message.


Add Open Sauce fonts (Open Sauce Sans, Open Sauce One, Open Sauce Two)

This PR Adds the Open Sauce Sans, Open Sauce One and Open Sauce Two fonts. Resolves #569
These files are copied as TTF from https://github.com/marcologous/Open-Sauce-Fonts and converted to WOFF and WOFF2 via https://onlinefontconverter.com/
The licenses for these fonts are also in that GitHub repo.

Hey @ayuhito thanks for the fast review!

Once ready, copy over the new package folder found in ./scripts/generic over to the main packages directory, ./fonts/generic. Make sure to delete the generated files in ./scripts/generic. Create a pull request, it'll be reviewed and added to the repository!

I was putting the fonts in fonts/generic because of this instruction in the scripts/generic/README.md, but I'll update that to fonts/other.

I was putting the fonts in fonts/generic because of this instruction in the scripts/generic/README.md, but I'll update that to fonts/other.

Yeah, this is really outdated. I'm planning on reworking the entire generic packager so adding new fonts would be much much easier in the future, but that's still a while away.

Updated!

Awesome. It should take around 15 mins to publish onto NPM once merged. Thanks!


Bug: Logixlysia incorrectly labels ESLint as a dependency, instead it should be a devDependency.

[X] I have checked the latest version.

Priority
💤 Low
Description
All ESLint dependencies are installed alongside logixlysia unnecessarily.
Steps to Reproduce
Simply run bun i logixlysia.
Expected Behavior
Installing Logixlysia should not also install ESLint.
Possible fix
Moving all ESLint-adjacent dependencies to devDependencies in the package.json should work.

Thank you for bringing this issue to our attention 💯

@SwanX1 I released latest-version at https://github.com/PunGrumpy/logixlysia/releases/tag/v3.3.1


https://balances.crypto-nerdz.org/ dead ?

Hello,
did you stop the website ?

Hello, yes the website was shutdown a couple of years ago, due to lacking of time & motivation issues.
I doubt I'll revive it from dead anytime soon.


[FLINK-34447][test] Hardens ActiveResourceManagerTest#testWorkerRegistrationTimeoutNotCountingAllocationTime

What is the purpose of the change
We're still seeing test instabilities here if the machine is slow because the release could be trigger resulting in the completion of the releaseResourceFuture. We're checking this before the assume call which would result in a failure.
Brief change log

Moves the assert down and adds another check earlier to make the purpose of this test clearer

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure

@xintongsong Do you remember what the actual purpose of this test was? Am I missing something or can we actually remove this test because it still can be unstable on slower machines. :thinking:


Add path completion for external commands

There is path completion for external files, but not external commands.  Given how often external commands are run, it would be good to have it for them as well.

There is a PR for rustyline at https://github.com/kkawakam/rustyline/pull/574 that has the required functionality, but it will need to be adjusted a bit to work properly here: executable completion should be active at the beginning of brackets, and after a single space or $, and filename completion should be active the rest of the time.  (It's probably a good idea to add completion for core forms, library functions, variables, etc. at the same time.)

Path/executable completion sorted out now.  Unsure about completion for core forms and similar: most are sufficiently short that it probably doesn't help much, so will punt that part for now.


hello are you with this program electron.js? You did

hello are you with this program electron.js? You did

Hi @omerfeyyaz,
Brackets is written on top of Chrome Embedded Framework (CEF), but there's also a Electron port available at https://github.com/brackets-userland/brackets-electron

Thank you for the information


object.method style of calling type class instance methods doesn't work.

Example,
Suppose we defined a type class
class SetVal a where
setVal :: (, a) -> bool
instance SetVal [char] where
setVal  = setHobbesArray
instance SetVal ()+( * long) where
setVal = setVariantPair
Then in your hobbes code:
mo.setVal("ABC") <-- this works
mo.setVal(mypair) <-- this doesn't  however, setVal(mo, mypair) works fine.
Error for the first call is:
Cannot unify types: (() + ( * long)) != [char]

I think that some information was lost in the website formatting here, I am guessing there are some "opaque C++ type names" above where there are blank space (e.g. "(,a)->bool" above probably has an opaque C++ type name in the first argument).
Offhand, my guess is that the type mismatch error (which you get for those two calls to .setVal) probably happens when both calls are part of a larger expression.  In isolation, my guess is that those two calls would work.
If so, this comes down to a restriction on field types -- we have a functional dependency saying that field types with the same name should have the same type.  This is probably something that we can address if somebody gets some spare cycles.


How to load controller file of modal directive dynamically

In web application there are a lot of modal template (angular foundation modal). When the modal is opened, we must give controller . because of a lot of modals , we writing  controllers of modal in js file.
if controller js of each modal  is loaded  seperately as js file ,this is not efficient for me.  so  , I want to load controller of the modal , when the modal is opened.
It is like :
var modalInstance = $modal.open({ templateUrl: 'myModalContent.html',
controller: 'modal-controller.js' })
can it be done? if so, how can I do that this,
thank your helps

The modal directive is not concerned with the loading of JS files. That should be handled by whatever method you use to compile and concatenate your app's JS files. The controller that you specify in the controller parameter is Angular's name for the controller, not the filename of the controller.


Update with latest changes

Signed-off-by: Akash Srivastava<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 681

111 of 224 (49.55%)  changed or added relevant lines in 11 files are covered.
4328 unchanged lines in 9 files lost coverage.
Overall coverage increased (+0.006%) to 26.995%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

render/storagerenderer.go
8
12
66.67%

render/host.go
10
16
62.5%

render/detailed/summary.go
8
17
47.06%

probe/kubernetes/storagepoolclaim.go
0
11
0.0%

probe/kubernetes/reporter.go
20
32
62.5%

probe/kubernetes/disk.go
0
15
0.0%

render/persistentvolume.go
15
30
50.0%

probe/kubernetes/pod.go
17
33
51.52%

probe/kubernetes/client.go
0
25
0.0%

Files with Coverage Reduction
New Missed Lines
%

probe/kubernetes/logreadcloser.go
2
81.55%

render/detailed/summary.go
2
89.66%

app/multitenant/consul_client.go
3
51.92%

report/sets.go
4
82.18%

report/node.go
4
87.1%

probe/docker/registry.go
5
72.7%

report/latest_map_generated.go
13
66.95%

render/detailed/detailed.codecgen.go
1774
30.33%

report/report.codecgen.go
2521
35.82%

Totals

Change from base Build 666:
0.006%

Covered Lines:
14452

Relevant Lines:
53536

💛  - Coveralls


Sort websites by first_published_to_production

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
Part of #1188
What's this PR do?
Changes the order field to first_published_to_production.  Does not make the additional changes necessary for new sites created in studio to show up in the list of new courses on the home page (see #1202 for that).
How should this be manually tested?
Go to https://localhost:8043/api/websites as an anonymous user and verify that the websites are listed in descending order by Website.first_published_to_production, and only imported ocw sites with metadata show up.

Codecov Report

Merging #1204 (c9bae77) into master (2ba1d71) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1204      +/-   ##
==========================================
+ Coverage   90.86%   90.87%   +0.01%
==========================================
  Files         211      211
  Lines        8738     8739       +1
  Branches     1626     1626
==========================================
+ Hits         7940     7942       +2
  Misses        689      689
+ Partials      109      108       -1

Impacted Files
Coverage Δ

websites/serializers.py
97.82% <ø> (ø)

websites/factories.py
100.00% <100.00%> (ø)

websites/views.py
97.57% <100.00%> (ø)

websites/api.py
97.24% <0.00%> (+0.68%)
:arrow_up:

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 2ba1d71...c9bae77. Read the comment docs.


Use paragraphs for Florabank1 abstract

The Florabank1 abstract is currently blocking the import of our DCAT feed, because it contains carriage returns. This should be solved in the next version of the IPT (https://github.com/gbif/ipt/issues/1231), but in the mean time, I have concatenated the abstract. Once the issue is solved, we should revert back to the old abstract (maybe also a good chance to review it), which might include multiple paragraphs.
We should test for:

[ ] line breaks in a single paragraph text field
[ ] the use of multiple paragraph text fields

Old abstract:

Florabank1 is a database that contains distributional data on the wild flora indigenous species, archeophytes and naturalised aliens) of Flanders and the Brussels Capital Region. It holds about 3 million records of vascular plants, dating from 1800 till present. Furthermore, it includes ecological data on vascular plant species, redlist category information, Ellenberg values, legal status, global distribution, seed bank etc. The database is an initiative of "Flo.Wer" (http://www.plantenwerkgroep.be), the Research Institute for Nature and Forest (INBO) (http://www.inbo.be) and the National Botanic Garden of Belgium (http://www.br.fgov.be). Florabank aims at centralizing botanical distribution data gathered by both professional and amateur botanists and to make these data available to the benefit of nature conservation, policy and scientific research. The occurrence data contained in Florabank1 are extracted from checklists, literature and herbarium specimen information. Of survey lists, the locality name (verbatimLocality), species name, observation date and IFBL square code - the grid system used for plant mapping in Belgium (Van Rompaey 1943) - are recorded. For records dating from the period 1972–2004 all pertinent botanical journals dealing with Belgian flora were systematically screened. Analysis of herbarium specimens in the collection of the National Botanic Garden of Belgium, the University of Ghent and the University of Liège provided interesting distribution knowledge concerning rare species, this information is also included in Florabank1.
The data recorded before 1972 is available through the Belgian GBIF node (http://www.gbif.org/dataset/940821c0-3269-11df-855a-b8a03c50a862), not through Florabank1, to avoid duplication of information. A dedicated portal providing access to all currently published Belgian IFBL records is available at: http://projects.biodiversity.be/ifbl.
All data in Florabank1 is georeferenced. Every record holds the decimal centroid coordinates of the > IFBL square containing the observation. The uncertainty radius is the smallest circle possible covering the whole IFBL square, which can measure 1 km² or 4 km². Florabank is a work in progress and new occurrences are added as they become available; the dataset will be updated through GBIF on a regularly base.

Two more resources were adapted to overcome the line breaks issue within string literals:

Testwat
Belgian Coccinellidae - Ladybird beetles in Belgium

These can be altered as well if the IPT issue is solved.

https://github.com/gbif/ipt/issues/1231 is fixed. I have split the description of florabank1 back over 3 paragraphs, but this time in 3 separate fields rather than in one field. No further review/adaption was done. As this was not blocking the DCAT feed, I have not republished the dataset: this metadata update will be included in the next auto-publication.
@stijnvanhoey regarding TestWat and Coccinellidae: I don't know in what sections the original descriptions were split, so I have not updated those.
Closing issue.


exports is not defined error with Webpack 2.2.0-rc.5

I'm submitting a bug report
Webpack version:
2.2.0-rc.5
Webpack Karma version:
2.0.1
Karma version:
1.4.9
Please tell us about your environment:
OSX 10.x
Current behavior:
My test suite was fully working with Webpack 2.2.0-rc.4. Upon updating to Webpack 2.2.0-rc.5 I'm getting an Uncaught ReferenceError: exports is not defined. error from any test file.
Expected/desired behavior:
Test suite continues to run correctly with Webpack 2.2.0-rc.5 as it did with Webpack 2.2.0-rc.4.
I have a feeling this has to do with the breaking change in rc.5 which made module.exports return undefined. I've made no other changes on my end and my .babelrc file is set to use the ES2015 preset with modules: false.

@farneman - This should resolve itself one the PR lands with the fix in webpack. Currently just waiting on review.

@farneman - Just checked this with webpack-rc.6 and all is well in the world of Karma again.


Forwarding click events in iOS Safari works only once

Description
In iOS Safari, in a button click event listener, calling click() on a <input type="file"> works only once.
Probably something to do with the ghost-click prevention in src/standard/gestures.html.
Live Demo
http://jsbin.com/poqiledega/3/edit?html,output
Steps to Reproduce

Add <button>Trigger</button> <input type="file"> to the document
Add a click event listener for the trigger button that forwards clicks to the file input by calling fileInput.click()
Click the button a few times

Expected Results
The file selector dialog opens in any browser any time the button is clicked.
Actual Results
In iOS Safari, only first click opens the file dialog. The second and subsequent clicks do nothing.
Browsers Affected

[ ] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari 9
[ ] Safari 8
[ ] IE 11
[x] iOS 10.2 Safari

Versions

Polymer:

master: only the first click works, as described here
v1.7.1: no clicks work
v1.7.0 and 1.6.0: all clicks work

webcomponents: v0.7.22, v0.7.23

In Polymer 2 is happening in Chrome Android too.
Comparing to Polymer 1, there is an extra timeout of MOUSE_TIMEOUT=2500, so there is a click canceling for 2,5s
https://github.com/Polymer/polymer/blob/2.0-preview/src/utils/gestures.html#L110
    POINTERSTATE.mouse.mouseIgnoreJob = Polymer.Debouncer.debounce(
          POINTERSTATE.mouse.mouseIgnoreJob
        , Polymer.Async.timeOut.after(MOUSE_TIMEOUT)
        , unset);

I'm not really sure what are you trying to accomplish with this, but couldn't this be too much intervention behind the scenes.? This is not the platform behavior.
Why not leave the platform as it is and let the user explicitly work around the problem?


vieb: 3.4.0 -> 4.5.1

Motivation for this change
supersedes #115708
tested execution and it runs

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 121845 at 7bdfa113 run on x86_64-linux 1

1 package built successfully:

vieb

2 suggestions:

warning: build-tools-in-build-inputs
rsync is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:292:7:
    |
292 |       buildInputs = [ yarn nodejs rsync ] ++ extraBuildInputs;
    |       ^

warning: build-tools-in-build-inputs
yarn is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:292:7:
    |
292 |       buildInputs = [ yarn nodejs rsync ] ++ extraBuildInputs;
    |       ^

Thanks!
You should also update the electron version to 12 (in all-packages.nix).

You should also update the electron version to 12 (in all-packages.nix).

Done. Testing now.

Package opens and works as intended.


What happens to "unverified tokens" on tonkeeper wallet?

I minted a token contract on the TON blockchain using the website "https://minter.ton.org/". Now the token is shown as "unverified token" in the tonkeeper wallet. How can I solve this problem?

Yes

Yes

How do I deal with it?

I minted a token contract on the TON blockchain using the website "https://minter.ton.org/". Now the token is shown as "unverified token" in the tonkeeper wallet. How can I solve this problem?

@luo53779525 Has your issue already been escalated?

I minted a token contract on the TON blockchain using the website "https://minter.ton.org/". Now the token is shown as "unverified token" in the tonkeeper wallet. How can I solve this problem?

@luo53779525 Has your issue already been escalated?

My problem hasn't been resolved yet.


No module named 'llama_cpp' issue, but it actually has

📜 Description
No module named 'llama_cpp' issue, but it actually has llama_cpp
👟 Reproduction steps

Source Docs select default
type "hi" in chat box
enter

👍 Expected behavior
normally response
👎 Actual Behavior with Screenshots

resoponse with "Something went wrong. Please try again later."
💻 Operating system
Linux
What browsers are you seeing the problem on?
Chrome
🤖 What development environment are you experiencing this bug on?
Docker
🔒 Did you set the correct environment variables in the right path? List the environment variable names (not values please!)
LLM_NAME=llama.cpp
EMBEDDINGS_NAME=huggingface_sentence-transformers/all-mpnet-base-v2
VITE_API_STREAMING=true
📃 Provide any additional context for the Bug.
llama-cpp-python actually installed, both python virtual/system env  (DocsGPT run on virtual python env)

[root@xxx-alma8 DocsGPT]# source venv/bin/a
-bash: venv/bin/a: No such file or directory
[root@xxx-alma8 DocsGPT]# source venv/bin/activate
(venv) [root@xxx-alma8 DocsGPT]# pip install llama-cpp-python
Requirement already satisfied: llama-cpp-python in ./venv/lib64/python3.11/site-packages (0.2.65)
Requirement already satisfied: typing-extensions>=4.5.0 in ./venv/lib/python3.11/site-packages (from llama-cpp-python) (4.11.0)
Requirement already satisfied: numpy>=1.20.0 in ./venv/lib64/python3.11/site-packages (from llama-cpp-python) (1.26.4)
Requirement already satisfied: diskcache>=5.6.1 in ./venv/lib/python3.11/site-packages (from llama-cpp-python) (5.6.3)
Requirement already satisfied: jinja2>=2.11.3 in ./venv/lib/python3.11/site-packages (from llama-cpp-python) (3.1.3)
Requirement already satisfied: MarkupSafe>=2.0 in ./venv/lib64/python3.11/site-packages (from jinja2>=2.11.3->llama-cpp-python) (2.1.5)

[root@xxx-alma8 ~]# pip install llama-cpp-python
Requirement already satisfied: llama-cpp-python in /usr/local/lib64/python3.11/site-packages (0.2.65)
Requirement already satisfied: typing-extensions>=4.5.0 in /usr/local/lib/python3.11/site-packages (from llama-cpp-python) (4.11.0)
Requirement already satisfied: numpy>=1.20.0 in /usr/local/lib64/python3.11/site-packages (from llama-cpp-python) (1.26.4)
Requirement already satisfied: diskcache>=5.6.1 in /usr/local/lib/python3.11/site-packages (from llama-cpp-python) (5.6.3)
Requirement already satisfied: jinja2>=2.11.3 in /usr/local/lib/python3.11/site-packages (from llama-cpp-python) (3.1.3)
Requirement already satisfied: MarkupSafe>=2.0 in /usr/local/lib64/python3.11/site-packages (from jinja2>=2.11.3->llama-cpp-python) (2.1.5)
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
[root@xxx-alma8 ~]#

📖 Relevant log output
backend-1   | [2024-04-27 07:12:40 +0000] [7] [ERROR] Error handling request /stream
backend-1   | Traceback (most recent call last):
backend-1   |   File "/app/application/llm/llama_cpp.py", line 17, in __init__
backend-1   |     from llama_cpp import Llama
backend-1   | ModuleNotFoundError: No module named 'llama_cpp'
backend-1   |
backend-1   | During handling of the above exception, another exception occurred:
backend-1   |
backend-1   | Traceback (most recent call last):
backend-1   |   File "/usr/local/lib/python3.11/site-packages/gunicorn/workers/sync.py", line 135, in handle
backend-1   |     self.handle_request(listener, req, client, addr)
backend-1   |  File "/usr/local/lib/python3.11/site-packages/gunicorn/workers/sync.py", line 183, in handle_request
backend-1   |     for item in respiter:
backend-1   |   File "/usr/local/lib/python3.11/site-packages/werkzeug/wsgi.py", line 256, in __next__
backend-1   |     return self._next()
backend-1   |            ^^^^^^^^^^^^
backend-1   |  File "/usr/local/lib/python3.11/site-packages/werkzeug/wrappers/response.py", line 32, in _iter_encoded
backend-1   |     for item in iterable:
backend-1   |   File "/app/application/api/answer/routes.py", line 179, in complete_stream
backend-1   |     for line in answer:
backend-1   |   File "/app/application/retriever/classic_rag.py", line 101, in gen
backend-1   |     llm = LLMCreator.create_llm(
backend-1   |           ^^^^^^^^^^^^^^^^^^^^^^
backend-1   |   File "/app/application/llm/llm_creator.py", line 27, in create_llm
backend-1   |     return llm_class(api_key, user_api_key, *args, **kwargs)
backend-1   |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
backend-1   |   File "/app/application/llm/llama_cpp.py", line 19, in __init__
backend-1   |     raise ImportError(
backend-1   | ImportError: Please install llama_cpp using pip install llama-cpp-python

### 👀 Have you spent some time to check if this bug has been raised before?

- [X] I checked and didn't find similar issue

### 🔗 Are you willing to submit PR?

None

### 🧑⚖ Code of Conduct

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

just install it in container will solve


object storage interface

S3-like object storage interface to ChubaoFS

moreover, to support billions of buckets, the metadata subsystem architecture needs to be refactored -  partition:namespace from 1:1 to 1:n.

S3-like object storage interface to ChubaoFS

The development of S3-compatible is in progress, which makes ChubaoFS provides high-level API based on http protocol.
It is designed to be compatible with native Amazon S3 interfaces, so that user can use the Amazon S3 protocol and POSIX protocol to operate ChubaoFS at the same time.
This feature will be released at late of 2019.

We need to put the design doc into the repo.

We need to put the design doc into the repo.

The design document and code of this feature have been pushed into s3 branch.

The interfaces for file data operations has been completed and merge to master branch by pull request #320 .
Following interface haven been supported:

HeadBucket
GetBucketLocation
HeadObject
PutObject
GetObject
ListObjects
ListObjectsV2
DeleteObject
DeleteObjects
CopyObject
CreateMultipartUpload
ListMultipartUploads
AbortMultipartUpload
CompleteMultipartUpload
ListParts
UploadPart
UploadPartCopy

We are working on Policy&ACL related functions and it will be done soon.

We need to put the design doc into the repo.

The design document and code of this feature have been pushed into s3 branch.

Notes:
The document about design and user guide of object subsystem which provides s3-compatible object storage interface have been merged to master branch.


2-Factor Authentication

I think the title is self-explanatory, hehe

this is going to take some time so I'm going to keep this open. if anyone has any ideas on how to tackle this please send them here lol.
I might be able to enable 2FA on my account and see what calls are being made but I want to make sure I can get all the endpoints first because that's more important right now.

It would be great if you can get a a hand on the 2FA issues first, as I believe there are more people with 2FA enabled than 2FA disable. Thanks for the repo!

I got something here @elijah-wright.
When I call the endpoint /api/v1/bloks/apps/com.bloks.www.bloks.caa.login.async.send_login_request/, as you guys know, we will get huge JSON there.
I noticed one key, namely two_factor_identifier and trusted_notification_polling_nonce. Then at intervals, Threads keep making requests to the endpoint /api/v1/two_factor/check_trusted_notification_status/, where the request header looks like this:
POST /api/v1/two_factor/check_trusted_notification_status/ HTTP/2
Host: i.instagram.com
X-Ig-App-Locale: en_US
X-Ig-Device-Locale: en_US
X-Ig-Mapped-Locale: en_US
X-Pigeon-Session-Id: UFS-fb74e206-bf44-4a25-ac3a-7530b7a66862-0
X-Pigeon-Rawclienttime:<PHONE_NUMBER>.542
X-Ig-Bandwidth-Speed-Kbps: -1.000
X-Ig-Bandwidth-Totalbytes-B: 0
X-Ig-Bandwidth-Totaltime-Ms: 0
X-Bloks-Version-Id: 5f56efad68e1edec7801f630b5c122704ec5378adbee6609a448f105f34a9c73
X-Ig-Www-Claim: 0
X-Bloks-Is-Layout-Rtl: false
X-Ig-Device-Id: 3fe9d785-c663-4d65-a377-9772cc086f8f
X-Ig-Family-Device-Id: 36c762e1-dc73-4d87-b02f-0723a0e4e9c1
X-Ig-Android-Id: android-75e76fb4f6897be0
X-Ig-Timezone-Offset: 25200
X-Ig-Nav-Chain: TwoFacLoginVerifyFragment:two_fac:1:button:1688869948.526::
X-Fb-Connection-Type: WIFI
X-Ig-Connection-Type: WIFI
X-Ig-Capabilities: 3brTv10=
X-Ig-App-Id:<PHONE_NUMBER>025917
Priority: u=3
User-Agent: Barcelona 2<IP_ADDRESS>.109 Android (33/13; 560dpi; 1440x3036; Google/google; sdk_gphone_x86_64; emu64xa; ranchu; en_US; 489720161)
Accept-Language: en-US
X-Mid: ZKoXywABAAHrsWHm2AF23gP0lQ2f
Ig-Intended-User-Id: 0
Content-Type: application/x-www-form-urlencoded; charset=UTF-8
Content-Length: 251
Accept-Encoding: gzip, deflate
X-Fb-Http-Engine: Liger
X-Fb-Client-Ip: True
X-Fb-Server-Cluster: True

two_factor_identifier=$2FA_IDENTIFIER&username=$USERNAME&device_id=android-$DEVICE_ID&trusted_notification_polling_nonces=%5B%22$NONCE%22%5D

Notes:

$2FA_IDENTIFIER is the value obtained from two_factor_identifier
$USERNAME is self-explanatory
$DEVICE_ID is self-explanatory
$NONCE is the value of trusted_notification_polling_nonce
in the parameter trusted_notification_polling_nonces you can see that there are %5B%22 and %22%5D which are [" and "] encoded, that is the trusted nonce array

I'll try using a 2-factor authentication app, or using OTP via Airtime after this.

awesome, I was just going to leave a comment about how I was going to start working on this. I'll be awaiting your next comment

@sooluh are you using frida to ssl unpin? I've tried with installing a certificate at root level, but the app still refuse to serve the requests through my proxy. I can help if I can get a hand on this.

@sooluh are you using frida to ssl unpin? I've tried with installing a certificate at root level, but the app still refuse to serve the requests through my proxy. I can help if I can get a hand on this.

Yes, of course, I'm using the patched app from here: https://github.com/Eltion/Instagram-SSL-Pinning-Bypass

For 2FA using TOTP, the application makes a request to the /api/v1/accounts/two_factor_login/ endpoint, and takes the signed_body parameter as such
signed_body=SIGNATURE.%7B%22verification_code%22%3A%22$TOTP_VALUE%22%2C%22phone_id%22%3A%22$FAMILY_DEVICE_ID%22%2C%22two_factor_identifier%22%3A%22$2FA_IDENTIFIER%22%2C%22username%22%3A%22$USERNAME%22%2C%22trust_this_device%22%3A%221%22%2C%22guid%22%3A%223fe9d785-c663-4d65-a377-9772cc086f8f%22%2C%22device_id%22%3A%22android-75e76fb4f6897be0%22%2C%22waterfall_id%22%3A%22$WATERFALL_ID%22%2C%22verification_method%22%3A%223%22%7D

There you get the JSON that must be sent, then it is encoded with URI encode and prepend with the string SIGNATURE.
{
  "verification_code": "$TOTP_VALUE",
  "phone_id": "$FAMILY_DEVICE_ID",
  "two_factor_identifier": "$2FA_IDENTIFIER",
  "username": "$USERNAME",
  "trust_this_device": "1",
  "guid": "$IG_DEVICE_ID",
  "device_id": "android-$DEVICE_ID",
  "waterfall_id": "$WATERFALL_ID",
  "verification_method": "3"
}

You may be quite confused by the new variables, and I'll explain.
$TOTP_VALUE must be obtained from your OTP application, and $2FA_IDENTIFIER, $USERNAME and $DEVICE_ID must be known.
$FAMILY_DEVICE_ID is UUID v4 which should be sent with params and request headers when logging in, you can check my previous comment on the X-Ig-Family-Device-Id header.
$IG_DEVICE_ID (guid) is the same as $FAMILY_ID, check the X-Ig-Device-Id header for the value.
$WATERFALL_ID is the same as the v4 UUID and should be sent in the params at login, but not in the header (or maybe I wasn't careful enough).
More or less the params sent at login are like this, they should be
{
  "client_input_params": {
    "device_id": "android-$DEVICE_ID",
    "password": "$PASSWORD",
    "family_device_id": "$FAMILY_DEVICE_ID",
    "contact_point": "$USERNAME"
  },
  "server_params": {
    "device_id": "android-$DEVICE_ID",
    "waterfall_id": "$WATERFALL_ID",
    "credential_type": "password",
    "family_device_id": "$FAMILY_DEVICE_ID"
  }
}

But if $FAMILY_DEVICE_ID, $IG_DEVICE_ID and $WATERFALL_ID are optional, then we don't need to add these right?

yeah, don't add optional parameters for now. anything that we can cut back on in the library, we should. I get what you're saying, it's just a different endpoint

I'm working on it, there's an error code in the reply function, there's already a pull request on #25, maybe you can merge it first.
oh btw, where should I put the utility code to parse the response? guide me so that I adapt your coding style.

the request function in the REST manager will handle everything. for advanced POST requests, look at the create function in the post manager. parsing the response isn't necessary since it's turned into JSON. eventually I'll combine it with typings

hi, in getToken function, it takes id from this.androidId, while getToken function is outside Client object, how to solve this?

there was some pr that changed that, just fixed it now

I have just made a pull request on #28. Next, I need to implement the 2FA methods through WhatsApp OTP, authenticator app, and SMS (airtime)

alr, good work everyone. closing this now, if there are any issues with the pr open another issue


[misc][OTCIS-4450] Update xml for sampleapp

Due to testkit-lite has been updated to support both and webdriver mode to run all tests, so update xml files to suit

@JianfengXu , please remove the exist manual tests(html) in xml files

Auto cases: UT pass
Manual cases: blocked by https://jira01.devtools.intel.com/browse/OTCIS-4516
let's merge it after this issue fixed


TypeError: supabaseClient.auth.signIn is not a function

Supabase client
import { createClient } from '@supabase/supabase-js';

const supabaseUrl = `${process.env.NEXT_PUBLIC_SUPABASE_URL}`;
const supabaseAnonKey = `${process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY}`;

export const supabase = createClient(supabaseUrl, supabaseAnonKey);

const handleLogin = async () => {
    const { error: signInError } = await supabaseClient.auth?.signIn(
      {
        email<EMAIL_ADDRESS>        password: '123456789'
      },
      { redirectTo: '/' }
    );
    console.log(signInError);
  };

dependencies
"next": "12.2.5",
"react": "18.2.0",
"react-dom": "18.2.0"
"@supabase/supabase-js": "^2.0.0-rc.6",
"@supabase/ui": "^0.36.5",

is supabase support react 18 ?

I think you need to call a different method in v2: https://supabase.com/docs/reference/javascript/next/auth-signinwithpassword

Thanks for @soedirgo
It work, so any version or package of supabase auth ui working with v2 ? i try using @supabase/ui and supabase-community/auth-ui-react also not supported.

I'm not familiar with these libraries, but I'm guessing they're still on v1. I suggest creating an issue on these libs to use supabase-js v2, though we need to move supabase-js out of prerelease first.

Hi, any update on this? I'm trying Auth with Next and Typescript and getting errors such as:
supabase.auth.signIn is not a function and
supabase.auth.session is not a function

Yes, you need to follow the migration notes here, though you may not be able to use auth-helpers, auth-ui, etc. atm since they're using supabase-js v1.

thanks a lot, VS code is also throwing  Property 'unsubscribe' does not exist on type '{ subscription: Subscription; }
from UserContext.tsx - anything you can suggest on how to refactor for unsubcribe?
useEffect(() => {
const { data: authListener } = supabaseClient.auth.onAuthStateChange(
async (event, session) => {
setSession(session)
setUser(session?.user ?? null)
}
)
return () => {
  authListener?.unsubscribe()
}
// eslint-disable-next-line react-hooks/exhaustive-deps

}, [])

thanks, that helped.
In UserContext.tsx I have:
return () => {
  authListener?.unsubscribe()
}
// eslint-disable-next-line react-hooks/exhaustive-deps

}, [])
VS Code is throwing: Property 'unsubscribe' does not exist on type '{ subscription: Subscription; }'.
I couldn't see anything in migration notes on this, are you able to suggest how to resolve please?

Maybe authListener?.subscription.unsubscribe()?

thanks, also helpful! Thinking I sholud've stuck with working in React and js - but sorry one last request - I've refactored session() to getSession(), but having trouble setting session in userContext as getSession returns a promise so VS code throws an error and the app doesn' recognise if the user is logged in:
const [session, setSession] = useState<Session | null>(
supabaseClient.auth.getSession()
)

Hmm I'm not familiar with React myself - cc @alaister

@strangecarr it might be worth checking out this section of the next.js guide in the supabase docs, as it's got a good example of storing the session in state, which you should be able to apply to your user context.
Hope it helps!

thanks a lot, yes got it working now. Main issue was me getting v1 and v2 mixed up. Thanks for your help!

Similar to this is the supabase.auth.user() also different now. I get a similar not function error.

Closing this as OP's issue is resolved. Please follow the migration notes linked above for other obsoleted methods.

change signIn function with signInWithPassword


meson: revise png loader meson script.

We hate spagettie code, not even build script.
this revise it to keep the build script consistency.
Enable png loader by this:
$meson . build -Dexamples=true -Dloaders="svg, png"

Description :

Issue Tickets (if any) :

Tests or Samples (if any) :

Screen Shots (if any) :

aside from that one comment I gave it builds and works fine


feat: Make Vaadin PUSH functionality work

Description
Added a build step to initialize a customized Vert.x handler that attaches
HTTP session from upgrade request to the WebSocket exhange so that Vaadin
PushHandler can work.
Fixes #5
Type of change

[ ] Bugfix
[X] Feature

Checklist

[X] I have read the contribution guide: https://vaadin.com/docs/latest/guide/contributing/overview/
[X] I have added a description following the guideline.
[X] The issue is created in the corresponding repository and I have referenced it.
[ ] I have added tests to ensure my change is effective and works as intended.
[X] New and existing tests are passing locally with my change.
[X] I have performed self-review and corrected misspellings.

Additional for Feature type of change

[ ] Enhancement / new feature was discussed in a corresponding GitHub issue and Acceptance Criteria were created.

I run mvn install multiple times, also after cleaning all modules, but can get to reproduce the logged exceptions

I couldn't see the exception in the PR build logs either. Also it doesn't fail anything as all tests pass.
Seems I get them for both dev and prod test executions.
I guess it's something with persistent connections as I found the information:
Vert.x HttpClient (which opens persistent connections by default) also throws java.io.IOException: An existing connection was forcibly closed by the remote host when the server terminates them.


Detecting face landmarks using previous frame's landmarks

Hi Patrik,
I have been testing your code for some time now for facial landmark training and tracking.
I modified rcr-track to use the landmark points from the previous frame as initialization, instead of the mean. Face detection is performed in the first frame. After that, I use the landmark points from the previous frame. But on doing this, the landmarks get scattered after just a few frames. This happens even if the face remains more or less stationary. I am pasting the code snippet that I used below. Am I doing something wrong?
(current_landmarks holds the landmarks form the previous frame, as also mentioned in your comments in the code)
if(!have_face) { ...
...
have_face=true;
}
else{
cv::Mat prev_landmarks = to_row(current_landmarks);
current_landmarks = rcr_model.detect(image, prev_landmarks);
rcr::draw_landmarks(image, current_landmarks);
have_face=true; // simply setting it to true for now for testing purposes.
}
I would greatly appreciate any help or insight from you.
Cheers!

Hi
What you're trying to do with initialising using the previous points was also my first intuition, and similar to you, it didn't work for the reason you mentioned. I think what's happening is the following:

Initialising with the mean landmarks in each frame acts as a kind of regularisation, we go back near the space of valid faces. If you're curious, what you could try is train a model and train it with a high regularisation value, so the training error would be quite large - and then see if the landmarks also get scattered in your testing. I guess this would help analyse this behaviour.

If you're initialising with the previous frame, the landmarks will be fairly close to the true location already (depending on how much the subject moves obviously). This is something that the regressor (or at least the first) actually has not learned from the training data. In the training, we initialise from the face-boxes, and thus quite far away from the true location. What I would do here is re-train a model with a different initialisation strategy, i.e. perturb around the ground-truth location of the training landmarks, and not around the face box. I've started this because I think it's a neat idea but never finished it so far.

I hope this helps, let me know how it goes and I'm glad to help more, I'm quite curious as well. (It's unfortunately the stuff that never gets mentioned in papers, for example Xiong & De La Torre don't disclose this information in their paper. But I'm actually pretty sure my idea should work (and might also be exactly how they do it).

Thank you for your detailed response. I'm working to implement your ideas. I'll post here if I manage to make any headway. Thanks again!

Cool to hear that! Just let me know if you need any help.

I have ever implement the idea "perturb around the ground-truth location of the training landmarks", however the tracking for rotation was week. So, did you have some better idea for simulating the perturbation of rotation.

You can correct for rotation by in-plane rotating the input image. That way you don't need to train it. I think there's a discussion about it in some of the (possibly closed) issues here.
Alternatively you can try perturbing in the training with a 2D rotation.
I'm closing this since the original question is resolved, and the new question is kind of off-topic.

Thanks for your response. I think it is a 3D rotation scene, so it may be
not appropriate to train with a 2D rotation. BTW, how about your progress
on the tracking work.
2016-12-27 17:57 GMT+08:00 Patrik Huber<EMAIL_ADDRESS>
Closed #19 https://github.com/patrikhuber/superviseddescent/issues/19.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/patrikhuber/superviseddescent/issues/19#event-905578815,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFxv_vE8xgED-kCX_guzlInRp7yo3T6lks5rMOETgaJpZM4GKYgn
.

I see.
The tracking works well enough for my purposes. Landmark detection isn't my main topic. I am focussing my effort on eos, the 3D fitting library.

ok, thanks very much. can you introduce the procedure for your current
tracking work.
2016-12-27 19:34 GMT+08:00 Patrik Huber<EMAIL_ADDRESS>
I see.
The tracking works well enough for my purposes. Landmark detection isn't
my main topic. I am focussing my effort on eos
http://github.com/patrikhuber/eos, the 3D fitting library.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/patrikhuber/superviseddescent/issues/19#issuecomment-269314408,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFxv_qA7pvLn3G7R58qjAITdwFEnraX7ks5rMPfegaJpZM4GKYgn
.


User Registration and Admin Updates

originally: #4718 and https://github.com/CenterForOpenScience/osf.io/pull/4796

@sloria This should be good to go

Occurs when clicking on the link after the registration email is sent and I am not logged in.

@Ghalko Thanks for catching these, I'll address them right away

I was able to change my password to a <5 character password.

@Ghalko How did you do that?

Clicked on the change password link
typed in old password
typed in new 4 char password
retyped new 4 char password
4a. hit submit
logged out
logged back in using my new password

@Ghalko I'm trying to reproduce using the change password link, but I'm getting a validation error for a 4 char password.
What was the 4 char password?

A suggestion for the admin user registration page: redirect back to the same form page rather than taking them to their home page. It is a bit disconcerting, and if you have a second one or a 100th one to add (not that we will ever get that big, though maybe) then you don't have to click the menu link again.

@zkraime The password was f1sh

@Ghalko I agree about redirecting back to the same, I had noticed that

At this moment the only thing I see that is really holding back this PR other than above issues is that you have a major problems with tabs and spaces. You should set your editor to use 4 spaces in place of tabs.

@zkraime  Let me see if I need to restart my server or run migrate.

Have you done makemigrations for your changes? @zkraime

@Ghalko I had before merging the other branch, but I needed to do it again; Ill push the migration files

@zkraime Still didn't fix the <5 char password. This time cl4m

@Ghalko  I know, I can't reproduce. I just tried that and f1sh and I'm getting that it's too short.
I'm not sure what the issue is and why it's not consistent.

@zkraime Unfortunately I am running from this same PR repo/branch. The commits I see here are the same ones I see on my environment. Is there something else that isn't getting committed, is setup dependent like local.py, or some other factor that would keep the change from taking effect on my end?

defining AUTH_PASSWORD_VALIDATORS  in settings takes care of validating the password length so if you have that, I don't see the cause of the problem

I am pinging Steve with this PR.

@zkraime I can update this if it is out of your scope for now.

@Ghalko Sorry for the delay; I'm coming in Monday and Tuesday and will finish updates to this and then finish up the Conferences PR. Thanks

Are there any tests for these changes? @Ghalko

Just a few drive-by comments. For now, see if you can clean up the URLs and maybe add some tests. Would like to merge this one soon, since this is a dependency of the Spam PRs.

@sloria Is this a good one for me to look at this week? The review queue on Jira is empty at the moment.

@whit537 Yes, go ahead with this one. After this one, there's this roadmap feature: https://github.com/CenterForOpenScience/osf.io/pull/5183 (currently not ready for review, but might be ready by the time you've finished with this PR).

@sloria Cool, will do.

What's the Jira ticket for this?

Here it is: https://openscience.atlassian.net/browse/ADM-47.

Lots of different things going on on this PR ...

Looks like this already got some review in #4796 and #4718.

Punchlist—for @zkraime

[ ] make Travis happy

Investigations—for @whit537

[ ] understand what this PR is trying to do

P.S. @sloria This seems like a great example of where smaller batch size would help keep work flowing more smoothly.

The purpose stated at https://github.com/CenterForOpenScience/osf.io/pull/4718#issue-121661618 matches the list at ADM-47, so that seems like a likely candidate for describing what this PR is about.

But there's a separate purpose at https://github.com/CenterForOpenScience/osf.io/pull/4796#issue-123025759. How does #4796 relate to this PR?

They both share a6d27826a247615b5de4838d76bf27b714aa144f, but their histories seem otherwise unrelated, for some reason (and why does GitHub report the commit comments only on the one PR and not the other?).

Looks like they sync up by the end, though:

Well, I guess we should bring in the purpose from #4796 as well, then.

I'm going back through my notes at https://github.com/CenterForOpenScience/osf.io/pull/5040#issuecomment-186386144 to stand up an admin instance again.

/me discovers an admin README ...

README added recently, in #5129.

Alright, I'm in again. Now ... what am I looking for?

These UI errors look pretty straightforward, let's start with those ...

Restrict home page by authenticated (fix "Logged in as Sign Out" bug)

Figured I'd start with this because I noticed it on develop as I was getting set up. I still see it on this PR, tho. :-/

Here's the list of changed files. Which files relate to which of the 11 or 12 purposes on this PR?
admin/base/settings/defaults.py
admin/base/urls.py
admin/base/views.py
admin/common_auth/admin.py
admin/common_auth/forms.py
admin/common_auth/migrations/0002_auto_20160205_1157.py
admin/common_auth/models.py
admin/common_auth/urls.py
admin/common_auth/views.py
admin/pre_reg/utils.py
admin/templates/base.html
admin/templates/common_auth/emails/account_creation_subject.txt
admin/templates/common_auth/emails/invitation_email.html
admin/templates/emails/account_creation_subject.txt
admin/templates/home.html
admin/templates/login.html
admin/templates/register.html
admin/templates/registration/password_change_done.html
admin/templates/registration/password_reset_email.html
admin_tests/common_auth/__init__.py
admin_tests/common_auth/test_views.py
celerybeat-schedule
tasks/__init__.py

Purpose
Files

Custom email templates

Password length restriction

UI Errors - Login view cutoff

UI Errors - Restrict home page by authenticated (fix "Logged in as Sign Out" bug)

UI Errors - Add password change page for users

UI Errors - Add forgot password link

UI Errors - Display proper Django message class type (current 'patch' handles only success and error)

UI Errors - Handle warning/failure cases for send_email_invitation in Django-admin

Set up OSF email server for admin app

Needs to be tested on staging

Add basic registration fields (email, first/last name, password)

Add FilterSelectMultiple widget for displaying/selecting/assigning Permissions (and Groups?)

Add email invitation upon new registration on custom page

admin/base/settings/defaults.py
admin/base/urls.py
admin/base/views.py
admin/common_auth/admin.py
admin/common_auth/forms.py
admin/common_auth/migrations/0002_auto_20160205_1157.py
admin/common_auth/models.py
admin/common_auth/urls.py
admin/common_auth/views.py
admin/pre_reg/utils.py
admin/templates/base.html
admin/templates/common_auth/emails/account_creation_subject.txt
admin/templates/common_auth/emails/invitation_email.html
admin/templates/emails/account_creation_subject.txt
admin/templates/home.html
admin/templates/login.html
admin/templates/register.html
admin/templates/registration/password_change_done.html
admin/templates/registration/password_reset_email.html
admin_tests/common_auth/__init__.py
admin_tests/common_auth/test_views.py
celerybeat-schedule
tasks/__init__.py

What does "Custom email templates" mean?

Is that, like, "start using custom email templates"? What were we using before? What could non-custom email templates mean in our context here?

I went through the password reset flow locally and it works afaict.

@whit537 Thanks for the review, I will address the rest next time I'm in the office which will be tomorrow

@zkraime I'm just getting started. :-) I'll update the punchlist as I work through items under "Investigations."

I've got my environment in an inconsistent state. I'm trying to run locally and getting an AttributeError when signing in. Also spent some time learning about reset and writing a kill-osf.sh script to work around kludges with this Procfile setup I'm building up.

Okay, figured it out (configuration mismatch between web app and fakecas). Back on track and proceeding with review ...

Of course, this PR is for admin and not website anyway ... :mouse:

The new admin user registration page seems to work. It's a little weird that we send a password reset instead of a "Welcome!" email, but I guess that's an MVP and this is just for internal users anyway.

[ ] Handle warning/failure cases for send_email_invitation in Django-admin

This appears not to have been done—which is fine! Another way to shed weight from this PR. :-)

Restrict home page by authenticated (fix "Logged in as Sign Out" bug)

Figured I'd start with this because I noticed it on develop as I was getting set up. I still see it on this PR, tho. :-/
Let's drop this too, I guess.

Alright, so I'm starting to wrap my head around this PR. I'm seeing three basic purposes here:

Basic Purpose
Purposes on https://github.com/CenterForOpenScience/osf.io/pull/4992#issuecomment-196511342

implement password changing
- Add password change page for users

implement password resetting
- Add forgot password link

implement user registration
- Custom email templates- Display proper Django message class type (current 'patch' handles only success and error) - Handle warning/failure cases for send_email_invitation in Django-admin- Add basic registration fields (email, first/last name, password)- Add FilterSelectMultiple widget for displaying/selecting/assigning Permissions (and Groups?)- Add email invitation upon new registration on custom page

extras
- Password length restriction- Login view cutoff- Restrict home page by authenticated (fix "Logged in as Sign Out" bug)

null
- Set up OSF email server for admin app- Needs to be tested on staging

@zkraime In the future, please scope your PRs more tightly. This PR probably should've been four or more separate PRs (per https://github.com/CenterForOpenScience/osf.io/pull/4992#issuecomment-199334873). Smaller, tightly focused PRs are easier to move through the pipeline. Thanks. :-)

Okay! First pass completed, @zkraime, and punchlist updated! :basketball: is in your court! :-)

Thanks @whit537, I will notify you when I'm finished responding to the checklist items

Sounds good, @zkraime. :)

@Ghalko Could you look into the failing test? I think it's in the post-merge-with-develop branch of the testing, but I'm not positive. Might just need to merge develop into this branch and verify that it still passes.

@Ghalko Is this PR still needed, or should it be closed?


Cannot apply tailwind CSS styles in Hextra

Description
Hi, I I'm wondering whenever it's possible to apply tailwindcss classes in Hextra, considerting Hextra already uses it?
Steps To Reproduce
Currently when I'm passing the classes they're not getting applied really:
# layouts/index.html

{{ define "main" }}

My cool string goes on here

{{ .Content }}
{{ end }}

What should have happened?
Classes are applied
** What happened instead? **
Classes are not applied
Environment
>  hugo
Start building sites …
hugo v0.138.0+extended+withdeploy darwin/arm64 BuildDate=2024-11-06T11:22:34Z VendorInfo=brew

Please take a look at the discussion here:
https://github.com/imfing/hextra/discussions/111
Hextra leverages Tailwind CSS only during the development process. Please include Tailwind CSS in your own project appropriately.

Hi @imfing ,
thanks a lot, now I've understood how to do this 😅


The OpenAPI specification obtained in XXXX is ill formed and cannot be parsed

In SwaggerHub I try to export the YAML or JSON file like this:

When I try to import the file in burp I get the error "The OpenAPI specification obtained in XXXX is ill formed and cannot be parsed".
I tried to validate the YAML (or JSON) file with the swagger-cli validate command:

It seems that the file is correct.
Unfortunately I cannot share the YAML or JSON files.
Any ideas what I can check or try on my side?

It seems that the version 3 format of OpenAPI files don't get parsed well yet.
A current workaround is to convert the file to version 2 format.
www.apimatic.io does a great job converting the files.

Thanks Mac! ❤ That did the trick.


Add missing semicolon to e2e template

Very small change to add a missing semicolon after the beforeEach() call in app.e2e.ts

Weird that the linter didn't catch it in our tests... Thanks! LGTM

Could you rebase on master please? @bschlosser

Rebased.
Yeah, I was surprised that the linter didn't catch it as well. I'll look into that.

I signed it! Corporate - BP3 Global Inc


Preparation of meeting and guide for analysis of current state of the art and implementation of citations - 2024#1

The following actions are to do, in order to prepare the meeting and the guide:

[x] #996

[x] #997

[x] #998

All done.


Improve inst.py for IoT platform

Update:

Use abspath when scp packages
Add a new inst.py for docroot on IoT

@Honry , requested by @BruceDai and had been tested.

LGTM


move all transformers under transformers/

Another daily code health workout.
Trying to reduce the number of top-level directories and eliminate hyphens in them, too.

:+1:


Please provide better progress information when building android apps.

It is quite frustrating watching a "blank" screen with no turn-indicator or progress info on how far a build for android has come:

It may stay for several minutes, during this time yoo could give som info that the build actually is running smooth and good and even how many % it has reached just by counting anticipated steps.
This will give the programmer a hunch to do other things when keeping an eye on the progress... ...instead of being suprised by the fact on what you should watch when the app started, just passed you by...
...Merry X Mas !

Hello @RoarGronmo. If you need more information on what Flutter / Android is actually doing, you can add the -v flag to flutter run and you'll get verbose information about the commands being run, files being downloaded, etc.

Thanx, just wathced around in the settings, and found this

This helped !
But visually, there should be a form for progressbar....

When build native android apps, you don't really get a progress bar either. It shows you the same verbose output, but just in a nice UI integrated into Android Studio's build tab. Since Flutter is primarily meant to work on a terminal, it makes sense that the verbose output would look like this.
In order to have a progress bar, Flutter would have to pre compute all the build tasks, from all plugins and your application, beforehand and then estimate the time it would take to build those tasks so that it can show a somewhat accurate progress bar. I my opinion, this does not seem feasible at this point.
Therefore, I am taking a decision to close this issue as, other than looking nice, a progress bar doesn't really give much practical value. For practicality, verbose logging is the best choice. If you disagree with my decision, please comment with your reasoning on it and I may reopen the issue. Thank you


Beta UniFi Network Application 6.4.47

UniFi Network Application 6.4.47
Beta firmware
Install command: fetch -o - https://git.io/JB3xL | sh -s
(install at your own risk, back up your DB first)

Upgrade from 6.3.51 to 6.4.47, successfull<<
OPNsense 21.1.9-amd64
FreeBSD 12.1-RELEASE-p19-HBSD
OpenSSL 1.1.1k 25 Mar 2021
CPU type	Intel(R) Core(TM) i3-7100U CPU @ 2.40GHz (4 cores)

Clean Install 6.4.47, successfull<<
OPNsense 21.1.9-amd64
FreeBSD 12.1-RELEASE-p19-HBSD
OpenSSL 1.1.1k  25 Mar 2021
CPU type | Intel(R) Celeron(R) CPU 1037U @ 1.80GHz (2 cores)

Clean Install 6.4.47, successfull
Upgrade from 6.4.47 to 6.4.47, successfull
OPNsense 21.7-amd64
FreeBSD 12.1-RELEASE-p19-HBSD
OpenSSL 1.1.1k  25 Mar 2021
CPU type | Intel(R) Celeron(R) CPU 1037U @ 1.80GHz (2 cores)


Merge changes back to Conveyal repo

Fast forward the Conveyal repo to incorporate recent PRs.

Codecov Report

Merging #388 into dev will decrease coverage by 0.07%.
The diff coverage is 0.43%.

@@           Coverage Diff            @@
##             dev    #388      +/-   ##
========================================
- Coverage   5.23%   5.15%   -0.08%
========================================
  Files        324     325       +1
  Lines      15405   15584     +179
  Branches    4649    4692      +43
========================================
- Hits         806     804       -2
- Misses     12449   12608     +159
- Partials    2150    2172      +22

Impacted Files
Coverage Δ

lib/editor/components/HourMinuteInput.js
0% <ø> (ø)
:arrow_up:

lib/editor/components/ZoneSelect.js
0% <ø> (ø)
:arrow_up:

lib/manager/util/index.js
2.04% <ø> (ø)
:arrow_up:

lib/types/index.js
0% <ø> (ø)
:arrow_up:

lib/editor/components/timetable/Timetable.js
0% <ø> (ø)
:arrow_up:

lib/editor/components/pattern/EditSettings.js
0% <ø> (ø)
:arrow_up:

lib/manager/reducers/projects.js
4.47% <ø> (ø)
:arrow_up:

lib/manager/components/version/FeedVersionTabs.js
0% <ø> (ø)
:arrow_up:

lib/gtfs/components/gtfsmapsearch.js
0% <ø> (ø)
:arrow_up:

lib/common/components/OptionButton.js
0% <ø> (ø)
:arrow_up:

... and 183 more

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 84e2662...a1016d9. Read the comment docs.

:tada: This PR is included in version 4.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Initial release to Play Store and F-Droid

Need to do an initial release to both the play store and f-droid. Subsequent releases will not require a ticket.

Play Store beta release completed

Issue added to f-droid repo for f-droid release


Fix Italian translations

Fixed prepositions
Better rephrasing of a sentence
Added more intents

Summary by CodeRabbit

New Features

Expanded Italian localization with new phrases for enhanced user interaction.
Improved query intent patterns for more natural user prompts in Italian.
Added flexibility in commands for the "wikiroulette" feature, allowing varied expressions for initiating actions.

Bug Fixes

Corrected phrasing and capitalization in Italian translations for clarity and correctness.

Documentation

Updated Italian dialog messages for improved linguistic accuracy and user experience.

hello and thanks for the translation!
these files are automatically managed via automations, the translations should be submitted via the translation portal
while the PR itself is valid, if it is merged the data in the translation platform will get out of sync, can you please submit the translations again via https://gitlocalize.com/users/OpenVoiceOS
tutorial - https://www.openvoiceos.org/translation
thanks!

@JarbasAl I used GitLocalize although I couldn't add all options I added here


Merged two tools: RANSAC-based and least squares-based landmark registration

Finally, all checks passed! It turned out that the previous package dependency was problematic. Updating versions of packages fixed the issue.
Thank you @bgruening


When you prompt open it is taking password , it is running only one command ?

i want to run like this.
but only first command is running and other have error
var sudoP = require('sudo-prompt');
var options = {
name: 'Productivity app',
icns: '/path/to/icns/file', // (optional)
};
sudoP.exec('apt-get install -y xdotool', options, function(error, stdout, stderr) {
console.log("stdout",stdout,"stderror",stderr);

});

sudoP.exec('xdotool', options, function(error, stdout, stderr) {
console.log("stdout",stdout,"stderror",stderr,"error",error);

});

sudoP.exec('apt-get install -y gawk', options, function(error, stdout, stderr) {
console.log("stdout",stdout,"stderror",stderr,"error",error);

});

Fixed by https://github.com/jorangreef/sudo-prompt/commit/1e11485bf4686e9cf12b82541aec9761d573f1ed
This was a bug in gksudo, which cannot be used to run multiple commands concurrently. It uses a lock file and exits if the lock file exists. I am not sure if there is a flag to disable this behavior in gksudo so I have removed support for gksudo.
Also, just on the side, I noticed that your test program runs apt-get install -y xdotool and then immediately runs xdotool. You should rather wait for the callback from the installation of xdotool to return before you assume that xdotool is installed. Otherwise, your program may try to run xdotool before the installation has finished.


Updated all adapters like bmw, broadlink2 and km200 to npm version in stable.

How long these versions are out? broadlink was updated to 1.8.0 3hours ago ... Is there already a "proof" of stability?

Th 1.8.0 was on Git for about 2 weeks and in forum used by several users. I just changed one file today to keep it in sync with the same files in other adapters before I pushed it to npm.
The others were in use and on npm since more than a month.

p.s.: The main changes in the last weeks were because of Apollon's test changes :)

Great!


Migrations

[x] Executar Migrations por linha de comando
[x] Executar Migrations pelo endpoint /migrations

https://alandev.com.br/api/v1/migrations


<EMAIL_ADDRESS>Failed to load bindings

After runnig npx create-solana-dapp@latest with default settings I got warning
Installing preset<EMAIL_ADDRESS>Failed to load bindings, pure JS will be used (try npm run rebuild?)

Hi there - thanks for creating this issue!
It's a duplicate of https://github.com/solana-developers/create-solana-dapp/issues/39, it's harmless and not something we can fix in the generator.


Fix: Remove unused method

This PR

[x] removes an unused method

Related to #908.
💁♂ Running
$ git grep -i createdFormattedOutput

yields no results other than in tests, the interface, and the single implementation.

@mdwheele
Rebased!

Thank you, @mdwheele!


Return Output with More Information Button

Is your feature request related to a problem? Please describe.
I occasionally need to have a dialog with 3 buttons. On some occasions, I can use the more information button for this purpose, but this doesn't work when I need to get output (see screenshot example below).
Describe the solution you'd like
My request is an option to get output when the more information button is selected.
Describe alternatives you've considered
This can be done with radio buttons, but that's not a good solution in all instances.
Additional context
The following is a screenshot of an example in the Shortcuts app. Thanks for the great app.

this is already available with the --alwaysreturninput argument

Thanks. I tested the --alwaysreturninput argument and that's just what I need.


TableRequestOptions for Table Storage bindings

Hi,
are there plans for implementing TableRequestOptions for Table Storage? I'm aware of the fact, that there's a workaround(not providing Table Storage binding and handling the client and configuration myself), but I don't know whether I'm reinventing the wheel.
Best regards,
Kamil

We have no plans to expose this. Your workaround is our recommended approach.


Add missing openid scope when requesting JWT token in Zitadel

Describe your changes
According to the Zitadel documentation, openid scope is required when requesting JWT tokens.
Apparently Zitadel was accepting requests without it until very recently. Now lack thereof causes 400 Bad Requests which makes it impossible to authenticate to the Netbird dashboard.
https://zitadel.com/docs/guides/integrate/service-users/client-credentials#2-authenticating-a-service-user-and-request-a-token
Issue ticket number and link
None
Checklist

[x] Is it a bug fix
[ ] Is a typo/documentation fix
[ ] Is a feature enhancement
[ ] It is a refactor
[ ] Created tests that fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fixed decimals in analysis

Old Charts:

New Charts:

@MikeMcC399 Thanks for the review (and suggestion). Looks better now.

@HelTob FYI

This PR was to resolve issue #2150.


WEBSHOP_URL = "http://<IP_ADDRESS>:3000" 遇到一些问题

请问一下，在webshop实验中
我们的网页地址是： WEBSHOP_URL = "http://<IP_ADDRESS>:3000"
请问我应该如何把这个网页替换为我自己的网页？有网页构建的代码吗

the live site is down for now. you can follow https://github.com/princeton-nlp/WebShop and create a localhost port, or use the python package directly. let me know if you run into issues, and happy to pull request if you make it work!

谢谢您的回复！

I have the Same issues I want run the code , but I got invalid observation
Please could I use any url into your code or I need to build my own webshop url @ysymyth

the website is down due to cost issues, so you need to follow https://github.com/princeton-nlp/WebShop and create a localhost port


New version: yt-dlp.yt-dlp.nightly version 2<IP_ADDRESS>704

Pull request has been created with komac v2.1.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Not sure why PowerTools logs are showing up for this package.

Automatic Validation ended with:

(Automated response - build 750.)

Automatic Validation ended with:

Installation failed with exit code -1978334972

(Automated response - build 750.)

@wingetbot waivers Add Validation-Installation-Error


Update spring cloud azure cosmos proxy and retry configuration

Fixes Azure/azure-sdk-for-java#25690
Changelog:
Update Cosmos proxy configuration, make the proxy configuration available;
Fix processor listener NPE when the consuming message exception and the consumer has not been initialized;

/azp run java - spring - tests


tmp dir might run out of space if partition is too small

To implement

[ ] change the hardcode paths for /tmp/ddc (use grep and search to find them all)
[ ] have them all reference a tmp-output-dir (or something related to that)

lack of resources


loadtable / JuliaDB

Would it be meaningful to add loadtable from JuliaDB?
Or is loadtable using one of the existing functions of your list.
I quickly glanced at your code and it seems it would be straightforward to add this (or am I missing something)?

I believe loadtable should be identical to TextParse.jl.

I think you are right. I did not realize this.


Playlist bug

Chrome / YouTube Center Developer Build 522

This has been reported several times already and is being investigated.


changefeed update topicRule failed

What did you do?

Pause changefeed, then update config with new dispatchers.topic:

Diff of changefeed config:
{Type:update Path:[Config Sink DispatchRules 0 TopicRule] From:{schema}_{table} To:{schema}2{table}}
Could you agree to apply changes above to changefeed [Y/N]
y
Error: invalid character 'p' after top-level value
What did you expect to see?
No response
What did you see instead?
As above
Versions of the cluster
Upstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):
(paste TiDB cluster version here)

Upstream TiKV version (execute tikv-server --version):
(paste TiKV version here)

TiCDC version (execute cdc version):
(paste TiCDC version here)
v6.2.0 nightly

dispatchers.matcher also cannot be updated, dispatchers.dispatcher update will be ignored.

please check the add the cdc owner server log


Adding Device information to uservoice tickets

Hello,
I am wondering if it would be possible to add device information (make and model) of the phone that is sending the uservoice ticket. If that method already exists, could you please tell me how it can be done? Otherwise, is it possible to add that on to the SDK?
Anything will be appreicated.
Best,
Jake

You should be able to set up a custom field for that information and pass it yourself. (It would also be great if the sdk would capture that information automatically, but we don't currently have that capability.)


Apk firebase-*.properties Files

Step 1: APK Analyzer firebase.properties Files? How to remove it？

Step 2:

Android device: Xiaomi
Android OS version: 8.1.0
Google Play Services version: 15.0.1
Firebase/Play Services SDK version: 16.0.1

@iiShadow are these .properties files causing a problem for you?  They just contain the SDK version and are extremely small (<100B) so they should not bloat your app or affect anything at runtime.

There hasn't been a response in 2 weeks and I think this is intended behavior, so closing the issue.

Why are these .properties files needed, what are they used for?