Ignore whitespaces in the grammar

Currently, the openCypher grammar has whitespaces modeled like the snippet below describing that ReadPart is a sequence of zero or more ReadingClause separated by whitespaces.
https://github.com/opencypher/openCypher/blob/8fa5658e433060a208971845a2b301d140461535/grammar/cypher.xml#L151-L155
To be more precise, the excerpt above also states that a ReadPart ends with a whitespace unless empty.
However, the next excerpt from ReadUpdateEnd is formalized more strictly to describe a sequence of one ore more of ReadingClauses that have whitespace separators strictly in between them, but not at the end.
https://github.com/opencypher/openCypher/blob/8fa5658e433060a208971845a2b301d140461535/grammar/cypher.xml#L111-L114
The reason behind the difference between the two excerpts is to make the grammar unambiguous also for whitespace matches.
If we could switch to whitespace-independent grammar, then the excerpt above could be rewritten in a more straightforward way like:
<repeat min="1">
  <non-terminal ref="ReadingClause"/>
</repeat>

I think it would be beneficial to switch to whitespace-independent formalization of the grammar. Also the patches in #300 could be rewritten in a more intuitive way then.

the railroad diagram generator already seems to skip whitespaces and does the hard work to illustrate one or more ReadingClause sequence of ReadUpdateEnd in its natural form despite the grammar says it is one followed by zero or more.
the generated ANTLR4 grammar could be rewritten to skip whitespaces (at least in theory, but I didn't try to do so)
is there a use for whitespaces in the original XML grammar or in the EBNF grammar artifact or anywhere else?

What do you think?

Note: related issues were discussed earlier in #56. Essentially, handling whitespaces is very easy in CF grammar-based parsers such as plain ANTLR and Xtext (which uses ANTLR 3 internally).
However, it is not trivial in PEG-based tools such as Parboiled. The Handling Whitespace wikipage of the parboiled project says this:

One disadvantage of PEGs over lexer based CFGs can be the handling of white space. In a traditional CFG based parser with a separate lexer (scanner) phase this lexer might simply skip all white space and only generate tokens for the actual parser to operate on. This can free the actual parser grammar from all white space treatment. Since PEGs do not have a lexer but directly operate on the raw input they have to deal with white space in the grammar itself. Language designers with little experience in PEGs can sometime be unsure of how to best handle white space in their grammar.

Thank you @szarnyasg for the reference and raising the point of PEGs.
However, PEGs usually do a greedy matching, so keeping whitespaces in the grammar and dropping (hiding) them in the ANTLR4 artifact:

might hide ambiguity for whitespace parsing, but greedy matching on whitespaces in PEGs will handle, i.e. for the ambiguous pattern SP* SP*, its first part will consume all whitespaces
the ANTLR4 tests will still raise ambiguity exceptions in case non-whitespace ambiguity is introduced
still allow for clarified repetition syntax similar to the one given in the issue description:

<repeat min="1">
  <non-terminal ref="ReadingClause"/> &WS;
</repeat>

I think this makes a lot of sense, and will certainly simplify the grammar as a whole. I don't think I can foresee what exact consequences (if any) this will have for the language, but I can't come up with any real problem, if the rule was formulated that whitespace is allowed between tokens in any position of the query.
Some examples that I've considered that don't seem to be a problem:

function invocations: RETURN   toInt  (  '1'   )
multi-word operators: RETURN   1   IS   NOT   NULL
patterns: MATCH (  a   :   A  )  -  [  r   :    TYPE   |   TYPE2   ]    -    >   (   b   :  B  {  b  :  1  }  )

As correctly identified in the above, the specification for whitespace as currently done is to model this very rule, but it turns out it's pretty tricky to get it right.
@thobe Do you have any input for this discussion?

Looks like I'm seeing this quite late.
I think what we should do is add a classification of non-terminals whether they are expected to be handled on a lexer level or on a parser level. A processing tool can then use that information to inject whitespace in appropriate places for output formats that needs to be explicit about it.

I agree, and we already have started such a scheme:
https://github.com/opencypher/openCypher/blob/64ec11f72be9e49b07ca21930a93b53f35ad57e8/grammar/basic-grammar.xml#L546-L559

Yes. I'm not sure I'm a fan of the particular naming, but it does the trick. "All" we need to do is use that for some sort of "whitespace injection", and we're set.


Install test version of Invenio

Install an intitial test version of InvenioRDM on AWS for testing and experimentation.
It will be accessible at invenio-dev.hcommons-staging.org

Local install is complete.


Workflow

Workflow
Proposal (in case we stay without a board)
The issues are running through stages:

pipeline ready: first draft of the pipeline is ready, so that it can be approved
table structure approved: the table structure has been approved: ready to make queries
queries ready: queries have made
queries approved: the queries are approved

Questions

These issues have been recently closed: should we reopen them?
Do we need a project board?

we continue to work with issues
we keep them open until everything is done (including upload to postgres, metadata and queries
we add new screenshot everytime the structure of the result changes
there is a label to approve the table structure


Could not load Text DB

I got the 3.0.0 Stable version and even with the Alpha i get the same Problem, searched trough the internet and could not find a solution.

Could you provide a screenshot of the error of the debug log file?

I did a lot of ini editing and such stuff, but nothing helped.
Am Mo., 10. Juni 2019 um 05:09 Uhr schrieb Etienne Lamoureux <
<EMAIL_ADDRESS>
Could you provide a screenshot of the error of the debug log file?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EtienneLamoureux/TQVaultAE/issues/137?email_source=notifications&email_token=AMI7DHMDO5F4N7CN6TO3F5LPZXAYFA5CNFSM4HWHAHEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXI2CWQ#issuecomment-500277594,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMI7DHJ5TQPCTLJPBBK5X3LPZXAYFANCNFSM4HWHAHEA
.

Can you give us a screenshot of the error message?

The only thing i can provide is the screen shot i added before
Etienne Lamoureux<EMAIL_ADDRESS>schrieb am Mo., 10. Juni 2019,
17:52:

Can you give us a screenshot of the error message?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EtienneLamoureux/TQVaultAE/issues/137?email_source=notifications&email_token=AMI7DHPLH56J4HR6JGONJNDPZZ2CRA5CNFSM4HWHAHEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXKI7SI#issuecomment-500469705,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMI7DHIQXFXK22LVCBOTIBLPZZ2CRANCNFSM4HWHAHEA
.

Ah, I think you may be replying by email. Attachments by email aren't shown in the issue discussion. Could yoy attach your screenshot in github directly?

i look it up how, never been on github, that will take some time
Am Mo., 10. Juni 2019 um 18:37 Uhr schrieb Etienne Lamoureux <
<EMAIL_ADDRESS>
Ah, I think you may be replying by email. Attachments by email aren't
shown in the issue discussion. Could yoy attach your screenshot in github
directly?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EtienneLamoureux/TQVaultAE/issues/137?email_source=notifications&email_token=AMI7DHMVRQEKDKOXGXLII6DPZZ7M3A5CNFSM4HWHAHEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXKNC7Q#issuecomment-500486526,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMI7DHJWYXK22I3K35MX3PDPZZ7M3ANCNFSM4HWHAHEA
.

i hope this works

Thanks

Do you play Titan Quest Anniversary Edition through Steam?

Also, did you follow the instructions in our F.A.Q. for that problem to the letter? https://github.com/EtienneLamoureux/TQVaultAE/blob/master/README.md#troubleshooting-and-faq

Yeah, through steam

yes i also followed those instruction but it didnt work.

Please try with the latest version.


[Excel] [PnP] Open in Excel (#2188)

adding draft of PnP article open in Excel

fix links

fix code language

format fixes

add intro image

Apply suggestions from code review

Co-authored-by: Linda Lu Cannon<EMAIL_ADDRESS>

typo fixes

link fix

Add the see it in action links later

add auth token warning

Co-authored-by: Linda Lu Cannon<EMAIL_ADDRESS>

Docs Build status updates of commit d4d325c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/pnp-open-in-excel.md
:white_check_mark:Succeeded
View

docs/images/embed-script-lab-sample-ui.png
:white_check_mark:Succeeded
View

docs/images/fluent-ui-wxp.png
:white_check_mark:Succeeded
View

docs/images/pnp-open-in-excel.png
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Atom for Windows 1.55.0 Crashes when trying to open

Each time I try to start Atom, it crashes immediately after displaying the following screen.

I've tried uninstalling/re-installing the latest version with no success.  Same issue.

I have the same issue, but I was able to open it from the console using safe mode : "./atom.exe --safe"
It could be an issue with the community packages since they are disabled in safe mode.
Here is my list of packages:

atom-ctags 5.1.2
block-comment 0.4.3
busy-signal 2.0.1
editorconfig 2.6.1
file-icons 2.1.47
html-tag-auto-complete 2.1.0
intentions 1.1.5
linter 3.3.1
linter-ui-default 3.3.1
minimap 4.39.13
pigments 0.40.6
prettier-atom 0.60.1
react 0.18.0
symbol-gen 1.3.1
teletype 0.13.4
jshint 1.8.6
language-babel 2.85.0

I had the same problem. However, with atom 1.55 it works just fine. All later releases (1.56, 1.57, 1.58) however throw this error on my machine! In safe mode, there is no issue.
I re-installed v. 1.55 and deactivated all installed community packages. Then I installed atom v. 1.58 and enabled all those packages one-by-one. I restarted atom each time I enabled a new package. After I enabled arduino-upload, the editor crashed.
Deinstalling arduino-upload (1.8.0) did the trick for me.
I could reproduce the error by deactivating all packages and installing arduino-upload 1.8.0. The editor crashed immediatly. Also it crashed instantly upon startup. I deinstalled the package again using apm uninstall arduino-upload in the Anaconda prompt on Windows 10.
You could try the same thing and check if one of your installed packages causes this error.

I had the same problem. However, with atom 1.55 it works just fine. All later releases (1.56, 1.57, 1.58) however throw this error on my machine! In safe mode, there is no issue. I re-installed v. 1.55 and deactivated all installed community packages. Then I installed atom v. 1.58 and enabled all those packages one-by-one. I restarted atom each time I enabled a new package. After I enabled arduino-upload, the editor crashed. Deinstalling arduino-upload (1.8.0) did the trick for me. I could reproduce the error by deactivating all packages and installing arduino-upload 1.8.0. The editor crashed immediatly. Also it crashed instantly upon startup. I deinstalled the package again using apm uninstall arduino-upload in the Anaconda prompt on Windows 10.
You could try the same thing and check if one of your installed packages causes this error.

Thank You I was finally able to solve it Atom needs to have better moderation of community packages

I wonder if there's a way to get the error message for why Atom crashed


A controller method that returns null/none is considered a 404

This is incorrect. Determine how to handle this situation so that it is not a 404 or make it clearer that a controller method must have a non-null return.

Solved by https://github.com/nox7/nox/commit/b42b04326867ee319d41f41f1d1bb02d17b2c46a


Civetweb: create separate directory

Is your feature request related to a problem? Please describe.
CivetWeb using samples have potential to spread over multiple directories (especially after merging of #28323).
Describe the solution you'd like
Put all CivetWeb using samples into one civetweb root directory. Thus allowing http/websocket and many more CivetWeb based samples existing in one root directory (e.g. samples/net/civetweb/http_server, samples/net/civetweb/websocket_server).
Describe alternatives you've considered
none. (Maybe your suggestion?)
Additional context
Should be done incrementally after merging of #28323

@jukkar It was mostly your idea, to put CivetWeb samples into same root directory. Please provide short look at this issue.

Agreed with separate directory. The samples/net/civetweb/ is good place to put the sample apps.

Reopened due to  #30175


Consider auto-initializing collection navigation properties when materializing entities and in other scenarios

Change #637 fixes all known cases in which we currently throw NullReferenceException from our stack due to uninitialized navigation collections. E.g. it stops trying to iterate over the contents of collections when the collections themselves are null and it auto-initializes the collections when we need to add items to them.
This issue is about initializing collections more consistently so customers don't need to initialize the collections everywhere to avoid NullReferenceExceptions from being thrown from application code. There are several places in which doing this could make sense:

Materialization
Simple creation, e.g. db.Set().Create()
Attachment, i.e. any time we start tracking entities

Creation of navigation properties happens on a best effort basis, e.g. we have convention that looks for specific types, specific constructors and specific patterns on the parent class, e.g. collection setters. We should decided what to do when we can't figure out the right type, constructor and setting mechanism, and whether we can factor this as a service.

Decided we should just do 1 from the initial list (reactivated https://github.com/aspnet/EntityFramework/issues/451 to track this).
2 doesn't apply (DbSet.Create doesn't exist)
We're not sure 3 is a good idea


[prim_sync_reqack] Check usage of EnRzHs parameter to make EDN interfaces partial-reset safe

Description
For Earlgrey ES, most hardware blocks interfacing with EDN are using the default non-return-to-zero (NRZ) handshake protocol in the corresponding prim_sync_reqack modules. This is faster from a peformance point of view but it doesn't allow safely resetting the hardware block without also resetting EDN as there is always a risk of the two sides of the CDC primitive getting out of sync. IIRC, this isn't an issue for the specific reset topology of Earlgrey ES but it might be for other top levels.
We should thus check all hardware blocks interfacing with EDN and decide whether it would better to switch to the slower but partial-reset safe return-to-zero (RZ) handshake protocol.
For details, refer to https://github.com/lowRISC/opentitan/issues/19232#issuecomment-1641854328 . This was also discussed in the Sec WG meeting of Jul 20, 2023.

For Earlgrey and Darjeeling, the entropy complex and all entropy-consuming security or crypto devices have the same logical root reset. Or in other words, none of the entropy consumers reside in the AON domain or are SW resettable through rstmgr:
https://github.com/lowRISC/opentitan/blob/143098eec598e92bd7dd20939d5c7587f4039705/hw/top_earlgrey/data/top_earlgrey.hjson#L155-L162
Therefore, it is not necessary to use the partial-reset-safe implementation of prim_sync_reqack in those cases and no further design changes are needed.


Update instructions on how to run it locally

I had to tweak it even further to get it to work

Thank you @johncornish


This commit updates the test results to work with the latest ocaml-dockerfile submodule.

Looks like another promotion is needed to satisfy the linting check: https://ocaml.ci.dev/github/ocurrent/opam-repo-ci/commit/edca6c4ba1a15f49da22c5d7eff19b0fa785a454/variant/(lint-doc)#L1028-1028

Looks like the change to deprecate Ubuntu 23.10 in https://github.com/ocurrent/ocaml-dockerfile/pull/214 which is due to be merged next week is already a part of the submodule on master. Do we want to revert it? Or are we okay to deprecate the OS a week earlier?

I'm happy to deprecate it a week early so save an extra redeployment.


How to pass globals from main thread to run-multiple run

Hi,
I am using globals with nodejs +webdriver and I cant see any working way to pass globals with un-multiple
same for run-workers.
I will be happy to see some examples

What kind of globals?

I am trying to set some data on global , this data is dynamic the set is from bootstrup, the issue when you run with workers the global change workers.any update about this issue?

@koushikmohan1996 Just bumped into your PR, thanks for the reference. I've got a question, would the import from inject supposed to work only within a test? Or would I be able to do it at root level of the file? In this example, you are importing the shared data from inject inside the test and it works fine. The same doesn't work when trying to import from root level, maybe because the root level imports are evaluated way earlier?


distributor: format go import

Part of https://github.com/keptn/keptn/issues/3150
/cc @christian-kreuzberger-dtx

@christian-kreuzberger-dtx I saw this package the go version is outdated, is it okay to open a follow PR to update the go version?

We have a separate issue for that: https://github.com/keptn/keptn/issues/2936
If you want, you can take it and upgrade go.mod files, Dockerfiles, and github actions etc...


decorator only trace very top level function

fixes #6492

Can one of the admins verify this patch?


Create sus-am-bktm.txt

Website: https://bk-tm.de/
Inforamtion About Courses: http://relaunch.bk-tm.de/bildungsgaenge/berufsschulabschluss/
Domain is E-Mail Domain "Schülerinnen und Schüler" = SUS, "am" = at, "bk-tm" = Berufskoleg Platz der Republik Technik und Medien

@Faruktfk Thank you for the clarification. Could you please still provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@Faruktfk You might miss my previous comment. Please react.

@Faruktfk I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!

Hi @philipto,
I'm sorry that i couldn't answer sooner, but it took a while to get an offical letter from my school about the recognition of the domain above.
I hope this letter below works.
Best regards
Faruktfk


Issues with Suggestions on Mobile

It seems that when using a mobile there is an issue with the suggestions not being clickable (when clicked it is as if I have clicked somewhere else on the map, closing the keyboard and not searching).
As a quick (temporary) fix, is there a way to turn off suggestions?
I have noticed people have brought this up already and that it should be fixed so hopefully its all on my side.

i couldn't reproduce this problem in v2.2.14.
happy to chat more if anyone can provide additional information.


New package: ISL v0.1.0

Registering package: ISL
Repository: https://github.com/JuliaLabs/ISL.jl
Created by: @vchuravy
Version: v0.1.0
Commit: bc99ecc2c296d80c46e3b684b747c0262f59ad65
Reviewed by: @vchuravy
Reference: https://github.com/JuliaLabs/ISL.jl/commit/bc99ecc2c296d80c46e3b684b747c0262f59ad65#commitcomment-51099956

This is includes the Julia wrappers for the ISL library, and is the front-end package to ISL_jll. As such I don't think there is really a better name and I would like to keep the current one.


How to cleanup (aka remove) secret files after they removed from home-manager config?

The title says it all. There should be a way to remove every via hm deployed secret that is not in the current generation. Otherwise the system could end up in a non reproducible state.
agenix has the same issue as mentioned in my https://github.com/ryantm/agenix/pull/27#issuecomment-963175438.

Currently all secret files are removed through the use of the secrets folder. It is a symlink to the home manager build (through the use of home.file). Therefore when a home manager upgrade happens, the old age file is not symlinked anymore. It is removed from the file system when you garbage collect your old home manager generations.

I think to have a good clean up, all then broken symlinks should be removed too. A solution to this could also solve the cleanup for copied files.

Ya I definitely agree with that. For now broken symlinks are workable (but not ideal) as they at least don't contain the file. A cleanup solution is needed. Would work by adding a home.activation.homeageCleanup script after the writeBoundary.


Badges added

Fixes Issue🛠
#98
Closes #
Description👨💻
added badges in readme
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I am an Open Source Contributor

Screenshots/GIF📷

Good! @khushimarothi


Dynamic arguments with upper case doesn't work

Dynamic arguments will not work when being specified by a data property with upper case.
Ref: Dynamic arguments upper case problem · Issue #9528 · vuejs/vue
In the example of Dynamic Arguments below, however, the name of the data property is attibuteName and eventName.
Ref: Template Syntax — Vue.js
I know that there is a description at Dynamic Argument Expression Constraints section saying that using uppercase keys should be avoided, but it's still confusing that the example with attibuteName and eventName don't work.
I think we can change the name of the data property from attributeName to attribute and from eventName to event. What do you think?

While I hear your concern, having the argument as mere attribute or event wouldn't be as expressive and may actually cause another confusion, as they more than often refer to the objects (the attribute or event themselves) and not the names. What do you think about this alternative:
<!--
Note that there are some constraints to the argument expression, as explained
in the "Dynamic Argument Expression Constraints" below.
-->
<a v-bind:[attributeName]="url"> ... </a>

@phanan
Thanks for replying back soon. I agree with the alternative!
Just to clarify, but what I understand from this report is that the attributeName get lowercased to attributename and Vue will see if there is attributename data. Please let me know if I didn't get it right.
If my understanding is right, I think we should also edit the following sentences as follows.

For example, if your Vue instance has a data property, attributeName, whose value is "href", then this binding will be equivalent to v-bind:href.

For example, if your Vue instance has a data property, attributename, whose value is "href", then this binding will be equivalent to v-bind:href.

Similarly, when eventName‘s value is "focus", for example, v-on:[eventName] will be equivalent to v-on:focus.

Similarly, when eventname‘s value is "focus", for example, v-on:[eventName] will be equivalent to v-on:focus.
I am not sure if the lowercased, not camel cased variable name is proper as an example though...

@mmiyahara Can you please check the PR linked above and see if it suffices.

@phanan
Thank you for creating PR. I just want to make sure if you check https://github.com/vuejs/vuejs.org/issues/2256#issuecomment-516743300. Except for the things I mentioned in https://github.com/vuejs/vuejs.org/issues/2256#issuecomment-516743300, everything looks fine.


updated proxy for production

Please cross check the port.

are you sure that the proxy is running on port 443

In the mail sent by Shomy to all, no port is mentioned.

Proxy server port updated after checking with shomy.


How to calculate receptive field?

Hi @mattmacy
I was going through VNet paper ,if you see at end they have given table for receptive field for each stage.
for example

stage 1 its 5x5x5
stage 2 it is 22x22x22

can you please help me how it is calculated , i went through lot of material for receptive field calculation but no luck,hope you will help me
thanks
sagar

This might be late, but I made an implementation to compute the RF numerically.
Here it is:
https://github.com/rogertrullo/Receptive-Field-in-Pytorch


Lets use 2 different (funded) wallets for our tests

Currently the test run for for the sequencer provider and for the rpc provider share the same wallet and keypair for testing provided by TEST_ACCOUNT_ADDRESS= and TEST_ACCOUNT_PRIVATE_KEY=.
I propose to add a 2nd funded wallet (we need some more funds for the existing wallet too, as it's almost empty)
we could rename the secrets into something like:
TEST_ACCOUNT_ADDRESS_1=***
TEST_ACCOUNT_PRIVATE_KEY_1=***
TEST_ACCOUNT_ADDRESS_2=***
TEST_ACCOUNT_PRIVATE_KEY_2=***

and map it inside the workflow file to the job
This would enable us to run release pipelines in parallel and would speed up the pipeline 3 times

We can't run them in parallel in the same wallet because it would cause a nonce issue right?
Would it be wise to name the wallets more explicitly? like
TEST_ACCOUNT_ADDRESS_RPC

&

TEST_ACCOUNT_ADDRESS_SEQUENCER

Right, currently we limit the matrix to run maximum 1 job in parallel because otherwise we would get nonce issues
See https://github.com/0xs34n/starknet.js/blob/develop/.github/workflows/release.yml#L29
I opted for more general names to avoid having explicit mapping to sequencer/rpc in the secret name
This mapping would be done in the workflow file
By keeping it this way we can add more/less accounts and just do the mapping in the workflow file, in example if we decide to have multiple test accounts for one kind of provider to parallelise tests further

Sounds good. Also, I added 0.015 ETH (max amount on layer swap) to the almost empty wallet.

rip

funded 0.25 eth in testnet

Closing this issue as the Sequencer provider is going to be deprecated, untill than we are going to run on 1 test acc.


Add Client header to REST Calls

This PR adds a metadata-client header to the REST calls made from the CLI.
The aids in identifying calls coming from the CLI explicitly and internal analysis.
Signed-off-by: daidokoro<EMAIL_ADDRESS>Ref: https://github.com/reliablyhq/entity-server/pull/104

My 2cts, usually extra headers are prefixed with X- for clarity.

Thanks @dmartin35, I wanted to a way to add additional metadata to our headers without necessarily logging all headers on the server of having to specify explicitly which headers to print.
So the server logs anything prefixed with metadata- but I can adjust that to X- at @Lawouach's request.
Didn't realise  we already defined user-agent so I'll kill this PR for now.


Table Services Requirements Scrub

Is your feature request related to a problem? Please describe.
TBL requirements out of date
Describe the solution you'd like
Update requirements
Describe alternatives you've considered
N/A
Additional context
Summary of changes (w/ links if individual issues exist already):

cTBL6011 - HK requirement not consistent (update wording)
cTBL6301 - Free app resources handled by ES_AppExit, delete
- cTBL6700 - Maximum tables (delete, mission config)
- cTBL6701 - Maximum simultaneous loads and dumps (delete, mission config)
NEW API Requirements:

cTBL6313: CFE_TBL_Validate
cTBL6314: CFE_TBL_DumpToBuffer
cTBL6315: CFE_TBL_Modified cfe-999
cTBL6316: CFE_TBL_Manage cfe-995

NEW Cmd Requirements:

cTBL6013: CFE_TBL_DeleteCDSCmd
cTBL6013.1: Delete CDS - Table In Use
cTBL6013.2: Delete CDS - Not Critical Table
cTBL6013.3: Delete CDS - Already Deleted

Requester Info
Jacob Hageman - NASA/GSFC
EDIT - Bold items missed in first round

Closing since PR #526 includes this fix


Redmine 3.2

This plugin will be updated to work with Redmine 3.2?

Yes, and it will be done this week, if possible

Hi, on redmine 3.2 I got an "Internal error" after plugin installation. Is it still not compatible with 3.2?
Thanks

For info, git clone of the plugin does not work under Redmine 3.2 but if you rename the redmine_advanced_roadmap_v2 folder to advanced_roadmap_v2, then it seems to work.

There's no issue with redmine 3.2. I'll just update information on redmine website.


Log error for failing calendar

One of my calendars broke at some point (no clue why). It took me forever to realize what was wrong because the only thing logged to my Firefox console was "TypeError: Object". This patch improves the error message so that users can actually find out what is wrong (example: Error: Couldn't load calendar.work: Response error: 400).
I want to see if I can at some point figure out how to show the error message IN the card and continue loading all the other events from other calendars, but for now, this is at very least an improvement over the current situation.

Closing this in favour of #52


Savefile operators don't work anymore

wixoa nuking IDreamProcIdentifier made savefile["entry"] << 1 and savefile["entry"] >> var  non-functional, which is blocking https://github.com/wixoaGit/OpenDream/issues/559

Also the fix is to make a << b a statement with a different instruction from the bitshift expression.
See https://discord.com/channels/847318834952405042/847318834952405045/944707503038529636

Fixed by #894


[admin] Publish

[Outlook] (smart alerts) Update walkthrough

Add image with attachment recommendation

Updates (#3291)

[Outlook] (event-based) Add how to update commands.html

Move loopback exemption note

Apply suggestions from code review

Co-authored-by: Sam Ramon<EMAIL_ADDRESS>

Tweaks

Tweaks

Updates based on feedback

Co-authored-by: Sam Ramon<EMAIL_ADDRESS>
Remove regkey info since not needed for these events

Co-authored-by: Sam Ramon<EMAIL_ADDRESS>

Docs Build status updates of commit 11b0a7a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/outlook-win-smart-alert.png
:white_check_mark:Succeeded
View

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

docs/outlook/smart-alerts-onmessagesend-walkthrough.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Handle non-www to www redirection better

We serve the M-Lab site on www.measurementlab.net, and we want to redirect requests for measurementlab.net to that domain. However, we need to point the root domain to an IP address (an A record).  CloudFront and S3 don't give you IP addresses to point to (that I know of), but simply domain names, and they manage and muck the underlying IP addresses as they need to. A lot of DNS hosting services provide a redirection service like this, include Amazon's own Route 53.  Since we host our own DNS for measurementlab.net, we're going to need to figure out how to redirect requests to measurementlab.net to www.measurementlab.net.  As a stop-gap, I pointed measurementlab.net to our Linode Nagios VM, and Apache on that machine redirects (301) all requests to www.measurementlab.net. While this works fine, we won't ultimately want the Linode VM having anything to do with the core website. We can find some service will provide this redirection for us, or we could deploy the redirection on some other server we manage, or we could move our DNS to another provider (like Google Cloud DNS or Amazon Route 53) who provides this sort of redirect.
Additionally the above temporary redirection mechanism does not work for HTTPS at the moment.

In https://github.com/m-lab/ops-tracker/issues/145  and https://github.com/m-lab/m-lab.github.io/issues/268
I've changed the behavior of our CloudFront entry to redirect HTTP to HTTPS. So this issue is largely resolved for now. As mentioned in those other tickets, https without www does not yet redirect, but we should do this in Amazon instead of using EB for redirects.

With the move to Firebase, this is now resolved. All redirects are handled in Firebase configs.


Incorrect extension suggestion

Version: 1.48.0-insider (user setup)
Commit: c828a5339bca7222fd8f56ab98361a98f736d185
Date: 2020-07-14T05:39:53.264Z
Electron: 8.3.3
Chrome: 80.0.3987.165
Node.js: 12.13.0
V8: 8.0.426.27-electron.0
OS: Windows_NT x64 10.0.19041
Steps to Reproduce:

Open the vscode repo in insiders
Open a typescript file

Does this issue occur when all extensions are disabled?: Yes

cc @chrisdias

Sorry about that. I reverted the fix yesterday, it should be fixed in today's Insiders.
Commit: cb31fc6ce2b4da3520392b622aa3ebc5b313533b
Date: 2020-07-15T05:31:08.767Z


Add Portuguese translation and show queue time to connect in session

Add regions name in settings
Add stream queue estimate time to connect

Queue time

@Sirherobrine23 Well done! The related changes will be released together in the next version.


Image Not load

What steps will reproduce the problem?
Why yii2 (2.0.6) not load images with names like digit.png, digita.png, digital.png, etc..? If the name of image contains name digit the image not load. Why?
The code?
echo Html::img(Yii::getAlias('@web'.'/img/asdfdigitasdfsdf.png'));
What do you get instead?

Failed to load resource: the server responded with a status of 403 (Forbidden)

Additional info

Q
A

Yii version
2.0.6

PHP version
5.6.15

Operating system
Windows 10

Yii can't deny image access so it should be a problem with server config.

Are u tried? I'm testing in localhost and only happens with Yii2.


Replace all mentions of VECTORs in the reference for MAPs

LDPL VECTORs have been formally renamed to LDPL MAPs. The old keyword can still be used but the correct way of declaring VECTORs is now to write MAP instead. This should be reflected in the documentation, and every mention of an VECTOR should be changed to MAP. One should mention also that MAPs were formally called VECTORs and that the keyword is still supported albeit deprecated.

Done!


StatefulSet becomes unavailable on worker node shutdown or crashes.

Hi,
We're currently testing a Kubernetes cluster with an IBM backend as storage solution and we ran into an issue where nodes cannot be shutdown properly due to an existing issue with the driver. The documentation has a page in regards to Recovering from a crashed Kubernetes node which states the following:

This section details a manual operation required to revive Kubernetes pods that reside on a crashed node due to an existing Kubernetes limitation.

When a worker node shuts down or crashes, all pods in a StatefulSet that reside on it become unavailable. In these scenarios, the node status is NotReady, and the pod status appears as Terminating.

Having to do a restore procedure after a node crash is to be expected but on a node shutdown not so much as Kubernetes runs on highly dynamic environments where nodes come and go this is a pretty big blocker for us.
I tried to find some more information on this but I can't seem to find any in the open or closed issues. Hence the reason for me creating this issue.
Could you please provide me with some more information in regards to this issue:

Technical description of the problem.
Possible mitigations to prevent it from happening.
A timeline if/when this issue is going to be resolved.
Any links or reference to upstream in regards to this issue.

Thanks in advance.

I'd like to point out that the procedure as describe in the docs does not work as advertised. You have to go in and remove the finalizer from the volumeattachments as well.


Support ARGB32 images

Fixes https://github.com/JuliaIO/FileIO.jl/issues/352

The "too many files" from that bug is triggered because FileIO falls back to ImageMagick which struggles to do anything properly :smile:. Fix this, and then the saving gets handled by PNGFiles.

Hi and thanks for the fix! Just fyi, I managed to get tests passing by applying the following diff to this PR:
@@ -367,11 +367,11 @@ function _save(png_ptr, info_ptr, image::S;
         end
     else
         image_eltype = eltype(image)
-        if (image_eltype <: BGR || image_eltype <: BGRA || image_eltype <: ABGR)
+        if (image_eltype <: BGR || image_eltype <: BGRA || image_eltype <: ABGR || image_eltype <: ARGB32)
             png_set_bgr(png_ptr)
         end

-        if image_eltype <: AbstractARGB
+        if (image_eltype <: ABGR || image_eltype <: ARGB)
             png_set_swap_alpha(png_ptr)
         end

@@ -455,7 +455,6 @@ end
 _prepare_buffer(x::IndirectArray) = UInt8.(x.index .- first(axes(x.values, 1)))
 _prepare_buffer(x::BitArray) = _prepare_buffer(collect(x))
 _prepare_buffer(x::AbstractMatrix{<:T}) where {T<:Colorant{<:Normed}} = x
-_prepare_buffer(x::AbstractMatrix{ARGB32}) = ARGB{N0f8}.(x)   # unclear why this helps (maybe endianness?)
 _prepare_buffer(x::AbstractMatrix{<:T}) where {T<:UInt8} =  reinterpret(Gray{N0f8}, x)
 _prepare_buffer(x::AbstractMatrix{<:T}) where {T<:UInt16} = reinterpret(Gray{N0f16}, x)
 _prepare_buffer(x::AbstractMatrix{<:T}) where {T<:Normed} = reinterpret(Gray{T}, x)

I.e. by not swapping alpha but treating the colorant as BGR. Apparently, libpng treats the bytes in ARGB32 type as BGRA, so alpha is in the right place, it just the BGR that needs to be "swapped to RGB".

Thanks, @Drvi! I credited that to you and will make you a co-author of the squashed commit.


Long card titles overflow

When a card title is too long it needs to be truncated

We'll need to truncate the text in the markup here, unless we want to allow cards to grow. There's no way in CSS to use an ellipsis to limit vertical flow.

@shanberg - That sounds like a good approach to me. 👍

Reassigning to @7ruth. Martin, please tag @shanberg and either Kait or Karina as reviewers on your PR.


DevSupport classes included in release apks (~40KB size impact) due to DevLoadingModule

Description
This is an issue with React Native 0.72.
According to the design of DevSupportManager
https://github.com/facebook/react-native/blob/d724007364c4315e2cabace0fc6eae6e6212431d/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DefaultDevSupportManagerFactory.java#L21-L26
BridgeDevSupportManager class and its dependencies should be stripped by Proguard in release builds, however on inspecting the release APKs with proguard enabled, I found that many DevSupport classes were included. On digging deeper, I found the issue here https://github.com/facebook/react-native/blob/d724007364c4315e2cabace0fc6eae6e6212431d/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/modules/devloading/DevLoadingModule.java#L16
On deleting the usages of DevLoading Module, I found that the unnecessary DevSupport classes were no longer included in the release APKs.
Please look into refactoring DevLoading Module so that it doesn't cause unnecessary bloat in the release APK
Steps to reproduce

npx<EMAIL_ADDRESS>init test072App --version 0.72.14
Enable proguard in app/build.gradle https://github.com/facebook/react-native/blob/d724007364c4315e2cabace0fc6eae6e6212431d/packages/react-native/template/android/app/build.gradle#L56
Build and check the classes.dex in APK Analyzer, find unnecessary classes like BridgeDevSupportManager. Total devsupport package size (~52KB), which should be ~14KB (checked when I deleted DevLoadingModule from react-native)

React Native Version
0.72.14
Affected Platforms
Runtime - Android
Output of npx react-native info
System:
  OS: Windows 11 10.0.22621
  CPU: (8) x64 11th Gen Intel(R) Core(TM) i5-11300H @ 3.10GHz
  Memory: 3.15 GB / 15.79 GB
Binaries:
  Node:
    version: 18.16.1
    path: C:\Program Files\nodejs\node.EXE
  Yarn: Not Found
  npm:
    version: 9.5.1
    path: C:\Program Files\nodejs\npm.CMD
  Watchman: Not Found
SDKs:
  Android SDK: Not Found
  Windows SDK:
    AllowAllTrustedApps: Disabled
    Versions:
      - 10.0.19041.0
IDEs:
  Android Studio: AI-221.6008.13.2211.9619390
  Visual Studio:
    - 17.0.32014.148 (Visual Studio Community 2022)
Languages:
  Java: 17.0.1
  Ruby: Not Found
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.72.14
    wanted: 0.72.14
  react-native-windows: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: Not found
  newArchEnabled: Not found

Stacktrace or Logs
Check screenshots

Reproducer
https://github.com/garvsgit/test072App
Screenshots and Videos

Updated with template reproducer with proguard enabled

This issue is not present in 0.73 and above. Have re-opened #44698 as a pick-request to 0.72 for the solve

This issue is not present in 0.73 and above. Have re-opened #44698 as a pick-request to 0.72 for the solve

Thanks for the investigation @garvsgit
As this issue is already fixed, what is preventing you from updating to 0.73 or 0.74?
0.72 is end of life and we limit the pick requests to only extremely necessary picks

I depend on a few out-of-support packages which do not work well with 0.73 so holding off on an upgrade for now.
I understand that 0.72 is end of life and this is only a nice to have pick, not a necessary one so I'll be closing this issue and the associated PR.
Thanks!


Add config to override shims provider class (#765)

Add config to override shims provider class

Signed-off-by: Jason Lowe<EMAIL_ADDRESS>
Update docs for shims provider override config

Signed-off-by: Jason Lowe<EMAIL_ADDRESS>

build

This shouldn't be a cherry-pick, it should be a re-merge of branch-0.2 to main.

Using merge instead of cherry-picking, new PR: https://github.com/NVIDIA/spark-rapids/pull/770

Using merge instead of cherry-picking, new PR: #770


Setup video

I was going over old Facebook comments and spotted a follow-up I missed months ago with somebody asking how to install. In my personal style of overwhelming knowledge I offered both normal game download and full DevSetup info - which was too much and the user suggested making a video.
I suppose we could do that for a local dev setup at some point, maybe especially the Git + SSH setup (even though that's pretty well documented on GitHub)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/487369-setup-video?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

@Cervator Isn't this issue covered? The gradle script seems to do the "magic".

With GitHub promoting https: connections over ssh and the fancy gradlew setup in place pretty much everything is covered. If you still want to do a video of sorts, I'd say it should contain the following:

log in to github and fork the project
clone from your fork (console command) and add upstream remote repo
fetch some modules
create a new module
run idea task and open up in intellij -> do changes
run the game via a) command line, and b) intellij

I'm not sure if it is worth the effort. The video would showcase more or less our custom setup, and how to use the gradle magic in place.

I think the wiki provided with this project is more than enough to get someone started but this issue is still open.

This issue is really old, yeah. It can be easy to think our Gradle utility will do everything really easy but that's probably in part because we're already developers and perhaps already familiar with some of the concepts (pinging @AWebb3701 / Kazimir on that note since he rightfully woke me up to that perspective)
A modder, texture artist, play test enthusiast, or other individual who hasn't done much if any coding before might get a lot out of seeing a basic video that covers what @skaldarnar mentions above. "A picture is worth a thousand words" so a video is probably worth about a million :-)
Still, a video can take some effort to set up and maintain. So we need somebody who doesn't mind re-recording an update once in a while. Maybe ideally a set of really small videos of a minute or less in length that each cover a very small part of the process. That makes it easier to re-record updates later. We can put them all together in a playlist and link from various tutorials.
Other ideas for what to cover:

Using a GitHub native client (GitHub Desktop or whatever it is called these days)
Bypassing Git entirely and using a src zip downloaded from GitHub
Bypassing source completely and using a downloaded game zip to mod in assets like new blocks, which is entirely doable with plain "directory modules" - aka extracted jar files that live in a regular directory
Grabbing and using useful tools that relate, like 2D texture editors or Voxelshop
Highlighting the runtime asset reloading (there is an existing video or two on this) and exactly what reloads in what situations - like what needs a fresh stack created in-game, what needs a new block placed vs. applies to an existing block, and so on
Using Eclipse instead of IntelliJ

There are lots of possibilities, really. I'm pretty sure we're talking more about a whole series of small snippets.

Closing in favor of #2569 - most of the discussion was brought over to the DoD 🙂


Possible to create a single key that can both sign and derive a shared secret/key?

As far as I can tell you cannot sign with X25519, and you cannot generate a shared secret and key with Ed25519. I do not want to create 2 different keys, if possible. Thanks!

Well I'm not sure if the issue I specifically mentioned is possible, but for what I'm doing I realized I can simply generate an ephemeral X25519 keypair, sign it with the long term Ed25519 keypair, and on each end exchange ephemeral X25519 public keys, then from THAT generate a shared secret. And it's probably in the best interest to not have a signing and encrypting key be the same. For now I am closing it. :)

Which specification are you implementing?

Currently not a set-in-stone specification, just looking to sort of duplicate the scuttlebutt protocol's decentralization and key system and the whisper protocol.
Essentially, I would want a single Ed25519 key to be the identity for a single person. Say Alice wants to talk to Bob. They both have each other's public keys (maybe a DHT-type system for public keys). Locally whenever a new username is registered or when they first connect, everyone calculates the shared secret between them and each user. That way, when something encrypted gets delivered, each user who sees it will firstly forward it on to other nodes, then try to decrypt it with each shared secret they have. From what I can tell you cannot calculate Ed25519 key shared secrets nor sign with X25519, so I may just have to make an Ed25519 and X25519 keypair and add them both.

It seems that Scuttlebutt is using static, long-term Ed25519 keys and ephemeral, short-term X25519 keys. So I don't see a need for keys that can both sign and perform key agreement here, or a need to distribute more than one public key. Maybe you could do the same?

This would require signing the short term X25519 public key. I'm looking to reduce such metadata in a system like eth's whisper where every node gets every message. It would divulge that X wants to talk to another user since X signed a keypair. However ultimately I will have to use both X25519 and Ed25519, seems like that is inevitable.

While it's technically possible to convert an Ed25519 key to a Curve25519 key, there are a number of security considerations related to using the same key for multiple purposes. So you could create two Ed25519 keys and then convert one of them to Curve25519, but that probably doesn't provide an advantage over creating an Ed25519 key and a Curve25519 key from the start. For this reason, NSec currently doesn't expose a method for converting keys.

Thanks!


Feat/fix table updates

Some fixing/refactoring to use the correct job/query properties.  Also cleaned up a few log statements. (repointed to dev branch for merging)

cherry picked and merged


Rocketchat: irc timeout after receiving x amount of messages from IRC

Describe the bug
RocketChat integration is crashing matterbridge when the RocketChat SDK has processed enough IRC messages. I am now testing whether this is due to RocketChat SDK generating poor random ids for messages with duplicate ids.
Example of a poor random id generated by Matterbridge seen in the direct link to a message:
https://rocketchat.example.com/group/testing?msg=0.609882
Native client generated "true" random id:
https://rocketchat.example.com//group/testing?msg=xffqwMDvqdgpfe38F
Unfortunately this is a vendor run matterbridge instance so I cannot provide any debug output at this time. Maybe later.
Thank you!
To Reproduce

Bridge RocketChat channel to an IRC channel
Send x amount of messages from IRC to RocketChat
Matterbridge crashes (disconnects  from IRC)

Expected behavior
No crashes / disconnects

Disconnect from irc is not a crash though.
Without debug logs I can't really do much, in another issue we traced irc disconnects to a discord bridge getting ratelimited which caused the irc bridge blocking and got a ping timeout because of that. See #1255 and #1300

This is the only log I can observe, and it is apparently a crash of the binary, not a ping timeout. The bot is clearly restarting the connection and joining back:
12:19 -!- rocket<EMAIL_ADDRESS>has quit [Read error: Input/output error]
12:19 -!- rocket<EMAIL_ADDRESS>has joined #channel

Sorry for not providing proper debug logs!

Okay got some logs now, looks to be the same as #992 but we can manage to have the bot connected for 1 - n messages, it is not crashing straight away.
panic: interface conversion: interface {} is nil, not map[string]interface {}
goroutine 141 [running]:
github.com/matterbridge/Rocket.Chat.Go.SDK/realtime.(*Client).SendMessage(0xb029560, 0xaeae1e0, 0x16, 0x0, 0x0)
/go/pkg/mod/github.com/matterbridge/!rocket.!chat.!go.!s!d!k@v0.0.0-20200411204219-d5c18ce75048/realtime/messages.go:44 +0x198
github.com/42wim/matterbridge/bridge/rocketchat.(*Brocketchat).Send(0xb032b40, 0xaf68000, 0x8a, 0xafd16b0, 0x6, 0xae72d00, 0x11, 0xae8e240, 0x25, 0x0, ...)
/go/src/github.com/42wim/matterbridge/bridge/rocketchat/rocketchat.go:176 +0x44c
github.com/42wim/matterbridge/gateway.(*Gateway).SendMessage(0xad53d70, 0xaf89030, 0xb032ac0, 0xad4b1c0, 0x0, 0x0, 0xc, 0xade9520, 0x17, 0xade9520)
/go/src/github.com/42wim/matterbridge/gateway/gateway.go:457 +0x586
github.com/42wim/matterbridge/gateway.(*Gateway).handleMessage(0xad53d70, 0xaf89030, 0xb032ac0, 0x5d9, 0xa782d00, 0x0)
/go/src/github.com/42wim/matterbridge/gateway/handlers.go:226 +0x1e3
github.com/42wim/matterbridge/gateway.(*Router).handleReceive(0xb0329c0)
/go/src/github.com/42wim/matterbridge/gateway/router.go:152 +0x1a1
created by github.com/42wim/matterbridge/gateway.(*Router).Start
/go/src/github.com/42wim/matterbridge/gateway/router.go:102 +0x76f

main process exited, code=exited, status=2/INVALIDARGUMENT

Should be fixed in master (together with your random ID patch).

Yes seems like it is working, thanks!


Community Notes

Describe the Feature
I would like to see community notes on this platform to help prevent the spread of misinformation.
Attachments

Describe Alternatives
No response
Additional Context
The code for this is open source. https://github.com/twitter/communitynotes

would be nice

it will be great to prevent misinformation

Would be great!

I'd love to lend a hand with this, but where would the annotations themselves be hosted? Can you add arbitrary annotations to a post in https://github.com/bluesky-social/atproto ? Then, of course, is the challenge of distributing the model, but maybe a PDS could be a requirement for that.

How should it work given the decentralized nature? The most likely solution is that its managed by its own instance, but some modularity like community notes being able to be handled by other instances or several other instances could have some use (the former more likely)

I'd love to lend a hand with this, but where would the annotations themselves be hosted? Can you add arbitrary annotations to a post in https://github.com/bluesky-social/atproto ?
Then, of course, is the challenge of distributing the model, but maybe a PDS could be a requirement for that.

I think the best way to do it is to have the suggested note live in the PDS of the DID that suggested it. And then whichever one has the most helpful votes gets displayed.

Also the original Community Notes from X can be greatly improved on, by:

separate discussion area
increased weighting for people with good track records (or some other way to get more notes through). Elon Musk doesn't get noted enough at all, for example

I do not think rewarding people would be a good idea, that just incentives people to add notes to every post they find, even if said post is f.e just a joke. I believe that would simply decrease the quality of the notes overall.

I get your concern but disagree. If they're not good notes, they don't get
approved and the note writer won't be rewarded. There should only be a
reward for good, approved notes
On Tue, Nov 19, 2024, 07:26 Mattis Kabella @.***> wrote:

I do not think rewarding people would be a good idea, that just incentives
people to add notes to every post they find, even if said post is f.e just
a joke. I believe that would simply decrease the quality of the notes
overall.
—
Reply to this email directly, view it on GitHub
https://github.com/bluesky-social/social-app/issues/5783#issuecomment-2484804070,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BLISJ4CLRYLWLZURSJIREQT2BLKY3AVCNFSM6AAAAABQCRR2I2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBUHAYDIMBXGA
.
You are receiving this because you commented.Message ID:
@.***>

I get your concern but disagree. If they're not good notes, they don't get approved and the note writer won't be rewarded. There should only be a reward for good, approved notes
…
On Tue, Nov 19, 2024, 07:26 Mattis Kabella @.> wrote: I do not think rewarding people would be a good idea, that just incentives people to add notes to every post they find, even if said post is f.e just a joke. I believe that would simply decrease the quality of the notes overall. — Reply to this email directly, view it on GitHub <#5783 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BLISJ4CLRYLWLZURSJIREQT2BLKY3AVCNFSM6AAAAABQCRR2I2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBUHAYDIMBXGA . You are receiving this because you commented.Message ID: @.>

Yes I think @Siebe-wq is right here. A user's weight would be calculated best on the track record of the notes they submit or vote as helpful. This is how it works on X/Twitter now. Posting arbitrary notes would result in a small or even a negative weight.
A user's weight and their rewards could be two different things. Rewards could be monetary. But you can only gain these rewards after you have gained reputation weight.
There is a great essay on this idea by a16z: https://a16zcrypto.com/posts/article/reputation-based-systems/

I still disagree that it should be monetary, a badge on the account would be cool, but monetary feels wrong to me personally

I still disagree that it should be monetary, a badge on the account would be cool, but monetary feels wrong to me personally

In any case, monetary rewards wouldn't be fundamental to the proposed feature. How could a monetary award even be implemented? A badge might be sufficient incentive for a lot of people. Community Notes contributors on X don't even get that and they still contribute.

The results of the Community Notes process could be published as "misinformation" labels in a moderation service.
In fact the Community Notes algorithm itself, which uses bridging-based ranking to identify notes that people with different points of view find helpful, could be used for other moderation labels.

A "Community Notes" feature, I imagine, would probably be built on (an extension of) the existing Labeling system in the underlying protocol. One such implementation might include:

A field on the labeler service definition indicating that it accepts Community Notes submissions.
A field on certain labels to indicate that that label might be associated with Community Notes; this would signal to the Bluesky app to look them up.
An XRPC endpoint, implemented at the labeler, which returns the Community Note for a given post, which the Bluesky AppView would use when it finds that a post has a Community Notes label.
XRPC endpoints for viewing, proposing edits to, and voting on Community Notes proposals for a post. This may have access controls, and this is where restrictions for who can submit Community Notes to a labeler service would be applied (i.e. verified email, minimum account age, etc).
The Bluesky AppView might have special treatment of the first-party Community Notes labeler, so that it shows i.e. "Readers added context" instead of i.e. "Bluesky Moderation Service added context" (actually, maybe having the contributions-accepted marker would make it say "Readers added context", and just add the service to the bottom of the box)

This would also allow, as suggested by @bou.bsky.social, known fact-checking services such as Snopes to offer a labeler service to give ratings and share the rationale.

An alternative, possibly simpler solution is to allow any third-party labelling service to act as a fact-checking service.
Any user could report content (e.g. a post) to that service, which could in turn label it as true / mostly false / false, and add a rationale of why the reported information is true or false. To do this, the system should allow them to link one of their own public posts or another external source (e.g. Snopes could link an article on their webpage).
Then, subscribers to that service would see something like this:

Do you think this could be enough?

An alternative, possibly simpler solution is to allow any third-party labelling service to act as a fact-checking service.
Any user could report content (e.g. a post) to that service, which could in turn label it as true / mostly false / false, and add a rationale of why the reported information is true or false. To do this, the system should allow them to link one of their own public posts or another external source (e.g. Snopes could link an article on their webpage).
[...]
Do you think this could be enough?

Hmm, I partially like the idea of people being able to add their own fact checkers. However, I also think that this is a really important feature and there shouldn't be a choose-your-own-adventure with this. Community Notes should imo be the default & obligatory viewing, as well as result in algorithmic suppression in all feeds. Once Community Notes is in place, the extra feature of adding extra noting could be added. (Let me know if I have misunderstood you!)

In fact the Community Notes algorithm itself, which uses bridging-based ranking to identify notes that people with different points of view find helpful, could be used for other moderation labels.

I think this is a very good idea!

However, I also think that this is a really important feature and there shouldn't be a choose-your-own-adventure with this. Community Notes should imo be the default & obligatory viewing

I think this is an important point. Community Notes discourages misinformation on X because having a Note attached to a post has real consequences. Everybody who sees the post will see the note. And people seem to have a grudging respect for Community Notes so getting "Community Noted" looks bad.
So to have teeth this feature really needs to be part of the first-party Bluesky Moderation Service, and the Bluesky app needs to show these notes by default.
That doesn't mean third-party labelers couldn't also start including Notes with their labels. But third-parties might make a bigger impact by just submitting notes to the first-party Bluesky Moderation Service. As @elsacodelcoco suggested the note can be a link to a external source such as a Snopes article.

as well as result in algorithmic suppression in all feeds

I think this could be valuable but the rules for algorithmic suppression would need to be considered carefully. Sometimes posts with notes are not false information, they are just misleading or incomplete without additional context. The context in the note can "fix" a misleading post in a sense.
Also algorithmic amplification of corrective information can be as effective as algorithmic suppression of misinformation. So amplifying the note itself (with or without the post) could be valuable. It is especially very valuable to push notes to people that previously like or reposted the post without the context.
It would be fascinating to observe if people that said they found a note helpful are less likely to like or share the post that was "noted". Since it can be hard to measure this effect, I would suggest the Apps can add a prompt for users who liked or reposted the post but subsequently voted the note as "helpful". The App would ask them if they would like to "retract" their post, like, repost, etc, given this new context. The retraction rate could determine the degree of algorithmic suppression.
This information could be rolled into a reliability score for users that could be used by ranking algorithms. Personally, I would prefer my feed exclude posts from users who have a history of submitting posts that get a lot of likes/shares initially but that don't stand up to scrutiny.

the system should allow them to link one of their own public posts or another external source (e.g. Snopes could link an article on their webpage).

Since a post can contain an external link, maybe a proposed note should always be a post. And since a note is always a response to a post, I suggest notes are always replies to the post. There would just need to be a way of designating a reply as a proposed note.
Perhaps this is done using the Reporting feature. Right now com.atproto.moderation.createReport has a reason field. Maybe this could be the DID of a reply?
However, I'm not sure that notes always needs to be associated with a misinformation label. Perhaps there is a "missing context" label for posts that are accurate but potentially misleading without context?
And of course notes could be associated with labels other than "missing context" and "misinformation", such as "misprepresentation", "unsafe link", "scam"...any time someone wants to give a reason for a label.

Or the actual label descriptions of the community note label could just contain the link to the external source, bc I don't think a label should also be a regular post, mby labelers should be changed in such a way where the label descriptions can be dynamically created, instead of being predefined. That'd allow for the label descriptions to be custom to a specific post
https://github.com/bluesky-social/social-app/issues/7016

A lot of good points @johnwarden ! I agree that algorithmic suppression should depend on the reason for the note.
I have my doubts about turning notes into posts. I think for now it would be good to keep things as simple as possible so it gets up and running ASAP.
I really like the idea of retraction but maybe that should first be built out as a separate feature (i.e. strikethrough text and no longer showing to new viewers?)

Idk if this would be to hard to realize on a technical level, or if i should make a seperate Issue due to the complexity, but i think this is what the architecture should be like.

The user would see the "proposed notes" label, click on it, see a window of the proposed note, then log in if they want to vote on or add a new label themselves, this would create an anonymous UUID to associate the BSKY user without publishig which notes the user made. This is done to track the the "quality" of their labels to replicate the ranking mechanism of the Twitter version.
When a bsky user reports a post it creates a new note on behavlf of the user while still linking the note to their UUID since - from my understanding - labelers are able to see who created a certain report so this should be possible to do

Another advantage of this architecture would be that it'd give third party labeling services more freedom, i.e the spotify example that I made
Of course this would require a lot of changes to the current system like the ability to assign specific labels to specific posts (so that the URL matches with the post that the label has been applied to)

~Also some people are so far gone that they dislike anything to do with "misinformation" notices and fact checking.~

The UI needs to use the most neutral language for this reason. No "gotcha" to latch on, if the information in the note itself is enough for people to draw conclusions.


Storage type defaults to shared even on setting to local

By default, the following parameters are passed :
{"GET":{"scheme":"http","host":"<IP_ADDRESS>:8080","filename":"/client/api/","query":{"issystem":"false","name":"test2","displaytext":"test2","provisioningtype":"thin","cachemode":"none","customized":"false","offerha":"false","limitcpuuse":"false","cpunumber":"1","cpuspeed":"200","memory":"200","isvolatile":"false","command":"createServiceOffering","response":"json"},"remote":{"Address":"<IP_ADDRESS>:8080"}}}

storagetype seems to be missing

After the fix:
{"GET":{"scheme":"http","host":"localhost:5050","filename":"/client/api/","query":{"issystem":"false","name":"test3","displaytext":"test3","storagetype":"local","provisioningtype":"thin","cachemode":"none","customized":"false","offerha":"false","limitcpuuse":"false","cpunumber":"2","cpuspeed":"200","memory":"2000","isvolatile":"false","command":"createServiceOffering","response":"json"},"remote":{"Address":"<IP_ADDRESS>:5050"}}}

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/738 (JID-3503)


Ask for permission

What a great work! Thank you very much for sharing. I have already submitted an application for use on HuggingFace. I hope you can approve it . Thank you!

Thanks for your interests in GigaPath. Just approved it!


Error when having more than one menu subitem with the same title

Bug Description
When there is more than one menu subitem with the very same name, it doesn't matter what subitem you click, slate will always pick the first item of the same name in the menu.
For example, https://docs.smoobu.com/, you will see most of the menu items have a "HTTP Request" subitem.
You can pick anyone in the list, slate will redirect to the first one every time.
Screenshots
N/A
Browser (please complete the following information):

OS: Any
Browser Any
Version N/A

Last upstream Slate commit (run git log --author="Robert Lord" | head -n 1):
commit a4854b03a4e4df8d016c2144fbcc831af0b6b8eb

The fix works for multiple headers in the same file that have the same name - but if you have multiple include files that have, for example, a 'Request' and 'Response' section, the calculation for unique headers gets wonky.  Each file begins numbering #request, #request-1, #request-2, etc.  So in my fourth file in my Includes, the first Request will link to the first Request entry from the first file, etc.

The fix works for multiple headers in the same file that have the same name - but if you have multiple include files that have, for example, a 'Request' and 'Response' section, the calculation for unique headers gets wonky.  Each file begins numbering #request, #request-1, #request-2, etc.  So in my fourth file in my Includes, the first Request will link to the first Request entry from the first file, etc.


Can't RMB Pull During Antigrav

Description

During the no gravity event you can pull stuff behind you, but you can't RMB + ctrl pull it.
Reproduction

Get no gravity.
RMB + Ctrl click
No pulling

has to do with you not having magboots active to "anchor" you to the ground, was pretty sure this has always been the case and is working as intended but i'm also not the one who coded it
i do know if you have magboots active you can RMB + Ctrl click and it'll aggressively reposition the item

I am pretty sure you should be able to move the item relative to you even if you don't have an "anchor".


Use Binding to manage color values

Instead of accepting two parameters in the ColorWellView initializer, color, and action, you could use a single Binding parameter. This is cleaner and what the rest of the SwiftUI interface uses.

This is in the process of happening. I've actually been reworking a lot of the API for both the Cocoa and SwiftUI views. I may end up archiving this repo and starting fresh once I get it in a state that I'm happy with.

You also have way more initializers than you need. Just make the parameters optional and/or add default values and you get the same interface for much less code.

The new repository is now available at https://github.com/jordanbaird/ColorWellKit, with a number of changes to the SwiftUI interface to more closely align with SwiftUI's conventions. Closing this now, but if there are still problems, feel free to open a new issue there.


Crew-1 (#107) patch image size error?

links > patch > small and links > patch > large are both returning an image of size 6126x5692.

Yeah that's massive 😆
Replaced with an appropriately sized one

Thank you! 😁


Feature request: Skip copy protection/exe scanning

Is your feature request related to a problem? Please describe.
Unnecessary scans of exe files when title is known to have no copy protection (mostly applicable to standard application discs)
Describe the solution you'd like
A flag to skip exe scanning or scanning for copy protection at all.

Just do not use /ns and /sf and then DIC will not scan exe files

Thank you, I was not using /sf, but I was using /ns and it was not clear from the included help that it would scan exe files.


Fixed PHPDoc of Themes and added Shopware File Header

Description
Please describe your pull request:

Why is it necessary?
Why not :D
What does it improve?
Fixes phpdoc
Does it have side effects?
Nope

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?

How to test?

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-17858.
Please use this issue to track the state of your pull request.


Restore Readme for the recipes

With the latest changes on the recipes, the readme are not providing the same level of details.
Now: https://github.com/redhat-et/locallm/blob/main/summarizer-langchain/README.md
Before: https://github.com/redhat-et/locallm/blob/5ff3767ff3e265a917a4f604211557d2511d41b4/summarizer/README.md
We would like to get back to the previous version of the content, aligned with the latest changes.

@slemeur I've updated the relevant README's and have closed this issue.


npm ci command fails with version 18.2.8

Command
other
Is this a regression?

[x] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
18.2.7
Description
Using command npm ci on a clean Angualr 18.2.8 project, the command fails with an error.
Minimal Reproduction

Run npm new angular182. Select SCSS and no SSR.
cd angular182
run npm ci

Exception or Error
npm error code EUSAGE
npm error
npm error `npm ci` can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with `npm install` before continuing.
npm error
npm error Invalid: lock file's<EMAIL_ADDRESS>does not satisfy<EMAIL_ADDRESS>npm error Missing<EMAIL_ADDRESS>from lock file
npm error Missing<EMAIL_ADDRESS>from lock file
npm error Invalid: lock file's<EMAIL_ADDRESS>does not satisfy<EMAIL_ADDRESS>npm error Missing<EMAIL_ADDRESS>from lock file
npm error Missing<EMAIL_ADDRESS>from lock file

Your Environment
Angular CLI: 18.2.8
Node: 20.18.0
Package Manager: npm 10.9.0
OS: win32 x64

Angular: 18.2.8
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1802.8
@angular-devkit/build-angular   18.2.8
@angular-devkit/core            18.2.8
@angular-devkit/schematics      18.2.8
@schematics/angular             18.2.8
rxjs                            7.8.1
typescript                      5.5.4
zone.js                         0.14.10

Anything else relevant?
Same issue occurs after regenerating package-lock.json.

delete package-lock.json
delete node_modules directory
run npm install - regenerates package-lock.json and node_modules
run npm ci

Looks like you are experiencing one of the npm ci bugs listed in: https://github.com/npm/cli/issues/6787

We ran into the same issue trying to upgrade our lock files with renovate.
Manual intervention with an npm install resolved it temporarily. But every time we run npm update again, it creates an invalid package-lock file.
It seems there's a certain constellation of dependencies at the moment that makes npm update fail reliably on chokidar.

This is, indeed, an npm issue, but if you got here and are looking for a workaround because npm ci is broken and complaining about chokidar and other invalid dedups, the simplest fix for us was to regenerate our package-lock.json files by running npm install twice in a row.
So, to get a fresh package-lock.json we will run:
rm -rf node_modules/; rm package-lock.json; npm install; npm install

Overriding the chokidar version in  package.json looks to be a good workaround:
"overrides": {
  "chokidar": "4.0.1"
}

This is, indeed, an npm issue, but if you got here and are looking for a workaround because npm ci is broken and complaining about chokidar and other invalid dedups, the simplest fix for us was to regenerate our package-lock.json files by running npm install twice in a row.
So, to get a fresh package-lock.json we will run:
rm -rf node_modules/; rm package-lock.json; npm install; npm install

This weird thing works! Many thanks for sharing. Any reasoning behind twice in a row execution?

This weird thing works! Many thanks for sharing. Any reasoning behind twice in a row execution?

Pure speculation on my part (based on reading the comments in the npm issue linked above and my own observations), but for whatever confluence of reasons, when running npm install in this situation once, npm doesn't fully resolve/install the dependencies requested by all packages.
For example a common situation is like:

package a depends on package<EMAIL_ADDRESS>package b depends on package<EMAIL_ADDRESS>
When npm install runs, it will install both versions of package c (at least one of which will end up under a package's own install such as node_modules/a/node_modules/c.
Well, again, for some reason in this situation with Angular and chokidar, both versions of the "same package" (chokidar) needed (3.6.0 and 4.0.1) are not getting installed satisfactorily for every library.  In my specific testing, I found that on the first run of npm install, chokidar 4.0.1 was installed in the "main" node_modules directory and some but not all packages that needed an older version had a local install of 3.6.0.  This would fail npm ci.  On a subsequent run of npm install, I found that the "main" directory version of chokidar changed to 3.6.0, and 4.0.1 was locally installed to the few places that needed it, such as @angular/compiler-cli.  At this point npm ci succeeds.
Why? Not sure, but it's an npm bug.  😄

So, to get a fresh package-lock.json we will run:
rm -rf node_modules/; rm package-lock.json; npm install; npm install

This weird thing works! Many thanks for sharing. Any reasoning behind twice in a row execution?

This also worked for me. I could not imagine I needed to run npm install; npm install twice in order to get this working. Thanks for sharing.


Deployment documentation：8.2.4 During script execution，install nfs-utils error [REPLACEMENT ISSUE]

The original issue
Id: 1186
Title: Deployment documentation：8.2.4 During script execution，install nfs-utils error

could not be created.
This is a dummy issue, replacing the original one. It contains everything but the original issue description. In case the gitlab repository is still existing, visit the following link to show the original issue:
TODO

In GitLab by @sokril on Nov 8, 2019, 08:20
closed

In GitLab by @sokril on Nov 8, 2019, 08:20
assigned to @cuiaq123


(BKR-1534) permit user environment on osx-10.14

👍

retest this please

re-kicking since last test results were purged over the holiday break.
retest this please

retest this please


Feature/add proxy tests

Codecov Report

Merging #24 (6dc23bf) into master (10cad90) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #24   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           66        68    +2
  Lines          905       928   +23
=========================================
+ Hits           905       928   +23

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

stweet/exceptions/__init__.py
100.00% <100.00%> (ø)

...eet/exceptions/invalid_requests_param_exception.py
100.00% <100.00%> (ø)

stweet/http_request/proxy_client_requests.py
100.00% <100.00%> (ø)

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 10cad90...6dc23bf. Read the comment docs.

I have added tests with docker-compose proxy. Additionally I remove option that user can put none in proxy -- when is no proxy user could use basic web client. All options should be listed in constructor params. Putting dict as params make user interface harder to understand. This is what I hate in Python -- many difficult to understand function params.

Of course -- I will add every param which users need.


macOS Monterey compatibility

Since PHP is removed this workflow will not work by default in macOS Monterey

@vmitchell85 I have a draft PR open in my repo, should have it updated by tomorrow: https://github.com/clnt/alfred-tailwindcss-docs/pull/10

This pattern seems to do the trick (and matches what @clnt has done): https://github.com/tillkruss/alfred-laravel-docs/pull/22


[Bug] autoskin failures lock renderer queue

Describe the bug
If any bug occurs during autoskin, e.g #863, then any renderer reservation is locked and the renderer cannot be used.
Steps to reproduce

Recreate any autoskin bug, e.g #863
Try an autoskin command with valid parameters
Observe "renderer queue full" error

Message link
https://discord.com/channels/226528541368385536/456218562428796929/1033359886454636564
Suggested Solution
I'd like to make a RendererReservation class that I can with:

__enter__: create a renderer reservation
__exit__: catch any exception that occurred release the renderer reservation, throw the exception again. I'm not sure whether the catch/throw is really needed, does with do this for you??

(optional) Your discord tag
Cryedra#2545
Suggested solution from Trimatix

fixed by with RendererReservation


Add functionality for students to 'draw' line

This PR adds functionality for the student to 'draw' a best-fit line on screen 3 using their mouse. This works as follows:

When the student first clicks the "Draw Fit Line" button, a listener is added to the scatter plot. This will follow the student's mouse inside the plot and create a line connecting their mouse location to the origin as they move.
When the student clicks, this functionality is turned off, and their line stays in place. This click also deselects the "Draw Fit Line" button.
When the student next clicks "Draw Fit Line", a circular point is added to the end of their line. If the student drags this point, they can adjust their line again. Until they do, their previous line is unaffected.

I think this is pretty close to what we had discussed the other day, but I can make changes to this if we think that this user experience isn't straightforward enough.

I'm merging this in, so we can have the basic functionality as we continue to develop the student experience, but we still need to adjust the ux so the user doesn't have to click back on "draw a fit line" button to grab & move the line end point.


BUG: brief description of the bug

Sealos Version
v4.1.3
How to reproduce the bug?
"500:rolebindings.rbac.authorization.k8s.io not found" is displayed when the database is deleted
What is the expected behavior?
ignore the not found error
What do you see instead?
nothing
Operating environment
- Sealos version:v4.1.3
- Docker version:
- Kubernetes version:
- Operating system:
- Runtime environment:
- Cluster size:
- Additional information:

Additional information
nothing

What database? So how to reproduce this issue, provide some details.


dechex may return incorrect ecid

Below is dechex description on php.net (http://php.net/manual/en/function.dechex.php)
string dechex ( int $number )
Returns a string containing a hexadecimal representation of the given unsigned number argument.
The largest number that can be converted is PHP_INT_MAX * 2 + 1 (or -1): on 32-bit platforms, this will be<PHONE_NUMBER> in decimal, which results in dechex() returning ffffffff.

Thanks!


Polymer.dart: The drawer in the core-header-panel cannot be closed on iPhone (Safari/Chrome)

Originally opened as dart-lang/sdk#20644
This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

The source code is at https://github.com/andersforsell/chessboard.dart/tree/74612d7a84931ad32e2870afffe7e03f1ec50aa3
Browse to http://andersforsell.github.io/chessboard.dart/example/app.html from an iPhone
Tap the top-right button to open the drawer
Try to close the drawer by tapping to the left of the drawer

What is the expected output? What do you see instead?
When running the example on a desktop browser the drawer closes when I click outside the drawer.
What version of the product are you using?
Dart Editor version 1.6.0.dev_09_05 (DEV)
Dart SDK version 1.6.0-dev.9.5
On what operating system?
Mac OS X 10.9.4
What browser (if applicable)?
Chrome 36.0.1985.57 on iPhone
Please provide any additional information below.

 Comment by jakemac53

Added Done label.

 Comment by jakemac53

Can you try adding 'touch-action="auto"' to your body tag? I am pretty sure this is related to https://github.com/Polymer/core-drawer-panel/issues/3.
Also, I believe that in the next version of polymer this will no longer be required, but the dart version of that won't be out until next week sometime as we are currently working through some IE 10/11 issues that came up.

Added Waiting label.

 Comment by dgrove

This should probably be filed on https://github.com/dart-lang/core-elements .

Set owner to @jakemac53.
Added Pkg-Polymer, Triaged labels.

This comment was originally written by<EMAIL_ADDRESS>
Yes, adding the 'touch-action=“auto”’ resolved the issue.
Thanks!


Running azurite with a custom host name

Which service(blob, file, queue, table) does this issue concern?
blob
Which version of the Azurite was used?
latest docker image
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
dockerhub
What problem was encountered?
I'm trying to run azurite under a different host name, as required for docker-compose networking, per this question:
https://stackoverflow.com/questions/59615330/connecting-to-azurite-using-a-hostname-fails
I'm trying to emulate Azure Blob Storage in docker using Azurite for integration tests.
All works well, to the point I have to access Azurite via a hostname (which is AFAIK required for docker networking)
My connection string looks like this (which is the default well-known connection string):
"AccountName=devstoreaccount1;AccountKey=Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==;DefaultEndpointsProtocol=http;BlobEndpoint=http://azurite:10000/devstoreaccount1;"
My docker compose part for azurite looks like this:
services:
  azurite:
    image: mcr.microsoft.com/azure-storage/azurite
    hostname: azurite
    command: "azurite-blob --loose --blobHost <IP_ADDRESS>"
    ports:
      - "10000:10000"
    volumes:
      - ./test/azurite:/data
    networks:
      - stillsnet

my code looks like this:
private const string ConnectionString ="AccountName=devstoreaccount1;AccountKey=Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==;DefaultEndpointsProtocol=http;BlobEndpoint=http://azurite:10000/devstoreaccount1;";

[Fact]
public async Task UploadFile()
{
   var container = new BlobContainerClient(ConnectionString, "images");
   await using var stream = File.OpenRead(@"C:\temp\output\3ee9bc41-40ea-4d05-b180-e74bd5065622\images\00000000.jpg");
   await container.UploadBlobAsync("test.jpg", stream);
}

this will throw an exception:
System.Xml.XmlException : Root element is missing.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.ParseDocumentContent()
   at System.Xml.XmlTextReaderImpl.Read()
   at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
   at System.Xml.Linq.XDocument.Load(Stream stream, LoadOptions options)
   at Azure.Storage.Blobs.BlobRestClient.Container.CreateAsync_CreateResponse(Response response)
   at Azure.Storage.Blobs.BlobRestClient.Container.CreateAsync(ClientDiagnostics clientDiagnostics, HttpPipeline pipeline, Uri resourceUri, PublicAccessType access, Nullable`1 timeout, IDictionary`2 metadata, String requestId, Boolean async, String operationName, CancellationToken cancellationToken)
   at Azure.Storage.Blobs.BlobContainerClient.CreateInternal(PublicAccessType publicAccessType, IDictionary`2 metadata, Boolean async, CancellationToken cancellationToken, String operationName)
   at Azure.Storage.Blobs.BlobContainerClient.CreateIfNotExistsInternal(PublicAccessType publicAccessType, IDictionary`2 metadata, Boolean async, CancellationToken cancellationToken)
   at Azure.Storage.Blobs.BlobContainerClient.CreateIfNotExistsAsync(PublicAccessType publicAccessType, IDictionary`2 metadata, CancellationToken cancellationToken)
If I change the connection string from azurite to <IP_ADDRESS> it all works fine.

Steps to reproduce the issue?
https://stackoverflow.com/questions/59615330/connecting-to-azurite-using-a-hostname-fails
Have you found a mitigation/solution?
no

In case of this error, I would at least expect a response that can be handled by the Azure Blob storage client instead of an empty response. Is there any workaround for this problem, or is it by design?

Hi @zidad, the error stack trace is from .net SDK, not sure it's a SDK issue or Azurite issue. Curious about   why there is an XmlException parsing issue when creating container operation. Can you enable Azurite debug.log and share here? I need to check what's the response body for Azurite, then we can check why .net SDK throws the exception.

@XiaoningLiu that's a good pointer, thanks!
It looks indeed like the BlobContainerClient itself strips away the account name from the URL if you use anything else than localhost or an IP address, so the client generates this:
"PUT /images?restype=container HTTP/1.1" 400 -
instead of
"PUT /devstoreaccount1/images?restype=container" HTTP/1.1 201
As a really ugly workaround we can include the account name in the container name when testing against azurite:
var container = new BlobContainerClient(ConnectionString, "devstoreaccount1/images");
Though await container.CreateIfNotExistsAsync(); doesn't work properly against azurite then (throws a 409 Exception when it already exist...
So we either have an ugly hack, dropping azurite in favor of a real blob storage account, or dropping the Azure Blob client 👎

@XiaoningLiu I was thinking, if azurite could be started with something like  "--DefaultAccountName=devstoreaccount1" and it would accept URL's without requiring the account name to be in there, might solve the issue?
I can't imagine I'm the only one running into this issue, and I'd expect docker-compose for spinning up test dependencies will become a popular way of integration testing?

Hi @zidad You can open an issue to storage .net SDK and see if they can improve the support for single word domain?
Single Azurite process supports multi storage account, so it's sounds not a good choice for Azurite fixing into one default storage account.

"Azurite fixing into one default storage account." that could be optional based on the parameter?
I could argue that the SDK assumes a specific URL structure, and Azurite doesn't follow that structure, hence it can be considered a problem of the emulator.
Is there any other way to determine the account name for azurite? Isn't it passed in the headers, or a claim?

Hi @zidad Azurite uses URL pattern http://IP:Port/Account which is well accpeted by Azure Storage, and should be supported by all storage SDKs.

Hmm right, then it does seem like an issue with the client SDK... I'll file an issue

Same or similar problem seems to be in the Java SDK. Using IP Address as host works, using a (docker) hostname does not.

@nilshartmann  Have you figured it out?
Still facing the same problems with Java

@iryaniv It should be fixed for the Java client as well, there's a Java answer here too: https://stackoverflow.com/questions/59615330/connecting-to-azurite-using-a-hostname-fails/59651621#59651621


Remove designer attribute from SqlCommand and SqlDataAdapter

Fixes #1110

For some reason they are also in netcore ref class, might as well clean that up.

Is there a VS issue opened to track adding this library to the types VS knows about?

There isn't any, I was thinking the same. I might get some conversations started internally, but it doesn't seem they would be replacing existing classes anyways. Most likely new set of classes would be available in future VS - then we'll include those here.


Remove dead FHT code.

Alternatively, we can invoke print_fht() here.

Alternatively, we can invoke print_fht() here.

This seems valuable for debugging

Alternatively, we can invoke print_fht() here.

This seems valuable for debugging

Yeah, can we keep that print_fht() function around (add the #[allow(dead_code)] attribute on it if necessary)?

Have amended this PR to leave print_fht() in place. To invoke it from FMC will require that we increase FMC's size from 16 to 17k.


Support handling of inputs provided via CLI

Prerequisites
Put an x into the box that applies:

[ ] This issue describes a bug.
[x] This issue describes a feature request.

For more information, see the Contributing Guidelines.
Description
We want to integrate a clean handling of arbitrary data provided to the application through the CLI.

Depends on #82.


v1.2: bump timeout on K8s upstream tests to 40 minutes

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-upstream-k8s

test-upstream-k8s

Prior build had 'Build VMs' stage take 32 mins, showing that this is the right fix:
https://jenkins.cilium.io/job/Cilium-PR-Kubernetes-Upstream/782/

This change is self-evident; the tests pass and the change is in the Jenkinsfile. Merging.


Add TargetAccount field to "Multiple Password resets by User"

I think it would be a good option to add the "TargetAccount" field to the query, in order to see which account the password was tried to change at a glance.
https://github.com/Azure/Azure-Sentinel/blob/master/Detections/MultipleDataSources/MultiplePasswordresetsbyUser.yaml
https://github.com/Azure/Azure-Sentinel/pull/2665

https://github.com/Azure/Azure-Sentinel/pull/2665

Thank you for submitting issue with PR. I have suggested a change, rest looks good. once changes done and PR is merged, this issue should get closed automatically.


Update list of iOS defaults UserDefaults keys and prefixes

Based on what the UserDefaults dictionnary returns on iOS 17.5 and iOS 18, both merged together.

Hello @memfrag, can you please give a quick look at this PR.
Thanks in advance.

Oops missed this. Looks good.


CODEOWNERS: add owner for /dts/bindings/

@galak is now the proud owner of /dts/bindings/.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7061c03). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #14982   +/-   ##
=========================================
  Coverage          ?   52.93%
=========================================
  Files             ?      309
  Lines             ?    45251
  Branches          ?    10447
=========================================
  Hits              ?    23953
  Misses            ?    16533
  Partials          ?     4765

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 7061c03...b6bac55. Read the comment docs.


Add autoCutoverConfiguration to local storage

This PR fixes adds autoCutoverConfiguration to local storage.
This fixes inconsistency of show/hide on cutover button and shows migration mode even during creating state.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.404%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26396

Relevant Lines:
57839

💛  - Coveralls


flip help screen design improvements

Ok moved it all up to the top, let me know what you think!

cc @malgorithms

@adamjspooner any thoughts on this? It looks pretty good to me.

One caveat is the blog link is below the fold, @malgorithms might want it to be viewable with no scroll.

@mmaxim, should we send the title to the frontend (instead of hard-coding it)?

Sure