Cross platform database openness check

Currently, Client::new() checks if ledger is open to another client with lsof command. If not starts a clean ledger and if it is just connects. This works fine but not cross platform. Probably not work on windows.
#63 was an attempt to this problem. But it had it's problem.
There is another option: Start a server and write it's port to database. Then new clients first tries to connect to that server. If server is not responding, start a new ledger and a server. This approach also has caveats: Simple ping checks takes lots of time. This is not acceptable for multiple tests.
Changing database is another option. Redis provides in-memory database with connection information. Could be a good replacement to SQLite3.

Another possible bug: Clear database when other RPC clients are dropped but test as a whole is not finished. Meaning current check for openness is also problematic. Possible scenario on a test:

Clients starts and does their job
Clients get dropped after
New client connects to ledger
Sees that no other process is connected to database
Clears database and inevitably clears that test's previous work

This scenario can happen even with lsof and needs to be addressed.


Add Sigma.js functions to network display

Desirable additions to how we use Sigma.js on the DraCor single-play pages:

Zoom in/out button --> #137
click on node shows other nodes directly connected to focal node (distance = 1) (see "Bind callbacks to events" example on Sigma.js homepage) --> #258
relate node size to word count of a character (using fixed numbers for min and max node size) --> #259
relate node colour to gender information (male, female, unknown) --> #184
Search function for nodes (cf. here) (not for now)
Colouring of clusters/communities (cf. here) (not now)

I think we should create separate issues for the items in the desirable additions list (where isn't one yet, and maybe even for the ones marked with "not now") and close this issue.

I created individual issues, closing this one.


Support deploy-to-azure button for enterprise plan

Sync code from https://github.com/Azure/azure-quickstart-templates/pull/13532
Try this button:

Deploy to Azure Spring Apps

Consumption plan
Not supported

Basic/Standard plan
Not supported

Enterprise plan

Hi @stliu , please help review this PR, thanks!


Different from torch.nn.DataParallel that will separate total batch to each gpu equally?

I missed to get a instance from DataParallel.
Changing DataParallelWithCallback(net, gpu_ids) to net = DataParallelWithCallback(net, gpu_ids) worked.


Update junit-interface, munit to 0.7.28

Updates

org.scalameta:junit-interface
org.scalameta:munit

from 0.7.27 to 0.7.28.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

labels: library-update, semver-patch

Superseded by #537.


Outgoing Contract Calls

Looks great so far @mvid, I will you do hook implementation in this PR too?

@jackzampolin we already have hooks merged in from a previous PR, and they are being called in the EventHandler. Or are you talking about something else?

🤦♂


This step doesn't produce the output I would like.

I have TFS 2015.4, it has a task in the Utility group called "Copy and Publish Build Artifacts", and it works and generally does the right thing in not making me setup separate sets of folder structures for combinations of programs that make up each artifact.
The only problem it has is when I download the artifact its added an extra level of foldering. For example, if I told the step to use

CopyRoot = $(artifact staging directory)
contents  = program1**
program3**
artifact name = "windowsApps"
Artifact type = Server

If i download the artifact to c:\temp and unzip it, program1 is at c:\temp\windowsapps\windowsapps\program1.
I'd like to get rid of that duplication of "windowsapps", as its unnecessary and just creates extra work for me to do an additional repackaging. That seems contrary to the statement  "It eliminates the most common root path for all paths matched."
Also the link heading at the top has the different versions of TFS, but clicking on the one appropriate for my version of tfs takes me to a different task entirely.  Deprecation of a feature in a point release is not something anyone would reasonably expect.

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

ID: 34301e0e-fc7e-bf7a-de61-c466756f22a3
Version Independent ID: 2595a9cf-03b2-4206-6dda-43ac1b903add
Content: Copy and publish build artifacts
Content Source: docs/build-release/tasks/utility/copy-and-publish-build-artifacts.md
Service: unspecified
GitHub Login: @erickson-doug
Microsoft Alias: alewis

@StingyJack  I'm looking at this one for additional actions.

@StingyJack
Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a few options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

@thomps - the docs took me to the wrong build task, so there is that. IDK if that's been fixed or not, I had a hard enough time finding the correct version of documentation in the first place.
It's rare to get useful MSFT advice on SO (outside of Eddie Chen and Hans Passant), and this would be for a thing that really only an insider would know.
I don't like the AZ support bot (https://developercommunity.visualstudio.com/content/problem/450140/the-azure-devops-support-bot-is-useless.html), but I may have reported this issue on devcomm as well. I wouldn't be able to locate it as devcomm can't remember all of the 500+ issues I've logged and can only show about 215 of them.
I don't use this version of TFS anymore. This is fine to remain closed, so long as it's recognized that it was indeed partially a problem with docs, and that the suggestions are not very useful.


InitAttribute(InitPosition.Top) produces empty

In a class with only one method, marking that method with [Init(InitPosition.Top)] generates an empty javascript class. I think it shouldn't generate it.
Steps To Reproduce
https://deck.net/79bf4bfa9d4e861a0fa6ae323f0db533
public class Program
{
    [Init(InitPosition.Top)]
    public static void Main()
    {
        var msg = "Hello, World!";

        Console.WriteLine(msg);
    }
}

Expected Result
/**
 * @compiler Bridge.NET 16.8.2
 */
var msg = "Hello, World!";

System.Console.WriteLine(msg);

Actual Result
var msg = "Hello, World!";

System.Console.WriteLine(msg);

/**
 * @compiler Bridge.NET 16.8.2
 */
Bridge.assembly("Demo", function ($asm, globals) {
    "use strict";

    Bridge.define("Demo.Program");
});

That's the intended behavior. This is meant to be a means to make startup commands when it is absolutely required to be done before actual Bridge code. If the method therein references to anything within the Demo space, it should be defined.
While this would be nice to have a way to produce clean and simple bridge output, there would also be so much limitation to what can be written in the code, that it will likely kill Bridge purpose at all.
I will mark this as 'wontfix' for now, but we'll be watching the community reaction to this possible feature despite the consequences.

I was just pointing it out, but It is not a big deal for me.

Thanks! Glad to know this is not really being a problem at all.


Setup Prow PR Dashboard

We should the Prow PR Dashboard to make it easier to see which PRs need attention.

Did we solve the issue?

Nope.

This is fixed now.


Question : Which format/check you are using for validating US phone/mobile numbers

Hi There,
We are using this bundle in our Symfony 2.x application from a long time and it worked for very well.
Recently, one of the customer reported that below numbers are not being recognized as valid US phone numbers.
Example :<PHONE_NUMBER>643,<PHONE_NUMBER>2,<PHONE_NUMBER>437,59169611818,555181404269, 440790375,
358440482158,<PHONE_NUMBER>6
We are curious to know what exact format/validation/checks you are using for validating US phone/mobile numbers.
Please do not consider this an issue, rather just a query.

@dbhattacharjee,
Thanks for reporting this! This is actually addressed in our documentation:
https://github.com/googlei18n/libphonenumber/blob/master/FAQ.md#where-do-we-get-information-from-to-determine-if-a-number-range-is-valid
As a result, we'll close this issue, but let us know if we may clarify the
answer and update documentation. Please consider joining the
discussion group if you'd like to discuss this further.


Wrong type when attribute name and type name clash

I stumbled upon an interesting issue: when an attribute name and type name clash inside a class definition the attribute type is wrong:
import attr

@attr.s
class AT_Base:
	a: str = attr.ib()

@attr.s
class AT_Owner:
	# attribute name and type name clash
	AT_Base: AT_Base = attr.ib()

for f in attr.fields(AT_Owner):
	print('AT', f.name, f.type)

This will output:
AT AT_Base _CountingAttr(counter=23, _default=NOTHING, repr=True, eq=True, order=True, hash=None, init=True, on_setattr=None, metadata={})

I tested the same thing using dataclasses and there the type seems to resolve correctly:
import dataclasses

@dataclasses.dataclass
class DC_Base:
	a: str

@dataclasses.dataclass
class DC_Owner:
	DC_Base: DC_Base

for f in dataclasses.fields(DC_Owner):
	print('DC', f.name, f.type)

which outputs:
DC DC_Base <class '__main__.DC_Base'>

These were tested on attrs==22.1.0 and Python 3.10.8

Interestingly...this seems to work:
import attrs

@attrs.define
class AT_Base:
	a: str

@attrs.define
class AT_Owner:
	AT_Base: AT_Base

as does:
import attr

@attr.s(auto_attribs=True)
class AT_Base:
	a: str

@attr.s(auto_attribs=True)
class AT_Owner:
	AT_Base: AT_Base

But this does not:
import attrs

@attrs.define(auto_attribs=False)
class AT_Base:
	a: str = attrs.field()

@attrs.define(auto_attribs=False)
class AT_Owner:
	AT_Base: AT_Base = attrs.field()

for f in attrs.fields(AT_Owner):
	print('AT', f.name, f.type)

auto_attribs=False (which is default in the old attr.s) APIs should ignore type annotations so I guess it gets confused by symbols of two types floating around? It's definitely not a bug, I would call it undefined behavior. :)

Indeed, updating the dataclasses examples to use dataclasses.field() does this:
``shell
Traceback (most recent call last):
File "/home/arni/dohop/libdohop/dc.py", line 12, in
class DC_Owner:
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/dataclasses.py", line 1185, in dataclass
return wrap(cls)
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/dataclasses.py", line 1176, in wrap
return _process_class(cls, init, repr, eq, order, unsafe_hash,
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/dataclasses.py", line 1094, in _process_class
str(inspect.signature(cls)).replace(' -> None', ''))
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/inspect.py", line 3207, in str
formatted = str(param)
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/inspect.py", line 2724, in str
formatannotation(self._annotation))
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/inspect.py", line 1369, in formatannotation
return repr(annotation)
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/dataclasses.py", line 283, in repr
return ('Field('
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/dataclasses.py", line 283, in repr
return ('Field('
File "/home/arni/.pyenv/versions/3.10.8/lib/python3.10/dataclasses.py", line 283, in repr
return ('Field('
[Previous line repeated 491 more times]


Design: Boss Galaga Hit Color Material

This branch includes the new Boss Galaga hit color material and logic to change the color material when the Boss Galaga has been hit once.

 Game Logic: Boss Galaga Hit System


Move to .Net Core 2.1 Preview2

.Net Core 2.1 Preview2 is released.
We should move to the new version.

Roll-forward in RTM and Self-contained Application Servicing
This can affect the creation of security patches.
Elimination of platform dependencies on libcurl
We have some workarounds for libcurl. We should review this. "On Linux, you need to deploy libcurl yourself if you want to use the old HttpClient implementation."

hmm.. Should we really use a preview release? It's one thing to test it in a branch or something, prep for changes, and test for regressions, but we shouldn't commit a Preview release of .NET Core into PowerShell's Master, IMO.

@SteveL-MSFT Ok, do we have a contract that states that we will not ship 6.1 until 2.1 is GA? What if 2.1 is delayed several months beyond the planned 6.1 GA? Will we revert and ship or will we delay release until 2.1 is final?
@iSazonov Right, no one is looking forward to getting rid of that libcurl dependency as much as I am. That doesn't mean we should merge a preview framework into master.

@markekraus We plan to have a new minor GA approx every 6 months, but we're not bound to that timeline.  So if 2.1 slips, 6.1 will slip with it.  Note that PSCore6 will not always have a hard dependency on latest dotnetcore.

@SteveL-MSFT

Note that PSCore6 will not always have a hard dependency on latest dotnetcore.

From https://blogs.msdn.microsoft.com/dotnet/2018/04/11/announcing-net-core-2-1-preview-2/ :

dotnet publish now publishes self-contained applications with a serviced runtime version. When you publish a self-contained application with the new SDK, your application will include the latest serviced runtime version known by that SDK. When you upgrade to the latest SDK, you’ll publish with the latest .NET Core runtime version. This applies for .NET Core 1.0 runtimes and later.

@iSazonov I believe what that means is that when we publish with 2.1, we would automatically get a 2.1.1 servicing release if available, not that it would automatically move to 2.2-preview1 (for example) which is what I meant by "latest dotnetcore".  Perhaps I should have said "latest dotnetcore preview".

Preview and RTM has different policy.
Previews "do not roll forward from one preview to another or from preview to RTM."
For RTM - "When you upgrade to the latest SDK, you’ll publish with the latest .NET Core runtime version. "

Correct, that means we need to explicitly move to latest Preview

Has anyone successfully made this work? I tried updating all of the references and installing the latest SDK but TypeGen ends up doing nothing and then the build dies with a sea of red.

Seems TypeGen depend cyclically on dotnet restore.

@markekraus I tried 2.1 preview2 sdk and I see the issue. We dynamically create a temporary proj file to generate powershell.inc file but the file doesn't created. I don't discover a root cause. Could you push PR from your branch? Then I'd confirm that we catch the same issue.
@SteveL-MSFT Seems April 25 is a Preview2 milestone - is someone from MSFT working on the Issue tomove to .Net Core 2.1 Preview2?

@iSazonov I'm going to try to work on this today, otherwise we'll have to wait until after PSCore6.1-Preview2 to pick up dotnetcore-2.1-Preview2 (or whatever version they have at that time)

I see rc1 in their repo.

@iSazonov I had it on a local branch and ended up deleting it when I couldn't get it to work. But that's the same issue I saw. The command to generate the powershell.inc completes without output or error btu the file is not created. If you let me know what branch in your repo I can skim the changes to see if it was the same. But it was basically changing all the version references to the new SDK and RuntimeFrameworkVersion references. I also updated all the netcoreapp2.0 to netcoreapp2.1.. not sure if that was the right thing or not.

@daxian-dbw TypeCatalogGen on this line runs, but it doesn't generate a dependency file if we build with dotnetcore 2.1-preview2.  Can you take a look?

@markekraus  See #6718

@iSazonov and @SteveL-MSFT have you had any success getting VSCode/Omnisharp to play nice with the RC preview SDK? I'm trying to work on redoing the logic for -PreserveAuthorizationOnRedirect but I'm having 0 luck getting a working development environment up and running. if you have some tips or tricks.. I'd appreciate it.
I'm seeing this in my Omnisharp output
[fail]: OmniSharp.MSBuild.ProjectLoader
        The "ResolvePackageAssets" task failed unexpectedly.
System.ArgumentOutOfRangeException: Non-negative number required.
Parameter name: value
   at System.IO.FileStream.set_Position(Int64 value)
   at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader.ReadMetadataStringTable()
   at Microsoft.NET.Build.Tasks.ResolvePackageAssets.ReadItemGroups()
   at Microsoft.NET.Build.Tasks.ResolvePackageAssets.ExecuteCore()
   at Microsoft.NET.Build.Tasks.TaskBase.Execute()
   at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
   at Microsoft.Build.BackEnd.TaskBuilder.<ExecuteInstantiatedTask>d__26.MoveNext()
[warn]: OmniSharp.MSBuild.ProjectManager
        Failed to load project file 'c:\Users\MarkLocal\Documents\github\PowerShell\src\System.Management.Automation\System.Management.Automation.csproj'.

@markekraus I had to remove all previous .Net Core SDKs before I could able to compile.

@iSazonov I can build fine. but I can't get VSCode and Omnisharp to work. I have deleted all the previous SDKs.

Hmm.. moving the repo from a deeper path to c:\git\PowerShell seems ro have fixed the problem. I saw some unlreated issue in dotnet/sdk about nupkg path lengths.. and decided to try a shorter repo path. *shrugs

:smile: DOS is still alive!

So now that the official 2.1 RC1 is out, do we need to make updates? https://blogs.msdn.microsoft.com/dotnet/2018/05/07/announcing-net-core-2-1-rc-1/

We should move to RC1-Final before Preview.3 ships.  Should just involve updating global.json as we've made changes in tests/product necessary to move to 2.1

@markekraus @SteveL-MSFT
The 2.1-rc1 SDK downloaded from the link given in the blog shows version 2.1.300-rc1-008673 with 2.1.0-rc1 Host and 2.1.0-rc1-final .NET Core rutnimes (see below). However, running Install-Dotnet -Channel release -Version 2.1.300-rc1-008673 with the same verison installs the same version of SDK but different Host/rutnimes versions.
I'm not sure what went wrong, but it seems to me on the dotnet core side ...
dotnet SDK download form the blog
C:\>C:\Users\dongbow\Downloads\dotnet\dotnet.exe --info
.NET Core SDK (reflecting any global.json):
 Version:   2.1.300-rc1-008673
 Commit:    f5e3ddbe73

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.16299
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Users\dongbow\Downloads\dotnet\sdk\2.1.300-rc1-008673\

Host (useful for support):
  Version: 2.1.0-rc1
  Commit:  eb9bc92051

.NET Core SDKs installed:
  2.1.104 [C:\Program Files\dotnet\sdk]
  2.1.300-rc1-008673 [C:\Users\dongbow\Downloads\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.0-rc1-final [C:\Users\dongbow\Downloads\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.0-rc1-final [C:\Users\dongbow\Downloads\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.0.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0-rc1 [C:\Users\dongbow\Downloads\dotnet\shared\Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

dotnet SDK downloaded with Install-Dotnet -Channel release -Version 2.1.300-rc1-008673
PS:8> C:\Users\dongbow\AppData\Local\Microsoft\dotnet\dotnet.exe --info
.NET Core SDK (reflecting any global.json):
 Version:   2.1.300-rc1-008673
 Commit:    f5e3ddbe73

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.16299
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Users\dongbow\AppData\Local\Microsoft\dotnet\sdk\2.1.300-rc1-008673\

Host (useful for support):
  Version: 2.1.0-rc1-26428-03
  Commit:  2c36b039fd

.NET Core SDKs installed:
  2.1.104 [C:\Program Files\dotnet\sdk]
  2.1.300-rc1-008673 [C:\Users\dongbow\AppData\Local\Microsoft\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.0-rc1-30682 [C:\Users\dongbow\AppData\Local\Microsoft\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.0-rc1-30682 [C:\Users\dongbow\AppData\Local\Microsoft\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.0.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0-rc1-26428-03 [C:\Users\dongbow\AppData\Local\Microsoft\dotnet\shared\Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

@daxian-dbw weird. I get the 2.1.0-rc1 [sic] host when I do Install-Dotnet -Channel release -Version 2.1.300-rc1-008673

I see the same as @daxian-dbw on macOS.  I'll reach out to the dotnet team on this.

I see rtm builds on MyGet.

I see rtm builds on MyGet.
https://dotnet.myget.org/feed/dotnet-core/package/nuget/Microsoft.NET.Sdk/2.1.300-rtm-62909-02

Dotnet team must be busy with Build as they aren't responding.  Perhaps for now we should just wait for RTM if they are starting producing those builds.

what ever happened to the 6.0.0 branch? I don't see it anymore. was it made private or moved somewhere?

@markekraus 6.0.x servicing updates are done on a private branch since they contain security updates and we don't want to disclose the changes prematurely before the bits being publicly available

Got a response now that Build is over.  They had an issue with CDN not having correct version, but that has since been fixed.  If you still see wrong info with dotnet --info, try cleaning out ~/.dotnet folder.


Add descriptions to exceptions

The repo has as dependency Flask = ">=2.0,<4.0"
The repo also uses Werkzeug (for example in the edited file)
Flask>=2.0 requires Werkzeug>=2.0
When accesing an protected route the following error appears

This is because HTTPException.description can no longer be None in werkzeug 2.x (commit)
In this PR I only add a very simple description to both exceptions to avoid the errors

Thanks!


is the library compatible with 0.18 monero node?

Hi!
I was using your lib on monero v.0.17 +  lws v.0.17 and it works fine. But after update to 0.18/0.2_0.18 submit_raw_tx not working .
E wrong variant type: N10cryptonote12txout_to_keyE, expected txout_to_tagged_key in transaction id=
Is the library compatible with 0.18 monero node?

Hi @Frazer-dl, we have updated the build scripts and  pushed out updated WASM's if you would like to give them a try.


Chef search does not work within attributes file?

If I have something I'm trying to grab from my databags in chef search, it doesn't appears attributes supports the "search" feature. I think I understand why this is happening, however some clarification would be nice.
Say I have the following:
default['foo']['bar'] = search('foo').first['bar']

I ( or think I do) understand that I am searching within the realm of 'foo' within my databags. However, what is the difference in precedence over this versus:
default['first']['bar'] = 'baz'
default['foo']['bar'] = node['first']['bar']

Are the databags only formed after the attributes file is read? If so, what is the chef-standard way of getting something from a chef search into a cookbook-wide variable?
Cheers!

The documented syntax for data bag search looks more like search(:admins, "*:*"), are you sure your search query as written even works in recipe code?
In general, though, this is bug tracker and not a place to ask questions.  Search and databags should work fine from within attributes files.  If you can turn this into a bug report it can stay open, but questions like this should really go to the mailing list, stack overflow, paid support channel, irc, etc..

The DataQuery DSL methods aren't included in Node I don't think, they are part of Recipe. You can use the underlying classes directly though.

The kinda odd thing is that i'd think it would still blow up.  It looks like it should hit method_missing equivalent to something like node.search.foo which should still trainwreck with a NoMethodError on NilClass.

Yeah, I think this question is hypothetical, you get method errors last time I tried :)

I can confirm my search syntax works in recipe code, but does not work when contained in my attributes file. I'll try to do some more digging to see if this is indeed a bug, or just something trivial I'm missing. I'll go ahead and close this for now.


Added writing full released version to a file

Prototype for writing versions of all packages when releasing.

    for (const [name] of this.repoVersions) {

We should already have all the version information.  I don't think we need to add more tracking.  All you need to do is to expand "Client" and "Server" into package names, whcih you should be able to get from the FluidRepo object.

Refers to: tools/build-tools/src/bumpVersion/versionBag.ts:223 in 73fb223. [](commit_id = 73fb22347fe44b3f72bc613a268d22e576747066, deletion_comment = False)

    for (const [name] of this.repoVersions) {

We should already have all the version information. I don't think we need to add more tracking. All you need to do is to expand "Client" and "Server" into package names, whcih you should be able to get from the FluidRepo object.
Refers to: tools/build-tools/src/bumpVersion/versionBag.ts:223 in 73fb223. [](commit_id = 73fb223, deletion_comment = False)

Sounds good. Does this look good for generating a sample file so we can test if this works for our partners?


Crashing when doing circle payments

Hi,
I've had this happen 3 or 4 times now when doing circle payments:

Traceback (most recent call last):
  File "./lndmanage.py", line 361, in <module>
    main()
  File "./lndmanage.py", line 317, in main
    dry=not args.reckless)
  File "/home/user/lndmanage/lib/rebalance.py", line 66, in rebalance_two_channels
    raise NoRouteError("Have tried this route already.")
lib.exceptions.NoRouteError: Have tried this route already.

Let me know if I can provide you with any addition info.
Thanks!
edit: I realize its because its trying the same route twice, but shouldn't the program just exit and not stackdump?

Actually I shouldn't have said "exit" above. I should have said "continue to try the next set of channels" This has been happening more and more, and if it happens at the beginning it won't continue on and try the next route and its got me stuck a couple times. I've also had some other crashes, but I haven't been keeping track. I'll try to post the next one I find.
Thanks again!

Rebalancing and circular payments haven't received a lot attention the last weeks, but once I have finished the regtest integration testing I'll get back to that.

hi there,
I think am also experiencing similiar issue:

Currently using:
LND 0.7.1
lndmanage 0.7.1
Windows 10

This should be fixed by #81 with better error reporting.


Replace hide/show func for CSS classes

Due to issue #2591 I have replaced show and hide jQuery functions because create conflict with the CSP rules by creating inline style code. With this, the content it's filtered correctly like in desktop version.

This restricts the result displayed correctly, however the article is not opened if it is tapped.

@Ein-Tim @MikeMcC399
Thanks for your quick responses! I'm glad it's working! https://github.com/corona-warn-app/cwa-website/issues/2594 will be covered in another PR, I hope to have it finished tomorrow. Have a nice evening!

Changed approved


Figure out how to protect login provider key

We were unable to do this in 2.1 due to key rotation issues.
Potential solns: Modify the scheme or make changes to the value converters to allow passing context ("the key to use")
@ajcvickers do you want me to file a bug in EF about value converter changes?

@HaoK Yes, please.


Install error in PowerShell (VS Code)

Within the integrated PowerShell of Visual Studio Code i cannot install Runtypes.
It fails with the error:
PS C:\Source\Ionic\flexApp> npm install runtypes --save
npm ERR! path C:\Source\Ionic\flexApp\node_modules\.staging\runtypes-20e0969c
npm ERR! code EPERM
npm ERR! errno -4048
npm ERR! syscall rename
npm ERR! Error: EPERM: operation not permitted, rename 'C:\Source\Ionic\flexApp\node_modules\.staging\runtypes-20e0969c' -> 'C:\Source\Ionic\flexApp\node_modules\runtypes'
npm ERR!     at Error (native)
npm ERR!  { Error: EPERM: operation not permitted, rename 'C:\Source\Ionic\flexApp\node_modules\.staging\runtypes-20e0969c' -> 'C:\Source\Ionic\flexApp\node_modules\runtypes'
npm ERR!     at Error (native)
npm ERR!   cause:
npm ERR!    { Error: EPERM: operation not permitted, rename 'C:\Source\Ionic\flexApp\node_modules\.staging\runtypes-20e0969c' -> 'C:\Source\Ionic\flexApp\node_modules\runtypes'
npm ERR!        at Error (native)
npm ERR!      errno: -4048,
npm ERR!      code: 'EPERM',
npm ERR!      syscall: 'rename',
npm ERR!      path: 'C:\\Source\\Ionic\\flexApp\\node_modules\\.staging\\runtypes-20e0969c',
npm ERR!      dest: 'C:\\Source\\Ionic\\flexApp\\node_modules\\runtypes' },
npm ERR!   stack: 'Error: EPERM: operation not permitted, rename \'C:\\Source\\Ionic\\flexApp\\node_modules\\.staging\\runtypes-20e0969c\' -> \'C:\\Source\\Ionic\\flexApp\\node_modules\\r
untypes\'\n    at Error (native)',
npm ERR!   errno: -4048,
npm ERR!   code: 'EPERM',
npm ERR!   syscall: 'rename',
npm ERR!   path: 'C:\\Source\\Ionic\\flexApp\\node_modules\\.staging\\runtypes-20e0969c',
npm ERR!   dest: 'C:\\Source\\Ionic\\flexApp\\node_modules\\runtypes',
npm ERR!   parent: 'flexapp' }
npm ERR!
npm ERR! Please try running this command again as root/Administrator.

Whereas the same command (without Admin privileges!) is running successfully in the normal Windows commandline:
C:\Source\Ionic\flexApp>npm install runtypes --save
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\runtypes\node_modules\ts-jest):
npm WARN enoent SKIPPING OPTIONAL DEPENDENCY: ENOENT: no such file or directory, rename 'C:\Source\Ionic\flexApp\node_modules\runtypes\node_modules\ts-jest' -> 'C:\Source\Ionic\flexApp\node_modules\runtypes\node_modules\.ts-jest.DELETE'

+<EMAIL_ADDRESS>added 427 packages in 83.487s

Any Idea whats going wrong in Powershell within VS Code?

I can't imagine that this is specific to Runtypes. Do you have similar issues installing other libraries from within VS Code?

Hmm, you're right, tested it once again on another computer, there was no problem.
But i ran into this issue several times on different computers, but only during installing runtypes.
Possibly project directory not in users directory could cause this..
(so i close this issue as it seems indeed not to be specific to runtypes..)


Fix documentation for ssd_hc() and ssd_hp()

ssd_hc() and ssd_hp() are missing descriptions of arguments.
See for example
https://bcgov.github.io/ssdtools/reference/ssd_hc.html

As part of this add documentation indicating that delta = 7 selected from Burnham and Anderson Model Selection and Multi-Model Inference


Recommended Image Crop and Resize Plugin

Can you recommend the best Image Crop and resize plugin to use with the CKEditor?
Thank you.

You can use the build in functions of the file browser
https://camo.githubusercontent.com/d8a05dcd2f53f2f0867ea7ac9e714b637d202914/687474703a2f2f7777772e77617463686572736e65742e64652f506f7274616c732f302f73637265656e73686f74732f646e6e2f434b456469746f72496d616765456469746f722e6a7067
https://camo.githubusercontent.com/728a0639b88d931b1fa41c27b5de1902a4bfe27f/687474703a2f2f7777772e77617463686572736e65742e64652f506f7274616c732f302f73637265656e73686f74732f646e6e2f434b456469746f72496d616765456469746f72322e6a7067

How do i install this plugin and configure this to work with the CkEditor?

Open the editor
click on the image icon in the editor toolbar.
Open the file browser via "Browse Server"
Select the image you want to modiy
Click on the Edit image icon in the browser and it opens the resize modus as seen in the screenshot above.

Thank you.  Are there any other plugins from this listing that are also compatible with your provider?
https://ckeditor.com/cke4/addons/plugins/all

Technically all plug-ins can be added. As described here
https://github.com/w8tcha/dnnckeditor/wiki/Add-a-Custom-Plugin

Thank you.  One last question on this thread.  Can you tell me where to look in the CK Editor Documentation to know exactly what the result is of each setting on the Editor Config Tab? https://www.screencast.com/t/OOzr5Lx2
Is this the correct guide? https://docs.ckeditor.com/ckeditor4/latest/guide/index.html

Also, which Upload Image Plugin are you referencing in this build 2.01.12 https://www.screencast.com/t/jNHOpefp

On top of the config tab is a link to the settings description page
https://docs.ckeditor.com/ckeditor4/latest/api/CKEDITOR_config.html

There is online one image upload plug in, which was introduced in 4.8.0
https://ckeditor.com/cke4/addon/uploadimage
This plug in is included in the provider package

But that page links to CKeditor 3.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Ingo<EMAIL_ADDRESS>Sent: Tuesday, March 20, 2018 12:30:54 AM
To: w8tcha/dnnckeditor
Cc: SMB; Author
Subject: Re: [w8tcha/dnnckeditor] Recommended Image Crop and Resize Plugin (#21)
On top of the config tab is a link to the settings description page
https://docs.ckeditor.com/ckeditor4/latest/api/CKEDITOR_config.html
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/w8tcha/dnnckeditor/issues/21#issuecomment-374481096, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AfaWRt7G9TCtAKN_QmrDlcUkjHSb-1Ljks5tgJQOgaJpZM4Q9eoE.

But that page links to CKeditor 3.

Yes but the settings are still valid in v4. But i update the link in the next release!


Bug in Bills

[x] sort by option- name,due date
[x] filter under detail page of a particular customer is displaying results of other customers too

ISSUE FIXED !


Update unique-values-of-array.md

I think the two choices are

unique values, i.e. values that only occur once in the list,
distinct items, the list with duplicate values remove

The naming was conflicting, so I already dealt with it, but I will change it as your suggestion is better than the naming right now (you can PR it if you like, too).


Sample code doesn't work

Sample code does not work. I do not find instruction how to setup folder path for given code.

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

ID: 2d1c05a5-3479-dea1-9d9b-23da4286242e
Version Independent ID: 2d1c05a5-3479-dea1-9d9b-23da4286242e
Content: Run your first ETL workload on Azure Databricks - Azure Databricks
Content Source: databricks/getting-started/etl-quick-start.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@KarthikATOS22
Thanks for your feedback! We will investigate and update as appropriate.

@KarthikATOS22
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1


Add electron builds to CI

Adds builds against Electron to CI configuration (both Travis and Appveyor).
The change to test/support/createdb.js is needed to quit electron only after the script ends,
as app.quit() kills the whole process immediately.
Please consider merging this PR and distributing resulting binaries same way as Node ones.

BTW appveyor build for node 0.10 and 0.12 started to fail somewhere this month.

Coverage remained the same at 86.312% when pulling 5a79cad5011180a0213028d9081a2cf5f0c97570 on tsufeki:electron-ci-builds into f1456b1cc96e89b82ac37b1896f23ef14aff80b6 on mapbox:master.

@tsufeki - thank you so much for this PR. This is a create contribution. It will require lots of testing to ensure it is working right - to what extent have you been able to test it? (have you published test binaries at all?)

landing in ##976


Classify heuristics into 3 buckets: paranoiac, "normal", "confident" and "experimental"

let's use a less stigma-related word for the highest level, what about something like "extremely-cautious" ?


Build status needs to be updated

Looks like the drone link you have is deprecated.

Dagnabbit.  drone.io is going away.  Now I have to find a new host.
Travis, I guess?  I dunno, I'll look for something.

Travis seems to be the best bet.

Solved with #36


Open links in external browser

Fixes: #4323
Based on #4343

Thanks, @WesleyAC, this LGTM! I tested on iOS and it works as expected.
I added one commit on top, to bump the new openLinkExternal toward more visibility/adoption. If that looks good, please merge at will.

@chrisbobbe Thanks! I added a setting as per the CZO discussion, would be good to get a review on the last commit in this stack :)

Ok, pushed a new version with defaults as discussed on CZO :)

Should be ready for another look @chrisbobbe :)

LGTM, pending that spot where we're waiting for @gnprice to weigh in, here. As I mentioned, I'd be happy enough with a small comment and no further action there, so I'd imagine we could time out waiting for Greg in a day or two. 🙂


Index Management be system-wide (rather than tenant-specific)

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I am frustrated when I need to create an index pattern that should apply to all tenants, but can only create it in tenants individually.
Describe the solution you'd like
A clear and concise description of what you want to happen.
It would be nice to manage index patterns globally rather than in a tenant-specific manner. For example, we have 10 tenants (a team per tenant), and we would like all 100+ index patterns we've created in one tenant to apply to all tenants. Rather than having to take the time to create the patterns across tenants or come up with some back-end hack, would be nice if this was a feature to begin with.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
We haven't had time to try anything else, but have asked our separate teams to use a common tenant to see all index patterns.
Additional context
Add any other context or screenshots about the feature request here.

Multi-tenancy is applicable for all kibana objects including index patterns, visualization and dashboards.
Global tenant/shared tenant is for such usecases when you want to share everything across all users/roles.
Additionally, if you do not wish to use multi-tenancy features and allow all kibana objects for everyone, you can disable multi-tenancy.  Details are documented here: https://opendistro.github.io/for-elasticsearch-docs/docs/security/access-control/multi-tenancy/#configuration

Multi-tenancy is applicable for all kibana objects including index patterns, visualization and dashboards.
Global tenant/shared tenant is for such usecases when you want to share everything across all users/roles.
Additionally, if you do not wish to use multi-tenancy features and allow all kibana objects for everyone, you can disable multi-tenancy.  Details are documented here: https://opendistro.github.io/for-elasticsearch-docs/docs/security/access-control/multi-tenancy/#configuration

@hardik-k-shah we don't want to eliminate multi-tenancy for our cluster - we have tenants who have specific permissions to see only certain indices/documents. The problem is that MOST of our tenants would be benefit from the 100+ index patterns we have written, but we want them to be able to store their own objects in their own tenants. But recreating and maintaining the same 100+ index patterns across individual tenants (which we have over 10) is not feasible. @dbbaughe thanks for the transfer, I'll take it up over there.

@hardik-k-shah we don't want to eliminate multi-tenancy for our cluster - we have tenants who have specific permissions to see only certain indices/documents. The problem is that MOST of our tenants would be benefit from the 100+ index patterns we have written, but we want them to be able to store their own objects in their own tenants. But recreating and maintaining the same 100+ index patterns across individual tenants (which we have over 10) is not feasible. @dbbaughe thanks for the transfer, I'll take it up over there.


Add instruction to install pxt CLI to readme

Just pulled it over from the pxt#running-a-target-from-localhost page

Handled in a different commit; closing


MINOR: Close timing window in SimpleAclAuthorizer startup

ZooKeeper listener for change notifications should be created before loading the ACL cache to avoid timing window if acls are modified when broker is starting up.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Seems like we should backport this.

@junrao @ijuma Thanks for the review. Yes, I was thinking of cherry-picking to older branches, but wasn't planning to create another RC since it has always been this way and no one has run into an issue.

Merged to trunk, 2.0, 1.1, 0.11.0 and 0.10.2 branches


esbuild serve not using configured outputPath, all assets missing

Command
serve
The previous version in which this bug was not present was
16.0.0-next-4 (this is a new feature)
Description
Esbuild using ng build seems to work perfectly, assets, script, css etc all end up in the configured outputPath in my angular.json, ready to go.
However using ng serve, I notice some strange behaviour:

it seems to build fine (see screenshot below)
. though the Sourcemap for "/styles.css" points to missing source files line I have no idea how to resolve.
however, when I go to http://<IP_ADDRESS>:4200/, I see a blank page. It seems that generated files have been created and being served successfully, but any assets are not being served breaking the whole initialization phase of my application.

When I look in my outputPath folder (which for me is cordova/www, so a non-standard value), it's just empty. I can't find anywhere that the files are being hosted (e.g. searching for chunk-Z53IBAJG.js on my filesystem gives me nothing). I don't see a dist folder or something

Minimal Reproduction
ng serve with "@angular/cli": "^16.0.0-next.6"
Exception or Error
No response
Your Environment
Angular CLI: 16.0.0-next.7
Node: 16.14.2
Package Manager: npm 9.6.3
OS: darwin arm64

Angular: 16.0.0-next.7
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.1600.0-next.7
@angular-devkit/build-angular      16.0.0-next.7
@angular-devkit/core               16.0.0-next.7
@angular-devkit/schematics         16.0.0-next.7
@angular/cdk                       16.0.0-next.4
@angular/flex-layout               15.0.0-beta.42
@angular/material                  16.0.0-next.4
@angular/material-moment-adapter   16.0.0-next.4
@schematics/angular                16.0.0-next.7
rxjs                               7.8.0

### Anything else relevant?

_No response_

I tried to to dug it
https://github.com/angular/angular-cli/blob/main/packages/angular_devkit/build_angular/src/builders/dev-server/vite-server.ts#L232
here req.url is set to<EMAIL_ADDRESS>where as it should be<EMAIL_ADDRESS>

I also notices
changing
{
						"glob": "**/*",
						"input": "projects/config/util/src/lib/swaggerK8s",
						"output": "/swaggerK8s"
					},
					{
						"glob": "**/*",
						"input": "node_modules/@volterra/ui-components/assets",
						"output": "/assets"
					},
                    {
                        "glob": "**/*",
                        "input": "node_modules/@volterra/bootstrap/assets/fonts/Inter",
                        "output": "/"
                    },

to
{
						"glob": "**/*",
						"input": "projects/config/util/src/lib/swaggerK8s",
						"output": "swaggerK8s" //removed / it the begining
					},
					{
						"glob": "**/*",
						"input": "node_modules/@volterra/ui-components/assets",
						"output": "assets"
					},
                    {
                        "glob": "**/*",
                        "input": "node_modules/@volterra/bootstrap/assets/fonts/Inter",
                        "output": ""
                    },

Another observation. Some of our assets have a query params to bust browser cache. example /voltuiicons.woff?bkszux these do not work now

Looks like this is fixed for me in rc0


Builds failing in github actions with `GLIBC_2.33' not found

We use the foundry-toolchain github action in our CI and it started failing recently with failures like this:
forge: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by forge)
forge: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by forge)
forge: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by forge)

Is there a problem with the nightly release?

having the same issue

Head over to https://github.com/foundry-rs/foundry/issues/3827 - the foundry guys are working through it. Seems like it's due to migration of Github Actions runners from ubuntu-20.04 to ubuntu-22.04. The short-term fix is to pin your build job to runs-on: ubuntu-22.04 . I would follow the other thread to see the long-term solution

Correct, closing and let's consolidate.


Update workflow to use Python version 3.12

🗣 Description
This pull request upgrades a GitHub Actions workflow to use the latest minor version of Python.
💭 Motivation and context
This instance will not be updated via the upstream pull request cisagov/skeleton-generic#154, so we must update it manually.
🧪 Testing
All automated tests pass.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

This PR should technically be part of the work of a Lineage PR that includes https://github.com/cisagov/skeleton-generic/pull/154. It should not be merged until such a PR exists (and is also being merged) if nothing else.

Closing this because the Python version is now populated by the python-version output from cisagov/setup-env-github-action.


feat: move forge build + abi + abi-ts to out

forge has a default out directory for build artifacts. When you forge build or forge test, everything gets built and written to this directory, including tests and scripts.
For the purposes of a clean ABI directory with matching TS types, I thought I would try to side step this by separately writing forge build output to an abi dir (via --out abi) that would also skip script (.s.sol) and test (.t.sol) files (via --skip script test).
This ended up working against us in our current CLI approach, where deploys need the systems ABI artifacts to generate an IWorld interface, and uses the configured forge output dir (via forge config) to get it. So we either had to build to both places (out and abi) or build to just abi but then be missing systems ABIs because they may not have been built to out, where forge is configured. And we can't update the foundry.toml to out = abi because that would cause forge test to put its artifacts there too.
We could work around this by updating our deploy pipeline to be configurable with a forge output dir (and fall back to forge config -> out), but then we'd still have forge build --out abi --extra-files abi ... commands littered around and that felt gross too.
So I've gone the simpler route for now and just default everything to forge build (with skips for tests/scripts when we're just deploying or building for ABI artifacts) and make sure to include the proper ABI files we need in our npm package artifacts.
This also moves src/MudTest.sol to test/MudTest.t.sol so it's included in --skip test and we don't have to manually skip it. Since this isn't used in store package and has a reference to a world, should we move this to the world package instead?

This also moves src/MudTest.sol to test/MudTest.t.sol so it's included in --skip test and we don't have to manually skip it. Since this isn't used in store package and has a reference to a world, should we move this to the world package instead?

good point, should be in world!


Document alias_priority

Documentation added for alias_priority in Fields and Model Config.
skip change file check
Change Summary

Related issue number
Removes a TODO in the docs.
Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


Preprocessing XML error

At the end of preprocessing the data and attempting to create the .fmerge file I'm given this error
Merge parsed files to generate *.merge file... Processing file: .../StageDP/rst_discourse_treebank/data/RSTtrees-WSJ-main-1.0/TEST/wsj_0602.out.text.xml Traceback (most recent call last):   File "preprocess.py", line 98, in <module>     main()   File "preprocess.py", line 93, in main     extract(fxml)   File "preprocess.py", line 19, in extract     sent_list, const_list = reader(fxml)   File "/home/blaise/Desktop/Wang-2017-RST/StageDP/src/utils/xmlreader.py", line 123, in reader     xmldoc = minidom.parse(fname)   File "/home/blaise/miniconda3/lib/python3.8/xml/dom/minidom.py", line 1959, in parse     return expatbuilder.parse(file)   File "/home/blaise/miniconda3/lib/python3.8/xml/dom/expatbuilder.py", line 911, in parse     result = builder.parseFile(fp)   File "/home/blaise/miniconda3/lib/python3.8/xml/dom/expatbuilder.py", line 207, in parseFile     parser.Parse(buffer, 0) xml.parsers.expat.ExpatError: syntax error: line 1, column 0
It appears there is an error in the creation of the xml files, could someone advise?

my solution was to change line 20 of run_corenlp.sh to
/usr/bin/java -mx2g -cp "$scriptdir/*" edu.stanford.nlp.pipeline.StanfordCoreNLP -annotators tokenize,ssplit,pos,lemma,ner,parse -file $FNAME -outputDirectory $PATH -outputFormat xml -outputExtension '.xml'


Docker image does not expose a /api endpoint

Steps to reproduce:
docker compose up

Open browser to http://localhost:8701/api OR http://localhost:8701/harness
Expected: A 200 response with JSON
Actual: A 404 response

The /api endpoint is only meant to be used by the flow-cli. The request should 404 when the app is run from source (locally of via Docker) and the default config will be returned, so you should be fine.
This isn't obvious though, and we're rethinking how this is designed so we can avoid confusion.

I've just setup the local dev wallet for the first time. The docker compose didn't work as noted, but flow dev-wallet and flow emulator seems to be happiest path here.

The API endpoint has been replaced with an all static build.


Disabling pipeline auth does not remove github webooks

Rancher versions: 2.0 master 6/26
Steps to Reproduce:

Enable pipeline
Create multiple pipelines to auto build on new PR
Disable pipeline

Results:  Webhooks are not removed from github.

+1
one can't remove webhooks. this is a deal braker when using multiple accounts on multiple clusters.

rancher/rancher:master 9/5
Webhook is being removed after repository authorization is removed.


Update README.md with high level description, useful links, license info etc.

Similar to https://github.com/open-telemetry/opentelemetry-js/blob/master/packages/opentelemetry-types/README.md
Packages:

[x] opentelemetry-core
[x] opentelemetry-types
[x] opentelemetry-scope-base
[x] opentelemetry-context-async-hooks
[x] opentelemetry-basic-tracer
[x] opentelemetry-node-tracer

I'll handle readme's for both scope packages

I can handle opentelemetry-core
Let me know if you want a proposal for opentelemetry-scope-base, opentelemetry-context-async-hooks (We can always improve them)

Closed via #134, #148 #122


Feature Request: Dashboard as entry point

Problem/Motivation
When I used Portainer with the standard docker image, it opened the dashboard first. With the Portainer Hass.io addon it starts with "home". So I have to select the primary endpoint first.
Proposed changes
I don't know if it's possible to go to the primary dashboard instantly, but it would be more convenient for most of the users (I think most hass.io users only have one endpoint).
Regards,
Nico

A user can create/delete their own endpoint, thus changing IDs of the endpoints in the URL. There is no way for me to guarantee this will keep working (once I've set it).

What about an optional config like "default_endpoint":"primary"?

I've looked at it again today. I really don't see a nice way to solve this without being "hacky".
I've decided to keep it the way it is.
Nevertheless, thank you for your suggestion! ❤


fix: update credential type logic

Signed-off-by: Junjie Gao<EMAIL_ADDRESS>

Codecov Report
Attention: Patch coverage is 85.71429% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 91.51%. Comparing base (ef0d057) to head (12427bf).

Files
Patch %
Lines

...ation.Plugin.AzureKeyVault/KeyVault/Credentials.cs
85.71%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #164      +/-   ##
==========================================
- Coverage   91.65%   91.51%   -0.14%
==========================================
  Files          17       17
  Lines         599      601       +2
  Branches       74       75       +1
==========================================
+ Hits          549      550       +1
  Misses         37       37
- Partials       13       14       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Gql gen via Github workflows

Closing for now but hoping to circle back on this per some recent Docker findings.


Defer adding editorKey until after initial render

Summary
currently, the generation of _editorKey during construction causes serverside rendering to break. This PR changes the behavior so that the _editorKey is either passed in as a prop or generated dynamically during componentDidMount if no prop was provided.
Test Plan
Included is a minimal universal/iso example which breaks before adding the subsequent commit. Upon building draft-js with the second commit, the universal rendering works if an editorKey prop is passed, or not, or whatever.
A slight gotcha here: EditorState.createEmpty() will create mismatched content states between the server rendered editor and the browser rendered one. For this reason, the example uses a dummy empty rawDraftContentState rather than using the other examples' approach of using createEmpty()
I'm 100% ok if this is the wrong approach or not good to merge, i'm opening it as an rfc mostly to get feedback or to understand if this is not a tenable approach to getting universal rendering unbroken with draft.

It's similar to #796 ))

@flarnie this can probably be closed. it's similar to #796 but keys blocks on mount via setState to maintain compat with the accessibility stuff. not sure what the status on the other pr is, but it seems to have greater mindshare and has probably been updated more frequently than this (as well as having inherited this pr's tests 😛 )

Thanks again - merged https://github.com/facebook/draft-js/pull/796 so closing this.


Unexpected error when searching for "dog"

Describe the bug
At yamz.net, I entered the search string "dog" and received the message, "An unexpected error has occured. The administrator has been notified. Sorry for the inconvenience."
To Reproduce
Steps to reproduce the behavior:
*1. Go to yamz.net
*1. Enter dog in the search string.
*1. Hit enter.
*2. Observe the error message.
Expected behavior
Search results for "dog" should be shown.
Desktop (please complete the following information):
*- OS: macOS 14.2.1
*- Browser: Google Chrome v.  123.0.6312.107

And "fod" search results are weird (implicitly empty?) but different.


x64 tools knob

VsDevCmd.bat sets the host tools to x86 by default.  With larger C++ projects--particularly with LTCG, it can sometimes be helpful to use the x64-hosted tools by passing "-host_arch=amd64" to VsDevCmd.bat.
There are other useful options for things like specifying the Windows SDK version, the platform, and the toolset.
For non-interactive use, "-startdir" should almost always be set to "none".

I added toolset_version, winsdk, and host_arch.


device reboot

lib:
ESP32_BLE
Backtrace: 0x40090c38:0x3ffd1dc0 0x40090e3b:0x3ffd1de0 0x4018315f:0x3ffd1e00 0x401831a6:0x3ffd1e20 0x40170bd3:0x3ffd1e40 0x400d4ebd:0x3ffd1e60 0x400d1a9d:0x3ffd1ec0 0x400d1b86:0x3ffd1f10 0x400d1d5f:0x3ffd1f70
device connect to ble server ,esp32 will reboot.

Use this to decode backtrace:
https://github.com/me-no-dev/EspExceptionDecoder

Its hard to say. Could you paste logs before app crash?

Try to comment this line, i think its bug in esp-idf:
WiFi.mode(WIFI_OFF);

My envionment changes to ESP-IDF.
This phnomenon disappeared.
https://github.com/espressif/arduino-esp32/blob/master/docs/esp-idf_component.md

You have to remember that arduino-esp32 is few commits behind esp-idf master.


Compilable examples

These patches add compilable examples to the MoCOCrW libary.
The examples provide an easy start for further development with the MoCOCrW library.

See #82.


feat: add withName function to File

Fixes #6474
⚠ This PR also includes @jedevc commit from f7b3a9cf7b3a9c to handle global client generation. Merged by #7492
Fixes OSS-62

Needs a changelog fyi.


Explain why Docker is needed

It's not trivial why Docker is needed, so I've added a short explanation

Great! 🤘


🐞[bug]: hover effect in footer section

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
i want to add hover effect in the footer section of social media link ( follow us ) to make it more user-friendly .The goal is to create a cohesive retro aesthetic with vintage-style fonts, colors .Hover effects on links and buttons will be added for interactivity, using subtle animations that reflect a retro vibe, such as pixelated transitions or glowing text.
Expected behavior
I wanted that when I click the icons on follow us section they should slightly pop up and hover with coordinating background colour.
Add ScreenShots

the (Follow Us) section.
What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@bhumikasinhaB it is already done , please check it.

And also I am closing this issue as completed.


Erin dev

This is intended to address :

https://github.com/UPHL-BioNGS/Cecret/issues/67
https://github.com/UPHL-BioNGS/Cecret/issues/81
https://github.com/UPHL-BioNGS/Cecret/issues/89

Just running some tests.
Currently needing :

update readme
update config template
update example files

Alright. Updated the config file and readme, and now just waiting on the tests.

The primer files aren't being read in correctly. Adjusting them and trying again.

All tests passed


ipset layer for docker

- What I did
An ipset layer for docker. Currently is only able to do create (with validation), save, delete, flush, destroy. A lot more left to do. This is to get a go ahead or feedback as to whether I can continue on.
- How I did it
Taken in parameters, verify validity, build execution output
- How to verify it
There's a sample program in the README.md which can be used to test it. you can change ipset to any script that will just echo out arguments passed to see what has been passed as arguments also.
seeking feedback, work in progress prototype

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:hackersdotmu/libnetwork.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:hackersdotmu/libnetwork.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.


Virtual keyboard

I'm on sway v1.8 using waybar v0.9.17 with the language module. Everything working fine except utilizing a wayland virtual keyboards (for example with 'wtype') causes the module to stop displaying the current keyboard layout. Switch the layouts fixes the issue.
Part of output of swaymsg -t get_inputs:
{
    "identifier": "0:0:wlr_virtual_keyboard_v1",
    "name": "wlr_virtual_keyboard_v1",
    "vendor": 0,
    "product": 0,
    "type": "keyboard",
    "repeat_delay": 600,
    "repeat_rate": 25,
    "xkb_layout_names": [
      "English (US)",
      "German"
    ],
    "xkb_active_layout_index": 0,
    "xkb_active_layout_name": "English (US)"
},

This looks related to  #1536


json log with sort_properties prints decimals for all numbers

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
The json logs when using Envoy FileAccessLog are serialized as floating numbers with decimals rather than integers.
An example is response codes which are logged as "HTTP" 200.0.
We see this when json_format_options.sort_properties: true is set (introduced in https://github.com/envoyproxy/envoy/commit/fc33e5d94f6919f6748e468b75df607bdd667e24 and enabled in istio in https://github.com/istio/istio/pull/48882).
{"bytes_received":0.0,"bytes_sent":11576.0,"duration":6.0,"method":"GET","response_code":200.0,"upstream_service_time":"6"}...

This creates inconsistent data and HTTP Response Codes should only be integers.
Ref https://github.com/envoyproxy/envoy/issues/34420.
Should sorting of the json be disabled until it is fixed in envoy?
Are there any workarounds to disable this?
Version
>= istio 1.22.0

Additional Information
https://github.com/envoyproxy/envoy/issues/34420
https://github.com/istio/istio/pull/48882

fxied by https://github.com/istio/istio/pull/51535


chore(main): release 1.2.7

:robot: I have created a release beep boop
1.2.7 (2023-09-02)
Bug Fixes

Plot Settings (98987c0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/chiba-ai-med/Landscaper/releases/tag/v1.2.7 :sunflower:


pip install takes too long and has dependency issues

Do you need to file an issue?

[x] I have searched the existing issues and this bug is not already filed.
[ ] My model is hosted on OpenAI or Azure. If not, please look at the "model providers" issue and don't file a new one here.
[ ] I believe this is a legitimate bug, not just a question. If this is a question, please use the Discussions area.

Describe the issue
Hi graphrag team, I have been trying to use graphrag by installing it from pypi. The installation takes up to 10 minutes (most times lot more time). I was finally able to get it to work locally in my virtual environment. But when I try to build a solution with it, I need to install and process data either through a bicep deployment script or as an azure function. This has been very painful process to build/debug/unit test because every install takes forever. The same applies to creating functions for querying. Please consider simplifying the dependencies and the time it takes to install graphrag.
Steps to reproduce
To reproduce, please run - pip install graphrag in VScode or any choice of editor terminal.
GraphRAG Config Used
# Paste your config here

Logs and screenshots
No response
Additional Information

GraphRAG Version: latest version or 0.3.6
Operating System: Windows
Python Version: 3.11
Related Issues:

This is just a suggestion - have you considered using containerized azure functions? Building a based image with graphrag installed would significantly reduce your development cycle.


fix(ui): fixes issue where submit btn was only loading onClick

Closes #15668
Problem
Hitting ctrl + enter in the script editor would submit the query without enabling the loading state of the button
Solution
Removed property that was only enabling the loading state of the button when the button was clicked. It seems like this was set by design so I'd love another opinion on why this was originally introduced and to see if there's a more correct solution.

[x] CHANGELOG.md updated with a link to the PR (not the Issue)

After speaking with @russorat about the original issue - it seems like it may have been addressed by a different UI modification. Put simply, this PR was meant to address an issue where users didn't have sufficient feedback to their actions whenever using the hot keys to submit queries. However, recent UI changes indicate that the data is reloading, and therefore renders these changes unnecessary


CLI

Sorry if I'm asking the question in the wrong place, but I don't really understand how to use Github. I kindly ask you to make a version for working on the command line. Or add the ability to execute commands in the current version via the console

Hi MBTYthe, NectarRcon has no plans to support CLI, If you use the command line, you can try go-rcon-cli


Release 0.2

Any chance of pushing a 0.2 crate with the sandbox_this_process() -> Result tweak?  I'm using it in https://www.github.com/Freaky/tarssh and I don't want to push a crate while it's depending on a Git repository.
The alternative is a release that proudly logs [INFO  tarssh] sandbox, enabled: dunno :)

Sure, done.
Interestingly, I lost the 0.1.0 commit, the master was 0.0.0 still…

Thanks :)


Add group quantization for whisper

What does this PR do?
Adds group quantization custom op to optimum. Adds the functionality to whisper's linear layers.
Still to do:

[ ] Automate the custom op build into setup.py
[ ] Add quantisation options into IPUConfig or whisper parallelize kwargs.

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


hotfix testing hash for best_cheng

Hotfix for a misspecified file hash.
Developers certificate of origin

[X] I certify that this contribution is covered by the MIT license as defined in our LICENSE and adheres to the Developer Certificate of Origin.

Codecov Report

Merging #716 (23d8c8b) into main (1ef8d12) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files


Web page typographic error

https://docs.microsoft.com/en-us/windows/win32/api/fwpstypes/ns-fwpstypes-fwps_filter1

Yes, I found that too.
Browser configuration: Chrome Browser, User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64;  X64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.111 Safari/537.36

up

@GrantMeStrength, this one's logged in drivers but the content is in win32. Formatting issue on a ref topic.
Feel free to open a new issue in the win32 repo and point to it from here, and I'll close this.

@HeMinzhang, It caused by unpaired tag </div>.
I've fixed the issue, but it takes few days for it to be applied to the page.

Yup, we are experiencing a small delay with the publishing pipeline.  Thanks!

@EliotSeattle, It is fixed and applied on page. Could you close this issue?


Generate ptr::null rather than zero literal

Clippy complains about this in crates that use clippy. Easy to ignore, but thought I'd fix it for everyone.

@bors-servo r+

:pushpin: Commit 430fb0e has been approved by emilio

:hourglass: Testing commit 430fb0eb236530eab68054caa03e0edee7cba477 with merge e3e6c730393f97daae93c2394d4af7bc9a5183b4...

:sunny: Test successful - status-travis
Approved by: emilio
Pushing e3e6c730393f97daae93c2394d4af7bc9a5183b4 to master...


Fix to ensure cross-platform compatibility for storing in the cache folder

This is a minor update to fix the case where ~ is not recognized as the user home directory.  Referencing the python documentation "Unlike a Unix shell, Python does not do any automatic path expansions. Functions such as expanduser() and expandvars() can be invoked explicitly when an application desires shell-like path expansion."  This fix ensures cross-platform compatability for storing and loading items in the hidden cache folder of the home directory.

@jpc I kept finding these folders named "~" in my working directory and made a fix for it with this PR

I'll test on Windows if you think it's ready.

Thanks @BBC-Esq, I think it's ready to test in Windows.  Hopefully it works seamlessly for you

Worked on windows for me.


Flush Logs After Time Period

Harvey will quickly build up hundreds or thousands of log files. Need to add some logic to flush logs after a certain date or time period? Allow the user to configure this?

Closing in favor of merging these details with #29


Remove erlang_js before releasing riak 2.2.6

There is nobody (to the best of my knowledge) using this functionality for several years now, but the dependency persists.
The dependency bundles and compiles several of it's dependencies, such as spidermonkey, nspr, and the quizzically named 'system'. These libraries are provided as standard on all contemporary Linux distributions and can be installed easily on OSX using 'brew'.
Anyway, nobody uses it, and it's currently failing to compile on Freebsd 11 (gist).
I propose we remove erlang_js from riak_kv, and take a step towards reducing needless complexity.

https://github.com/basho/erlang_js/pull/67 Should fix the readline issue on FreeBSD

Tested @Licenser patch and can confirm it works - would be nice to apply it as a stopgap measure to fix the freebsd build before beginning the work to purge erlang_js from the codebase.

How about before 3.0?
I will help with the exorcism.

It's already been removed in the develop-3.0 build of Riak we are working on.

Removed in 3.0


Fixed error in chrome packaged apps

Chrome throws errors in packaged apps if trying to use localStorage
solution inspired by pouchdb

Cool, thanks! I'll try to push an official release soon!

Thank you!

Released in 2.3.0


rebalancing and estimateCount and triggers

Did all of these:

[ ]  mention atomic increment / decrement in the top of the readme

[ ]  docstrings for functions & class

[ ]  in the backfill section make it more clear that there’s a “simple” version that just loops over old values (like I do in [the PR for mid](https://github.com/ianmacartney/mid-embeddings/pull/1))

[ ]  change type param to just key type for explicit generics? e.g.
<Key extends string, Shards extends Record<Key, number>>

so we can still capture the named shards, but also just allow <Id<"users">> instead of <Record<Id<"users">, number>>

[ ]  add subtract/inc/dec? so the .for isn’t strictly more feature-ful?

[ ]  compact function that checks for extra shards for a given name, and adds their value to a valid shard

[ ]  trigger method so you can attach it to a table with a trigger

Although for compact i did it differently: we now have a rebalance method which does compaction but also equalizes all shards, and an estimateCount method that benefits from rebalancing while also providing a way to read the count without taking a full read dependency

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

sg
On Mon, Oct 21, 2024 at 6:03 PM Lee Danilek @.***>
wrote:

@.**** commented on this pull request.
In README.md
https://github.com/get-convex/sharded-counter/pull/3#discussion_r1809706916
:

+may get a fractional count from counter.count. Suppose you have three
+shards and you counter.inc to add one. If you rebalance, each shard will have
+0.3333333333333333 and the count will become 0.9999999999999999, which might

I don't want to drop precision if you do counter.add(0.001).
And actually this precision problem happens even without estimates or
rebalancing, because floating point arithmetic isn't commutative. If it's
on one shard, (1e55-1e55)+1 = 1, but on two shards (1e55+1) + (-1e55) = 0.
If you have either large numbers or high precision like 0.33333333, you get
these precision issues.
—
Reply to this email directly, view it on GitHub
https://github.com/get-convex/sharded-counter/pull/3#discussion_r1809706916,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZQW345EZZXACHCC3PTCDZ4WP4LAVCNFSM6AAAAABQLIXUSKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGOBTGU2DSNZVGM
.
You are receiving this because your review was requested.Message ID:
@.***>


TolaSite settings fallbacks #820

Purpose
As soon as we split the Community Edition we won't need TolaSites anymore. I splitted as much as possible the code requiring TolaSites so we are prepared to do the fork faster.
Ticket
https://github.com/toladata/TolaActivity/issues/820

Coverage increased (+0.4%) to 53.325% when pulling 701a4a3d539f75d5b767eee322850f280c213b6b on Menda:tolasite-settings-cleanup into b577dd78b117f056832efe04efcedcc8deb97b9e on toladata:dev-v2.

Coverage increased (+0.4%) to 53.356% when pulling e04f69f168334066daedb905310f583b9390f6e1 on Menda:tolasite-settings-cleanup into b643a6ea5fc0ba82bd4bda405f72e611c6f73d02 on toladata:dev-v2.


TST/ERR: GH10369 read_msgpack checks argument type

To close #10369

nice - pls add a release note

Sure. In which section does this belong?

I'd call this a bug fix

@jreback tests are green.

@kawochen thanks!


Java based

Really good investigation of Thymeleaf support in Intellij IDEA

test with Spring 4 / 5
test with thymeleaf-spring4 / thymeleaf-spring5
test with MyWebAppInitializer in 2 modes
test with DispatcherMvcConfig WITH implementing WebMvcConfigurer and WITHOUT
test with ViewResolver inside WebMvcConfigurer

By mistake, sorry.


Proper capitalization of PlistBuddy

Hello! I encountered following error while trying to build this framework.
/Users/sust/dev/swift/macos/VolumeTest/VolumeTest/Carthage/Checkouts/AMCoreAudio/bin/autoincrement-buildnumber.sh: line 19: /usr/libexec/Plistbuddy: No such file or directory
This happens because I’m using case-sensitive file system.
Changing it to PlistBuddy resolved this error.

@sustavov Good catch! Merging into develop.


Mixing sync and async IO

I've just started having a play with the tokio based async Hyper on master and have a simple question. As I understand things Service.call should return a future immediately without blocking. However, the implementation of the server I'm working on has a bunch of synchronous code that interacts with a database, it looks like I should set up something like a threadpool to do that work on and return futures executing on that threadpool, is that the correct? That feels like a strange approach but I think that's probably just because there aren't yet async interfaces for things like Diesel available yet?

Yes what you describe is exactly the correct approach. If you have things that would block a thread, you should run them in a  different thread. For example, if you look at nodejs, its file operations (fs.readFile and etc) are all executed on a thread pool, and the results sent back to the event loop thread.
Closing as a usage question, and not a bug with hyper.

i want know ,how send  result to the event loop thread???


[5.8] Alleviate session race problem

This is a re submission of #29399
related issue : #18187
This is a re submission of #29399 , With the difference that, now we always send the cookie to the client, no matter what.
This PR brings help to the issue, Only (and only) in cases where the session ID is not regenerated by the other concurrent request and only the session data is modified.
Also it may save an unnecessary I/O operation, at the end of the request.
I thought a lot, but I was not able to find a better solution to cover more cases, unfortunately.

I'm not messing with session handling in a patch release.

Ok I re-created it targeting the v6.0 master branch.


need to pairing again by chiptool after ubuntu restared

Problem
<what's wrong or missing, please include any applicable:

expected behavior
commissioner and commissionee are able to interaction with each other everytime since paired.

actual behavior
it lost connection when I restarted ubuntu, I have using "sudo ./chip-tool pairing ble-thread" to pair it again for establish a connection.

steps to reproduce
just reboot ubuntu in VMware.

system configuration
ubuntu 20.04

@Leo2442926161 Did you pass --KVS arguments to the relevant applications so that their persistent storage files would go somewhere that's not /tmp?  Because if not, then an OS restart would clear /tmp and hence delete the storage, so the "I am commissioned" information would be lost.

Hi bzbarsky, thanks for your help.
how to add --KVS arguments to relevant applications?

You just do it:
chip-all-clusters-app  --KVS /home/whatever/something

Hi bzbarsky, thanks again,
I am using chiptool, there no --KVS parameter

@Leo2442926161 When you say "I restarted ubuntu", is the thing running on ubuntu chip-tool or the server side?

@bzbarsky-apple the chip-tool run on ubuntu

chip-tool supports setting TMPDIR now to the value of your choice.


Doesn't work

Pulled repo ran the commands to get running, Doesn't work. Abandonware

Hey, @Cally99.
Yeah, it could be dated, to some extent. For example, NuxtJS now recommends using fibers instead of node-sass, which has been deprecated. Though it was more like a one-time thing, for learning purposes, I'm probably making a nice update this week.
Thanks for passing by.
P.D.: Abandonware. That's a new one for me 😁

Done!


doc: recommend lake exe mk_all in the README

Instead of the find incantation.

Currently, this doesn't seem to work on Windows yet (due to issues with lake imports, see zulip).

The Zulip link in the description is broken.
Is it a good idea to do this if it is not working on windows yet? Maybe we can replace the find incantation with scripts/mk_all.sh until it gets fixed?

If I understand correctly, #13854 and #13850 will fix the script on Windows. (Help testing the second PR is welcome; I don't have a windows computer. Check out that branch, run lake exe mk_all and report if this works or not.)
Let's mark this PR as depending on them. (And I'll fix the zulip link shortly.)

bors d+
Sorry about causing the merge conflict you're about to have to resolve!

No worries - thanks for reviewing the dependent PR and this one. Since I just bors-ed the other PR, let me approve this one already.
bors r+

bors r+


Revert "OCPBUGS-8713: Create a second scheme that always registers prometheusoperatorv1 GVKs"

Reverts openshift/hypershift#2292
We will use OBO to ship hypershift metrics, so no need to register this version group, and we are seeing this error in the installer pod:
 oc logs -n open-cluster-management-agent-addon hypershift-install-job-25cr8-nnlvw
already exists: PriorityClass /hypershift-control-plane
already exists: PriorityClass /hypershift-etcd
already exists: PriorityClass /hypershift-api-critical
already exists: PriorityClass /hypershift-operator
applied Namespace /hypershift
applied ServiceAccount hypershift/operator
applied ClusterRole /hypershift-operator
applied ClusterRoleBinding /hypershift-operator
applied Role hypershift/hypershift-operator
applied RoleBinding hypershift/hypershift-operator
applied ServiceAccount hypershift/external-dns
applied ClusterRole /external-dns
applied ClusterRoleBinding /external-dns
applied Deployment hypershift/external-dns
applied Deployment hypershift/operator
applied Service hypershift/operator
applied Role hypershift/prometheus
applied RoleBinding hypershift/prometheus
Error: Incorrect version specified in apply patch. Specified patch version: monitoring.coreos.com/v1, expected: monitoring.rhobs/v1
Incorrect version specified in apply patch. Specified patch version: monitoring.coreos.com/v1, expected: monitoring.rhobs/v1

/approve
/lgtm

/cherry-pick release-4.13

/retest


chore: Add failing test for beam files with ? in them.

Related to https://github.com/ash-project/reactor/issues/75 - compiling on Windows will fail if the filename contains a question mark.

So, I've merged this, but I'm partly wondering if this is an Elixir issue, not a spark issue. Or maybe kind of both? Specifically, that shouldn't be an allowed module name in Elixir if it doesn't work on windows.

Otherwise, what should we do about this? Ensure our module names don't use question marks and replace it with QuestionMark or something?

Okay, I did that last thing :)

So, I've merged this, but I'm partly wondering if this is an Elixir issue, not a spark issue. Or maybe kind of both? Specifically, that shouldn't be an allowed module name in Elixir if it doesn't work on windows.

Yeah, I was thinking the same thing. Maybe we should raise an issue with Elixir?

Yeah, that would make sense. The trouble is the reproduction. I don't have a windows environment set up. But it should be pretty easy to make a test that does a bad defmodule and say "this will break on windows". Would you want to do that?

I don't have access to a windows machine either. Maybe @puruzio who raised the initial issue with Reactor may care to?

I can't wait for the bug report when someone is trying to define sections ending in ? and QuestionMark 🤣

lol. Well, for any future person coming to this issue:
https://media.giphy.com/media/DRlN7T0IXTAqcgpd8S/giphy.gif


Model calibration

So I know it's much more useful to interrogate models and look at uncertainty than to history match fluxes and groundwater elevations, but.... client wants old fashioned calibration. At least I can look at uncertainty before and after calibration.
This wasn't really touched on in the July course. Are there any examples of pyEMU being used for history matching? Or even a clue as to the command to do so?
Thanks,
Rebecca

Essentially, pestpp-glm and pestpp-ies are history matching (calibrating - barf) when they adjust parameters to reproduce observed states.  With ies, the “base” realization is (in theory) the unbiased, minimum error variance parameter set.  With glm, just use the final parameter estimates for the same.

OK. That's what I thought. But I still can't get my head around where to put my actual real physical (albeit uncertain and biased) measurements of groundwater head and flux. I'm trying to chuck them into pst.observation_data, but this keeps getting overwritten with model observations (was it John who started calling model outputs 'observations' to confuse 99% of all living groundwater modellers? because if it was, I'm going to have stern words with him) whenever I do anything.

Actually, just found my coding error so that it is not overwritten, but I'm still not sure if I have by field observations in the correct location.

That’s the tricky part with pest in general I think - it’s up to users to make sure the output from the model can and should be compared with the specified observation value in the control file.  It’s not easy to do from a mechanics standpoint.  And it will almost always require some kind of filtering/processing to remove unwanted signal components.  I wish I had an easier way to get this done...open to suggestions...

On a side note, and to avoid any upset stomachs, what is a more acceptable
way to say "the model was calbrated to historical records of..."? I just
can't craft a suitable sounding sentence that includes any form of "history
matching."
Sent from my phone without spell checker.
On Wed, Nov 20, 2019, 6:56 PM J Dub<EMAIL_ADDRESS>wrote:

That’s the tricky part with pest in general I think - it’s up to users to
make sure the output from the model can and should be compared with the
specified observation value in the control file. It’s not easy to do from a
mechanics standpoint. And it will almost always require some kind of
filtering/processing to remove unwanted signal components. I wish I had an
easier way to get this done...open to suggestions...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jtwhite79/pyemu/issues/109?email_source=notifications&email_token=ADSJXRDGX5UHTQ2EQOLUHN3QUX2GLA5CNFSM4JP4BWP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEYV2AY#issuecomment-556883203,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSJXRF527IP7BWOUWYT3N3QUX2GLANCNFSM4JP4BWPQ
.

Perhaps change to 'parameters were modified to minimise (or provide one of many possible model realisations that minimised) the RMSE between model outputs and historical observations.'

Someone once told me I should consult an over-imbibed homeless individual for help with my grammar so you don’t want my opinion here!

@jtwhite79 Yeah, I'm not sure. I think it is very project dependent. What I would like is to be able to drag in a number of head and flux timeseries, and for <magic> to transform it into a dataframe with the correct observation titles, aggregated (or possibly interpolated) appropriately to the model output frequency, filtered to remove all the Nans and then linked with the appropriate model observations in pst.observation_data.
I'm sure I could do that myself. It's just tedious. Is there something in common that all groundwater modellers do in this space? Surely head obs and flux obs are fairily commonly used?

Plus add some what that makes it easy to extract and aggregate flux outputs from a cbc file. - Drains, ET, etc. Which is what I am about to attempt to do (agonisingly slowly and badly)

Also, isn't it sleep time in the US?

Your favorite utility (pyemu.gw_utils.setup_hds_timeseries()) can accept a cell budget file and “text” for what component your are after.

Hurrah. Thanks. Good old hds_timeseries.

So I know it's much more useful to interrogate models and look at uncertainty than to history match fluxes and groundwater elevations, but.... client wants old fashioned calibration. At least I can look at uncertainty before and after calibration.
This wasn't really touched on in the July course. Are there any examples of pyEMU being used for history matching? Or even a clue as to the command to do so?
Thanks,
Rebecca

Hi  did you find an example or something like that, I want to calibrate a model too and I'm lost

pyEMU itself doesnt do any parameter adjustments for history matching.  Instead it sets up and post-processes PEST(_HP,++) analyses, including FOSM-based uncertainty and dataworth calcs.
https://github.com/mnfienen-usgs/GW1876/tree/fy20/activities_stpete/notebooks - there is a sequence of jupyter notebooks here that walk thru setting up and history matching a synthetic transient model.  See the notebook_order.readme to instructions


Implement User Registration and Login Functionality

Implements User Registration: Allows users to create accounts by providing necessary details such as name, email, and password.
Enables User Login: Users can now log in using their registered credentials, enhancing the security and usability of the application.

Good work brushed through the code but to ask in this scenario and Oauth would have made sense people hate oversigning ,what do u think about that just an playlist google to handle all this ,maybe convenient,that's my thought though @marvinhosea @oyamo what do u think?

Good work brushed through the code but to ask in this scenario and Oauth would have made sense people hate oversigning ,what do u think about that just an playlist google to handle all this ,maybe convenient,that's my thought though @marvinhosea @oyamo what do u think?

You are saying we use OpenID Connect to implement authn & authz?

Good work brushed through the code but to ask in this scenario and Oauth would have made sense people hate oversigning ,what do u think about that just an playlist google to handle all this ,maybe convenient,that's my thought though @marvinhosea @oyamo what do u think?

You are saying we use OpenID Connect to implement authn & authz?

yes u get me now


[WIP] Application archtitecture rewrite for easier adapter handling, auto org/repo detection

Q
A

Fixed Tickets
#358

License
MIT

Major work in progress, please only focus direct problems and typo's ;) commits need to be rebased and tests are currently broken :+1:

Screen was taken without the org/repo configured, else this NOTE is not shown.

--org and --repo options are now automatically detected when they are not provided or configured. So no more need to always use --org when your origin is not the upstream :smile:
Configuring allows to use dual workflow where your issues are on another org/repo or adapter, and speed-up the commands. (configuring is done using the core:init command which already detects sensible defaults).
The adapter can now be provided using the --repo-adapter and issue-adapter options, so even when no default is set and you are not in a repository you still use Gush!

I'm really happy with how its looking so far :metal:
Todo:

[ ] Fix tests
[x] Help command does not contain auto filled options (need to look at that, properly just extend the Help command for our own logic :) )
[x] Update issue:take to use the new --issue-org and --issue-repo options.
[ ] Clean-up old dead/deprecated code.
[ ] Lots of testing
[ ] Rebase to clean-up wip commits

Current broken tests:

[x] TemplateHelper
[ ] TableSubscriber
[ ] Template\PullRequest\Create\PullRequestCustomTemplate

I rewrote the Config handling as this has been a problem for a really long time, and also overcomplicated the testing. Don't worry, existing configuration is still compatible ;)

I actually don't care too much about BC, if we rewrite let's not have any BC code, because that gets stagnated, I don't like that at all anything that prevents simplicity so if we can remove it let's do that. It is super easy to create configuration again so :+1: for removing the BC code.

I am not mad at all for you rewriting Gush :blush: is cool mate :metal:

outstanding

The test are blazing fast now, easy to understand and less fragile.

Instead of testing the exact outcome we only look for those things that are important or easy to break.
This makes it possible to at any point change our Styling or IO logic without having to update all the tests.
Only mock when there is an actual interface (Config is not mockable anymore).
Testing questions now uses an Input stream (as described in the Symfony docs) to provide the actual input, and not mock the QuestionHelper.
A Virtual system is used for mocking file operations (including meta:header command), testing with the real one is still possible (export GUSH_USE_TMP=false but should only done in Travis or when there is a problem).
Test if the Adapter operations actually did what we expect (needs further improvements, but its a start :+1: )
No more Unix/Windows only tests.

Note: Only the broken tests were updated, I have some other idea's to further improve testing, but thats something for a follow PR as this one is already big enough :smile:
GitRepoSubscriberTest is missing some tests to ensure the Org/repo is properly set for the Adapter. this actually works, but its best to keep it that way :dancers:

Note for tomorrow, create a temp-folder as the system tmp folder is not working with sudo: false :+1:

Wow I cannot believe it :scream: its done! This is properly my biggest pull-request for Gush so far :smile: I also tested all the commands manually and they are working perfectly :+1: :metal:
@gushphp/contributor Please do thorough review! Also on spelling as I'm not a native speaker and got bitten by that before ;)

Thanks for reviewing @pierredup

@sstok Great work! There are quite a lot to review and I'm not even halfway through yet :flushed: I'll continue when I get some more free time

I'll do an English check when I get back to my computer
—
Aaron Scherer
http://bldr.io/
On Sun, May 3, 2015 at 9:41 AM, Pierre du Plessis
<EMAIL_ADDRESS>wrote:

@sstok Great work! There are quite a lot to review and I'm not even halfway through yet :flushed: I'll continue when I get some more free time
Reply to this email directly or view it on GitHub:
https://github.com/gushphp/gush/pull/416#issuecomment-98502343

Ok, my eyes are getting tired..... lol

All should be good now, great to some amazing teamwork :+1:

:shipit: :+1:

outstanding ...
team  @gushphp/contributor join me in thanking @sstok :clap:


Action alias CLI commands and minor consistency changes

This pull request adds CLI comments for viewing action aliases (list, get).
In addition to that, I made some other changes so it's consistent with other content pack resources (actions, rules, etc.):

Also allow user to retrieve action alias by ref
Update supported query param filters
Add enabled attribute to the ActionAlias model

Overall I am ok with the change. Few questions but not blockers -
+1


No  variables PUID and PGID

Seems as if agents are forced to run as '100:101' in container. Not nice. Would be nicer if what now appears to be a standard is used.

environment:
  PUID=1005 # User
  PGID=102 # Group
  othervars=....

This will also allow for required user access outside the container without requiring root. Forgive me if there is already a way to do this.

Yes please add PUID and GID variables. I recently moved my agent docker container to openmediavault but I think it's failing to record events due to permissions problems and normally this is fixed by having the user/group ID configurable via variables (see also linuxserver's docker containers)

Would this be sufficient (https://docs.linuxserver.io/general/understanding-puid-and-pgid). It mentions:
We are aware that recent versions of the Docker engine have introduced the --user flag. Our images are not yet compatible with this, so we recommend continuing usage of PUID and PGID.

Would this work as well? https://docs.docker.com/engine/reference/run/#user

@yllekz  @neutralvibes  any thoughts?

Just came across this issue, for K8S if I set runAsNonRoot: true, I will also need to set runAsUser: <uid>.  @cedricve From the dockerfile I can see there is an non-root user, but there seems not to be an uid set for the user? A static UID would be nice and easier for config


Inspector view of patch file after successful submit does not allow the user to copy the last newline of the patch file.

When using the inspector and viewing a trajectory with a successful submit, users are able to retrieve the patch file.
However, when a user goes to copy the patch file in the localhost:8000 UI, the last newline of the patch file is unselectable (see screenshot)

I assume this is universal on all browsers and is a function of the  tag (https://stackoverflow.com/questions/638648/getting-linebreaks-in-pre-tags), but just in case, I am using Firefox 124.0.1 (64-bit).
Workaround: After copying what is selectable from the patch file, add a newline to the patch file manually.

Actually, I was wrong, it was selected. Its just a bit tricky and doesnt look like it.


Fix mysensors callback

Description:

All messages were not triggering proper updates. Fix by checking all
child value types each update.
Upgrade pymysensors to fix a bug with pickle persistence, when upgrading pymysensors.

Checklist:
If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Flaky tests?

Don't worry about CI, it's aiohttp having dependency issues.

Cherry-picked for 0.42.4


where is the training datasets?

can you tell me where is the training datasets? or the link of the datasets,thank you very much!
(files train.smiles, valid.smiles and test.smiles,

You can get the datasets via links in the guacamol repository.

Thank you very much! @kmaziarz


PENV-411: set special character for empty jetid

- What I did
- How I did it
- How to verify it
- Description for the changelog

Codecov Report

Merging #77 into master will increase coverage by 0.00%.
The diff coverage is 80.00%.

@@           Coverage Diff           @@
##           master      #77   +/-   ##
=======================================
  Coverage   74.98%   74.99%
=======================================
  Files          43       44    +1
  Lines        2770     2779    +9
=======================================
+ Hits         2077     2084    +7
- Misses        555      556    +1
- Partials      138      139    +1

Impacted Files
Coverage Δ

api/handlers.go
79.56% <ø> (ø)

etl/models/models.go
100.00% <ø> (ø)

instrumentation/converter/platform.go
83.33% <0.00%> (-16.67%)
:arrow_down:

api/mappers.go
100.00% <100.00%> (ø)

etl/transformer/transformer.go
83.42% <100.00%> (-0.09%)
:arrow_down:

instrumentation/comparator.go
100.00% <100.00%> (ø)

test/api/apiwrapper.go
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0e6d09...827b5cb. Read the comment docs.