[.NET 9.0 Preview 5] Could not load file or assembly NuGet.ProjectModel

Description
Hi, I'm looking forward to AndroidMavenLibrary, this is what I've been waiting for for a very long time and want to try. After all, for full-fledged development, we always lacked the libraries that developers use in Android Studio.
So I installed:
VS 2022 Preview
.NET 9.0 Preview 5
.NET 9.0 MAUI Workload
And added to Android Binding project
<ItemGroup>
  <AndroidMavenLibrary Include="com.squareup.okhttp3:okhttp" Version="4.9.3" />
</ItemGroup>

then output error:
1>C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\34.99.0-preview.5.308\tools\Xamarin.Android.Bindings.JavaDependencyVerification.targets(27,5): error XAJDV7028: System.IO.FileNotFoundException: Could not load file or assembly 'NuGet.ProjectModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The specified file cannot be found.
1>C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\34.99.0-preview.5.308\tools\Xamarin.Android.Bindings.JavaDependencyVerification.targets(27,5): error XAJDV7028: File name: 'NuGet.ProjectModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'
1>C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\34.99.0-preview.5.308\tools\Xamarin.Android.Bindings.JavaDependencyVerification.targets(27,5): error XAJDV7028:    at Xamarin.Android.Tasks.NuGetPackageVersionFinder.Create(String filename, TaskLoggingHelper log)
1>C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\34.99.0-preview.5.308\tools\Xamarin.Android.Bindings.JavaDependencyVerification.targets(27,5): error XAJDV7028:    at Xamarin.Android.Tasks.DependencyResolver..ctor(String lockFile, TaskLoggingHelper log) in /Users/runner/work/1/s/xamarin-android/src/Xamarin.Android.Build.Tasks/Tasks/JavaDependencyVerification.cs:line 124
1>C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\34.99.0-preview.5.308\tools\Xamarin.Android.Bindings.JavaDependencyVerification.targets(27,5): error XAJDV7028:    at Xamarin.Android.Tasks.JavaDependencyVerification.RunTask() in /Users/runner/work/1/s/xamarin-android/src/Xamarin.Android.Build.Tasks/Tasks/JavaDependencyVerification.cs:line 77
1>C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\34.99.0-preview.5.308\tools\Xamarin.Android.Bindings.JavaDependencyVerification.targets(27,5): error XAJDV7028:    at Microsoft.Android.Build.Tasks.AndroidTask.Execute() in /Users/runner/work/1/s/xamarin-android/external/xamarin-android-tools/src/Microsoft.Android.Build.BaseTasks/AndroidTask.cs:line 25

Steps to Reproduce
see above
Link to public reproduction project repository
No response
Version with bug
Unknown/Other
Is this a regression from previous behavior?
No, this is something new
Last version that worked well
Unknown/Other
Affected platforms
Android
Affected platform versions
No response
Did you find any workaround?
No response
Relevant log output
No response

@alexeygritsenko I would imagine this works fine with dotnet build? And it's only broken in Visual Studio or .NET framework MSBuild.exe?

@drasticactions, Yes, this is reproduced in android native project (without MAUI), I didn't know that you have dotnet/android repo. Sorry if I posted this topic in the wrong place.
@jonathanpeppers, I did not think about that. You're right. I just checked through the command line and there is no such error.
In this case, when can I expect Visual Studio support?

While we should of course address this issue, also note that we already maintain an okhttp3 binding: https://www.nuget.org/packages/Square.OkHttp3.

I had a memory of "how does this work without us shipping NuGet.ProjectModel.dll"?
But I can't remember where I posted that comment:

https://github.com/dotnet/android/blob/main/build-tools/installers/create-installers.targets

While we should of course address this root issue, also note that we already maintain an okhttp3 binding: https://www.nuget.org/packages/Square.OkHttp3.

Thanks. As for Square.OkHttp3 library, I know that it already exists in Nuget, I was just interested in testing how it works. In the future, I want to connect libraries that are not available on Nuget in this way, for example mikepenz/FastAdapter

I had a memory of "how does this work without us shipping NuGet.ProjectModel.dll"?

I suspect it works because the msbuild process will very likely already have the NuGet libraries loaded in order to do the project build.  However they may not be the version we expect, or perhaps a VS build can be broken up into small enough pieces that NuGet isn't in that msbuild process.
I think we will likely need to ILRepack the libraries to ensure we always have and use our expected versions.

I think we will likely need to ILRepack the libraries to ensure we always have and use our expected versions.

I wanted to remove ILRepack, can we just ship the assembly like we normally would?

@drasticactions, Yes, this is reproduced in android native project (without MAUI), I didn't know that you have dotnet/android repo. Sorry if I posted this topic in the wrong place.

No worries!  We just wanna make sure it gets to the right place. It's always better to file an issue somewhere than not file it at all.

I wanted to remove ILRepack, can we just ship the assembly like we normally would?

Given that there is very likely a copy of this assembly already loaded in the msbuild process, and it may not match our version, I think we need it ILRepack'd but I could be wrong.  I would think the same about other assemblies we currently ILRepack like Newtonsoft.JSON as well.

No it's strong named and versioned, so it should work to just ship a copy.

No it's strong named and versioned, so it should work to just ship a copy.

That should work for .NET framework which supports strong naming, but I suspect it won't work on .NET 6+ which doesn't check strong names.

For .NET Core and .NET 5+, strong-named assemblies do not provide material benefits. The runtime never validates the strong-name signature, nor does it use the strong-name for assembly binding.

(https://learn.microsoft.com/en-us/dotnet/standard/assembly/strong-named)
For example, let's say we use an API added in Newtonsoft.Json 12.0, and a user's project includes another MSBuild task assembly that references and ships Newtonsoft.Json 11.0.
On a dotnet build, if their MSBuild task assembly is loaded first it will load 11.0 and I suspect our task would fail.

The old problem was that VS had third party extensions  (ILSpy/Refactor)  which shipped incompatible assembly versions for things like Cecil and NewtonSoft. These would break our tasks, so ILRepack was the only option.
Someone needs to test this in VS to make sure we do not hit any clashes.

I think this PR will work in both VS and dotnet build:

https://github.com/dotnet/android/pull/9098

An old sample I have is:

https://github.com/jonathanpeppers/dotnet-using-task

VS / .NET framework should work, because the NuGet assemblies are strong-named and versioned. .NET framework can only load a single assembly with the same "identity", no matter if it is a different file path.

dotnet build should work. Even though it doesn't validate strong names, it can successfully load different assemblies from <UsingTask/>. dotnet build should be able to load assemblies of the same name with different paths.

ILRepack seems more useful for fixing Mono MSBuild and VS Mac, as Mono didn't do either 1 or 2.
If we are concerned, we could try different versions of Newtonsoft.Json in my sample above?


Can't done newly created video or photo

The newly created video or photo is display selected,but can not save it.(it should be choose again)

I checked Latest version 0.9.7 on iPhone6 (iOS8.3).
it works well. could you explain more ?
please check your UzysAssetsPickerController Version.

I checked Latest version 0.9.7 on iPhone5s (iOS8.12)
The detail problem is this：
when the newly created video or photo be use, the newly item is selected,
and bottom -right button number is 1,
but the button can not be tap.

제가 풀리퀘 드린 부분에서 버그가 있는것 같습니다. 죄송합니다. ㅠ
선택 순서대로 array에 저장하는 기능을 넣으면서, 카메라로 찍은 asset을 거기에 포함하지 않았습니다.
얼른 다시 풀리퀘 드리겠습니다.

Thanks for the bug reporting. I fixed bug. that caused by new feature (select order).
I would recommend you to use taged version not latest commit.


pydantic validators fail silently

This code works fine if the password conforms to the validator rules.
Especially during assignment of a parent JsonModel. However when I run a get() method to retrieve data I run into errors.
I have the follow EmbeddedJsonModel.
class identity(EmbeddedJsonModel):
    '''
    A Password|Token verification class
    '''
    password: str | None
    auth_token: str | None
    # session_token: str | None

    @root_validator
    def password_or_token(cls, values):
        if values['password'] or values['auth_token']:
            return values
        raise ValueError("Password or Token must be provided")

    @validator("password", pre=True)
    def password_hasher(cls, password):
        if password:
            if len(password) >= 8 and len(password) <= 16:
                if not (any(char in special_characters for char in password)
                and any(char.isdigit() for char in password)):
                    raise ValueError("Password must contain at least 1 number and 1 special character")
            else:
                raise ValueError("Password must be between 8 and 16 characters")
            return ph().hash(password)

x = await RedisUser.get(pk) This fails with the following error: KeyError: 'password'
This indicates that the Model completely rejected the key 'password'  on getting it back. Since the value is hashed and doesn't necessarily pass the validation check anymore on retrieval from the DB.
If I comment out the @root_validator then my 2nd validator returns the correct error.
Similarly,
If I comment the 2nd validator @validator("password") then the program works. My root validator runs.
This problem may extend to the sync version as well.
This is what I sent to the DB:
identity(pk='01GPZQCPEDZG928P7XCTHJYB36', password='<password hash that fails the validator check>', auth_token=None)
This is what I got back:
identity(pk='01GPZQCPEDZG928P7XCTHJYB36', auth_token=None)

This could be a pydantic problem. Will need to debug further.


Add a data validator

Problem

Transforming unknown/untrusted data into valid prisma input types can be a bit annoying, we should add a function that takes a type and data and validates the data against the type.
Suggested solution

def validate(type_: Type[T], data: Any) -> T:
    ...

We might be able to make use of pydantics dynamic create model feature and if not we can just roll our own validation.
def foo(data: Dict[str, Any]) -> User:
    create = prisma.validate(UserCreateInput, data)
    return client.user.create(data=create)

Implemented with 386e6056f16d538194ae6fc007306f62dca484c3


Can't vendor two compatible crate versions from two projects

More info at -- https://bugzilla.mozilla.org/show_bug.cgi?id=1336528#c3

Note that using -x is of no help, even though setting the option you would expect cargo vendor to save both versions and prefer the specific version first before falling back to the most recent version.
The reason why this isn't a showstopper for Mozilla is that they delete the directory every time:
https://bugzilla.mozilla.org/show_bug.cgi?id=1336528#c5
Which apparently works around this bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=1336528#c4
(though I can't get the workaround to work, I am stuck)


Refactor remaining [Next|Previous] method pairs?

Just did a quick pattern search to identify the remaining Next/Previous method pairs. These two would fit the Adjacent pattern well:

WorkspaceManager.MoveWindowTo[Next|Previous]Monitor
MonitorManager.Get[Next|Previous]Monitor

This one also lends itself to a merge, but the Adjacent naming seems slightly less fitting:

Workspace.[Next|Previous]LayoutEngine

Will be replaced with the updated structure in #890


Highlight selected column

Hi,
I'm building csv reader using your libs and I wonder how solve column highlight problem. I was looking for some function to set attributes and I found one(AddAttribute) but this only read attribute from the column and set it to cell. Do you maybe have some other solution for my problem, I would be very pleased.
Greetings PK

What widget do you use? Can you provide code reproduce?

Sure, this make columns and add text that reads from .csv file. And now I want to highlight column somehow when I click on.
dw.treeView.column = make([]*gtk.TreeViewColumn, w)
dw.treeView.cell = make([]*gtk.CellRendererText, w)
column := gtk.NewTreeViewColumnWithAttributes("#",
  gtk.NewCellRendererText(), "text", 0)
column.SetClickable(true)
column.Connect("clicked", func() {
  fmt.Println("All")
})
   f := func(i int) func() {
     return func() {
       fmt.Println(i)

  }
}

dw.treeView.tree.AppendColumn(column)
if hasHead == false {
  for i := 0; i < w; i++ {
    dw.treeView.cell[i] = gtk.NewCellRendererText()
    dw.treeView.column[i] = gtk.NewTreeViewColumnWithAttributes("empty",
      dw.treeView.cell[i], "text", i+1)

    dw.treeView.column[i].SetClickable(true)
    dw.treeView.column[i].SetReorderable(true)
    dw.treeView.column[i].Connect("clicked", f(i))
    dw.treeView.tree.AppendColumn(dw.treeView.column[i])

  }
} else {
  sName := dt.GetHeader()
  dw.treeView.tree.SetModel(store)
  for i := range sName {
    dw.treeView.cell[i] = gtk.NewCellRendererText()
    dw.treeView.column[i] = gtk.NewTreeViewColumnWithAttributes(sName[i],
      dw.treeView.cell[i], "text", i+1)

    dw.treeView.column[i].SetClickable(true)
    dw.treeView.column[i].SetReorderable(true)
    dw.treeView.column[i].Connect("clicked", f(i))
    dw.treeView.tree.AppendColumn(dw.treeView.column[i])

  }

}

for i := 0; i < h; i++ {

  var iter gtk.TreeIter
  store.Append(&iter)
  record := dt.GetRecord(i)
  count := strconv.Itoa(i + 1)

  record = append([]string{count}, record...)
  for j := 0; j < w+1; j++ {
    store.Set(&iter, j, record[j])

  }
}

You can try following patch on examples/listview.
diff --git a/example/listview/listview.go b/example/listview/listview.go
index 409d6c2..f16c7dd 100644
--- a/example/listview/listview.go
+++ b/example/listview/listview.go
@@ -17,12 +17,12 @@ func main() {

 	swin := gtk.NewScrolledWindow(nil, nil)

-	store := gtk.NewListStore(glib.G_TYPE_STRING, glib.G_TYPE_BOOL, gdkpixbuf.GetType())
+	store := gtk.NewListStore(glib.G_TYPE_STRING, glib.G_TYPE_BOOL, gdkpixbuf.GetType(), glib.G_TYPE_STRING)
 	treeview := gtk.NewTreeView()
 	swin.Add(treeview)

 	treeview.SetModel(store)
-	treeview.AppendColumn(gtk.NewTreeViewColumnWithAttributes("name", gtk.NewCellRendererText(), "text", 0))
+	treeview.AppendColumn(gtk.NewTreeViewColumnWithAttributes("name", gtk.NewCellRendererText(), "text", 0, "background", 3))
 	treeview.AppendColumn(gtk.NewTreeViewColumnWithAttributes("check", gtk.NewCellRendererToggle(), "active", 1))
 	treeview.AppendColumn(gtk.NewTreeViewColumnWithAttributes("icon", gtk.NewCellRendererPixbuf(), "pixbuf", 2))
 	n := 0
@@ -34,6 +34,7 @@ func main() {
 			0, id,
 			1, (n == 1),
 			2, gtk.NewImage().RenderIcon(id, gtk.ICON_SIZE_SMALL_TOOLBAR, "").GPixbuf,
+			3, "red",
 		)
 		n = 1 - n
 	})

Thanks I also made solution like this, but I think it don't solve my problem. I am doing "excel" for big CSV file. So add additional column with color is not so satisfied. Anyway that's for your reply. Can I ask You a question, when will You finish this gtk lib ?

not column. it should be appended into list store. see the diff above. I don't append into column.

This is how it looks,

And I don't want additional column.

Sorry my mistake

closable?

One question, can you give me example, I want to replace old column with new one with background color when i press on column header.

okay but if i can take enough time to do. :-o

Look what I've done, maybye that would help you.
https://gist.github.com/PiotrKlosek/5414aeede32c0517538830d309e049bf#file-reader-go
And now in line 54 I want to change background color of column. But I want to done it without display additional column with color, do you see any solution ?

okay. closable?

Yes, sure


During fingerprinting, AVHashes returns empty when specifying secondsToProcess and startAtSecond while creating both audio and video fingerprints

Describe the bug
When fingerprinting (haven't tested query yet) both audio and video simultaneously, if you supply secondsToProcess and startsAtSecond paramaters the AVHashes returned is always empty (isEmpty=true).  Fingerprints are returned as expected if only MediaType.Audio or MediaType.Video are specified, but both (MediaType.Audio | MediaType.Video) does not. Based on the time it takes, it seems likely it is doing neither video or audio hashing of any kind, and just returning the empty AVHash.
To Reproduce
Steps to reproduce the behavior:

Try to fingerprint a file (containing audio and video) specifying secondsToProcess, startsAtSecond, and "MediaType.Audio | MediaType.Video" paramaters in the .from method of the FingerprintCommandBuilder.Instance.

Expected behavior
Returned fingerprint hashes for both video and audio from the timerange specified by secondsToProcess and startsAtSecond, instead of an empty AVHashes containing no fingerprints.

OS: Win7
IDE: VSCommunity 2017
Targeting: .Net v4.6.1
Using nuGet Packages: SoundFingerprinting.8.23.0, SoundFingerprinting.Emy.8.23.23, FFmpeg.AutoGen.Abstractions.<IP_ADDRESS>
Using git build of ffmpeg pre-compiled from here : https://github.com/BtbN/FFmpeg-Builds/releases/download/autobuild-2023-06-21-15-24/ffmpeg-N-111233-g83b3ff741b-win64-gpl-shared.zip

Can you please attach a file and provide the exact parameter values for both secondsToProcess and startsAtSecond so I can reproduce the issue?

Here is the sample file (downloaded format #22 with yt-dlp)
yt-dlp -f 22 https://www.youtube.com/watch?v=4sebEgloQEM
secondsToProcess = 10
startsAtSecond = 33
While playing around with it, it does seem that the particular values above are involved in the issue. There were some, like secondsToProcess = 10, and startsAtSecond = 1 which did appear to work.
In my use case, the secondsToProcess and startsAtSecond values are always doubles (many decimals) and are retrieved from previous query result matches. The above is just a contrived example with a sample file.

It is fixed in v8.24.0, you will need to upgrade SoundFingerprinting.Emy to fix version.
Also, I suggest you re-index your database if you've used secondsToProcess parameter with both Audio | Video flags during track insertion.
Link to NuGet package: https://www.nuget.org/packages/SoundFingerprinting.Emy/8.24.0


Possible visual bug: There is no margin between div elements anymore

Gitea version (or commit ref): 1.11.0
Git version: -
Operating system: -
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist: -

Description
It seems that div elements are no longer separated with margin between them.
Screenshots

I think this is also a duplicate but, again a good reminder that we never got this fixed after the move to fomantic

Example for logged in users: https://try.gitea.io/user/settings
Seems like a underlying bug in Fomantic but I guess we can workaround:
.ui.attached.segment + .ui.attached.header {
  margin-top: 1rem;
}


Integrate the stat feature in director's redirect request

In #682, we introduced a stat feature at the director to query against all origins for an object in the federation and see which origins can serve the object. We want to integrate this check when the director received a object request.

This is blocked by the fact that the current setup of the stat feature only supports public namespaces as noted in the PR: https://github.com/PelicanPlatform/pelican/pull/682
We need to add the support for protected namespace first before the stat call returns correct result

I wouldn't be able to get this in 7.8. Bump it to 7.9


How to load appollo-client with system.js

I just added angular2-apollo to an angular2 project bootstraped with angular-cli.
Angular 2 is loading modules with systemjs. However it's pain with apollo-client: You have to add every individual lodash function as a module to systemjs to be loaded. Moreover this ends in many requests to the server and inconvenient imports.
Thus I do have the following questions / ideas:

Isn't it possible to load apollo-client as a node_module with and also load its sub-depenencies automatically without adding every single to system js? Also asked this at the angular-cli repository and stackoverflow
Does it make sense to load lodash at once and not every individually needed function separately?
Does it make sense to make one single loadable apollo-client file wich includes sub-dependencies?

However I'm not jet that much into SystemJS and module loading (I appreciate) to tell if my thoughts make sense?

Honestly, this seems like a problem with systemjs rather than with Apollo Client. It shouldn't be up to the module author to pre-package dependencies, that sounds like the build system's job. Otherwise the whole concept of dependencies doesn't really make sense.

Does it make sense to load lodash at once and not every individually needed function separately?

No, because many apps don't use lodash. In this case we would be loading all of lodash just to use a few functions. With Webpack/Browserify/Meteor this is completely transparent to the app developer.

Does it make sense to make one single loadable apollo-client file wich includes sub-dependencies?

We could do this pretty easily if it makes systemjs easier. How do people usually bundle packages for system? Is there an example I can look at?
@Urigo @kamilkisiela did you run into this problem with Angular?

I have no experience in working with SystemJS. There is an issue on angular2-apollo on that topic too.
In some way you have to specify which modules you want to load and where the build system can find them. I didn't come up with any rational solution.
Here's an example of SystemJS in use. https://github.com/angular/quickstart

I've just answered it on Stack Overflow here: http://stackoverflow.com/a/38317093/1426570
It's been hard to solve and I totally agree with @stubailo that this is a System.js/Angular CLI issue.
In the answer I link to the solution but also to an active issue about this on Angular CLI.
I think we can close this now


refactor: Remove dependency on netaddr

Fixes #253

Remove netaddr dependency in favor of builtin ipaddress
Refactor tests for inventory model
Add error catching
Factorise the function using DEBUG for Exceptions
Fix typing for **kwargs to match with mypy recommendations

Works on my machine.


Date range min max dates

minDate and maxDate already exists in Date Range Picker http://www.daterangepicker.com/#options, I just add them to our component (which uses Date Range Picker).
So we can use this to restrict date range select (for example for future or past dates).

 (8) Include 'Add Branding Campaign' to Mass Adjustment Tool in BO


Welcome Modal is not shown

Bug description
I have never seen the Welcome Modal in the UI. While it may be technically possible, I think that in practice it is not possible.
The relevant section of code is in main_page.py:
def __panel__(self):
    asyncio.ensure_future(self.refresh_data())

    objects = [self.left_sidebar, self.central_view, self.right_sidebar]

    if self.chats is not None and len(self.chats) == 0:
        ...  # logic to add welcome modal to layout here

self.chats is initialised as None. It is subsequently populated by a JSON structure by self.refresh_data(). However, self.refresh_data() is scheduled as a background task (asyncio.ensure_future) and is unlikely (impossible?) to complete before hitting the if-block of code that determines whether to show the welcome modal. Hence the welcome modal logic will never be shown.
How to reproduce the bug?
Open the UI!
Versions and dependencies used
Affects all versions.
Anything else?
My favoured proposal would be to remove the welcome modal entirely.
If this is not desired, it becomes a bit more complicated.
The welcome modal should only be shown the first time that someone uses ragna. I consider it quite a bad/annoying UX otherwise - it's just another button to click to close something and is frustrating.
The question is then: how does ragna determine a first-time user? At the moment, the only clear way is to call the /get_chats endpoint and determine the length of the chat history (a length of 0 means new user). To show the welcome modal, the application would need to wait for a response from this endpoint before showing the main_page (at the moment the application does not wait for a response, it just schedules it as a future task). However, waiting for a response in this way might introduce a performance issue if /get_chats returns something large, or if the API is slow for some reason, so I'm not sure if this is a good idea.
Another solution is to introduce a new endpoint to the API that checks whether a user already exists in the DB (so, it doesn't return the entire user history, just a boolean indicating a if the user exists). This should not create the same performance issues, but it does mean that we now need to support a new endpoint. (Which is fine, but is it really worth it?)
Another partial solution is to call the welcome modal anytime a user needs to authenticate. It's not great, as it's not only first-time users who have to authenticate, but it does reduce the frequency of the welcome modal pop-up.

The idea of the modal was to provide a "landing page" of sorts when the user starts the app, instead of the chat config modal directly.
A solution I see to reduce the number of clicks would be to move this from a modal format, to on-screen text, so that the user doesn't have to dismiss the modal to start a new chat. And you would still have a "landing page" that can always be used to provide info, updates etc.
Wdyt?

So if there are no chats open, we have static text where usually the chat interface is?

Yep, we still have the left panel, and static text on the right side, where the chat usually is, without the input field.


Skip link is hardcoded to point at #content

GOV.UK Template hardcodes the skiplink to point at #content (see this line). But GOV.UK Elements adds styles to #content (see this block).
There are cases where we don’t want the thing we identify as #content to have those styles.
Would it be possible to override where the skiplink points to, so we could have <a href="#anything-else" class="skiplink">?

Alternatively, we could set GOV.UK elements to use a class instead, for example, .main-content, that would allow id="content" to be reserved for the target of the skiplink.

@gemmaleigh Yes. Although that would be more of a breaking change for users of Elements I think.

It would! I was going to add that. Sorry, this is something from GOV.UK elements that you're having to work around.

No problem, I don’t think it’s a use case you could’ve reasonably foreseen.

Following the launch of the GOV.UK Design System, GOV.UK Template will now only get major bug fixes and security patches, so I'm going to close this.
We've captured this issue on GOV.UK Frontend here.


[Question] CuFile GPU behavior in Compatibility Mode

When kvikio/CuFile is not running in compatibility mode it is possible to have the cuFileReadAsync/WriteAsync API calls executed in different cuda streams. kvikio.CuFile.raw_read_async() and the corresponding raw_write_async() function can take a cupy.cuda.Stream object as an argument to define which stream to perform the read/write. These functions do not support host memory and do not use posix threads.
When in compatibility mode reads are done through posix, and kvikio/CuFile parallelizes this by creating multiple posix threads. In compatibility mode during a read, would it be possible to start another read that is not blocked by the one before it with kvikio in another cupy.cuda.Stream (or something like it)? Is this stream functionallity only supported when accesing data on a GDS enabled storage in CuFile?

When kvikio/CuFile is not running in compatibility mode it is possible to have the cuFileReadAsync/WriteAsync API calls executed in different cuda streams. kvikio.CuFile.raw_read_async() and the corresponding raw_write_async() function can take a cupy.cuda.Stream object as an argument to define which stream to perform the read/write.

Yes, raw_read_async and raw_write_async can be called with different cuda streams. Just make sure that the returned IOFutureStream is kept alive alive until all data has been read/written.

These functions do not support host memory and do not use posix threads.

Correct.

When in compatibility mode reads are done through posix, and kvikio/CuFile parallelizes this by creating multiple posix threads. In compatibility mode during a read, would it be possible to start another read that is not blocked by the one before it with kvikio in another cupy.cuda.Stream (or something like it)? Is this stream functionallity only supported when accesing data on a GDS enabled storage in CuFile?

This isn't implemented in KvikIO but the workflow should work with KvikIO. Say you have 4 threads in Python, you can have each thread read a chunk of the file using their own cuda stream. Since all calls to kvikio releases the GIL, it should enable concurrent reads.


Node 16+ ESM TypeScript error

Hi y'all, I'm getting this error for all worktop packages
Could not find a declaration file for module 'worktop'. '.../node_modules/worktop/router/index.mjs' implicitly has an 'any' type.
  There are types at '.../node_modules/worktop/router/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'worktop' library may need to update its package.json or typings.

I'm on 0.8.0-next.14 does anyone have suggestions on how to fix this?

Fixed & available as 0.8.0-next.15 – thanks!


Added then answer void method

Status
READY
Breaking Changes
NO
Description
Generally, much developers has to use when method to return void value.
Thus, this PR added simple extension for when class to make more easy works with void returns values.
Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Thanks for opening a pull request! This is similar to #77 so perhaps https://pub.dev/packages/mocktailx would be of use here.
Closing for now but feel free to comment with additional questions and I'm happy to continue the conversation 👍


Mouse control

Up down left right control dont works

Do you have raw input turned off in Minecraft? Robot won't be able to put input into it since it can't send direct mouse events it's just blindly moving the cursor around the screen.
This was very specific to the setup I was using it in so you'll have to make sure your game is running at 1280x720 and is placed in the middle of the screen unless you changed it to account for a difference.

Keep in mind I don't have Windows installed so if you are running this under Windows you are on your own since I don't actually have a Windows install to test on. I only use Linux

Closing due to inactivity


Username Characters Length Limitation & Overflow Fix

This solve the issue #9

[x] Added line break in overflow

[x] Added character limit up to 40

Hi, looks great, thank you again! One minor thing: could we combine the error messages for "please enter a username to continue" and "please enter a username between 1 and 40 characters" into a single message?

Hi, looks great, thank you again! One minor thing: could we combine the error messages for "please enter a username to continue" and "please enter a username between 1 and 40 characters" into a single message?

Yes. I will update the code.

I have updated please check. If it's okay or not.

Looks good, thank you again!

Looks good, thank you again!

welcome 😄


3.1.x fixes - CXF-6454

CXF-6454 - Orphaned JMS connections created in endless loop
Changing the code to limit the number of retries to a pre-defined value. It can be configured as maxNoOfRetries in JMS Enpoint.
Adding another property to make the sleep time configurable. We were seeing high CPU consumption because of low sleep interval time.

@cschneider - can you please check this. If this fix looks fine to you.

Will look into it. The tests look good already.
I will need to squash the commits into one commit but that is fine.

I think the max retries does not really solve the problem with the InvalidClientIdException.
I have pushed an alternative solution with just retryInterval and special handling of the exception to
the CXF branch https://github.com/apache/cxf/tree/CXF-6454. Can you try if this fixes your problem?

@cschneider We are also trying to address the 2nd point mentioned in mentioned in CXF Bug CXF-6454, I think we should still try to get rid of infinite loop problem as well. We are trying to give a feature of maxRetries as well so that customers can configure on their own as per their needs. Thus, I would still suggest we put a loop controller with maxRetries as in the original code from this pull request. Further there are 2 problems in addition

If a thread interrupts the loop, you are not cleaning up the resources done in method deactivate(). It simply is coming out without cleaning up the resources.
We are still not giving control to the customer using this feature to control the maxRetries and coming out of the loop without interrupting the thread.
Also, in addition to that, Thread.sleep() should be put in try block before deactivate as done in original code in this pull request since it does not make sense to retry immediately the JMS Connection. Thread.sleep will allow a grace period for Queue Mgr to come up and thus, we should keep Thread.sleep before deactivate and put maxRetries back in loop to come out of the endless loop

I do not yet see a good case for limiting the number of retries as the cxf endpoint or client would be unusable at this point. Can you elaborate how the max retries would help in practice? Especially what would happen after the retries are stopped. Btw. This might be something we could discuss on IRC. I am always in the channel #apache-cxf.

Not sure I understand this case. Can you elaborate the steps that would happen?
I think the retry thread should run as long as the CXF endpoint is active. My intention is that you can simply call shutdown on the Destination which ends the retry thread. Interrupting a thread is not such a good idea to achieve this.
I agree to have the sleep there. I wrongly thought that this code would also be run on the first connect.

@cschneider The use case for point-1 mentioned by me above is when the application wants the flexibility to stop the retries after certain number of attempts if the Connection couldnt be established to JMS queue managers or JMS Destinations itself. The client may not like to continue the retry infinite times due to the CPU Spikes that they can experience unnecessarily after every completion of thread.sleep. Also, this situation becomes worst if a single JVM has >50 JMS Destination Listeners defined and a single physical host has multiple such JVMs. In such scenario, client would still see high CPU when it continues to do infinite retries. Thus it makes sense to give flexibility to client in terms of exposing both retries and retryinterval.
Also for point 2, I was trying to avoid need for client to put custom solution to call explicitly shutdown on JMSDestination as well. ALso, if you note, shutdown variable is proned to race condition since its not defined volatile. If you call shutdown from another Thread, and the retry thread is still looping in another thread, the change in the value of shutdown variable may not be instantaenously visible to other thread.
Thanks!

hi @cschneider Can you please comment on my last comment. If you are fine then we would like to go ahead with the original fix provided by @vikash32504

We are hitting critical issues due to this bug in our applications

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished. 717 tests run, 2 skipped, 0 failed.


Removing unnecessary meta.openstack in openstack stub

See related thread at https://groups.google.com/a/cloudfoundry.org/d/msg/vcap-dev/pGqlNn_MOO0/C1bU8vHPwIMJ
This is a prereq to submitting a docs PR as it is sourcing this file, see https://github.com/cloudfoundry/docs-deploying-cf/blob/master/cf-stub-openstack.html.md.erb#L18

Merged as of 4502f945a1584168163214c17797bc686db5e405.
@gberche-orange In the future, you should make your pull requests to cf-release against the develop branch as that is our working branch.
Thanks for the PR!
@zaksoup + @jfmyers9, CF Runtime Team

thanks @zaksoup and @jfmyers9, and sorry about forgetting to start from the develop branch.


rviz fails to transform frames with leading slashes

I have a bagfile with PointCloud2 message in the /left/driver frame.
When I try to visualize these in rviz, I set the fixed frame to "/left/driver", but it automatically changes to "left/driver" after I type it in, and I get the error "Failed to transform from frame [/left/driver] to frame [left/driver]"
If I process the bagfile to change /left/driver to left/driver, it displays correctly.
I think this is the same error that is being asked about here: https://answers.ros.org/question/355595/rviz-tf-error-failed-to-transform-from-frame-map-to-frame-map/
Your environment

OS Version: Ubuntu 20.04
ROS Distro: Noetic
RViz, Qt, OGRE, OpenGl version as printed by rviz:

[ INFO] [1602544755.809240504]: rviz version 1.14.1
[ INFO] [1602544755.809284506]: compiled against Qt version 5.12.8
[ INFO] [1602544755.809296759]: compiled against OGRE version 1.9.0 (Ghadamon)
```

http://wiki.ros.org/tf2/Migration#Removal_of_support_for_tf_prefix:

tf2 asserts that all frame_ids do not start with /.

I'm afraid you need to remap your bag file.


Add missing mod statement for securitybaseapi

Title
Cargo.toml and build.rs seem to reference it.
I'm not sure how to test that this will actually fix the problem, but this seems like the obvious fix.

Is there anything that needs to happen for this to be merged?

For me to fix the test I assume. Will do it this week-end if I don't forget. :)

Solved in #568. Once merged, your PR will be able to be merged as well.

Cool, glad I was able to contribute 👍


Temporary fix to handle broken CE 2.0.8 to 2.0.9 updates

Update tests are broken because htcondor-ce-2.0.8-2 unconditionally restarts on uninstallation so the CE tests see that the service is already running and subsequent CE tests are skipped. The service restart is a temporary fix (required to reload the condor_mapfile to accept our daemons using the generated host cert) to be removed when htcondor-ce-2.0.9-3 is released so that we don't have a blind spot in the test coverage while it is in testing.
Test results: http://vdt.cs.wisc.edu/tests/20160929-1113/results.html

Applied fixes to both. The latter turned out to work because the initial packaging problem was EL7-only.


fatiloquent, escarps (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


Fail to install with install.sh script

Operating system and version:
Vagrant, using latest version of Ubuntu ('ubuntu/focal64' (v20240801.0.0))
How did you install nvm?

# Install nvm (if not already installed)
if ! command -v nvm &> /dev/null; then
  echo "Installing nvm..."
  sudo chmod +x /usr/bin/tail
  sudo mkdir -p /usr/local/git/share/git-core/templates && sudo chown -R $USER /usr/local/git
  curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.40.0/install.sh | bash
  export NVM_DIR="$HOME/.nvm"
  [ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"
  [ -s "$NVM_DIR/bash_completion" ] && \. "$NVM_DIR/bash_completion"
fi

What happened?
 Installing nvm...
   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                  Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 16555  100 16555    0     0   274k      0 --:--:-- --:--:-- --:--:--  274k
 => Downloading nvm from git to '/home/vagrant/.nvm'
Cloning into '/home/vagrant/.nvm'...
warning: templates not found in /usr/share/git-core/templates
main: line 166: 18496 Segmentation fault      (core dumped) git clone https://github.com/nvm-sh/nvm.git --depth=1 /home/vagrant/.nvm
Failed to clone nvm repo. Please report this!

What did you expect to happen?
NVM to install and be usable.
Is there anything in any of your profile files that modifies the PATH?

No.

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

default:   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying 2606:4700:10::6814:172e:443...
* TCP_NODELAY set
* Connected to nodejs.org (2606:4700:10::6814:172e) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/certs/ca-certificates.crt
  CApath: /etc/ssl/certs
} [5 bytes data]
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
} [512 bytes data]
* TLSv1.3 (IN), TLS handshake, Server hello (2):
{ [122 bytes data]
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
{ [19 bytes data]
* TLSv1.3 (IN), TLS handshake, Certificate (11):
{ [4835 bytes data]
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
{ [520 bytes data]
* TLSv1.3 (IN), TLS handshake, Finished (20):
{ [52 bytes data]
* TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
} [1 bytes data]
* TLSv1.3 (OUT), TLS handshake, Finished (20):
} [52 bytes data]
* SSL connection using TLSv1.3 / TLS_AES_256_GCM_SHA384
* ALPN, server accepted to use h2
* Server certificate:
*  subject: CN=*.nodejs.org
*  start date: Feb 28 00:00:00 2024 GMT
*  expire date: Mar 30 23:59:59 2025 GMT
*  subjectAltName: host "nodejs.org" matched cert's "nodejs.org"
*  issuer: C=GB; ST=Greater Manchester; L=Salford; O=Sectigo Limited; CN=Sectigo RSA Domain Validation Secure Server CA
*  SSL certificate verify ok.
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
} [5 bytes data]
* Using Stream ID: 1 (easy handle 0x5558b3b5f0e0)
} [5 bytes data]
> HEAD /dist/ HTTP/2
> Host: nodejs.org
> user-agent: curl/7.68.0
> accept: */*
> accept-encoding: deflate, gzip, br
>
HTTP/2 200
date: Sun, 04 Aug 2024 22:58:57 GMT
content-type: text/html
cache-control: public, max-age=3600, s-maxage=14400
cf-cache-status: HIT
age: 8627
last-modified: Sun, 04 Aug 2024 20:35:10 GMT
vary: Accept-Encoding
strict-transport-security: max-age=31536000; includeSubDomains; preload
x-content-type-options: nosniff
server: cloudflare
cf-ray: 8ae227b2ea11c3fd-EWR
content-encoding: br

{ [5 bytes data]
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
{ [230 bytes data]
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
{ [230 bytes data]
* old SSL session ID is stale, removing
{ [5 bytes data]
* Connection state changed (MAX_CONCURRENT_STREAMS == 100)!
} [5 bytes data]
< HTTP/2 200
< date: Sun, 04 Aug 2024 22:58:57 GMT
< content-type: text/html
< cache-control: public, max-age=3600, s-maxage=14400
< cf-cache-status: HIT
< age: 8627
< last-modified: Sun, 04 Aug 2024 20:35:10 GMT
< vary: Accept-Encoding
< strict-transport-security: max-age=31536000; includeSubDomains; preload
< x-content-type-options: nosniff
< server: cloudflare
< cf-ray: 8ae227b2ea11c3fd-EWR
< content-encoding: br
<

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
* Connection #0 to host nodejs.org left intact

I'm not familiar with "git-core/templates", but the segfault seems to be coming from git itself.
The rest of the issue template would be helpful, particularly git --version.

Any chance it’s related to this? https://stackoverflow.com/a/6501915

Based on my understanding, it shouldn't be. In my Vagrantfile I only copy over the local source code, and ignore all the other stuff that isn't needed:
# Avoid syncing unnecessary files to the VM
config.vm.synced_folder ".", "/projectfolder", type: "rsync", rsync__auto: false, rsync__exclude: [
  # Ignore the Git directory
  ".git",
  # Avoid permission errors in VMs by allowing the Docker containers
  # to create their own internal versions of the database files
  "backend/docker/data/*",
  "backend/docker/questdb/db/*",
  # Ignore testing data
  "testdata/",
  # Avoid copying dependencies and build artifacts to the VMs
  "*node_modules*",
  "*__pycache__*",
  "rust/target",
  # Skip Virtual Machine files
  "images/*",
  "*.ovf",
  "*.ovf",
  "*.vmdk",
  "*.zip"
]

I installed git inside the setup.sh provisioner:
# Provisioning with a shell script
config.vm.provision "shell", name: "Initial setup", privileged: false, path: "./vagrant/setup.sh", env: {"SHELL_ARGS" => ENV["SHELL_ARGS"]}
``

Like so:

```sh
sudo -E apt-get -qq install git coreutils

# Install nvm (if not already installed)
if ! command -v nvm &> /dev/null; then
  echo "Installing nvm..."
  curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.40.0/install.sh | bash

  export NVM_DIR="$HOME/.nvm"
  [ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"
  [ -s "$NVM_DIR/bash_completion" ] && \. "$NVM_DIR/bash_completion"
fi

Open to any suggestions, I'm nearing my wits' end on this one, I've been stuck on this issue for three days.

 If anyone else hits this issue, check your e-BPF rules, you may be unintentionally blocking the install at the kernel level 🙉
My apologies, have a good day.

Glad you figured it out!


[fix](group commit) Fix group commit error log when decommission (#39899)

pick https://github.com/apache/doris/pull/39899

run buildall

run buildall


Null Safety 0.6.0

Fixes #41

Support Sound Null Safety
Function comments for possible thrown platform exceptions
Version bump to 0.6.0
Update example project

~RaisedButton~ -> ElevatedButton

gitignore some generated files
Prefer final instead of var across the plugin and the example

@bdlindsay Thank you very much!

Thanks for pulling in and tagging the pub.dev release so quickly!


geopandas vs geopandas-base dependency

Aside from fiona, geopandas-base has all the dependencies required for dask-geopandas. Would it be preferable to drop the dependency down to geopandas-base for the suffix-free package (and ensure fiona is in the dep list), or publish a minimal version under dask-geopandas-base?

Yes, we should use geopandas-base here, thanks for bringing this up (I suppose this is from before we had that base package). Especially since we don't even use fiona in dask-geopandas.
I don't think we need to add extra dependencies to the list. If we people want the full dependency list of geopandas, they can install both conda install geopandas dask-geopandas? (and then just conda install dask-geopandas only gives you the limited set of dependencies)


Update file_functions.R

Fixes to accommodate upcoming major update this.path_2.0.0

Thanks for the pull request.  But it won't pass the cmd checks.  I already tried it.  Since the sys.pathI() function doesn't exist in this.path yet, the cmd check fails.  I'm still trying to figure out what to do.

I gues "fails" is too strong. There is a warning.  But that will be enough to bounce it on CRAN.

When are you planning to deploy?

Yeah, I see what you mean, I forgot about that. I think something like this.path::sys.path() would work without a warning.
I was hoping to deploy the update in the next two weeks, that is if the maintainer of {neptune} gets back to me before then. If not, I guess I'll wait longer. The code is ready to go, just doc fixes remaining.

Yes. The cmd check is too smart.  I was thinking about deploying a max version of this.path until the new version is ready.

I just submitted to CRAN with a < 2.0 restriction on this.path.  Let's see what they say.  I already removed the this.path dependency on the logr package.  It is now pointing to common.  So I have only one call to this.path.  Thank you very much for coordinating with me on this change.

I'm thinking they'll reject it, I think they expect all CRAN packages to play nicely with the most recent versions of other CRAN packages. I think I found a solution, using this.path::this.path(envir = NULL, srcfile = NULL) instead of this.path::sys.path() would produce the same behavior without throwing a warning.

Another thing that could work would be to turn off keep.source before creating the function, then switching it back to default before continuing.

OK.  I'll keep this as backup in case they reject.

It passed.  But it may still cause trouble when you deploy the update.  If so, I'll change to the code you have here or something else.

I removed the dependency on this.path().  So this change won't be necessary.  Thanks for the suggestion and your concern about breakages.

So you stole my code from Github and Stack Overflow and passed it off as your own, but you also did a bad job at that too because there are several mistakes that make it practically unusable. Do you need me to point them all out? C'mon man, you can do better than this.


Polymer binding to on- events throws exceptions in 0.8.7

In 0.8.7 of polymer you start needing to make event bindings of the form
on-click="doStuff"
but when you do, you get exceptions of the form
Class '_EventStreamSubscription' has no instance getter 'value'.
NoSuchMethodError : method not found: 'value'
Receiver: Instance of '_EventStreamSubscription@0x3918afae'
Arguments: []
You can see this in the click counter example, presumably anywhere else that uses event handlers. The exception will get caught, so if you aren't getting all exceptions it will continue and work. But then you can't ever debug an actual exception in the application.

This issue has been moved to dart-lang/polymer-dart#485.


Add setTabColor, openTabColorPicker to docs

Does what it says on the label.
See https://github.com/microsoft/terminal/pull/6567

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

File
Status
Preview URL
Details

TerminalDocs/customize-settings/key-bindings.md
:white_check_mark:Succeeded
View

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

I think that, as a maintainer of this repo, you can retarget the PR @cinnamon-msft :)

Oh I didn't know! I'll look into how I can do that. Looks like this is all covered in #90 though, so I'm going to close this one out. :)


CIS Scan - scheduled scan keeps trying to execute after the cluster is deleted

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a cluster and enable scheduled CIS Scans
Delete the cluster

Result:
After some time the Scheduled scan tries to get executed in the deleted cluster:
2020/03/10 23:23:03 [ERROR] ClusterScanController scheduledscan/c-vk7cw:23 * * * * [scheduledScanHandler:sync] failed with : scheduledScanHandler: sync: error fetching cluster c-vk7cw: cluster.management.cattle.io "c-vk7cw" not found

Other details that may be helpful:
Environment information

Rancher version: 2.4 master-head (03/10/2020) commit id: 8ede17d10
Installation option (single install/HA): single

Verified on master-head - commit id: 330a4bed0

Deploy a cluster.
Enable Scheduled CIS scan on the cluster.
Deleted the cluster
Scheduled scan does not run on a deleted cluster.


docs(misc/Downloading): Latest version

Updated the CDN link and description to 1.3.14

CLA signed.


[Bug] .conf mayhem

Adding 2 new lines at the end of the conf file stops the program from starting.
my config file:
`max_stack_size = 100
show_tray_icon = true
pop_keybinding = Control + Alt + C
swap_keybinding = Control + Alt + S
clear_keybinding = Control + Alt + D
prevent_duplicate_push = false
`
Also, can we have the default config file being in the same folder as the executable? Or maybe have it check for a config there first.

Fixed the empty line parsing in f2ca1478c2fa8317d7b2a1f2c301946cba40b3b7 (now we allow blank lines  anywhere throughout the config.) I'll include that in the next release (probably with other suggestion) but lmk if you need it earlier.
As for the configuration file in the same folder as the executable, yes that's a good idea. I think I'll take the approach of checking the local directory for a configuration and prioritizing that one; but I want to think on it for a bit.
Thanks again for the bug reports and suggestions :D

No I don't need the fix to be released. Also I was wondering why you bumped up MINOR instead of PATCH for the last bugfix: cargo crates use Semantic Versioning.

Yes, it should have been a patch release (thought about that this morning.)
But, since this a binary and not a library the versioning is much less
important anyway (imo.)
On Thu, Sep 27, 2018, 17:00 Karrq<EMAIL_ADDRESS>wrote:

No I don't need the fix to be released. Also I was wondering why you
bumped up MINOR instead of PATCH for the last bugfix: cargo crates use Semantic
Versioning https://semver.org/.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DenialAdams/ripclip/issues/6#issuecomment-425241461,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB-fVWAVhNj88LvYOlBTzVVB_UXcrfS1ks5ufTxcgaJpZM4W9S0f
.

Added a test for this issue in 1c84e63f2276600995c9aeb0c78d2b1b04336c9c


[1.12] Mob Crusher gives essence without killing

**Industrial Foregoing Version: **
1.12.2 - 1.9
**TeslaCoreLib Version: **
1.12.2 - 1.0.15
Issue
So apparently the mob crusher killing mobs isnt actually a requirement for essence. Just mobs dying within its area of coverage. So with a max range upgrade and a mob crusher from mob_grinding_utils to kill with looting 10, i dont even need to power the machine to collect essence at a rate faster than a powered crusher can collect it

:wave: Sorry but Industrial Foregoing has ended support for version 1.12. I'm closing this issue as it will no longer be updated for that version. Thanks for playing with the mod.


kubeadm部署高可用案例中api-server连接etcd问题

我发现如果以kubeadm这样的方式来做高可用的，他每个master上的api-server的pod里面连接etcd的参数都会是 --etcd-servers=https://<IP_ADDRESS>:2379而不是etcd上的节点ip地址，这样会不会有问题的，因为当api-server没问题但是本地etc有问题时候那这个master始终会访问不了etcd了。但如果强行把这个改掉或者单独将这个部署成外部etcd的话，那证书不会很难搞呢。

This is design choice～
If there is something wrong with the local etcd, it's highly possible that there is some malfunctioning in the machine, and is not recommended to serve as part of the master-node.

This is design choice～
If there is something wrong with the local etcd, it's highly possible that there is some malfunctioning in the machine, and is not recommended to serve as part of the master-node.

但有没有人是部署好以后进行修改的呢，还有现在生产环境上用的二进制部署的比较多还是kubeadm呢？

You can check the "external-etcd setting" for kubeadm.
It may fit your requirement:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/high-availability/#external-etcd-nodes

You can check the "external-etcd setting" for kubeadm.
It may fit your requirement:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/high-availability/#external-etcd-nodes

好的。非常感谢，我也正在看这个配置考虑是否在外部部署etcd


Pallet-Multisig to framev2

relates: #7882
Following the upgrade guidelines here: https://crates.parity.io/frame_support/attr.pallet.html#upgrade-guidelines.
:warning: Breaking Change :warning:
From https://crates.parity.io/frame_support/attr.pallet.html#checking-upgrade-guidelines

storages now use PalletInfo for module_prefix instead of the one given to decl_storage: Thus any use of this pallet in construct_runtime! should be careful to update name in order not to break storage or to upgrade storage (moreover for instantiable pallet). If pallet is published, make sure to warn about this breaking change.

So users of the Multisig pallet must be careful about the name they used in construct_runtime!. Hence the runtime-migration label, which might not be needed depending on the configuration of the Multisig pallet.

Kusama and Polkadot both use Multisig in construct runtime! no need for migration. Rococo does not use pallet-multisig.

Don't mind me, just seeing if it works :-)
/cla run-cla-check

Haven't been free the last week, thanks for the little fixes :), should be good to go now

bot merge


Allow for study-specific arguments

It would be good to have a loose dict in the StudyConfig class that can have key/value pairs added via a CLI arg, so that study authors can tweak behavior that may be unique to their specific use case.

addressed via #248


[broker] topic closed normally may still try to closeFencedTopicForcefully

Describe the bug
Pulsar cluster with topicFencingTimeoutSeconds>0, the topic which is closed normally should not having the log [A] Topic remained fenced for 10 seconds and is already closed (pendingWriteOps: 0).
To Reproduce
Steps to reproduce the behavior:

Deply a cluster with topicFencingTimeoutSeconds > 0, for example topicFencingTimeoutSeconds=10
Trigger topic A to close normally
closeFencedTopicForcefully still will be run, as [A] Topic remained fenced for 10 seconds and is already closed (pendingWriteOps: 0) will be printed.

Expected behavior
If topic A close normally, [A] Topic remained fenced for 10 seconds and is already closed (pendingWriteOps: 0) should not be logged.
Desktop (please complete the following information):

OS: linux
Pulsar version: 2.8.3

Just a warning log, it is ok.


How to do count many-to-many?

Sails version:0.12.7
Node version:6.2.1
NPM version:3.9.3
Operating system:centos 64

I have too model like bleow:
// User.js
module.exports = {
    attributes: {
        name: {
          type: 'string',
          required: true
        },
        app: {
          collection: 'app',
          via: 'user',
          dominant: true
       }
    }
}

// App.js
module.exports = {
    attributes: {
       name: {
          type: 'string',
          required: true
       },

       user: {
          collection: 'user',
          via: 'app'
       }
    }
}

Now, how can I do count user numbers by a app.id or app.name whit condition which part of some attributes in user like this:
User.count({some condition}).exec(...)

I am sorry the same issue which in waterline(https://github.com/balderdashy/waterline/issues/1399). But nobody reply in there

my waterline version is 0.11.4

Hey @cage1618, what you're describing is a WHERE subquery, which is not supported out of the box by Waterline alone.  Instead, you'll want to take advantage of .query()
Note that we've made good progress on making this easier and cleaner in Sails v1.0/Waterline 0.13 (see http://github.com/node-machine/driver-interface for more info).

@mikermcneil, my database is mongodb, so I can't use query method, has another to do this? thanks!

@cage1618 you'll want to use .native()


Merge branch 'branch-23.12' into main

Merge branch 'branch-23.12' into main
Change version to v23.12.0
NOTE: merge this PR with "Create a new commit"
Signed-off-by: Tim Liu<EMAIL_ADDRESS>

This PR is not the final merge to main change.
Draft the PR for the in-advance review purpose
NOTE: There are still some open PRs targeting 23.12, will do the final merge after them get merged.
https://github.com/NVIDIA/spark-rapids/pulls?q=is%3Apr+is%3Aopen+base%3Abranch-23.12+

build


IOS - Security - Default contact method error when closing an account open with phone number

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

Action Performed:

Open settings
Go to security
Close account
Write the the phone number that is the default contact method

Expected Result:
Writing the phone number that is the default contact method should allow us to close the account
Actual Result:
Writing the phone number that is the default contact method caused error
Workaround:
Unknown
Platforms:

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

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

Version Number: 1.319-7
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/252a5b8d-18e0-4de1-a27a-f05441b72625
Expensify/Expensify Issue URL:
Issue reported by: requested
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686242248690479
View all open jobs on GitHub

I will review by EOD.

I'm unable to reproduce this when entering my default contact method. For the record, mine is an email. It prompts me to close the account when I enter it in newdot.

This issue is only reproducable when your default contact method is a Phone number that is outside US.
As the above proposal suggests, the contact method is formatted with formatPhoneNumber it will include the space, so you only texted your phone without any space, it won't pass the validation.
Can you please review it again

Hi, @isabelastisser just following up on the review. I am still able to reproduce this, have you checked my comment?
https://github.com/Expensify/App/assets/49120316/eb0f440f-77c5-4e74-90d8-2de3153e195b


[Threading] Fix threading package defaults.

These were supposed to work for cross-compilation, but because of an error in Threading.cmake, they didn't.  Also, the test rig should be using threading_package_name() rather than threading_package_default(), so that any user override takes precedence.
Fixes #59568.

@swift-ci Please test

I have a few more changes to go into this one yet, I think.

@swift-ci Plese test

I think this will work, but it is setting the threading library very late in the configuration process and at each location, which can get hairy.
Do you mind rewriting this to follow the established process of setting SDK-specific variables in the configure_sdk_*() macros instead? That way it's set once in a SDK-specific global variable that you can print out in _report_sdk(), which will be less error-prone over time.


Collect information about the vpn provider and use it in white- and blacklists

Added to constructor in 69f94f83bf12f5b29121731135c9540be0022e20.


Different flags inputs for deploy and update-traffic commands

TL;DR
When passing flags input they are passed on to both deploy and update-traffic commands, this is problematic because for example you can't pass --min-instances if you're also passing an input like tag_traffic because it will error out with unrecognized arguments when it tries to call update-traffic with the same flag.
Detailed design
No response
Additional information
No response

Hi @mrkstwrt - this is working as intended. flags is an advanced option, and we do not parse or validate the flags. I just updated the documentation in 4a1d803.


Sound and Music

Adding Sounds/Music to our project.

Check out NCS (NoCopyrightSounds) on YouTube!

implemented and documented


Where is the  batches.meta.txt file?

Hi, @gskielian
Where is the image batches.meta.txt file?
I cannot change the file because of no file.

+1 - im curious about this too...

I added examples for the batches.meta.txt file in latest push -- this isn't used by the Python scripts but is useful later for Tensorflow processes, which can utilize this to map numerical labels with the labels in "batches.meta.txt" file.
This is more of an artifact of my "Png-2-Cifar10" repo which I adapted this from, but could be one way to keep track of your mapping of folder numbers to categories.


Research requirements for Maskbook plugin

Maskbook.com
From me:

The Maskbook plugin requires a category of permission that we haven't explored yet: What is the simplest way we can request permission to, say, decrypt messages on facebook? Permission to edit text by selector? We'd love to make it less than "permission to view and edit all info on all pages", and while "all info on Facebook" is much better, I wonder if we could be even more specific?

From yisiliu:

We are requesting only "Browsing History" and "Manage Downloads" permission and the rest are just web hacks :)

Posting here for more discussion

Hi, the "" permission is "webNavigation" in the manifest file.
Maskbook use this permission to let the script of Maskbook inject to Facebook as early as possible. ("document_start" in the manifest file is not early enough).
And on Chrome, there is no chrome.contentScripts API (Firefox does), so without the "webNavigation" permission, Maskbook cannot inject content script with the optional permission "<all_url>" (Maskbook require for the specify site in the runtime).
https://github.com/DimensionDev/Maskbook/blob/master/src/background-service.ts#L48

Hi, the "" permission is "webNavigation" in the manifest file.
Maskbook use this permission to let the script of Maskbook inject to Facebook as early as possible. ("document_start" in the manifest file is not early enough).
And on Chrome, there is no chrome.contentScripts API (Firefox does), so without the "webNavigation" permission, Maskbook cannot inject content script with the optional permission "<all_url>" (Maskbook require for the specify site in the runtime).
https://github.com/DimensionDev/Maskbook/blob/master/src/background-service.ts#L48

@danfinlay  What's your opinion? :D

I was actually hoping to get a higher-level description of your needs: Rather than listing the current WebExtension APIs that you use, I was hoping to refine the definition of the minimum possible API that MetaMask could provide to enable the same use case.
One way I can tell that WebExtension's APIs are failing here is that the permissions you're requesting don't sound obviously related to what you're doing, and so it is hard for a user to provide informed consent for these permissions.
Instead, maybe "edit the way posts are displayed and posted on Facebook"? Or maybe just "ability to manipulate facebook", etc..

Maskbook will request access for a certain site (like facebook.com or twitter.com) if user choose to connect their facebook/twitter account.
After the permission is granted, Maskbook will be able to read the post and try to decrypt them. When user create encrypted posts by Maskbook, Maskbook will automatically paste the encrypted post into the post box of facebook/twitter but not sending them automatically.
Maskbook will use the EC key to encrypt the post, sign the content of the post.
Maskbook will collect info like nickname, user id, avatar and relationship(friend/non-friend) to other person.
All of information collected above will be stored locally. In future, Maskbook may have cross device sync so the information will be transported by end-to-end encryption.

So for right now, Maskbook could operate entirely with:

permission to view & edit info on twitter and facebook

And you wouldn't even need ability to access the internet?
We provide keys to every plugin, so you don't even need permission for that.
This would represent very strong privacy guarantees, and I think I can endorse giving you a method to edit these sites.
Async Site Editing
Now unlike a WebExtension, where you get a contentscript per page, we would not be able to give you direct DOM access, your access to the page would always be async. Is this a deal breaker? I'm curious if our registerApiMethodHandler() method could be good enough for this purpose. We will probably at least need this feature in capnode first, or we can switch to CapTP.
We could also consider making specialized permissions just for this use case, like "Permission to modify how I post and view posts on twitter/facebook", although this gives us a moving target to try to support.

Alright, I thought connect to internet is too basic to mention. Maskbook need to connect to the internet to dispatch AES keys to other Maskbook users.
Maskbook is using GunDB (a decenterialized graph database) which is using WebSocket to connect to the bridge server. In future, Maskbook will connect to Matrix (a decenterialized IM protocol) by HTTPs.
Our infrastructure of injecting UI to the webpage is heavily depends on DOM operation. https://github.com/DimensionDev/holoflows-kit/
But Maskbook doesn't require the DOM to be the "real" DOM because our infrastructure already wrap the DOM into a Proxy (https://github.com/DimensionDev/Holoflows-Kit/blob/master/src/DOM/Proxy.ts).

@Jack-Works @Tedko I'm going to close this issue since I'm about to archive this repository, but if you are still interested in building a Maskbook snap and documenting your present requirements, please open an issue in https://github.com/MetaMask/snaps-skunkworks.


Fatal: detected dubious ownership in repository

We started receiving the following error in our App Service Build Service (Kudu) continuous deployment setups today.  We're not sure when it would have started, but we had successes last week.  We've had no change in configuration, and I have verified on 3 separate app service/repository combinations that this is occurring universally for us.  Any ideas on what might be going on?
fatal: detected dubious ownership in repository at '[redacted]'\n'[redacted]' is owned by:\n	'[redacted]'\nbut the current user is:\n	[redacted]'\nTo add an exception for this directory, call:\n\n	git config --global --add safe.directory '[recacted]'\n\r\nC:\Program Files\Git\cmd\git.exe remote add -t master origin "git@github.com:Lightburn/Doral.Website.git"
Repro steps.
Triggered a build with a commit into the associated branch (multiple repos/app services)

We also started receiving this error in our production build server around the same time.

Please navigate to SCM  (Kudu) / Debug Console and run git config --global --add safe.directory "*" command.   See if that mitigate the issue.    Do share the stamp name env information (WEBSITE_HOME_STAMPNAME) - it should be something like waws-prod-.....

Hi
Please file incidents in Azure, so we can better track and assist on this case:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
Thanks,
Joaquin

Sorry, I forgot to mention that we already tried that. The result on our app server with problems is:
error: could not lock config file C:/home/.gitconfig: Permission denied
We'll look into filing this on azure support.

Thank for reporting.   We have identified the issue and currently is working on the fix.  The issue is due to the new git (2.39.2) rollout by Azure App Service along with missing write permission on c:\home directory for sites created on or prior to 2018 - leading to the issue.   We will update the progress of fix rollout.  Thanks for your patience,

To update, the fix is being rolled out and may apply to different regions at different time.   The ETA for completion to all regions is end of Sat 9/16/2023 if not earlier.

Fix has been rolled out completely - apology for the incoveneince.


update readme and ui

readme (and code comments) updated re: loopback connections in development and side-load scenarios
ui updated to add device information (machine name and os) to the context menu and to split the tag and ipd setting to separate buttons / dialogs.
this closes #31, #39

Hi @davidkline-ms, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (David Kline (XBOX)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Per document read-access security

Summary
Allow to control, who can retrieve document by declaring security object along with the document
Desired Behaviour
This proposal may look slightly less complex than other proposals I have found in the suggestion box while writing this article. For example #1504. My proposal should be easy to implement and backward compatible.
Default behaviour is remain unchanged. Once the user or its role is listed in members section of the _security, all of the documents of the database can be retrieved without limitation. The change comes with a new reserved key "_security" included into the document
{
"_id":"example",
"_security":{"roles":{...}, "users": {...}},
}

The key _security have exactly the same format as members sections of the security object. You can define roles and users. If there is a match with the current user or its role, the document can be retrieved through the HTTP API to the user. If there is no match, the error object is retrieved instead. The key itself is visible to the readers and can be modified. There can be a VDU function  to validate the content of _security field
When the such security is applied? It is applied generally everywhere, where document is prepared for output to the response. It includes: GET, _bulk_get,  include_docs, etc... However, it doesn't prevent to read the document by the query server to build views. So document (or some part) still can be retrieved through the view (but not though include_docs)
Design documents
To prevent the user to access the such documents through the design's document functions, the security object is also applied to design documents. In this case, it also prevents unauthorised users to execute functions on the design docs. (excluding VDUs)
{
"_id":"_design/topics",
"_security":{"roles":{...}, "users": {...}},
"views":{...},
"shows":{...}
}

Mango indexes
Mango indexes introduces little challenge to this proposal because, the _find function allows to specify any fields from the document and acts as include_docs with filtering. Applying the above proposal can cause, that _find will be forced to filter out the inaccessible documents from the result
This has a several impact to usefulness of the mango queries. So my proposal suggest to extend _security for the _index definition
{
    "index": {
        "fields": ["foo"]
    },
    "name" : "foo-index",
    "type" : "json",
   "_security": {
      "roles":["reader"],
      "fields":["_id","_rev","bar","baz"]
    }
}

So the index definition above makes index available only to users with the role "reader". They will be able to retrieve "_id","_rev","bar" and "baz" fields from the documents that fails to security check otherwise for that users. Accessing the other fields causes that these documents will be filtered out of the result.
Replication
Replication is executed under a specified user context and  all the above rules are also applied. If the replication doesn't have access to a document, the document is not replicated.
Possible Solution

check and apply _security key when document is serialised to the output
check and apply _security key while execution function in a design document (exclude VDU functions)
check and apply _security key while searching for the index
implement field filtering for _index and _find

Additional context
Currently, the database is open to everyone who has a role in members section. It is hard to hide some documents to some users, There is only solution to use two or more databases while indexing these databases can be difficult. CouchDB has also one special database, where only owner of the document can retrieve its content - _users - but this behaviour is hard-coded and can't be implemented otherwise. By introducing per-document security this hard-coded behaviour is no longer needed.
Related issues #1558, #1724, #1504
PS: Sorry for any weird wording and mistakes in my English. I hope that everything is understandable.

@ondra-novak heya, I am working on implementing _access and haven’t yet been very open about the progress. I’ll update 1524 soonish. I’ll take your issue here as another vote that a feature like this is very needed.

I know the post is closed, but I'm highly interested in the status of this feature as the "per-user" database model is real NO-GO for my projects.

Updates will be on #1524 when available.


list index out of range [x] run.Feed

Initial Check

If the issue is a request please specify that it is a request in the title (Example: [REQUEST] more features). If this is a question regarding 'twint' please specify that it's a question in the title (Example: [QUESTION] What is x?). Please only submit issues related to 'twint'. Thanks.

Make sure you've checked the following:

[] Python version is 3.6;
[] Updated Twint with pip3 install --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint;
[] I have searched the issues and there are no duplicates of this issue/question/request.

Command Ran

c = twint.Config()
c.User_id = '792349094861824000'
c.Store_object = True
c.Since = '2016-01-01'
output =  '792349094861824000.csv'
c.Format = "{id}\001{date}\001{time}\001{user_id}\001{timezone}\001{tweet}\001{location}\001{hashtags}\001{replies}\001{retweets}\001{likes}\001{link}\001{is_retweet}\001{mentions}"
c.Output = output
nest_asyncio.apply()
twint.run.Profile(c)

Description of Issue
I have run the command above, to collect all favorite in CSV file, I met the problem below:
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:noDatalist index out of range
CRITICAL:root:twint.run:Twint:Feed:Tweets_known_error:list index out of range
list index out of range [x] run.Feed
[!] if get this error but you know for sure that more tweets exist, please open an issue and we will investigate it!
However, there are retweets in this user's timeline https://twitter.com/Fw669Tq2UvJdvYE?lang=zh-cn
Environment Details

Using Windows, Linux? What OS version? Running this in Anaconda? Jupyter Notebook? Terminal?
Windows 10, Spyder 3.7, updated to latest version with pip install --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint

Please add c.Profile_full = True before twint.run.Profile(c)

Now I edited the code so that you will have to call twint.run.Profile(c)
So your code will be:
import twint

c = twint.Config()
c.User_id = '792349094861824000'
c.Store_object = True
c.Since = '2016-01-01'
output =  '792349094861824000.csv'
c.Format = "{id}\001{date}\001{time}\001{user_id}\001{timezone}\001{tweet}\001{location}\001{hashtags}\001{replies}\001{retweets}\001{likes}\001{link}\001{is_retweet}\001{mentions}"
c.Output = output
nest_asyncio.apply()

twint.run.Profile(c)

Thanks so much for your quick response and editing of the codes!
Twint is such a wonderful project!!!

Another question: is there any possibility to get the replies data? I mean the "tweet and replies" in user's home timeline, like this https://twitter.com/Fw669Tq2UvJdvYE/with_replies?lang=zh-cn
Thanks so much for your efforts for this great project!

Great to hear!
I'm going to close this issue. If you get other errors related to this issue feel free to re-open.


Embedded Schemas Intro

A continuation of #3761 started by @matteing

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Made it possible to disable the deletion of Unknown Commands with SmartQueue

I have made it possible to disable the automatic deletion of unknown commands when using the SmartQueue feature.
While this might not be used by many people, it can be useful for those that can't have their commands deleted if they aren't used by something using DIH4JDA. Might be especially useful since a disabled SmartQueue overrides all commands, therefore also deleting unknown commands

An "unknown" command is a command, which exists in a guild/global, but was not found in the current code.
I think it makes sense to automatically remove them, as they have no associated handler etc.
But, sure. If somebody wants to not have their command deleted ig thats fine, as long as its not enabled by default.
Suggestions:

Log commands which would've been deleted
Change the boolean and method's name to deleteUnknownCommand & disableUnknownCommandDeletion, respectivly

An "unknown" command is a command, which exists in a guild/global state, but was not found in the current code.

I know, I made this change so people who might have multiple things using the same token (not necessarily simultaneously) can do so without these "instances" deleting each other's commands.

Great PR


Add flag for Makefile dependency rules generation

Add flag that will make cython output a rule suitable for make describing the dependencies of the main source file.
like this:
$ cython -M foo.pxd

foo.c: bar.pxd

Migrated from http://trac.cython.org/ticket/655

This would be useful for meson, too. For example, the pxd file is not tracked in this test case.

Since the cython tool does not deal with dependencies (it's a one-shot command), a better place to implement this seems to be the cythonize frontend.

cython still needs to deal with C headers though, correct?

Cython's dependency collection in Cython.Build.Dependencies does not look at transitive header files but only those that are mentioned directly via cdef extern from clauses. It wouldn't be able to decide which of them will be included in a platform/configuration specific way by the C preprocessor anyway.
It transitively collects .pxd, and .pxi files, as well as include files from cdef extern from sections (.h, .c, whatever people use there).
I'm pretty sure it also currently does not look into verbatim C code sections (docstrings of cdef extern from), so if they include anything, it won't be noticed. That's probably ok – if users want to include something here, they can just name it as the input file of that section. And if they include different files in a platform dependent way, then a) Cython can't decide it anyway and b) we are probably dealing with system provided header files which don't normally change.

I guess for cython the header files don't matter as long as they shouldn't trigger cython to re-compiler/transpile (at least from Meson's point of view) as the C/C++ compiler will generate its own depfile that would cover the inclusion of C language headers, so for Meson I guess that just means pxd files?

The feature was only added to the cythonize command for now, in #4563.
Whether it should be added to the cython command as well is less obvious since it does not currently deal with dependencies at all. Only cythonize has this intention.

Backported to 0.29.27 in https://github.com/cython/cython/pull/4576


Catch exceptions for all commands sent light.zha.Light.

Description:
Catch exceptions for all operations which may fail because of device
reachibility
More verbose debug logging on operations
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

Thanks @Adminiuga -- I actually just submitted a pull request for the same thing, but closed it now -- this looks great!

@Adminiuga Been testing this in 0.79.2 and so far it seems that it's working well -- If you do turn on a light that is physically switched off you will get "Couldn't change brightness level" message for the bellows delivery error, but I assume that is based on the internals of how the turn_on service calls the light with a set brightness. . . .

but I assume that is based on the internals of how the turn_on service calls the light with a set brightness

Yes, the light.zha.Light.async_turn_on() actually issues move_to_level_with_on_off Zigbee command which turns light on/off depending on the brightness level. So if the light is physically off, then move_to_level_with_on_off is going to throw DeliveryError so you  respectively the error states it couldn't change the brightness level.


Fix mavros and it's dependencies in melodic

geographiclib-tools wasn't defined, added it as a package based on the debian package's definition
uuid-msgs, geographic-msgs, mavros-msgs and mavlink didn't have native dependencies configured
Small QA issue in mavros due to a file's bash shebang

Is there anything left to merge? #663 wasn't merged yet but there aren't any overlaps and all of the comments were resolved, I don't see a reason to wait any longer.

Milestone 11 changes were finally merged on Sunday, please rebase on current thud-draft and squash all your commits into 1. Then I'll integrate it together with my milestone 12 changes.

And with webOS OSE it also shows bunch of return-type errors:
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp: In function 'Eigen::Quaterniond mavros::ftf::detail::transform_orientation(const Quaterniond&, mavros::ftf::StaticTF)':
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp:88:1: error: control reaches end of non-void function [-Werror=return-type]
   88 | }
      | ^
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp: In function 'Eigen::Vector3d mavros::ftf::detail::transform_static_frame(const Vector3d&, mavros::ftf::StaticTF)':
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp:102:1: error: control reaches end of non-void function [-Werror=return-type]
  102 | }
      | ^
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp: In function 'mavros::ftf::Covariance3d mavros::ftf::detail::transform_static_frame(const Covariance3d&, mavros::ftf::StaticTF)':
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp:122:1: error: control reaches end of non-void function [-Werror=return-type]
  122 | }
      | ^
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp: In function 'mavros::ftf::Covariance6d mavros::ftf::detail::transform_static_frame(const Covariance6d&, mavros::ftf::StaticTF)':
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp:153:1: error: control reaches end of non-void function [-Werror=return-type]
  153 | }
      | ^
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp: In function 'mavros::ftf::Covariance9d mavros::ftf::detail::transform_static_frame(const Covariance9d&, mavros::ftf::StaticTF)':
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp:186:1: error: control reaches end of non-void function [-Werror=return-type]
  186 | }
      | ^
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp: In function 'Eigen::Vector3d mavros::ftf::detail::transform_static_frame(const Vector3d&, const Vector3d&, mavros::ftf::StaticTF)':
/jenkins/mjansa/build-ros/webos-melodic-zeus/tmp-glibc/work/raspberrypi4-webos-linux-gnueabi/mavros/1.0.0-1-r0/git/src/lib/ftf_frame_conversions.cpp:227:1: error: control reaches end of non-void function [-Werror=return-type]
  227 | }
      | ^
cc1plus: some warnings being treated as errors

will add a fix for this as well.

Sorry for delay, this was finally merged together with other Milestone 12 changes now. If I've missed something then please create new PR.


Update to Swift 2.2

This commit fixes:

Warnings: "Use of string literal for Objective-C selectors is deprecated; use '#selector' instead"

Thanks @albinekcom


WAVALET transform analysis for photometry light curve

I am working on binary systems, for this, we are analyzing light curves and some of them are periodic. We have this data through observations with different telescopes. The data we obtain are Time, Flux, and error_flux. And I would like to do an analysis with your ssqueezepy code, to analyze the light curves through a WAVALET transform.  how do I define my time, and flux to be able to run the code?

I suggest familiarizing with some other libraries and their tutorials, or asking on StackOverflow for others' attention. This isn't a basics support forum, I'm afraid.


Upgrade Elasticsearch to version 6.x

Currently, only project indexing working. Investigating and trying to index the pages/files in a cleaner way. (Fixed)
Things need to be done

[x] Implement a cleaner way to index File
[x] Search for project
[x] Search for File
[x] Add highlighter for result description
[x] Implement facet searching in a cleaner way
[x] Configure travis to run the tests
[x] Fixup the tests in order to pass

This fixes #4183

@ericholscher I think you can take another look into this!
I will integrate the view and templated later today!

@ericholscher I have fixed the integration in view and template. But currently the description with highlighting is not showing in the file search result. I will look over into it later.
Can you run it locally and let me know about your feedback?

@ericholscher I have fixed the tests and now it is passing. 🎾
possible to merge?

@ericholscher Renamed the migration. ready to merge! :shipit:

Looks good. 👍


diversified_sampler is currently failing

diversified_sampler is currently failing with the error: Sampler aggregation must be used with child aggregations.
E.g. /builder?q=(dataset:mosaic_nih_v7,form:secondary,forms:!((name:primary,params:(interval:1),selection:(aggregation:histogram,field:year_fiscal_funding,type:integer)),(name:secondary,selection:(aggregation:diversified_sampler,field:title_of_funder,type:ES_text_w_keyword))),size:0)

This can't be created from scratch anymore as diversified_sampler is disabled for the primary axis.
We might want to manage all Bucketing nested aggs not used as nested aggs with a specific error message but for the time being it would seem overkill, closing.


Using BlockSDK in Marketing cloud.

Is it possible for now to use BlockSDK and custom block in MarketingCloud?
I have implemented my own application:
This app works in blocktester
But
Does not in MarketingCloud.
In addition, I deployed the sampleblock to Heroku but I see the same issue:
It does not work in MarketingCloud.
After debugging and modifying the source code of SDK I found the issue. Parent window in MarketingCloud doesn't answer on handShake.
In my local project, I have changed the code of SDK. And after that my custom block starts working in MarketingCloud.
  this._postToEditor = function (payload, callback, ttl) {
    var self = this;
-    // we only message up if we have
-    // validated the origin
-    if (!this._parentOrigin) {
-      if (ttl === undefined || ttl > 0) {
-        window.setTimeout(function () {
-          self._postToEditor(payload, callback, (ttl || 5) - 1);
-        }, 20);
-      }
-      return;
-    }
    this._messages[this._messageId] = callback;
    payload.id = this._messageId;
    this._messageId += 1;
    // the actual postMessage always uses
    // the validated origin
-    window.parent.postMessage(payload, this._parentOrigin);
+    window.parent.postMessage(payload, '*');
  };

Hello. It will not be possible to add your block to the Marketing Cloud until the April 21 release.


1.28.2 sap.ui.model.odata.v2.ODataModel createEntry(...)

I am currently trying to implement our MasterData CRUD Template for our OData services on HANA and PHP.
Besides my issues with setting batchMode = false and not able to handle success/error on using submitChanges() i ran into problems creating new entries.
Using
var oEntry = this.getView().getModel().createEntry(sCreatePath)
and using it via
oView.bindElement(sObjectPath);  // something like "/UmeUsers('id-1427727787066-30')"
There is no valid Context inside my View!
Also the oDataModell is doing a GET request on the creatingPath
/UmeUsers('id-1427727787066-30')
The .createEntry return an context and the entry is stored inside model, but it seems that the entry is not able to be bound using oView.bindElement using the binding Path.
that seems to be invalid for me, because i try to create a new entry?!?
Since 1.28.2 i use the default settings for defered batchGroup="changes" so editing existing entries works for me and i can use submitChanged() to update.
There is also a bug inside the createEntry line
// remove starting / for key only
sKey = sPath.substring(1) + "('" + jQuery.sap.uid() + "')";
because this works only for KEYS of type String!!!
What about keys of type integer, which needs an uri like
/UmeUsers(6)
If the UID will only be used internally as a phantomId for newly generated entries and filtered out on submitChanges(), this would be ok, but how can i disable the GET request on non existing data?
Do you have a small code snippet for me, how to be able to do CRUD operations using the new v2/OData model.
Currently the SAPI5 diagnostics Controll tree will not show binding infos for me, even if the binding is valid?!?
Any help would be greatly appreciated ;-)
Cheers Holger

Hi Holger,
thanks for reporting! I`ll forward this to the responsible OData expert to verify and will get back to you asap.
Best! Thilo

Hi Holger,
bindElement always triggers a request if the referenced entity does not exists in the models cache. So it could work only for entities that exists at the backend.
createEntry returns a context object that can be set directly to the control tree via setBindingContext.
It creates a transient entity object in the ODataModels cache. The context points to this object.
To create the entity at the backend you have to call submitChanges. If the creation was successful the context will be updated by the model so it points to the newly created entity.
The ID we create for the transient object is for binding purposes only and is absolutely correct. The real ID has to be created by the backend.
var oContext = this.getView().getModel().createEntry(sCreatePath);
this.getView().setBindingContext(oContext);
using CRUD methods should be easy:
oModel.read("/pathToMyEntity",{success: mySuccessHandler, error: myErrorHandler});
oModel.delete("/pathToMyEntity",{success: mySuccessHandler, error: myErrorHandler});
oModel.update("/pathToMyEntity", myEntityData, {success: mySuccessHandler, error: myErrorHandler});
oModel.create("/pathToMyEntitySet",myEntityToCreate, {success: mySuccessHandler, error: myErrorHandler});
Have a look at the API doc for the complete parameter set these methods provide.
BR,
Jan

Hi Jan,
thanks for the help.
I figured out after some test, that my problem consists of initially using oView.bindElement(...) (onMasterLoaded) and then try to set the createEntry via setBindingContext().
I fixed it using oView.unbindElement() before setBindingContext
var oView = this.getView(),
oContext = oView.getModel().createEntry(sCreatePath);
oView.unbindElement();
oView.setBindingContext(oContext);
Now it works as expected ;-)
Is this the right way of doing it?
Cheers Holger

Hi Holger,
the bindElements context is the leading one on a control/view. If you set a context via setBindingContext to a control that already has an ElementBinding it does only have an effect if the ElementBinding is relative. The newly set context will become the parent context for this relative ElementBinding.
I think in your case the ElementBinding is absolute, so the setBindingContext has no effect until you unbind it. So your approach is fine.
BR,
Jan


Make handlers pure functions

As discussed in #23 this PR changes how function action handlers are called and now the first argument is always the whole machine:
Example for a handler function in 6.x:
'add todo': function ({state}, todo) {
  return {
    name: 'idle',
    todos: [...state.todos, todo]
  };
}

Example for an arrow function as handler in 6.x:
'add todo': ({state}, todo) => ({
    name: 'idle',
    todos: [...state.todos, todo]
})

I've tried to update all examples and tests to match this change, let me know if I've missed anything.

👍 😸

6.0.0 published to npm.


Bring require()s up-to-date in sample code

The code samples were not actually runnable, because require("Repository.js") is no longer the correct way to import the Repository module.  Brought them up-to-speed.

LGTM


Fix for Codesmell check step1

Code Smell completed the following categories:

close resources for database
close resources for stream
equals()
unmodifiable collection
Random

Can this PR be maintained anymore?
if not, let's close it.

Can this PR be maintained anymore?
if not, let's close it.