Filter out duplicate collateral inputs in transaction build and build-raw

Resolves #4744

This is a small code change so I don't mind but for the future:
The way you re-used my commit kicks me out of the got commit authors (I saw you wrote it there but its not registered by got itself). If someone has contributed a large code change they might want this to show in the project statistics or on their profile. For that, you can copy the branch of that person (including their authored commits) and then modify in your own repository (even merging commits if thats what you are aiming for). The benefit is that the original author stays co-contributor, recognized by git. Something like the following should do the trick (in this particular case):
git remote add nielstron https://github.com/nielstron/cardano-node
git fetch nielstron
git checkout fix/filter-collateral-inputs
# < do your modifications of the branch >
git push -u origin fix/filter-collateral-inputs # prevents access errors by pushing into own/controlled repo

This is a small code change so I don't mind but for the future:
The way you re-used my commit kicks me out of the git commit authors (I saw you wrote it in the commit message but its not registered by git itself). If someone has contributed a large code change they might want this to show in the project statistics or on their profile. For that, you can copy the branch of that person (including their authored commits) and then modify in your own repository (even merging commits if thats what you are aiming for). The benefit is that the original author stays co-contributor, recognized by git. Something like the following should do the trick (in this particular case):
git remote add nielstron https://github.com/nielstron/cardano-node
git fetch nielstron
git checkout fix/filter-collateral-inputs
# < do your modifications of the branch >
git push -u origin fix/filter-collateral-inputs # prevents access errors by pushing into own/controlled repo

Thanks for this, I will follow this procedure from now on.


How to lock commands (using lockstrings?)

Hi everyone,
I´m completly new to rpi/MM/python but somehow managed to keep it running with some modules.  I have following problem (dont know if its a Problem about MMM-voice or MMM-Facial-Recognition):
If I hide my modules with the voice module, every module will be showed again when my MMM-Facial-Recognition Module logs in/out a user. I already read about using "lockStrings" which could block the Facial Recognition to not show the modules again as long as they are hidden with the mmm-voice command, but i have no clue how and where to set them correctly.
Maybe someone faced my problem before and is able to help me out, that would be really awesome. I´m grateful for every help.
Cheers Felix

ok i think i solved it. I was testing few things and added some lines in the MMM-voice.js data.
        } else if (notification === 'HIDE') {             MM.getModules().enumerate(function(module) {             module.hide(1000, function() {                 Log.log(module.name + ' is hidden.');             }, {lockString: module.lockStrings.join});         });         } else if (notification === 'SHOW') {             MM.getModules().enumerate(function(module) {             module.show(1000, function() {                 Log.log(module.name + ' is hidden.');             }, {lockString: module.lockStrings.join});         });
I dont know if this is the right way to do it but it works for me.


filesystem: correct the pool name in the ux backend handler

In the expandstorage handler pool name is incorrect,
Rook adds the filesystemName prefix to every datapool, which was missing in the storageclass parameter
https://github.com/rook/rook/blob/44e96c8ab67a27121354a04b121b7f16b6a38894/pkg/operator/ceph/file/filesystem.go#L321

/cherry-pick 4.18

/assign @malayparida2000 @SanjalKatiyar @iamniting @Madhu-1 @travisn

/cherry-pick release-4.18


Stay on Top FancyZones Zones

Allow FancyZones zones to be designated as a Stay-on-top Window.
The premise is simple, give the option in the Layout Editor to designate a zone as a Pined or Stay-on-top zone. So, when a Window is dragged into a zone that is labeled as Stay-on-top, that Window will stay on top of other Windows even if it is not in focus. This would be particularly useful in a Canvas layout, because it would allow layouts to be built with zones that intentionally overlay other zones. Examples where this would be useful include: having a calculator stay open in the corner, and keeping a perpetual chat window open. In a more abstract sense, this proposal allows for better multitasking on computers that do not have multiple monitors.
There is also a separate issue, https://github.com/microsoft/PowerToys/issues/13, to allow for generic Stay-on-Top behavior that would be activated by a hotkey or integrated into the general Windows context menu. I am not sure if this proposal can be completed without the groundwork in that issue or if this proposal can exist completely independent from that issue. Regardless, I want to highlight the differences between this and https://github.com/microsoft/PowerToys/issues/13. This proposal sets up the behavior in the Layout Editor and exists as a FancyZones feature. Whereas when activated, https://github.com/microsoft/PowerToys/issues/13 adds general Stay-on-Top behavior to all Windows at all times.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

@yak27
this is tracked in #6611 by #919


HIVE-28440: unblock hcatalog parquet project pushdown

What changes were proposed in this pull request?
to unblock hcatloader parquet table project pushdown. HCatLoader sets the needed columns intending to be used for project/filter pushdown, however, org.apache.hadoop.hive.ql.io.parquet.read.DataWritableReadSupport#init ReadContext skips project/filter pushdown when job properties 'columns' is not set. This change sets the 'columns' property.
Why are the changes needed?
to unblock hcatloader parquet table project pushdown
Does this PR introduce any user-facing change?
No
Is the change a dependency upgrade?
No
How was this patch tested?
tested locally

LGTM


Step 5b in the documentation problem

When I make a request to http://localhost:9000/docs/swagger-ui/index.html?url=/docs/swagger.json Chrome just downloads a file called index.html.
It seems to be the right file. I don't know why it is downloaded instead of being displayed. Maybe because the endpoint uses Assets?
Help.

I haven't seen this problem before, u are the first one who reported this. So I am not sure if using Assets would causing such an issue. It might be your play mime configuration?

@i-am-the-slime can you try the new sbt-plugin setup?

https://github.com/playframework/playframework/pull/6058
just upgrade play to 2.5.3


compile fall when change source structure

Hi all
Another problem with ceedling. I really want to separate my source code *.c in /src and header files *.h in /inc  likes bellow:
├── inc
│   ├── module_A
│   │   └── module_a1.h
│   ├── module_b.h
│   ├── vfAppCache.h
│   ├── vfMemory.h
│   └── vfRingbuffer.h
├── project.yml
├── rakefile.rb
├── src
│   ├── module_A
│   │   └── module_a1.c
│   ├── module_b.c
│   ├── vfAppCache.c
│   ├── vfMemory.c
│   └── vfRingBuffer.c
├── test
│   ├── module_A
│   │   └── test_module_a1.c
│   ├── support
│   ├── test_module_b.c
│   └── test_vfAppCache.c
├── vendor

And configure the the path field ceedling project file as
:paths:
  :test:
    - +:test/**
    - -:test/support
  :source:
    - src/**
  :include:
    - inc/**
    - vfInclude/**
  :support:
    - test/support

module_b will call some function from module_a1. The test case works fine.
In another test case test_vfAppCache with the same theory. vfAppCache will call some function in vfRingBuffer
But when I tried to run the test case. The compile noticed a compile error likes bellow:
Test 'test_vfAppCache.c'
------------------------
ERROR: Found no file 'vfRingbuffer.c' in search paths. However, a filename having different capitalization was found: 'src/vfRingBuffer.c'.
rake aborted!

And why is this a ceedling problem? It actually gave you a very descriptive error.
Rename your .h file with proper capitalization or check what is capitalization with #include

Yeah, I must agree with Letme in this case. This is one of the cases where Ceedling actually provides a nice descriptive error (our goal, but one which often falls short). C is a case sensitive language (even if Windows isn't). It's noticed you have an error and is kindly pointing it out for you.


Ability to open a specific folder inside a nested folder structure

Problem:
One of the most common ways of organizing things on a computer/smartphone etc. is to organise things into some kind of nested folder structure. E.g. 2023/summer/photoProjects/birdProject/photos/bird3/portrait.
But Aves currently doesnt allow you to see/utilize the nested nature of this structure, instead showing all folders in a flat way.
Here are some examples where this leads to problems:
Example 1:
Suppose you organize your hobbies (and jobs) into projects, each living in a separate folder. Over time these accumulate, there could be hundreds of them. Inside each project there is a sub-folder called "photos" which documents that project.
How can one browse the photos of a specific project?
With the current app version I see two ways to do this:

Enter "photos" into the search box and then go through the (possibly hundreds) of same-named entries, trying to guess which one is the one you were looking for. Problem: cumbersome and annoying until you found the right folder. It's worse if you want to quickly compare different projects, now you have to jump between the folders quickly, which is basically impossible with this method.
Find the right folder using a file management app, then from within that app open one of the photos via Aves. Problem: When Aves is invoked in this way then one can only see individual photos in Aves, but cannot utilize the mosaic view.

Example 2:
I photograph birds. Lots of them, many different individuals from many different species. Each individual bird lives in its own folder. Naturally, these folders tend to have similar names, e.g. something like date_speciesName_locationName. Many of these bird-folders contain subfolders naming body parts e.g. head, wing, tail, and so forth (it's a reference database for bird-identification purposes). The same problem as in Example 1 occurs: How can I navigate to the "wing" folder of a specific bird and see it in mosaic view? Another problem that arises here is that if i want to compare several birds its difficult to quickly navigate from (the mosaic view of) one to the next, even though each has a unique folder name. Basically one has to either do it the same way as in example 1 or manually type in the folder name.
Example 3:
Sticking with the scenario of example 2: suppose I have found the folder of a specific bird and want to browse all of its photos. Problem: If some of these photos are in subfolders Aves doesn't show them. I can't even see that this problem exists from within Aves, it just looks as if these photos wouldn't exist.
Possible solutions:
Solution 1: Incorporate some kind of file system view (e.g. by showing folders alongside photos and allow navigating to the enclosing folder)
Solution 2: Allow entering mosaic view after an image has been opened in aves from a file manager app.
Solution 1 may (depending on how it's implemented) have the advantage of allowing users to more quickly navigate subfolders within a folder, and, if the subfolders are shown alongside the photos, the advantage of making the user aware that subfolders exist.
Alternatives:
One natural alternative would be if it was possible to open a folder in aves from a file manager. But my file manager doesn't allow that, it only allows opening files in external apps, not folders. I suspect that is how it generally is on android?

Interesting use cases and ideas, thank you.
Generally about nested folders, there is already this issue:

#268

About opening a folder from the file manager, it's true that it is not currently possible, but i could try and allow opening it from the Info page of the opened image.
About album comparison, what you can do is pin them. Let's say you searched for the album, then you can long press on the chip and select Show in Albums, then pin it to the top. Do the same with the other albums of interest. Then from the Albums page you can quickly switch between the pinned albums.
Also, i don't intend to change your workflow, but it seems to me that your intricate folder structure would be better represented by tags instead. You can use tags for the name of the bird, body part, project, commissioner, whatever... Because they are multidimensional (compared to folders that are just a tree), they are preferred by photographers, journalists, archivists, etc. who need to precisely document their images.

True, I should have mentioned #268 .
These two issues are related, but not necessarily equivalent, because albums are not necessarily the same thing as folders. E.g. a folder could contain all kinds of things, only some of which being subfolders containing photos. It is not clear to me what the definition of "album" used by Aves is, and what the vision for their user interface is.
As a sidenote: I have accidentally deleted files in other photo-viewing apps because I thougt the folder was empty. So in case you end up implementing folder view then it's probably a good idea to either display non-image files, or not allow deleting folders from within Aves, or to emitting a warning if one tries to delete a folder containing non-image files.
Thanks for the tip to pin albums for comparison. When I want to compare back and forth between several albums that could work well. But I also have the use case where I linearly traverse a large number of folders without going back to previous ones. This use case comes up when I do bird identification training, when I explain some bird id stuff to someone, or when I encounter a bird in the field and compare it to my photos to identify it.
About tags vs folders for organisation: True! Unfortunately I want to organise not only my photos but many other file types as well. I dont want to have separate systems for photos and the rest. There is software for that, but the solutions I found all had problems like vendor lock-in, being too use-case specific, too complicated or expensive.

It is not clear to me what the definition of "album" used by Aves is

I favoured the word "album" rather than "folder" because it would be future-proof in case I someday connect with cloud services like Google Photos, where the notion of "album" exists but not "folder". In practice, for local storage, an album is backed by a single folder (without its subfolders).

I have accidentally deleted files in other photo-viewing apps because I thougt the folder was empty

In Aves, when you rename/delete an album, it actually moves/deletes media files in it, but it never touches non-media files. Even if I expose folder structure as it is on storage, I would do file manipulation the same way. Data loss is to me the most important thing to avoid.

Good day dev. The only thing I want for Christmas is this feature to arrive as soon as possible.

Rant Alert!
Good day dev. Any time line on this feature? Too many things are being prioritized over nested folder/groups.
Almost everyday I check up on your app for updates and features, although always appreciated I never see this 1 feature to make Aves perfect in my eyes.
I don't serve to annoy or upset you. I'm just tired of seeing one long strip of albums, and blindly🧑🦯 waiting for when this will be added.

Any time line on this feature?

No timeline, no promise. It may never be implemented.

Almost everyday I check up on your app for updates and features, although always appreciated I never see this 1 feature to make Aves perfect in my eyes.

This is useless. You can subscribe to issues (or simply watch repo releases) for updates.

I don't serve to annoy or upset you. I'm just tired of seeing one long strip of albums, and blindlyperson_with_probing_cane waiting for when this will be added.

I don't mind banning people who are rude, feel entitled, or are generally annoying. Don't rant on this repo.

I fully understand. However it's not useless.
On Sat, Jul 15, 2023, 7:07 AM Thibault Deckers @.***>
wrote:

Any time line on this feature?
No timeline, no promise. It may never be implemented.
Almost everyday I check up on your app for updates and features, although
always appreciated I never see this 1 feature to make Aves perfect in my
eyes.
This is useless. You can subscribe to issues (or simply watch repo
releases) for updates.
I don't serve to annoy or upset you. I'm just tired of seeing one long
strip of albums, and blindlyperson_with_probing_cane waiting for when this
will be added.
I don't mind banning people who are rude, feel entitled, or are generally
annoying. Don't rant on this repo.
—
Reply to this email directly, view it on GitHub
https://github.com/deckerst/aves/issues/592#issuecomment-1636748439, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A4EOLRRCFQ6RF5SGC7DUBUTXQKBXXANCNFSM6AAAAAAXCICZZA
.
You are receiving this because you commented.Message ID:
@.***>


404 for Contributing Guide

Hi all,
I noticed the link to the contributing guide on the community page is broken. Since many Jupyter projects (at least the ones I work on) reference this in their own contributing guidelines, I think it would be great if we could figure out where this page has moved to?
Cheers!

The page formerly at
https://jupyter.readthedocs.io/en/latest/contributor/content-contributor.html
is now at
https://jupyter.readthedocs.io/en/latest/contributing/content-contributor.html

Pull request:
https://github.com/jupyter/jupyter.github.io/pull/378

Thanks so much @slel for noticing and making the patch!

Credit goes to @sgibson91 who noticed the faulty link and opened this ticket!


Allow creating deployment scripts so that they can be used in test suites

The way I imagine it - I want to be able to test the system that will be deployed in production.
I argue that the way you deploy the contracts can effect your system, so you want to make sure that your tests are testing the same thing.

Research results:

deploy script → migration script
similar migration approach to Truffle
new command: migrate
common test/migration cheatcodes:

build -> trigger building
declare and deploy_contract should accept compiled contracts as input
deploy_declared — prepare cheatcode won't be available in the migration environment

Examples

migration/01_migrate.cairo

%lang starknet

func migrate(args_count: felt, args: felt*):
  %{
      account_address = ids.args[0]
      result = {}
      build()
      result["foo_class_hash"] = declare("./build/foo.json")
  %}
  return ()
end

migration/01_migrate_test.cairo

from migration.01_migrate.cairo import migrate

@view
func test_migrate():
    let (local args : felt*) = alloc()
    migrate(0, args)
    %{
        result["foo_class_hash"]
        # ...
    %}
end

- `protostar migrate --args $(echo $ACCOUNT_CONTRACT)`

func up(a,b,c):

end

protostar migrate-up --inputs 1 2 3
[protostar.migrate-up]
inputs=[1,2,3]

Protostar commands must be one level deep to generate CLI docs so:
protostar migrate-up
protostar migrate-down

Hmm, with this constraint I think protostar migrate as you originally proposed was fine 😄 I have seen somewhere following design protostar migrate --down so maybe we could reserve this one for us...
I do wonder though if we will reach a need for down migrations now 🤔

I thought about this, but I'd have to preprocess the migration script everytime before argparse is configured.

I think the implementation would have to escape limitations of argparse here and do parsing itself. That's why I thought about using a different prefix for argument names (@x instead of --x). From argparse point of view, the migrate command would accept arbitrary amount of extra string parameters.

Also, migration args couldn't be defined in the protostar.toml, unless Protostar combines CLI's unknown-args and protostar.toml::[protostar.migrate] into one dict and passes it as a hint. All arguments would be a string though.

Do we need to support this case, though? I cannot imagine benefits, as that would gradually litter protostar.toml.

I think the implementation would have to escape limitations of argparse here and do parsing itself. That's why I thought about using a different prefix for argument names (@x instead of --x). From argparse point of view, the migrate command would accept arbitrary amount of extra string parameters.

For the time being, I'll make it consistent with the inputs argument or the deploy command. In the future, we can think about the custom parses for both of these commands.

Do we need to support this case, though? I cannot imagine benefits, as that would gradually litter protostar.toml.

Somebody may try to run migrate in the devnet network over and over. Defining inputs in the protostar.toml would avoid providing them manually every time.
It's also about consistency. We try to make every argument configurable from the protostar.toml. Actually, it shouldn't be a problem:
[protostar.migrate]
inputs = "@x 1 @y 2"

Blocked by https://github.com/software-mansion/protostar/issues/415

Blocked by #439


Feature request: display commit hash on submission_info

Hi,
enjoying your work so far! I want to make a small suggestion for 'script' competitions
currently, my submission_info displays the following:
{
"datetime": "2024-05-28 05:53:55",
"submission_id": "0fb3273c-ede2-4dd6-8dae-7fdb210e269a",
"submission_comment": "",
"submission_repo": "<omitted>/dummy",
"space_id": "<omitted>/comp-0fb3273c-ede2-4dd6-8dae-7fdb210e269a",
"submitted_by": "<omitted>",
"status": 3,
"selected": true,
"public_score": {
"accuracy_score": 0.4444444444444444
},
"private_score": {
"accuracy_score": 0.45454545454545453
}
}

I found it rather hard to trace back the submitted script, since it is prone to change per every submission.
would it be possible to add commit hash to submission_info?

will be fixed.

now, in this case, the submission id is the commit hash


Fix accepter occasionally being incorrect

High Level Overview of Change

Occasionally the address sending the NFTokenAcceptOffer transaction won't be the one that buys/sells the NFT. This is fixed by using the acceptedOfferNode.Destination whenever it's not undefined.
Context of Change

I was looking at https://livenet.xrpl.org/transactions/ and was wondering why the buyer wasn't the same as other explorers were showing.
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Release

TypeScript/Hooks Update

[ ] Updated files to React Hooks
[ ] Updated files to TypeScript

Before / After

Test Plan

Works locally.

@will2022 Looks great.  Can you add a unit test for this scenario?

I've looked a little more at the transaction that caused this bug. It doesn't look like this would be the final fix


Update release version schema to match JitPack's expectations

Use a JitPack-comapatible version naming schema for future releases, according to https://github.com/jitpack/jitpack.io/issues/448#issuecomment-189927888.
Build a new release using this schema.

Fixed.


Using Parse SDK in Windows 10 Universal App

Hi, we are working on Windows 10 Universal app and trying to integrate Parse push notifications functionality. We have tried both v1.6.0 and 1.5.5 of Parse SDK to subscribe to a channel using installation.SaveAsync()  or using shorthand like SubscribeAsync() , but, the device is not getting registered (i.e. not subscribing to the channel on Parse portal Installation table). Kindly , let us know if there is a fix available for it.

Hello,
I added support for W10, you can check it there : https://github.com/Julien-Mialon/Parse-SDK-dotNET

Same here but it keeps throwing System.Aggregate exception

@Julien-Mialon we are unable to Build the Parse.UWP project in VS 2015. Errors show HttpClient,Httprequest, etc. not accessible due to its protection level. Please check from your side and Suggest what we can do from our side. Thanks.

@Vaibhavmalik3122 are you using the Parse nuget ? If yes, you should use the Parse source code directly. There is a small modification on Parse code in order to Parse.UWP use the internal types.

@Julien-Mialon @richardjrossiii  we are able to build our project using the Parse.UWP reference , and sucessfully registered the username in the Installation table. But, when trying to send a Test push notification via Parse push portal, it is NOT able to sent.

@Vaibhavmalik3122 Yep, that sounds par for the course. Our backend does not currently support UWP push, and won't until we 'officially' support the UWP version of the SDK in an official capacity, sadly.

@Vaibhavmalik3122 : that sound weird, I'm currently using it with an UWP app without any issues, push notifications work great. It uses the same system than WinRT app.

@Julien-Mialon could you please elaborate what small modifications on parse code were you talking about for Parse.UWP to use internal types in previous comments ?

@Vaibhavmalik3122 : you can check my commit for uwp support there : https://github.com/Julien-Mialon/Parse-SDK-dotNET/commit/d6ac21ae89e8b22d30b17daf127c3ac123912b01
Basically, modification in Parse code was in ParseClient.cs to add Parse.UWP to assembly list to use when looking for implementation of interfaces and in AssemblyInfo.Portable.cs add this line [assembly: InternalsVisibleTo("Parse.UWP")] to authorize Parse.UWP project to access internal types of Parse project.

@Julien-Mialon Could you provide sample solution to use Parse.UWP solution? I tried to use it by creating new UWP project and referencing the library. On app launch adding this line "await ParsePush.SubscribeAsync("");". But it throwing exception Unable to cast object of type 'Parse.ParseObject' to type 'Parse.ParseInstallation'. please help.

@Julien-Mialon we have the latest pull of the repository and hence all the modifications you talked about are already there(check the screenshot attached).But still we are not able to send push Notifications from parse dashboard. We need to complete this as our official project deadline is coming to an end soon . Would really appreciate if you could assist us on this, maybe a skype call or teamviewer session would help. Looking forward.

I was able to fix cast exception by referencing Parse.dll and initializing ParseClient.Initialize("key", "key") method. But still no luck with push notifications from Parse console :(

I just tested this procedure, tell me if it works for you. Otherwise, we can plan teamviewer or skype session to debug that issue.

create vs project UWP W10

clone parsesdk github from my profile (with modification for UWP)

copy Parse & Parse.UWP directory in your project directory

right click your solution in vs2015 and import project Parse/Parse.csproj and Parse.UWP/Parse.UWP.csproj

Check that reference to Parse in Parse.UWP is correct

Add Parse and Parse.UWP in your app reference

Check if everything build correctly

Go to dev.windows.com

Create a new app by reserving its name.

In my test it will be called "storeTestingStorm"

Look at your app overview and go to section "Services"/"Push notifications"

Click on link "Live Services site"

Copy package SID & client secret to parse configuration page

Go to parse.com and create an app (I called it "testParseUWP")

Go to push section and scroll down to "Windows Push Credentials". Paste your app SID and client secret

Go to keys section and copy "Application Id" & ".NET Key"

Let's go back in VS 2015

right click on your app and associate it with your store app (store -> associate app with the store...)

In your app.xaml.cs constructor, add "ParseClient.Initialize(APPLICATION_ID, .NET_KEY)" where APPLICATION_ID and .NET_KEY are values from parse keys page.

Add "ParseInstallation.CurrentInstallation.SaveAsync();" just after this line

Launch your app and suspend it via "Lifecycle events" in VS toolbar (if not available, right click on VS top toolbar and enable "Debug Location") Do not stop the app

Go back to parse.com and send a push notification

If everything went fine, it shows a push notification on your desktop after a few seconds

Great guide, thank you @Julien-Mialon! I have made only a small modification: I'm calling await ParseInstallation.CurrentInstallation.SaveAsync(); in async onLaunched() (and not in constructor), as it's an asynchronous method.

@Julien-Mialon Thanks for steps!
As from the steps you mentioned, I was not doing "ParseInstallation.CurrentInstallation.SaveAsync();"
but I will try with this change and see if it works. Thanks again!

Although right now I have the (small) problem that the push opens don't get tracked in Parse.com (only push sends - I see e.g. 2 pushes sent and 0 opened).
My code looks like this:
    public App()
    {

(...)
ParseClient.Initialize(app_id, app_key);
}
(...)
    protected override async void OnLaunched(LaunchActivatedEventArgs e)
    {

(...)
await ParseInstallation.CurrentInstallation.SaveAsync();
        // By convention, the empty string is considered a "Broadcast" channel
        // Note that we had to add "async" to the OnLaunched definition to use the await keyword
        await ParsePush.SubscribeAsync("");

        // Track this application being launched
        await ParseAnalytics.TrackAppOpenedAsync(e);
    }

Any suggestions?
Thanks so much!

I tried with above steps but still cannot send push successfully, this is what i see on Parse dashboard

@lukaszkalnik @mehul9595 about the opened notification count, it's an issue from Parse. I just tried with the WinRT sdk (nuget 1.7.0), it doesn't work either. The sdk expect to receive a json with "push_hash" field to track push notification open count. And this field is not sent by Parse backend, just receiving something like {"alert":"push notification message"}

@Julien-Mialon cool, thanks so much for the information! You are really helping a lot, I appreciate it!

@lukaszkalnik I just opened a dedicated issue (#162) for the problem with the missing field in json notification.

@Julien-Mialon Notifications are working good. Thanks for the steps. But, one issue is bugging us , whenever the app is in Foreground state , the toast notification banner does not appear , even after the app is closed , no notification appears in Action center. The ParsePush.PushNotificationReceived is hit everytime , but on the OS level no notification shows up.

@Vaibhavmalik3122 I just tested foreground notifications and it works without issue, can you check with the project available there : https://github.com/Julien-Mialon/Parse.UWP.Sample, you just have to associate the app with the store and write your Parse appid and .net key in Constants.cs.
If it's doesn't work for you, check if in the event handler of PushNotificationReceived, args.NotificationType == "Toast" and args.Cancel is not set to true.

I followed your steps, the app has been associated with the store, and I had it working in December, now it is randomly not working (I had not tested it for some time because I had got it working). Do you have any more suggestions?

@michstmatt If it's randomly working, I can not really think of something from the app that can block this. But I saw some issue from Windows notification service working sometimes.

@Julien-Mialon I was wondering if its possible to send 'adaptive toast notificatoins' which are supported by win10 adaptive templates (ref: http://blogs.msdn.com/b/tiles_and_toasts/archive/2015/07/08/toast-notification-and-action-center-overview-for-windows-10.aspx)
As I can see from Parse dashboard it only allows to send Plain Text or Json data into Push notification. Is there additional things needs to be done in order to these custom push notifications?

@mehul9595 Not supported for now but I think there is another opened issue about that. https://github.com/ParsePlatform/Parse-SDK-dotNET/issues/137

@Julien-Mialon well exactly what I was looking for. thanks again!

@Julien-Mialon I created a new project, I found out that when I copy the Package SID its adds a blank character at the end. Went back into my original projects parse push settings, sure enough blank character at the end, and its working. Thanks for making this library!

@Julien-Mialon is it possible to send push notification with deeplink? if yes, how can it be handled?

@mehul9595 I think you can do that by sending json content in your notification and then parsing this content in your application. You can embed deeplink info in this json.

@lukaszkalnik @mehul9595  I just implemented support to track opened notification count in WinRT & winPhone (https://github.com/ParsePlatform/Parse-SDK-dotNET/pull/164) and did the same modification on my uwp fork you can find here : https://github.com/Julien-Mialon/Parse-SDK-dotNET/tree/features/uwpSupport

Thanks @Julien-Mialon, I have just tested the opened pushes reporting and it works flawlessly. Great job!
The only weird thing is that now I get an error when starting the Release build on a Windows 10 Mobile device:
0x76CCE031 (combase.dll) in ApplicationName.exe: 0xC000027B: An application-internal exception has occurred (Parameter: 0x038E60F8, 0x00000002).
When I start a Debug build though everything is working correctly.
On a Windows 10 PC both Debug and Release builds are working correctly.

Hmm, I just checked and with your previous version of the SDK I have actually the same problem (Release build not working on a smartphone). Probably this is unrelated to your changes then.
Thanks for your awesome help!

@Julien-Mialon Thank you, great job! I have a problem with receiving push notifications. After implementing ParsePush.ParsePushNotificationReceived, Parse.UWP throws System.NullReferenceException when the app receives a notification

@SamAlbuga I think you have to subscribe to 2 event, ParsePushNotificationReceived and PushNotificationReceived.

@Julien-Mialon is it under master yet? which branch do you suggest to use?

For uwp, you can use the branch "features/uwpSupport" on my repo. Not merged with official parse repo for now. https://github.com/Julien-Mialon/Parse-SDK-dotNET

cool, I was using master branch from your repo so far working fine for me without issues. I will change to your feature branch.

@Julien-Mialon Yes that worked perfectly. Thank you :)

Hey @Julien-Mialon , when app is in foreground, how can we create a custom toast notif. for the user such that, when clicked, a custom event can be hit instead of Onlaunched. for reference , WhatsApp for WP is a good example.

@Vaibhavmalik3122 I don't think if that's possible, seems it is an part of enhancement, 137 . Once interactive notifications is supported this can be achieved.

I found a case where push notification is sent during device is power off, notifications are not delivered on tablet device when tablet is powered on.
On Windows phone, push notification is seen when device is powered on.
Any clue?

@Julien-Mialon we are currently testing and turns out I cannot start the app with Parse SDK in Release mode on a Lumia 930 phone. It starts in Debug mode though... I get the following exception:
Exception at 0x777AE031 (combase.dll) in App.exe: 0xC000027B: An application-internal exception has occurred (Parameter: 0x00894FC8, 0x00000002)

Based on the latest version in the master branch using the Parse SDK on UWP platforms does work.
If any specific problem re-occurs with the latest version from the master branch, feel free to create a new issue to discuss possible solutions.


LineChart - Strip and Points not showing beyond x index 0 for DataSet

Description
When trying to show a strip and datapoint when pressing a LineChart that uses the dataSet property, the points and strip disappears when pressing anything other than the beginning of the graph (x = 0).
I've found what I believe is the cause of the issue, which lies here: https://github.com/Abhinandan-Kushwaha/react-native-gifted-charts/blob/20a98cf084b787f6d1beb3a990f51ba4363ec932/src/LineChart/index.tsx#L1362-L1373
Where the active X point is set. The calculation for finding the x value uses the cumulativeSpacing1 list, which is empty when the component uses dataSet instead of data, data2 etc.
I suppose this should be using cumulativeSpacingForSet instead?
Steps to reproduce

Add a LineChart with data passed to dataSet
Add a pointerConfig to enable pointers and strip to appear.
See how the  pointers and strip only appear when pressing right in the beginning (left side) of the chart area.

Snack or a link to a repository
No response
version of react-native-gifted-charts
1.4.47
React Native version
0.75
Platforms
Android, iOS
Workflow
React Native

Vertical Strip is not visible after some index point on the data, used using data prop. Circular pointer is visible but the strip is hidden. Do you have any Idea how this could be fixed?
pointerConfig={{                     pointerStripHeight: 200,                     pointerStripColor: '#454545',                     pointerStripWidth: 1,                     pointerColor: '#FFFFFF',                     radius: 8,                     pointerLabelWidth: 120,                     pointerLabelHeight:0,                     pointerVanishDelay:20000,                     strokeDashArray:[5,5],                     pointerStripUptoDataPoint: true,                     activatePointersOnLongPress: true,                     autoAdjustPointerLabelPosition: true,                     persistPointer:true,
this is the pointerConfig.

@Abhinandan-Kushwaha , poniterConfig is not showing the pointerStrip after certain X data points. I used data prop of the LineGraph using one of the examples shown.
Data object: {
value: 8,  // PM2.5 level in µg/m³
date: '07:00 AM',
label: '07:00 AM',
labelTextStyle: { color: '#FEFEFE', width: 60, fontSize: 8, fontWeight: "bold" },
},
{ value: 5, date: '07:05 AM' },

any updates on this? I'm experiencing the same issue 😅

Hi @sondreluc @lucasBruma @adityaranjan006 @MartinCura 👋
Thanks for pointing out this issue. I have fixed it and now the pointers work fine with dataSet.
It is available from versions 1.4.48 onwards. Please use the latest version of the library.
Thanks @sondreluc for suggesting the fix.


Migrate tests/app/submitter and tests/app/utilities to pytest

What is the context of this PR?
Migrate tests/app/submitter and tests/app/utilities to pytest
How to review
Check the tests make sense
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Migrate tests/app/submitter and tests/app/utilities to pytest (2 SP)


docs: add missing instruction for MLEM<>DVC workflow

close #120

@alex000kim, please review. Now it should work!


Epona's Hog Mane incorrect description

The last line in "Bonus to All Pets" apply to whole item, not pets:
In TQVault:

In-game:

looks like a lot of item bonuses does not appear in correct order; some should be above the "Bonus to All Pets" are displayed under; some should be above "Grant Skill" are also displayed under

That's simply due to the fact that the game and the tool display the bonuses in different order. All items have a bonus to all stats, but most of these bonuses are actually 0. We display the ones that are not 0, but apparently not always in the same order as the game does (with pet bonuses always at the bottom).

@graceoflives can you share the item using a vault file ?

I'm gonna help
eponashogsmane.zip

I'm gonna help
eponashogsmane.zip

@afjback : Thank for your help we can look at it now.


Name dists afer commits

This names the tarballs after the HEAD commit, to help ipfs/distributions#134

How does the final filename output look like?

cd bin/ && tar -zcf dist/orbit_87db6da_darwin-amd64.tar.gz Orbit-darwin-x64/
cd bin/ && tar -zcf dist/orbit_87db6da_linux-amd64.tar.gz Orbit-linux-x64/

@RichardLitt excellent, thank you!

Thank you @RichardLitt <3


All scalars in matrixes are now strings

This PR does two things:

(Fixes a verification bug) All scalars inside Matrix are stored as strings. During parsing they are all fmt.Sprint-ed. MatrixScalars is gone. MatrixSetup is now map[string][]string and MatrixAdjustmentWith is now map[string]string. This should fix verification when non-string scalars are used, because the backend at some point stringifies them all, before they are reflected back in the job payload, meaning that the JSONified Matrixes are different during verification, manifesting as a signature verification failure. Stringifying everything in advance should be no issue, because there is no difference between the results of interpolating 47 and "47" into a matrix token.
(Willing to go back on this) MatrixPermutation is now map[string]string, removing SelectedDimension. The alternative is to make the backend output be an array, but using a map removes one of the error cases in permutation validation (dupes) and also lets us remove UnmarshalJSON (without altering the current backend output - either way it's a PR here or there). But I didn't mind MatrixPermutation so much really.

i do wanna call out the assumption here:

Stringifying everything in advance should be no issue, because there is no difference between the results of interpolating 47 and "47" into a matrix token.

... that we know of. i think this is definitely fine for now though.


test utility emissions channel and update documentation

Follow steps from https://github.com/hyperledger-labs/blockchain-carbon-accounting/blob/main/utility-emissions-channel/README.md, get the utility emissions channel up and running.
Pls also suggest any improvements and updates to the documentation.

Hi, I am new to this project, and would love to try working on this issue as a first-time contributor.

Great.  Pls try it out and let us know.
Si Chen
Open Source Strategies, Inc.
Video: Fighting Climate Change with Blockchain and Open Source
https://youtu.be/NgxNWXa_IjE
On Sat, May 15, 2021 at 3:59 AM Kris Stern @.***> wrote:

Hi, I am new to this project, and would love to try working on this issue
as a first-time contributor.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hyperledger-labs/blockchain-carbon-accounting/issues/98#issuecomment-841639981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANAS4KROSBHSWA6NMZQGT3TNZH2RANCNFSM4ZHN7VMA
.

Sure, let me start work on this issue this week

Hi! I got some questions that need some clarification. Regarding the running-the-fabric-network-and-express-api section on the README.md for the utilities-emissions-channel, I think it is not very clear how to go about getting the necessary credentials for configuring Ethereum. I could go by trial-and-error, but that would take a bit long besides being a bit (mentally) uncomfortable:

Do you guys have any information to share so I would quickly set up an Infura project quickly as well as the needed Ethereum wallet?
I prefer to use ETH2 over Ethereum, would that be okay?
And, what is the best way for me to start a smart contract for this?

Any help would be much appreciated!

Just go to https://infura.io/ and you can sign up for an account
there.  Infura is basically a gateway to different ethereum testnets and
main nets

It could work with any ethereum test network, so you can use Goerli for
testing.  We're using xDai for production data.  It is a Layer 2 network
which is much more energy efficient.

We already have smart contracts under the net-emissios-token network.
The express api that connects to ethereum will call that to create
emissions tokens.  Please check the instructions for the net emissions
token network.

Si Chen
Open Source Strategies, Inc.
Video: Fighting Climate Change with Blockchain and Open Source
https://youtu.be/NgxNWXa_IjE
On Mon, May 24, 2021 at 7:32 AM Kris Stern @.***> wrote:

Hi! I got some questions that need some clarification. Regarding the
running-the-fabric-network-and-express-api
https://github.com/hyperledger-labs/blockchain-carbon-accounting/blob/main/utility-emissions-channel/README.md#running-the-fabric-network-and-express-api
section on the README.md for the utilities-emissions-channel, I think it
is not very clear how to go about getting the necessary credentials for
configuring Ethereum. I could go by trial-and-error, but that would take a
bit long besides being a bit (mentally) uncomfortable:

Do you guys have any information to share so I would quickly set up
an Infura project quickly as well as the needed Ethereum wallet?
I prefer to use ETH2 over Ethereum, would that be okay?
And, what is the best way for me to start a smart contract for this?

Any help would be much appreciated!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hyperledger-labs/blockchain-carbon-accounting/issues/98#issuecomment-847083227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANAS4OJSANY5SKPEDDVGLDTPJPQPANCNFSM4ZHN7VMA
.

Thanks @sichen1234  for the info. I noticed that @Pihu1998 has self-assigned the issue, so is she also working on it now?

HI @slim-patchy , yes I am also working on it now. I am currently fixing bugs in the utility emissions channel, will create commits soon today about those fix. :)
I guess, you may also try to get the channel up and running and fix bugs along the way. :)

Hi @Pihu1998 That's great! Maybe I can checkout your code and update the wordings of the documentation later, like adding the context by providing some backstories and motivations, especially to the README.md's.

Hi @slim-patchy , that's a great idea. I got the channel up and running by following the instructions under:  "Running the Fabric network and Express API" in the README.md.
In perspective of the README update, I believe two things need to be more clear here:

First is this step:
`
Fill in Ethereum configuration settings in typescript_app/src/config/networkConfig.ts:
export const PRIVATE_KEY = "private_key_of_ethereum_dealer_wallet";
export const CONTRACT_ADDRESS = "address_of_ethereum_contract_to_connect_to_on_goerli";
export const INFURA_PROJECT_ID = "infura_id";
export const INFURA_PROJECT_SECRET = "infura_secret";`

I guess we need to mention here:
You will need to create an Infura project (https://infura.io) for getting the INFURA credentials. And for the Ethereum wallet credentials, we need to download the Metamask extension (https://metamask.io/) and get some Ether (https://faucet.goerli.mudit.blog/) in the wallet.
2. Second thing is the below step:
Start network, create channel, and deployCC:
sh ./scripts/reset.sh && sh start.sh
While running this command, the sh ./scripts/reset.sh command would first look for net orderer's and net peer's volumes to remove. For Eg: volume net_orderer1.auditor1.carbonAccounting.com.
And this might have not been initially created by the user, till they run the ./start.sh script. In my case, it was the same and I got below errors:

Removing network net_carbonAccounting
WARNING: Network net_carbonAccounting not found.
Network net_carbonAccounting is external, skipping
Removing volume net_orderer1.auditor1.carbonAccounting.com
WARNING: Volume net_orderer1.auditor1.carbonAccounting.com not found.
Removing volume net_orderer1.auditor2.carbonAccounting.com
WARNING: Volume net_orderer1.auditor2.carbonAccounting.com not found.
Removing volume net_orderer1.auditor3.carbonAccounting.com
WARNING: Volume net_orderer1.auditor3.carbonAccounting.com not found.
Removing volume net_peer1.auditor1.carbonAccounting.com
WARNING: Volume net_peer1.auditor1.carbonAccounting.com not found.
Removing volume net_peer1.auditor2.carbonAccounting.com
WARNING: Volume net_peer1.auditor2.carbonAccounting.com not found.
Removing volume net_peer1.auditor3.carbonAccounting.com
WARNING: Volume net_peer1.auditor3.carbonAccounting.com not found.
Removing volume net_pgdata
WARNING: Volume net_pgdata not found.
Removing volume net_walletstore
WARNING: Volume net_walletstore not found.
---- No containers available for deletion ----
---- No images available for deletion ----
rm: cannot remove 'emissionscontract.tar.gz': No such file or directory
Resetting wallets and fabric-ca-server dbs...
Wallets at ../typescript_app/dist/src/blockchain-gateway/wallets do not exist, skipping.
db for auditor1 does not exist, skipping.
db for auditor2 does not exist, skipping.
db for auditor3 does not exist, skipping.
Finished removing all fabric-ca-server dbs.
Pruning volumes...
WARNING! This will remove all local volumes not used by at least one container.
Are you sure you want to continue? [y/N] Total reclaimed space: 0B
Removing build under typescript_app/dist...
rm: cannot remove '../typescript_app/dist': No such file or directory
rm: cannot remove 'chaincode*_log.txt': No such file or directory

Hi @sichen1234 ,
I got the channel up and running successfully. But I see that it just created two Auditors: Auditor1 and Auditor2in the process:
{ 	"approvals": { 		"auditor1": true, 		"auditor2": true 	} }
, whereas in the video you created, I saw there are 3 Auditors and organizations created.
Is this an issue

Sure @Pihu1998 Let me follow your suggestions and submit a PR soon regarding the docs. I agree some parts need substantially more info.

HI @slim-patchy , sure. You may also create your commits on this PR (which is created for this issue): #169
Please let me know if you need any help. I am also constantly improving the docs and the repo constantly, as I see any flaws.

No problem @Pihu1998 I will do anything I can to help to fill in the gaps so that people will not be put off trying out the code due to missing or insufficient information. I will try to get everything done by this weekend.

@Pihu1998 That's not a problem.  That video was from a while ago and we've changed the startup config.  If you notice anything missing or incorrect with the documentation, though, definitely please create a pull request.

I have submitted PR #181 as a follow-up on this discussion.

I have some doubt regarding whether we should also update the versions to use say:
./bootstrap.sh 2.3.2 1.5.9 -d -s

instead of the original in the Running the Fabric network and Express API section under the instruction "Install binaries for linux distribution". This is because the original seems outdated. But am not sure if it was intentional or not to pin it to an earlier version.

Hi @slim-patchy , thanks for creating a PR. Your changes looks good to me. :)
Regarding the Linux Binaries, we need to check if the new version sync up with other dependencies in the build process. Otherwise we would get several dependency errors while building.

I see, just wanted to double check if the versioning of the fabrics-samples dependencies need updated. Thanks for the info!


yt uninstall needs sucessful unintall message

Problem
when you run yt uninstall moduleName there is no indication to the user that the action has completed sucessfully
Solution
add "moduleName uninstalled successfully" printout message or fail message as appropriate.

This behaviour is the same as, for example, rm. In general unix commands exit silently if there was no error.
yt clean, yt logout, yt outdated and probably other commands will also exit silently if successful.


getting NA class string

Adding a function to get the class name from an na_class_t will let me not worry about carrying around the class name through the code after initialization.
/** Returns the name of the NA class.
 * \param na_class [IN] pointer to NA class
 * \param class_name [OUT] pointer to NA class name
 *
 * \return NA_SUCCESS or corresponding NA error code
 */
na_return_t NA_Get_class_name(na_class_t *na_class, const char **class_name);

Yes thanks that was something missing


rotate_center

rotate_center is default to be [100, 100], that is ok for base config, which has bev resolution of 200x200,
but for small and tiny config, which has 150x150 and 50x50, it seems rotate_center is not the center of the bev image for these configs.
is this on purpose or a mistake?

Yeah, it's a mistake. Fixing it will obtain higher scores😂.

Thanks for your reply.
Another question:
In temporal_self_attention.py,  you concate the ‘query’ and ‘value[:bs]’ in line 195, which i guess you mean to concatenating previous bev_query and current bev_query in feature dimension, howevery 'value' is reshaped from shape [B,2,query_len,F]( in encoder.py line 203), which has batch in first dimension, after reshape, ’value[:bs]‘ will not be the first batch(which is according to pre_query), but half pre_quey and half cur_query.  Maybe reshape from [2, B, query_len, F] to [2*B, query_len, F] is what you intended?

Thanks for pointing out our mistakes, would you like to raise a pull requests and become a contributor of BEVFormer?

very glad to, i will make a PR after train and eval.

@zhiqi-li @minrui-hust
hello,I have a question.
if I change the bev coords to [200,100](means bev_h=200,bev_w=100),and I change the rotate center to be [50,100], will it work?
Thanks.

ge the bev coords to [200,100](means bev_h=200,bev_w=100),and I change the rotate center to b

I think it will work, but we don't need to set the center.
The rotation function by default rotates around the center of the tensor.


[Editor] Sirraide's PRs

Made for #76
Add to syntax highlighting:

inline keyword(Vim+Emacs)
__builtin_syscall function(Vim)

Legendary; this will be merged as soon as #76 is

TODO:

[ ] Add noinline
[ ] Add __noopt__
[ ] Add __builtin_line()
[ ] Add __builtin_filename()
[ ] Add __builtin_debugtrap()
[ ] Add __builtin_inline

There’s also used, const, and pure if you want to add all of the attributes :eyes:

Oh, and I think I forgot about noreturn


Add DNS Event Support

Add DNS events to the Security Events schema and the mapping for Sysmon conversion.
Field mappings
query_name = "QueryName"
query_results = "QueryResults"
query_status = "QueryStatus"

I think the rest should be generic fields already mapped.

For reference to this thread, here's an example event from OSSEM
https://github.com/Cyb3rWard0g/OSSEM/blob/master/data_dictionaries/windows/sysmon/event-22.md
<Event xmlns="http://schemas.microsoft.com/win/2004/08/events/event">
  <System>
    <Provider Name="Microsoft-Windows-Sysmon" Guid="{5770385F-C22A-43E0-BF4C-06F5698FFBD9}" />
    <EventID>22</EventID>
    <Version>5</Version>
    <Level>4</Level>
    <Task>22</Task>
    <Opcode>0</Opcode>
    <Keywords>0x8000000000000000</Keywords>
    <TimeCreated SystemTime="2019-06-12T00:57:56.373798200Z" />
    <EventRecordID>6612437</EventRecordID>
    <Correlation />
    <Execution ProcessID="2312" ThreadID="3252" />
    <Channel>Microsoft-Windows-Sysmon/Operational</Channel>
    <Computer>DESKTOP-WARDOG.RIVENDELL.local</Computer>
    <Security UserID="S-1-5-18" />
  </System>
  <EventData>
    <Data Name="RuleName" />
    <Data Name="UtcTime">2019-06-12 00:57:55.254</Data>
    <Data Name="ProcessGuid">{A98268C1-4DDF-5D00-0000-00102D794100}</Data>
    <Data Name="ProcessId">416</Data>
    <Data Name="QueryName">chrome.google.com</Data>
    <Data Name="QueryStatus">0</Data>
    <Data Name="QueryResults">type: 5 www3.l.google.com;<IP_ADDRESS>;</Data>
    <Data Name="Image">C:\Program Files (x86)\Google\Chrome\Application\chrome.exe</Data>
  </EventData>
</Event>

I think normalizing the QueryResults will be a little tricky, because we'll probably want something a little more structured. Maybe an array of just the IPs for now?

@rw-access What are your thoughts on a more structured format? I'm not sure what all we could use here. How is it currently being done in Endgame?


fix: incognito pages and user data dir

When trying out Playwright I found that neither the userDataDir nor useIncognitoPages options of browser-pool work in the SDK. I attempted a quick fix, but apparently there is also a problem in the design, because useIncognitoPages works on the plugin level for Playwright (hence the passing tests), but on the BrowserController level in Puppeteer. Hence the failing test, because Apify.launchPuppeteer does not work with BrowserController at all.
We need to add more tests to make sure this works in all cases:

[x] useIncognitoPages: false works in Apify.launchPlaywright
[x] useIncognitoPages: true works in Apify.launchPlaywright
[x] useIncognitoPages: false works in Apify.launchPuppeteer
[ ]  useIncognitoPages: true works in Apify.launchPuppeteer
[ ] userDataDir works in PlaywrightCrawler
[ ] userDataDir works in PlaywrightCrawler
[ ] userDataDir works in PuppeteerCrawler
[ ]  userDataDir works in PuppeteerCrawler

@mnmkng that's exactly what I broke in browser-pool couldn't figure out the issue (don't know the code in-depth)

Fixed in #1145


Make flog an importable package

There are cases where you want to generate log lines directly in your application. Therefore it is necessary to make the module github.com/mingrammer/flog importable in other projects.
Moving the code that should be importable out of the main package into its own will allow others to use import flog.
Signed-off-by: Christian Haudum<EMAIL_ADDRESS>

Please resolve the conflict


[text analytics] Updates to Healthcare design

This PR contains design updates for the Healthcare SDK based on direct feedback on the original Python design and also aligning with the updated design in other SDK languages (Java and JS).

/check-enforcer reset

/check-enforcer override


[Deserialization] Do not assert on a missing witness if allowing errors

When allowing errors we may have serialized a requirement without a
witness, do not assert when deserializing.
Resolves rdar://81884767

@swift-ci please test

This is just an assertion - it's already handled on the next line as the requirement as the requirement could have been optional/unavailable.
Another found from the stress tester.

Build failed
Swift Test Linux Platform
Git Sha - 8634f3726d39f0e8b625c56baf8e4bc258296fae

@swift-ci please test


Gh 1282 federated demo

Codecov Report

Merging #1283 into develop will increase coverage by 15.13%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             develop    #1283       +/-   ##
==============================================
+ Coverage      64.51%   79.65%   +15.13%
==============================================
  Files            688       65      -623
  Lines          22225     2271    -19954
  Branches        2734      298     -2436
==============================================
- Hits           14339     1809    -12530
+ Misses          6738      353     -6385
+ Partials        1148      109     -1039

Impacted Files
Coverage Δ
Complexity Δ

.../commonutil/iterable/WrappedCloseableIterable.java

...ore/optimiser/AbstractOperationChainOptimiser.java

...ulostore/key/core/AbstractCoreKeyRangeFactory.java

.../binaryoperator/BoundedTimestampSetAggregator.java

...q/gaffer/accumulostore/key/AccumuloKeyPackage.java

...re/key/exception/BloomFilterIteratorException.java

...tore/serialisation/impl/ByteParquetSerialiser.java

...peration/handler/AddElementsFromSocketHandler.java

...spark/examples/GetJavaRDDOfAllElementsExample.java

...fer/spark/operation/scalardd/GetRDDOfElements.java

... and 613 more

Continue to review full report at Codecov.

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


Added back round corners to copy command button

rounded-md wasn't working because shadcn has a custom config of it in the tailwind.config.mjs file. So I had to set it like this rounded-[6px] for it to show.

Web
Before:

After:


Support for asynchronous communication

Following our discussion on #28, I'd like to make a feature request / plea :) one more time. I've been working on a p2p communication client for a while and as you can imagine, in such environment is hard to predict the order of delivery, basically to predict delivery at all. According to this blog post: https://whispersystems.org/blog/advanced-ratcheting it would seem that I'm doomed. The author had implemented a "double ratchet" algorithm presented here: https://github.com/trevp/double_ratchet/wiki but as far as I know, no attempt has been made to "backport" it into libotr.
However, when I tried some basic use cases like "send (encrypt) messages 1, 2, 3" and "receive (decrypt) 3, 2, 1" or "receive (decrypt) 3, 1" within a session, there wasn't any problem. Right now I don't care about forward secrecy, I'm happy to have one session per user pair forever (i.e. I'd use OTR mainly as an authentication tool, or, as mentioned in #28, as a symmetric key generator based on the shared secret).
So my questions would be:

In what scenarios OTR does fail when it comes to the order of delivery? Am I safe during the session at least?
If the session is safe, what do I have to store to be able to renew the session later? Would be making Session Serializable enough to renew the session?
And most importantly, what are my chances to see the double ratchet algorithm (or something similar) implemented in otr4j? :-)

I intend to answer your questions for as far as I can, but it will take a few days as I am currently very busy.

That's all right, I'm looking forward for your answer!

Hey @mikrosk !
Just some news here: the new version of OTR will support asynchronous communication. Check it here: https://github.com/otrv4/otrv4
With OTRv4:

In what scenarios OTR does fail when it comes to the order of delivery? Am I safe during the session at least?

None.

If the session is safe, what do I have to store to be able to renew the session later? Would be making Session Serializable enough to renew the session?

With OTRv4, you only need to store message keys for a certain amount of time.

And most importantly, what are my chances to see the double ratchet algorithm (or something similar) implemented in otr4j? :-)

In OTRv4. Check the C implementation, which already has some parts of the double ratchet algorithm:
https://github.com/otrv4/libotr-ng


linclust generates duplicate representative sequence IDs

I'm using mmseqs 9.d36de, and running linclust as in the following:
# cluster
mmseqs linclust --threads 24 seq_db cluster_db tmp_dir

# creating _h file
mmseqs createsubdb cluster_db seq_db_h cluster_db_h

# get representative sequences for each cluster
mmseqs result2flat seq_db cluster_db output_rep_seqs.faa --use-fasta-header

While the entire rep seq ID is unique (eg., >1000000 2+209	3), the base ID (eg., >1000000) is not, so any downstream tool that strips off the characters past the first space ( 2+209	3 in this example) will result in ambiguous mappings to the sequences.

You could use convert2fasta instead of result2flat. This should printout the whole header information.
 mmseqs convert2fasta cluster_db output_rep_seqs.faa

I tried using convert2fasta instead of result2flat, but I still get many duplicate IDs, even without removing everything after the first space. For example:
>93922 149-149	3
PTTGELYKVAGKENNDSIKGYRSAYNKATETAKAGYYSVELTDYHIKVESSATPHCGILHFTYPSSDQSRIQIDLARRVGGTSTSQYIKVVDDYTIQGWMKFTPDGGGCGNGEGKADYTVYYYAQFSKPLANYGFWSADIPEHWVRKRDEVV
--
>93922 149-149	3
LARFGAKVTQTDTSVTVEKGELNAIDIDASQIPDLVPVLTVCAAFAKGTTRIYN
--
>93922 149-149	3
ITDTVTKDMESCFSTDCQETYNQTPVSFTFITPETEELFKPDNYPERFSSWFEKFVEHV

Could you please provide a sample set to reproduce the error?

Was seq_db created by extractorfs? That module creates a special header file to identify where the ORF came from. I could build something to would allow to resolve the original header again, however that would still not result in unique headers.
The easiest solution might be to write some small 'awk/sed/perl' script that replaces the spaces in the FASTA header with an underscore or something similar.

Please reopen if the error still occurs.

I experience the same problem. After running linclust and extracting the representative sequences, I have duplicate IDs
>k99_9972_2 # 266 # 886 # 1 # ID=81340_2;partial=00;start_type=GTG;rbs_motif=AGGAG;rbs_spacer=5-10bp;gc_cont=0.655
>k99_9972_2 # 500 # 1270 # 1 # ID=10959_2;partial=00;start_type=TTG;rbs_motif=GGxGG;rbs_spacer=5-10bp;gc_cont=0.471

So reads mappings and downstream tools break because of this.

We are aware of the problem. The issue is that prodigal does not respect the fasta definition here. Everything after a white space is a comment, not an identifier.
I recommend combining the first seven columns into an identifier before clustering.


Download rootfses specific to appid livebuild rootfs

We now have per appid livebuild produced rootfs. Adjust the paths to download them generically.

well, as a general rule, a PR shouldn't be approved if the tests are failing. Otherwise add a comment explaining why they are failing and if it is related or not to the changes.

For the end to end tests to pass, after investigation, we need to download the rootfs with the new naming convention. This means we need to validate from the last livecd-rootfs build in the ppa, then upload it to noble, and wait for one image from the CPC team to be built (the action is taking Ubuntu-Preview, fortunately)

Ok, so apart from the WSL official images, (we still need to have "Ubuntu" published there, the rest is passing. Current is broken too until we can finish the transition.
The naming here follow the new convention and we need to iterate too in the store for the on demand build. I’m merging this one to unblock the future work.


Support for sops decryption methods other than GPG

Is there a roadmap for adding the other decryption engines supported by sops to kustomize-controller?

Switching this to discussions as it's not an 'issue'.

Switching this to discussions as it's not an 'issue'.


update workshop 12 - sector coupling with multilink default

Since the PR: https://github.com/PyPSA/PyPSA/pull/669, my feeling is that this code block is not necessary. I am happy to get assigned to this task and deal with it (as well as the Linopy workshop)
Some sector-coupling technologies have multiple ouputs (e.g. CHP plants producing heat and power). By default, PyPSA links have only one input (bus0) and one output (bus1) with a given efficieny (efficiency). Thus, we have to tell PyPSA that links can have multiple outputs by overriding the component attributes.
override_component_attrs = pypsa.descriptors.Dict(
    {k: v.copy() for k, v in pypsa.components.component_attrs.items()}
)

override_component_attrs["Link"].loc["bus2"] = [
    "string",
    np.nan,
    np.nan,
    "2nd bus",
    "Input (optional)",
]
override_component_attrs["Link"].loc["efficiency2"] = [
    "static or series",
    "per unit",
    1.0,
    "2nd bus efficiency",
    "Input (optional)",
]
override_component_attrs["Link"].loc["p2"] = [
    "series",
    "MW",
    0.0,
    "2nd bus output",
    "Output",
]

Yes, requires to set minimum PyPSA version in environment yaml.

[like]  Maximilian Parzen reacted to your message:

From: Fabian Neumann @.>
Sent: Wednesday, September 27, 2023 7:55:04 PM
To: fneum/data-science-for-esm @.>
Cc: Maximilian Parzen @.>; Assign @.>
Subject: Re: [fneum/data-science-for-esm] update workshop 12 - sector coupling with multilink default (Issue #7)
This email was sent to you by someone outside the University.
You should only click on links or attachments if you are certain that the email is genuine and the content is safe.
Assigned #7https://github.com/fneum/data-science-for-esm/issues/7 to @pz-maxhttps://github.com/pz-max.
—
Reply to this email directly, view it on GitHubhttps://github.com/fneum/data-science-for-esm/issues/7#event-10491612074, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOYZENNMSSH7NR36RU7UNQ3X4SABRANCNFSM6AAAAAA5J3O3BA.
You are receiving this because you were assigned.Message ID: @.***>
The University of Edinburgh is a charitable body, registered in Scotland, with registration number SC005336. Is e buidheann carthannais a th’ ann an Oilthigh Dhùn Èideann, clàraichte an Alba, àireamh clàraidh SC005336.

Nice! :+1:


Add "FAQ" text to the description for FAQ page

Fix #1137
Not certain if this will help (the main FAQ page is not ranked at all when you search for "FAQ")

(Can we test changes to the search, or does it only index from the live site?)

If this change will have any effect, it doesn't appear to be possible to tell from the preview site.


Release with aws sdk2?

Not an issue, just a question: this PR https://github.com/scanamo/scanamo/pull/719 has been merged 4 months ago (May), but the latest release was still in Jan? Any plan for the new release?

 I'll publish a release this weekend.


Add fixed basicaa to NOELLE

LLVM's basicaa is not safe for parallel applications because it breaks alias edges that should exist. However basicaa is the only (as far as I can tell) AA that reads function attributes.
To test for correctness:
for (i=0..N) {
  arr[i-1] = ...
  ... = arr[i]
}

The two instructions above should alias, however they currently do not.

Working on getting this to integrate will NOELLE AAResults
Moving to an analyze and patch method to fix loop bug in LLVM basicaa.

Finalizing unit test

Delaying this feature until we need it. For the near future we do not need to generate the PDG.
Feature is implemented but specification is not entirely clear so we don't mess with NOELLE's AA system.


Allow duplicate role players

What is the goal of this PR?
Allowing duplicate roles to be played in a relation as of https://github.com/graknlabs/grakn/pull/5667.
This means now that relation.getRolePlayerMap() should return a map from Role to List<Thing> rather than Set<Thing>
What are the changes implemented in this PR?

Write new test for retrieving duplicate role players
Update types for rolePlayerMap in the concept API

TODO revert dependency to graknlabs dependencies because of dependency cycle right now: grakn core -- console -- client-java -- grakn core


Rename "master" branch to "main"

It is generally common practice now to avoid the "master" label for its link to slavery.
https://www.theserverside.com/feature/Why-GitHub-renamed-its-master-branch-to-main
We should switch when we can. I think Github makes it pretty painless these days.

It appears this can be closed now.

You're right - thanks!


Add carveouts for downloading legislative districts for places that don't have them

DC, NE

No longer necessary


Use hostname as key in annotation.json

This change is

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
506

Relevant Lines:
506

💛  - Coveralls


Incorrect return type for $fetch in TypeScript

Shouldn't the return type of $fetch be Promise<void> instead of just void (or at least void | Promise<void>), based on this?
I'll make a PR to fix it if it's indeed an error.


limine: 8.4.0 -> 8.6.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/limine-bootloader/limine/releases.
meta.description for limine is: Limine Bootloader
meta.homepage for limine is: https://limine-bootloader.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.6.0 with grep in /nix/store/nn8pgcdb80cd6lp7f4319qfv3606w5dk-limine-8.6.0
found 8.6.0 in filename of file in /nix/store/nn8pgcdb80cd6lp7f4319qfv3606w5dk-limine-8.6.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

limine

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/cb52981ac0751f014530b55c1a5b95dbff8a7e15#limine

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nn8pgcdb80cd6lp7f4319qfv3606w5dk-limine-8.6.0
ls -la /nix/store/nn8pgcdb80cd6lp7f4319qfv3606w5dk-limine-8.6.0/bin

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

x86_64-linux

:white_check_mark: 4 packages built:

limine
limine.dev
limine.doc
limine.man

Maintainer pings
cc @48cf @phip1611 @lunabunja @surfaceflinger for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


npm install of BotBuilder uses depricated uuid module

Running through the tutorials found here:
https://docs.botframework.com/en-us/node/builder/overview/
The first real step is "Install the Bot Builder":
npm install --save botbuilder
This immediately gives a warning:
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead
Everything else went smoothly during the install process.

Yes, "node-uuid" has been recently deprecated in favor of "uuid". It's not a big issue (just a deprecation notice) but it should be easy to fix it.

Thanks... I'll take a look.

I've checked in a fix that will go out with next release.

Fantastic! Thanks Steve!

Just pushed update.


Project Feedback!

We have graded your work, and it looks like the following features are not reflected on your GIF walkthrough:

User can sign in to Twitter using OAuth login.
User is displayed the relative timestamp for each tweet "8m", "7h".

The GIF helps us to make sure we don't miss any required or optional stories you have completed. We highly suggest that you provide us with a GIF that completely shows all the features you have implemented.
Your assignment is incomplete if all required features are not shown in your GIF. Once resolved, please push your updates and submit your assignment again through the Course Portal. Still confused about how to properly submit your assignment? Check out the Submitting Coursework for detailed instructions.
Whenever you make updates to your project that require re-grading, you need to re-submit your project using the submit button on the associated assignment page in the course portal. This will flag your project as “updated” on our end and we know to re-grade.

Thank you for the feedback! I have completed the missing required features and submitted again.


Insert WP Snippets

List in constant expansion.

Duplicate of https://github.com/sindresorhus/awesome/pull/436.


[EMB-87] Add node's implicit contributors endpoint

Purpose
Allow the front-end a way to retrieve a node's implicit contributors via apiv2
Changes
Added v2/nodes/<node_id>/implicit_contributors endpoint, with tests and the relationship on the node serializer.
QA Notes
Test filtering on the new endpoint, and whether its contents match what is currently shown in the contributors page under Admins on Parent Projects
Ticket
https://openscience.atlassian.net/browse/EMB-87

Can you put a sample of the json output into the Changes section of the PR notes, please? Just copy and paste into a code block is fine.

accidentally commited my docker-sync config. Forcing a commit to reverse that.


paho-mqtt-cpp: 1.2.0-4 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository paho-mqtt-cpp to 1.2.0-4:

upstream repository: https://github.com/eclipse/paho.mqtt.cpp.git
release repository: https://github.com/nobleo/paho.mqtt.cpp-release.git
distro file: noetic/distribution.yaml
bloom version: 0.10.7
previous version for package: 1.2.0-3

As per suggestion here: https://answers.ros.org/question/389836/buildfarm-dpkg-shlibdeps-error-cannot-find-library/?answer=390167#post-id-390167


hey i saw u removed the email verifier is it possible to get that back or can u send it to me?

hey i saw u removed the email verifier is it possible to get that back or can u send it to me?


What do the service level codes mean?

Just what do the following codes mean and what is the difference between them?  A link would be helpful in this page if not an explanation.  S0, S1, S2, S3, S4, S6, S7, S9 or S12

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

ID: 228c33b8-2005-40e1-584a-77c32b0433f8
Version Independent ID: 96b3d88c-7343-48a5-37bb-854a94471be1
Content: sys.database_service_objectives (Azure SQL Database) - SQL Server
Content Source: docs/relational-databases/system-catalog-views/sys-database-service-objectives-azure-sql-database.md
Service: sql-database
GitHub Login: @stevestein
Microsoft Alias: sstein

@JoeMoyle Hello Joe, Thank you for reaching out through our Feedback mechanism. We shall look into this Issue #2674 .
@stevestein Hi Steve, You might know of a good link to add to the article cited in this Issue. This issue is now assigned to you.

@JoeMoyle - thanks for the feedback. I added some links to the article that should go live today.
The info you're looking for is here: https://docs.microsoft.com/en-us/azure/sql-database/sql-database-dtu-resource-limits-single-databases
Thanks.


Make explanations collabsible

Use details and summary tags to make them collapsible. This would help to hide them until readers are done with guessing the reason and need to see the explanations after that.

https://gist.github.com/satwikkansal/83a4929f7c164506cdc15897eacde4e3

Not a worthy enough feature to substitute a lot of markdown in the project with html tags. Closing this.


use same odds as box user clicked on

use getSizedPrice odds when user clicks outcome square. Keep using getSizedPrice odds if the inputted wager is less than the wager from getSizedPrice. Only adjust odds and show odds adjusted message, if user's inputted wager is greater than getSizedPrice wager.
example:
user clicks on odds 13% the betslip should say 13% even though spot price would show 10%.

show message if user is getting worst odds than displayed.
message: "Highlighted odds changed because of wager size"

@Chwy5  is this ticket still needed if so, is the description correct?

closing this ticket, behavior not needed.


hugo-theme-bootstrap not found when following Install via Hugo Module

Describe the bug
I followed Install via Hugo Module step by step, including the optional step 4. In step 5, hugo mod npm pack fails with:

Error: module "hugo-theme-bootstrap" not found; either add it as a Hugo Module or store it in "[path]/my-new-blog/themes".: module does not exist

To Reproduce
Steps to reproduce the behavior:

Follow steps 1 through 5 of Install via Hugo Module.

Hugo version
hugo v0.94.2+extended darwin/amd64 BuildDate=unknown
Desktop

OS: macOS

Additional context
Hugo, npm, and Go all updated as of today via homebrew.

Please replace the theme as github.com/razonyang/hugo-theme-bootstrap, and restart the hugo server.

That fixed it. Looks like another substep is needed in the docs at the end of step 4. I can submit a PR to fix the docs bug if you point me to the repo for the docs.

@breyed PR are welcome. The docs are located at https://github.com/razonyang/hugo-theme-bootstrap/blob/master/exampleSite/content/docs/installation/hugo-module/index.md.


Update Teaching Section

Update teaching section to move sections from "current" to "previous".
This also involves updating the feedback page to have the appropriate sections.
Delete help center hours, those do not need to be included under previous.
In current, add "GTA for Physics 2211 SOUP Groups 3 and 4"

Done


feat(Overlay): Use MDN recommendation level 2 "overflow" definition

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.3.8-alpha.b77a0f.0@codecademy/gamut-labs@7.6.4-alpha.b77a0f.0@codecademy/gamut@19.8.1-alpha.b77a0f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fe223bd27f67f74c60a4d0e--gamut-preview.netlify.appDeploy Logs


Non-deterministic 'DataManager' object has no attribute 'storage_access'

Sometimes when creating Files with Globus URLs, the creation fails with the below exception. This is non-deterministic, and I expect it is based on non-deterministic order at which _get_globus_endpoint() iterates over executors.
I think this was introduced by PR #805, which changes the list of executors used by the data manager, to now include itself as an executor.
  File "parsl/tests/test_staging/test_implicit_staging_globus.py", line 29, in test_implicit_staging_globus
    unsorted_file = File('globus://03d7d06a-cb6b-11e8-8c6a-0a1d4c5c824a/unsorted.txt')
  File "/home/benc/parsl/src/parsl/parsl/data_provider/files.py", line 53, in __init__
    self.dman.add_file(self)
  File "/home/benc/parsl/src/parsl/parsl/data_provider/data_manager.py", line 112, in add_file
    self._set_local_path(file)
  File "/home/benc/parsl/src/parsl/parsl/data_provider/data_manager.py", line 115, in _set_local_path
    globus_ep = self._get_globus_endpoint()
  File "/home/benc/parsl/src/parsl/parsl/data_provider/data_manager.py", line 121, in _get_globus_endpoint
    for scheme in executor.storage_access:
AttributeError: 'DataManager' object has no attribute 'storage_access'

fixed by #815


Repo does not mirror automatically, only works when using "Synchronise now"

Gitea Version
1.16.1 (latest-rootless docker image)
Git Version
2.30.2
Operating System
Whatever is inside that container
How are you running Gitea?
Docker latest-rootless from 6th of February.
Database
SQLite
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
https://gist.github.com/AlexuDragon/510b139eb1f670609c591947916050a0
Description
I created a rule for mirroring with default 8h. Then I run synchronise just once so I won't wait 8 hours. Then I decided to delete it (since I couldn't edit it )and add the entry again with 1h timer. I noticed that the repo never syncs on its own, unless I manually click on "Synchronise now". Then I saw the logs, the one with "GetPushMirrorByID" repeats pretty much every 10 minutes. The other one containing "RemovePushMirrorRemote" probably was triggered when I removed the mirror entry, I'd guess that this step triggered the bug. I tried again to remove and re-add the mirror and same pattern happens. I'm guessing that manual Synchronising operates a bit different than the cron script that reads the info from wherever it is stored (SQLite?) before running the push operation. I didn't try this with the try.gitea as it uses a different dev version. I never submitted a bug, I can provide any addition information needed.
Thank you!!
Screenshots
No response

It appears doMirrorSync in services/mirror/mirror.go calls SyncPushMirror passing mirror RepoID. In the end, this ends up in models/repo/pushmirror.go which looks up mirror information from the push_mirror table, but, instead looking at the idcolumn and not repo_id. To be continued.

Further investigation shows that doMirrorSync uses SyncRequest which contains RepoId, this is passed in doMirrorSync (mirror.go) to SyncPushMirror (mirror_push.go) which expects mirrorId and not RepoId.
So the question is, should SyncPushMirror be changed to expect repoId, or SyncRequest to contain mirrorId?

Same here.  Push mirrors never sync unless manually. Gitea 1.16.1 on Windows with SQLite

Same here. Push mirrors never sync unless manually. Gitea 1.16.1 on Windows with SQLite

Seems to have been fixed in 1.16.2

Same for me.


Clarify expected behavior in README

After reading the README, it's not clear to me where the stats will appear...in the browser console, I guessed. To test this, I did:

Downloaded rtcstats.js
Uploaded it to the same folder where my verto js lib files are
Made this the first script execution in my test page:
Made a test call and monitored the browser console

The call went fine (in Chrome72/incognito/Win10), but I didn't see anything new once the peer must have gotten to ice connection state "completed", nor after hangup.
I would expect that I could pass a callback, probably as a listener to iceconnectionstatechange, and that I could inspect the collected, periodic stats that way. If you would explain a bit what to expect, I'll submit a PR for the README to help clarify. Thanks.

no, they will be sent to an instance of https://github.com/opentok/rtcstats-server via websocket.
https://github.com/opentok/rtcstats#require is the more common way to use this nowadays, the other one should probably be removed from the README


fix flaky eventlog integration tests

What does this PR do?

Fix race condition in event log check tests. MockSender is now prepared prior to generating events for the test.
Fix flaky event log service stop by using hard kill instead of SCM commands which sometimes fail to due EventLogs dependent services.
Motivation

https://datadoghq.atlassian.net/browse/WINA-588
https://datadoghq.atlassian.net/browse/WINA-480
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge

/merge


Add mention insertion methods

This change is dependent on https://github.com/matrix-org/matrix-rich-text-editor/pull/699. Will retarget main when that happens.
This PR:

changes name of internal function to set_mention_from_suggestion and updates bindings
expands some of the ffi tests for this function
creates a new mentions.rs file to handle the renamed function
creates a new insert_node_at_range.rs file to allow insertion of a node at a cursor with unit tests
adds extensive tests for mentions in test_mentions.rs
tweaks behaviour of suggestion monitoring to disallow it inside links
implements the corresponding selection changes required in web due to mention nodes now having length 1

Codecov Report
Patch coverage: 99.10% and project coverage change: +2.01 :tada:

Comparison is base (70a2fbf) 88.22% compared to head (441b39d) 90.23%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #698      +/-   ##
============================================
+ Coverage     88.22%   90.23%   +2.01%
============================================
  Files           144       81      -63
  Lines         16633    14213    -2420
  Branches        762        0     -762
============================================
- Hits          14674    12825    -1849
+ Misses         1773     1388     -385
+ Partials        186        0     -186

Flag
Coverage Δ

uitests
?

uitests-android
?

uitests-ios
?

unittests
90.23% <99.10%> (+2.49%)
:arrow_up:

unittests-android
?

unittests-ios
?

unittests-rust
90.23% <99.10%> (+0.44%)
:arrow_up:

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

Impacted Files
Coverage Δ

bindings/wysiwyg-wasm/src/lib.rs
20.54% <0.00%> (ø)

crates/wysiwyg/src/composer_model/hyperlinks.rs
98.74% <ø> (-0.09%)
:arrow_down:

bindings/wysiwyg-ffi/src/ffi_composer_update.rs
98.29% <98.97%> (+0.87%)
:arrow_up:

bindings/wysiwyg-ffi/src/ffi_composer_model.rs
33.45% <100.00%> (+8.72%)
:arrow_up:

...rates/wysiwyg/src/composer_model/example_format.rs
97.47% <100.00%> (ø)

crates/wysiwyg/src/composer_model/mentions.rs
100.00% <100.00%> (ø)

crates/wysiwyg/src/composer_model/menu_action.rs
100.00% <100.00%> (ø)

crates/wysiwyg/src/dom/dom_struct.rs
91.23% <100.00%> (+1.46%)
:arrow_up:

crates/wysiwyg/src/dom/insert_node_at_cursor.rs
100.00% <100.00%> (ø)

crates/wysiwyg/src/dom/nodes/dom_node.rs
88.76% <100.00%> (+0.20%)
:arrow_up:

... and 65 files with indirect coverage changes

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

I've made those changes @aringenbach . Extracting the logic check to a util is probably a good move too as we're going to change that behaviour when issue #702 gets addressed anyway.


No animation in IE11

On chrome example works ultra smooth and fast, however in IE11 there is NO animations at all. How can i eliminate this ?

This is a bug, I'm fixing it as we speak. Also, duplicate of #7

@clauderic thanks! can i help you with it ?

Sure! Feel free to chime in the PR for it here: https://github.com/clauderic/react-sortable-hoc/pull/10


merge: dev to master

:tada: This PR is included in version 0.82.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


miss the image.pt file

When I ran the code I encountered the problem in the image, the image.pt file was missing

Thank you for using this repository.
I have a question, which script did you run?

thank you for reply.
the script is : roslaunch intersection_detector create_dataset.launch

Thank you for teaching me the script.
I'm sorry, but my response will be delayed as I am currently preparing for a new phase of my life.


How to use batch env

Hello.
Thank you for providing us with a great algorithm.
I have a question about batch env.
I have specified two envs in batchenv because of the slow progress of the game agent I am currently training.
However, when I checked the progress in sensorboard, there was no change and it was not doubled in speed.
Is it correct to use batchenv? Also, what does the parallel parameter represent?
Thank you in advance.

Hi, if you use train.py (as opposed to the simplified example.py) then you already get parallel training for free. Doubling the number of environments might not speed up things that much though, because it only speeds up data collection (up to your available CPU resources and constant overhead of copying observations/actions to/from the GPU for the policy forward pass). Often, the bottleneck is training not data collection. You can speed that up by training with multiple GPUs.