Problem in running the Tutorial

URL
https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/run
Operating system
windows
More information about your system
No response
Description
The Solution Explorer did not appear.
The Start Debugging and Run Without Debugging did not respond as expected. Instead, these commands opened the Welcome window. I don't yet know how to go ahead further.
dotnet --info
No response

Can you confirm that you installed the .NET SDK as instructed in step 1?
If so, have you tried restarting VS Code?
Can you try clicking on the Program.cs file to see if this triggers the Solution Explorer to appear?


AT:  S-1.2 - As a user, I want to see the building I am currently in.

This acceptance test is in reference to issues #17, #19
Acceptance Criteria
Given

that the map is open

When

I am inside of a campus building,

Then

I should see my current location within the perimeter of the building.

Steps

Open the app, the app should launch and Google maps opens with my current location within the perimeter of the building.

Example

ACCEPTED.


Add nwb as module test and build tooling

Related to #71 --

Switches to nwb for building the module

Builds three targets (lib, es, umd)
https://github.com/insin/nwb#npm-modules-for-the-web

Refactors tests to use fetch-mock instead of nock, allowing tests to be run in browsers

Currently, tests are run in PhantomJS but there is an opportunity to use something like SauceLabs and run against a matrix of browsers
Tests can still be run in node via new test:node script

General package.json, eslintrc, travis.yml etc cleanup

Coverage remained the same at 100.0% when pulling e802a1a4391bf4418330c892a13a406751531ff8 on nason:nwb into cbf0553fac905dc0e7434193d3593dba677d9559 on agraboso:master.

@nason will tests be splitted across multiple files? it's getting pretty big.

will tests be splitted across multiple files? it's getting pretty big.

I'm definitely open to that @iamandrewluca -- my goal in this PR is more to update the build and test tooling, not really to re-organize the tests 🤠

Closing this per #195. I'm going to update the test setup in a separate PR


Update CONTRIBUTIONS.md

@catalin-hritcu @artagnon @protz @mtzguido @aseemr

This is fine by me. Things to note:

the "green" as referred to in this sentence means "green on slack", which is more stringent than "green on the pull request"
the reviewer may (in rare cases, when the change touches extraction) need to bootstrap twice to reach the fixpoint.

the "green" as referred to in this sentence means "green on slack", which is more stringent than "green on the pull request"

I guess the proper name for the Slack thing would be "Success without breakages".
And can one know that a PR is already a "Success without breakages" without kicking an extra build that reports to Slack? Is there a way to tell this just from the VSTS logs?

And can one know that a PR is already a "Success without breakages" without kicking an extra build that reports to Slack? Is there a way to tell this just from the VSTS logs?

Actually the first thing in the "Extra Logs" is a "Build Status" field, so never mind, it should be easy to check even without a build that goes to Slack.

I've further improved the explanation on snapshots and while I was at it also improved the rest of the document.


_FixedOffset should be a shim rather than relying on conditional imports

Look at:
https://github.com/Azure/azure-sdk-for-python/pull/16800#discussion_r579554218

Update, code using conditional import because of Python 2 should disapear:
try:
    from datetime import timezone

    TZ_UTC = timezone.utc  # type: ignore
except ImportError:
    TZ_UTC = _FixedOffset(0)  # type: ignore

or
        try:
            tzinfo = datetime.timezone(datetime.timedelta(minutes=delta))
        except AttributeError:
            tzinfo = _FixedOffset(delta)

https://github.com/Azure/azure-sdk-for-python/pull/27072


Upgrade go-mod-messaging and paho.mqtt lib

🚀 Feature Request
Relevant Package [REQUIRED]

 This feature request is for...
All service
Description [REQUIRED]

Upgrade go-mod-messaging and paho.mqtt lib
golang.org/x/sync should also be added into attribution.txt

Close by https://github.com/edgexfoundry/edgex-go/pull/4055


Support for DATABASE_URL or similar

It seems quite common in tooling these days to put all the connection information in a single environment variable as a url.
It would be handy for squirrel to support this too.
An example can be seen in the dbmate docs: https://github.com/amacneil/dbmate?tab=readme-ov-file#connecting-to-the-database
I hope this is sufficient information to go off.

Thank you for the suggestion! Now you can use a DATABASE_URL env variable with a connection string in v1.3.0


Support objects with same name on different schemas

Goals
I am developing a framework that uses Realm. I want to make sure the client apps won't crash when using Realm as well, regardless of the name they decide to provide to their own objects.
Expected Results
Client apps can use Realm without any trouble.
Actual Results
Client apps crash if they try to create an Object with a name already used in the Framework schema.

*** Terminating app due to uncaught exception 'RLMException', reason: 'RLMObject subclasses with the same name cannot be included twice in the same target. Please make sure 'UserObject' is only linked once to your current target.'

As you may see below, I am overriding the func shouldIncludeInDefaultSchema() so that the framework objects aren't included in the app schema, but it still crashes.
Steps to Reproduce
Run the client app with the code below.
Code Sample
// In the Framework:

class UserObject: Object {
    @objc dynamic var lastName = ""

    override class func shouldIncludeInDefaultSchema() -> Bool {
        return false
    }
}

class PersistenceController {

   // Method used to access the Realm instance, properly set with its object types
   func createRealm() throws -> Realm {
        let objectTypes: [Object.Type] = [UserObject.self]
        let configuration = Realm.Configuration(schemaVersion: 1, objectTypes: objectTypes)
        return try Realm(configuration: configuration)
    }
}

----------------------------------------------------------

// In the client app:

class UserObject: Object {
    @objc dynamic var firstName = ""
    @objc dynamic var lastName = ""
    @objc dynamic var age = 18

}

Version of Realm and Tooling
Realm framework version: 3.11.2
Xcode version: 10.1
iOS/OSX version: iOS 12.1

This check exists because when you have a property like @objc dynamic var obj: UserObject?, the obj-c runtime just tells us that the type is UserObject, and not what the namespace of that type is. As a result, if a UserObject type is defined in two namespaces, we don't know which one it is.
A workaround for this is to override the obj-c names of the classes defined in a framework to give them a unique prefix, as you would when defining obj-c classes directly. The syntax for this is @objc(FrameworkUserObject) class UserObject: Object { ... }.


Create GN build for Dart SDK

"Some people want to extend the GN build to the full Dart SDK.".  It is currently working to build the runtime in the flutter context.
Here is a checklist of things that we would need to add:

[ ] Add Wiki page "Building with GN"

[x] Add gn binary download to repo checkout?

[x] Create working standalone GN build for runtime

[x] Analyzer + summaries

[x] Snapshots of tools

[x] Copy files to SDK directory

I'll do "Add gn binary download to repo checkout".

Started working on this here: https://codereview.chromium.org/2350583002/

The GN build can now produce the SDK on all platforms, including cross-builds, Android, and ASAN on Linux. It's wired up to both gclient runhooks and build.py. It is used as the default if the environment variable DART_USE_GN is defined, and can otherwise be selected with the flag --gn. It also produces project files for VS and XCode on Windows and Mac, respectively.
@whesse Can we get this set up on a builder or two?

Wonderful!  I'll set up builder/testers for this.  We will need documentation about how to update this and GYP (and blaze) in simple cases like adding a source file to the build of a target.  But that is mostly in VM, and you guys know already how to do it.

________ running 'D:\VRProject\depot_tools\python276_bin\python.exe sdk/tools/generate_buildfiles.py --gn' in 'D:\VRProject\dart\dartbuild'
Command failed: D:\VRProject\dart\dartbuild\sdk/buildtools/win/gn gen --check --ide=vs out\DebugIA32 --args=is_release=false is_clang=false dart_zlib_path="//runtime/bin/zlib" use_goma=false dart_use_fallback_root_certificates=true target_cpu="x86" goma_dir="None" dart_use_tcmalloc=false dart_target_arch="ia32" is_asan=false is_product=false is_debug=true dart_runtime_mode="develop" dart_host_pub_exe="D:\VRProject\dart\dartbuild\sdk\tools\sdks\win\dart-sdk\bin\pub.bat" dart_debug=true host_cpu="x86" target_os="win"
output: Traceback (most recent call last):
  File "D:/VRProject/dart/dartbuild/sdk/build/toolchain/win/setup_toolchain.py", line 154, in <module>
    main()
  File "D:/VRProject/dart/dartbuild/sdk/build/toolchain/win/setup_toolchain.py", line 121, in main
    args = _SetupScript(cpu, win_sdk_path)
  File "D:/VRProject/dart/dartbuild/sdk/build/toolchain/win/setup_toolchain.py", line 68, in _SetupScript
    return [os.path.normpath(os.path.join(os.environ['GYP_MSVS_OVERRIDE_PATH'],
  File "D:\VRProject\depot_tools\python276_bin\lib\os.py", line 423, in __getitem__
    return self.data[key.upper()]
KeyError: 'GYP_MSVS_OVERRIDE_PATH'
ERROR at //build/toolchain/win/BUILD.gn:28:18: Script returned non-zero exit code.
toolchain_data = exec_script("setup_toolchain.py",
                 ^----------
Current dir: D:/VRProject/dart/dartbuild/sdk/out/DebugIA32/
Command: D:/VRProject/depot_tools/python276_bin/python.exe -- D:/VRProject/dart/dartbuild/sdk/build/toolchain/win/setup_toolchain.py "C:\Program Files (x86)\Microsoft Visual Studio 14.0" ../../third_party/gyp/pylib/gyp/win_tool.py "C:\Program Files (x86)\Windows Kits\10" "C:\Windows\Sysnative;C:\Windows\SysWOW64" x86
Returned 1.
See //BUILD.gn:6:1: which caused the file to be included.
group("default") {
^-----------------
Error: Command 'D:\\VRProject\\depot_tools\\python276_bin\\python.exe sdk/tools/generate_buildfiles.py --gn' returned non-zero exit status 1 in D:\VRProject\dart\dartbuild

I get the above error when attempting to use the GN build on windows.

@mfrancis107 Thanks for the report. Did you follow the instructions here and set DEPOT_TOOLS_WIN_TOOLCHAIN=0. It looks like open source contributors may also need to define the environment variable GYP_MSVS_OVERRIDE_PATH to be the path to Visual Studio, e.g. C:\Program Files (x86)\Microsoft Visual Studio 14.0. I'm unable to test that at the moment, but if you're able to give it a shot, that would be helpful.

@zanderso Manually setting the environment variable worked.
However, I think this can be resolved in the build steps. It looks like in toolchain\win\BUILD.gn a call to exec_script("setup_toolchain.py .... which uses the environment variable GYP_MSVS_OVERRIDE_PATH is called before exec_script("vs_toolchain.py".... which sets the environment variable if it is not already set.

@whesse @zanderso let's get this completed -- did you look at the env var issue mentioned above?

I've created a new issue for the environment variable situation, which is not a blocker for the GN build. There may also still be a few other small issues remaining with the GN build, but we should file issues for those separately as well as they come up, so I'll go ahead and close this one.


Docs: Fix js options to what is really inside js

Fixes #36936.

Plus, you may include https://github.com/twbs/bootstrap/issues/36936 too

Plus, you may include #36936 too

I think this is already taken into account in https://github.com/twbs/bootstrap/pull/37019/files#diff-eea88f447d7813cdc4b41d8d41bfd86111df371dc511a094bf8af223fe07cb00R149. Am I wrong ?

Sorry, my mistake


Limit drivetrain acceleration during teleop

Closing this in favour of running the drivetrain in open loop mode during teleop.


Does getting a key reset the TTL?

Does the process of getting a key reset the TTL for that key? I am wondering if I can use thing to keep track of JWT tokens.

getting a key does not reset it's TTL, but your could use:
let cacheKey = "foo";
let newTTL = 600;
// ...
let value = cache.get(cacheKey);
if (value != null) {
    cache.ttl(cacheKey, newTTL);
}

Thanks!


Made Shopware()->DocPath() compatible with composer installations

1. Why is this change necessary?
When used in a composer installation the method Shopware()->DocPath() would return vendor/shopware/shopware/ instead of the actual project root directory.
2. What does this change do, exactly?
The project root is now determined by dirname(dirname(dirname(__DIR__))) and will be used throughout the config. Also the Shopware()->DocPath() method now return the config value of app.rootDir.
3. Describe each step to reproduce the issue or behaviour.
Run Shopware()->DocPath() in a composer installation.
4. Please link to the relevant issues (if any).
Dunno.
5. Which documentation changes (if any) need to be made because of this PR?
Dunno.
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hi @jkrzefski,
your changes would result in the DocPath not being available in our config.php anymore.
Could you please fix this in the Composer project instead?

As @shyim pointed out in a discussion about this PR, my changes would break the usage of $this->DocPath() inside the config.php file. So now the approach is to decorate the application class inside of the composer project and leaving this implementation as it is. The new PR can be found here https://github.com/shopware/composer-project/pull/28/files


Version 0.2.x

Coverage increased (+1.4%) to 89.409% when pulling 02283ffe73a53426de3875a7ba68933c9731108b on version_0.2.x into bc527acfa76e20e91d92062e3ae8a2d4f0c01533 on master.

Coverage increased (+1.4%) to 89.409% when pulling e5e098b485b965aa96bcaf9263e1db1447c0f73b on version_0.2.x into bc527acfa76e20e91d92062e3ae8a2d4f0c01533 on master.


Force install of auxiliary files in makedist.sh

What it says on the tin.

Turns out this doesn't work as expected. I was notified the shebang was changed for the install script. RC1 shipped with #!/usr/bin/sh while normally that is #!/bin/sh. I'm assuming it has to do with me testing this on a Linux workstation(?) Something to look into for the actual 4.9.0 release.


fix: rate limit store requests and sanitize path

Fixes LGTM issues

Pull Request Test Coverage Report for Build 512359434

0 of 35 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 22.777%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ZwaveClient.js
0
1
0.0%

src/modules/ColumnFilterHelper.js
0
1
0.0%

app.js
0
33
0.0%

Totals

Change from base Build 512272671:
-0.02%

Covered Lines:
2009

Relevant Lines:
9029

💛  - Coveralls

Pull Request Test Coverage Report for Build 512359434

0 of 35 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 22.777%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ZwaveClient.js
0
1
0.0%

src/modules/ColumnFilterHelper.js
0
1
0.0%

app.js
0
33
0.0%

Totals

Change from base Build 512272671:
-0.02%

Covered Lines:
2009

Relevant Lines:
9029

💛  - Coveralls


⚠ Bofrost.be has degraded performance

In f429ae9, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1459 ms

Resolved: Bofrost.be performance has improved in c53fa2d after 6 minutes.


Lines in new sidebar icons fuzzy at 1x DPI scale.

VSCode Version: 1.37.0
OS Version: macOS 10.14.5 (likely applies to all operating systems)

Steps to Reproduce:

Launch the latest version of Visual Studio Code using a display with 1x DPI scaling (non-retina).
Look at the new icons in the sidebar. Note that the lines appear rather fuzzy around the edges.

Here's what they look like at 1x scale:

And at 2x (retina) scale, as shown in the update changelog, they're much sharper in appearance:

It's clear that they were drawn at 2x scale using line widths with an odd number of pixels, specifically catered toward retina displays. On close inspection, the number appears to be three. This issue would be easy to correct simply by adding or subtracting one pixel's width from those lines, so that dividing the size by 2 would result in a whole number of pixels. Doing so would make these appear a lot nicer on monitors with 1x dpi scale, which many people still use. Even just using different vector images with one-pixel-wider lines only at lower scaling would be sufficient to create a consistent appearance across most displays.

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

This was actually intentional, we designed the icons at 16x16 with a 1px border and then scaled them up to 24x24 for the activity bar so that the stroke widths would be consistent. This causes the borders to be off which causes pixelation on non-hdpi screens. We did explore making these 1px but they were too thin and 2px was too thick so we opted to make them consistent with the rest, see https://github.com/microsoft/vscode/issues/8017#issuecomment-502809582

Well, that's just a little bit sad. I'd really rather have them "too" thin or thick than a blurry "1.5" pixels, but alright.


Package request: Nuclear

[x] I have checked that this app does not already have a pending request and has not previously been declined.
App Name: Nuclear
Download URL: https://github.com/nukeop/nuclear/releases
Website URL: https://nuclear.js.org/
App Summary: Streaming music player that finds free music for you

Core dump is fixed and two working semantic releases since yesterday give hope that it may be worth re-opening the PR


Revert "Kwxm/eutxo spec"

Reverts input-output-hk/plutus#1177

Rebasing overwrote changes in the stuff I was attempting to commit.


Update chaos-bugbounty-list.json

remove traveloka program (no longer available since a long time) URL WAS: https://www.traveloka.com/en-id/bugs

The program may be private, maybe the maintainers can add a comment option for saying the program no longer exists.

Indeed, the program IS still running, but private on bugcrowd. I think the public VDP page link has been removed for a reason, so i still think it's relevant to remove it from a repo named

public-bugbounty-programs

@nil0x42 thanks for this update - can you fix the conflicts so that we can merge this?

https://www.traveloka.com/en-en/vulnerability-disclosure-program links to https://bugcrowd.com/2b73a955-01f6-4fdf-98ac-54b55d556fc8/external/report. The program is still active.


Allow different x-axis for line charts

The x-axis need not necessarily be a time/date, it could just be any number. Would be great to make that possible!

This feature would be great, when you analyse the data it's not just time series on the x axis, looking forward for it.

Checking if this feature of supporting Line char X-axis to be non-time/date has been added yet?
Thanks

This would be as a new visualization of the nvd3.js. The x axis might be either a numeric column that must be "grouped by", or a metric that is an aggregate function. We may need a new control that represents a superset of columns and metrics for that purpose.

Notice: this issue has been closed because it has been inactive for 349 days. Feel free to comment and request for this issue to be reopened.

Please add this, it's vary constraining to have datetime as the only option for the x axis.

did this get resolved?

@pjpringle we're currently working on this - we should hopefully have this supported in Superset 1.4.

Current progress: https://github.com/apache-superset/superset-ui/pull/1356 and #16660 (first prerequisites for adding support for generic x-axis)

Hi, any updates on this? Any workaround for this maybe?


convert all return values to strings, unless?

Why can't we just convert all values returned by the tag's block to a string?
If I say
li { foo }
and foo is for example a Time it fails.

I am trying to reproduce the issue, but it seems like passing a Date as foo won't raise any exception. Would love to help if @fzingg could provide more information.
I am not sure about convert return values to string automatically, but at least we could make the exception raised more clear for debugging.

@zetachang
In fact it doesn't raise any exception for me as well but the value doesn't display at all without the to_s :
 td(width: '33%') { params.post.posttitle }
 td(width: '33%') { params.post.description }
 td(width: '33%') { params.post.created_at.to_s }

Here Posttile and description are displayed but created_at only displayed with to_s
created_at is a timestamp here but I tried with date and I have the same issue.
I pushed it on a repo :
https://github.com/fzingg/hyperloop-eventsapp

@fzingg , thanks for updating the information. Does a exception like the following help?
Exception raised while rendering #<MyComponent:0x203c>:
Instead of String or React::Element, a Time object was returned in one of child building block.

Yes, I think it would be great.

Note the better error message will be in Hyper-React 0.12, plus in Hyper-Component there will also be automatic conversion for the most common types, via the acts_as_string? flag


malloc(): Not enough space in some MiniZinc subsolvers

solver: MiniZinc's int, cp, lcg and chuffed subsolvers
CPMpy: version v0.9.9 commit e79b3afedc934a9437c2ddb3a9f54d7e2d7bd3ee
minizinc-python release version 0.7.0 commit {https://github.com/MiniZinc/minizinc-python/commit/a195cf63fcfbc98665d70ab64efb5424db25bd7e)
While using CPMpy to model a problem and some Minzinc's subsolvers to solve the problem, I found a malloc() error. This error is probably for https://github.com/MiniZinc/minizinc-python, but after advise from Ignace I'm posting it here just to be sure. The amount of constraints are minimized, but the remaining constraints can probably still smaller.
almostMinimimal.zip
from cpmpy import *

file = "almostMinimimal" # don't forget to unzip first
m = Model().from_file(file)
m.solve(solver="minizinc:lcg") # or int, cp, lcg, chuffed

bug found while working on my master thesis.

Hi Ruben,
Yes, so either our translation to MiniZinc is too naive (for example, because we do not identify sufficiently tight domains), or it is a bug in python-minizinc or actually the solver itself.
We should look at the raw minizinc text to be able to judge that. If it is on the solver or mzn side, we should file it upstream.
There is no really good way to get the minizinc textfile from a CPM_MiniZinc model I found, so I hacked it like this:
import requests
import tempfile
import os
from io import BytesIO
from zipfile import ZipFile

from cpmpy import *

url = "https://github.com/CPMpy/cpmpy/files/10056699/almostMinimimal.zip"

headers = {"User-Agent": 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.93 Safari/537.36'}
response = requests.get(url, headers=headers)
myzip = ZipFile(BytesIO(response.content))
# make temp dir
tmpdir = tempfile.mkdtemp()
myzip.extractall(tmpdir)
zipfnames = myzip.namelist()

cpm_file = os.path.join(tmpdir, zipfnames[0])
m = Model().from_file(cpm_file)
s = SolverLookup.get("minizinc", m)
print("".join(map(str, s.mzn_model._code_fragments)))

(generally applicable if you change the url)
which returns:
% Generated by CPMpy
include "globals.mzn";

var bool: BV486914;
var -2147483659..2147483636: IV10;
var 0..2147483659: IV11;
var 0..5: x_1;
var -2147483648..2147483647: IV3;
constraint xorall([false,true,true,((BV486914) \/ ((abs(IV10)) == (IV11))) /\ ((BV486914) \/ ((
    let { array[int] of var int: arr=array1d(0..5,[1,2,5,7,13,21]) } in
      arr[x_1]) == (IV3)))]);

So... the minizinc people will probably not accept this as a bug.
So the real question is: why does IV3 not have a more proper domain? e.g 1..21
That is the CPMpy bug here...

(note that it ties to #96, which is one of the multiple open points that your fuzzer so accurately identifies ; )

Thanks! This explanation helped me with a (what I now know to be the same) bug I struggled to reproduce.

The file above got created by taking a random (sub)solver and a random existing (solvable) problem executing 4 of the 40 created sat-preserving transformations on some (or all) the constraints,then test if the problem is still solve-able and extracting it if it failed via to_file.
I did find some places where I take a random argument's ".lb" without first checking if that argument had a ".lb". So it could be that I set those values to gibberish assuming Python allowed it (normally Python would crash I think). Tomorrow I will search further to see if this is what happened.


Apps & Packages Feature Exploration!

Dev Home Feature Exploration: Apps & Packages
Please Provide Feedback & Ideas!
Please let us know what you think about this feature by commenting on this issue! We'd love to hear your ideas and feedback!
Suggested New Feature: Apps & Packages
This new feature aims to create an all-in-one apps & packages management experience in Dev Home that allows you to manage your apps & package in a single, streamlined interface:

Scenario
As a developer, I want to be able to easily manage my installed apps & packages in one place so that I can quickly update, pin, uninstall, and view my apps & packages.
Additional Details
A more detailed spec is in progress and will be linked here when it is complete.
High level features include the ability to:

Batch uninstall apps & packages
Detect and apply updates for packages
Easy version control + pinning

#1983

Here are some thoughts that pop in my head after seeing this:

As you said: "This new feature aims to create an all-in-one apps & packages management experience in Dev Home that allows you to manage your apps & package in a single, streamlined interface:", will this also detect and populate packages and apps that are not dev related? From the screenshots I see Edge Dev listed. If it does not, how is it differentiating between dev and non-dev packages and apps.
I install many apps from winget that may not be dev related, will they show up here?
Will I be able to manually add a package here? If it is already installed, in theory it should be there in the list but if not, can I add it? If it is not already installed, will this serve as an interface to let me install apps through winget? Also while giving me the ability to view the code/command it will use to do it?
I dont see the ability to generate winget configuration files. This is something that Dev Home lacks and this particular GUI looks like a viable option to have that option. Importing, exporting, visualizing, editing winget configurations should be an option in dev home.
For all the packages installed, it should also be able to show where they are installed on the system. While at it, for installing winget packages through this interface, having options to install packages through a url, zip, or any other install/download option that winget supports would be very useful!

Here are some thoughts that pop in my head after seeing this:

As you said: "This new feature aims to create an all-in-one apps & packages management experience in Dev Home that allows you to manage your apps & package in a single, streamlined interface:", will this also detect and populate packages and apps that are not dev related? From the screenshots I see Edge Dev listed. If it does not, how is it differentiating between dev and non-dev packages and apps.
I install many apps from winget that may not be dev related, will they show up here?

To me it looks like it's just a GUI for winget and as far as I know there are no package categories in winget. So filtering just dev related packages would require maintaining some kind of a whitelist in Dev Home.

As already mentioned, this seems to be a good WinGet GUI approach. However, I think the extension of being able to directly see the current and newest version of a package as well as its source and having an update button is something that could be quite useful for regular users to have in the Windows settings as well. Dev Home could then offer a list of Apps & Packages that only contains the "dev packages" and similar items.

I dont see the ability to generate winget configuration files. This is something that Dev Home lacks and this particular GUI looks like a viable option to have that option. Importing, exporting, visualizing, editing winget configurations should be an option in dev home.

FYI: there's now a script now over at winget-create repo to help you create a winget configuration file. Currently it's very limited but it's a start.

For all the packages installed, it should also be able to show where they are installed on the system and where they came from by hovering over the branches icon under Version and Status column (using winget metadata I guess...?). While at it, for installing winget packages through this interface, having options to install packages through a url, zip, or any other install/download option that winget supports would be very useful!

For portable packages, I imagine this should be possible through winget's metadata since WinGet acts as a logical installer for those packages and dictates where it should be put in the system & Registry. For other installers, it may not be a simple task. I'm thinking this would depend on

https://github.com/microsoft/winget-cli/issues/2298

Use case
I don't want to install the package right away, but download its installer for a manual installation later.
Proposal
I would also like to see support for just downloading the version instead of installing it. WinGet has support for downloading packages via the winget download command that can be leveraged here.

Use case
Some packages I know don't behave well in a silent installation. It may default to installation in another location while I perform installations for packages in another drive or custom location.
Proposal
Some installers expose the ability to give a custom install location via the CLI args and that can be accessed through InstallerSwitches: Location field in a winget manifest. DevHome can expose a default install location settings field that users can set and if the installer honors the location switch, the custom location can be passed in. There should also be a way to select an install mode i.e., silent, silentWithProgress or interactive for a package.

I would also like the ability to edit the data of the applications displayed in the list.
Use Case: An application doesn’t write the DisplayVersion into registry, making it show up as Unknown
Proposal: Have an edit button per app entry where the package properties such as DisplayVersion can be updated. Updating the fields would update the registry values. Requires a user to confirm the changes they make could affect the ability to correlate packages or check for updates

You definitely don't need to do that, You can use Widgets for UniGetUI and UniGetUI

@mdanish-kh - really like this idea.


Set runner.InitRequest.Upgrade to false by default

Prevents terraform runner to upgrade all providers, and will respect the definition of the terraform lock file if any. If required, this could be parametrized, so that the user can decide whether to enable upgrade option or not.

@hmonsalv thank you for this PR.
Would you like to try adding a new field to the CR?
Maybe like this:
kind: Terraform
spec:
  upgrade: false # default to true

The new upgrade flag would still be true by default to retrain compatibility. We would come back later for example in v1alpha3 or v1beta1 to set this value to false by default.

Here's another field, RefreshBeforeApply, which providers a flag but for the apply phase: https://github.com/weaveworks/tf-controller/blob/main/api/v1alpha2/terraform_types.go#L214

Make sense!
If many agreed, we could introduce this as a breaking change + docs by having the upgrade field default to false.
Then if users wanted the old behavior, they would go setting it to true.
@defreng would you be able to help with the docs part for this PR?

Can you add the user configurable field in the spec? As far as I see the correct behaviour would be to NOT upgrade by default. The whole purpose of the controller is to maintain consistency with the git repository (gitops) and prevent most events that would result deviation from the source.
(also docs 😆)


Shouldn't even non-optional attributes be nullable?

In machine.h.motemplate we have:
@property (nonatomic, strong<$if Relationship.optional$>, nullable<$endif$>) <$Relationship.destinationEntity.managedObjectClassName$> *<$Relationship.name$>;

i.e.:

optional relationships are nullable (makes sense!)
non-optional relationships are not nullable (does this make sense?)

In actuality, during deletion and tear down, even optional relationships get niled, and this can even be encountered during KVO observation. I have been bitten in the past assuming non-optional relationships are never nil.
Of course I can just use my own template, but we could consider changing the default template too.
Thanks for any thoughts on this...

even optional relationships get niled

You mean non- optional here?
This is similar to the Garbage Collection Object Resurrection issue. The common case is you want non-optional to be non-nullable, expect for deletion, right?
For deletion, it may make sense to bypass the setters.

You mean non- optional here?

Yes, sorry, that made things more confusing!
I don't have an exact example on hand, but I'm pretty sure I recall being surprised encountering required relationships being nil, probably during undo, redo, KVO, faulting, or something like that.
I've just tried Xcode's own ManagedObject codegen, and it seems to make all relationships nullable, be they optional or required relationships.


Can't load 2nd loading screen

When i open the engine, the second loading screen after the mod loading doesn't load:
https://user-images.githubusercontent.com/94700814/174452647-27f10505-3d57-4c48-afab-82dcae26889b.mp4

I know, uploading a hotfix rn

@this-is-bennyk Thanks !


feat: Allow menu.url to accept dynamic urls and text

Current Signature
url(text: MaybeDynamicString<C>, url: string)

Proposed Signature
url(text: MaybeDynamicString<C>, url: MaybeDynamicString<C>)

@KnorpelSenf I notice that most of the functions takes the form func(text, ...middlewares). Will this feature require a rewrite of the core?

@KnorpelSenf I notice that most of the functions takes the form func(text, ...middlewares). Will this feature require a rewrite of the core?

No. All we need to do is to also resolve url properties from the buttons, rather than just resolving the text properties.

This needs to be done at https://github.com/grammyjs/menu/blob/a9818bfb35780ed5834c81e521b831753a92789e/src/menu.ts#L768 by calling uniform also for the URL.

@yohanesyuen do you want to contribute back the changes?

It seems like no one is willing to work on this. Note that you can already solve this problem using dynamic ranges. If anyone is willing to pick it up again, feel free to leave a comment. Closing.


Could not load package...

Hi, I was testing this out for the first time on my mac 10.9 and I got a package error for drush.
Could not load package drush/drush in http://packagist.org: [UnexpectedValueException] Could not parse version constraint ^2.6.3: Invalid version string "^2.6.3"

I changed the version from "dev-master" to "master" and it seemed to work, but now there is another package issue.
Could not load package bilge/doctrine-module in http://packagist.org: [UnexpectedValueException] Could not parse version constraint ^0.8: Invalid version string "^0.8"

What am I doing wrong or what is missing? I used the generator-gadget to build the source files.
Thanks

Hi @kalevitan,
This sounds like a known issue if you have an older version of Composer installed. Try resolving this by updating Composer by running: composer self-update

That solved it, @arithmetric. Thanks


QG4 System Team checks Release 3.1.0

QG checks
Please keep this issue open until QG 4 is concluded and will be managed by the Issue Creator!
We will inform you about finding and proposals in separated issues, this issue here is for the Overview of the Checks!
Product Name: SLDT Digital Twin Registry
Product Owner: Thomas Henn
Dev SPOC: @tunacicek @LuLeRoemer
Helm Chart Version: 0.2.13
App Version: 0.2.0-M9-multi-tenancy
QG4 Approval: yes/no
TRG 1 Documentation
TRG 1.01
https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-1

[x] an appropriate README.md file

TRG 1.02
https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-2

[ ] an appropriate INSTALL.md #92

TRG 1.03
https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-3

[x] an appropriate CHANGELOG.md

[x] the versioning itself follows semantic versioning

TRG 2 Git
TRG 2.01
https://eclipse-tractusx.github.io/docs/release/trg-2/trg-2-1

[x] default branch is named main

TRG 2.03
https://eclipse-tractusx.github.io/docs/release/trg-2/trg-2-3

[x] /docs directory
[ ] must contain detailed product related documentation for the Tractus-X product #93
[x] /charts directory
[x] must contain the Helm chart for the Tractus-X product IF available
[x] AUTHORS.md file (optional)
[ ] #106
[x] CONTRIBUTING.md file
[x] DEPENDENCIES file(s)
[x] run Dash Tool to update Dependencies File #102
[x] LICENSE file
[x] NOTICE.md file
[x] SECURITY.md file

TRG 2.04
https://eclipse-tractusx.github.io/docs/release/trg-2/trg-2-4

[x] a Leading product repository
[x] repository name must be productname without prefix or suffix
[x] should contain the release
[x] references/urls to the product's other repositories
[x] might contain product helm chart
[x] README.md: contains the urls for the backend and frontend applications

TRG 2.05
https://eclipse-tractusx.github.io/docs/release/trg-2/trg-2-5

[ ] .tractusx metafile is present in the root of the repository #94
[ ] file has a proper format

TRG 3 Kubernetes
TRG 3.02
https://eclipse-tractusx.github.io/docs/release/trg-3/trg-3-2

[ ] if data persistence is needed in Kubernetes the use of PersistentVolume and PersistentVolumeClaim resource #95

TRG 4 container
TRG 4.01
https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-1

[x] All images must be tagged following semantic versioning
[x] container is labeled correctly additionally to the latest tag

TRG 4.02
https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-2

[x] must add a section to your top level README.md file, that contains information about the used base image
[x] Java, Kotlin, ... if JVM based language use base image from Eclipse Temurin

TRG 4.03
https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-3

[x] image has USER command to specify a non-root user to run the container
[x] deployment.yaml has runAsUser and allowPrivilegeEscalation: false properly set

TRG 4.05
https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-5

[x] released image must be present on GitHub Package registry or Dockerhub

TRG 5 Helm
TRG 5.01
https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-01

[x] Helm chart must be released
[x] appropriate semantic versioning for version and appVersion has to be used in Chart.yaml
[x] must not contain any environment specific values-xyz.yaml
[x] values.yaml file must contain proper default values/placeholders
[x] No hostname provided for ingress
[x] Ingress is disabled
[x] No references to any secret engine service (e.g.: Hashicorp Vault)
[x] Dependencies should be prefixed with the nameOverride and/or fullnameOverride properties
[x] Image tag is set to the Chart.yaml appVersion property
[x] must be deployable to any environment without overwriting default values with a simple helm install command
[x] dependencies have to be declared in Chart.yaml NOT requirements.yml
[x] recommended use GitHub Chart Releaser Action

TRG 5.02
https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-02

[x] Helm chart location inside Git repository in /charts directory
[ ] chart file structure #101

charts/
    chartNameA/
      Chart.yaml
      ...
    chartNameB/
      Chart.yaml
      ...
AUTHORS.md
DEPENDENCIES.md
LICENCE
README.md

[x] each file must contain the Apache 2.0 Licence
[x] latest tag is not used in helm chart be default

TRG 5.04
https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-04

[ ] CPU and memory limits and requests are properly set #103

TRG 5.06
https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-06

[x] every startup configuration aspect of your application must be configurable through the Helm chart (ingress class, tls, labels, annotations, database, secrets, persistence, env variables)

TRG 5.07
https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-07

[x] if dependencies are present in the Chart.yaml they are properly configured

TRG 5.08
https://eclipse-tractusx.github.io/docs/release/trg-5/trg-5-08

[x] a product has a single deployable helm chart that contains all components (backend, frontend, etc.)
[x] name of the Chart should be just the product-name without prefix or suffix
[x] values file should contain all available variables (even from subcharts) with default values and comments about what they do
[x] helm install command should successfully install the chart to any supported Kubernetes version cluster (without overwriting default values)
[x] helm test runs without errors

Testing

[x] installed and running on pre-prod without errors #100

Hints
Information Sharing

Hi @FaGru3n ,
sure.
Product Name:  Digital Twin Registry
Product Owner:  Thomas Henn
Dev SPOC:  Lukas Römer, Tunahan Cicek
Helm Chart Version:  0.2.13
App Version: 0.2.0-M9-multi-tenancy
QG4 Approval: no

Hi @LuLeRoemer and @tunacicek
little update for #106

#102 is already fixed

https://github.com/eclipse-tractusx/sldt-digital-twin-registry/issues/93 is fixed now

#94 is fixed now

#95 is not valid (check comment in issue for details)

#101 is not valid (check comment in issue for details)

https://github.com/eclipse-tractusx/sldt-digital-twin-registry/issues/103 needs further investigations and will not be closed for now

Hi @LuLeRoemer @bs-jokri
please inform us which current tag we have to test after changes
Helm Chart Version: 0.2.13 →  ?
App Version: 0.2.0-M9-multi-tenancy → ?

@FaGru3n
Chart Version: 0.2.17 (once this PR has been merged: https://github.com/eclipse-tractusx/sldt-digital-twin-registry/pull/115)
App version: 0.2.0-M12-multi-tenancy

succesfully synced on argo pre-prod


Add beacon Archive command to the CRON tab

We already have a command for archiving beacons but currently we are not running it.
The command is in class: BeaconArchiveBundleCommand
The cron tab could be adjusted by modifying: docker/crons/crontab

Hello @korzol
Could you pick this ticket?
The change should be trivial but it will require some testing.


Define temporary directory

On my linux machine my /tmp is very limited. Is there a way to define a temporary directory for unzipping files to be something else?

Digging through the code, the scripts use python's tempdir module.
By prefixing the sircel command with TMP=somedirectory the temporary directory can be defined easily


Document Pod Deletion Issue when using K8s < 1.13.0

Document Pod Deletion Issue when using K8s < 1.13.0.
Refer: https://github.com/kubernetes/kubernetes/issues/45688

https://github.com/oracle/coherence-operator/pull/152


#2755 show info about pending force settlement orders in account info

General
Closes #123
added settlement table
General
Please make sure the following is done:

Pull request is onto develop
If you haven't already, have a look at

https://github.com/bitshares/bitshares-ui/blob/develop/CODE_OF_CONDUCT.md
https://github.com/bitshares/bitshares-ui/blob/develop/CONTRIBUTING.md

Code Preparation
Please review all your changes one last time before committing

[x] Check for unused code
[ ] No unrelated changes are included
[ ] None of the changed files are reformatting only
[ ] Code is self explanatory or documented
[ ] All written text is properly translated (english language)

Testing
The branch has been tested on the following browsers (desktop and mobile view)

[x] Chrome
[ ] Opera
[ ] Firefox
[ ] Safari

User interface changes
Delete this section if there weren't any UI changes. Please make sure you tested your changes in all themes

[x] Dark
[ ] Light
[ ] Midnight

@sschiessl-bcp  Please review it

The task was to add this as a second table under open orders, not a new tab. please adjust.

The task was to add this as a second table under open orders, not a new tab. please adjust.

Something like this?

or

Forwarded


Error when using db.all()

I have data like this:

(data is an empty object) and it throwing error:

(error in file src\Main.js, line 170)

Hello, I'm closing this for inactivity (sorry I was inactive as well). If you need help, you can contact us here: https://discord.gg/uqB8kxh


static_assert the existence of an << ostream operator

Fixes #22 (maybe only partially).

Nice! is_detected for the win :smiley:
Unfortunately, the user will still see the operator<< compiler error from the stream << value line below. Could we maybe use a custom specialization/overload of prettyPrint for types without and ostream operator and just put the static_assert inside?

I don't really see the advantage of suppressing the stream << value error message, because this is a classic case of adding more code complexity just for the sake of a nicer compiler error.  Anyway, I have implemented your request using a tag dispatch.

Why does AppVeyor even run?  There is no appveyor.yml, so of course it fails immediately.

@hmenke AppVeyor works even without appveyor.yml in the repository. They provide a "control panel" with everything you could write in appveyor.yml. If it doesn't find that file it uses the "online settings".

@hmenke AppVeyor works even without appveyor.yml in the repository.

Oh, weird. I didn't know that. I now enabled the "Skip branches without appveyor.yml".

I don't really see the advantage of suppressing the stream << value error message, because this is a classic case of adding more code complexity just for the sake of a nicer compiler error.

What do you mean by "because this is a classic case of adding more code complexity just for the sake of a nicer compiler error"? That was kind of the whole point of #22. I think better error messages (for the user) are a perfectly valid reason to justify an increase in amount of code / code complexity. Error messages are part of the UX.

Anyway, I have implemented your request using a tag dispatch.

Thank you very much!


Replace Questions Remaining Alert with Progress Bar

Right now there's the "X questions remaining" alert at the top of the page when you're taking the survey, but I think a progress bar would be more appropriate.

Game Rules:

Level 1 - achieved by answering 1/3 of questions
Level 2 - achieved by answering 2/3 questions
Level 3 - achieved by answering all the questions.

Behavior:

Progress bar describes number of questions that have been answered on this level.  When a new level is reached, it resets.
Once user has reached at least level one, the current matches can be checked by clicking a submit button that appears.

Other thoughts:

Badges?
Animations?

Same rules as above, but this has more character.


[simulation] fix non-virtual time uart test

Codecov Report

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

@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
+ Coverage   50.78%   50.83%   +0.04%
==========================================
  Files          37       37
  Lines        4503     4503
==========================================
+ Hits         2287     2289       +2
+ Misses       2035     2034       -1
+ Partials      181      180       -1

Impacted Files
Coverage Δ

web/web.go
47.61% <0.00%> (-3.18%)
:arrow_down:

dispatcher/dispatcher.go
60.24% <0.00%> (+0.61%)
:arrow_up:


Add security multiparts for MIME (RFC 1847)

Add encrypted multipart subtype
Add pgp-encrypted application subtype
Add application/pgp-encrypted type
Add signed multipart subtype
Add pgp-signature application subtype
Add application/pgp-signature type

Closes #113.
updated version of #114

compared to #114 i fixed some typos and made the tests pass.
#114 also included an attempt at using two params for a mime type. This is not supported yet.
I did not know how to add it so i removed the test instead.


Login with Openshift

I was trying to deploy the operator with Code Ready Containers using this example:
https://github.com/argoproj-labs/argocd-operator/blob/master/examples/argocd-oauth.yaml
I am not seeing the "Login with Openshift" button. Is there any other setup that is required to get it to work?
Thanks!

@werne2j , you also have to follow the instructions in this blog post:
https://blog.openshift.com/openshift-authentication-integration-with-argocd/

@werne2j , I was discussing this with @jmckind today and it should work "out of the box" in version 0.0.5.

Thanks @pittar


feat(core) handle native 400 response codes

Summary
In addition to BAD_REQUEST messages sent by Kong, we should also
handle natively-generated 400 response codes.
Full changelog

handle native 400 response codes

Issues resolved
Fix #1816, #2085

Should it be added here as well:
https://github.com/Mashape/kong/blob/master/spec/fixtures/custom_nginx.template#L90

@bungle sure, yeah we should keep these in sync. Updated.


Cannot set SetConsoleCtrlHandler

Windows Terminal version (or Windows build number)
1.9.1523.0
Other Software
I wrote a console application in C++ and wanted to handle CTRL_CLOSE_EVENT.
I set my default terminal as Windows Terminal, so the app launched in it by default.
I wasn't sure why the method wasn't working and after a while I decided to try out running the program in conhost and it worked.
Here's the code:
BOOL WINAPI CtrlHandler(DWORD fdwCtrlType)
{
    switch (fdwCtrlType)
    {
    case CTRL_C_EVENT:
        printf("Ctrl-C event\n\n");
        Beep(750, 300);
        return TRUE;

    case CTRL_CLOSE_EVENT:
        Beep(600, 200);
        printf("Ctrl-Close event\n\n");
        return TRUE;

    case CTRL_BREAK_EVENT:
        Beep(900, 200);
        printf("Ctrl-Break event\n\n");
        return FALSE;

    case CTRL_LOGOFF_EVENT:
        Beep(1000, 200);
        printf("Ctrl-Logoff event\n\n");
        return FALSE;

    case CTRL_SHUTDOWN_EVENT:
        Beep(750, 500);
        printf("Ctrl-Shutdown event\n\n");
        return FALSE;

    default:
        return FALSE;
    }
}
int main()
{
    SetConsoleCtrlHandler((CtrlHandler, TRUE);
    while(1)
    {
        std::cin.get();
    }
}

I guessed this could be because Windows Terminal uses Win32 windows, but this could cause compatibility issues with existing console apps.
Steps to reproduce
Run the program and try to trigger a CTRL_C_EVENT
Expected Behavior
CtrlHandler handles the event
Actual Behavior
CtrlHandler is not invoked

Very high chance this is resolved with #10415

@blacklightpy Would you mind testing on Windows 22000.65 or higher with Windows Terminal Preview 1.10?

Sorry for the delay, I was sick for a while.
I'm now on 22000.100 and on Terminal Preview 1.10.1933.0
Still, the same result.
I noticed that I can hear a beep sound when I'm closing the window, meaning CTRL_CLOSE_EVENT is triggered.
But CTRL_C_EVENT and CTRL_BREAK_EVENT are still not triggered in the terminal. I didn't try the logoff and shutdown events.
The beep sound wasn't working the last time, but it wasn't working in conhost too then. Now it's working on both.
CTRL_CLOSE_EVENT too is working differently from what's expected. In conhost, the program exists only after the beep is performed. In Terminal however, the tab is close immediately and the beep is heard afterwards. So if there are any print statements in the control handler, they won't be seen. I tried setting 2 beeps and a print statement between them, and both beeps were still executed.

I'll look into this when I get a chance.

Okay. So far...
CTRL_BREAK_EVENT is coming in as a break if started within the Terminal, but as a CTRL_C if started from DefApp. So that one is DefApp related.
CTRL_CLOSE_EVENT seems to behave the same for both and makes the noise after the tab goes away. That slightly tracks with how tab handing works and we'd likely have to introduce a delay/ordering of sorts there to stop that one and is likely NOT DefApp related.
CTRL_C_EVENT just works in both.
Now trying to figure out how to do CTRL_LOGOFF_EVENT and CTRL_SHUTDOWN_EVENT as I think those are weird/difficult.

There's another issue with CTRL_CLOSE_EVENT related to CloseTest.exe (from our tools folder). It is not propagating to all the children...

now is 2024, Any progress on this issue? @zadjii-msft

Good point. Progress seems to have been achieved and the issue can be closed now. I'm not sure what fixed it, but it definitely beeps and boops in Windows Terminal 1.20.


CAMERA_ERROR (3): submitRequestList - cannot use a surface that wasn't configured

Hi, Iv'e tested the example app with 3 devices:
Samsung J7, Android version 8.1.0:
As soon as I click the "publish" button:
Screen freeze but the audio still working and broadcasting
Samsung 9+ , Android version 10:
As soon as I click the "publish" button:
Working but crashing after few minutes for unknown reason
Xiaomi Mi A2 , Android version 10:
As soon as I click the "publish" button:
crash with this exception:
CAMERA_ERROR (3): submitRequestList - cannot use a surface that wasn't configured
Can you provide a workaround for this issue please?

This project is a work in progress. It's buggy.
It works for me the feature/opengl-camera branch.
Thank u.


Adding phone number as first user in new group makes the message below group name appear as null in search

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

Action Performed:

Open the app
Open search and search for any geniune number from your country and click on the result to chat with it
Click on plus and click on new group
Select number as first member and add any other members in group and click on Create group
Open search and observe that it shows null below group name in search

Expected Result:
App should display first user email or phone number in search below group name as it does for email
Actual Result:
App displays 'null' as message in search below group name if first user added is a number
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

[] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

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
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/233508149-2fff72f7-19e0-485a-9a20-1dcf7ed0b980.mp4
https://user-images.githubusercontent.com/43996225/233508170-28507d60-d74b-4dac-af70-a6b18f06822b.mp4
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1681991862691079
View all open jobs on GitHub

Triggered auto assignment to @puneetlath (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

I'm not able to reproduce on prod, so closing. Reopen if you're still able to reproduce.


chore: Remove obsolete utils

Removing the no longer used utils.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Why is this not a direct revert of https://github.com/cloudscape-design/collection-hooks/commit/b37682ee59b49158403fdde7815a25e4698f3909

b37682e

Thanks, I just didn't think of it.

Did we have a dry-run of these changes?

Did we have a dry-run of these changes?

Yes


feat: use bgp router annotations from node when creating cilium node

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

When using cluster-pool, a new CiliumNode Resource is created. This also needs the BGP Control Plane Annotations if present. This PR copies the annotation from the Node to the CiliumNode at creation time.
Use BGP Control Plane annotations from Node Resource for creation of CiliumNode Resource

Hi, can someone with appropriate rights reapprove the pipelines? The commit length issue from previous checks should be fixed. Thanks!

/test

There's a few test failures here.

ConformanceGKE hit https://github.com/cilium/cilium/issues/25267
k8s-1.26-kernel-net-next hit https://github.com/cilium/cilium/issues/25119
Runtime hit a failure in running the unit tests, not sure if related to this PR or not. I've filed a new issue here to initiate discussion: https://github.com/cilium/cilium/issues/25276

/ci-gke

/test-1.26-net-next

/test-runtime

/ci-e2e

The runtime tests have failed in the same way for a second time in a row, it seems likely that there is some cause related to the code changes here. Can you take a look?
I think it should be possible to reproduce with sudo -E make tests-privileged TESTPKGS=./pkg/datapath/linux.

Ah, correction, it's a different package this time. Either way, can you see whether you observe these issues locally?

Locally, this test seems to work:
OK: 36 passed
--- PASS: Test (238.17s)
=== RUN   FuzzNodeHandler
--- PASS: FuzzNodeHandler (0.00s)
PASS
	github.com/cilium/cilium/pkg/datapath/linux	coverage: 55.0% of statements
ok  	github.com/cilium/cilium/pkg/datapath/linux	238.197s	coverage: 55.0% of statements

OK, thanks for taking a look. I've filed https://github.com/cilium/cilium/issues/25291 for that latest failure. Given that we haven't seen reports for that test failing otherwise, I'll re-run that test once more to gain further insights.

/test-runtime


[Invalid] Unwanted behavior of avoidDense (ALT post)

MC version 1.18.2
Fabric version 0.13.3
Fabric API 0.51.1
Saplanting v1.1.0
If more than one sapling drops stay close to each other, there is a chance that all of them may be auto-planted at the same tick, which causes large density. This phenomena happens especially frequently while using a mod "Falling Trees".
This issue is sent on behalf of one of the mod player named "strrricato_"

Bug fixed in Saplanting v1.1.1


Extend TinyPubSub with generics

This is perhaps not a good idea, but worth a discussion.
// Specific type, specific channel (the this parameter is the owner, good for memory management)
TinyPubSub.Subscribe<T>(this, "MyChannel", (T t) => { ... });

// Specific type, any time an event with this type is published no matter what channel
TinyPubSub.Subscribe<T>(this, (T t) => { ... });

The use case is when you want to send types arguments. The channel and type must match in order to subscribe.

In pullrequest

Fixed in PR! Thanks!


[RFR] Move webpack to example

We don't need webpack in the main project, only in the example. Moving the related dependencies to emphasize that admin-on-rest is bundle agnostic.

missing style-loader and css-loader


updated information sources for chemnitz

Due to the migration to gluon we use the meshviewer now. I also updated the image file the logo. The image without sponsors is better to use.

 Gruss zum Nischel.


SVN: Patch file containing (revision  ...) after a/ or b/ filename not working

DFetch uses Patch-ng  which seems to have troubles with revision behind the filenames:
Index: build-deb.sh
===================================================================
--- build-deb.sh        (revision 4007)
+++ build-deb.sh        (working copy)
@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/bash

 LANG=C

As a temporary workaround removing the (.*) behind the filename works for making the patch apply

May be a copy-paste issue on my side during a test :)


HTTP redirect from https://www.instagram.com/username/ to https://www.instagram.com/accounts/login/

When I attempt to scrape who have username such as "username" I see a message like
"HTTP redirect from https://www.instagram.com/username/ to https://www.instagram.com/accounts/login/" and nothing happens.
What is this exactly and how can I fix it?

@goker67 Duplicate of https://github.com/instaloader/instaloader/issues/388

When rate limit exceeded, how long does it take? How I long should I wait before another scraping attempt?

Duplicate of #388.


Is there any way to export just audio? like mp3?

Is there any way to export just audio?like mp3?
Or import audio output audio?

auto-editor example.mp4 -o out.mp3

or
auto-editor example.mp4 --export_as_audio

Thank you very much!
But I use :
auto-editor example.mp3 -o out.mp3  --frame_margin 16 --my_ffmpeg exportMediaOps  --silent_threshold 0.08 --audio_codec mp3 --audio_bitrate 192k --no_open
The --audio_bitrate is not work, still 128k. But mp4 is work fine. I just want to cut audio sometimes. Hope to fix it.


Fix dupe table rows

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

@stummsft - I was just about to approve but looks like there's a merge conflict now. Any chance you can resolve?
Thanks!

@stummsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@stummsft - Thank you!
#sign-off


helm: add Kubernetes discovery support

The K8 discovery mechanism, enabled by the druid-kubernetes-extension, relies on each pod advertising its name and namespace on the env variables POD_NAME and POD_NAMESPACE [1]. Add env variables to all deployments/statefulsets.
[1] https://druid.apache.org/docs/latest/development/extensions-core/kubernetes.html
Signed-off-by: Alejandro del Castillo<EMAIL_ADDRESS>Description
Helm deployment is not compatible with the druid-kubernetes-extension since the extension relies on each pod advertising its name and namespace on the env variables POD_NAME and POD_NAMESPACE
Add POD_NAME and POD_NAMESPACE env variables to all deployments/statefulsets.
Release note

Add K8 discovery support to helm charts.

This PR has:

[X ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ X] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ X] been tested in a test Druid cluster.

thank you for your contribution @adelcast


implement adv. search by gene, variant, variant group, user

For clarity, each of these searches should probably be on their own tab.  i.e. Evidence Items -> Genes -> Variants -> Variant groups -> Users.
@acoffman or @jmcmichael if you could stub in the basic framework for these tabs, one of us could perhaps work through setting up and testing various queries using the existing search code as an example.

Implemented gene, variant, vg advanced searches. User search can be accomplished on the Community page.


Confused about opts.fullPaths

from the readme:

opts.fullPaths disables converting module ids into numerical indexes. This is useful for preserving the original paths that a bundle was generated with.

afaict it always does this anyway - is this a bug?
I really want to remove the system paths. Explicitly setting this to false doesn't help. Is there any way?

You can now use https://www.npmjs.com/package/bundle-collapser

@callumlocke What exactly are you trying to accomplish: keep absolute paths out of the bundle? Keep any path data (even relative) out? Please create a repo with a simple reproduction and explanation of how the actual output differs from your desired output.

@jmm I don't want any paths in the bundle.
The issue is the statement I quoted from the readme seems to be wrong: it suggests that paths are usually converted into numerical indexes (unless you explicitly enable opts.fullPaths). But in my experience this isn't true, you always get string paths.

@callumlocke Ok, thanks for elaborating, I see what you mean. It must have worked that way at one point, but like you said it's obviously inaccurate now. My understanding is that, pretty much what @maxlath said, fullPaths currently toggles absolute local paths instead of relative paths (to the require()'ing file or base dir).
In any case, as @py-in-the-sky said I think the solution to your problem is bundle-collapser. Can you please take a look and post back here whether that does the trick for you?
I'm putting a documentation label on this issue to get that description straightened out.
Thanks!


[FEATURE] Possibility to stop test execution after n test failures

Is your feature request related to a problem? Please describe.
In the big and bug-prone APIs it could happen that the schemathesis output is so large it becomes useless.
Describe the solution you'd like
Add an option to limit the test failures. For example --limit-fails=5. The execution stops after 5 failures.
Describe alternatives you've considered
Interrupt execution manually
Redirect output to a file
Additional context
That would definitely help to onboard more people.

I am leaning toward the pytest's --maxfail=N option, or maybe --max-fail / --max-failures to have more consistent naming


Correct AtlasMap UI top offset to be relative (margin-top) to play well with Syndesis

syndesisio/syndesis#5579


Using username and password I get error [SharkIQ] Error with login.

Describe Your Problem:
running on a raspberry pi 4.  I use my shark username and password and I get [SharkIQ] Error with login.
I cannot get the other way to work either.
Logs:
Show the Homebridge logs here, remove any sensitive information.

Plugin Config:
Show your Homebridge config.json here, remove any sensitive information.

Screenshots:

Environment:

Plugin Version:
Homebridge Version:
Node.js Version:
NPM Version:
Operating System:

Pi 4 requires manual OAuth login. Leave the email and password field blank and follow the instructions from the readme
#25


watchStream keeps getting triggered

I would like to implement an example of how to use watchStream as per documentation.
It seams like the watchStream gets triggered repetitively, and I'm not sure what's causing that.
This is how it looks like after implementation:
https://github.com/user-attachments/assets/aef95f5e-d70b-4920-b997-c0d1a1373fcf
You may check the implementation in this branch. and you may also find the changes in this Pull Request.
Please help look into this issue and let me know if im doing anything wrong in the implementation.
Thank you.

I mean your stream issues new events, so shouldn't it rebuild then?
Not sure what you think is behaving wrongly.
Am 8. Aug. 2024, 04:31 +0100 schrieb Yuri Zagori @.***>:

I would like to implement an example of how to use watchStream as per documentation.
It seams like the watchStream gets triggered repetitively, and I'm not sure what's causing that.
This is how it looks like after implementation:
https://github.com/user-attachments/assets/aef95f5e-d70b-4920-b997-c0d1a1373fcf
You may check the implementation in this branch. and you may also find the changes in this Pull Request.
Please help look into this issue and let me know if im doing anything wrong in the implementation.
Thank you.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I mean the only thing that I don't understand is that in your linked branch you have a delay of 20s which seems not to be the case in the video.
Not sure why you think it does not behave as it should?
A better example would probably using a StreamController<List in the manager and then build the full list in the UI after watchStream

Thanks for replying @escamoteur.
The stream, i set, is supposed to emit events every 20 seconds. But, as you can see in video, the rebuilds happen faster than that (widget rebuilds every few milliseconds).
I have tested the source stream watchWeatherInCities() without use of watchStream, and the events are emitted as expected (every 20 seconds)
But when I use watchStream to subscribe to the stream, I get the behaviour in video. I was suspecting that watchStream is re-subscribing to the stream every time the widget is rebuilt, but i think that is not the case after looking at the watchStream documentation:

/// When you call [watchStream] a second time on the same Stream it will
/// return the last received data but not subscribe another time.

I have tried using StreamController at source of the stream as you've suggested, and watchStream is able to subscribe and rebuild widget only once every 20 seconds (expected behaviour 👍 ).
Real Scenario
In my real scenario, I am using Drift for Database and getting data as follow:
Stream<List<WeatherEntry>> watchWeatherInCities() => select(tableName).watch();

Then in the widget, I subscribe to the stream as follow:
class WatchStreamExample extends WatchingWidget {
  const WatchStreamExample({super.key});

  @override
  Widget build(BuildContext context) {

    // subscribe to stream of weather entries from DB
    final List<WeatherEntry>? weatherEntries =
        watchStream<WeatherController, List<WeatherEntry>?>(
      (controller) => controller.watchWeatherInCities(),
    ).data;

    print('++++> new list of entries: $weatherEntries');

    return Scaffold(/*more logic*/);
  }
}

Question
Does watch_it package allow subscribing to the stream directly as in the example above? or do I still need to use StreamController or StreamSubscription to manage the stream

It should work directly with the Stream from your Drift  Database.
I will test your initial version myself again to understand what is going on there. I m pretty sure it is something not related inside watch_it
Am 10. Aug. 2024, 06:46 +0100 schrieb Yuri Zagori @.***>:

Thanks for replying @escamoteur.
The stream, i set, is supposed to emit events every 20 seconds. But, as you can see in video, the rebuilds happen faster than that (widget rebuilds every few milliseconds).
I have tested the source stream watchWeatherInCities() without use of watchStream, and the events are emitted as expected (every 20 seconds)
But when I use watchStream to subscribe to the stream, I get the behaviour in video. I was suspecting that watchStream is re-subscribing to the stream every time the widget is rebuilt, but i think that is not the case after looking at the watchStream documentation:

/// When you call [watchStream] a second time on the same Stream it will
/// return the last received data but not subscribe another time.
I have tried using StreamController at source of the stream as you've suggested, and watchStream is able to subscribe and rebuild widget only once every 20 seconds (expected behaviour 👍 ).
Real Scenario
In my real scenario, I am using Drift for Database and getting data as follow:
Stream<List> watchWeatherInCities() => select(tableName).watch();
Then in the widget, I subscribe to the stream as follow:
class WatchStreamExample extends WatchingWidget {
const WatchStreamExample({super.key});

@override
Widget build(BuildContext context) {
// subscribe to stream of weather entries from DB
final List? weatherEntries =
watchStream<WeatherController, List?>(
(controller) => controller.watchWeatherInCities(),
).data;
print('++++> new list of entries: $weatherEntries');
return Scaffold(/more logic/);
}
}
Question
Does watch_it package allow subscribing to the stream directly as in the example above? or do I still need to use StreamController or StreamSubscription to manage the stream
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I think I know what is going on. The problem is that you call the function with the yield inside the watch Stream function. And I suspect that although Dart treats the return value as a Stream, if you call it multiple times the returned stream instance won't have the same object id so watch_it doesn't know it is the same stream and so subscribes again.
Please try to assign the return value of your yield function to a Stream property in your controller class in its constructor and watch that properly.
Alternatively assign it  to some property of your widgets state and use the target parameter of watchStream to watch that property
Am 10. Aug. 2024, 06:46 +0100 schrieb Yuri Zagori @.***>:

Thanks for replying @escamoteur.
The stream, i set, is supposed to emit events every 20 seconds. But, as you can see in video, the rebuilds happen faster than that (widget rebuilds every few milliseconds).
I have tested the source stream watchWeatherInCities() without use of watchStream, and the events are emitted as expected (every 20 seconds)
But when I use watchStream to subscribe to the stream, I get the behaviour in video. I was suspecting that watchStream is re-subscribing to the stream every time the widget is rebuilt, but i think that is not the case after looking at the watchStream documentation:

/// When you call [watchStream] a second time on the same Stream it will
/// return the last received data but not subscribe another time.
I have tried using StreamController at source of the stream as you've suggested, and watchStream is able to subscribe and rebuild widget only once every 20 seconds (expected behaviour 👍 ).
Real Scenario
In my real scenario, I am using Drift for Database and getting data as follow:
Stream<List> watchWeatherInCities() => select(tableName).watch();
Then in the widget, I subscribe to the stream as follow:
class WatchStreamExample extends WatchingWidget {
const WatchStreamExample({super.key});

@override
Widget build(BuildContext context) {
// subscribe to stream of weather entries from DB
final List? weatherEntries =
watchStream<WeatherController, List?>(
(controller) => controller.watchWeatherInCities(),
).data;
print('++++> new list of entries: $weatherEntries');
return Scaffold(/more logic/);
}
}
Question
Does watch_it package allow subscribing to the stream directly as in the example above? or do I still need to use StreamController or StreamSubscription to manage the stream
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you for looking into my code @escamoteur.
Exactly, I think what you mentioned here is the case I had:

And I suspect that although Dart treats the return value as a Stream, if you call it multiple times the returned stream instance won't have the same object id so watch_it doesn't know it is the same stream and so subscribes again.

Assigning value returned from the source stream to a property that is pre-defined in the controller class, and watching that property resolved the issue. Incorporating a StreamController is an alternative to achieve same expected behaviour.
Thanks Again.


Consider Slide In & Add root actions

Currently available options allow only to remove (slide out) a non-root branch.
Possibilities to add (aka slide in) a new subbranch (for current or selected) and to add a new root branch seem to be interesting features. The point is to handle machete file edits with plugin actions.
Scope of the task:

discuss details together :palms_up_together:
add a slide in action (for current & selected dropdown) :droplet:
add add root action (from toolbar) :hammer:
add remove root action (probably from selected dropdown only) :wastebasket:

Notes:
If branches that do not exists in the repository are not gonna be properly presented to graph table (falling to VcsNotification error) then these (slide in, add root) actions should automaticaly revert instead of causing new error.

I think we could drop the idea of roots addition/deletion because:

The toolbar got very (even too) populated
Roots deletion/addition is not considered to happen often
We now have very comfortable access to machete file edition


Enable support for workflow dispatch events

These events include the same information as pull request events if they are triggered in the context of a PR.
If they are triggered as a standalone workflow run, there will be no associated PRs to find matching reviews and they will remain unapproved.
Change-type: patch

/deploy please


Change default database to postgresql

@MathieuAA Is there anything more to do to change that?
To be merge once https://github.com/jhipster/generator-jhipster/issues/11736 is complete.
Please make sure the below checklist is followed for Pull Requests.

[ ] Checks are green
[ ] Tests are added where necessary
[ ] Documentation is added/updated where necessary
[ ] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

@agaspardcilia only this file has to be updated. And quite a lot of tests ;)

@MathieuAA I think once jhipster-core is merged into generator-jhipster, jdl should not set default values at all, should let generator-jhipster use it defaults.

@mshima 100% agreed, the future JDL/core/whatever module will have to consume defaults from a source :) Be it the generator or jhipster base

@agaspardcilia just be aware this repository is about to be discontinued.
@MathieuAA are you merging the merge PR with this repository?

I'll handle the code changes from this repo to the "super jhipster" thing (I don't have a name for it yet, working on it). As long as the CI here passes, it should be merged. However, as this PR is related to v7, I won't merge it.

@MathieuAA are you planning to create a new repository for generator-jhipster?

@mshima no, there's a PR in the generator for this: https://github.com/jhipster/generator-jhipster/pull/11694. It's very basic, but it's intended: refactorings, design decisions, etc. should be made after this. The PR doesn't even change the generator's behavior at all.

Here's the mail

Yep I remember that, I was confused about your "super jhipster"

Ah! that's the name of this thing. I know, it's a poor name but this was that or calling "the merged version" :(

@MathieuAA Yeah, I feel like this database change will be more about testing than making actual changes. 😅
@mshima Alright, I just hope I'll be done with this one before this repos is discontinued then.

@agaspardcilia Hello there! I'm tidying up the merge PR (https://github.com/jhipster/generator-jhipster/pull/11694). As this PR is quite small (tests excluded), I suggest you do another one in the generator once the merge is... merged. Or I can do the update for you. Whatever suits you best.

Closing this as JCore has been merged into generator-jhipster


Android keycode getter

Could potentially improve priority of results by logging 'time pressed' for latest keycode in input tick, but none of the existing uses Input has for keyboard_key currently benefit from distinguishing priority when keys are chorded anyway.

Thank you kindly!


/home/vsts/.bccontainerhelper/alLanguageExtension/15.0.1143290/extension/bin/linux/altool not found

Describe the issue
With the release v15.0.1143290 of the AL Language, it seems that the altool binary for Linux is missing.
/home/vsts/.bccontainerhelper/alLanguageExtension/15.0.1143290/extension/bin/linux/altool not found
In our Azure DevOps pipelines we use the New-BcNuGetPackage and Push-BcNuGetPackage on a ubuntu-latest Hosted Agent.
Scripts used to create container and cause the issue
$package = New-BcNuGetPackage -appfile $appFile
Push-BcNuGetPackage -nuGetServerUrl $nuGetServerUrl -nuGetToken $nuGetToken -bcNuGetPackage $package

Full output of scripts
Starting: Generate NuGet Package
==============================================================================
Task         : PowerShell
Description  : Run a PowerShell script on Linux, macOS, or Windows
Version      : 2.245.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/powershell
==============================================================================
Generating script.
========================== Starting Command Output ===========================
/usr/bin/pwsh -NoLogo -NoProfile -NonInteractive -Command . '/home/vsts/work/_temp/9929f2da-9f06-48f2-9642-898353220580.ps1'
**************************************
* App File = [VanRoey_<redacted>_24.0.166048.0_sandbox_24.5_be.app]

Get-AppJsonFromAppFile -appFile VanRoey_<redacted>_24.0.166048.0_sandbox_24.5_be.app
BcContainerHelper version 6.0.22
BC.HelperFunctions emits usage statistics telemetry to Microsoft
Running on Linux, PowerShell 7.4.5
Downloading AL Language Extension (Prerelease) to /home/vsts/.bccontainerhelper/alLanguageExtension/15.0.1143290.zip
using Expand-Archive
/usr/bin/chmod: cannot access '/home/vsts/.bccontainerhelper/alLanguageExtension/15.0.1143290/extension/bin/linux/altool': No such file or directory
Exception: /home/vsts/.local/share/powershell/Modules/BcContainerHelper/6.0.22/HelperFunctions.ps1:129
Line |
 129 |  …             throw "Command $command not found, you might need to inst …
     |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Command
     | /home/vsts/.bccontainerhelper/alLanguageExtension/15.0.1143290/extension/bin/linux/altool not found, you might need to install that command.

##[error]PowerShell exited with code '1'.
Finishing: Generate NuGet Package

The problem is in the pre-release version of the AL Language extension. The altool is now altool.dll (which seems to be wrong) - I will report this.
In the meantime, I have changed BcContainerHelper to use the latest version as the default instead of the pre-release.

Thank you for resolving this so quickly!
I can confirm changing to to latest version (instead of the pre-release) has resolved this issue.

Unfortunately the release of the AL Language v14.0.1161190 today, has reoccurred this issue.
Starting: Generate NuGet Package
==============================================================================
Task         : PowerShell
Description  : Run a PowerShell script on Linux, macOS, or Windows
Version      : 2.245.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/powershell
==============================================================================
Generating script.
========================== Starting Command Output ===========================
/usr/bin/pwsh -NoLogo -NoProfile -NonInteractive -Command . '/home/vsts/work/_temp/a14d4cd2-5a82-404a-8872-b8df890c8d61.ps1'
**************************************
* App File = [VanRoey_<redacted>_24.0.168095.0_sandbox_25.0_be.app]

Get-AppJsonFromAppFile -appFile VanRoey_<redacted>_24.0.168095.0_sandbox_25.0_be.app
BcContainerHelper version 6.0.24
BC.HelperFunctions emits usage statistics telemetry to Microsoft
Running on Linux, PowerShell 7.4.5
Downloading AL Language Extension to /home/vsts/.bccontainerhelper/alLanguageExtension/14.0.1161190.zip
using Expand-Archive
/usr/bin/chmod: cannot access '/home/vsts/.bccontainerhelper/alLanguageExtension/14.0.1161190/extension/bin/linux/altool': No such file or directory
Exception: /home/vsts/.local/share/powershell/Modules/BcContainerHelper/6.0.24/HelperFunctions.ps1:129
Line |
 129 |  …             throw "Command $command not found, you might need to inst …
     |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Command
     | /home/vsts/.bccontainerhelper/alLanguageExtension/14.0.1161190/extension/bin/linux/altool not found, you might need to install that command.

##[error]PowerShell exited with code '1'.
Finishing: Generate NuGet Package

Latest BcContainerHelper - 6.0.25 will be able to work with AL Language extensions with no executables.


Filter labels in project doesn't work

when I search 'abc'the images with label 'abcd' are also shown

when I search 'abc'
the images with label 'abcd' are also shown

Checked, it's already fixed. Close it


Support --grep option to only undebt the files that contain a particular string

Most of the time when I use undebt, I grep a particular string to find (then cut/pipe) the filenames I want to run on instead of running on a directory with massive number of files.
Wonder if it worthy to incorporate this feature into undebt?
For example, I normally do:
git grep <some string> | cut -d ":" -f 1 | sort | uniq > filenames.txt
cat filenames.txt | xargs -n 1 undebt -p <my_pattern> -i
It would be nice to have something like:
undebt -p <my_pattern> --grep <some string>

I can add this enhancement on the next week if no one start doing it.

@b5y That would be great! This is definitely a feature we would like to add.

@b5y may be prudent to open a PR once you've started so that others can see that you are working on it (less chance of duplicating work); then just ping us on there when it's ready (ps - @evhub thoughts on this contribution pattern?)

@ajm188 Yes, it is good idea. I will open PR soon.
I wanted to ask about assigning this issue to me, but it is possible to do only for contributors, I think.

What benefit is this going to have over git grep -l | xargs undebt ... or find ... | xargs undebt ...?  Seems odd to reinvent the grep wheel.

@ajm188 Yeah, makes sense to me. Although you can't open a PR until you actually have at least some changes on your branch.
@asottile Why use pyparsing when you can use regex? Why use a GUI when you can use the command line? The benefit is simplicity, and especially in this case, it will make it much easier for people to use Undebt in what we think is the right way to use it—that is, grepping for what files to change so Undebt doesn't have to scan every single one.

Why use pyparsing when you can use regex?
I thought the whole point of pyparsing was you weren't using regular expressions and are actually using a real parser (which is more powerful than regex).

Why use a GUI when you can use the command line?
Should we add a winforms app to undebt :D

The benefit is simplicity, and especially in this case, it will make it much easier for people to use Undebt in what we think is the right way to use it—that is, grepping for what files to change so Undebt doesn't have to scan every file.

I'd argue that adding complexity isn't really increasing simplicity.  What happens when someone says "well I want perl regular expressions" then you grow a -P argument.  What happens when someone says "well I want to filter filenames with globs" then you add some option for that.  What happens when someone says "well I want to filter filenames with regular expressions", "well I only want specifically named folders", etc. etc. etc.  Now your "simple" tool has responsibilities well beyond taking a file and performing operations on it.  Plus there are already tools (xargs, grep, git, etc.) which support all of these operations out of the box and better than you could ever reimplement.

@asottile The idea is to make it as easy as possible to do the right thing, and as hard as possible to do the wrong thing. If it's possible to grep to determine what files should be scanned, then that's the right thing to do, since it yields a huge performance increase. But right now, it's much harder than just scanning every file, since you can do that with just one command. The rationale isn't "whenever somebody requests a feature, we add it" but "whenever a feature makes it easier to do the right thing and harder to do the wrong thing." That's why we wouldn't add any of the other examples that you give, but this one is valuable.

@evhub, @asottile: I agree with that we shouldn't reinvent the wheel and also supporting grep is a pretty useful feature. Instead of making the undebt cmd more complicated, how about we provide a mechanism for people to add hooks to pre-process (or post-process). I am stealing this idea from the pre-commit hook. The goal is to have undebt load the new hooks from a config-file and a script to support the ways people use undebt. In this case, we only need to decide whether a new hook is valuable to be added into this repo, and not worrying about complicating the undebt core (that is, taking a file, a pattern, and process).

@paiweilai The idea of calling bash grep was in my mind. I will work with this feature on the next week.
Is there any IRC channel for undebt project?

@b5y Good point! I'm going to create a Gitter chatroom.

I'm actually with @asottile on this one. grep has a manpage as well as years of examples and expertise. The only thing I'd say is worth doing is grep style syntax of trailing files are inputs (so I don't have to do git grep -l | sed 's/^/-i /' | xargs undebt ..., I can just do git grep -l | xargs undebt ....
If the only value we're adding here is not having to pipe to xargs, then the only thing we're doing is reinventing the wheel.

@firba1 The problem with that is that if you have undebt -p pattern and you tack on file, it'll treat it as another pattern file since it comes after -p. The way to avoid sed 's/^/-i /' is just to do git grep -l | xargs undebt -p pattern -i, since -i, like -p, can accept multiple inputs.

@evhub if that's the main problem we're trying to solve, why not update the cli to have a -- option, like git subcommands usually do? So then undebt -p pattern -- file1 file2 ...

This might be worth moving to another issue.

@ajm188 The CLI already supports --, although it doesn't do anything since we don't have any positional arguments right now. My point is just that -i can already do everything that -- can (just replace -- with -i in your example).

Then I'd suggest changing the CLI to instead be:
undebt -p pattern [-p pattern ...] FILENAME [FILENAME ...]

This is pretty standard for most linting / rewriting tools and certainly helps with xargs as above (and would make undebt compatible to use as a pre-commit hook)

@asottile I think I do actually like that better, although it would break backwards compatibility, so we'd want to bump the version to 0.3.0.

@asottile @firba1 @ajm188 I created #27 for implementing the above change to the CLI.

After reading all these discussion, I start to agree that instead of making undebt cli more and more complex, probably a better idea is to add a "tips" section in the README/docs to show the best practices of "how to use xargs for parallel processing/directory traversal/grep string".
In this way, we don't have to reinvent wheels, but we still provide guidelines and examples to show how to use undebt efficiently.

I am going to close this issue as won't fix since we added tips in the doc #31


python38Packages.shapely: fix build

Motivation for this change
https://hydra.nixos.org/build/137025189
Things done

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

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

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

Should be fixed when 1.8.0 is released. Toblerity/Shapely#1079

@GrahamcOfBorg build python38Packages.shapely

Result of nixpkgs-review pr 114027 at 05221f18 run on aarch64-linux 1

1 package marked as broken and skipped:

python39Packages.mask-rcnn

15 packages failed to build:

python37Packages.cartopy
python37Packages.descartes
python37Packages.geopandas
python37Packages.osmnx
python38Packages.cartopy
python38Packages.descartes
python38Packages.geopandas
python38Packages.osmnx
python39Packages.aplpy
python39Packages.basemap
python39Packages.cartopy
python39Packages.descartes
python39Packages.geopandas
python39Packages.osmnx
python39Packages.spectral-cube

40 packages built:

apache-airflow (python37Packages.apache-airflow)
buku
cura
curaLulzbot
flatcam
mapproxy
python37Packages.aplpy
python37Packages.basemap
python37Packages.flask-admin
python37Packages.geoalchemy2
python37Packages.imgaug
python37Packages.labelbox
python37Packages.mask-rcnn
python37Packages.owslib
python37Packages.pyosmium
python37Packages.pyproj
python37Packages.shapely
python37Packages.spectral-cube
python37Packages.uranium
python38Packages.aplpy
python38Packages.basemap
python38Packages.flask-admin
python38Packages.geoalchemy2
python38Packages.imgaug
python38Packages.labelbox
python38Packages.mask-rcnn
python38Packages.owslib
python38Packages.pyosmium
python38Packages.pyproj
python38Packages.shapely
python38Packages.spectral-cube
python38Packages.uranium
python39Packages.flask-admin
python39Packages.geoalchemy2
python39Packages.imgaug
python39Packages.owslib
python39Packages.pyosmium
python39Packages.pyproj
python39Packages.shapely
python39Packages.uranium

1 suggestion:

warning: license-missing
Package is missing a license
Near pkgs/development/python-modules/shapely/default.nix:42:3:
   |
42 |   meta = with lib; {
   |   ^


Using DHCP Options Sets limitation on AWS

We need to document that when using DHCP Options Sets with Windows, only one domain name can be specified in the domain-name field per this doc
Excerpt below:
Important
Some Linux operating systems accept multiple domain names separated by spaces. However, other Linux operating systems and Windows treat the value as a single domain, which results in unexpected behavior. If your DHCP options set is associated with a VPC that has instances with multiple operating systems, specify only one domain name.

@maggieliu who should review this?

@luthermonson can review

Closing because https://github.com/rancher/docs/pull/2845 is merged


Websocketd Capacity

Is there any benchmark about how many connections Websocketd could handle, how many msg/s, etc?

It does depend on application that you're running. Work on writing capacity testing tool and do some profiling is in my todo,  I just don't know when I'll get there.
There is way to do something simple if your script is not requiring much input. You could write curl request that does upgrade to ws:// and reads data. Then you could realistically gnu-parallel it and see what kind of throughput rates you're going to see.

No, there isn't. I'm going to close this issue as I think it's something more appropriately implemented by an unbiased third party.


Make the jvmTarget the same as per the docs.

According to the CONTRIBUTING.md file, we should have at least the JDK 17 to compile the project.
However, the current setup is pointing to JDK 11, which prevents the project from compiling with JDK 17.

Thanks for the PR, but this requires downstream apps also upgrade to jvm target 17, which might not be everyone's choice, and there's an issue #121 before which make me back to jvm target 11.

Thanks for the PR, but this requires downstream apps also upgrade to jvm target 17, which might not be everyone's choice, and there's an issue #121 before which make me back to jvm target 11.

Got it.
Should I change this PR to update the documentation to 11 instead of 17?

Yes, PreCompose needs at least jdk 11 to develop, we can update the documentation to clarify this.


Investigate these settings in resin.properties and if they are in effect

https://github.com/TechEmpower/FrameworkBenchmarks/blob/master/toolset/setup/linux/webservers/resin/resin.properties#L114-L121
# Enable the proxy-cache - for caching static content in memory
proxy_cache_enable : true

# Sets the proxy cache memory size
# proxy_cache_size : 256m

# Enable clustered persistent sessions (for failover)
session_store : true

Will not do anything.