Uncaught Error: Cannot call function 'getPosition' on missing remote object 1168

[What I did:]

Working on my LateX file.
CTRL+S to Save.

[Is the error reproducible?]
Not that I know no
Atom: 1.35.1 x64
Electron: 3.1.6
OS: "Antergos Linux"
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot call function 'getPosition' on missing remote object 1168
Error: Cannot call function 'getPosition' on missing remote object 1168
at throwRPCError (/usr/lib/electron/resources/electron.asar/browser/rpc-server.js:145:17)
at EventEmitter. (/usr/lib/electron/resources/electron.asar/browser/rpc-server.js:367:7)
at EventEmitter.emit (events.js:182:13)
at WebContents. (/usr/lib/electron/resources/electron.asar/browser/api/web-contents.js:294:13)
at WebContents.emit (events.js:182:13)
At /usr/lib/electron/resources/electron.asar/renderer/api/remote.js:219

Error: Cannot call function 'getPosition' on missing remote object 1168
Error: Cannot call function 'getPosition' on missing remote object 1168
    at throwRPCError (/usr/lib/electron/resources/electron.asar/browser/rpc-server.js:145:17)
    at EventEmitter.<anonymous> (/usr/lib/electron/resources/electron.asar/browser/rpc-server.js:367:7)
    at EventEmitter.emit (events.js:182:13)
    at WebContents.<anonymous> (/usr/lib/electron/resources/electron.asar/browser/api/web-contents.js:294:13)
    at WebContents.emit (events.js:182:13)

    at metaToException (/usr/lib/electron/resources/electron.asar/renderer/api/remote.js:271:17)
    at Object.exception (/usr/lib/electron/resources/electron.asar/renderer/api/remote.js:219:30)
    at metaToValue (/usr/lib/electron/resources/electron.asar/renderer/api/remote.js:223:28)
    at Object.remoteMemberFunction (/usr/lib/electron/resources/electron.asar/renderer/api/remote.js:123:16)
    at AtomEnvironment.getWindowDimensions (/usr/lib/atom/src/atom-environment.js:684:34)
    at AtomEnvironment.storeWindowDimensions (/usr/lib/atom/src/atom-environment.js:721:34)
    at WindowEventHandler.handleWindowBlur (/usr/lib/atom/src/window-event-handler.js:191:26)

Commands
     -3:12.1.0 core:move-up (input.hidden-input)
     -2:47.3.0 autocomplete-plus:confirm (input.hidden-input)
     -2:46.5.0 core:backspace (input.hidden-input)
     -2:45.9.0 editor:newline (input.hidden-input)
 22x -2:43.7.0 core:backspace (input.hidden-input)
     -1:55.1.0 intentions:highlight (input.hidden-input)
     -1:55 core:paste (input.hidden-input)
     -1:53.7.0 core:backspace (input.hidden-input)
     -1:53.3.0 core:move-right (input.hidden-input)
     -1:53.2.0 core:move-down (input.hidden-input)
     -1:51.5.0 editor:newline (input.hidden-input)
     -1:51.1.0 core:backspace (input.hidden-input)
     -0:31.1.0 editor:newline (input.hidden-input)
     -0:27.3.0 intentions:highlight (input.hidden-input)
     -0:27.2.0 core:paste (input.hidden-input)
  6x -0:24.5.0 core:backspace (input.hidden-input)

Non-Core Packages
autocomplete-ansible 0.1.24
busy-signal 2.0.1
file-icons 2.1.31
file-types 1.0.1
ftp-remote-edit 0.13.6
intentions 1.1.5
language-ansible 0.2.2
language-latex 1.2.0
latex 0.50.2
latex-completions 0.3.6
linter 2.3.0
linter-spell 0.15.0
linter-spell-latex 0.11.0
linter-ui-default 1.7.1
move-panes 0.2.0
pretty-json 2.0.3

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/16630 so you can subscribe there for updates if you'd like (we believe that this will be fixed in a future Electron upgrade).

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Generic authorship and uncle inclusion module

This module aims to make an author available to runtime modules in order to query. It does so in a way which intends to be used with the inherent digest abstraction: consensus engines will implement a FindAuthor trait for extracting an author ID from a block header. This will probably have to coupled with a special FindAuthor wrapper for Session, which converts from session keys to account IDs.
It also manages a set of uncles (from up to configurable number of blocks back) and handles import of them. Generation of the Authorship::set_uncles inherent is not currently done in this PR. Extra care has to be taken about importing uncles in a PoS scenario because of the possibility of an unbounded number of equivocations. This PR includes an uncle inclusion filter for mitigating that.
TODO:

[x] SessionFindAuthor wrapper (deferred to https://github.com/paritytech/substrate/issues/2949 )
[x] scrape author
[x] Tests

@DemiMarie-parity Opinions on whether this is using inherent digests as you intended as well as compatibility with #2929 would be appreciated along with general review.
@dvc94ch Thoughts on #2949 ?

Pushed a commit to cover an index-based variant of https://github.com/paritytech/substrate/issues/2949 as well. The idea is that BABE/Aura/other consensus tells us the index of the author in the canonical list of validators.


TypeError

python3 train.py --problem wmt32k --output_dir ./output --data_dir ./wmt32k_data
Traceback (most recent call last):
File "train.py", line 211, in
main()
File "train.py", line 156, in main
opt.batch_size, device, opt)
File "/home/vivien/Bureau/transformer-pytorch/dataset/problem.py", line 12, in prepare
translation.prepare(max_length, batch_size, device, opt, data_dir)
File "/home/vivien/Bureau/transformer-pytorch/dataset/translation.py", line 157, in prepare
pad_token=pad, lower=True, eos_token='')
TypeError: init() got an unexpected keyword argument 'is_target'

Hello. it requires the latest version of torchtext. please upgrade your torchtext, and actually is_target is not necessary, so I just removed it for now.


Add yourself to the website

In the getting started guide, there are instructions. Please do this before the next group meeting. Any questions can be asked here in this github issue.
http://arfc.npre.illinois.edu/manual/guides/website
(also, Ansh, Gwen, and Xin)

@hzhang21 completed!

@gwenchee @kamuda1 @smpark7 completed!

I think the only remaining member to add themselves is Xin . I'll make a new issue for that and this one can be closed. Thanks team!!!


Add a tooltip to the fingerprint duration parameter in Dashboard

Add a tooltip to the fingerprint duration input in the Dashboard explaining what fingerprint duration is.

Hey @tanyas-codes! Thanks, yes sure.
The idea is to add a tooltip next to the Fingerprint duration tag when creating an off-chain group. The text will be:
The fingerprint is a value that represents a group. Right now the fingerprint is the Merkle Root of an Incremental Merkle Tree. Every time the group changes, the fingerprint changes too.

The fingerprint duration indicates how long old fingerprints will be valid to verify proofs successfully.

More information about it: https://github.com/semaphore-protocol/semaphore/issues/98

yes ! do assign me this issue .
I will start working on this ,also if the text seems lengthier then i will briefly shorten it (summarize)  and ask you if it's okay to proceed .

Hey @tanyas-codes also following up on this one :)
Were you able to work on it? Any troubles or work pending?

@aguzmant103 Yes ,I had exams all this while so could'nt find the time ,I'll start with this issue this week !

Amazing no worries :) @tanyas-codes  thanks for letting us know! Looking forward to your contributions 🙌

@vplasencia Hello ! I've made some draft design using the color pallete of bandada,I've attached the screenshots below,do let me know the modifications.
1)If the text length is apt?
2)the width of the tooltip
https://github.com/bandada-infra/bandada/assets/107319582/2a02f8fb-9756-4463-9d3d-0a70e4ffffdd
changed the width

hello!Thankyou so much ! @vplasencia ,Since you liked both of them for now I made a PR with design 1.
And made the changes you said above .


Standalone simple binner

@sidneymau @bechtol do we want to pull out the simple binner into it's own repo that depends on ugali? Maybe @sidneymau did this already, or maybe it's more work than we want?

I think this has pretty much happened with sidneymau/simple.


Enum complex type is not supported in the Avro schema definition

The latest (0.4.0 version) Gradle Avro plugin seems to have problems with the 'enum' complex type in Avro schema. It throws a Gradle exception in the following part of the code:
if (!nextPass.isEmpty()) {
    throw new GradleException(String.format("Failed to compile schema definition files due to
    undefined names: %s", nextPass));
}

I'm sorry to hear that you're having difficulty with the plugin.  I've added functional test coverage for enum handling to the automated test suite, and wasn't successful in reproducing this issue.  Can you provide a set of schema files that demonstrates the failure?
https://github.com/commercehub-oss/gradle-avro-plugin/blob/master/src/test/groovy/com/commercehub/gradle/plugin/avro/EnumHandlingFunctionalSpec.groovy

Here is the modified example file from your distribution.
{
 "namespace": "example.avro",
 "type": "record",
 "name": "User",
 "fields": [
     {"name": "name", "type": "string"},
     {"name": "kind",  "type": "enum", "symbols" : ["ONE", "TWO", "THREE"]}
 ]

}
It seems that the 'symbols' is not supported. Here is the link to the Avro spec.: http://avro.apache.org/docs/current/spec.html#schema_complex

Here is the modified example file from your distribution.
{
 "namespace": "example.avro",
 "type": "record",
 "name": "User",
 "fields": [
     {"name": "name", "type": "string"},
     {"name": "kind",  "type": "enum", "symbols" : ["ONE", "TWO", "THREE"]}
 ]

}
It seems that the 'symbols' is not supported. Here is the link to the Avro spec.: http://avro.apache.org/docs/current/spec.html#schema_complex

The test coverage I added this morning includes symbols, such as in this file.
However, thanks to your example, I'm now able to reproduce the failure.  As far as I an tell, it doesn't affect enums defined in their own schema file (which can then be referenced from other schema files) or enums defined within a union.  So far, it looks like it's only affecting enums defined directly in a field definition.  I'm investigating whether this may be an upstream bug in avro-compiler.

Pretty sure I figured out the problem.  Your syntax is incorrect.  Instead of
{
  "namespace": "example.avro",
  "type": "record",
  "name": "User",
  "fields": [
    {"name": "name", "type": "string"},
    {"name": "kind", "type": "enum", "symbols" : ["ONE", "TWO", "THREE"]}
  ]
}

you need
{
  "namespace": "example.avro",
  "type": "record",
  "name": "User",
  "fields": [
    {"name": "name", "type": "string"},
    {
      "name": "kind",
      "type": {
        "name": "Kind",
        "type": "enum",
        "symbols" : ["ONE", "TWO", "THREE"]
      }
    }
  ]
}

The error message given by avro-compiler was:
"enum" is not a defined name. The type of the "kind" field must be a defined name or a {"type": ...} expression.

I'm going to pursue some changes to the plugin to make errors thrown by the compiler more visible to aid in troubleshooting this type of issue.

Thanks a lot! At any rate, I fully agree that a more helpful error message would be much appreciated.

@evarga Please try version 0.5.0 when you have a chance.

I have just gave it a try, and it works perfectly! Thanks again!

According to Avro 1.10.2 (https://avro.apache.org/docs/current/spec.html#Enums), the below should be supported, however it fails with the same error message:
"enum" is not a defined name. The type of the "type" field must be a defined name or a {"type": ...} expression.
{
  "type": "enum",
  "name": "Suit",
  "symbols" : ["SPADES", "HEARTS", "DIAMONDS", "CLUBS"]
}


Selected text is invisible on Cursor (an editor based on VS Code)

When using Blockman on Cursor, which is a fork of VS Code, we can't see the box containing the text selection.
For example, here is Cursor with Blockman turned off. While I'm not selecting anything, the editor looks like this:

If I activate Blockman, it looks like this:

When I try to select text anywhere:

I know that this is an extension for VS Code, but I thought that maybe you knew a simple solution/setting value for Blockman to also work on Cursor.
Thank you for this great extension!

Thanks for the lovely feedback. Well, it is super strange, because blockman boxes have z-index about -9000 and even less. I tried to investigate it now but so far I could not find any difference between classic vs code and Cursor. Classic VS Code does not seem to have such issue.
I inspected the elements with F1 then Toggle Developer Tools in both IDEs. Blockman renders blocks with the vs code api method vscode.window.createTextEditorDecorationType and vs code renders them as css before and css after boxes. VS Code renders the selected text rectangle as regular div box with class selected-text. I tried to play with z-index values with many elements, but I could not find what causes the issue. Maybe you could also help me investigate it?

Honestly, I wouldn't know how to modify your extension to make it work on Cursor (I don't know how to code extensions!)
But I somewhat randomly found a workaround, I don't know exactly why it works 😅 It was thanks to this comment https://github.com/microsoft/vscode/issues/21898#issuecomment-2009938306 from

I installed Custom CSS and JS Loader and followed the instructions.
In the custom CSS file, I put the rules from the Github comment, but I changed the z-index value at line 31 from 1 to 5 (the problem is still there at 4 and below)

/* prevents border overlap */
/* `height` value might need to be changed to suit your font settings. */
.cdr.selectionHighlight,
.cslr.selected-text{
  height: 17px !important;
}
/* hides extraneous leading/trailing selection edges. */
/* `width` value might need to be changed to suit your font settings. */
/* potentially could require additional workarounds if additional artifacts show up in specific contexts, or if it unintendedly affects selections that should be visible (single characters, etc.). */
:is(
  .cdr.selectionHighlight,
  .cslr.selected-text
)[style*="width:10px"]{
  display: none !important;
}

/*###  Selection.Matches  ###*/
.cdr.selectionHighlight{
  border:        1px solid hsla(334, 100%, 60%, 45%) !important;
  border-radius: 4px !important;
}

/*###  Selection.Active  ###*/
/* inactive pane. */
.cslr.selected-text{
  background-color: hsla(217, 50%, 58%, 20%)           !important;
  border:           1px solid hsla(217, 60%, 70%, 50%) !important;
  border-radius:    4px                                !important;
  pointer-events:   none                               !important;
  z-index:          5;
}
/* active pane. */
.monaco-editor.focused .cslr.selected-text{
  background-color: hsla(217, 76%, 63%, 50%)            !important;
  border:           1px solid hsla(217, 100%, 60%, 78%) !important;
  border-radius:    4px                                 !important;
  opacity:          0.65                                !important;
  mix-blend-mode:   plus-lighter                        !important;
}```

I guess, we are not able to fix this issue by modifying the extension itself, because vscode extension api is very limited and does not give us much permissions.
your workaround seems good, well, maybe it's not the best thing to modify vscode styles forcefully (unoficial way) like that, but I guess it's fine temporarily. Actually I think we should inform this issue to the Cursor team as a bug and hope that they will fix it on their own.


new logo

New logo handwriting font

new logo handwriting font

Hiya @JessicaML! Thanks very much for the Logo :) Merging!


Done conditions are distinguished

Done conditions are special and not quite like assignments. This PR updates the ir::Group structure to make done conditions a field so that we don't have to do the janky "iterate over the group assignments to find the done condition" nonsense.
Also, more importantly, we no longer allow a group's done condition to have guards. The textual programs can still write something like:
group upd {
  upd[done] = c.done & d.done ? 1
}

But the compiler immediately canonicalizes it to:
cells {
  dwc = wire(1)
}
group upd {
  upd[done] = dwc.out
}
dwc.in = c.done & d.done ? 1;

There are a couple of benefits of this change. First, we don't need the group[done] hole anymore; we can simply inline the group's done condition directly into the location since the condition is guaranteed to be a port. It also makes sure that figuring out if a group is done straightforward since it does not require any computation.
Finally, w.r.t. undefined behavior, it clarifies the responsibility of the port–it must always have a non-x value. The guard does not play a role in defining the "doneness" of a group.
Fixes #1119 and fixes #864.
TODO

[ ] Fix rewriting of ports
[ ] Update tests to have done conditions
[ ] ReadWriteSet::uses on group assignments returns different results

I'm not sure if this incarnation of solving #864 is a good idea. A couple of problems:

Iterating over the group's assignments was a guaranteed way of getting all the used ports. By making done_cond a separate field, we lose this uniformity.
The above canonicalization, where guarded assignments to the group's done condition are externalized to the group, means that we'll have to assume extra-long live ranges for certain cells even though they are only used in a specific group disabling a bunch of sharing possibilities.

I think solving #864, or even attempting to solve it, requires a more thoughtful plan than this PR. I'll try to extract the useful changes from this PR to solve #1119 but undo the change that makes done_cond a separate field. We'll lose out on some niceness of the canonicalization but the trade-off doesn't seem worth it. We can revisit it in the future.


update to React16.0.0

Ref: https://github.com/facebook/react/issues/9103

The Reconciler API for the current targeted revision is available at:
https://github.com/facebook/react/blob/ca4325e3eff16b86879188eb996ebcc9a933336a/src/renderers/shared/fiber/ReactFiberReconciler.js#L48-L104
A renderer is the public interface to a React reconciler. With Fiber you
create a reconciler by calling ReactFiberReconciler with a HostConfig
object.
All types for creating a react reconciler are manually extracted into
../react-types for the current revision (16.0.0-alpha.3).

https://github.com/facebook/react/tree/master/packages/react-reconciler

https://www.npmjs.com/package/react-reconciler

https://github.com/facebook/react/blob/master/packages/react-reconciler/package.json

Closed by 72b38287886a72967e8578b886f510044cbbb1c5


Avoid sending null fields in the cloud metadata

What does this PR do?
Fixes a problem reported in the forum - following the recent addition of (#1599), the agent may fail to work with APM Server 7.9.x, which doesn't allow null values, if one of the cloud metadata fields is not populated.
This was fixed in 7.10, but since we did not release yet, we would prevent it in the agent side by not serilalizing cloud metadata null values.

Checklist

[x] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc - No need, the cloud metadata discovery feature was not released yet
[x] I have added tests that would fail without this fix
[x] I have reproduced the problem and verified the fix works as expected

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #1648 opened

Start Time: 2021-02-02T14:52:55.472+0000

Duration: 43 min 36 sec

Commit: 028eaa29fbf832b435262219125aee1c2a6d4458

Test stats :test_tube:

Test
Results

Failed
0

Passed
1698

Skipped
12

Total
1710

:green_heart: Flaky test report
Tests succeeded.

Expand to view the summary

Test stats :test_tube:

Test
Results

Failed
0

Passed
1698

Skipped
12

Total
1710

Codecov Report

Merging #1648 (9ce076c) into master (7edd598) will increase coverage by 4.66%.
The diff coverage is 91.11%.

@@             Coverage Diff              @@
##             master    #1648      +/-   ##
============================================
+ Coverage     59.09%   63.76%   +4.66%
- Complexity       92     3371    +3279
============================================
  Files           403      349      -54
  Lines         18327    16504    -1823
  Branches       2522     2257     -265
============================================
- Hits          10831    10523     -308
+ Misses         6729     5241    -1488
+ Partials        767      740      -27

Impacted Files
Coverage Δ
Complexity Δ

.../elastic/apm/agent/impl/CloudMetadataProvider.java
63.49% <77.77%> (-0.68%)
21.00 <0.00> (+21.00)
:arrow_down:

...stic/apm/agent/impl/payload/CloudProviderInfo.java
96.82% <85.71%> (+0.35%)
15.00 <0.00> (+15.00)

.../apm/agent/report/serialize/DslJsonSerializer.java
90.93% <100.00%> (+0.12%)
201.00 <6.00> (+201.00)

...ic/apm/agent/profiler/collections/LongHashSet.java
16.66% <0.00%> (ø)
12.00% <0.00%> (+12.00%)

...gent/hibernatesearch/HibernateSearchConstants.java

...t/kafka/ConsumerRecordsRecordsInstrumentation.java

.../helper/KafkaInstrumentationHeadersHelperImpl.java

...t/v5_6/ElasticsearchClientSyncInstrumentation.java

.../v5_6/ElasticsearchClientAsyncInstrumentation.java

...stic/apm/agent/kafka/BaseKafkaInstrumentation.java

... and 53 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 7edd598...9ce076c. Read the comment docs.


How to use jupysql with ODBC Driver and DSN?

I have a been trying to use the jupysql magic (%sql) with some of the databases I have set up with ODBC and Azure authentication but could not find a way to set up the configuration.
I have:

DSN called 'FUSION'  with ODBC Driver 17 for SQL Server
it has Azure Active Directory Integrated authentication and I don't have plain text password and username.

I can query the database using pyodbc like this and it is SUCCESSFUL:
with pyodbc.connect(DSN='FUSION') as conn:
    df = pd.read_sql(query, conn)

I tried to do same thing in jupyter notebook in my Windows Laptop,
%load_ext sql

%sql mssql+pyodbc://DSN=FUSION

%sql select top 2 * from mytable;

But this does not work, how to work with pyodbc and jupysql?
Error
(pyodbc.InterfaceError) ('IM002', '[IM002] [Microsoft][ODBC Driver Manager] Data source name not found and no default driver specified (0) (SQLDriverConnect)')
(Background on this error at: https://sqlalche.me/e/14/rvf5)
Connection info needed in SQLAlchemy format, example:
               postgresql://username:password@hostname/dbname
               or an existing connection: dict_keys([])

currently, jupysql relies on sqlalchemy for db connections. it seems like sqlalchemy supports mssql via pyodbc.
if you can get the create_engine from sqlalchemy up and running, you can pass the engine object to jupysql:
mssql_engine = create_engine(...)

%sql mssql_engine

%%sql
SELECT * FROM ...

let me know if this works!

@edublancas Thanks, I figured out how to work with Windows Authenticaion and Azure Authentication. Here is the snippet, if it helps other programmers:
import urllib

# Look at Driver Name from ODBC data source admin app in your windows computer
# In my case it is odbc driver 17
params = urllib.parse.quote_plus("DRIVER={ODBC Driver 17 for SQL Server};"
                                 "SERVER=XXX;"
                                 "DATABASE=XXX;"
                                 "Trusted_Connection=yes")

engine = sqlalchemy.create_engine("mssql+pyodbc:///?odbc_connect={}".format(params))

query = 'select top 2 * FROM [myschema].[mytable]'

with engine.connect() as conn:
    dfx = pd.read_sql(query,conn)

dfx

awesome! thanks for sharing the solution, closing this now!


Please add support for clangd's 'remark' severity on problems panel

Type: Feature Request
Recently, clangd developers have decided to change the severity of some diagnostics to 'remark'. This has the unfortunate side effect that it makes these diagnostics all but invisible in the vscode GUI -- in my case, with my settings, they are literally invisible unless I click on them. But even worse, it makes them not searchable or discoverable in any way other than to go looking for them manually in each file, meaning that the command line is currently a better user experience.
While the merits of their decision are debatable, this issue is not for debating this; go to clangd/vscode-clangd#579 or llvm/llvm-project#75706 for that.
Instead, this issue is about making these diagnostics more visible in the vscode GUI, preferrably in the problems panel with a way to filter through them. I am not sure exactly what kind of severity the clangd 'remark' severity gets mapped to in VSCode; I believe it would be 'hint'?
VS Code version: Code 1.85.2 (8b3775030ed1a69b13e4f4c628c612102e30a681, 2024-01-18T06:40:10.514Z)
OS version: Windows_NT x64 10.0.22631
Modes:

I do not think VS Code support severity type remark, can you please confirm this?
https://github.com/microsoft/vscode/blob/e91bff758bade17f7432ac3a787a87f54f14083c/src/vscode-dts/vscode.d.ts#L6641-L6663

I think that clangd extension remaps it to hint. I do know that these remarks appear in VS Code usually in strike through, with mouse over tips, and with quick fixes.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Possible docs typo

In the Hugo Bookshop guide, it instructs to put the mount configuration at components/config.toml. I thought this meant /component-library/components/config.toml. After looking at example sites, I've found it should be /component-library/config.toml.

I think this is addressed well in the new documentation
(also, the init command will now create this file for you 😄)


html script code runs on OSX versions with no file extension.

If there's no extension on the file and html script code is inserted, the script gets executed without any filters.
I'm currently using latest edition of OSX Brackets version.
the way to trigger the problem is as follows:

create a filename with no extension (example, 'testtest')
insert the html script tag with appropriate javascript codes. (example, '')
open the file in brackets.

this bug might lead to security problems (precisely XSS/CSRF problems).

This happened on the brackets-minimap package.
Created a new issue on the brackets-minimap package issue tab.
https://github.com/zorgzerg/brackets-minimap/issues/33


Pass vectors by reference

All the vectors are copied on the stack in each function call, use & to pass them by reference.

Hi @drake7707 ,
Thank you for suggestion. Now, I pass vectors by reference. You can take a look at last commit.
Martin


[Mouse Without Borders] - refactoring "Common" classes

Description of the new feature / enhancement
As a precursor to https://github.com/microsoft/PowerToys/issues/34126 (Integrate Mouse Jump with Mouse Without Borders) I've spoken with @crutkas and @ethanfangg about doing some mechanical refactoring to break the large "Common" class into multiple smaller classes as general maintenance / housekeeping work.
Each "Common.*.cs" partial file will be turned into a separate static class so as to reduce the scope of the private members inside them and make it easier / safer to perform other changes (e.g. integration with Mouse Jump) at a later date.
Changes to each "Common.*.cs" file will be submitted as a separate PR.
Scenario when this would be used?
No change in functionality - this is a refactoring / maintenance change designed to make some future work easier / safer to implement.
Supporting information
No response

@mikeclayton  isn't this https://github.com/microsoft/PowerToys/pull/35156 or is there more

There's more PRs to come - there's about a dozen "Common.*.cs" classes and we talked about moving cautiously and doing one file at a time so as not to break anything.
I've been trying to get the Mouse Jump style work over the line in #35521 - once that's done I can spend a bit more time on Mouse Without Borders (I've got a second refactor lined up but I've not had time to go through the regression tests yet)...

What's now? is #35858 related to this?

This ticket is unrelated to your feature request in #35858.


SPIRVProducerPass refactor - phase2.1

Removed pointer management, SPIRVInstructionLists are static
Changed DeferredInst mechanism, addSPIRVPlaceholder then replace in 2nd pass (HandleDeferredInstruction)

Enhancement for clspv issue #545

This doesn't look like a formatting error:

tar xf clang-llvm.tar.xz
tar: This does not look like a tar archive
xz: (stdin): File format not recognized
tar: Child returned status 1
tar: Error is not recoverable: exiting now

Yes, LLVM server has been returning 503s lately.

Can you rebase on master? I've update some bot scripts.

@alan-baker should be up-to-date

Hmm, looks like my local clang-formatter is not right anymore..

Now it's pointing at the right clang-formatter, I think.


The Denpa Men: They Came By Wave (USA)

Game Name (and version if necessary)
The Denpa Men: They Came By Wave (USA)
Hardware Specs (If necessary)

CPU: Intel i5-8400
GPU: NVIDIA RTX 3070

Operating System
Windows 10 22H2
Game State
Menu text cannot load properly and falls back to "Dummy". Insane flickering on top screen while bottom screen is partially cut off. Crashes after starting a new game and attempting to access the camera EDIT: Since 2cf4563, the emulator can get into gameplay, but due to requiring hardware features to detect Wi-Fi Signals, you cannot progress very far w/o save editing.
Screenshots

Additional information
Provided below is a log file of console output while running the game. It seems the game attempts to access the cameras with a strange port value, I'll try to see what exactly the value is if I can compile the application on my machine:
pandaLogEdited.txt

Looks like the problem is in CAMService::startCapture, when it indexes with port.getSingleIndex() in a for loop with iterator values based on what is returned in port.getPortIndices(). The function does not account for when the value could be 3, indicating its trying to use both cameras.
Replacing the call to getSingleIndex() with the iterator value gets into the main game, but since this game relies on hardware features to detect wifi signals, you can't get very far without save editing to collect more Denpa Men.

At some point between 2cf4563 and 61e2e71, compatibility regressed for this game.
Booting the game now seems to throw Thrown fatal error @ 00208EA0 (pid = 1, type = 0)

Can't reproduce, can you give some more info/a save?
Think it might be this PR that breaks it https://github.com/wheremyfoodat/Panda3DS/pull/524

Looks like having a save was causing the crash. I cleaned my Panda3DS folder and the game is operating as it was before. Once a save is made sometime after the intro, I need to restart the entire emulator to get the crash to occur again.

I see. So I guess it's not a regression but a bug you hadn't been hit with before?

Possibly, I didn't test too extensively the last time.


Feel-it dataset

Hi,
Is the Feel-it dataset somewhere available?
Thanks

Yes, send me an email<EMAIL_ADDRESS>


Add CMake Support

Added basic CMake support

interface (header only) library for atomic_queue
option to build samples
option to build tests + CTest support
gcc/clang linux cmake build

Because of the way dependencies are set up in benchmarking I did not implement benchmark target. Those could be implemented with either cmake's install + find_package or with FetchContent if respective libraries use CMake and are small enough.
Boost, which is used by the test target, is found using find_opackage. This causes some issues on Windows - the easiest way to get it to work on windows is by using vcpkg. To mitigate it, it would be the best to replace boost-test with google-test, which can be easily downloaded and installed per-repo with fetch_content.

I will create Windows fix (conditional pthreads) once this is accepted, there is currently no way to build this on Windows (unless we use MinGW or something to build from make files). pthreads are only included by test target or manually if someone wants to use spinlock.

What problem does your CMake support tries to solve?
Boost is a number one dependency in all C++ projects I worked on since year 2004. Asking to ditch Boost and change code is an absurd request, in my opinion.

What problem does your CMake support tries to solve?

The goal is to build natively on Windows and support CMake build systems without requiring 3rd party package managers or manual CMake configurations, even if this is just a header-only library.

You can run Ubuntu natively in Windows and build Linux projects without having to write or change a line of code.

Yes, that doesn't however solve the problem of targeting Windows natively.

Boost is a number one dependency in all C++ projects I worked on since year 2004. Asking to ditch Boost and change code is an absurd request, in my opinion.

That's fair. I proposed it as a way of getting around having to install 3rd party dependencies manually or using external package managers.

CMake was created with a goal of being able to build C++ projects on different platforms. CMake support would need to be able to build tests and benchmarks on Windows to fulfil its purpose. Conan package manager solves the problem of acquiring libraries and making them available in CMake.

I can work on bringing the benchmark target to CMake and making it work on Windows. The main goal for me was to port the library and test targets to CMake.

What problem does your CMake support tries to solve?

The goal is to build natively on Windows and support CMake build systems without requiring 3rd party package managers or manual CMake configurations, even if this is just a header-only library.

Ah, I see now.

You can run Ubuntu natively in Windows and build Linux projects without having to write or change a line of code.

Yes, that doesn't however solve the problem of targeting Windows natively.

Fair enough.

Boost is a number one dependency in all C++ projects I worked on since year 2004. Asking to ditch Boost and change code is an absurd request, in my opinion.

That's fair. I proposed it as a way of getting around having to install 3rd party dependencies manually or using external package managers.

Fine, amputate boost.

CMake was created with a goal of being able to build C++ projects on different platforms. CMake support would need to be able to build tests and benchmarks on Windows to fulfil its purpose. Conan package manager solves the problem of acquiring libraries and making them available in CMake.

I can work on bringing the benchmark target to CMake and making it work on Windows. The main goal for me was to port the library and test targets to CMake.

Okay, benchmarks can be a tall order.


Fixed Issue #534 - Made the main page more visually appealing by adding cards and colours

IMPORTANT: When creating your Pull Request, please use the following format for the title: Category - Resource Name & Section
Fixed Issue #534
PR Title samples:

Web development - CSS Portal & Tools
Web development - Leetcode & Challenges

Closes #10
Description:

Made the main page more visually appealing by adding cards and colours.

Resource URL:

Insert the URL of the resource you're proposing

Checklist:
Before submitting your pull request, ensure that you have completed the following tasks:

[x] I have carefully reviewed and adhered to the contributing guidelines before creating this pull request.
[x] I followed the prescribed PR title template. (Check this if you're adding a resource)

fill the bracket with x. Something like this [x]

Give me the screenshot


DEVOPS-501 Allow building with branch-specific dependencies

Pull request test result: success :star:

Pull request test result: success :star:


please support range syntax

array define : [1,2,3,4,5]
range define: [1..5]
This syntax is useful in an inline environment to effectively shorten script lengths.
Some language support it like kotlin, groovy...etc

This is actually not a bad idea. @d5 any thoughts on this? I wouldn't mind making some time for this.

Sure. Can we discuss more details on the syntax first?

Just looking up how this is done in other languages now but from a cursory overview I’m leaning towards the python style range builtin function rather than changing the array syntax.

Haha. Yeah I like that!

So the new builtin function will return an array of ints?

Yes, an array of ints from 0 to a n-1 If one argument is provided, start to end-1 with two args, or start to end-1 with step Increment with three args

I think if it's just a built-in, function, and we already have enough APIs to implement it ourselves, the built-in bracket syntax support would be much easier to read for line expression scenarios

I’m not sure I agree that brackets add something to expressiveness that a builtin range function doesn’t.
It’s also IMO an unnecessary addition to the syntax which I’d rather leave as is unless there’s a really good reason to change it.
Additionally, the bracket syntax is limited. It doesn’t easily allow you to add a step size.

I use Tengo in a database sharding proxy project, it allows me to simplify some definitions, such as database node definitions: DS_1, DS_2, DS_3. If I have this syntax I can say DS_{[1..3]}, this makes the configuration file simple and easy to read. If it's a function: DS_{rang(1,3)}, I don't think it's as clear so.
In my project, the more complex
inline expression:
ds_{[1..9]}.table_{[1..5]}
This expression is in the configuration file, so avoiding the function is easier to read.

BTW: Because missing this syntax, I implemented a "range" function on my project, and this issue comes from a feeling that the configuration file is ugly.

I suppose we can achieve something like that with [start:step:end] and [start:end] syntax but I’m not convinced of the need for it. While your example expression does not look pretty it is readable. I’m reluctant to change the language syntax to make this pretty.
@d5 thoughts?

Yeah I agree with @geseq. It's a nice-to-have feature and not worth all the extra efforts.

In that case @endink would you be willing to create a PR with your implementation of the range function? If not I’ll work on implementing it.

@geseq Alright, these days are the China New Years. If need, I can provide this PR after this holiday .

@endink any luck with the PR? If you prefer I can work on this as well.

@geseq The Chinese New Year ends on February 27th, wait two days for me to do it or you do it. I'm fine with it anyway.You decide

Oh no worries! I didn’t know that. Take your time.

@geseq Pr has been commited, and the specific usage can be seen in unit tests


Check secrets are not empty

We have to check the following secrets are not empty, this will avoid hpc-rocket unexpected errors.
REMOTE_GROUP
REMOTE_HOST
REMOTE_USER
SSH_PRIVATE_KEY

#10 solves this issue.


Indicate in documentation when something has to imported via require

Discussion
Currently some parts of the standard library must be imported via require x before usage. Whatever the reason for this, it leads to some usage problems:

It seems arbitrary which ones must be imported (there may be an internal reason, but it's not clear to a user)
It is not clear what is to imported (JSON is json but BigInt is just big?)

And it is never specified anywhere in the api documentation or crystal book, what is to be imported and why. In some cases it is shown in the example case near the top, in others such as BigInt not even that. In any case, simply having it in an example is very inexplicit.
In light of this I suggest:

Every class/module/whatever that needs an import should state so near the top
A section should be added to the crystal book and/or api homepage explaining how to use the standard library and that require may be needed sometimes

I am not sure about the style. The ruby docs have a heading dedicated to it (example) but perhaps a NOTE would be better.

So in general, the primary code examples should usually include a require statement when necessary. This is ensured by tests that we run on the code examples, so AFAIK it's pretty complete (when there are examples). BigInt has no examples in the type documentation. But there are examples for several methods, including the constructor which includes the require.
Note that subsequent examples usually don't repeat the require for brevity. I wouldn't object adding them everywhere, though to make the examples really self-contained.
https://github.com/crystal-lang/crystal/blob/master/src/prelude.cr is the standard prelude. It contains a list of require that are included by default in the core library. You never need to explicitly require anything that's also required from the prelude. Everything else needs to be required.
Regarding style, I don't have a clear suggestion. Maybe this could eventually be expressed as some kind of directive for the doc generator. But we don't have that yet, so we need to start with a free format. Probably just prose. Maybe we could use some kind of admonition style (e.g. NOTE: To use this class, you need to explicitly require it via `require "foo"` ).
Keeping the same format everywhere allows to easily transform it into something else later on.


SRS and SRSSpec/SRSValue

We use a wrapper around an SRS.
These are the current implementations of SRSValue and SRSSpec:
pub enum SRSValue<'a, G: CommitmentCurve> {
    Value(SRS<G>),
    Ref(&'a SRS<G>),
}

impl<'a, G: CommitmentCurve> SRSValue<'a, G> {
    pub fn get_ref(&self) -> &SRS<G> {
        match self {
            SRSValue::Value(x) => &x,
            SRSValue::Ref(x) => x,
        }
    }
}

pub enum SRSSpec<'a, G: CommitmentCurve> {
    Use(&'a SRS<G>),
    Generate(usize),
}

impl<'a, G: CommitmentCurve> SRSValue<'a, G>
where
    G::BaseField: PrimeField,
    G::ScalarField: CommitmentField,
{
    pub fn generate(size: usize) -> SRS<G> {
        SRS::<G>::create(size)
    }

    pub fn create<'b>(spec: SRSSpec<'a, G>) -> SRSValue<'a, G> {
        match spec {
            SRSSpec::Use(x) => SRSValue::Ref(x),
            SRSSpec::Generate(size) => SRSValue::Value(Self::generate(size)),
        }
    }
}

Note that this type is re-declared in two places: dlog/commitment/src/srs.rs and dlog/plonk-15-wires/src/index.rs
The idea is that there's a single SRS, and it is shared amongst many circuits (ConstraintSystem). This API forces the SRS to either be passed by reference, or created from scratch.
There's a few problem with that. First, as it is only a reference, as soon as the SRS goes out of scope then these references cannot exist. This would not be a problem with a shared reference pointer.
The second problem is how we use it in our OCaml bindings:
pub struct CamlPastaFpPlonkIndex<'a>(pub Box<DlogIndex<'a, GAffine>>, pub Rc<SRS<GAffine>>);

// ...
    let urs_copy = Rc::clone(&*urs);
    let urs_copy_outer = Rc::clone(&*urs);
    let srs = {
        // We know that the underlying value is still alive, because we never convert any of our
        // Rc<_>s into weak pointers.
        unsafe { &*Rc::into_raw(urs_copy) }
    };
    let t = index_serialization::read_plonk_index(
        oracle::pasta::fp::params(),
        oracle::pasta::fq::params(),
        srs,
        &mut r,
    )?;
    Ok(CamlPastaFpPlonkIndex(Box::new(t), urs_copy_outer))

Not only this is hard to follow, but it should either be refactored to use caml_pointer (a custom type around an Rc), or abstracted behind a cleaner interface (so that we can ignore it at this level, and review it in a self-contained manner).
The third problem is mentioned here: https://github.com/o1-labs/proof-systems/issues/164 and is about the fact that the SRS is too tightly coupled with a ConstraintSystem, which makes deserialization (or serialization) awkward, and handling of types awkward as well.

I'm trying to understand what is going on here.
First, in OCaml-land we handle an SRS through the custom type CamlFqSRS:
CamlPointer<Rc<SRS<GAffine>>>

This is weird already, because a CamlPointer is already an Rc:
struct CamlPointer<T>(pub Rc<T>);)

So we have an Rc<Rc<...>> here. I'm assuming this is not normal.
An SRS is useless without a constraint system.
When we want to deserialize an index (which contains a constraint system), we pass that SRS CamlPointer to some function (is the Rc pointer cloned when we do that?) that returns a CamlPastaFqPlonkIndex:
struct CamlPastaFqPlonkIndex<'a>(pub Box<DlogIndex<'a, GAffine>>, pub Rc<SRS<GAffine>>);

Since the index structure (first field) doesn't have an RC pointer to the SRS, and just a reference, we need to bundle it with an RC pointer of the SRS (second field).

So perhaps this proposal would work:

delete the SRSSpec and SRSValue types
use Rc<SRS> in the constraint system and index types
CamlPastaFqPlonkIndex can now represent the index directly, no need to bundle the SRS

this is done


Added Content for ML Community.

Solves Issue #125

Description:  Removed Lorem Ipsum for Community of Machine Learning.
Checklist

[x] I've followed the Contributing guidelines provided in the repository.
[x] I've made the changes which were demanded in the linked issue.
[x] I've tested my code on a Chromium based browser.
[x] I've tested my code on Mozilla Firefox.
[x] My code gave a clean console on debugging. (no warnings/errors)

Testimonial

@Vishal19111999  I have made those changes, please Review


podspec source_files includes everything

Hi there,
Is that line of the podspec correct?
s.source_files  = 'JTCalendar/**/*'
It includes everything including Info.plist.
Should it be
s.source_files  = 'JTCalendar/**/*.{h,m}' ?

Hi,
both version are / were correct, yours is cleaner, in a recent merge with a PR I didn't see the info.plist file added.
Have you a problem with the pod?

Sometimes I'm getting this issue, cannot replicate on demand, but always with JTCalendar:
dyld: Library not loaded: @rpath/JTCalendar.framework/JTCalendar
So I started to search and found that discussion on SO:
https://stackoverflow.com/questions/39160654/signing-errors-with-use-frameworks-and-unique-provisioning-profiles/39182887#39182887
I'm not sure they are related, but probably having *.{h,m} will be more straightforward.
I can see Info.plist in the pod folder.

I release a new version 2.2.1, but I already seen your bug with others libraries without finding a good reason. It seems to happen on projet where I manage multiple targets with a target depending of another target (I have an app and a core lib). The only solution I have found is to clear the projet and recompile.

Thank you! Yes, that's exactly the scenario you're describing, I cannot find a good solution too.


*: update protobuf

Fix https://github.com/coreos/etcd/issues/2138

@bdarnell FYI we are updating gogoproto for raft too.

OK. FYI we've been using the latest github version of gogoproto in the rest of cockroachdb, and there haven't been any issues caused by the fact that we use one version and raft uses another.

lgtm


Can you give me consult ?

Hello evreybody. Recently I start study kubernetes. And I try to understand PV and PVC with nfs.
And I really cant understand:

When I using nfs. Where need to deploy nfs server ? It is some independent or need to deploy in master node or minion node ?
In my pvc - yaml
======================================================
apiVersion: v1
kind: PersistentVolume
metadata:
name: pvc1
spec:
accessModes:

ReadWriteMany
capacity:
storage: 10Gi
nfs:
server: - I think need to past nfs server ????
path:  --- which path need to past ???
===============================================

Each PV contains a spec and status, which is the specification and status of the volume.
apiVersion: v1
kind: PersistentVolume
metadata:
name: pv0003
spec:
capacity:
storage: 5Gi
volumeMode: Filesystem
accessModes:
- ReadWriteOnce
persistentVolumeReclaimPolicy: Recycle
storageClassName: slow
mountOptions:
- hard
- nfsvers=4.1
nfs:
path: /tmp
server: <IP_ADDRESS>
path is where you want the nfs to be on. It can be on host, GCP or AWS.
I’m not 100% about the server, hope this helps.


[WIP] Enable Python 3.7 in Linux

Taking over from @BrettCannon
Test-fixes to follow...

Fixes (part of) #962
This pull request:

[x] Has a title summarizes what is changing
[ ] Has unit tests & code coverage is not adversely affected (within reason)
[ ] Works on the latest Python 3 release
[ ] Works on ~Windows 10, macOS, and~ Linux

Also, taking over from PR #1507

Thanks @DonJayamanne, a few more tests to go though. (TBH I'm super slow at this, not due to the problems, but due to learning how the tests are set up... hopefully I'll get faster soon).
I'll ping you when I'm all done and the 3.7 build is running smoothly.

VSTS!!

@DonJayamanne & @brettcannon, just waiting for a little bit of feedback and I can submit this sucker!

@brettcannon please have another look.

...requesting changes based on coming up with an "official" way to skip tests.

You cannot use this in an anonymous method, @DonJayamanne set me straight on that! Now I'm skipping the test the correct way (and got rid of the extra cruft commented out beneath the test).
Also, I updated the Black test to only run on Python v3.6 or higher.


generate blocks rpc commands

I was not able to find the rpc command generate_blocks
see https://github.com/monero-project/monero/blob/9a124f681119855949f6406ecd69c2ad91da9770/utils/python-rpc/framework/daemon.py#L213-L226
I think I can hack it using the raw_jsonrpc_request method but would you accept a PR that adds it ?

I dont think it is documented, it apparently can only be used with the regtest flag, hence my other issue opened.
for the backgroupnd I'm trying to setup a fixture that spawn a private testnet that mines faster, and this discussion on irc led me to both regtest and generate_blocks
<euri10> Greetings, I'm currently using monero-python and I have questions about the way people test their code when it deals with monero.
<euri10> So far I have setup a private test network in a container, that runs 2 daemons, and creates 2 wallets with deterministic adresses, and uses a low difficulty.
<euri10> My test suite then setup a pytest fixture that runs the container, starts mining on one of the 2 addresses, waits for the wallet to have enough xmr to spend.
<euri10> The "issue" I'm facing is the following: mining takes time. Afaik the first mined xmr becomes available only at height 60, is it something I can tweak to increase speed ?
<euri10> Another question is : can I mine up to a given height and stop mining once reached ? I didn't see an rpc command parameter for that (up to x height)
<euri10> But after I made all those questions, I'm reaching that point where I'm just wondering if integrated testing is the way to go and if I should not mock the various monero part I'm interested in interacting with.
<euri10> thanks for the read !
<sech1> euri10 use --fixed-difficulty parameter for monerod to mine faster
<euri10> yep already set at 1
<sech1> then it should be very fast to mine 60 blocks
<moneromooo> There's a generate_blocks RPC, you tell it how many blocks to mine. See tests/functional_tests.
<moneromooo> Also, set MONERO_FASTER_POW=1 if you don't care about verifying the PoW from another program. It'll use keccak IIRC.
<moneromooo> Ah nvm that never got merged. You can apply it from my "crash" branch if you don't mind running a slightly patched daemon.
<euri10> ok ok :) good infos here, will definitely check that
<moneromooo> https://git.townforge.net/townforge/townforge/commit/d8e4f4078598221a2f7320564a507c9d5bbce6a3 is the one with keccak, it doesn't seem to be in my crash branch. Should be easy to merge.
<moneromooo> Also, generate_blocks only works in regtest mode for some reason, but I doubt that'll be a problem here.
<euri10> what is regtest mode ?
<moneromooo> A global state set by --regtest IIRC, which enables that and a "delete chain before we start".
<euri10> ok ok, with all that hopefully I can speed up my test suite, many thanks
<moneromooo> Which made me add a --keep-fake-chain because useful, especially to investigate when stuff went wrong. So not sure why this mode exists in the first place.
<euri10> @moneromooo as soon as I add the regtest flag to the daemon command, it seems the daemon is not accepting rpc connections anymore, is that possible ? I raised an issue here, maybe you have better insights: https://github.com/monero-ecosystem/monero-python/issues/117


Tour tutorial implementation

Bubbles to implement:

Search by neuron
search by promoter
search by dev stage
card component
video player swap between histone marker video and promoter video

pushed in feature/53


Newsletter

It would be nice to have a monthly newletter for IndieHosters.
And this should include our personal newsletter also.
Do you plan to write a newsletter for your users?
What do you think?

I don't really have much interesting news for my users. :) We could write about our efforts to agree on a packaging format, and how hard this turned out to be in practice. But it's quite a technical topic, and we want to offer a service to non-technical users, right? The only other news I can think of is that I migrated one of my servers from ssd to sata, but I don't know if that's very interesting either.
Maybe we have more news to blob about for other hosters than for our own users?

I think it could be interesting to at least share uptime on a monthly basis,and also canary warrant.
And them, you can also say, ok now I offer cozy, if you want to try, please contact me.
And then, on IH newsletter, we can speak about plan to have common apps.

I think newsletter is important at least to report uptime. When you have downtime then, you are reminded that you'll have to report to all your users and other hosters. And this is kind of a nice pressure.
But well, it is up to you.
I'll let you close the issue.


Azure B2C

HI,
I followed this article
https://docs.microsoft.com/en-us/azure/active-directory-b2c/azure-monitor
However, I get a message while configuring the diagnostics settings
"To setup Diagnostic settings to use Azure Monitor for your Azure AD B2C directory, you need to set up delegated resource management. You can start here to learn how to set this up."
The resource delegation is configured as mentioned in the article. But I still get the aforementioned error
Please assist
Thanks
PG

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 24e34c89-9a70-b5b2-3b38-bc81660f83c1
Version Independent ID: 77fa8d3c-56dd-294f-13ae-ac6ba44018a7
Content: Monitor Azure AD B2C with Azure Monitor - Azure AD B2C
Content Source: articles/active-directory-b2c/azure-monitor.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@praveensoni5  I followed the same article, I was also getting the same thing you are getting.
1)I waited for 15 minutes, signed in again with the B2C tenant user who was the member of a security group I created.
2)After logging in, I selected the Directory as my tenant where my subscription is attached and not with the B2c tenant.
3)Once done, I now have option to Add diagnostic setting:

@praveensoni5  I followed the same article, I was also getting the same thing you are getting.
1)I waited for 15 minutes, signed in again with the B2C tenant user who was the member of a security group I created.
2)After logging in, I selected the Directory as my tenant where my subscription is attached and not with the B2c tenant.
3)Once done, I now have option to Add diagnostic setting:

@praveensoni5 @praveensoni5  I followed the same article, I was also getting the same thing you are getting.
1)I waited for 15 minutes, signed in again with the B2C tenant user who was the member of a security group I created.
2)After logging in, I selected the Directory as my tenant where my subscription is attached and not with the B2c tenant.
3)Once done, I now have option to Add diagnostic setting:

@praveensoni5 @praveensoni5  I followed the same article, I was also getting the same thing you are getting.
1)I waited for 15 minutes, signed in again with the B2C tenant user who was the member of a security group I created.
2)After logging in, I selected the Directory as my tenant where my subscription is attached and not with the B2c tenant.
3)Once done, I now have option to Add diagnostic setting:

@praveensoni5 Closing this for now, please let us know if you need any further help.

@praveensoni5 Closing this for now, please let us know if you need any further help.

I still get the error despite waiting for >24 hrs. What could be possibly wrong here?

I still get the error despite waiting for >24 hrs. What could be possibly wrong here?

I still get the error despite waiting for >24 hrs. What could be possibly wrong here?

The problem I had was that the Subscription wasn't selected when I was in my B2C tenant. Make sure you have All Subscriptions selected in the following dropdown:


added new map to storage page

Closes #862

This change is

Pushed the fixes as recommended . Please check

Just pushed another fix, all should be good now

@quentin-abei build fails in CI

@enitrat  what might be the issue ? I can see your commit have an issue , did you see it ?

I don't know, try to build and see

@enitrat , I’am going to open another PR with all the changes we done here . Should avoid the bug

@quentin-abei no, please check what's wrong with these files

but you just need to git pull

See

git pull origin add-new-map
for the rest check with claude or chatgpt how to properly config your git repo

Added new imports related to the new MAP types as seen here: https://community.starknet.io/t/cairo-v2-7-0-is-coming/114362

Pushed a new fix, the very last one. Problematic files are compiling well locally. So the merging should work great ! Please review it

The command that is failing « Cairo-listings verify » is also failing in my local machine ( almost every cairo file fails ) . So I have lot of verification fails . I don’t know if it’s related to my scarb version maybe ?

It says that listing_01_reference_contract is not formatted properly. If you are using scarb with ASDF, everything should be fine, the scarb versions required are the ones in the file .tool-versions

At which line does the fmting error appear ? I can’t find a way to view the errors .

yous can just go to that directory and run scarb fmt


Update test retrigger

This is just #228 which had problems with travis not triggering.

@meltsufin @patflynn @elharo


Order not being removed from current-orders.json on sale

When an order is sold the action goes through correctly but the order is not removed from current-orders.json. This results in the bot trying to repeatedly sell the item but not being able to as the Binance wallet no longer has the item to sell.

This is a weird bug only happens on Testnet, please refer to this question.
In short, the sell orders on Testnet usually doesn't really fulfilled because Testnet doesn't truly reflect a real market. Therefore your order status often marked as "EXPIRED".
Currently, I'm deleting the order in the current-orders.json only if the status is "FILLED"
    if (sellData.status === 'FILLED') {
      if (coinRecentPrice >= TP_Threshold) {
        console.log(`${symbol} price has hit TP threshold`);
      } else if (coinRecentPrice <= SL_Threshold) {
        console.log(`${symbol} price has hit SL threshold`);
      }
      await saveSuccessOrder(order, coinRecentPrice);
      await removeSymbolFromPortfolio(symbol);
    }

The solution I propose is if we're using Testnet, even if the order marked as "EXPIRED" or whatever, we pretend that's a success order and proceed the next actions.
The ternary operator looks something like this
    if (TEST_MODE ? (sellData.status) : sellData.status === 'FILLED') {
    // Code
    }

If anyone has a better alternative, feel free to propose.

I was running against mainnet when this occurred. Not sure what debugging info would be useful here but I'll watch for it again and try to capture more information.


DSA/RSA_print(): Fix potential memory leak

Fixes #10777

Good for 1.1.1 and master.

Should we be doing fixes in master for deprecated functions?

Yes - it is simply deprecated - not actually removed.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master and 1.1.1 branches. Thank you for the review.


Possibility to subclass BaseTap

Is your feature request related to a problem? Please describe.
I figured out that it is not possible to subclass BaseTap and create custom Tap due to internal access modifier of doHandleTapBlock function.
Describe the solution you'd like
I will suggest making it open so we can create custom Tap.
Describe alternatives you've considered
As an alternative solution I think will be to provide BufferTap which will send raw buffer (AVAudioPCMBuffer) to the handler without any modifications.
Assign yourself to this task
Please confirm which option is preferable for you and I can create a PR for that.

Um, it is open?
https://github.com/AudioKit/AudioKit/blob/bd206fa2109df78da9bfe0059cbaa7544d9e1e74/Sources/AudioKit/Analysis/BaseTap.swift#L101

Reopen or comment on this, but I'll close it for now.

@aure Thanks.
It seems it was updated recently, and I haven't updated it in my project.


maintainers: fix IDs and update github handles

Description of changes
These were found with a nix eval ... | jq ... | xe -s 'gh api ... && echo' oneliner which bash failed to preserve in my history. I then got a merge conflict with #259990, where SamirTalwar erroneously was renamed. There was a typo in his githubId.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

I think the principle of least merge conflict is a good stance. Maintainers are however welcome to themselves treewide change their maintainers.<attribute>.
I decided not to drop any maintainer in this PR, even though there were candidates. I think we should discuss whether it is acceptable to drop removed accounts from maintainer-list.nix or not at all. Git is not the only info they provide.

Github is the only place to contribute, and maintainership requires keeping aware of and interacting PRs and issues related to their package. (Which requires a GH account.)
So if their account is deleted, it does not make sense for them to stay as package maintainers, it is just bad data quality.

I updated both the github handle and the name of Ocean, because the old name struck me as a dead name. (Only the name change is visible post-merge however.) For picnoir whether this is the case or not was not as apparent to me, it could be a refresh of the online identity alone and not the IRL one. I decided not to dig.
Touching maintainer-list.nix in general can quickly get finicky, and clear guidelines should be discussed and put in place.


[TextField] Fix typo in FormControlLabel declaration file

Signed-off-by: Nathan Hughes<EMAIL_ADDRESS>

[x] I have followed (at least) the PR section of the contributing guide.

@hghmn It's a great first pull request on MUI 👌🏻. Thank you for working on it!


Add eclipse project files

as per title

@m133225 can do this one?

@m133225 any updated on this?


2.0: Fully use the HeaderBag and some cleanup

Hi, welcome to the next round. :smile:
As promised, I worked on replacing the $headers array with the HeaderBag instance. This was mostly lacking the extra properties that you saved with each header - this is now all stored in the Header instances within the HeaderBag.
I also did some smaller refactorings (shorten methods, flatten code). I hope I kept the style changes to a minimum, although I would really advise going for a well-specified standard like PSR-2.

Hmm, something's not quite right yet, for example when looking at the importHeaders() function. I suppose there are some tests missing...

Ahaha, nice changes once again! :D

As promised, I worked on replacing the $headers array with the HeaderBag instance. This was mostly lacking the extra properties that you saved with each header - this is now all stored in the Header instances within the HeaderBag.

👍

I also did some smaller refactorings (shorten methods, flatten code). I hope I kept the style changes to a minimum, although I would really advise going for a well-specified standard like PSR-2.

Some of the "strange brace placements" can probably be explained by my trying to keep to a <= 80 char line width 😜

Hmm, something's not quite right yet, for example when looking at the importHeaders() function. I suppose there are some tests missing...

Yup, writing more tests is definitely something that I need to get done! The tests are essentially just focused on making sure safeMode hasn't broken for the moment. Will get on that once I can grab a couple hours to sit and think of different ways things can break!
I think the line I commented on is probably the one causing the issues you're seeing?

You were right with your comment - I removed the line and that fixed my new test case. :)

And some more generic tests for code that I am about to refactor.

👍
Thanks for adding those tests in too (definitely a little overdue on my part).

👍

@aidantwoods - not totally sure that StyleCI is enabled for branch 2.0?

So this most recent commit will fail...
Have commented on the line causing the error.
Basically this went unnoticed because all the headers involved in the current test are either added in by import from the HttpAdapter, or are grabbed and decomposed as policies before being added to $this->headers. These then later appear when policies are compiled, so no tests noticed that all existing headers were being overridden by this line 😨
Guess this just adds to the case for me needing to add some proper tests!

@lucasmichot
Yeah, it's not enabled for the time being.
To be honest, it made me a little uncomfortable. Kind of a combination of StyleCI needing write access to all my repos and that there appears to be code that deletes branches without asking.
Not saying that I have any reason to think they have bad intent – just seems like something that could easily cause a lot of damage if they made a mistake in the wrong place on a future update.
I guess if/when they no longer require write access I'd be glad to add it as a check again. Or if there are any alternatives?

Good job in catching that. :+1:

As for StyleCI alternatives, there is also NitpickCI... maybe that isn't that greedy in terms of permissions?

@franzliedke
Looks like NitpickCI requests everything 😭

In theory styling shouldn't need more than read permission and being manually added as a service, right? (unless I'm missing something here)

Also, with those latest commits we should now fully utilise the HeaderBag class by supporting multiple headers with the same name on public functions and imports (and fixed a small bug where the GlobalHttpAdapter wasn't able to set multiple headers of the same name).
Think that wraps up everything that makes sense to add in this PR?

Sure, looks good. I am (as we speak) working on one last major refactoring - the separation between configuring and application that I was talking about when 2.0 came up as a topic. But that's way bigger than what can be done in this PR.

Awesome, looking forward to the next one! 😄


[System Tests] Automatically starting npm testing on dev.sh

This PR updates the dev.sh configuration script in order to automatically start a testing version of webpack and automatically configure proxy.dev to use the CLUSTER_URL  specified.
Effectively, using the ./system-tests/driver-config/dev.sh driver configuration script you can serve the current branch's version without the need of building a dist first.
For example, you should now use the following to run the tests:

docker run -it --rm --ipc=host \
  -v `pwd`:/dcos-ui \
  -e CLUSTER_URL=$OPEN_CLUSTER_URL \
  mesosphere/dcos-ui dcos-system-test-driver \
  ./system-tests/driver-config/dev.sh

L👀ks good!


Add logfile support

It would be nice to have Kraken be able to dump logs to a file for easier parsing/debugging.

It might make sense to also migrate to a standard logging framework like logrus


fix: recognize ipv6 error messages

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 5

Complexity increasing per file
==============================
- pastepwn/scraping/pastebin/tests/pastebinscraper_test.py  3

See the complete overview on Codacy

Pull Request Test Coverage Report for Build 394

1 of 3 (33.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.905%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pastepwn/scraping/pastebin/pastebinscraper.py
0
2
0.0%

Totals

Change from base Build 384:
0.0%

Covered Lines:
3148

Relevant Lines:
3891

💛  - Coveralls

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 5

Complexity increasing per file
==============================
- pastepwn/scraping/pastebin/tests/pastebinscraper_test.py  3

See the complete overview on Codacy


fix: CVE-2023-45288 ensure golang/x/net is 0.23+

Part of ACM-12257
Also build container with ubi9 images
The following files do not seem to be updated when stolostron fork is upgraded to v0.27.0 level via [PR 61] (https://github.com/stolostron/prometheus-alertmanager/pull/61). Updated these files as part of this PR as well.  - FYI: @coleenquadros
	modified:   template/email.html
	modified:   template/email.tmpl
	modified:   ui/react-app/dist/26.js.gz
	modified:   ui/react-app/dist/26.js.map.gz
	modified:   ui/react-app/dist/main.js.gz
	modified:   ui/react-app/dist/main.js.map.gz

Here is the file diff between v0.27.0 upstream fork now
# git remote -v
origin	https://github.com/subbarao-meduri/prometheus-alertmanager (fetch)
origin	https://github.com/subbarao-meduri/prometheus-alertmanager (push)
stolostron	https://github.com/stolostron/prometheus-alertmanager.git (fetch)
stolostron	https://github.com/stolostron/prometheus-alertmanager.git (push)
upstream	https://github.com/prometheus/alertmanager.git (fetch)
upstream	https://github.com/prometheus/alertmanager.git (push)

# git diff v0.27.0 --stat
 COMPONENT_NAME           |  1 +
 COMPONENT_VERSION        |  1 +
 Containerfile.operator   | 30 ++++++++++++++++++++++++++++++
 DCO                      | 37 +++++++++++++++++++++++++++++++++++++
 Dockerfile.prow          | 30 ++++++++++++++++++++++++++++++
 Makefile.common          |  1 +
 OWNERS                   | 17 +++++++++++++++++
 README.md                |  1 +
 SECURITY.md              |  2 +-
 go.mod                   | 14 +++++++-------
 go.sum                   | 32 ++++++++++++++++----------------
 sonar-project.properties | 10 ++++++++++
 12 files changed, 152 insertions(+), 24 deletions(-)

/retest-required


Fix typo

Fixes #28184

@Potapy4 : Thanks for your contribution! The author, @wmgries, has been notified to review your proposed change.

@Potapy4 - Thanks for making this fix.
#sign-off

@tamram: I'm sorry - only the author of this article, @wmgries, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@tamram - I like the message from the PR merger :)


Terragrunt cache folder still contains deleted files in source folder

When using terragrunt, we noticed that if we renamed a source file or deleted a source file, the working folder still contains the original file / deleted file in its worker folder.
It seems that the copy files operation performed by Terragrunt does not do a sync, but just copy modified or new files.

Hm, it should delete all *.tf files: https://github.com/gruntwork-io/terragrunt/blob/master/cli/download_source.go#L322
Are you referring to other file types?

Oh yeah, we have some .yaml files and some template files being referenced in our .tf files.
Can we make it configurable such that it will sync all files against the working folder?

PRs for improving how Terragrunt does copying are welcome. There are several requests around this functionality:

Clean up all files/folders, not just .tf.
Copy some hidden files and folders, but not all (e.g., not .git). See https://github.com/gruntwork-io/terragrunt/issues/394.

@brikis98 I'm working on a related issue (#725) and figured that I could address cleaning up all files/folders as well. Is this functionality still desired, and if so, should this be the default behavior? I did a quick search on the open issues and wasn't able to see any discussion on whether we wanted that behavior to be configured or default.


malformed json indentation when using multiline string templating

Describe the solution you'd like
Retrieving an external stored json secret value via multiline string templating creates a malformed indentation json value in the kubernetes secret.
Give us examples of the outcome

Stored GCP Secret Manager value to retrieve "my-sa-creds":

{
  "type": "service_account",
  "project_id": "project-0123e456",
  "private_key_id": "abc0123abc",
  "private_key": "-----BEGIN PRIVATE KEY-----\value",
  "client_email"<EMAIL_ADDRESS>  "client_id": "000000",
  "auth_uri": "https://accounts.google.com/o/oauth2/auth",
  "token_uri": "https://oauth2.googleapis.com/token",
  "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
  "client_x509_cert_url": "https://www.googleapis.com/robot/v1/metadata/x509/iam.gserviceaccount.com"
}

ExternalSecret object used:

apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: thanos-spike
  namespace: monitoring
spec:
  secretStoreRef:
    kind: ClusterSecretStore
    name: gcp-secret-manager

  refreshInterval: 24h

  target:
    name: thanos-spike
    creationPolicy: Owner
    deletionPolicy: Delete
    template:
      engineVersion: v2
      type: Opaque
      data:
        thanos-store.yaml: |
          type: GCS
          config:
            bucket: thanos-testing-123a123
            service_account: |-
              {{ .sacreds }}

  data:
    - secretKey: sacreds
      # GCP secret details
      remoteRef:
        key: my-sa-creds

Afterwards k8s secret is generated, the resulting decoded b64 value:

type: GCS
config:
  bucket: thanos-testing-123a123
  service_account: |-
    {
  "type": "service_account",
  "project_id": "testing-0123e456",
  "private_key_id": "abc0123abc",
  "private_key": "-----BEGIN PRIVATE KEY-----\value",
  "client_email"<EMAIL_ADDRESS>  "client_id": "000000",
  "auth_uri": "https://accounts.google.com/o/oauth2/auth",
  "token_uri": "https://oauth2.googleapis.com/token",
  "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
  "client_x509_cert_url": "https://www.googleapis.com/robot/v1/metadata/x509/iam.gserviceaccount.com"
}

See how service_account  json value gets wrongly indented, thus my end-service fails to parse/read it so it can't use this secret.
This is how it should correctly look like:
type: GCS
config:
  bucket: thanos-testing-123a123
  service_account: |-
    {
      "type": "service_account",
      "project_id": "testing-0123e456",
      "private_key_id": "abc0123abc",
      "private_key": "-----BEGIN PRIVATE KEY-----\value",
      "client_email"<EMAIL_ADDRESS>      "client_id": "000000",
      "auth_uri": "https://accounts.google.com/o/oauth2/auth",
      "token_uri": "https://oauth2.googleapis.com/token",
      "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
      "client_x509_cert_url": "https://www.googleapis.com/robot/v1/metadata/x509/iam.gserviceaccount.com"
    }

Observations (Constraints, Context, etc):
I've tested to retrieve the same secret without multiline templating and recover it as a k/v , in this case the json doesn't get malformed and is correctly retrieved.
  target:
    name: thanos-spike
    template:
      engineVersion: v2
      type: Opaque
      data:
        bucket: thanos-testing-123a123
        service_account: "{{ .sacreds }}"

Hi @dllegru ! Have you tried using nident function?
like this?
        thanos-store.yaml: |
          type: GCS
          config:
            bucket: thanos-testing-123a123
            service_account: |-
              {{ .sacreds | nident 8}}

Hi @gusfcarvalho! Thank you for the hint! 😄
I managed to get it working with the following:
        thanos-store.yaml: |
          type: GCS
          config:
            bucket: thanos-testing-123a123
            service_account: |-
          {{ .sacreds | indent 4}}

Now It is generating the correct indented output in the secret  🎉
I've used indent instead of nindent to remove the newline at the beginning, also had to place the {{ .sacreds | indent 4}} bit indented left so the first line of the generated json value also gets correclty placed.
Thanks again for your help! 👍


Put time measurement steps into a function

Currently, time measurement is done by repeating a large amount of code. This should be simplified.
The idea would be to provide a function that measures the current time and returns the time lapse w.r.t. a previous time instant. This function would be added to the generated .h and .c files in some way during the code-generation process.

This has been added with the new snippet feature to all solvers as of release 'v0.3.10' (bb8406630ea952b0b5ba63e180d0030f91e8390c).


Revert "python37Packages.pyspf: 2.0.12 -> 2.0.13"

This reverts commit e3f9c4d79041c6114a67ff67772186b737477393.
This fixes #72791.
Version 2.0.13 introduced a bug where some incoming mails can cause a
crash and be falsely rejected. (https://github.com/sdgathman/pyspf/issues/17)

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] 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.

Notify maintainers
cc @FRidh

Could you include a comment next to the version info that 2.0.13 should not be used. That way we likely notice it when the bot proposes the change again.

Could you include a comment next to the version info that 2.0.13 should not be used. That way we likely notice it when the bot proposes the change again.

Good idea, done.

@GrahamcOfBorg build pypolicyd-spf python27Packages.pyspf python37Packages.pyspf python38Packages.pyspf


[FS Overview] "MORE..." link inaccessible in some screen sizes

Description of the issue
After filling out a Fact Sheet with two windows side by side, I wanted to browse existing Factsheets. However, I was unable to click on them. The "MORE..." link was inaccessible to me, because the shrunk down container pushed the link out of the visible area. There was no option to scroll to the link
Ideas of solution
1 Make the entire container clickable OR
2 enable scrolling within the container OR
3 create a button that is always shown, even in a shrunk down container.

I guess this should be looked at when the design is final.

I entirely changed the overview page. I tried to adapt it to the OEP logo. Therefore I used hexagons and removed the 'more button', and made the hexagons clickable:

Looks good, thanks!


Add type, values and required properties to api operation header structure

Fixes #862 - header properties are missing from extracted template

@microsoft-github-policy-service agree [company="Bentley Systems, Incorporated"]

@microsoft-github-policy-service agree company="Bentley Systems, Incorporated"

This project is now in "maintenance" mode. This means that no new features will be accepted and fixes will be limited to security fixes. We expect to retire the DevOps Resource Kit by the end of 2023. At that time, the DevOps Resource Kit repository will be archived and there will be no further updates.
While you are free to fork the DevOps Resource Kit yourself, we recommend that you migrate to the following products:

If you are using creator, switch over to using Bicep templates. Bicep is a descriptive language for describing infrastructure, fully supports all features of Azure API Management, and is covered in any support agreement that you have with Azure. You can find bicep modules and examples for handling API Management APIs at https://github.com/azure-samples/api-management-sample-apis.
If you are using extractor, switch your pipelines to use APIOps. APIOps applies the concepts of GitOps to API deployments. It is faster and more flexible than the DevOps Resource Kit.


[modify] webmail / replace FAST

Coverage decreased (-0.01%) to 83.19% when pulling 4deff903f585e12e7d3289cd41eede500906bdbe on tany:webmail-fast into 1cd0f846cf1118857bcb835b43b5d13f7edcb0a1 on shirasagi:master.