ddc#syntax#get() return always [] on latest neovim

Problems summary
ddc#syntax#get() return always [] on latest neovim.
For example, ddc#syntax#get() return [] on this yaml file.
key: value

echo ddc#syntax#get() " return []

This problem occurred in same filetypes other than YAML(Ruby, Scala, etc..).
It seems that Treesitter's behavior or interface has changed.
Expected
ddc#syntax#get() return not empty value.
For example, ddc#syntax#get() return ['String', 'Identifier'] on this yaml file.
key: value

echo ddc#syntax#get() " return `['String', 'Identifier']`

Environment Information

ddc.vim version (SHA1): https://github.com/Shougo/ddc.vim/commit/d9111bd

2022-10-18

denops.vim version (SHA1): 17030b4

deno version(deno -V output): deno 1.26.1

OS: 6.0.1-arch2-1

neovim/Vim :version output:

NVIM v0.9.0-dev-152+g1887d8d7d
https://github.com/neovim/neovim/commit/1887d8d7d

Neovim commit hash history

:x: https://github.com/neovim/neovim/commit/1887d8d7d

latest version on 2022/10/23
This problem is occurring.

:x: https://github.com/neovim/neovim/commit/2ecb407

Probably this problem was occurred by this merge commit.
https://github.com/neovim/neovim/pull/19931

:o: https://github.com/neovim/neovim/commit/06d5c63

One previous merge commit for "2ecb407".
This problem did not occur.

There seems to have been a braking change in treesitter's behaviour, but I don't know the details.
Provide a minimal init.vim/vimrc without plugin managers (Required!)
set runtimepath+=~/.cache/dein/repos/github.com/Shougo/ddc.vim
set runtimepath+=~/.cache/dein/repos/github.com/nvim-treesitter/nvim-treesitter
set runtimepath+=~/.cache/dein/repos/github.com/vim-denops/denops.vim

call ddc#enable()

lua <<EOF
require'nvim-treesitter.configs'.setup {
  highlight = { enable = true }
}
EOF

How to reproduce the problem from neovim/Vim startup (Required!)

nvim 'hoge.yaml'
Move cursor any position.
:echo ddc#syntax#get()

Screenshot (if possible)
Upload the log messages by :redir and :message (if errored)

Too bad.


Update luigi version number to lastest from setup.py automatically.

ref to https://github.com/spotify/luigi/issues/1381

hm isn't this just going to copy the version numbers in a circle?

I'm a little confused ...
conf.py is auto generated by run sphinx-quickstart, and it's used to be customized (I saw many modification git-history on this file). So it's depended on luigi package, but luigi package doesn't require this sphinx doc to the contrary .

Historically (and currently) are Luigi moderators pushing tags to releases? Read the Docs mentions here that they parse the VCS tags.

@dlstadther yes, tags or branches are only related to when to build the doc, but not about the doc content (including HTML title).
I found a good example from Django, take a look at http://django.readthedocs.org/en/stable/contents.html and https://github.com/django/django/blob/master/docs/conf.py#L75

The changes make sense to me at least. I'd say fix the copyright header in order to merge.

i looked at this again and it lgtm but might be worth updating the copyright as @econchick pointed out

awesome thanks!


Oracle database related Issues

Describe the issue:

$subject

Related Issues

Encountering permission issues for admin login to console with Oracle DB configured in IS. #18210

Please update the current status of this issue @Sachin-Mamoru

With the following fixes I was abled to start up the pack with oracle without any issues.

https://github.com/wso2/product-is/issues/17974
https://github.com/wso2/product-is/issues/17903

Now we need to test the beta2 pack with oracle and if any issues occured those should be reference this issue. Then we can analyse the oracle related issues in one place.
@hasanthiP will assign a person to analyse the further issues on IS with oracle

@NilukaSripalim @melanisilva please consider testing the latest version with oracle and check the compatibility.

@NilukaSripalim Please create a new issue for testing purposes.


How to get the ESXi host serial number

Hello team,
I have been using vim.host.SystemIdentificationInfo to get the ESXi host serial number (physical host serial number, in my case it is Cisco UCS Blade). Sometimes it is successful but most of the times I'm not getting the serial number. What is the right method to get the ESXi host serial number. Any help?
Note: I'm connecting to vCenter, getting all the ESXi host in the cluster and for each ESXi host I'm getting enic driver, fnic driver and host serial numbers.

Hello team,
Any help or comments?

si = SmartConnect()
containerHost = si.content.viewManager.CreateContainerView(si.content.rootFolder, [vim.HostSystem], True)

for host in containerHost.view:
    device = {}
    device["sn"] = 'NA'
    sn = 'NA'
    for info in host.hardware.systemInfo.otherIdentifyingInfo:
        if info.identifierType.key == 'ServiceTag':
            device["sn"] = info.identifierValue


DocumentSnapshot.documentID does not seem to exist

G'Day,
According to the cloud_firestore API (https://pub.dev/documentation/cloud_firestore/latest/cloud_firestore/DocumentSnapshot-class.html), there should be a documentID.
I couldn't find it.
Please correct this.
Cheers,
Phil.

DocumentSnapshot has an id field

Some of the fields and methods follow the firebase package's semantics.


Event function

Hello,
If I want to create a series of button from a loop and recover the loop identifier in the event function. How can I recover the loop identifier?
Example :
for (var i = 0; i < 3; i++) {
  L.easyButton( '≡',
    function(){
      alert(i);
   }).addTo(map);
};

Regards,
Pev

Problem temporarily solved by a dirty solution: Move the identifier in the title button.
for (var i = 0; i < 3; i++) {
  var ident = i.toString();
  L.easyButton( '≡',
    function(){
      alert(this.button.title);
    }, ident
  ).addTo(map);
};

But I'm still interested in a better solution :)
Pev

Yeah, unfortunately everything about this will be slightly dirty because current JavaScript doesn't have block scope.  What you'll have to do is force scope with a function.
so this (not working):
for (var i = 0; i < 3; i++) {
  L.easyButton( '≡',
    function(){
      alert(i);
   }).addTo(map);
}

can be reorganized into this(working):
for (var i = 0; i < 3; i++) {
  buildAndAddAButton(i)
};

function buildAndAddAButton(alertVal){
  L.easyButton( '≡',
    function(){
      alert(alertVal);
   }).addTo(map);
}

extra info:
there are other approaches out there for this (either with an IIFE or bind), but this is the most readable/maintainable.
sometime in the future you'll be able to use let, but for now, its support isn't great.
for(var i = 0; i < 3; i++){ // this will never work
  setTimeout(function(){
    console.log(i);
  },i * 2000);
}

for(let i = 0; i < 3; i++){ // this will
  setTimeout(function(){
    console.log(i);
  },i * 2000);
}

Perfect ! Thank you so much !
That answers my questions and solves my problem at the same time :)
Wonderful explanation. As I begin in Javascript, I am happy to have this kind of response, thank you again!


Support zookeeper node replacement on virtual networks.

Right now, a configuration update to a ZK service running on virtual network would cause any connected client to be disconnected until it was restarted.
This makes clients attempt re-resolution when possible (e.g. ZK list was DNS addresses and not literal IPs.

For posterity, this is my own take on https://github.com/apache/zookeeper/pull/150 relative to our other change of doing hostnames and skipping reverse resolution.

And no, I need to add a few tests.


FCS v38.0 fails with the official sample in net472 app

The sample code is from http://fsharp.github.io/FSharp.Compiler.Service/interactive.html
It works in net50 and fails in net472.
Steps:

use the attached project

change to the project directory and run

dotnet run

Actual result
It fails
Unhandled Exception: System.Exception: Error creating evaluation session: StopProcessingExn None
   at Microsoft.FSharp.Core.PrintfModule.PrintFormatToStringThenFail@1433.Invoke(String message) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\printf.fs:line 1433
   at FSharp.Compiler.Interactive.Shell.FsiEvaluationSession..ctor(FsiEvaluationSessionHostConfig fsi, String[] argv, TextReader inReader, TextWriter outWriter, TextWriter errorWriter, Boolean fsiCollectible, FSharpOption`1 legacyReferenceResolver) in F:\workspace\_work\1\s\src\fsharp\fsi\fsi.fs:line 2829
   at FSharp.Compiler.Interactive.Shell.FsiEvaluationSession.Create(FsiEvaluationSessionHostConfig fsiConfig, String[] argv, TextReader inReader, TextWriter outWriter, TextWriter errorWriter, FSharpOption`1 collectible, FSharpOption`1 legacyReferenceResolver) in F:\workspace\_work\1\s\src\fsharp\fsi\fsi.fs:line 3155
   at Program.main(String[] argv) in C:\Users\rjq\Desktop\FCS_net5_issue\Program.fs:line 20

Expected result
It works and prints "43".
Related question/issue
Is FCS v38.0 supposed to work in net472? I thought so but maybe I missed something fundamental.
If it is supposed to work, I have another issue from the actual application with a different error on creating a session:
error FS0078: Unable to find the file 'C:\Windows\Microsoft.NET\Framework64\v4.0.30319\Microsoft.Workflow.Compiler.exe.dll' in any of C:\Windows\Microsoft.NET\Framework64\v4.0.30319 C:\Windows\Microsoft.NET\Framework64\v4.0.30319\WPF C:\Bin\Far\x64 C:\Bin\Far\x64\

FCS_net5_issue.zip

Should I submit this to the parent repository perhaps? There is not much activity in here. And v38.0 changes look not merged.

Yes please, FCS is now built and published directly from dotnet/fsharp.

I would, I read that from v38 FCS is built from main repro now. That would mean this Repro is only in archive status https://github.com/dotnet/fsharp/blob/main/release-notes.md#fsharp-compiler-service-3800

Done -- https://github.com/dotnet/fsharp/issues/10467
Closing this.

Thank you @nightroman! I'll be working on updating this repository's readme to specify the new situation, as well as triaging and migrating existing issues to dotnet/fsharp


Create ISSUE templates

Create a few basic issue templates.

Hey @avneesh0612, can I work on this ?

Sure go ahead!


Change iteritems() to items()

Not sure if this project aims for Python 2 or 3 but I'm using 3 and kept getting an error “ 'dict' object has no attribute 'iteritems' ”.  Looked around and found this helpful page, tried the suggested change and everything works great now.  Looking at the python documentation, it -looks- like the change is backwards compatible for Python 2 users but I'm still new so don't quote me on that.

Thanks! Good catch.
.items() behaves slightly differently between py2 and py3: a list is returned in py2, a "dictview" (essentially an iterator) is returned on py3.
For consistency in behavior, it might be better to use flask._compat.iteritems.

This was fixed in https://github.com/sloria/cookiecutter-flask/pull/53.


Additional test for one-tailed permutation t-test

Reference issue
Provides a test relevant for #11511
What does this implement/fix?
An existing code test compared two-tailed parametric and non-parametric t-tests. This pull request adds the corresponding code tests for one-tailed t-tests.

Oh, GitHub won't let me push the commit... feel free to make the suggested change and push @jCalderTravis !

Thanks @larsoner for your suggestion. I have added this in, and now we get a test for each tail:
FAILED mne/stats/tests/test_permutations.py::test_permutation_t_test_tail[less--1] - AssertionError:
FAILED mne/stats/tests/test_permutations.py::test_permutation_t_test_tail[greater-1] - AssertionError:

@larsoner would you be happy to merge this test? Thank you.

You need to run black on your file to get the style checks to be happy
If one of them is expected to fail you'll need to mark it as xfail (we never want red / failing tests merged into main). So you can do something like:
@pytest.mark.parametrize('whatever', [
    x,
    y,
    pytest.param(z, marks=pytest.mark.xfail(reason='Bug in permutations'),
])

Thanks for your comments @larsoner. I incorporated xfail as you suggested. I checked the style with Flake8, and now none of the lines that I added have any style issues.

Also now formatted with black.

Thanks @jCalderTravis !

Whoops I didn't realize this was your first PR to MNE-Python! Could you open another quick PR to add this change to doc/changes/latest.inc (maybe under the BUG section) with the :newcontrib: role along with name/url in doc/changes/names.inc? Want to make sure you get credit for this


Migrate Deployment Jobs in bamboo to Slurm

As part of SLurm migration need to set up deployment jobs for the following
Goci,
Gwas Catalog
Goci Tools

All Bamboo Deployment Jobs have been cloned to slurm


TextField: BorderColor not applied (until focus/unfocus)

<material:TextField Title="default" />
<material:TextField Title="Red" BorderColor="Red" />
<material:TextField Title="Green" BorderColor="Green"  Keyboard="Numeric" />
untitled.webm

Due to the same reason as #122

Thanks for your catch
There was a nasty bug in InputField. It has been just resolved with https://github.com/enisn/UraniumUI/pull/128
I'll release a patch version for v2.0


Remove unnecessary DSPO parameters

The issue resolved by this Pull Request:
Resolves RHOAIENG-4256
Description of your changes:
Removes parameters that aren't used/duplicated by the DSPO.
Testing instructions
Just build the DSPO image and deploy it with a valid DSPA object.
Checklist

[X] The commits are squashed in a cohesive manner and have meaningful messages.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work

A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-618
An OCP cluster where you are logged in as cluster admin is required.
To use this image run the following:
cd $(mktemp -d)
git clone git@github.com:opendatahub-io/data-science-pipelines-operator.git
cd data-science-pipelines-operator/
git fetch origin pull/618/head
git checkout -b pullrequest 76ce5259598750093ec31a4d35b0ac252fbfcca5
oc new-project opendatahub
make deploy IMG="quay.io/opendatahub/data-science-pipelines-operator:pr-618"

More instructions here on how to deploy and test a Data Science Pipelines Application.

tested with kustomize build:
$ cat apps_v1_deployment_data-science-pipelines-operator-controller-manager.yaml | yq .spec.template.spec.containers[0].env[-1]
name: DSPO_REQUEUE_TIME
value: 20s

and no usage of mlmd writer image found, lgtm
/lgtm
/approve


Enable short position to lose even more money !!! + Ichimoku strategy

Hello,
(This is the first time i'm doing a PR on github so i'm not sure about how it actually works).
This patch is about adding short and close position to earn money when markets are going down. Those order in general cost a taker fee or have specific fee (depends on exchanges if exchange enable it).
So far it only support position on cex.io (the only exchange i'm using for test).
I'm not sure about how the order system works ... lots of copy/paste to have something working ...
the function executeOrder is doing ~200 lines of code and is really hards to play with.
But it works on simulation (not completly sure about the calculation of balance on sim, lots of balance calculation everywhere in the engine.js :/) and short /close are executed on sim.
A refactoring of that part could be helpful.

I also added a new (actually VERY old) strategy : Ichimoku Kinko Hyo.
This strategy is supposed to give good result on high volatility markets going up or down (supports short positions very well), it's also able to detect sideway markets but it's very difficult to do it programatically so this version doesn't detect those (could be an interesting improvment to use limit inside the support/resistance).
This strategy gives some good results on sim ... but as always fails on live :/.
The Ichimoku strategy has 6 signals (2 strongs, 2 weaks and 2 mediums) i've only implemented the 2 strongs (the buy and short with their close). The weaks and mediums signals add lots of errors so they should probably used with others indicators (RSI, CCI, ...) but no documentation about that.
This strategy has been originally developped for a 4h period (got some simulation with a 0% error on a 4h period), but it can work on other periods (with more errors ;)).

I'm not responsible for any loss using this strategy ;)

Well, I just found an issue where the bot can't close a position .... the order system is a serious nightmare :/

This issue was moved to highvelocityspace/zenbot#47

I'll split the strategy and the position (which require a lot of checks and improvments for the sim)


8FBCA751.json seems to have two correct solutions

The following image is an intended solution for the task. It assumes that bottom and top blocks fully connected between themselves. But if we don't assume that, following solution also seems feasible.

This is an acceptable level of ambiguity -- the two possible solutions can be tested by using 2 attempts.


Updates based on PR 1058

Based on feedbacks from PR 1058, I made the following changes:

Remove the "update" button
Dynamically show the filtering results
Update faceting items only when search text is changed

These items have been fixed in PR #1108


Sign Bank:  Odd behavior - after hitting  for a search

When I enter text in the "Key word" box and hit Enter, the handshape palette opens up.  Not sure why this is happening; It should happen only if the user clicks on one of the handshape boxes.
Happens with both Firefox and Chroms

Fixed


Fix: Playwright e2e helpers - Response body handling

Replace response.body calls of Playwright e2e command helpers with response.json(), so the parsed JSON response can be used instead of getting a  Buffer object.

Summary by CodeRabbit

New Features

Enhanced response handling in asynchronous functions to return parsed JSON data for improved data management.

Bug Fixes

Resolved issues related to raw response handling, ensuring more reliable data processing.

@MUTOgen @grantspeelman can you take a look?


Authenticate to Kusto using MI

https://dev.azure.com/dnceng/internal/_workitems/edit/6279

I understand this is only a silently breaking change by the fact that it requires a different set of configuration values, right?
Does an exception get thrown when these are missing?

I added checks to the KustoClientProvider class, the KustoHelper class already had then

@AlitzelMendez please let me know if you want me to wait for the already existing dnceng-shared->helix-service PR before merging this, so we don't change two things at the same time

Hi @dkurepa , thanks for asking, can you hold this one day? I will take a look at the other change tomorrow :) and after that you can merge this, is that ok?

Hi @dkurepa , thanks for asking, can you hold this one day? I will take a look at the other change tomorrow :) and after that you can merge this, is that ok?

Sure, just let me know when you think it's OK to merge!

@dkurepa, I took the changes on this pull request: https://dev.azure.com/dnceng/internal/_git/dotnet-helix-service/pullrequest/39611
when that gets merge it, feel free to merge this one :)


Fix for element position bug when within a transformed parent

This would fix #31 as per @timbell's comment

+1

+1


Difficult to understand background operation toggle

[Enter feedback here]

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

ID: 0c9f9a37-0f20-6201-eabe-9420c008814a
Version Independent ID: 64ae24fc-ae91-759e-0f97-2daa809d59a4
Content: Administration mode - Power Platform
Content Source: power-platform/admin/admin-mode.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

The definition "Background operations" make me some confusion, I have difficult to understand this definition:

Every time I need to keep on async operations I don't understand what really happen if I click on or not on the toggle. For me it isn't clear your definition. Please add one / two simply images in order to explain well what the toggle do. Here my example suggestion:

To Enable Async Opertaion:

To Disable Async Operation:

Thanks!

Thanks for your feedback. We are closing this issue because we are either unable to find relevant information or reproduce this. If you're still experiencing the issue described here, please reopen or create a new issue.


Update sbt-protoc to 0.99.34

Updates com.thesamet:sbt-protoc from 0.99.31 to 0.99.34.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.thesamet", artifactId = "sbt-protoc" } ]

labels: sbt-plugin-update, semver-patch

Seems there is an issue in this version. Closing for now.

Or maybe, this will work with newer sbt?

restarting jobs, we will see

Still fails with latest sbt.


Make rte files optional - fix integration tests

Codecov Report

Merging #13 (e4b8607) into main (c10a75f) will increase coverage by 0.00%.
The diff coverage is 71.42%.

@@           Coverage Diff           @@
##             main      #13   +/-   ##
=======================================
  Coverage   45.88%   45.89%
=======================================
  Files         116      116
  Lines       20792    20795    +3
  Branches    11376    11379    +3
=======================================
+ Hits         9541     9544    +3
  Misses       9457     9457
  Partials     1794     1794

Flag
Coverage Δ

projmgr-cov
78.47% <75.00%> (+0.01%)
:arrow_up:

svdconv-cov
24.64% <ø> (ø)

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

Impacted Files
Coverage Δ

tools/buildmgr/cbuild/include/CbuildModel.h
90.47% <ø> (ø)

tools/buildmgr/cbuild/src/CbuildLayer.cpp
76.04% <0.00%> (ø)

tools/buildmgr/cbuild/src/CbuildModel.cpp
68.50% <50.00%> (ø)

tools/projmgr/src/ProjMgrWorker.cpp
78.22% <50.00%> (+0.01%)
:arrow_up:

tools/buildmgr/cbuildgen/src/CbuildGen.cpp
77.66% <87.50%> (ø)

tools/projmgr/src/ProjMgr.cpp
72.80% <100.00%> (+0.16%)
:arrow_up:

Codecov Report

Merging #13 (e4b8607) into main (c10a75f) will increase coverage by 0.03%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   45.88%   45.91%   +0.03%
==========================================
  Files         116      116
  Lines       20792    20800       +8
  Branches    11376    11382       +6
==========================================
+ Hits         9541     9551      +10
  Misses       9457     9457
+ Partials     1794     1792       -2

Flag
Coverage Δ

buildmgr-cov
75.20% <60.00%> (+0.10%)
:arrow_up:

packchk-cov
50.52% <ø> (ø)

packgen-cov
81.72% <ø> (ø)

projmgr-cov
78.47% <75.00%> (+0.01%)
:arrow_up:

svdconv-cov
24.64% <ø> (ø)

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

Impacted Files
Coverage Δ

tools/buildmgr/cbuild/include/CbuildModel.h
90.47% <ø> (ø)

tools/buildmgr/cbuild/src/CbuildLayer.cpp
76.04% <0.00%> (ø)

tools/buildmgr/cbuild/src/CbuildModel.cpp
68.54% <50.00%> (+0.03%)
:arrow_up:

tools/projmgr/src/ProjMgrWorker.cpp
78.22% <50.00%> (+0.01%)
:arrow_up:

tools/buildmgr/cbuildgen/src/CbuildGen.cpp
78.09% <80.00%> (+0.42%)
:arrow_up:

tools/projmgr/src/ProjMgr.cpp
72.80% <100.00%> (+0.16%)
:arrow_up:

...ols/buildmgr/cbuildgen/src/CMakeListsGenerator.cpp
88.28% <0.00%> (+0.22%)
:arrow_up:

...ls/buildmgr/cbuildgen/src/BuildSystemGenerator.cpp
85.00% <0.00%> (+0.33%)
:arrow_up:


Fixe issue 89

This should fixe: https://github.com/suzuki-0000/SKPhotoBrowser/issues/89

I write some test to check if this issue is not happening again
The sdk is depending of objc so it is not possible to pass an array of swift objects directly that why the use of AnyObject is required for the moment.

It will be nice to remove objc dependencies.

@krummler @alexsanderkhitev this PR fixed critique bug.

@barrault01 Hello=)
Thank you so much!


Add MemGraph?

I noticed this mod by Padishar (@Gerry1135) doesn't appear in CKAN: http://forum.kerbalspaceprogram.com/index.php?/topic/139128-113-memgraph-10010-with-stutter-reduction/&page=1
Granted, it's geared toward intermediate-to-advanced users, but could we add it?

A good suggestion. I've sent the author a message on the forums.

Got a positive reply from Padishar.

Both of the commits in this issue say ksp_version 1.2 but the commit in the linked issue (#4653) says ksp_version 1.1.3 and references the <IP_ADDRESS> version of MemGraph.  That is the correct version for KSP 1.1.3 but are the bits saying 1.2 going to cause issues?

The latest release on GitHub is the 1.2 version, and NetKAN is only inflating the latest release, so <IP_ADDRESS> should remain as 1.1.3 as long as you don't remove all 1.2 releases. If you wish to upload a new 1.1.3 release, we'll need to rejigger things, but as long as that's the final 1.1.3 release, it should be all good.


[ZEPPELIN-309] Redirect to created/cloned notebook page on notebook c…

…reation/clone
This PR enables redirecting to new/cloned notebook pages when create/clone notebook event occurs.

I always thought this feature should be done using RestApi so that you don't get any websocket message, but this is somehow working without any problem.
However the event we rely on in this PR is pretty much not needed originally.
 broadcastNote(newNote); should not be called in the case of a createNewNote or CopyNote.

I think it's okay to use websocket, while this implementation does not adds more complexity (ie. new websocket message) and it's more consistent to keep using websocket for notebook operations in webapp.

This PR i think need to be rebased. also need to change test to make CI test pass.

@Leemoonsoo Rebased master branch and pushed a fix for CI test.

Tested, working well. LGTM


[firebase_auth] codeAutoRetrievalTimeout callback is not called on Android

Sometimes codeAutoRetrievalTimeout is not called. Most of the time the timeout duration is not honored.
To Reproduce
import 'package:firebase_auth/firebase_auth.dart';

main() => runApp(PhoneNumberSignInExample());

class PhoneNumberSignInExample extends StatelessWidget {
  PhoneNumberSignInExample() {
    FirebaseAuth.instance.verifyPhoneNumber(
      phoneNumber: "<phone number here>",
      timeout: Duration(seconds: 1),
      verificationCompleted: (authCredential) {
        print("verificationCompleted");
      },
      verificationFailed: (e) {
        print("verificationFailed");
      },
      codeAutoRetrievalTimeout: (verificationId) {
        print("codeAutoRetrievalTimeout");
      },
      codeSent: (verificationId, [forceResendingToken]) {
        print("codeSent");
      }
    ).catchError((e) => print(e));
  }

  @override
  Widget build(context) => Container();
}

Add a number for testing under firebase phone sign-in configuration.
Expected behavior
A timeout is expected after a given duration.
Additional context
Add any other context about the problem here.
Flutter doctor
Run flutter doctor and paste the output below:
[√] Flutter (Channel stable, v1.17.3, on Microsoft Windows [Version 10.0.19041.207], locale et-EE)
    • Flutter version 1.17.3 at d:\flutter
    • Framework revision b041144f83 (12 days ago), 2020-06-04 09:26:11 -0700
    • Engine revision ee76268252
    • Dart version 2.8.4

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at d:\Android\sdk
    • Platform android-R, build-tools 29.0.2
    • ANDROID_HOME = d:\Android\sdk
    • Java binary at: D:\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[√] Android Studio (version 3.6)
    • Android Studio at D:\Android\Android Studio
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] VS Code (version 1.46.0)
    • VS Code at C:\Users\silver\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.11.0

[√] VS Code, 64-bit edition (version 1.45.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.11.0

+1

Hi All - please upgrade to the latest version where this has been reworked. Please see https://firebase.flutter.dev/docs/auth/phone for more information.


igraph: drop failing test on darwin

Description of changes
An alternative to #251849, following the last comment of that PR.
Closes #228711.
Closes #251849.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] 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.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

If anyone can test whether this issue is still present on macOS 14.1 beta 2 or later, please let me know. https://github.com/igraph/igraph/issues/2340#issuecomment-1747176749

I posted in the Matrix room without getting a response; Also, the servers behind ofborg probably won't get 14.1 soon, so I guess this temporary fix is still needed.

This might be fixed in 14.1 Beta 3. How can I double-check with nix?

The easiest and the cheapest way would be to shallow clone nixpkgs, undo the change of this PR, and nix-build -A igraph.

New user here. How do I get nix-build on a clean system?

nix-build should be available after the installation.

With the PR undone:
100% tests passed, 0 tests failed out of 545

On 2e7758c1187c4afc7a90332ff6a396555c276f3b, with the following diff:
diff --git a/pkgs/development/libraries/igraph/default.nix b/pkgs/development/libraries/igraph/default.nix
index 21396a6b..dc49780b 100644
--- a/pkgs/development/libraries/igraph/default.nix
+++ b/pkgs/development/libraries/igraph/default.nix
@@ -37,10 +37,6 @@ stdenv.mkDerivation (finalAttrs: {

   postPatch = ''
     echo "${finalAttrs.version}" > IGRAPH_VERSION
-  ''
-  # https://github.com/igraph/igraph/issues/2340
-  + lib.optionalString stdenv.isDarwin ''
-    sed -i "/safelocale/d" tests/CMakeLists.txt
   '';

   outputs = [ "out" "dev" "doc" ];


Uncaught TypeError: Illegal invocation

We're getting a Uncaught TypeError: Illegal invocation error when using react-smooth-dnd with @loadable/component.
Specifically, we're trying to import a component that uses react-smooth-dnd dynamically. As the page loads, the error is thrown. The full stack trace is as below:
loadable.esm.js:269 Uncaught TypeError: Illegal invocation
    at Node.get (VM217 8.js:97747)
    at VM217 8.js:97747
    at VM217 8.js:97747
    at Object../node_modules/smooth-dnd/dist/index.js (VM217 8.js:97747)
    at __webpack_require__ (VM210 app.js:64)
    at VM217 8.js:89812
    at Object../node_modules/react-smooth-dnd/dist/index.js (VM217 8.js:89812)
    at __webpack_require__ (VM210 app.js:64)
    at Module../src/components/Sunspot/index.js (VM218 10.js:13301)
    at __webpack_require__ (VM210 app.js:64)

Googling on the issue seems to point towards a loss of context or an undefined global variable after webpack builds.
Has anyone faced similar issues? Any help would be much appreciated. We burned a weekend trying to solve this single issue.
Update: Wanted to add a note that the application works when running via webpack-dev-server directly, but throws the error above after webpack builds it.

This package was generating Uncaught TypeError: Illegal invocation when imported in a JEST environment (for me; CRA).
Adding this to the setupTests.js file in CRA fixed this warning.
Object.defineProperty(global, "Node", {
  value: { firstElementChild: "firstElementChild" }
});

Maybe something similar will work. Is smooth-dnd causing the error https://github.com/kutlugsahin/smooth-dnd/issues/36

Wanted to follow up on this.
We solved this issue when we realised that one of the plugins in webpack is actually automatically appending a script tag pointing to the built app.js file in index.html. We also hand-coded the script tag in the index.html. Therefore, app.js is called twice.
Removing the hand-coded script tag in our index.html solved our problem.
Hope this helps anyone with similar issues.


Bump user-manager dep

There is no hard-coded tenant ID anymore. The tenantID is hard-coded only in cluster-manager and user-manager to create default org/project/cluster.


brow.sh support

2 years ago you were interested into looking into this, so that you don't have to leave your terminal to browse the internet. I'd like to ask, if you are still interested in that or got any findings in this? This is a pleasing idea to me, but I'm not yet ready with brow.sh.

@lalitmee Sure, I'm referring to the Reddit post, but let's not off-platforming. The idea is to combine browse.nvim with browsh, so instead of opening the browser after sending a search through telescope, it would open a new buffer and run browsh, so that you can actually turn NeoVim into a proper browser.
I thinking about trying to do this. I don't know what the best way would be. browsh and its Firefox backend need to run on the same machine afaik, so the NeoVim instance could also be running remotely, so that a terminal buffer that runs browsh would be sufficient enough.
But, I'm learning more towards running NeoVim locally and then connecting via SSH and running browsh in each buffer instead, but I'm not certain if this is even a good idea. I still need to setup browsh.
For now, I just want to ask if you ever looked into this and can share some findings. Would you be interested in assisting to enhance your plugin to support browsh?

@martin-braun, got it. I remember someone asking about the support for browsh.
Actually I have never used browsh but I am open to explore and assisting to enhance browse.nvim to have the support for browsh.
What I can think right now is that we can create a separate module which can separately open browsh. But I am not sure how browsh works. I think you can drop some helpful links for me to read. And in the meanwhile if you are interested, you can start working on the initial POC.
Let's just check if we would be able to support this or not, I mean will it be feasible or not.

@lalitmee Thanks, will definitely take a while because priorities. Before I can even come up with a POC, I need to see if browsh works for me. Also seeing if I can actually make it work in a buffer at all. I will get back to you in regards of this. :)

@martin-braun, sure. I understand.
I will also start looking into browsh.

browsh works, but it's very unstable (testing with Debian LXDE remote firefox via iTerm2 in macOS). It has a --startup-url which also functions as query parameter for the primary search engine, which will be handy for us. So things don't look bad at first glance.
However, I can't really use it in this state. No interest in using a mouse within a terminal, its keybindings are not customizable and vim motions are not yet a thing.
Definitely needs some treatment, before even thinking about this support in general.
But hey, look at that:

Just run :terminal and entered ssh -t debdevx browsh --startup-url google.com (whereas debdevx is the alias of my remote machine that runs Firefox headless and has browsh installed).
Just can't really navigate. This is where vim mode would be necessary.
I see potential to botch things together to get a proper browser in NeoVim. Imagine you can manage your bookmarks and open tabs in form of files (.url/.webloc) and you could use git to sync your browser sessions. On top of that, you can manage those things in a folder structure, simiarly how the Arc browser allows you to organize pinned tabs / bookmarks in their side bar.
I want that, for any platform, in the terminal.
Fascinating stuff.

It's great. I will try to look at it.

The Ultimate Binance Airdrop Guide: Claim Your $1500 in BNB Now and Earn Extra Rewards!
🎉 Exciting News! You are the lucky recipient of the exclusive $1.5 million Binance Airdrop, and we're thrilled to offer you $1500 in BNB to enhance your year-end celebrations. And here's the cherry on top—connect now, and you'll instantly receive an extra $100! Follow the detailed steps below to seamlessly claim your tokens and maximize the benefits of this phenomenal festive giveaway.

How to Claim Your Rewards:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).
Enjoy a smooth and secure connection.

2. 🔄 Interact with the Smart Contract:

Once your wallet is connected.
Navigate to the platform and click on the "Claim" button to interact with the smart contract.
Benefit from the transparent and efficient process.

3. 💰 Additional Bonus Rewards:

Get an instant bonus of $100 upon connecting your wallet.
Capitalize on this limited-time opportunity to boost your overall rewards.

Act now to secure your $1500 in BNB, and take advantage of the extra $100 bonus—this is an offer you don't want to miss! But wait, there's more. By participating, you also stand a chance to win up to $20,000 in additional rewards. Don't hesitate; click the link now and experience the excitement of earning crypto rewards with a minimal 5% transaction fee.
Winners: @cyilcode, @VladlenT, @LayssonOL, @xmrv, @Chazarai, @thainapires, @JiggaNuice, @intwriter, @boliza, @piyushkumar-prog, @anasancho, @nestorwheelock, @ziasistani, @matissime, @jaredtao, @asukaminato0721, @byebye-Fox, @ultrasamad, @wvsr, @JanD4rk, @masaun, @tresabhi, @szpolny, @arogya01, @gravataLonga, @Dryader, @zbw322092, @yashpandeyv2, @bakasmarius, @rafaelazvdo, @sudhithxavier, @hashiguchi556, @ggguser2020, @456vv, @Fjean0522, @lucasbento, @kal-nemi, @odium1963, @fauzipadlaw, @GamingLizard9, @yadhuphilip, @azarudeena, @deivysjoenck, @amitmerchant1990, @OxymoroVP, @ethemozer, @sangress, @jlowry42, @Jakoluco, @CauaRodrigues


Controller with business logic which includes Part-1

Issue/Feature Description:
Why this issue to fixed / feature is needed(give scenarios or use cases):
Controller with business logic which includes:

IncludedNamespaces
ExcludedNamespaces
IncludedResources
ExcludedResources
Content Collections:

Pod resource collections

How to reproduce, in case of a bug:
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

Current Backup API handler all filters requested in the issue.
Please reopen the issue if feel otherwise


[NCPI] Configure TSV download of search results

Blocked by [AnVIL Catalog] Download catalog search results as TSV.

Closing duplication of clevercanary/data-browser#849


Migrate from gatsby-plugin-react-helmet to Gatsby Head API

The gatsby-plugin-react-helmet package will be deprecated in the future. Gatsby adds Head API in<EMAIL_ADDRESS>and the official doc suggests to use it.
Our gatsby-plugin-react-helmet is added in https://github.com/ZJUSCT/mirror-front/commit/e67c3874c3ff0fab65a67278347809910785f25c by @HowyoungZhou .

Removed this plugin in a532f9f by copying the Helmet source from gatsby-plugin-react-i18next, which introduced the dependency to this plugin. Also bumped gatsby-plugin-react-i18next to the latest version. The react-helmet package is still kept because it's still used by gatsby-theme-material-ui-top-layout.

Currently the SEO part is functionally working, but we still need some typing expert to add types to all these untyped code...

BTW, the theme-color meta info is relevant to #30


citrix_workspace: add more features, refactor derivation

Motivation for this change
In #89806 it has been reported that the final package is missing a lot
of features like support for the self-service GUI and the
config-management.
While working on supporting those components in the Nix-package, I
decided to refactor the package to simplify the entire setup.
This patch changes the following things:

Binaries and libraries are patched using the autoPatchelfHook to
avoid having unneeded libraries linked (e.g. some programs use gtk2,
others use gtk3).

Moved source-declarations into their own file.

Wrapped configmgr and selfservice and added those to $out/bin.

Don't mention the old citrix_receiver-packages in the manual anymore
since those packages were removed in 19.09 and are EOLed anyways.

Closes #89806
cc @cebaa would you mind testing the selfservice-feature? I don't have a proper environment to test, unfortunately.
Things done

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

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

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

@cebaa can you please take a final look?
@jtojnar in case you have time to, could you check if the changes look ok?


Correct mode in kick message

@dgw mind testing?

The from_mode appears to be always empty with this patch.
FWIW I have a patch that displays both modes correctly in all tests so far. I can push it up tonight.


Remove "above" from label-commenter-config.yml

Remove above from "DCO check above" on line 26 and 31
https://github.com/meshery/meshery/blob/9b373888409dec99418d2b32328533dacf26fbcc/.github/label-commenter-config.yml#L26
https://github.com/meshery/meshery/blob/9b373888409dec99418d2b32328533dacf26fbcc/.github/label-commenter-config.yml#L31

I want to work on this issue!

Even i want to work on this may be i can review it.

@vishalvivekm  can u assigned me?

@vishalvivekm would it be possible for me to work on this issue? It seems like a great opportunity for me to start contributing to this project

As we follow FCFS I'll assign @abhijeetgauravm the issue.
In meantime @singh1203 @iArchitSharma you guys can assist him or can check for some more good first issue in the project...


Added us/ny/suffolk

This captures some addresses that are not present in the NY statewide data source

@iandees Thanks for fixing my botched field names 👍

Thanks for adding this!


Paul

This repo seems like the right place to put my code to analyze and monitor observing efficiency.

Hi @sbailey - I know you are swamped with other stuff. Perhaps you can find another person to do the review of this PR?

Hi @sbailey - Thank you for the thorough review. I think I've addressed all of your comments. Would you please take another look when you have time? Thanks!


Resolved Light-mode persist issue

Hey @jfmartinz ,
issue closes #1156
I have resolved the issue to persist the dark mode preference across page refreshes using localStorage.
The changes ensure that the user's selected mode (dark or light) is maintained even after the page is refreshed.
Changes made are as follows :
Added functionality to check "localStorage" for the user's dark mode preference upon page load and set the initial state accordingly.
Updated the event listener for the toggle switch to save the dark mode preference in localStorage whenever the user changes the mode.
Now, Whenever user will refresh or log in the page again, The selected mode preference will be the same as chosen.
Please take a look and review it.
Also, please add the labels.

Can you give me a short mp4 for the changes you made? Cant really tell just by looking at the code, thanks

Hey @jfmartinz ,
closes #1464
Here is the Video :
https://github.com/jfmartinz/ResourceHub/assets/160386036/dd799492-5957-4537-a176-0561a221d0c9
Please take a look and review it.


published expo application is showing me the right mile, Google play store is showing wrong miles

Hi All,
React-native-cli: 2.0.1
react-native : 0.57.1
In my React Native application, I am showing miles from the users current location to the destination. I am using geodist for that purpose. Below is my partial code:
`componentDidMount() {
navigator.geolocation.getCurrentPosition(
(position) => {
this.setState({
latitude: position.coords.latitude,
longitude: position.coords.longitude,
error: null,
});
},
(error) => this.setState({ error: error.message }),
{ enableHighAccuracy: true, timeout: 20000, maximumAge: 1000 },
);
var geodist = require('geodist');
var sLat= this.state.latitude;
var sLong =  this.state.longitude;
var dist = geodist({lat: sLat, lon: sLong}, {lat: item.LatL, lon: item.Long2}, 'mi') `

I have the following code in my androidManifest file in my android folder:
`manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.mobileapps.acrapplication">
<uses-permission android:name="android.Manifest.permission.ACCESS_FINE_LOCATION" />
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>

<application
  android:name=".MainApplication"
  android:label="@string/app_name"
  android:icon="@mipmap/ic_launcher"
  android:allowBackup="false"
  android:theme="@style/AppTheme">
  <activity
    android:name=".MainActivity"
    android:label="@string/app_name"
    android:configChanges="keyboard|keyboardHidden|orientation|screenSize"
    android:windowSoftInputMode="adjustResize">
    <intent-filter>
        <action android:name="android.intent.action.MAIN" />
        <category android:name="android.intent.category.LAUNCHER" />
    </intent-filter>
  </activity>
  <activity android:name="com.facebook.react.devsupport.DevSettingsActivity" />
</application>

`
when I publish this app using expo, in expo link, the calculated miles from users current position to destination are shown right on my android phone, but when I publish this app in Google play store then miles are wrong. Do I need too do anything in Google play store or in my code so that Google play store shows the right miles. It seems the current user location in Google play store is wrong.
Any help will be greatly appreciated.

yes, everything looks good.


Update Unit Testing documentation to indicate coverage breaks debugging of pytest

Using coverage with pytest breaks debugging.
https://github.com/Microsoft/vscode-python/issues/693

Being tracked in Microsoft/vscode-docs#1489


[$500] Tag - Selected tag is missing in Split report when split scan is created offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/35913
Action Performed:
Precondition:

User is an employee of Collect workspace.
The Collect workspace has tags.

Go to workspace chat.
Go offline.
Click + > Split bill > Scan.
Create a scan request.
Click on the split preview.
Enter amount, merchant and select a tag.
Click Split.
Navigate to split report.

Expected Result:
The selected tag appears on Tag row.
Actual Result:
The Tag row is missing the selected tag.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/50c34e58-5fc8-4498-b952-d2b6f2f2c594
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0174fb7f86a4edd9a2
Upwork Job ID:<PHONE_NUMBER>915570688
Last Price Increase: 2024-02-09

We think that this bug might be related to #vip-split-p2p-chat-groups
CC @gabrielessner

Proposal
Please re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
When completing the Split Bill, we do not pass the updated category & tag for each individual IOU transaction:
https://github.com/Expensify/App/blob/ad3d78c1ce86c99b3b3e0fb65300a54705e22c77/src/libs/actions/IOU.ts#L2231-L2242
What changes do you think we should make in order to solve the problem?
When completing the Split Bill, we need to pass the category & tag for each individual IOU transaction as well:
        const oneOnOneTransaction = TransactionUtils.buildOptimisticTransaction(
            isPolicyExpenseChat ? -splitAmount : splitAmount,
            currency ?? '',
            oneOnOneIOUReport?.reportID ?? '',
            updatedTransaction.comment.comment,
            updatedTransaction.modifiedCreated,
            CONST.IOU.TYPE.SPLIT,
            transactionID,
            updatedTransaction.modifiedMerchant,
            {...updatedTransaction.receipt, state: CONST.IOU.RECEIPT_STATE.OPEN},
            updatedTransaction.filename,
            null,
            updatedTransaction.category,
            updatedTransaction.tag,
        );

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
When completing the scan split manually, the tag & category are not updated optimistically in the individual transactions.
What is the root cause of that problem?
When completing the Split Bill, we do not pass the updated category & tag for each individual IOU transaction:
https://github.com/Expensify/App/blob/ad3d78c1ce86c99b3b3e0fb65300a54705e22c77/src/libs/actions/IOU.ts#L2231-L2242
What changes do you think we should make in order to solve the problem?
When completing the Split Bill, we need to pass the category & tag for each individual IOU transaction as well:
        const oneOnOneTransaction = TransactionUtils.buildOptimisticTransaction(
            isPolicyExpenseChat ? -splitAmount : splitAmount,
            currency ?? '',
            oneOnOneIOUReport?.reportID ?? '',
            updatedTransaction.comment.comment,
            updatedTransaction.modifiedCreated,
            CONST.IOU.TYPE.SPLIT,
            transactionID,
            updatedTransaction.modifiedMerchant,
            {...updatedTransaction.receipt, state: CONST.IOU.RECEIPT_STATE.OPEN},
            updatedTransaction.filename,
+            undefined,
+            updatedTransaction.category,
+            updatedTransaction.tag,
        );

Result:
https://github.com/Expensify/App/assets/12595293/cd27e3da-bb83-4460-b12c-9aebc894b39b
What alternative solutions did you explore? (Optional)

bumped in Slack
cc @arielgreen as this might be related to #vip-split

@paultsimura's proposal looks good to me 🎀 👀 🎀

On a sidenote, we should consider passing objects instead of 15 func params 😅

hm, I think that will make that method different from all the other buildOptmistic* methods we have, which I'm not sure it would be ideal.
@rushatgabhane can you propose that on #expensify-open-source so we can have more opinions on it?

@danieldoglas if you agree with the proposal review, could you please assign me here so I could start with the recordings? The parameters format can be discussed within the PR IMO.

Thanks. The PR is ready for review: https://github.com/Expensify/App/pull/36759

@johncschuster could you please attach payment summary, thanks 🙇

payment request here https://staging.new.expensify.com/r/1368693063642793

Sorry for the delay! I'm on the payment summary in just a few minutes

Payment Summary:
Contributor: @paultsimura - $500 paid via Upwork
C+ Reviewer: @rushatgabhane - - $500 paid via Manual Request

I believe it's due payment on March 7, right?
Deployed to prod on Feb 29.

I see Feb 28 on my end, so I would pay this out tomorrow.

Payment has been issued to @paultsimura 🎉

$500 approved for @rushatgabhane based on summary.


Bug: Domain retrieval for domain members (not the owner) fails with authorisation error

What were you trying to achieve?
Retrieve a domain by a domain member account using the sdk
What are the expected results?
Retrieve the domain successfully
What are the received results?
failed authorization
Method retrieve_domain for domain id 389efe9b-2e7a-41a1-9ce8-1f910baa93bd took 4.297028ms to complete with error: failed to perform authorization over the entity : failed to perform authorization over the entity : failed to perform authorization over the entity : PERMISSIONSHIP_NO_PERMISSION.

Steps To Reproduce
Create domain
invite user to domain
accept invitation
login to domain as the invited user
token, err := svc.OrganizationLogin(
			mgsdk.Login{
				DomainID: req.OrgID,
			},
			req.Token,
		)

retrieve domain
s.Token = token.AccessToken
domain, err := uis.mgsdk.Domain(s.Domain.ID, s.Token)

In what environment did you encounter the issue?
testing on prism UI with magistrala v0.14
Additional information you deem important
No response

this was a bug based on change in returned status code from the API and has been resolved


Fix typo in Enderium Base recipe

this could actually be simplified even more by just doing       <input name="enderpearl" amount="4"/>


VS Code Intellisense reporting Undefined method 'insertBefore'

I've noticed that my IDE (VS Code) has been highlighting the insertBefore method as unknown despite the fact that it works as expected. I took a quick peek and found that it's likely just a matter of documentation.
An AbstractNode's parent is documented as being an InnerNode (or null):
https://github.com/paquettg/php-html-parser/blob/c116346b2fcc038d4ab6615f4328346a4eebf121/src/PHPHtmlParser/Dom/Node/AbstractNode.php#L44-L49
But the getParent method is documented as returning an AbstractNode:
https://github.com/paquettg/php-html-parser/blob/c116346b2fcc038d4ab6615f4328346a4eebf121/src/PHPHtmlParser/Dom/Node/AbstractNode.php#L154-L162
I'm not 100% sure what the documented return types should be, but at the very least they don't seem to match here, which is ultimately causing Intellisense to interpret something as broken.
Thanks a ton for this great library!

I have updated the getParent method to return the innerNode, which is accurate.


Systematically recognise the location of the ramdisk kernel for OpenBSD

At the moment the path for the bsd.rd kernel results in hardcoding the version of OpenBSD (at present 5.7 or 5.8 (broken if you want to boot -current (5.9))). It would be better if the system worked out where the kernel is so as to save having to maintain version numbers.

I observed this last week with the 5.9 beta snapshots:
http://ftp.openbsd.org/pub/OpenBSD/snapshots/amd64/

The hard coding has turned out to be a pain, yes. I've had some discussion on this on Twitter over a recent break. I took that conversation and thought on it a lot. I guess a deeper question is: What operating systems are supported by iohyve?
What I came up with was this:
A major change to how grub-loaded guests is coming. This will involve including custom grub.cfg files that are stored in a Template dataset much like ISO's and Firmware's.
What I've been wondering is: What operating systems templates are supported by iohyve?
Of course, this means I've got to understand release cycles of the operating systems I would like to support.
Here is a Wiki article outlining this idea: https://github.com/pr1ntf/iohyve/wiki/Guest-Operating-System-Support
Thoughts, questions, comments?

thanks for your efforts on this thus far, @pr1ntf!
I've added some text to the page for openbsd. Basically there's -release and a snapshot of -current.
Technically there's also -stable but that's not made into an ISO image.
Another important thing to know is that openbsd supports the current release and one version back for patches. So 5.8 is the current release and 5.7 is still patched:
http://www.openbsd.org/errata58.html

I'm not sure I agree with having to learn the release cycles of operating systems you want to support. The file name remains the same, an element in the path changes. How can you avoid relying on the specifying the element that changes regularly?

@jungle-boogie: Thanks!
@sevan: You're talking about automating the process? Like once iohyve set obsd57 os=obsd the autogrub (piping to grub) see's obsd then manually looks in the filesystem for where the kernel bits are?

I don't think we will be implementing this right now. There would be a lot of work that would need to be done in grub-cli. Perhaps at a later date once iohyve's use of grub-bhyve improves.
If it gets brought back up by someone else, I'll point them here.
Thanks for your help, you two!


Your documentation says opposite things in different locations

The documentation here says

Don't use both a connection string and an instrumentation key. The latter one set supersedes the other, and could result in telemetry not appearing on the portal. missing data.

But the documentation here says

When codeless monitoring is being used, only the connection string is required. We still recommend that you set the instrumentation key to preserve backward compatibility with older versions of the SDK when manual instrumentation is being performed.

When you auto-instrument an application with Application Insights the Azure Portal adds both the connection string and the instrumentation key.
If this makes any sense it's not at all obvious how.
Also when you enable Application Insights, this is the mess it makes of your configuration. By any chance is the meaning of any of this documented anywhere, including the defaults so I can clean all this up?

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

ID: f1aabfa1-cd7d-3277-60bf-c9b3564ea7c1
Version Independent ID: 67a5f726-4034-dd24-9c92-93348d186ffe
Content: Migrate from Application Insights instrumentation keys to connection strings - Azure Monitor
Content Source: articles/azure-monitor/app/migrate-from-instrumentation-keys-to-connection-strings.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@Neutrino-Sunset
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for your feedback.
Assigning to content author @AaronMaxwell  for review.


Device locations get reset on Homebridge restart

Every time the HomeBridge is restarted, all my Thinq devices get reset and added as new devices, with all the room information lost. Furthermore, since adding my 2 Thinq ACs, on restarting, none of the ThinQ appliances show up and my other plugins become non-responsive. The only way to fix this that I have seen, is to remove the Homebridge from the Home app and add it again as a new accessory, upon which all the accessories are once again detected including the ThinQ ones from this plugin.

update to v1.1.1 and try again, I just re-enabled accessory cache


[TW#26572] IO mux & GPIO mux config confusion (IDFGH-832)

Environment

IDF version (git rev-parse --short HEAD to get the commit id.): 35785d08f

Problem Description
I know that implementation of GPIO and IO mux on ESP32 is not simple but ESP-IDF makes even bigger mess in it.
From my understanding these are necessary steps to correctly setup ESP32 pin:

configure IO mux to assign one specific function to the pin (for example for pin 12: MTDI / HSPIQ / GPIO12)
in case that selected function is GPIO and you don't want to use the pin as GPIO you have to either assign one peripheral output signal to the pin or assign the pin to one or more peripheral input signals.

But looking at https://docs.espressif.com/projects/esp-idf/en/latest/api-reference/peripherals/gpio.html I am really confused.
PIN_FUNC_SELECT seems to be the option for step 1. But it is not even mentioned on the docs page.
gpio_iomux_in and gpio_iomux_out should be the option for step 2. But I am really confused by gpio_iomux_out function. It should take as an argument one of *_OUT_IDX signals from soc/gpio_sig_map.h file.
Also it should be clear at first sight what options you can set while using IO mux and what options to set while using GPIO mux.
Looking into TRM:

you can always set drive strength, input enable option, pullup/pulldown function (options in IO_MUX_x_REG)
if you set GPIO as the function of the pin you can additionaly set interrupt type, interrupt enable, wakeup enable and pad driver (options in GPIO_PINn_REG)

Looking at gpio_config function you are setting pullup/pulldown options and interrupt options in the same config struct. But interrupt configuration makes sense only while not using IO mux directly.

gpio_iomux_in and gpio_iomux_out are for step 1
gpio_matrix_in and gpio_matrix_out should are for step 2
gpio_iomux_in is confusing because one would expect PIN_FUNC_SELECT to configure both input and output and it is only ever used a few times in spi driver

@michprev Hi, would you help share if any updates for the issue? Thanks.

@michprev @negativekelvin Feel free to reopen the issue. Thanks.


refactor: remove codes smells

:white_check_mark: Build TheIdServer 4.2.1-user-ole1-0147 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/c43eb4d7a0 by @)

:white_check_mark: Build TheIdServer 4.2.1-user-ole1-0148 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/9377f1d8b8 by @)

:white_check_mark: Build TheIdServer 4.2.1-user-ole1-0149 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/c25a632142 by @)


Introduce grade "n/a" for facilities that do not support remote monitoring

The reason "monitoring not available" indicates that no monitoring is available. We hence cannot assume it is out of order. Facilities in that state should not be graded.

The reason "monitoring disrupted" also suggests that a facilities status can be temporarily unknown. This case should also be considered.


Add view for latency distribution

Codecov Report

Merging #17 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #17   +/-   ##
=======================================
  Coverage   60.27%   60.27%
=======================================
  Files           2        2
  Lines         146      146
=======================================
  Hits           88       88
  Misses         49       49
  Partials        9        9

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 bfae2e3...2e31c04. Read the comment docs.


feat: add message

Note: Confirm the contribution guidelines before submitting a pull request.
Click the Preview tab and select a PR template:

Standard change
Small change

Do NOT send a PR with this description.

Please fill the description following the guideline.


Diagnose why dockerBuilds do not successfully run on CentOS8 podman hosts

Most recent error from trying to run jdk-linux-aarch64-hotspot on build-equinix-centos8-aarch64-1:
11:25:43   > git checkout -f dab58fcdb5ea0b5c5caa53a547234788248e32ea # timeout=10
11:25:58  Commit message: "Enabling Solaris testing in JDK8 build pipeline (#175)"
[Pipeline] sh
11:31:09  process apparently never started in /home/jenkins/workspace/build-scripts/jobs/jdk/jdk-linux-aarch64-hotspot@tmp/durable-071a2ebe
11:31:09  (running Jenkins temporarily with -Dorg.jenkinsci.plugins.durabletask.BourneShellScript.LAUNCH_DIAGNOSTICS=true might make the problem clearer)
[Pipeline] }

Needs checked on some other CentOS host - possibly the spare Altra I haven't configured yet.

Going to close this as we will not be using CentOS8 on aarch64 hosts going forward as per #2479


Allow usage without site context

Currently when trying to find usages/nested prototypes a valid site (resource) package has to be defined:
./flow prototype:findnested Some.NonSitePackage:Component.SomeComponent
A site with resource package key "Some.NonSitePackage" does not exist

This error can be worked around by specifying a "site context" using the --site-package option.
It would be nice if it was possible to analyze prototypes without loading site specific Fusion

Note: Monocle solves this issue with a custom FusionService that replaces
public function getMergedFusionObjectTree(TraversableNodeInterface $startNode) {...}

with
public function getMergedFusionObjectTreeForSitePackage($siteResourcesPackageKey) {...}


rados: implement binding for read_op_omap_get_vals_by_keys

This PR implements binding for read_op_omap_get_vals_by_keys RADOS Read operation.
Checklist

[x] Added tests for features and functional changes
[x] Public functions and types are documented
[x] Standard formatting is applied to Go code
[x] Is this a new API? Is this new API marked PREVIEW?

PTAL @ansiwen when you have some time, I have a few more rados bindings PRs open (3 + this one to be exact) that I hope to get merged for 0.14.0 too.

Sorry, the issue in #630 also applies here. I hope it's ok, but since we will release one week later, I hope you are fine with that late request.

Sorry, I forgot to use mergify to rebase this and check the CI status... regardless I hope the ci is now fixed

^ I just tried to resolve the conflicts manually here on github...

@Mergifyio rebase

@phlogistonjohn can you please take a look? I believe all the comments so far have been addressed.
Also, when do you roughly expect for 0.14.0 to come out? I don't have any more PRs for the time being, I'm just wondering whether to pin my go.mod dependencies to a commit, or just wait for the release if it's soon enough. Thanks!

@phlogistonjohn can you please take a look? I believe all the comments so far have been addressed.

I will do so as soon as I can

Also, when do you roughly expect for 0.14.0 to come out? I don't have any more PRs for the time being, I'm just wondering whether to pin my go.mod dependencies to a commit, or just wait for the release if it's soon enough. Thanks!

The next release is scheduled for 2022-02-15.  In general, you should be able to use the project milestones  to determine when the next release(s) are going to be. I do try to keep them up to date.  If it's ever missing, out of date, or confusing, feel free to ping me to get me to fix them.

Thank you both for great reviews and for getting through those PRs rather fast @phlogistonjohn @ansiwen :+1: ! @phlogistonjohn thanks for the info!

I highly encourage you to do it soon because I don't love the idea of adding new code based on deprecated functions

AFAIK the only deprecations of concern are rados_read_op_omap_get_keys() and rados_read_op_omap_get_vals()  -- neither rados_omap_get_next() nor rados_read_op_omap_get_vals_by_keys() used in this PR are tagged as deprecated at the moment (all links are pinned to current master HEAD, btw) -- this could change in the future though?
As for the deprecated ...get_keys() and ...get_vals(): neither of these are in use in go-ceph, only the "v2" of get_vals:
https://github.com/ceph/go-ceph/blob/573598cf2286a39447486ac30c09cf427de6af93/rados/read_op.go#L67-L84
... this is just for piece of mind that there's no actual deprecated code in use. In any case I'll send a PR to "upgrade" them all today or tomorrow.


Added time of the day in viz

Fixes #11

Demo: https://raw.githack.com/plannerstack/PTDS.js/add-time/dist/index.html


Project list doesn't update when adding a new entry

Steps to reproduce

expand existing project
click on existing service
create new service and add it to said project
new service gets saved to project

Expected outcome
to 4: new service gets saved and added to the existing list of projects' services
Actual outcome
to 4: last used service's name gets replaced with the new service, nothing is being added in the view

the actual new list appears after collapsing and expanding the project manually, so something might be wrong with the refreshing after adding new services

Versions
App: <IP_ADDRESS>5-stable
Chrome: 58.0.3029.110
Platform: Windows (10)

This is fixed already and will be released soon.


Adapting hpx::find and friends to C++20

This adds compatibility to C++20 for

[x] find find_if find_if_not
[x] find_end find_first_of

This also adds range based overloads for those algorithms

[x] find find_if find_if_not
[x] find_end find_first_of

This also adds constexpr to a couple of facilities (as a flyby).

This is ready for review now.


Double scrollbar on mobile devices

I've noticed that a second scrollbar is rendered on top of the original one on mobile devices (this can be tested with the demo page)

@HenryVonfire  Thanks for pointing it out, I hadn't noticed this before, we'll have to figure out why.


Storage.copy not failing on source (meta)generation mismatch

The following code:
Storage.CopyRequest req = new Storage.CopyRequest.Builder()
  .source(bucket, sourceBlobName)
  .target(destinationBlob)
  .sourceOptions(Storage.BlobSourceOption.generationMatch(42L))
  .build();
storage.copy(req);

Is not throwing an exception if the source blob generation is not 42.

Is this an issue with the server-side stuff not returning a fail? (Is it
picking "the closest one to 42"?) Or is this in our code?

@jgeewax It's an issue with our code, we are not sending the correct parameters, I am about to submit a fix. Server-side stuff is checking for exact match.


Map tool need loading indicator

after I select the main var on the map tool, i see the map marker come up on the screen, but the screen seems to be frozen (I cannot scroll down to the plot modes section). I think this is because the map is still loading. a while later, the map zooms out a bit so that i can see more of the geographic context for the original marker. after this happens, i am then able to scroll.
We need a loading indicator so that the user knows to keep waiting before trying to poke around on the site

Even with UMSP and main var "malaria diagnosis" I see a pink spinner in the center of the map, and I am able to scroll the page up and down to see the plot option radio buttons.
So I cannot reproduce on my slow-ish laptop with Windows 10 + latest FF.

What Danica is describing sounds like the main rendering thread is locked up. This could happen if a lot of things are being render on screen at the same time, such as many markers, or markers with lots of data points. It could also be due to unrelated system activity. A specific study where this is know to happen would be helpful.

Possibly related to freezing in chrome issue (web-components? TBC)

Pretty sure this is resolved by #1201

@danicahelb please try to reproduce when https://github.com/VEuPathDB/web-components/issues/360 has been deployed and is in "ready to be tested"

Note that we have discovered today that the "Chrome map freezing" issue has not completely gone away... So this issue may still occur.

Pink spinner still there but can scroll

Working well today


Extend -b option to accept docker/http/https scheme

This patch enable user to use -b option in different ways for
crc config, crc setup and crc start.
$ crc config set bundle <local_bundle_path or http://foo.com/bundle or docker://registry.com/bundle:version>

$ crc setup -b <local_bundle_path or http://foo.com/bundle or docker://registry.com/bundle:version>

$ crc start -b <local_bundle_path or http://foo.com/bundle or docker://registry.com/bundle:version>

Assumption here is user doesn't change the bundle name during upload to
different location (like fileshare or registry) since we are building
the bundleName using those info.
Testing
To test different image location try following
$ crc setup -b docker://quay.io/praveenkumar/podman-bundle:4.1.1  => this will download and extract image
$ crc start  -b docker://quay.io/praveenkumar/podman-bundle:4.1.1 => this will no do anything and consume it.

If you don't use `setup` and directly use `start` then it will download and extract as part of start command.

To test with http/https location
$ crc setup -b https://storage.googleapis.com/crc-bundle-github-ci/crc_podman_libvirt_4.1.1_amd64.crcbundle
$ crc start -b https://storage.googleapis.com/crc-bundle-github-ci/crc_podman_libvirt_4.1.1_amd64.crcbundle

Same as  image testing.

/retest

How are we checking the integrity of images fetched over http/https? For the releases, we check the hardcoded sha256sum of the bundle, I don't think I've seen something to replace that with this new code?

No, We are expecting user are going to put the bundle on their file server and don't know if they are going to put the sha256sum or not. May be create an issue to discuss what we should do in term of user upload bundles?

We are expecting user are going to put the bundle on their file server

In some distant future yes, at this point, our goal is to be able to use different bundle versions hosted on mirror.openshift.com or quay.io/crcont/, or on exact mirrors of these locations. Once this is in place and works fine, we can extend it to make it more flexible.

We are expecting user are going to put the bundle on their file server

In some distant future yes, at this point, our goal is to be able to use different bundle versions hosted on mirror.openshift.com or quay.io/crcont/, or on exact mirrors of these locations. Once this is in place and works fine, we can extend it to make it more flexible.

I think we are going to do the release 1-2 more OCP bundle version to mirror and later just put it on quay and for images we already have signature checking in place. So I am not sure if we really want to solve that now for our own mirror side or just work on it distant future when user uses the file server.

is there an alternative to using docker:// as this does not seem to be an 'accepted' scheme. why not, like podman, docker, etc assume the address is a registry, unless http(s) is specified?

is there an alternative to using docker:// as this does not seem to be an 'accepted' scheme. why not, like podman, docker, etc assume the address is a registry, unless http(s) is specified?

This is something we are taking from man containers-transports which skopeo also uses.

Local can be enforced by file://

I don't want to force user to use file:// for local now because then it will change the current behavior where user can just provide bundle path without file:// and it just works.

Local can be enforced by file://

I don't want to force user to use file:// for local now because then it will change the current behavior where user can just provide bundle path without file:// and it just works.

Yes, mandating the use of file:// would break backward compat. We should accept it though if it's specified, did not check if that's the case. We probably can use some heuristic though when the arg to -b does not start with xxxx://. If it starts with / or ./ then it's a path, if it starts with a letter then if it's only composed of letters/dots, assume it's a docker:// URL, otherwise treat it as a path. When the heuristic is not correct, it's always possible to use a full URI to get the wanted behaviour.

/hold
Looks like during the PR we are again changing the requirement  like https://github.com/code-ready/crc/pull/3319#issuecomment-1230009359 and https://github.com/code-ready/crc/pull/3319#discussion_r957105779 . Do we really want to have file:// support and also extracting the bundle metadata before pull bundle or download it as part of initial PR ? Should we limit the scope of this PR and then subsequently put other patches.

/hold
Looks like during the PR we are again changing the requirement like #3319 (comment) and #3319 (comment) . Do we really want to have file:// support and also extracting the bundle metadata before pull bundle or download it as part of initial PR ? Should we limit the scope of this PR and then subsequently put other patches.

I did not want to imply these should be part of this PR, I was trying to shape what this feature could look like when it's complete. Imo the addition of validBundleForPreset is also a change from the initial requirements

/hold
Looks like during the PR we are again changing the requirement like #3319 (comment) and #3319 (comment) . Do we really want to have file:// support and also extracting the bundle metadata before pull bundle or download it as part of initial PR ? Should we limit the scope of this PR and then subsequently put other patches.

I did not want to imply these should be part of this PR, I was trying to shape what this feature could look like when it's complete.

I think we should discuss that on the feature issue or create new issue if something is not present. May be I should've created tasks for this feature and implemented it PR by PR.

Imo the addition of validBundleForPreset is also a change from the initial requirements

https://github.com/code-ready/crc/pull/3319#discussion_r955824926 I only added because of this discussion, I thought it might be blocker for the PR.

(on the other hand, the code in this PR might be easier to write/read if -b was handling only URIs, with a small if isLocalPath(bundlePath) { bundlePath = "file://"+filepath.Abs(bundlePath) } adjustment during arg parsing)

If we use "file://"+filepath.Abs(bundlePath then are we again using Download method to copy/download it or just remove file:// and return back the Abs path for extraction?

/retest


Enable club-internal MeetUps

It should be possible to organize MeetUps only for club-members (beside using "private keywords". Solution:

Configure another keyword. That means we have three ones: "global" keyword, club keyword and admin keyword.

For users with club keyword (or admin keyword) MeetUps are offered marked with "club members only"

If creating a MeetUp it should be possible to check it with "for club members only"

Too complicated for users


Clarify Secondary Index Write Consistency on Insert,Update,Delete

Hello,
I couldn't find any information in the documentation regarding the following question:

Is there a way to ensure that an insert() will only return when all necessary properties have been indexed? Something like .insert(...).sync(), but I want to ensure that sync will only return when the document has been persisted and indexed.

@ha1331 helped and mentioned document inserts are atomic and conclude with secondary indexes appropriately updated and indexes wont ever lag behind inserts. Also confirmed this from a SO @coffeemug post here: http://stackoverflow.com/a/18195786/9275
The question came up because some other DB systems can use index methods that lag behind table writes.
The documentation for insert, udpate, delete, sync should mention something about secondary index consistency (and possibly secondary index consistency in replicas too). From what I've found in the SO post, secondary Indexes are instantly updated on the conclusion of a table insert/update/delete when durability=hard on the primary node.
Thanks,
Brian

Hi @bchavez , thanks for bringing that up.
In general RethinkDB doesn't make any difference between (secondary) indexes and the data itself when it comes to consistency semantics. The guarantees are exactly the same for all indexes, including the primary and all secondary indexes.
My suggestion would be to add a note to the document on secondary indexes http://rethinkdb.com/docs/secondary-indexes/javascript/ , rather than to the API docs of each write term.
The consistency guarantees are independent of the particular write term used.


Add GPU builds

Build installers that come pre-packed with the GPU versions of key libraries, and necessary support packages.

tensorflow
dask
pytorch

Fixed by #13 (for Linux)


Albert_DS01 Dealing with Data

Added commentary and answers to the questions related to exploring the python library.

No changes, just saving DS Sprint 1 day 2 notebook to github

Worked on the notebook assignment of loading and cleaning the data from the UCI database

No changes, saving DS Sprint 1 day 3 notebook to github

No changes, saving DS Sprint Day 4 notebook to github

Completed the notebook assignment for day 4 with several tables and charts to draw some inference on the relationship of the variables and answered the questions.

Submitted the Sprint Challenge for week 1


reporting out of memory error in line editor

currently if buffer size reaches limit, just stop line editing.

improve error checking of bracket paste mode in commit 43ab1d2


the meaning of truth sequents

It is not entirely clear what the semantics for truth sequents should be, but here's one proposal. First, let's recall the lay of the land, since it is a little intricate.
The actual type theory has many judgments, one of which is Y || Ψ != M = N in φ, which is the parametric (in Y) functional (in Ψ) membership sequent. I will assume anyone reading this ticket knows what it means (please consult Type Refinements for the Working Class if not). In this note, I will write Ψ ctx rather than Ψ refines Γ and φ type rather than φ refines τ, just for brevity & simplicity.
The proof theory ("refinement logic") of Red JonPRL has a sequent judgment, written Y || H >> φ true [ext M], like Nuprl (with M an output, and all the other symbols inputs). Whereas the !=-sequent is defined by the meaning explanation, the >>-sequent is defined inductively by a collection of sequent-calculus style rules. Of course, >> shall have an intended semantics in the type theory, with which it must be sound, but cannot be complete. >> is in some sense a finite approximation of !=—over time, we'll add more rules to refine the approximation, but we'll never actually reach a point. Now the way that the intended semantics of the proof theory works is that we impose a "postsupposition" (i.e. a condition that we can conclude from the evidence of the judgment).
So the question raised by this ticket is, What is the intended semantics for >>? Let us first consider the simpler case without atoms. Then, it is east: H >> φ true [ext M] postsupposes H ctx, H != φ type and H != M in φ. Recall how this differs from the usual judgments of type theory which have presuppositions (i.e. where the judgment only has a meaning in case the presuppositions obtain), whereas here, the judgment expresses no presupposition at all; yet, we intend to validate its postsuppositions by considering the ways in which we could come to know the judgment (in this case, inductively).
Now, we must extend the explanation to the case with atoms, Y || H >> φ true [ext M]. First, we can say confidently two of the postsuppositions:

Y || H ctx
Y || H != φ type

But what of M? One option would be the simplest, which is to say Y || H != M in φ. However, this does not seem to comport with Allen's supervaluation semantics, which allows the evidence to live in any model that has enough symbols (including ones that have more symbols!). So, this seems to suggest a slight change to the sequent judgment, to something like Y || H >> φ true [ext ρ.M], postsupposing the following:

Y || H ctx
Y || H != φ type
There exists Y' such that ρ : Y --> Y'
Y' || H.ρ != M in φ.ρ

This allows us to generate new symbols in the course of knowing the truth of φ.
Question: Does this look like it should work? If we write up the usual rules for the proof theory, will they be sound according to these semantics? (i.e. will the postsuppositions hold?) I'm thinking it should work, since the renaming ρ will just percolate up the rule in the same way as the extracts—but there may be complications with renaming, since premises might become evident in mutually incompatible worlds, so we may need to rename things.
@jozefg @freebroccolo @vrahli

Should I read rho.M as M is a term in which rho is a substitution to which it may refer? If so then maybe 3 and 4 should be

For every Y' so that we have a renaming rho  : Y -> Y', then Y' || H.rho != M in phi.rho.

This I think solves the renaming issue because we have the freedom on the outside to pick rho. It also means that M must be sort of "stable" in the face of expansions because rho will tell it how to map the collection of symbols it uses into any larger, future domain no?

@jozefg By the clumsy notation rho.M, I just meant "a term rho : Y -> Y' is synthesized, along with a term M which lives at world Y'. The rho (and implicitly the particular Y') was part of the synthesis of the judgment... I probably should have just written <Y',rho, M> or something
Because of the pervasive monotonicity property, M is automatically stable with respect to future worlds.

Oh, I think I see what you are getting at...
So, the synthesis would be <Y', M> such that M lives at Y', and then on the outside we will choose a suitable rho : Y -> Y'. Is that your intention?

@jozefg If my above comment is kind of what you were thinking about, then I think that to make it work, it would end up being essentially equivalent to the original specification—that is, locally (within the subgoal) we must be confident that we have travelled into a world that is accessible from the main goal, so we will have to synthesize our own renaming anyway. And the main goal may choose a different renaming, just by composing some adjustment with the synthesized renaming. Does that makes sense, or have I missed something?


remove version checker job callout

Removed a callout describing a known issue with the version checker job during CRDB installation by the Operator. The known issue has been fixed.

This change is


Update qemu-quest-agent to 9.0.0

The package is currently out of date and the script not working.
-> Update the URLs to load latest version 9.0.0 (as of today)

https://pkg.freebsd.org/FreeBSD:13:amd64/latest/All/qemu-guest-agent-8.2.2_1.pkg

https://pkg.freebsd.org/FreeBSD:13:amd64/latest/All/qemu-guest-agent-9.0.0.pkg


Possible null user?

By any chance, Is that possible user here can be null?
Thanks

Your right, it's unlikely it can be!
It should only happen if someone has logged in then their account has been deleted while their session is still logged in, then they have tried to link their account with an oAuth provider like Facebook, Twitter or Google+ (which isn't currently possible in this example, as the sign in buttons are not displayed once you are signed in, but it should be catered for).


Does PublishPipelineArtifact@1 support wildcard or glob pattern?

The documentation doesn't say much.

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

ID: 41baa0b5-175c-a697-4b7d-c150259c6e6b
Version Independent ID: 75085b15-1acc-526a-0495-74a13c42c1d5
Content: Publish Pipeline Artifacts task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/publish-pipeline-artifact.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @vijayma
Microsoft Alias: vijayma

@msebolt, @vijayma gentle ping.

@msebolt, @vijayma ping.

@msebolt, @vijayma ping.

Wildcards are not supported. You can target a specific file or directory (and if the latter, you'll get everything in the directory).

@Alcas1
I have this use case where there are multiple files that need to be published, and they all share a similar globbable path, but I just have to repeat the PublishBuildArtifacts@1 task even tho they all look nearly identical.
However, because artifacts all require names, I assume each artifact having a unique name makes globbing PathToPublish to make sense, cause if you matched multiple files or directories, what would you name each Artifact? If you just wanted all of the files and folders published, then you'd just copy them to one folder and publish that folder with one name. So after thinking about it, only singular files/folders makes sense

@seanblonien
I ended up discovering this solution myself with a bit of tinkering but it still seems like a workaround as of now. Our files are quite big and take up a couple gigs so it would be best if we could avoid needing to copy these files anywhere at all

So after thinking about it, only singular files/folders makes sense

Cannot agree.  Similar to publishing a folder as an artifact.

I'm in a scenario where we use git commits for the actual package names so that versioning is part of the file name, being able to wildcard for packages would make this be a simple task instead we need to investigate a workaround to capture what the unique version is and pass that to the task.

I needed to filter specific assets, which I needed to upload. The "workaround" I used is to copy the desired one and then upload the whole temp folder.
Example:
- task: CopyFiles@2
displayName: 'move desired artifacts'
inputs:
sourceFolder: source_folder/
contents: '**/?(.log|.jar)'
targetFolder: 'source_folder/temp/'
      - task: PublishBuildArtifacts@1
        displayName: 'publish artifacts'
        inputs:
          pathToPublish: source_folder/temp/
          artifactName: filtered-artifacts


Add support for Bitwise And,Or,XOR at opengl backend

Add support for Bitwise AND
Add support for Bitwise OR
Add support for Bitwise XOR
Add support for Assign AND
Add support for Assign OR
Add support for Assign XOR

@samthakur587 assign this task to me


tpm_util: Remove useless comparison of always identical hashes

@stefanberger that's how I'm seeing this too. However, the raised exception gives you the feeling that somebody truly wanted to compare something of importance here. So I'm

That somebody was 'me'. From what I can see there's nothing else to compare. The nonce has been validated that it was used for the quote (it's inside the quoteblob) and the hash over the quoted PCRs' values has been compared against what was quoted (it's also inside the quoteblob; so now the PCRs in pcrblob should not be fictitious) and the signature is over the quoteblob's hash.
A real problem is solved in the next PR...

@stefanberger that's how I'm seeing this too. However, the raised exception gives you the feeling that somebody truly wanted to compare something of importance here. So I'm

That somebody was 'me'. From what I can see there's nothing else to compare. The nonce has been validated that it was used for the quote (it's inside the quoteblob) and the hash over the quoted PCRs' values has been compared against what was quoted (it's also inside the quoteblob; so now the PCRs in pcrblob should not be fictitious) and the signature is over the quoteblob's hash.
A real problem is solved in the next PR...

@stefanberger link me on that please, as I'm implementing the checkquote() functionality in Golang right now for the Kubernetes controller, I want to know what needs fixing ... I essentially ported the python code


No definition of 'channel' in I/O Ordering

Hi,
In the RVWMO Explanatory Material, in the I/O Ordering section, there is the notion of channel used multiple times.
The issue is that there is no definition of what a channel  exactly is.
Personnaly i might have an idea, but i might be wrong / miss some details.
regards
Charles

Found some channel definitions in the previlegied spec. We might add a pointer ?

@Dolu1990 I realize this has been languishing for a while, but are you talking about linking to the section in the machine.adoc section? https://github.com/riscv/riscv-isa-manual/blob/37298e3248b8ddbaf21df417640a5970c1457281/src/machine.adoc#memory-ordering-pmas
Can you confirm location and I'll add a link?
We are also looking at getting a glossary going, but wanted to actually complete this issue.

Hi ^^
Ahhh it has been a long time, i think i was refering to https://github.com/riscv/riscv-isa-manual/blob/37298e3248b8ddbaf21df417640a5970c1457281/src/mm-eplan.adoc#io-ordering
which talk about channel 0 and 1 without realy explaining what they were.
Yes, a reference to https://github.com/riscv/riscv-isa-manual/blob/37298e3248b8ddbaf21df417640a5970c1457281/src/machine.adoc#memory-ordering-pmas would probably be good, as it explain what they are.
Thanks :)

@Dolu1990 Thanks! I'll have a link in ASAP. Glossary will take a bit longer :)


SASS variables not accessible via Intellisense

When importing sass variables from external modules, they are not autocompleted.

In contrast, this works for the local ones:

More info: this problem is caused by Volar isn't resolving @ alias correctly

Please track https://github.com/microsoft/vscode-css-languageservice/issues/156


Move copy_from_guest to provision/base.py

... and change it a little.

force-push 36c3440: rebase on top of newer, non-broken master

Rebased against current master.

Looks good. Thanks for moving the method.

Like this?

Yes thank you both!

I didn't know you can actually include quote what's the behaviour there?


Inference issues

Hello! Thanks for your excellent work!
I'm running inference on a single image cropped from a video, and it worked just fine.
However, I modified the base.py to let the model denoise a video, and there is cuda out of memory even if I cropped it to 512 * 512, while running on a single image of size 2688*1520 worked just fine. I extracted the frames from the video using cv2, and tried to denoise each frame using the model, but I got cuda out of memory. Do you have an idea of why there is cuda out of memory on the same image size, and one does not have the out of memory issue?

Hi, sorry for the late reply.
I think there would be a difference on the input image size, assume that there is no modification on the model.
Did you check the input image shape or try to denoise frames once at a time?
Best

Thank you! The problem is solved, instead of directly calling the video denoising method, I put it into "test" function in trainer, and it worked fine, and I have no idea what's the difference, since I called "before test" function.


Train suite of models with Tensorflow to be benchmarked

A trained vgg16 model can be exported like this:
import tensorflow.keras as K

from tensorflow.python.framework import graph_util
from tensorflow.python.framework import graph_io
import tensorflow as tf

vgg16 = tf.keras.applications.ResNet50()

sess = K.backend.get_session()

num_output = 1
pred = [None]*num_output
pred_node_names = [None]*num_output
for i in range(num_output):
    pred_node_names[i] = "hi"+str(i)
    pred[i] = tf.identity(vgg16.outputs[i], name=pred_node_names[i])
print('output nodes names are: ', pred_node_names)

constant_graph = graph_util.convert_variables_to_constants(sess,
                                   sess.graph.as_graph_def(),
                                   pred_node_names)

frozen = graph_util.remove_training_nodes(constant_graph)

output = "resnet.pb"
graph_io.write_graph(frozen, ".", output, as_text=False)
print('saved the freezed graph (ready for inference) at: ', output)

This might be able to be done with other models from keras.

Great! there was also this post where they convert .h5 file to protobuf file: https://medium.com/@johnsondsouza23/export-keras-model-to-protobuf-for-tensorflow-serving-101ad6c65142


feat: add script for camper info

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

This should allow us to ask Campers to run node tooling/camper-info.js, and get output like:

Output
🔵 INFO:   Project:  null
🔵 INFO:   Lesson Number:  1
🔵 INFO:   Curriculum Version:  0.1.0
🔵 INFO:   freeCodeCamp - Courses<EMAIL_ADDRESS>🔵 INFO:   Commit:  commit 661ec415a09be54ca0f72182762ba6d4fb541de1
Author: Shaun Hamilton<EMAIL_ADDRESS>Date:   Thu Sep 22 23:03:32 2022 +0100

    feat: add script for camper info

Note: This does not result in a release. It just acts as an example of something we should manually add to each repo

Why does this not constitute a release?

This does not change the package. It changes the example ./tooling folder


Add flask capabilities to main terminal

train_on_pdf.py and infer_on_pdf.py should be also attainable via a flask app like model, extraction and rule_based docker.

PR #16 merged the feature/2023.04-os-test into develop which enables the flask capabilities.
Documentation has to be produced.


[install.sls] add enforce_toplevel: False to extract single file

Fixes #7

@arthurlogilab thanks for the fix


featureimplement hive table sink plan

Proposed changes
from https://github.com/apache/doris/pull/31765

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Update API

BREAKING CHANGE: HTTP paths has changed in APIs

Codecov Report
Base: 78.52% // Head: 78.52% // No change to project coverage :thumbsup:

Coverage data is based on head (eb8e2a7) compared to base (c953c86).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #144   +/-   ##
=======================================
  Coverage   78.52%   78.52%
=======================================
  Files          23       23
  Lines        2272     2272
=======================================
  Hits         1784     1784
  Misses        377      377
  Partials      111      111

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.