Godot fails to execute expression with class name inside

Godot version:
3.4.beta.custom_build. 74174676b
OS
Ubuntu 21.04 - Ubuntu, Gnome shell 3.38 X11
Issue description:
When executing first line, everything works fine,
{}.get(Particles.new())

var expr = Expression.new()
if expr.parse("{}.get(Particles.new())") != OK:
	print(expr.get_error_text())
	assert(false)
expr.execute([],self)
assert(!expr.has_execute_failed())

but when running same code inside expression class, then I got this errors:
ERROR: execute: Invalid named index 'Particles' for base type Object
   At: core/math/expression.cpp:2204.

I think there is something wrong with the test code.
As mentioned in the doc, only "arithmetic operation, built-in math function call, method call of a passed instance, or built-in type construction call" are supported by Expression.
Particles is not a built-in type, so one have to pass it in:
var expr = Expression.new()
var bindings = {'Particles': Particles}
if expr.parse('{}.get(Particles.new())', bindings.keys()) != OK:
    print("Parse Failed: ", expr.get_error_text())
else:
    expr.execute(bindings.values())
    print("Execute Success? ", not expr.has_execute_failed())


Const snippet

define('constname', 'value') is very PHP-ish.
Added a const type name = value; snippet.

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


Component is not accessible.

Toggle or Menu components of this dropdown are not accessible from the keyboard.

+1 Actually I'm glad we found this, because with the inability to disable the dropdown this makes this unusable for us.


fix: remove profile settings and active learning

Remove all unimplemented icons from view. This fix keeps any functionality required to add these features in the future.

Before:

After:


[ROCM] Fix hipsolverSsyevd tests due to align with the rocm behavior.

@superbobry could you please take a look? Thanks!


Remove telemetry for MSAL CPP to reduce the binary size

Proposed changes
Describe what this PR is trying to do.
Type of change

[ ] Feature work
[ ] Bug fix
[ ] Documentation
[x] Engineering change
[ ] Test
[ ] Logging/Telemetry

Risk

[ ] High – Errors could cause MAJOR regression of many scenarios. (Example: new large features or high level infrastructure changes)
[ ] Medium – Errors could cause regression of 1 or more scenarios. (Example: somewhat complex bug fixes, small new features)
[x] Small – No issues are expected. (Example: Very small bug fixes, string changes, or configuration settings changes)

Additional information
MSAL CPP has its own telemetry code, so we'd like to remove the objc telemetry implementation to reduce the binary size and unblock Office folks.
PBI: https://identitydivision.visualstudio.com/Engineering/_workitems/edit/1766424

This looks good to me as a short-term solution (in case if this change needs ASAP). As a long term solution I would prefer to split this logic into modules/separate classes so we don't need EXCLUDE_FROM_MSALCPP macro. For example:

Split MSIDSilentTokenRequest into 2 classes (parent with shared logic and child that won't be included into MSAL CPP)
Introduce a telemetry manager that acts as a wrapper around current logic. Internally It will have a dependency on the "telemetry" module. In MSAL CPP this module can be disabled.

Thanks, @antrix1989
Excluding the objc telemetry is kind of urgent since it is blocking Office from adopting OneAuth/MSAL. Will consider your suggestion in the future, thanks!

@kaisong1990 , can you take a look at this PR?
@oldalton , I believe I have resolved all the comments, can you approve it? Thanks!

Thanks @oldalton @kaisong1990 for the review. I verified with the changes in this PR, MSAL CPP and OneAuth pipelines will all pass. Part of the OneAuth manually tests:
iOS: PkeyAuth with broker flow, CBA, PoP with legacy broker flow, Migrate from Legacy auth stack to OneAuth, TrueMAM  (Fresh Broker flow
macOS: SSO Extension, PkeyAuth
will pass.
I didn't run all the manual tests since some of them are quite time consuming like I need to upgrade iOS and install the newest xcode.


How to flush?

How do I flush the database so that the shm/wal files are removed and the changes committed to the database .db file? I found a non react-native thread on SO that mentions closing the cursor but the "cursor" terminology is missing here. What is the equivalent?
Thanks.

This repo is for reporting issues not for questions. Please post your questions on Stack Overflow. Thanks

Thanks. Already posted on SO but I doubt if I will get an answer.
The main issue is that even if I do a db.close(), the database is not flushed out so if I make a copy to external storage after doing a db.close(), the database remains empty even if records are inserted. db.close() does not seem to flush out the changes. Logically, the transaction itself should close cursor and changes committed but this is not happening.
So the end result is that while the App can see all the latest changes, the DB file does not have all the changes to make a copy.

Please feel free to ask here:
https://github.com/storesafe/cordova-sqlite-storage-help
I am supporting cordova-sqlite-storage that this package was originally
based on.

Please feel free to ask here: https://github.com/storesafe/cordova-sqlite-storage-help I am supporting cordova-sqlite-storage that this package was originally based on.

Thanks. Will do.


disable identity references

one node, one object

iFit applies matlab-ish copy-by-value to assignments, this can be used to create a new object easily.


Allow filename dates to be followed by either an underscore or hyphen

Description
Many (most?) static site generators use a filename format where any date time is separated from the post slug using a hyphen.
Jekyll:

To create a post, add a file to your _posts directory with the following format:
YEAR-MONTH-DAY-title.MARKUP

Hexo:

By default, Hexo uses the post title as its filename. You can edit the new_post_name setting in _config.yml to change the default filename. For example, :year-:month-:day-:title.md will prefix filenames with the post creation date.

Eleventy:

If a date key is omitted from the file, we then look for a YYYY-MM-DD format anywhere in the file path (even folders).

(I can’t find a specific documented example of the full format but in my own testing, I note that using an underscore between the date and slug doesn’t work, and Eleventy will assume that the date is part of the slug.)
This means that, if you want to port an existing site over to Lume, you will need to update post filenames, which is one more thing to think about.
By parsing filenames using an explicit regex that matches the documented formats (dates yyyy-mm-dd or datetimes: yyyy-mm-dd-hh-ii-ss), dates can be followed by either an underscore (_) or a hyphen (-).
Check List

[x] Have you read the
CODE OF CONDUCT
[x] Have you read the document
CONTRIBUTING

[x] One pull request per feature. If you want to do more than one thing,
send multiple pull request.
[x] Write tests.
[x] Run deno fmt to fix the code format before commit.
[x] Document any change in the CHANGELOG.md.

Hmmm, actually looking at the diff, any time is not included in a post’s date (I misread my local diff, I thought it was improving this and adding the time). May need to tweak this some more…

Okay, so have fixed up the regex to not break the tests. Though, thinking about this some more, I’m guessing the original regex is inexact for speed? Will a stricter regex here impact overall build times?

Initially, Lume supports not only dates but order numbers, so you could name your files as 1_first-article.md, 2_second-article.md.
I decided to use underscore to distinguish between the date/order stored in the filename and numbers that can be in the slug (for example 10-rules-of-programming.md outputs /10-rules-of-programing/ but 10_rules-of-programming.md outputs /rules-of-programing/).
Ordering is no longer supported, so I think your change make sense. It's unlikely that the slug starts with something that can be intepreted as a date.
About the regex, probably you're right. In fact, the createDate function used to create the Date object has another regular expression. Perhaps we only need one regex, that can be used to both detect is the date exists and parse it. Something like:
const match = this.src.slug.match(regex);

if (match) {
  const [, year, month, day, hour, minute, second, slug] = match;

  const date = createDate(year, month, day, hour, minute, second);
  this.src.slug = slug
  this.baseData.date = date;
}

What do you think? Do you mind to change that?

Ah, interesting, wasn’t aware of the ordering background.
I agree, 2 very similar regexes seems wasteful and expensive – that there were 2 is what had me concerned about performance. Consolidating into 1 and providing values directly to the createDate function makes a lot of sense.
Will update this PR with such a change. 👍

Have update the implementation so that:

only one regex is needed
uses named capture groups to help parse the regex
uses types for values sent to createDate function

Also:

updates one of the file fixtures so that it includes the date separated by a hyphen
updates resulting snapshots

Awesome. Thanks!


git-cola: 3.9 -> 3.10.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for git-cola is: "A sleek and powerful Git GUI"
meta.homepage for git-cola is: "https://github.com/git-cola/git-cola"
meta.changelog for git-cola is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin/git-cola passed the binary check.

/nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin/git-cola-sequence-editor passed the binary check.

/nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin/git-dag passed the binary check.

/nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin/.git-cola-wrapped passed the binary check.

/nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin/.git-cola-sequence-editor-wrapped passed the binary check.

/nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin/.git-dag-wrapped passed the binary check.

/nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin/.git-cola-wrapped_ passed the binary check.

/nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin/.git-dag-wrapped_ passed the binary check.

8 of 8 passed binary check by having a zero exit code.

6 of 8 passed binary check by having the new version present in output.

found 3.10.1 with grep in /nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1

found 3.10.1 in filename of file in /nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1

directory tree listing: https://gist.github.com/c0e3258cc32bb1ba854ba5ae2fc83e36

du listing: https://gist.github.com/3199198209cad4b6f98c476cf50e256a

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
git-cola

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A git-cola https://github.com/r-ryantm/nixpkgs/archive/ad28b7f10654ca2804270781e2b879e31b79b572.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1
ls -la /nix/store/0rhnab7h02fjzcj9aw94g5lz0k892rax-git-cola-3.10.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

git-cola

Maintainer pings
cc @bobvanderlinden for testing.

Works on my machine :+1:


Branch ug update

add-leave, delete, find, find-all-tag, find-some-tag

What i actually did are these two commits, the rest are just sync things from other branches. Update UserGuide for add-leave, find, find-all-tag, find-some-tag and…
5a02463
@jean-cq
Amend the error message grammar error"


cant find .ts extension

Extension version: Typescript Version (4.8.4)
VSCode Version: 1.73.0
OS: Windows 10

Steps to Reproduce:

I installed typescript by typing [ npm install typescript ]

After Installation I tried to open up the .ts extension file to edit but it didn't open. Instead, it starts searching maybe with that blue bar under the editor, and yes ... at the right bottom corner vs code didn't show up typescript compiler also.

I have some screenshots so you can understand what am I saying (if this helps)
(searching one under .ts file extension)

(and this while it couldn't find the .ts extension)

Apologies one more ...
where it doesn't even recognise typescript language

@AshainX would you prefer this was filed with VS Code in general? I don't see the connection to the GitHub pull request extension.


Add cover letter, resume and EngSci letter

Hello! My name is Micol Altomare, and I'm a 2nd year Engineering Science student majoring in Machine Intelligence at the University of Toronto. In this pull request, I have added files in support of my application to the #12, #14, #15, #24, #26, #27, #36 internship positions.

Hi @micol-altomare,
Thank you for your interest! Unfortunately I need to close this issue as invalid. Please follow the instructions at https://github.com/MLDSAI/puterbot#instructions and try again.
Thank you! 🙏

@micol-altomare thank you for your interest! Please grab some time on my calendar at your earliest convenience on Weds/Thurs this week: https://www.getclockwise.com/c/richard-abrich/quick-meeting


Text error when doing nvm install iojs

######################################################################## 100.0%
WARNING: checksums are currently disabled for io.js
manpath: can't set the locale; make sure $LC_* and $LANG are correct
Now using node iojs-v1.0.3

May wanna change it to Now using iojs v1.0.3 or something.

Thanks, fixed


[App Service] az staticwebapp create and az staticwebapp update: Add Dedicated as supported SKU

Related command

az staticwebapp create
az staticwebapp update
Description

For Static Web Apps, we have recently previewed the Dedicated SKU. Currently, the related staticwebapp commands only support Free and Standard SKU. This PR adds support for the Dedicated SKU, as well as adds it to supported features in help texts.
Testing Guide

Added and ran a new unit test test_create_staticapp_with_dedicated_sku.
Also ran the following command (with actual values in place of sample values) with this code and verified a Dedicated SKU Static Web App was successfully created:
az staticwebapp create -n name1 -g rg1 -s https://github.com/Contoso/My-First-Static-App -b dev -l EastUs2 --app-location app --sku Dedicated
History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

App Service

Hi @jsntcy @zhoxing-ms @yanzhudd could I please get a review on this change and your help getting it merged? Thank you!

Hi @jsntcy @zhoxing-ms @yanzhudd, any updates here? Could you please help get this PR merged?


BUG: to_graph crashes when  'oneway' is not specified

While collecting my network windows, I got the following error on the 22nd grid that caused the script to crash.
  File "G:/My Drive/Codebase/ExtractingOSMData_v02.py", line 252, in getOSMnetwork
    thisWalkingGraph = thisAreaData.to_graph(nodes, edges, graph_type="networkx", osmnx_compatible=False, retain_all=True)

  File "C:\WinPython\python-3.6.5.amd64\lib\site-packages\pyrosm\pyrosm.py", line 756, in to_graph
    network_type, retain_all, osmnx_compatible)

  File "C:\WinPython\python-3.6.5.amd64\lib\site-packages\pyrosm\graphs.py", line 145, in to_networkx
    network_type)

  File "C:\WinPython\python-3.6.5.amd64\lib\site-packages\pyrosm\graphs.py", line 26, in get_directed_edges
    col=col)

ValueError: Required column 'oneway' does not exist in edges.

It is not hard to imagine that lots of roads fail to specify whether it is one-way or not, especially for pedestrian or footpath edges.
I think that, in these cases, it would be better to use a default of oneway=false instead of crashing.
Similar reasonable defaults could/should be done for other required variables.
And maybe it could throw a warning instead of an error...although that may explode my terminal, so please make that optional.
(i.e., I want to know if this is happening when testing, but if I decide I'm okay with it, I don't need to see thousands of warning messages in my console feed.)
...or maybe I misunderstand the meaning of this error.

@bramson Thanks for reporting this 🙏
Hmm, yes I believe this is a situation where indeed none of the ways in your data has a tag "oneway". Didn't think that this might happen but surely it does, as shown here.
I think the solution could indeed be to assume that all roads are bidirectional in such case, and raise a warning. 👍 Thanks! Will fix this asap.

As a quick fix, you can add the column "oneway" to edges with value None if the column does not exist, before using the to_graph() function.

@bramson Okay, now this should be fixed in the master. Thanks for pointing this out! 👍


"Allow reject empty" configuration option for RSpec/CompactBlank

Is your feature request related to a problem? Please describe.
The new RSpec/CompactBlank cop will replace reject(&:empty?) with compact_blank but it is not a straight swap;
['hi', 'there', '', ' ', 'hello'].reject(&:empty?)
# => ["hi", "there", " ", "hello"]
['hi', 'there', '', ' ', 'hello'].compact_blank
# => ["hi", "there", "hello"]

Describe the solution you'd like
Would it be possible to have a configuration option so that reject(&:empty?) is permitted while reject(&:blank?) is flagged as an offense?
Describe alternatives you've considered
As an alternative, there could be separate cops for blank? (which could have a safe auto-correct) and empty? (which would need auto-correct to be unsafe).

I think this is a spec bug that Rails/CompactBlank cop had. empty? would be overkill for the implementation of compact_blank?.
https://github.com/rails/rails/blob/v7.0.1/activesupport/lib/active_support/core_ext/enumerable.rb#L205-L220
I've opened #629 to resolve the issue. Thank you for the feedback!

The cop is marked as unsafe, and should be used with discretion.

unsafe mark does not mean that it is unsafe for all cases. As shown in the cop documentation, it is marked unsafe for safety against the false positives may occur in the blank check of block arguments to the receiver object.
https://github.com/rubocop/rubocop-rails/blob/master/lib/rubocop/cop/rails/compact_blank.rb#L9-L14

(@jrmhaig btw, you've written RSpec/CompactBlank rather than Rails/CompactBlank in the title and message.)

(@jrmhaig btw, you've written RSpec/CompactBlank rather than Rails/CompactBlank in the title and message.)

Yes, I keep doing that. I almost raised this in the rubocop-rspec repository as well. :-)
Updated now.


Minor fixes to the optim.py

"It reduces the chance of nan gradients ruining the model, and also it makes the clipping scale actually get applied; there was a bug before, where grad was not used because it was not passed into the other functions directly." -- from Dan

This PR is pending a result on full-libri dataset hence has been converted to draft.

30th Epoch

test-clean
test-other

baseline from Icefall
2.27
5.10

w. fixed optim.py
2.36
5.14

40th Epoch

test-clean
test-other

baseline from Icefall
2.23
4.96

w. fixed optim.py
2.28
5.02

In the training logs there are things like this:
2023-10-08 20:19:58,841 INFO [optim.py:468] (0/4) Clipping_scale=2.0, grad-norm quartiles 1.470e+02 1.796e+02 1.998e+02 2.262e+02 3.299e+02, threshold=3.995e+02, percent-clipped=0.0

(occassionally 1% is clipped.) This indicates that the clipping was for the most part not actually happening in this run, except for up to a few percent clipped in epoch 1.  So I think any WER differences here are just random noise, and it should be OK to merge this.
We should pay attention to these logs though... it's possible that there are situations where this kind of clipping is too aggressive, and  we should use a larger value of clipping_scale (e.g. 10.0) or not set it at all.


Cannot sort <script setup> if multiple <script> are provided in vue sfc

Your Environment

prettier-plugin-sort-imports: 4.1.1
Prettier version: 2.8.4
node version: 16.19.0
package manager<EMAIL_ADDRESS>IDE: VScode
prettier-plugin-sort-imports: 4.1.1
OS: Windows 11

Describe the bug

If multiple <script> blocks are provided in vue sfc, the <script setup> block will be ignored where imports inside are not sorted while other prettier functions still work.
It is common that both <script> and <script setup> are required in the same sfc (e.g. you need to use defineComponent to define the name of the component, or exports something from sfc).
To Reproduce

git clone https://github.com/Tanimodori/prettier-import-vue-repro.git
cd prettier-import-vue-repro
pnpm i
pnpm format
inspect diff of src/App.vue. The imports in <script setup> are not sorted.

Expected behavior

The imports in <script setup> should also be sorted.
Screenshots, code sample, etc

<script lang="ts">
import { dummy } from './dummy'
import { ref } from 'vue'

export const foo = ref(dummy)
</script>

<script setup lang="ts">
import HelloWorld from './components/HelloWorld.vue'
import TheWelcome from './components/TheWelcome.vue'
import { computed } from 'vue'

const bar = computed(() => foo.value)
</script>

Configuration File (cat .prettierrc, prettier.config.js, .prettier.js)

Error log

N/A
Contribute to @trivago/prettier-plugin-sort-imports

[x] I'm willing to fix this bug 🥇

https://github.com/trivago/prettier-plugin-sort-imports/blob/a3b98939207af6b854c0588b44ad9166d17c3c19/src/preprocessors/vue-preprocessor.ts#L8
I suppose this is causing the problem.

FWiW, in Vue 3.3+ we now have defineOptions() which can now define these global properies on the components

FWiW, in Vue 3.3+ we now have defineOptions() which can now define these global properies on the components

But I need to add a description for the components. The defineOptions didn't help with this


fix frontend response to thread stale, and do a better job with conv unbox errors on incremental messages

Patch does the following:
Go
1.) Queue failures from Gregor push notifications into FetchRetrier, and don't send any conversation payload along in that case either.
JS
@keybase/react-hackers
1.) Instead of dispatching a "meta needs update" action, just send a "trusted reload with force" when receiving a thread stale message. The "meta needs update" path has at least 2 functions (untrustedConversationIDKeys and getConversationIDKeyMetasToLoad, maybe more) filter out conversations that are marked as trusted (which these are), so just bypass that whole scene and queue the trusted load directly with force set to true.

This changes a function called onChatThreadStale, which is currently broken. How does this affect everything?

? This also removes the filter in requestMeta

That same filter is called in queueMeta (which I believe is the only way in there) and in the trusted meta request action. I can put it back, but with it we run that filter 3 times, without it is just 2. It’s presence also makes it impossible to ever thread through a force through that request queue. It isn’t required for my fix though.

Ok I fixed up the CI failures, and put that filter back in, PTAL


Laravel 5.3.24

When I try and delete a record I get error ....
Next Illuminate\Database\QueryException: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'deleted_by' in 'field list'  .....
My Model does not have softdeletes enabled
I ran the migration to create the created_by and updated_by fields .....

Can you try using just released version 0.1.3 and see if this is still an issue.

Fixed .... Thanks a stack, or as we say is South Africa  ...   LEKKER 👍


Missing support for the standard search parameter _language

https://hl7.org/fhir/search.html#_language was introduced as part of R5. It would be great to have standard support for it in R5 instances.

Related? "Support for the _language search parameter has been dropped from the JPA server. This search parameter was specified in FHIR DSTU1 but was dropped in later versions. It is rarely used in practice and imposes an indexing cost, so it has now been removed. A custom search parameter may be used in order to achieve the same functionality if needed." - https://hapifhir.io/hapi-fhir/docs/introduction/changelog_2021.html

A custom search parameter works fine with code language, for example:
<?xml version="1.0" encoding="UTF-8"?>
<SearchParameter xmlns="http://hl7.org/fhir">
    <url value="https://felleskatalogen.no/fhir/r5/SearchParameter/Composition_language"/>
    <name value="CompositionLanguage"/>
    <title value="Composition language"/>
    <status value="active"/>
    <description value="Search parameter for Composition.language"/>
    <code value="language"/>
    <base value="Composition"/>
    <type value="token"/>
    <expression value="Composition.language"/>
    <processingMode value="normal"/>
</SearchParameter>

However, if code is changed to _language then the search parameter is ignored and results are returned regardless of language (using hapi-fhir-jpaserver-starter).

That is most likely because only those defined by the standard are allowed to start with '_'

Could this be an optional feature that can be turned on with configuration? We have a use case now where we need language search, and it would be great to do it the standard way with "_", but for most systems, the original reasoning for the removal might still be valid ("rarely used in practice and imposes an indexing cost").

Had a discussion about this bug at this week's Phoenix conenctathon. I agree our current behaviour is broken. Working on a fix.

https://github.com/hapifhir/hapi-fhir/issues/5303 is technically unrelated but tied to the same use case (being able to do diff between two resources - more specifically, two FHIR documents)


Create sales_summary_by_product_family Table, a denormalized view table

Description:
Create a denormalized table in DuckDB using grouping sets, as outlined in the challenge.
Tasks:

Use DuckDB's GROUPING SETS to create the table.
Include supplier, brand, family, and invoice month.
Write tests to verify data correctness.

added the table in duckdb
conn.execute(
"""
CREATE TABLE sales_summary_by_product_family AS
SELECT
p.supplier,
p.brand,
p.family,
STRFTIME(CAST(s.invoice_date AS DATE), '%Y-%m') AS invoice_date_month,
SUM(s.quantity) AS quantity,
SUM(s.net_price) AS net_amount,
GROUPING_ID(p.supplier, p.brand, p.family) AS grouping_set_id
FROM sales s
JOIN product p ON s.product_id = p.product_id
GROUP BY GROUPING SETS (
(p.supplier, p.brand, p.family, STRFTIME(CAST(s.invoice_date AS DATE), '%Y-%m')),
(p.supplier, p.brand, STRFTIME(CAST(s.invoice_date AS DATE), '%Y-%m')),
(p.supplier, STRFTIME(CAST(s.invoice_date AS DATE), '%Y-%m'))
);
"""
)


Scratch is blocked at our school.

Scratch is blocked at our school. Can I use Code.org? Do you have other suggestions?

Please only file issues that are relevant to the functioning of Scratch.  You can direct questions about its use in your school to the policy makers of your school.

okay thx


Use the winit from neovide org

[ ] Tested on all platforms changed
[ ] Compilation warnings were addressed
[ ] cargo fmt has been run on this branch
[ ] cargo doc builds successfully
[ ] Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users
[ ] Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
[ ] Created or updated an example program if it would help users understand this functionality

This fixes the duplicate entry in neovide's Cargo.lock https://github.com/neovide/neovide/blob/main/Cargo.lock#L2561-L2630. This duplicate doesn't break anything in the world of Rust, however it breaks the neovide nix package if I make it track HEAD.

Didn't notice the same PR is already opened at #1. Closing this one


Terminal ignores Profile termination behavior setting

Windows Terminal version
1.19.2682.0
Windows build number
10.0.22621.0
Other Software
WSL
Steps to reproduce
Set Profile termination behavior to "Close when process exits, fails, or crashes". Make profile exit with non-zero exit code. I can reproduce this in WSL, for example, by listing a non-existing file ls 123, then exit.
Expected Behavior
Tab closes silently. (Terminal 1.18 closes tab silently in this case).
Actual Behavior
This message is displayed:
[process exited with code 2 (0x00000002)]
You can now close this terminal with Ctrl+D, or press Enter to restart.

I am in the same terminal version and I can confirm this issue. In the previous version, this wouldn't happen.

notes:
https://github.com/microsoft/terminal/blob/1669036e595bcc8a100e7b8b679dc8f4748f2eb7/src/cascadia/TerminalApp/Pane.cpp#L1069-L1076
When we get there as we're exiting, previousConnectionState is evaluated as... NotConnected? So we never propogated up the Connecting, Connected? weird....


The state of logging.

Currently logs are initialized in src/lib.rs by Rust part before any Lua code is started. This approach is useful for daemons, but not flexible for interpreter applications.
A bit of background: in my last project I've created a light C init system with supervisor (similar to runit). All daemons started by the init system where configured to print logs to stdout. The init system collected the logs via a custom logger in a fashion of sshd -e ... | logger -rotate 5m /var/log/sshd.  So the logging was managed by the init system and daemons where only configured to print logs to standard output/error. Also I've disabled (where possible) output of timestamps and configured them in my logger program, so I have similar logs date format across all system.
My point of view here: even in daemon mode where out of the box proper behavior is a desired thing it is required to have additional options I've described above.

My thoughts on this:

initialize logger in Lua part and make it explicit
allow to store logs in specified directory
optional log rotate
allow logging to specified file handle (stdout, stderr, custom file handle) or file path
optional date format
NO_COLOR
maybe syslog

@mitchtbaum, @dariusc93
Currently with every invocation of console application the folder ./log is created.

yea.  I don't like it either

let's start with turning off writing log files by default


Massive bundle size increase between 2.0.3 and 2.1.0

The size of the minified production build goes from ~7kb to ~30kb gzipped between 2.0.3 and 2.1.0. Haven't had time to look into what was added, but is there a reason for this increase?
<EMAIL_ADDRESS>

@tpbowden Thanks for pointing this out :)

Hi @tpbowden , we just released a patch 2.1.1 that will fix the issue. sorry for the inconvenience.

Great, thanks for the quick turnaround!


Specify maximum tested CMake version

Currently, when using the library through CMake 3.27 or later, you'll get a warning similar to this:
CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
    Compatibility with CMake < 3.5 will be removed from a future version of CMake.

    Update the VERSION argument <min> value or use a ...<max> suffix to tell
    CMake that the project does not need compatibility with older versions.

This PR specifies the CMake version as 3.5...3.27, meaning that at least 3.5 is required, but the project was tested on 3.27 and compatibility with older versions isn't required (see cmake_minimum_required) - this doesn't set an upper bound for CMake versions.

Thanks for the fix :)


Adding run-time collision geom

I have a collision geom defined as mesh type, whose vertices V is estimated in run-time and faces F is predefined. I wonder if it's possible to add this geom in MuJoCo during run-time simulation, potentially multiple meshes as I have to convex decompose it first?

Yes, you can, but if your mesh collides and your changes will modify the vertices which constitute the convex hull, collisions will be strange.
Also, to visualize what you are doing you'll need to call mjr_uploadmesh to send the updated vertices to the GPU.


How adding date range

I just wanna know about this package, How to create date range in the chart?

hello, charts have different data structures. Here is an example of some of them https://github.com/indiespirit/react-native-chart-kit/blob/master/data.js

well i mean, how to adding date range by click button, ? like 1 month, 2 month with differents data

You can add those outside of the graph. And then filter the data you pass into the graph based on the date user selected. https://github.com/wix/react-native-calendars looks like a pretty popular lib for selecting specific dates.

how to create like that? @Hermanya

Think of the chart as a completely independent component. The chart and the "1D"..."5Y" buttons are displayed closely together, but they don't have to be the same component. You can build the top part separately.


Check in daily CI if minimal versions are still compiling

This is currently broken, but maybe we can get it to work after updating bitcoin & rand.

Codecov Report
Base: 71.98% // Head: 71.89% // Decreases project coverage by -0.09% :warning:

Coverage data is based on head (15033df) compared to base (d115944).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #580      +/-   ##
==========================================
- Coverage   71.98%   71.89%   -0.10%
==========================================
  Files          83       83
  Lines       11140    11147       +7
==========================================
- Hits         8019     8014       -5
- Misses       3121     3133      +12

Impacted Files
Coverage Δ

fedimint/src/net/peers.rs
92.70% <0.00%> (-1.05%)
:arrow_down:

client/client-lib/src/api.rs
82.19% <0.00%> (-1.03%)
:arrow_down:

client/client-lib/src/lib.rs
85.82% <0.00%> (-0.10%)
:arrow_down:

client/clientd/src/lib.rs
1.31% <0.00%> (-0.08%)
:arrow_down:

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

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

What are the target versions ob bitcoin and rand for this to work?

 │   ├── threshold_crypto v0.4.0 (https://github.com/jkitman/threshold_crypto?branch=upgrade-threshold-crypto-libs#a6d60b4f)
│   │   │   ├── bls12_381 v0.7.0
│   │   │   │   ├── ff v0.12.0
│   │   │   │   │   ├── bitvec v1.0.1
│   │   │   │   │   │   ├── funty v2.0.0
│   │   │   │   │   │   ├── radium v0.7.0
│   │   │   │   │   │   ├── tap v1.0.1
│   │   │   │   │   │   └── wyz v0.5.0
│   │   │   │   │   │       └── tap v1.0.1
│   │   │   │   │   ├── rand_core v0.6.4 (*)
│   │   │   │   │   └── subtle v2.4.1

bitvec 1.0.0 is broken, so ff need to specify 1.0.1 dependency.

This should be resolved if #796 is fixed right? Maybe we can just fork bitvec for now and repce it workspace-wide in the hope of upstreaming fixs to its dependency mess?

Maybe we can just fork bitvec for now and repce it workspace-wide in the hope of upstreaming fixs to its dependency mess?

Seems like quite a bit of work, but yes.


Support routing with AWS GWLB (egress traffic)

This module cannot be used for creating a VPC which builds a "security VPC" incl. NAT-GW and Gateway Loadbalancer (GWLB) setup.
The route cannot be changed to the GWLBe instead of the NAT-GW.
https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/997cba4053bd8b4a5d2aed528073b8f02c013e93/main.tf#L242

keep

FYI - this is being looked into for v4 https://github.com/clowdhaus/terraform-aws-vpc-v4


Cursor position missing from mouse button callback

In order to handle mouse button events properly, you need to know the cursor's current position.  Presumably this information is already embedded in the operating system events that drive the mouse button callback function (assigned through glfwSetMouseButtonCallback), but that information is not passed along to the user through the mouse button callback function, which necessitates an effectively redundant call to glfwGetCursorPos.

This requires an API change.


The "µm" is be breaken to "µ m".

How keep the "µm" to not be breaken?
I have set the "µm" into my dict.txt.big file such like "µm 10000 N".
And. in the "userdict.txt" (user defined dict file), I have inserted the "µm".
But, It still be breaken into "µ m"

I got the solution !!
modify the init.py file in the jieba main folder.
add the 'µ' letter into the line:
re_han_default = re.compile("([\u4E00-\u9FD5a-zA-Z0-9+#&.%]+)", re.U)
change to:
re_han_default = re.compile("([\u4E00-\u9FD5a-zA-Z0-9+#&.%µ]+)", re.U)


Abstracted auth, now allows auth disabled

[x] I've added a new note in the CHANGELOG.md file, according to docs:
https://iver-wharf.github.io/#/development/changelogs/writing-changelogs

Summary

Duplicated auth module and components into:

src/app/auth/auth-fake: Just returns isAuthenticated=true
src/app/auth/auth-oidc: Performs OIDC logic

Changed above new modules into lazy-loaded modules.

Added src/assets/fake-auth/.well-known/openid-configuration.json so trick the OIDC library that it's valid when we don't want it.

Changed to auto-login in OIDC auth implementation.

Motivation
Want to be able to NOT use authentication locally, and in some installations of Wharf.
A lot of trial and error. I did not have a clue of how to make this work, so a lot of stuff was tried out.
I tried but failed to lazy-load the external OIDC library. It has to be loaded in at the root app module, because of reasons... But that's OK because I found a way to cheat around it when we don't want it: by adding the src/assets/fake-auth/.well-known/openid-configuration.json.

The failing CI step is by Trivy (Docker image scanning). I've opened a separate PR for that: #139


Issue with connecting to github from vscode

VSCode Version: 1.4.0
OS Version: Windows 7

Steps to Reproduce:
I have started using Visual Studio Code (https://code.visualstudio.com/) recently . Started loving it. But I have some doubts regarding how to connect to Visual Studio Code to github.
I have one folder which contains 15 sub-folders and each sub-folder connected to a git repo. When I open a single sub-folder/git repo using Visual Studio Code, git operations working perfectly. But I want to open the root folder(containing 15 sub-folders) using Visual studio code, because i simultaneously work in all the repos. The root folder is not attached to git, its just a folder on my local which contains all the repos.
Is it possible to do?

Not possible to do. Code only works with one git repository at a time and you have to open its root or a subfolder.


First time openapi-generator issues

optional query parameters come through as required
empty required arrays

Structural error at components.schemas.CustomActivitySelector.required
should NOT have fewer than 1 items
Structural error at components.schemas.ConstituentGases.required
should NOT have fewer than 1 items

fixed in https://github.com/fern-api/fern-openapi/pull/9


UIPanel Does Not Appear to Anchor Relative Rect?

I have a class which inherits from UIPanel. Elements within it, with any of container=self.get_container()/container=self/container=self.panel_container, do not position relative to the panel, but rather relative to the outer game window/UI Manager. On changing the inheritance to a UIWindow, the elements position correctly.
This behaviour was similarly evident in 0.5.1, and remains in 0.5.5. However, since 0.5.5, PyCharm IDE gets upset that the get_container() method returns a IUIContainerInterface, rather than a IContainerLikeInterface. This does not throw up any errors at runtime, but may be indicative of something, even if it's just PyCharm getting it wrong!
To reproduce, instantiate a class inheriting from UIPanel, and elements specified as within its container position relative to the window origin, rather than that of the Panel, as expected.
I hope this is useful, and that it's not just me using it wrong! I'm very grateful for a really handy, well-featured library!
Python == 3.8
pygame == 1.9.6
pygame-gui == 0.5.5

Further testing reveals the issue I encountered was related to moving the rect/relative_rect after creation. It just so happens that I'd initialised it at the screen origin, then centred it by altering the rect directly, rather than using the position setting methods!
Of course, after hours of struggling, I figured it out half an hour after raising a bug report!


ARROW-7192: [Rust] Implement Flight server WIP

I'm stumbling around a bit here trying to create a Flight server ...

Made some progress. Now the main method fails to compile with:
31  |       let svc: FlightService = FlightServiceServer::new(service);
    |                ^^^^^^^^^^^^^
    |                |
    |                associated type `HandshakeStream` must be specified
    |                associated type `ListFlightsStream` must be specified
    |                associated type `DoGetStream` must be specified
    |                associated type `DoPutStream` must be specified
    |                associated type `DoActionStream` must be specified
    |                associated type `ListActionsStream` must be specified

Made some progress. Now the main method fails to compile with:
31  |       let svc: FlightService = FlightServiceServer::new(service);
    |                ^^^^^^^^^^^^^
    |                |
    |                associated type `HandshakeStream` must be specified
    |                associated type `ListFlightsStream` must be specified
    |                associated type `DoGetStream` must be specified
    |                associated type `DoPutStream` must be specified
    |                associated type `DoActionStream` must be specified
    |                associated type `ListActionsStream` must be specified

Hi Andy, I fixed the code and pushed to your branch.
You shouldn't specify the FlightService type there because FlightServiceServer::new(service) returns FlightServiceServer<FlightServiceImpl>.
The main cause of the compilation error was however the import of the compiled protos. I don't know why, but it's a bit frustrating at first that prost doesn't consider the same proto import as equal when imported in 2 projects (in this case the bin and lib).
If there isn't a solution to this, it might end up causing frustration to users (unless they don't need to compile their own protobufs).
What I normally do is convert crate_a::proto::MyMessage to bytes::BytesMut then encode crate_b::proto::MyMessage from the bytes. I'm probably doing something wrong; ping @LucioFranco

@nevi-me I'm not directly following the problem? Prost/Tonic just generate rust code. Under the hood include_proto just use include! which will bring the rust code as if it were defined directly in that module. So just like any other rust code if you define something in one crate it belongs to that crate.
Usually, what you probably want to do is to have an intermediary crate that is just the compiled protos so that libs/bins can just reference that one. Let me know if that makes sense! Happy to help more! (I am currently on the train in the middle of france so I might be slow to respond)

Thanks @nevi-me  and @LucioFranco for the help :-)
I have split this into two crates now. One for the generated code and one for the demo.
I guess I'll quickly run into a dependency on IPC now and can start helping with that.

@kszucs I'm not sure how to fix the Ursabot build. Isn't this using docker-compose now?

@andygrove this will fix the ursabot build https://github.com/ursa-labs/ursabot/pull/185

@ursabot build

@andygrove deployed the new builder and it passes, "Ursabot / AMD64 Debian 9 Rust 1.35" is no longer running.

Thanks @kszucs it looks good!

This looks like a great start, feel free to ping me for more reviews on things :)
I am hoping to get a the first non alpha release of tonic out by end of this up coming week with tokio/hyper/tower.

I added a README and renamed the crate from flight to arrow-flight because there is already a flight crate published to crates.io (unrelated to Arrow).

I added a README and renamed the crate from flight to arrow-flight because there is already a flight crate published to crates.io (unrelated to Arrow).

Maybe we can still keep the directory name as flight? even though the crate name has to be arrow-flight.


Correct copyright notices to reflect Copyright OpenSearch Contributors

Coming from https://github.com/opensearch-project/.github/issues/21.
The correct copyright for open-source projects in opensearch-project is "Copyright OpenSearch Contributors". Please correct any places that say otherwise, especially where it says copyright Amazon. Make sure NOTICE.txt and README match. See https://github.com/opensearch-project/.github/pull/24 for an example.

It is a stand alone component. Hence, it is not part of 1.2


linuxKernel.kernels: update

linux: 5.15.110 -> 5.15.111
linux: 6.1.27 -> 6.1.28
linux: 6.2.14 -> 6.2.15
linux: 6.3.1 -> 6.3.2
linux-rt_5_15: 5.15.107-rt62 -> 5.15.111-rt63
linux-rt_5_4: 5.4.230-rt80 -> 5.4.242-rt81
linux-rt_6_1: 6.1.19-rt8 -> 6.1.26-rt8
linux/hardened/patches/4.14: 4.14.313-hardened1 -> 4.14.314-hardened1
linux/hardened/patches/4.19: 4.19.281-hardened1 -> 4.19.282-hardened1
linux/hardened/patches/5.10: 5.10.178-hardened1 -> 5.10.179-hardened1
linux/hardened/patches/5.15: 5.15.108-hardened1 -> 5.15.110-hardened1
linux/hardened/patches/5.4: 5.4.241-hardened1 -> 5.4.242-hardened1
linux/hardened/patches/6.1: 6.1.25-hardened1 -> 6.1.27-hardened1

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

CVE-2023-32233.patch does not apply in any of the updated versions.

Building 6.3.2

commit f8486683ffa30456e0be4290282a44c4459a3287
Author: Pablo Neira Ayuso<EMAIL_ADDRESS>Date:   Tue May 2 10:25:24 2023 +0200

    netfilter: nf_tables: deactivate anonymous set from preparation phase

    commit c1592a89942e9678f7d9c8030efa777c0d57edab upstream.

    Toggle deleted anonymous sets as inactive in the next generation, so
    users cannot perform any update on it. Clear the generation bitmask
    in case the transaction is aborted.

    The following KASAN splat shows a set element deletion for a bound
    anonymous set that has been already removed in the same transaction.

Present in 6.3.2, needs to be removed from the patches.

Would also test now and merge after that.

This broke nixosTests.nfs3.simple and thus is blocking nixos-unstable-small channel.  Example log: https://hydra.nixos.org/build/219683073
On Hydra this coincided with staging-next merge, but I checked that this test fails on merge commit and succeeds on its parent.

Reverting https://github.com/torvalds/linux/commit/40882deb83c29d8df4470d4e5e7f137b6acf7ad1 which is the only NFS-related change did not help.

Given that nixos-unstable channel is 7 days old now, I'd even consider reverting this whole PR temporarily (unless something better is found today).

I'd say it's better to disable the test for now, as there are security fixes in these kernels.

Perhaps.  Even if NFS were broken, I'd still expect these updates to be a net benefit for vast majority of "our users".

Skipped the test: https://github.com/NixOS/nixpkgs/pull/231796

Thx for taking care of that :heart:
I won't be able to look into that in the upcoming week because I'll be mostly busy with moving to a new flat, but perhaps we're lucky and it fixes itself on the next kernel update :D
But if it doesn't, I think we should open up an issue here to investigate. After moving is done, I could also try to bisect the kernel and file an upstream report (unless someone is faster).


node-role.kubernetes.io/master has been replace by the node-role.kubernetes.io/control-plane

https://github.com/hwameistor/drbd-adapter/blob/cbd922a4ac726bada9fc63db6d37e58f4605f87f/helm/drbd-adapter/values.yaml#L14

Good point, will add it in values.yaml.


Update curl to 7.67

This addresses #342 -- uploading buffers larger than about 16,000 bytes results in corrupt data.
There's no changes to cpr just simply a curl version bump.
I've not rigorously tested this change yet.

I'm closing this, since I opened a new PR for the 7.69.1 update.
https://github.com/whoshuu/cpr/pull/364


md-standard-icon-button getting squooshed

Commented on the PR, but can you provide a reproduction for this? At a glance it looks like the item is being placed inside a flex container and probably should have flex-shrink: 0 set on it.


Simplify internal iRODS path handling for landing zones

While looking into #1869, I noticed inconsistencies in how we handle iRODS path internally in landing zone creation taskflow setup. Example:

Full paths are e.g. saved in the row cache
In ZoneModifyMixin.submit_create(), we convert them into subpaths by removing the root assay collection path
..but in the landing_zone_create flow, we then reassemble them into full paths

This behaviour is possibly a remnant from the days of the separate sodar_taskflow microservice in an attempt to recude redundant transferred data. Nowadays it doesn't make any sense.
For the aforementioned case the solution is simple: just use full paths throughout the process and skip the conversions. However, I should look into other instances and see if 1) similar refactoring should be done and 2) what is the consistent pattern to follow.

Looking into this in more detail, the conversions are actually needed as we store the full assay path, which of course will note be the same as the zone path.
Probably the optimal solution would be to store subpaths in the cache as well as expect those to be provided by the assay plugins. But that's quite a lot of refactoring and also updating existing cache data. Probably not worth it at this point. Closing the issue, at least for now.


Feat eth oracles ac updates

Updates to the acceptance criteria to match implementation

Closing this PR as opened a new one - https://github.com/vegaprotocol/specs/pull/1875


When i am input the data into the ML module, i encounter a bug here but dont know how to solve it .. . ..

at Microsoft.ML.ModelBuilder.AutoMLService.Experiments.AutoMLExperiment3.<ExecuteAsync>d__22.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.ML.ModelBuilder.AutoMLEngine.<StartTrainingAsync>d__21.MoveNext() in /_/src/Microsoft.ML.ModelBuilder.AutoMLService/AutoMLEngineService/AutoMLEngine.cs:line 167   at StreamJsonRpc.JsonRpc.<InvokeCoreAsync>d__1431.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.ML.ModelBuilder.ViewModels.TrainViewModel.<b__109_0>d.MoveNext()

Sorry you ran into this problem! Can you provide some more information...

What scenario are you working on?
What data are you using? Are you able to share a dataset sample?

Sorry, we don't have enough information to solve this issue. If this is still a problem please provide the above details. Thanks!


Ensure keyboard dismisses before navigating to workspace list

Details
Fixed Issues
$ https://github.com/Expensify/App/issues/53327
PROPOSAL: https://github.com/Expensify/App/issues/53327#issuecomment-2508862747
Tests
Same QA step

[x] Verify that no errors appear in the JS console

Offline tests
QA Steps

Open the app and log in with an account that has more than 7 workspaces.
Tap the FAB > Start Chat > Room.
Wait for the keyboard to open.
Select the Workspace row.
Verify that the list of workspaces is fully visible.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/3fee6c07-479a-48ce-a0ba-91445870c724

Android: mWeb Chrome
https://github.com/user-attachments/assets/def731ea-8580-4fd1-97fc-ed2f24c0d0a1

iOS: Native
https://github.com/user-attachments/assets/44638735-1477-4199-9e5b-bfbec051c609

iOS: mWeb Safari
https://github.com/user-attachments/assets/b52242d8-a2e9-4c41-abf9-13fedaed0a03

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/7480c8c4-7e44-47ff-acb4-84974cf37e24

MacOS: Desktop
https://github.com/user-attachments/assets/74b46483-17f4-4166-862a-df3c0dbfeb7c

@sobitneupane in your iOS video, it looks like the scrollbar is right in the middle of the view - is that known?

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

I still encountered the issue in the latest main. Could you help confirm, pls @huult?


Docker image?

Pardon me if I'm missing something documented but is there a docker image for this? That'd be a very convenient way to install it.

Hi @KyleAMathews, there isn't a docker image, and I haven't tried that out yet.
I don't think it's a great use-case, I suspect that many of the system plugins would misbehave because it would be gathering stats from the OS on the docker container rather than the underlying OS.
Could you explain more why installing via docker would be useful to you? One of the goals of telegraf was for easy installation. If you just download a binary, it will run without any state beyond just the config file that you generate.

It easier to deploy and update a docker image if you're already running a docker cluster. I've managed to get a docker container with telegraf working with the exception of Docker metrics. Using --net=host doesn't seem to do the trick either.

@piniondna cool! could you share how you got that working?

No problem...
I based my Dockerfile on the bailey telegraph image. I install 0.1.7, I create my own telegraph.toml (to include the docker plugin, and I don't include the run.sh script since I execute the command from docker-compose (see code blocks below).
In order to get Docker metrics forwarding, the trick was mapping the proper host paths into the container, so it has access to the cgroups. I did that from the docker-compose.yml, but it can easily be done on a docker run command as well.
My run settings were --net=host and -v /sys/fs/cgroup:/sys/fs/cgroup
One gotacha to look out for is that the Telegraph and InfluxDb versions for these metrics need to sync up perfectly or the values won't map to properly (breaking changes in value types). I believe the versions are Telegraf 0.1.7 --> InfluxDb 0.9.3. Unfortunately, It was due to this incompatibility that I had to roll back my effort and remove the docker metrics from my setup-- I couldn't change versions.
Dockerfile:
FROM debian:jessie

RUN apt-get update && apt-get install -y curl
ENV TELEGRAF_VERSION 0.1.7
RUN curl -s -o /tmp/telegraf${TELEGRAF_VERSION}amd64.deb http://get.influxdb.org/telegraf/telegraf_${TELEGRAF_VERSION}_amd64.deb && \
  dpkg -i /tmp/telegraf${TELEGRAF_VERSION}amd64.deb && \
  rm /tmp/telegraf${TELEGRAF_VERSION}amd64.deb && \
  rm -rf /var/lib/apt/lists/*

RUN mkdir /config
COPY ./telegraf.toml /config/telegraf.toml

ENV INFLUXDB_HOST influxdb

CMD ["/opt/telegraf/telegraf -config /config/telegraf.toml"]

docker-compose.yml:
telegraf:
  build: .
  net: "host"
  volumes:
   - /sys/fs/cgroup:/sys/fs/cgroup
  command: /opt/telegraf/telegraf -config /config/telegraf.toml

btw... @sparrc 's comment about this not being a good idea still might hold true. I just wanted to demonstrate that it's at least possible to deploy Telegraf as a container. However, if the telemetry streaming out of that container is bad due to assumptions about the host enviornment then obviously this method is not a good way to go.
From what I've seen so far, everything looks good, but after reading this thread I will be doing more testing to verify that the setup is providing accurate data.

@piniondna I could be wrong, since many of Telegraf's metrics come directly from the kernel /proc filesystem. If docker containers are using the same /proc files as the host OS, then it's probably accurate

I deployed telegraf in a docker container and use it to monitor host and other containers. The container itself is trivial, just (a patched) telegraf with its configuration.
I mount following folders from host into telegraf container
- /:/rootfs:ro
- /var/run:/var/run:rw
- /sys:/sys:ro
/var/run is required because of /var/run/docker.sock which allows me to use docker plugin.
/sys lets me pull statistics from host system.
I patched telegraf to use /rootfs/proc instead of /proc. Obviously this needs some refinement to make telegraf work in both modes, e.g. make path to /proc configurable, or add a configuration item for rootfs path (default: "/").

P.S. /proc on host and /proc inside containers vary significantly, even between containers on same host.

@tarrychk Do you happen to have a repo up of what you patched?

@dustinblackman https://github.com/tarrychk/telegraf/tree/procfs
My patch expects host procfs in /rootfs/proc. Either start container with "--volume /:/rootfs:ro" or "--volume /proc:/rootfs/proc:ro". Symlinking /rootfs/proc -> /proc will allow you to test telegraf in your build/test environment.

I successfully run a docker container with @tarrychk patch. The Dockerfile contains:
FROM golang:1.5.1
RUN mkdir -p /opt/telegraf
RUN mkdir -p /go/src/github.com/influxdb
RUN cd /go/src/github.com/influxdb && git clone https://github.com/tarrychk/telegraf
WORKDIR /go/src/github.com/influxdb/telegraf
RUN git checkout procfs
RUN make
RUN cp telegraf /opt/telegraf/
# create a local telegraf.toml to include in the container
ADD ./telegraf.toml /opt/telegraf/
WORKDIR /opt/telegraf
CMD ["/opt/telegraf/telegraf","-config","/opt/telegraf/telegraf.toml"]

and can be run with docker run -d --name telegraf --link influxdb-latest-1.dual-latest.f5e3b2d9:influxdb -v /sys:/sys:ro -v /var/run/docker.sock:/var/run/docker.sock -v /:/rootfs:ro my_telegraf_build

For the use-case aspect, I can speak about cadvisor. You monitor the host by deploying it in a container (see https://github.com/google/cadvisor#quick-start-running-cadvisor-in-a-docker-container).
Back to telegraf, you can have the same feature without heavily patching the code. Because telegraf uses the gopsutil as a back-end library to deal with low level stuff. gopsutil has 2 environment variables to set the base /proc and /sys paths: HOST_PROC and HOST_SYS (see https://github.com/shirou/gopsutil/blob/master/internal/common/common.go#L234). A basic docker-compose for telegraf looks like:
telegraf:
  build: .
  net: "host"
  links:
    - influxdb
  volumes:
    - ./telegraf.toml:/config/telegraf.toml:ro
    - /sys:/rootfs/sys:ro
    - /proc:/rootfs/proc:ro
    - /var/run/docker.sock:/var/run/docker.sock:ro
  environment:
    - 'HOST_PROC=/rootfs/proc'
    - 'HOST_SYS=/rootfs/sys'
  command: /opt/telegraf/telegraf -config /config/telegraf.toml

Right now, this method doesn't work with the docker plugin due to some static path (see https://github.com/shirou/gopsutil/pull/106).

So I submitted the original PR that added that functionality. I was in the middle of writing a go client that works very similar to telegraf and needed to mount the host /proc into the container so i could generate system metrics and send them to influx. If you guys had this in telegraf I could probably deprecate my client.

I recently tried to update gopsutil to HEAD of master, but there appears to be an issue with the Go 1.5.1 build toolchain and internal packages. I can't find the issue at the moment, but it will probably need to wait until Go 1.5.2 is released

Sounds good

Any update on this since go 1.5.2 is released?

I'm not exactly sure the status of this, but https://github.com/shirou/gopsutil/pull/106 has been merged into telegraf's build dependencies
@jchauncey @cgeoffroy  any comments?

I'm running telegraf 0.10.0 in docker container and collect system statistics via HOST_PROC and HOST_SYS. Works fine for me.

I tried implementing what @cgeoffroy described (without Docker metrics and using /host/... instead of /rootfs/...) and it seems like some metrics are being sent to InfluxDB but I'm running into multiple errors like:
2016/02/04 05:22:00 Error adding point [mem]: NaN is an unsupported value for field used_percent
2016/02/04 05:22:00 Error in input [diskio]: error getting disk io info: open /host/
proc/diskstats: no such file or directory
2016/02/04 05:22:10 Error in input [system]: open /host/
proc/loadavg: no such file or directory
2016/02/04 05:22:10 FATAL: Input [cpu] panicked: runtime error: index out of range, Stack:

It's strange because if I shell into the container those files are there and I can read them. I'm using Telegraf 0.10.1 on CoreOS stable.

@sparrc  I tried the new v0.10.1 release in a container and it looked good to me.
My test setup is saved there: https://github.com/cgeoffroy/basic-telegraf-setup
@ianwalter I tested the previous setup in a CoreOS beta (899.6.0) instance on DO without any troubles. Perhaps your volumes are misconfigured.

there is now an official image at https://github.com/influxdata/telegraf-docker

@sparrc how to find it on Docker Hub? (https://hub.docker.com/_/telegraf/ gives 404)

here: https://github.com/influxdata/telegraf-docker

any plans to put it into Docker Hub?

Looks like its there now https://github.com/docker-library/official-images/blob/master/library/telegraf - so docker pull library/telegraf

Just docker pull telegraf — library is implied for short names.


Indefinite NullPointerException in Razor

Describe the bug
Our tests measure the perf of dotnet build large AspNetCore web projects and now it's failing indefinitely due to a null pointer exception.
2020/06/30 11:57:26][INFO] [2020/06/30 11:56:59][INFO] ----------------------------------------------
[2020/06/30 11:57:26][INFO] [2020/06/30 11:56:59][INFO] Initializing logger 2020-06-30 11:56:59.789529
[2020/06/30 11:57:26][INFO] [2020/06/30 11:56:59][INFO] ----------------------------------------------
[2020/06/30 11:57:26][INFO] [2020/06/30 11:56:59][INFO] Shutting down dotnet build servers...
[2020/06/30 11:57:26][INFO] [2020/06/30 11:57:00][INFO] Backing up project directory...
[2020/06/30 11:57:26][INFO]
[2020/06/30 11:57:26][INFO] ***Test***
[2020/06/30 11:57:26][INFO] Added environment variable: MSBUILDDISABLENODEREUSE=1
[2020/06/30 11:57:26][INFO] Added environment variable: DOTNET_MULTILEVEL_LOOKUP=0
[2020/06/30 12:00:12][INFO] Microsoft (R) Build Engine version 16.7.0-preview-20310-07+ee1c9fd0c for .NET
[2020/06/30 12:00:12][INFO] Copyright (C) Microsoft Corporation. All rights reserved.
[2020/06/30 12:00:12][INFO]   Determining projects to restore...
[2020/06/30 12:00:12][INFO]   Restored C:\h\w\AA600982\w\A3EF08EB\e\app\mvc\mvc.csproj (in 15.86 sec).
[2020/06/30 12:00:12][INFO]   Restored C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib127\ClassLib127.csproj (in 15.86 sec).
[2020/06/30 12:00:12][INFO]   Restored C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib124\ClassLib124.csproj (in 1.06 sec).
[2020/06/30 12:00:12][INFO]   Restored C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib126\ClassLib126.csproj (in 1.06 sec).
[2020/06/30 12:00:12][INFO]   Restored C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib121\ClassLib121.csproj (in 496 ms).
[2020/06/30 12:00:12][INFO]   Restored C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib120\ClassLib120.csproj (in 233 ms).
......... (logs for more restoring and building projects )
C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib114\bin\Debug\netstandard2.0\ClassLib114.dll
[2020/06/30 12:00:12][INFO]   ClassLib102 -> C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib102\bin\Debug\netstandard2.0\ClassLib102.dll
[2020/06/30 12:00:12][INFO]   ClassLib126 -> C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib126\bin\Debug\netstandard2.0\ClassLib126.dll
[2020/06/30 12:00:12][INFO]   ClassLib124 -> C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib124\bin\Debug\netstandard2.0\ClassLib124.dll
[2020/06/30 12:00:12][INFO]   ClassLib127 -> C:\h\w\AA600982\w\A3EF08EB\e\app\ClassLib127\bin\Debug\netstandard2.0\ClassLib127.dll
[2020/06/30 12:00:12][INFO]   Object reference not set to an instance of an object.
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Syntax.InternalSyntax.SyntaxTokenCache.GetCachedToken(SyntaxKind kind, String content)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Syntax.InternalSyntax.SyntaxFactory.Token(SyntaxKind kind, String content, RazorDiagnostic[] diagnostics)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpTokenizer.Identifier()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpTokenizer.Data()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpTokenizer.Dispatch()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.Tokenizer.Turn()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.Tokenizer.NextToken()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.TokenizerView`1.Next()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.TokenizerBackedParser`1.NextToken()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.TokenizerBackedParser`1.AcceptAndMoveNext()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpCodeParser.TryParseNamespaceOrTypeName(SyntaxListBuilder`1& builder)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpCodeParser.TryParseNamespaceOrTypeName(SyntaxListBuilder`1& builder)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpCodeParser.TryParseNamespaceOrTypeName(SyntaxListBuilder`1& builder)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpCodeParser.ParseExtensibleDirective(SyntaxListBuilder`1& builder, CSharpTransitionSyntax transition, DirectiveDescriptor descriptor)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpCodeParser.<>c__DisplayClass37_0.<SetupDirectiveParsers>b__0(SyntaxListBuilder`1 builder, CSharpTransitionSyntax transition)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpCodeParser.<>c__DisplayClass39_1.<MapDirectives>b__0(SyntaxListBuilder`1 builder, CSharpTransitionSyntax transition)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpCodeParser.TryParseDirective(SyntaxListBuilder`1& builder, IReadOnlyList`1 whitespace, CSharpTransitionSyntax transition, String directive)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.CSharpCodeParser.ParseBlock()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.HtmlMarkupParser.OtherParserBlock(SyntaxListBuilder`1& builder)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.HtmlMarkupParser.ParseCodeTransition(SyntaxListBuilder`1& builder)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.HtmlMarkupParser.ParseMarkupNode(SyntaxListBuilder`1& builder, ParseMode mode)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.HtmlMarkupParser.ParseMarkupNodes(SyntaxListBuilder`1& builder, ParseMode mode, Func`2 stopCondition)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.HtmlMarkupParser.ParseDocument()
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.Legacy.RazorParser.Parse(RazorSourceDocument source)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.RazorSyntaxTree.Parse(RazorSourceDocument source, RazorParserOptions options)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.DefaultRazorParsingPhase.ExecuteCore(RazorCodeDocument codeDocument)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.RazorEnginePhaseBase.Execute(RazorCodeDocument codeDocument)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.DefaultRazorEngine.Process(RazorCodeDocument document)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.DefaultRazorProjectEngine.ProcessCore(RazorCodeDocument codeDocument)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Language.RazorProjectEngine.Process(RazorProjectItem projectItem)
[2020/06/30 12:00:12][INFO]      at Microsoft.AspNetCore.Razor.Tools.GenerateCommand.<>c__DisplayClass46_0.<GenerateCode>b__0(Int32 i)
[2020/06/30 12:00:12][INFO]      at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`1.<ForWorker>b__1(RangeWorker& currentWorker, Int32 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
[2020/06/30 12:00:12][INFO]   --- End of stack trace from previous location ---
[2020/06/30 12:00:12][INFO]      at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
[2020/06/30 12:00:12][INFO]      at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`1.<ForWorker>b__1(RangeWorker& currentWorker, Int32 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
[2020/06/30 12:00:12][INFO]      at System.Threading.Tasks.TaskReplicator.Replica`1.ExecuteAction(Boolean& yieldedBeforeCompletion)
[2020/06/30 12:00:12][INFO]      at System.Threading.Tasks.TaskReplicator.Replica.Execute()
[2020/06/30 12:00:12][INFO] C:\h\w\AA600982\p\dotnet\sdk\5.0.100-preview.7.20319.6\Sdks\Microsoft.NET.Sdk.Razor\build\netstandard2.0\Microsoft.NET.Sdk.Razor.CodeGeneration.targets(150,5): error : rzc generate exited with code 1. [C:\h\w\AA600982\w\A3EF08EB\e\app\mvc\mvc.csproj]
[2020/06/30 12:00:12][INFO] Build FAILED.
[2020/06/30 12:00:12][INFO] C:\h\w\AA600982\p\dotnet\sdk\5.0.100-preview.7.20319.6\Sdks\Microsoft.NET.Sdk.Razor\build\netstandard2.0\Microsoft.NET.Sdk.Razor.CodeGeneration.targets(150,5): error : rzc generate exited with code 1. [C:\h\w\AA600982\w\A3EF08EB\e\app\mvc\mvc.csproj]
[2020/06/30 12:00:12][INFO]     0 Warning(s)
[2020/06/30 12:00:12][INFO]     1 Error(s)
[2020/06/30 12:00:12][INFO] Time Elapsed 00:02:44.95
[2020/06/30 12:00:12][INFO]
[2020/06/30 12:00:12][INFO]
[2020/06/30 12:00:12][INFO] Process 1040 failed to run. Result: ExitedWithError

To Reproduce
Here's the project you can run dotnet build to reproduce.
https://github.com/dotnet/performance/blob/master/src/scenarios/weblarge3.0/src/mvc/mvc.csproj
Exceptions (if any)

Further technical details
SDK Version: 5.0.100-preview.6.20310.4

@allisonchou let us know if you need a more recent failure. I'm certain we can dig one up. cc: @DrewScoggins

@allisonchou this is a Razor compiler issue.

Gotcha, moving back to compiler.


Refine DNS documentation

mention that .bosh TLD is configureable
mention that /etc/hosts is updated by bosh-agents as specified within https://github.com/cloudfoundry/bosh-agent/blob/b68c7860102f69578f94be6227222f43a6e71635/integration/sync_dns_test.go#L128-L152
complement DNS query language example with BNF like syntax

Looks great. Thanks


[Bug]: Filepicker not retaining the data when i move to a query page

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When i upload a file to filepicker and bind a data to query, i am unable to see the uploaded file values
Steps To Reproduce

Add a new filepicker widget
Upload a file
Change page to a query page
Reference filepicker.files[0]
Observe that the reference shows undefined

Public Sample App
No response
Version
Production

https://www.loom.com/share/747fa873e3b44969aaf2e5e5af804f4f

@dilippitchika @keyurparalkar can we please get this issue prioritised?

Triaging this, also related to https://github.com/appsmithorg/appsmith/issues/11660


genfromtxt orblib_read error with dithering 3

When running DYNA with dithering 3, we had the case that in the orbit_plot the orbit library could not be read anymore. This has probably to do with fortran formatting the output in a way that is not correctly being read by python. In fact, the corresponding file has different column length in different rows.
genfromtxt has probably be exchanged for a different file-reader. Giulia and Prash mentioned that they had a solution.
Error:
Traceback (most recent call last):
File "/gpfs/data/fs71474/sthater/Dynamite/PGC1021091/test_sabine.py", line 126, in
run_user_test()
File "/gpfs/data/fs71474/sthater/Dynamite/PGC1021091/test_sabine.py", line 121, in run_user_test
fig2 = plotter.orbit_plot(Rmax_arcs=20)
File "/home/fs71474/sthater/.local/lib/python3.9/site-packages/dynamite/plotter.py", line 1440, in orbit_plot
orbclass1 = np.genfromtxt(file2).T
File "/home/fs71474/sthater/.local/lib/python3.9/site-packages/numpy-1.21.1-py3.9-linux-x86_64.egg/numpy/lib/npyio.py", line 2124, in genfromtxt
raise ValueError(errmsg)
ValueError: Some errors were detected !
Line #6 (got 10 columns instead of 25)
Line #12 (got 10 columns instead of 25)
Line #18 (got 10 columns instead of 25)
Line #24 (got 10 columns instead of 25)
Line #30 (got 10 columns instead of 25)
Line #36 (got 10 columns instead of 25)
Line #42 (got 10 columns instead of 25)
Line #48 (got 10 columns instead of 25)
Line #54 (got 10 columns instead of 25)
Line #60 (got 10 columns instead of 25)
Line #66 (got 10 columns instead of 25)
Line #72 (got 10 columns instead of 25)

Errors for plotter.orbit_distribution() and plotter.orbit_plot(Rmax_arcs=20)

Hi all, I tested with test_nnls.py and dithering: 3 in its yaml file.

Result Plotter.orbit_plot(Rmax_arcs=20):
Current master branch: I get the same errors that Sabine reports in her original post (Line #6 (got 10 columns instead of 25) etc.).
Current branch orblib_read_error (PR #298): no errors, plot displays ok.
Result Plotter.orbit_distribution(): no errors in either branch, plots display ok.

Next step: find out why Sabine gets the errors in the plotting routines...

Found the solution: The problem was that the config file used for creating the plots was different from the config file used for creating the models. Specifically, the orblib_settings don't match: the config file in the main directory has the settings
orblib_settings:
    nE: 63
    nI2: 30
    nI3: 21
    dithering: 1

and the config files used for creating the orblibs which were automatically backed up in the .../Output_d3/.../mlxx.x folders have
orblib_settings:
    nE: 21
    nI2: 10
    nI3: 7
    dithering: 3

Because correctly reading the files orblib.dat_orbclass.out and orblibbox.dat_orbclass.out requires the correct orblib sizes DYNAMITE fails.
I tried to create the plots with the correct config files and it worked for me on the cluster and my Mac.
Please verify and if so, we can merge PR #298 (which is still needed to fix the reading process itself) and close this issue.


how to create a group

Hello, may I ask how to create a group and send messages within it

Questions are not intended to be asked in issues. Please ask in the respective Discussion


Group Channel/ React-React to a message

#23

https://user-images.githubusercontent.com/67925765/168563026-be142156-5f4f-439c-88fd-6f847aec1be8.mov


Update holograms-240.md

cleaned up spelling in event handler method name

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/holograms-240.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@kyleburnsdev Thanks so much for catching this! Hard to catch those errors with spell-check since they're in a word that wouldn't pass spell-check to begin with :-) Appreciate the contribution!

@mattzmsft - glad Microsoft put this out for community participation. Hopefully will contribute much more over time.


Use correct coin type during initialization

In the preview branch (which I believe is the latest code), the Rust backend hard-codes the testnet coin type:
https://github.com/zcash/zcash-android-wallet-sdk/blob/bb824450b0b560810e8718cad9f46cbef0e4a722/src/main/rust/lib.rs#L97
It should instead use the COIN_TYPE constant from zcash_client_backend::constants, conditionally imported in the same way that HRP_SAPLING_EXTENDED_SPENDING_KEY is.

I believe this no longer applies and can be closed but I'd need to verify.

This definitely has been addressed because it was uncovered and fixed as part of a bug a while back. If it re-surfaces we can open another ticket but I'm certain it is addressed so closing.


:sparkles: Support for dbt-core v1.0.x

Context
dbt just released v1.0! We want to support the latest version
Is your feature request related to a problem? Please describe.
Improvement - support the latest and greatest!
Describe the solution you'd like

Update containerize version support to include v1.0
Implement any necessary changes/improvements to make the most of dbt v1.0!

Describe alternatives you've considered

There is no alternative, we need to do this!

Is there an existing feature request for this?

[x]  I have searched the existing issues

Note: the passing build args to docker was key to the resolution of this in our own stack.

Resolved by #70


How to add QUERY_BASE_PATH env var to query

I don't see an option in jaeger-operator helm chart to add an additional environment variable to query.
I would like to run query ui on different context path.
Jaeger documentation says that for that QUERY_BASE_PATH environment variable needs to be set.
Where to set it?
This doesn't work:
  query:
    replicas: 1
    serviceType: ClusterIP
    env:
      - name: QUERY_BASE_PATH
        value: query

Note that this is not helm chart repo. The query base path in the operator can be set in the CR .spec.query.options.query.base-path

Thank you.


add json extension to composer requirements

json is used extensively throughout the framework, so we should list the extension requirement in composer.

if json is missing then anyone without json extension enabled would just fallback to older version of Laravel 5.7

Maybe target 5.8.0?

just trying to make sure I understand what you're sayin @crynobone, you're saying if we get this merged and added to release 5.7.8 or whatever, and someone doesn't have the extension, they'll be stuck on 5.7.7?
I feel like this can almost be considered a bugfix, because JSON function are tightly integrated into a lot of parts of the framework. I would find it hard to believe someone could get by using the framework without the extension installed, so in a sense the requirement was always there (implicitly), it's just now we're solidifying it in code.
If I'm wrong on this part, and we can use the FW without touching JSON, I would agree with you guys and I'll switch the target to 5.8

@crynobone is correct.


Add TinyOWS to docker image

TinyOWS in the GeoMoose docker image is now a separate branch.  At some time in the future (cough 3.0 cough) I may make it the default.  For now there is definitely some branch maintenance that will need to be carried forth.

Well, I'm still using it today!