Antrea e2e tests consistently failing in Jenkins for TestFlowAggregator

Describe the bug
I see e2e tests failing in Jenkins pretty much all the time with the following error:
    fixtures.go:400: Deleting 'antrea-test' K8s Namespace
I0408 01:50:33.091871 1234845 framework.go:630] Deleting Namespace antrea-test took 49.033647974s
    fixtures.go:386: Error when gracefully exiting Flow Aggregator: error when gracefully exiting Flow Aggregator - copy flow-aggregator coverage files out: cannot retrieve content of file 'flow-aggregator.cov.out' from Pod 'flow-aggregator-7cdb6f9849-hzmh7', stderr: <cat: flow-aggregator.cov.out: No such file or directory
        >, err: <command terminated with exit code 1>
--- FAIL: TestFlowAggregator (248.43s)

It seems like it should be easy to fix.

I saw it happen for other tests as well (coverage file for the agent this time, not the flow-aggregator):
2022-04-08 23:07:30.692449 I | Error when gracefully exit antrea agent: error when graceful exit Antrea agent - copy antrea-agent coverage files out: cannot retrieve content of file 'antrea-agent.cov.out' from Pod 'antrea-agent-p4tgj', stderr: <cat: antrea-agent.cov.out: No such file or directory
>, err: <command terminated with exit code 1>
FAIL	antrea.io/antrea/test/e2e	4155.525s
FAIL
=== TEST FAILURE !!! ===


Release 0.9.24

This is the 0.9.24 branch merged back into master.
I believe this will be the contents of #213 and #215 along with the one addition commit to revert the restriction against moving no-copy items out of no-mod object Contents.

If this PR is applied, we can ignore #213 and #215, this one includes those.


Unsure of solution for fixing "contents: write" security recommendation

Is your feature request related to a problem? Please describe.
Hello! We are seeing the scorecard recommendation in the image under "Additional context". You can find the line we are getting the recommendation for here. The scorecard recommendation ID is TokenPermissionsID.
Describe the solution you'd like
The remediation for this scorecard recommendation currently tells us to set permissions to read-all, although we already have permissions set to read-all in this file. I believe we have implemented the solution that is being recommended, so it seems like the remediation might be misleading, unless I'm mistaken. Is there an additional change that should be made in this configuration?
Additional context

I believe this is because contents: write grants write access to multiple APIs. https://docs.github.com/en/github-ae@latest/rest/overview/permissions-required-for-github-apps#permission-on-contents
What is the recommended best practice in this case? find a more limited permission?

do you need these permissions to write to the GitHub pages using peaceiris/actions-gh-pages?
Which API is it using? I think it may be worth asking GitHub for a more granular permission for this, because pushing docs is less scary than pushing to source code. Well, docs can trick users into running arbitrary commands.. but it's not very stealthy if this is the public doc.
In this case you probably have no way around it and I would dismiss the alert.

To answer your first question: yes, contents: write is currently being used for peaceiris/actions-gh-pages

peaceiris/actions-gh-pages seems to be a widely used Action to do that. Since GitHub does not have a special permission for documentation push, I'm inclined to make the permission check aware of this action and not warn if it's used in the workflow that requires the write permission. We do this for other standard action like Goreleaser, for example.


App: Display chats from Firestore and data sync (App <-> Firestore)

Depend upon #185, After that we can merge it.


Correct judgment condition

$definitions is defined in the parent class and is private.

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
Correct the conditions for determining whether a definition exists.

Thank you!
You are right - I've added tests for Factory::get().
Please see 58aa6c6.
But I don't agree with use of getDefinition instead of isset($this->definitions[$id]).
Why do you think it is right?

I think the PHP interpreter has problems determining the accessibility of member variables of a class.
If it is private in the parent class and not defined in this class, and the member variable to be accessed is treated as an array, then the exception should also be thrown instead of returning undefined. But at this point PHP returns undefined.

According to the definition of getDefinition() method of AbstractContainer,
!isset($this->definitions[$id]) and $this->getDefinition($id) == null are logically equivalent.


React Native Support

I am unable to Integrate this in React Native. Is it possible to use in React Native Application?

Hello @muqeet98 ,
We have customers that successfully integrated Talsec into React Native apps. Could you please provide more detailed information about what you do and what problems you encounter? Feel free to make a video recording.
Have a great day!

Hello @muqeet98 ,
We have customers that successfully integrated Talsec into React Native apps. Could you please provide more detailed information about what you do and what problems you encounter? Feel free to make a video recording.
Have a great day!

Hello @muqeet98 ,
We have customers that successfully integrated Talsec into React Native apps. Could you please provide more detailed information about what you do and what problems you encounter? Feel free to make a video recording.
Have a great day!
@syakymchuk Can you please share any docs for React Native Apps? Android and IOS both.

@syakymchuk  Can you please share any docs for React Native Apps? Android and IOS both.

Not Solved

@syakymchuk Getting this error When I put the maven url in build.gradle.
> Could not get resource 'https://nexus3-public.monetplus.cz/repository/ahead-talsec-free-rasp/com/facebook/fresco/imagepipeline-native/2.2.0/imagepipeline-native-2.2.0.pom'.             > Could not HEAD 'https://nexus3-public.monetplus.cz/repository/ahead-talsec-free-rasp/com/facebook/fresco/imagepipeline-native/2.2.0/imagepipeline-native-2.2.0.pom'.

Hi @muqeet98 ,
Could you please share the gradle file with us? Ensure that you are only adding our nexus repositories to the list and that the mavenCentral() and google() are still available on the list.
Best regards,
Talsec Team

@syakymchuk Can you please share any docs for React Native Apps? Android and IOS both.

@muqeet98 , the documentation for React integration is not ready yet. This is the reason why we are not claiming the support of React Native officially. Though we would be happy to help you to pass through the integration process. You are more than welcome support / contribute #freeRASP with drafting your integration guideline, self-check list, and issues that you have experienced. Thank you in advance!

Hi @muqeet98 , Could you please share the gradle file with us? Ensure that you are only adding our nexus repositories to the list and that the mavenCentral() and google() are still available on the list.
Best regards, Talsec Team

`// Top-level build file where you can add configuration options common to all sub-projects/modules.
buildscript {
ext {
buildToolsVersion = "30.0.2"
minSdkVersion = 23
compileSdkVersion = 31
targetSdkVersion = 31
}
ext.kotlin_version = '1.5.21'
repositories {
google()
jcenter()
    // gradlePluginPortal()
}

dependencies {
    classpath("com.android.tools.build:gradle:4.0.2")
    classpath 'com.google.gms:google-services:4.3.14'
    // classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:1.5.31"
    // NOTE: Do not place your application dependencies here; they belong
    // in the individual module build.gradle files
}

}
allprojects {
repositories {
google()
// jcenter()
    mavenLocal()
        maven { url "[https://nexus3-public.monetplus.cz/repository/ahead-talsec-free-rasp]" }

    maven {
        // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
        url("$rootDir/../node_modules/react-native/android")
    }
    maven {
        // Android JSC is installed from npm
        url("$rootDir/../node_modules/jsc-android/dist")
    }
        maven {
  url "https://cashshield-sdk.s3.amazonaws.com/release/"
}

    // gradlePluginPortal()
    jcenter()

    // maven { url 'https://dl.bintray.com/ironsource-mobile/android-sdk' }
    maven { url 'https://www.jitpack.io' }
    maven { url 'https://zendesk.jfrog.io/zendesk/repo' }
    //  maven { url "https://dl.bintray.com/ironsource-mobile/android-sdk" }
    // maven { url "https://dl.bintray.com/ironsource-mobile/android-sdk" }
}

}
`

We suspect that the issue is with the jcenter repository which is no longer supported. From the log you sent, you can see that the dependency com.facebook.fresco:imagepipeline-native cannot be found. Also, there is no version 2.2.0 available.
We do not use this dependency in our project, so the issue should still remain even if you comment our repository out.
There is a version 2.3.0 available on mavenCentral() (here), but you need to update the repositories (add mavenCentral() right bellow google()) and increase the version of com.facebook.fresco:imagepipeline-native.
Let us know, if it helped.


Persist reservation ids

Summary:
The reservation id is added to core.instance.Reservation and is persisted with each
instance write. Legacy instance have no reservation id persisted. In these cases the
migrations MigrationTo17, MigrationTo18 and MigrationTo18100 read a reservation
without the id and then infer it based on the instance's tasks and id. The migrated
reservations and then saved alongside migrated instances. MigrationTo18200 migrates
all instances that were not caught by earlier migrations.
JIRA issues: MARATHON-8626

Please add a changelog item

Isn't this just an internal item. There is no bug fix, no new feature, no performance improvement nor behavioral change. No user should notice any difference. It is a change for the Marathon library we publish to Maven but I really hope that Metronome is the only consumer.
We could expose the reservation id to the user at some point. This would be part of a bigger feature involving resizing reservations. I'll brief you today.
This patch was just a log hanging fruit after I dug into reservation ids yesterday. I then discussed with @meichstedt how we could remove the last dependency between reservations and instances. We will have to spec any follow up.

Do you think we should add a change log entry for this? Not sure if there is any external user facing changes here, so maybe not necessary.

It's not even a bug fix. I think we can go without.


MMS Restore

I can't find a way to access my backed up MMS messages.  I can't restore them either since the feature is not yet available.  Is there any way at all to get the pictures?  I need some very bad!
Thank you
SMS backup+

They will all be on your Gmail account.
When will this feature be supported? If it's a long conversation history, missing many MMS messages defeats the purpose of a backup. Is there work in progress?

It's not planned: https://github.com/jberkel/sms-backup-plus/#faq-restore-MMS

I agree that MMS messages are an important part of a messaging thread.
I have a suggestion as an alternative to supporting MMS restore that we might all be happy with.
Instead of creating a MMS on the device, create a SMS with the Text (from the MMS) and a Link to the Media (that's added as an attachment in the email backup). That way the messaging thread is kept and the content is view able just not within the SMS app. There would need to be a Special TAG in the SMS to indicated it was a restored MMS and not needing to be backed up uniquely.
If you can't link to the email attachment, a link to the email would do (if possible to load up in gmail?).
If that is too difficult, perhaps the attachments can be stored on Google Drive and linked in that way.
Could use a URL shorter and I was going to suggest https://goo.gl/  but I have just read that google are going to shut this down and replace it with Firebase Dynamic Links which looks like a better option for this usage case. https://firebase.google.com/products/dynamic-links/
i.e. the attachment could be stored in firebase (linked account) with 5 GB of free storage.

@b8two interesting idea! You can link to specific emails in Gmail, (e.g. https://mail.google.com/mail/u/0/#inox/[1636afe1d0f09b90]) but I'm not sure if these URLs are stable, and I couldn't find any documentation (how to determine the id part of the url). If you can find anything, let me know.
In general I'm hesitant to add dependencies to other (non-free) services such as Drive or Firebase.

Hi,
I just played with this online API function to get a list of ID's (Appears to be stable if it's part of the API);
https://developers.google.com/gmail/api/v1/reference/users/messages/list
Then used the ID to get the related message;
https://developers.google.com/gmail/api/v1/reference/users/messages/get
& hence I could use this ID with the URL (without the square brackets); https://mail.google.com/mail/u/0/#inbox/1636afe1d0f09b90
BTW, Google Drive is 15GB Free and is shared with the gmail inbox Quota not really a separate service.

just to re-assure you about the ID.
id | string | The immutable ID of the message.
From: https://developers.google.com/gmail/api/v1/reference/users/messages

I guess this would also be a way to check for uniqueness of the SMS/MMS regarding avoiding duplication backups. If message contains; "https://mail.google.com/mail/u/0/#inbox/1636afe1d0f09b90" and Message ID "1636afe1d0f09b90" exists, skip

thanks for doing the research. Do the links open correctly  in the Gmail app on the phone?
I need to get the message ID from the IMAP conversation, which seems to be possible with an extension:

Access to the Gmail unique message ID: X-GM-MSGID
Gmail provides a unique message ID for each email so that a unique message may be identified across multiple folders. Retrieval of this message ID is supported via the X-GM-MSGID attribute on the FETCH command. The message ID is a 64-bit unsigned integer and is the decimal equivalent for the ID hex string used in the web interface and the Gmail API.

https://developers.google.com/gmail/imap/imap-extensions

I have tested but I was unable to find the right link to open on the phone.  The links work well on a desktop web browser but trying on a mobile, gmail tries re-directs to a mobile version of the page, dropping the message thread ID.   if you replace the #inbox with #all, then any message ID will link correctly on a desktop.
I could not find any threads about links that work on a mobile phone.  just ways to view messages in Original View to get different ID's etc. It appears that it would make more sense to put a copy of the attachment on cloud storage and put a link to that.
i.e.  https://drive.google.com/file/d/0B7RLMHD6ozpSMEZmRUhPMXFOMjg4Q1JDMjZfRWdaZTltX2xj/view
^ this file was added to google drive via gmail with the option "Save to Drive" (Icon in Gmail next to download),

I was able to move the file (on google drive) to a "MMS" sub folder (but it keeps the same link as above).
It's still google storage (same Auth)
The storage isn't public.
When I click the link in a SMS, Google Drive app opens the link and (probably because I have two gmail accounts on my phone) checks which account to open it from (authenticated)

I understand that you don't have access to do this via IMAP interface but perhaps this is another check box option in the app,
Backup "MMS attachment on Drive Storage".
Restore "MMS attachements linked to Drive Storage"

I have a Note 8 on Verizon and I see my MMS messages in Gmail, and SOME did come back after a Restore, but not all...


Adding Route table

Issue #, if available: #489
Description of changes:

adds route table resource
refactors e2e tests to share common resources (i.e. VPC)

otherwise, vpcs would need to be created for each test which is inefficient and may exceed quota
now, only 2 vpcs are ever created per test run
could not use @pytest.fixture(scope="session") on vpc because we use pytest-xdist

adds route table tests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/cc @vijtrip2

/assign @RedbackThomson

/lgtm


Setup Dependabot for GitHub Actions

Dependabot found a security vulnerability in a GitHub Actions used here, but didn't automatically open a PR to fix it like it did in https://github.com/amarvin/fantasy-football-bot/pull/99.

I enabled the ability for Dependabot to open PRs:


returned to N times and vue-router BUG

Https://vue-hn.now.sh/top
Jump page 2 times after the return is also returned to 2 times to return

wat


Move dockerfiles and deploy in contrib directory

Current coverage is 45.54% (diff: 100%)

Merging #227 into master will increase coverage by 1.41%

@@             master       #227   diff @@
==========================================
  Files            11         11
  Lines          8989       8989
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3967       4094   +127
+ Misses         3982       3839   -143
- Partials       1040       1056    +16

Powered by Codecov. Last update cabe1eb...2685c95


Please invite me to the GitHub Community Organization

Name
Khirasagar
Discord Username (if applicable)
No response
Additional Context
twitter

Break issues into actionable tasks. Tackle complex issues with task lists and track their status with new progress indicators.


DATA URIs are stripped from href attribute

Bug description
I'm trying to include links to PDF files in a chart, where the links include the PDF report as base64 data URI, e.g.
PDF Report
This tag works fine when included in a regular HTML page.
In the Superset TABLE chart, the value of the href attribute gets stripped.
Setting HTML_SANITIZATION = False in the config doesn't seem to make a difference.
Is this expected, that Superset strips the value of the href attribute, when it contains a data URI (or maybe it's related to length), when HTML_SANITIZATION = False is set?
How to reproduce the bug
Add a column with  HTML tags to a data source, where the href attribute contains a data URI.
Create a TABLE chart that includes that column.
The href attribute will not have a value in the resulting links in the chart.
Screenshots/recordings
No response
Superset version
3.1.3
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

Seems like the bot pretty much nailed the answer. I'll close this as complete, since it answers the question and gives you the info to work around things. Holler if this needs reopening for any reason.


Atmos Crab Pet + Ghost Role

About the PR

Instead of a couple of maps having an unnamed crab mob, Atmos in every map now has a proper pet crab: Tropico, the noble defender of Atmosia! He is a round-start ghost role, and can be converted to a Revolutionary to honor his namesake.
Why / Balance

Atmos's pets are in a weird state--sometimes it's Exception, sometimes it's a nameless crab, sometimes it's nothing at all. This PR would add a new, consistent pet for Atmos in the form of Tropico the Crab. Note that Exception hasn't been removed from any of the maps, just had his spawner moved, if anything.
Tropico is a new round-start ghost role, as well. The idea behind this is to give Atmos a new player-controlled companion to keep them company during lonely shifts holed up in Atmos, as well as something to rubberduck ideas off of. Tropico can also be converted into a Revolutionary as a nod to the game Tropico, which gives Atmosian Revolutionaries another dimension to consider. It also is just kind of funny I think.
Technical details

A new mob and spawner have been added for Tropico, using MobCrab as a parent, and the spawner has been added to the atmos rooms in all current maps. MobCrab has also been edited slightly to facilitate players controlling it. Specifically, mostly for Tropico, a new crab replacement accent has been added, and MobCrab has been given arachnid speaking sounds for when it talks. When successfully petting, crabs will now play an arachnid laugh sound, since it's the most similar sound in the game to sounds that real life crabs make.
Media

https://youtu.be/_eHYbIYwLx8

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

N/A
Changelog

add: Added a new ghost role in the form of Tropico the Crab, the stalwart companion of Atmosia! Be aware--Revolutionaries may flash him and use his claws for evil.

with the ghost role removed, this should be the CL instead:

add: Added a new pet in the form of Tropico the Crab, the stalwart companion of Atmosia! Viva!

note: leaving the ability to convert him into a rev even though it doesn't do anything is intentional because it's funny

can you remove the mapping changes, multiple maps being changed in one pr isnt standard we like to atomize them, its easier to just incorporate additions into the normal mapping updates

can you remove the mapping changes, multiple maps being changed in one pr isnt standard we like to atomize them, its easier to just incorporate additions into the normal mapping updates

oh that makes a lot of sense actually, i’ll fix that in a bit

turn off rev convertible and change the changelog

turn off rev convertible and change the changelog

do i have to edit the CL in the original PR post for it to work?

also, should i add it to the list for the thief antag or is that out of scope for this PR


Setting RazorPagesOptions.AreaRootDirectory and RootDirectory to same folder breaks all searches

Change Startup.ConfigureServices(...) to include
            services
                .AddMvc()
                .SetCompatibilityVersion(CompatibilityVersion.Version_2_1)
                .AddRazorPagesOptions(options =>
                {
                    options.AreaRootDirectory = "/";
                    options.RootDirectory = "/";
                });

Expected: At worst, some confusing between area-specific and non-area pages.
Actual: 404s for all pages, even full paths such as https://localhost:44312/About.cshtml or https://localhost:44312/MyArea/Index.cshtml

Problem may be more general. 404s seem to be the new norm when only AreaRootDirectory is changed to /. In that case i.e. with RootDirectory set to its /Pages default, all page searches fail whether looking for an area-specific page or not.

We specifically disallow files that are under the area route to be reachable via their non-area route. They should however be routeable if they are located in /[AreaName]/[FileName].cshtml. What's the path of the files on disk?

With the configuration shown above,
root / About.cshtml
          Contact.cshtml
          Index.cshtml
          MyArea / About.cshtml
                          Contact.cshtml
                           Index.cshtml
          Shared / *

And, nothing is routable.

As part of addressing https://github.com/aspnet/Mvc/issues/7608, we removed the option to modify the AreaPagesRoot. This is no longer a valid scenario.


[python] Include OS info in show_package_versions

Python counterpart to #1176

@eddelbuettel using os might be underkill :)
I think distro is exactly what we want -- thank you! :)


Question about GraphQL Subscriptions and Adonis WebSocket

It's difficult to explain problem, but I will try:
I have tried to realize GraphQL Subscriptions (througs apollo subscriptions-transport-ws) with AdonisJS framework and WebSockets, and it works, but:
When I connect SubscriptionServer through adonis-websocket WS server - I works, but with some troubles, adonis-websocket send ping packets and after timeouts and limits it disconnect connections with clients. (it is correct logic, but GraphQL SubscriptionServer cannot work with it)
I had to connect GraphQL SubscriptionServer through new WebSocket server, it works fine, but ONLY if I disable Adonis WebSocket. Because two WS servers in one http server don't work correctly see issue
Therefore if I will use GraphQL Subscriptions, I cannot use Adonis WebSockets.. and same to backward...
I ask only your opinion, guys, and maybe some solution
Thank YOU!

Hello @yariksav
Sorry for getting late on this. Ideally it's not possible to make Adonis work with apollo subscriptions seamlessly.
Adonis adds a lot to the Websocket protocol, and I am not sure if apollo subscriptions uses vanilla Websocket connection.
I have plans to make GraphQL as the first class citizen in 5.0 and then I can re-visit apollo subscriptions and see if a clean integration is possible.


Explore different CSS-in-JS libraries

Necessary since Stitches is now not being currently maintained.
Current Options:
✅ PandaCSS
🚫 Emotion
🚫 Styled Components
🚫 Linaria

Currently doing a POC in a SK project with Panda

would it be viable to style components using css in the svelte files in <style> tags, and having shared .css files for common theme styles, and allowing users to inject styles inline using the style prop? Would this significantly reduce the bundle size by not requiring a runtime library?

Right now, without using any runtime library, the only way to override in <style> with be with :global(), since svelte does not provide any other way. One of the things we are exploring is to pass css variables as props (svelte allows this), but we still want to enforce the theme (so passing a blue with result in using our color variable for blue600, for example). This is still in POC phase, since we want to maintain the same feature set, even if the style API totally changes

Another possibility would be vanilla-extract.

What about using SUIT CSS naming convention and SASS in component <style> tags? Then users could import a global stylesheet via JS import statement in the layout, which could override needed parts. The naming scheme would make it clear what class belong to what component.
Also, as it would not depend on external library, but Svelte built-in solution, it would reduce the risk of having to refactor again in case a 3rd party styling library is selected, and it gets abandoned.


How to run ssn on custom videos

Hi @yjxiong ,
Thanks for your cool work, it's really useful. I wonder how to run ssn action detection on custom videos(which are not included in ActvityNet and Thumos) using pretrained model, can you give me some hints? Thank you in advance.

+1

Maybe some step as follow:
1 load a video, extract all frames with a interval of 6
2 compute optical flow on these frames
3 use TAG algorithm get temporal action proposals
4 process each proposal using TSN network
5 save all detect result(such as in json file), then re-edit the test video with the detect result to get a visualizing demo.

Hi @Tonyfy ,
Could you explain what ´s the meaning of each line in the file thumos14_tag_test_normalized_proposal_list.txt. How can we get the file for our own videos? Thanks.

https://github.com/yjxiong/action-detection/wiki/A-Description-of-the-Proposal-Files

Hi, @yjxiong. Thank you very much. I just try to test a few videos in thumos14. But the result is very weird. I only got a file called ´result´ with one line like ¨ <80>^D<95>^C^@^@^@^@^@^@^@}<94>¨ after I run python ssn_test.py thumos14 RGB none result_inceptionv3_thumos14_imagenet_rgb.npz --arch InceptionV3 --use_reference. I don´t think my result is correct. Could you tell me what the result should be like? The scores are stored in the file ¨result¨?

hello.
I come cross a trouble that I can't solve it.so(I am a green hand using pytorch) ,I hope your help,thanks a lot.
when i load the module parameter ,and it will say:
RuntimeError: While copying the parameter named inception_5a_3x3_reduce_bn.running_mean, whose dimensions in the model are torch.Size([192]) and whose dimensions in the checkpoint are torch.Size([1, 192]).
and then I want to modify the model to adapt it ,but how to do it ? or you have the other method to do with it?? @vra @vikashch @Tonyfy @jiaozizhao

We released the TAG proposal code. Now you can run your own proposal models to get proposals for any video.

Hi @yjxiong
Thank you for this great work
I'm trying to find a way to use my own video (i.e. video not from any dataset)
I understand that "TAG proposal code" can help me with this
But I don't find this code
can you point me to the file?

@SHaiHosh
You can find the instructions here
https://github.com/yjxiong/action-detection#train-tag-models

I meant that I want to evaluate my own video which does not appear in the THUMOS14/activitynet list. Just to upload a movie and get it trimmed (and the trimmed sections classified)
You referred me to the training section
Am I missing something?

@SHaiHosh You are not missing anything. That part contains the instruction for running TAG proposal generation.

@yjxiong
I see that I have to run the testing and validation for generation of series of sliding-window proposals on THUMOS14
Only then I can train a binary actionness classifier
Then I should be able to split the video into actions (which is my goal)
My question is: Do I really have to go over all this process in order to be able to split my video?

@SHaiHosh
You can use the pretrained actionness classifier here https://github.com/yjxiong/action-detection/blob/master/README.md#obtaining-actionness-score

Hi @Tonyfy ,
Could you explain what ´s the meaning of each line in the file thumos14_tag_test_normalized_proposal_list.txt. How can we get the file for our own videos? Thanks.

Hi, have u resolved the problem?


When running a new workflow, you have to manually click refresh and there are not automatic updates.

Describe the bug
When running a new workflow, you have to manually click refresh and there are not automatic updates.
To Reproduce
Steps to reproduce the behavior:
When I re-run a workflow, I see the steps progress:

When I run a new workflow, I have to hit refresh to see it:

Expected behavior
Running a new workflow should work similarly to re-running a workflow and update the UI without needing to manually hit the refresh button.
Screenshots
If applicable, add screenshots to help explain your problem.
Extension Version
v0.25.8
Additional context
Found by @Dazz563 and posted to Discussion here https://github.com/github/vscode-github-actions/discussions/208

Hi, are there any updates on this issue please? Thanks!

Hey @KetchupOnMyKetchup  - do you have any updates whether this will be investigated?  I see it was moved to Triage over a year ago, so maybe slipped through the cracks?

I’ve developed (for me a better) a workaround for this issue that might help you as well. This extension displays the latest build status for GitHub Action workflows and GitLab pipelines in the status bar, automatically refreshing for branch or tag builds and when switching files or repositories.
https://marketplace.visualstudio.com/items?itemName=DanielChristianSchroeter.git-tag-release-tracker


feature: better logging in the action runner

hey @omBratteng can you explain a bit about the issue and if possible assign it to me.

I actually don't remember what I had in mind when I opened this issue. Think it was something similar to how https://github.com/lowlighter/metrics/ does their logs. Well structured.
────────────────────────────────────────────────────────────────────────────────────────
Metrics                                                         │
Setup                                                           │ complete
Version                                                         │ 3.34.0
Using prebuilt image                                            │ true
Debug mode                                                      │ false
Debug flags                                                     │ (none)


Add evolution for applicants table

Describe the solution presented in this issue.
Add an evolution that modifies the applicants table as defined in the TDD.  This adds the following columns:

first_name - varchar
middle_name - varchar
last_name - varchar
email_address - varchar
phone_country_code - varchar
phone_number - varchar
date_of_birth - date

Indexes should also be applied to each of these columns.  First/middle/last names and phone number should use a GIN index on a trigram using the pg_trgm extension.  Email and DOB should use default btree indexing.
Include downs to drop the columns and indexes.
See https://github.com/civiform/civiform/commit/fe9f0d1e34296dac643da6249c2d7b70abd83eb3 (this can probably be used as-is).
Additional Context
TDD
Done when:
What criteria need to be met for the issue to be finished?

[x] applicants table has the new columns and indexes

When upgrading to the latest version, our instance fails on this evolution.  The civiform database user does not have anough permission to run the create extension command.
Our current pattern is to create the extensions when creating the database, before loading data.  When we manually create the extension ahead of time, the evolution crashes because the extension already exists.
Do you typically grant the civiform database user role SysAdmin permissions?

Rock on, we granted civiform superuser, ran the evolutions, then revoked superuser.


the link is fail

See Feed Different batches to Student and Teacher, Feed Cached Values for details of the above features.
the link can't visit

Still cannot visit it?
The link looks ok to me now.


Navbar Should be Sticky

UI Enhancements : Navbar Should be Sticky for better user experience
https://github.com/user-attachments/assets/922a2da7-f73c-4153-ae23-7d09e317535c

no need of this issue #118 is open if interested i will assign you


fix: Unable to see apps in home page after git connect fails

Description

If the user aborts git connect flow or if there is any error then the app is missing from the homepage.

Fixes #13385
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Locally
JUnit tests

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

We should fix this in connect flow as that's the root cause for failure cases. Also what happens if we end up in having empty default branch for multiple branches of course considering the error scenario that user has reported?

@abhvsn This fix is for the empty default branch, when only the ssh key is present and no other information is present in db.
Not sure what is the issue in connect flow. AM i missing something here?

@AnaghHegde this is required but I'm suggesting, what happens when connect flow is not successful because of some failure in that case also we end up in stale application should we remove the gitApplicationMetadata to keep the DB clean

@abhvsn it is good to remove the git metadata from db in case of failures to keep things clean.But this won't affects us.
I have added test cases for the current scenario. But we don't have to specifically check for both branchName and defaultBranchName as we don't have case where only one of them is present.

Still I think adding a testcase for connect and create branch flow is must though removing a gitMetadata can be optional, I'm asking this because we may end up in branched applications in DB without defaultBranchName but have all the other fields which will result in duplicates in homepage

@abhvsn In create branch flow, we set both defaultBranch and branchName fields. So if the resource is created in DD then both of them will be present. It won't be a case where only branchName is present and not defaultBranchName. We copy the gitApplicationMetadata from the parent app and then we update fields before saving it to DB. So i don't see how we end up in a state you are mentioning. The same goes for connect flow. If we save the branch name in db then the default branchName also gets updated as they are all updated in the same map.

/ok-to-test sha=55a19c8

/ok-to-test sha=4dfd800

@abhvsn I have added test case for the above case in this PR. If you look at this L.304 , i am testing the same scenario.
https://github.com/appsmithorg/appsmith/blob/4dfd800d9b2f53b7f4338615767a1e675cf8d7bc/app/server/appsmith-server/src/test/java/com/appsmith/server/solutions/ApplicationFetcherUnitTest.java#L304

/ok-to-test sha=bd9ae14


[Suggestion] Search by mod name

If you were able to search items by mod name I would use this to replace EMI instantly. Please add this if you ever get a chance as I hate the cluttered look of JEI and EMI, and would love to be able to use this to replace them.
Thanks!

Hello! I dug around in the code and i believe this is vanilla functionality


HPKE + cmake (c++): Missing transitive target?

Original Issue
While trying to build a simple demo project doing Hybrid Encryption based on the official example cc/examples/hybrid_encryption (but flattening / isolating / simplifying everything into a fresh tink-external project) using cmake  i stumbled upon the following issue:
This nearly example-identical cmake config:
cmake_minimum_required(VERSION 3.8)
project(tink_poc CXX)

set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_STANDARD 17)

add_subdirectory(third_party/tink)

add_executable(example cli_example/example.cc cli_example/util.cc)
target_include_directories(example PUBLIC)
target_link_libraries(example tink::static absl::flags_parse)

leads to this error during linking:
/tink_poc/cli_example/example.cc:92: undefined reference to `crypto::tink::RegisterHpke()'

Remark: example.cc has these parts (without the original macros/flags) and compiles (until linking)
#include "tink/hybrid/hpke_config.h"
...
result = crypto::tink::RegisterHpke();
if (!result.ok()) return result;

Workaround
I fixed this and verified the correct working of the example built by explicitly declaring the target: tink_internal_hybrid_hpke_config in the cmake config:
...
add_executable(example cli_example/example.cc cli_example/util.cc)
target_include_directories(example PUBLIC)
target_link_libraries(example tink::static absl::flags_parse tink_internal_hybrid_hpke_config)      # <- CHANGE

Question
This explicit declaration looks like something which should have been automatically added when declaring tink::static.
As this example currently seems to be a second-class citizen when cmake (instead of bazel) is used (https://github.com/google/tink/commit/68ea64916bcf6bdce07c926ef0d844f34c0d6da8), it's hard to tell if this is oversight or if there is a good reason for this?
Version
tink @ 27b061bb
= 1.7.0
(Ubuntu 20.04, cmake version 3.21.3)
Remark / Thanks
The library, docs and examples look really nice in general!

Hi @sschnug as you correctly pointed out, HPKE is not exposed by tink::static, and ideally it should be there; Unfortunately, simply adding it to tink::static would break compatibility with OpenSSL. So the exclusion is intentional :)
If you use BoringSSL, adding the tink_internal_hybrid_hpke_config target is the only option AFAICT.

Closing this issue. Feel free to reopen it if there are any further issues/clarifications needed!


timeout

Hey MDHIGGINS,
I don't know what happened to my problem report from before, but I found a possible solution. This is the one about Sparkfun WiFi IR Blaster WRL-15031
In the IR rec demo, I found that I needed to adjust the timeout of UNKNOWN signals. I set it to basically ignore unknown signals (1000ms) and that seems to have solved the problem with receiving. I don't remember having that problem with this board, the first board I tried I did not.
The LED problem persists. This board used GPIO 5.

https://github.com/mdhiggins/ESP8266-HTTP-IR-Blaster/issues/294#issuecomment-674339622
You original issue is here but I'll crosspost this


ImportError: cannot import name 'get_terminal_size' from 'click.termui'

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the Typer documentation, with the integrated search.
[X] I already searched in Google "How to X in Typer" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to Typer but to Click.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
import typer

Description

Install latest versions of typer (0.4.0) and click (8.1.0)
Run script with single line import typer
Import fails due to method get_terminal_size having been removed in click 8.1.0 (https://github.com/pallets/click/pull/2130)

Traceback (most recent call last):
  File "test.py", line 1, in <module>
    import typer
  File "/.venv/lib/python3.8/site-packages/typer/__init__.py", line 12, in <module>
    from click.termui import get_terminal_size as get_terminal_size
ImportError: cannot import name 'get_terminal_size' from 'click.termui' (/.venv/lib/python3.8/site-packages/click/termui.py)

Operating System
Linux
Operating System Details
Ubunutu 20.04
Typer Version
0.4.0
Python Version
Python 3.8.10
Additional Context
No response

I think that'll be fixed in #375

подтверждаю проблему
появилась при  импорте
import spacy

Same here, also running into this exception with import spacy for spacy version 3.2.3. Reverting to spacy version 3.1.3 "solves" the problem for now.

Same issue also with the minimal example from the docs on a clean Conda environment with Python 3.9.11. Actually, the error is raised when merely running import typer in the Python console:
➜ python
Python 3.9.11 (main, Mar 28 2022, 05:06:49)
[Clang 12.0.0 ] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import typer
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/drorata/anaconda3/envs/foo/lib/python3.9/site-packages/typer/__init__.py", line 12, in <module>
    from click.termui import get_terminal_size as get_terminal_size
ImportError: cannot import name 'get_terminal_size' from 'click.termui' (/Users/drorata/anaconda3/envs/foo/lib/python3.9/site-packages/click/termui.py)

Exactly the same error for me with Python 3.7.13 :
Python 3.7.13 (default, Mar 18 2022, 17:46:12)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import typer
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/var/www/ai/env/lib/python3.7/site-packages/typer/__init__.py", line 12, in <module>
    from click.termui import get_terminal_size as get_terminal_size
ImportError: cannot import name 'get_terminal_size' from 'click.termui' (/var/www/ai/env/lib/python3.7/site-packages/click/termui.py)

After downgrading Typer to 0.3.2 the problem is gone.

After downgrading Click to 0.4.1 the problem is gone.

This temporary solution : https://github.com/explosion/spaCy/issues/10564#issuecomment-1081570327 works for me → pip install click==8.0.4

This temporary solution : explosion/spaCy#10564 (comment) works for me → pip install click==8.0.4

That correct !! I just fix the same your way. I downgrade click from 8.1.0 to 8.0.4 It's working now.

I am seeing the same on a dependabot PR proposing to update the click version to 8.1
All my "import typer" statements fail with this error:
.tox/py3/lib/python3.8/site-packages/typer/__init__.py:12: in <module>
    from click.termui import get_terminal_size as get_terminal_size
E   ImportError: cannot import name 'get_terminal_size' from 'click.termui' (/home/feganj/wip/.tox/py3/lib/python3.8/

This temporary solution : explosion/spaCy#10564 (comment) works for me → pip install click==8.0.4

Thanks, I uninstall click and install again with 8.0.4 and works

Click is an absolutely terrible dependency.
They make deprecation warning one minor version ahead for functions that do reimport and could be working for years.
That's a terrible management for such a wide dependency, and now all tools are split into demanding click <8.1, demanding 7.* and demanding > 8.1.
If you fix black, prefect and typer are still not working. 😠

I removed that first line.

I think the basic point is that Click has caused problems for their consumers, including typer, by breaking backward compatibility without a lot of warning or prep time. Have you (or anyone) reported that to the Click project? Maybe they would choose to reintroduce this symbol in a 8.1.1 to restore compatibility, and then set about removing it again in a more controlled fashion. That seems the best solution to me. As far as the typer project goes... I don't know the history, but if Click do this sort of thing a lot then I agree it would make sense for their consumers to try to limit vulnerability to it where possible.

@jofegan yes, that was done before me several times. You can track their reaction is the Click repo.

Maybe they would choose to reintroduce this symbol in a 8.1.1 to restore compatibility

That what I expected being done. You're very welcome to suggest this to click team.

Maybe they would choose to reintroduce this symbol in a 8.1.1 to restore compatibility, and then set about removing it again in a more controlled fashion.

But we did remove it in a controlled fashion. Using this function issued deprecation warnings starting in 8.0, released 10 months ago. The deprecation was mentioned high up in the changelog, as was the removal. If projects didn't respond to the deprecation yet, why would adding it back in and then removing it again change anything? It would just delay the same events, and then when we removed it again we'd be in the same situation.
It doesn't matter how long we provided warning, because Typer (and apparently no one using Typer either) wasn't testing with deprecation warnings treated as errors, so they would never have been able to proactively respond to the deprecation. I've opened #379 to show how that can be enabled. This is not a problem specific to Click: Python itself, or any libraries used in test/docs environments, can also be issuing deprecation warnings that Typer (or applications using it) is unaware of right now.
One way you, or applications you use, can control this is to use a project management tool like pip-tools, pipenv, pdm, or poetry to pin your dependencies and control when updates are seen and applied. Again, this is not specific to Click. If you pin any dependency (or if a library you use doesn't proactively test warnings), then you need to pin your entire dependency tree to avoid one library updating while another is pinned or not updated yet.

It doesn't matter how long we provided warning, because Typer (and apparently no one using Typer either) wasn't testing with deprecation warnings treated as errors, so they would never have been able to proactively respond to the deprecation.

If nobody does, you still insist that's the way?
I know most of my projects just can't pass such testing because click is not the only dependency, and there are always some things that are not yet fixed in the freshest version (pandas-depending stack, numba-depending stack, torch and tf are other snowflakes).
Once again, you can just reintroduce function because its existence does not break anything and unlikely will. Absence of it already breaks a lot.

@davidism please verbalize your position, that was your diff. Thumbs are not an argument.

I hear what you're saying. In my case, the package versions are pinned and all that has happened is that dependabot proposed an upgrade to Click 8.1.0 and that PR failed CI because it would break Typer. Not a big deal; there's nothing in 8.1.0 that I specifically need and that's just my pins and CI doing their job and stopping me from breaking my own build. But this incompatibility does mean that I won't be able to accept any future updates to Click at all until Typer makes a change to compensate. That can be the tricky part - if one of the affected packages isn't well maintained I may be stuck on the old click version for a long time. But it sounds like that will be okay here.

But it sounds like that will be okay here.

Unless other project you depend critically on requires fresh black (updated after recent update), which requires fresh Click. Which typer can't work with.

Thanks for the report @jmhummel!
And thanks everyone for the conversation.
I just released Typer 0.4.1 that should handle this. :rocket: :nerd_face:

Thank you very much for the understanding, in particular to those showing your support to these free, open source projects, and the maintainers, like David and me. :heart: :bow:

I'm sorry this made your lives and work difficult for these last couple of days. What happened was that Typer was importing and exporting that utility function from Click just as a convenience for users, but it wasn't used internally, it wasn't called, and that way I didn't catch the deprecation warning.
It was unfortunate indeed, but I wouldn't actually think it's something worth shaming me very aggressively and even less the maintainer(s) of Click that always do an amazing and thorough job.
These types of errors and accidents happen, they happen in open source and even more in closed source, they happen to projects of all sizes and types. The best we can do is to continue to try and make the whole community a welcoming place where we try to help each other instead of blaming and shaming others.

@jmhummel if you confirm that the new release solves your problem you can then close this issue. :nerd_face:

Thank you!

I can happily confirm that the issue mentioned in this comment above is removed once updating typer to 0.4.1. Thanks!

Apologies to add to noise. Reinforcing @drorata's comment that upgrading Typer to 0.4.1 resolved the issue and managed not to break any other dependencies in my gcp + pandas data pipeline.

Bot closed this but it appears to be unresolved.

Here's the missing function definition at the time of removal:

def get_terminal_size() -> os.terminal_size:
    """Returns the current size of the terminal as tuple in the form
    ``(width, height)`` in columns and rows.

    .. deprecated:: 8.0
        Will be removed in Click 8.1. Use
        :func:`shutil.get_terminal_size` instead.
    """
    import shutil
    import warnings

    warnings.warn(
        "'click.get_terminal_size()' is deprecated and will be removed"
        " in Click 8.1. Use 'shutil.get_terminal_size()' instead.",
        DeprecationWarning,
        stacklevel=2,
    )
    return shutil.get_terminal_size()

I believe it should be trivial to fix this particular issue (and it wouldn't even break compatibility with click-8.0).

…Waitaminute
According to the docs, this was fixed back in v0.4.1 over a year ago.


ImportError: No module name html

Traceback (most recent call last):
  File "linkfinder.py", line 11, in <module>
    import re, sys, glob, html, argparse, jsbeautifier, webbrowser, subprocess, base64, ssl, xml.etree.ElementTree
ImportError: No module named html

Solution: pip install html
or
sudo apt-get install html

:+1:

python linkfinder.py -i http://xxxx/js/hoverIntent.js -o results.html Traceback (most recent call last):   File "linkfinder.py", line 372, in <module>     ''' % (html.escape(url), html.escape(url)) AttributeError: 'module' object has no attribute 'escape'
any solution for this error ?

@hasnat0x01, make sure you are running Python 3.

@EdOverflow Thank you its working fine now

Install before requirement file, use this command for install pip
sudo apt install python-pip


Pulumi learn resource talks about terraform destroy

Problem description

Hi folks, I hope this is the right place for this but I stumbled upon https://www.pulumi.com/learn/importing/importing-via-code/ which has the following line: If you were following along, don’t forget to tear down your stack and tear down the Terraform resources you stood up! pulumi destroy and terraform destroy
Not sure where it's generated from but it probably shouldn't suggest terraform destroy
Affected product version(s)

Suggestions for a fix

Nope, this is actually what was meant, my bad!


[explorev2] making Datasource an Viz controls not clearable

Other SelectField and new ones can now be set as non-clearable for places where null isn't right.

lgtm


Error

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

VS Code Version:
OS Version:

Steps to Reproduce:

Hello, Im not sure if this was a accident or something but the issue is blank, could you please fix this or close it?


PHP code not highlighted

I am trying to add PHP code tab. here that php code was not highlighting. but java code was highliting

Hi @vinothkumar-k, could you please post an example of the markdown you're having trouble with? You should also be sure to naming PHP like this:
```php
code here
```

Not:
```PHP
code here
```

sorry
i was added "<?" this one , after that it was highlighted. .
thank u .  . .

Ah, cool!


Validate locale tag

Hello!
Sorry, if I missed that, but I can not find an easy way to validate given locale string in order to use it with globalize. E.g. if a locale string is valid and is supported by globalize, then I do other loading stuff, instead of catching errors.

Please, have you read https://github.com/jquery/globalize#locales? Let me know if that helps and if you have further questions.

Yeh, I have read that, but it is not exactly what I'm lookng for.
For example, when calling Globalize('en') we won't get any error, but then, if cldr data for 'en' has not been yet loaded we'd get an error. But in this case we can load 'cldr/datat/en/currencies.json' and so on, because we have such data in cldr set. But what if some of locales missed? And I want to check if I can load valid locale data before actual loading, cause I don't want to get errors.
For example, currently I have 352 locales in cldr/main/data (I got them after "npm module install cldr-data") so it is obvious that some locales are missing and I can not safely load every locale I want.
So it would be nice to check if necessary locale data is present and then safely load it or jump to fallback with default locale settings.

I've found availableLocales.json in cldr installation. It seems, that it might be useful to use it for my purpose.

Gotcha. The answer to your question is still open and lies in https://github.com/rxaviers/cldrjs/issues/30. I suggest we focus any discussion there, so I'm closing this issue here in favor of that. I'd be happy to apply here any outcomes of that issue.


make debug message easier to read

Hi
This little patch help us read debug message easier. Please consider.

Added this as an option in the latest version. See debug documentation at the bottom of:
https://sosimplesoftware.com/fmREST.php


Type error TS2322(resolvePageComponent + vite's import.meta.glob)

Laravel Vite Plugin Version: 0.6.1
Laravel Version: 9.39.0
Node Version: 16.17.0
NPM Version: 8.19.1
Host operating system: Ubuntu22.04(Laravel Sail 8.1) on Docker on Ubuntu20.04(WSL2)
Web Browser & Version: no relation
Running in Sail / Docker: Sail

Description:
I consulted on Discord about an issue #161 I created earlier, and got a reply saying it might be a bug, so I created this new one.
I think the cause of the error is that the type of the second arg of resolvePageComponent of laravel-vite-plugin and the return type of import.meta.glob of vite do not match.
Because if I use import.meta.globEager instead of import.meta.glob I get no error. (Unfortunately using import.meta.globEager causes no code-splitting for the pages, so this is not a workaround)
Currently I avoid type errors by calling resolvePageComponent on the javascript file, but hopefully the laravel-vite-plugin will fix it.
Thank you.
// laravel-vite-plugin/inertia-helpers/index.d.ts
export declare function resolvePageComponent<T>(path: string, pages: Record<string, Promise<T> | (() => Promise<T>)>): Promise<T>;

// vite/types/importGlob.d.ts
// https://github.com/vitejs/vite/blob/v3.2.4/packages/vite/types/importGlob.d.ts#L39

・・・

export interface ImportGlobFunction {
  /**
   * Import a list of files with a glob pattern.
   *
   * Overload 1: No generic provided, infer the type from `eager` and `as`
   */
  <
    Eager extends boolean,
    As extends string,
    T = As extends keyof KnownAsTypeMap ? KnownAsTypeMap[As] : unknown
  >(
    glob: string | string[],
    options?: ImportGlobOptions<Eager, As>
  ): (Eager extends true ? true : false) extends true
    ? Record<string, T>
    : Record<string, () => Promise<T>>
  /**
   * Import a list of files with a glob pattern.
   *
   * Overload 2: Module generic provided, infer the type from `eager: false`
   */
  <M>(
    glob: string | string[],
    options?: ImportGlobOptions<false, string>
  ): Record<string, () => Promise<M>>
  /**
   * Import a list of files with a glob pattern.
   *
   * Overload 3: Module generic provided, infer the type from `eager: true`
   */
  <M>(
    glob: string | string[],
    options: ImportGlobOptions<true, string>
  ): Record<string, M>
}

・・・

Steps To Reproduce:
See #161.

In addition to treating resolvePageComponent as JavaScript, I also avoid the error in the following way.
// app.ts

・・・

// impot DefineComponent type
import { createApp, h, type DefineComponent } from 'vue';

・・・

createInertiaApp({
    // specify the DefineComponent type to import.meta.glob
    resolve: name => resolvePageComponent(`../ts/Pages/${ name }.vue`, import.meta.glob<DefineComponent>('../ts/Pages/**/*.vue')),

・・・

});

If I want to solve in react, what I need to use??

Hi folks, this function is generic so you can specify whatever you need.
// Lazy globing...

resolve: (name) => resolvePageComponent(`./Pages/${name}.vue`, import.meta.glob<DefineComponent>('./Pages/**/*.vue')),

// Eager globing...

resolve: (name) => resolvePageComponent(`./Pages/${name}.vue`, import.meta.glob<Promise<DefineComponent>>('./Pages/**/*.vue', { eager: true })),

@timacdonald Thank you for your reply.
As I also mentioned above, I see that your solution avoids type errors.
However, this method is not documented. If this method is Laravel's official solution, I think it would be useful for many people to mention it in the docs.
By the way, when I use import.meta.globEager, Typescript's type inference works and I don't get an error even if I don't explicitly specify the type.
I think this is the result of the type definition of resolvePageComponent being properly adapted to Vite's import.meta.globEager.
Is it difficult to make type inference work for import.meta.glob as well?

@HaneulChile Hi.
In the inertia-react type definition, options.resolve passed to createInertiaApp is described as follows.
//<EMAIL_ADDRESS>
・・・

export type ReactComponent = React.ReactNode;

export type PageResolver = (name: string) => ReactComponent|Promise<ReactComponent>|NodeRequire // TODO: When shipped, replace with: Inertia.PageResolver<ReactComponent>

・・・

export type BaseInertiaAppOptions = {
    title?: HeadManagerTitleCallback,
    resolve: PageResolver,
}

・・・

export type InertiaAppOptionsForSSR<SharedProps> = BaseInertiaAppOptions & {
    id?: undefined,
    page: Inertia.Page|string,
    render: typeof renderToString,
    setup(options: SetupOptions<null, SharedProps>): ReactInstance
}

export function createInertiaApp<SharedProps = Inertia.PageProps>(options: InertiaAppOptionsForCSR<SharedProps>): Promise<CreateInertiaAppSetupReturnType>
export function createInertiaApp<SharedProps = Inertia.PageProps>(options: InertiaAppOptionsForSSR<SharedProps>): Promise<CreateInertiaAppSSRContent>

・・・

And the PageResolver is a Promise that returns a ReactComponent or a ReactComponent itself , which is also defined in inertia-react.
So if you use React, I guess you can solve it by using ReactComponent. (Like Vue3's DefineComponent)
Like this:
// app.ts

・・・

// import ReactComponent type
import { createInertiaApp, type ReactComponent } from '@inertiajs/inertia-react';

・・・

createInertiaApp({

・・・

    // specify the ReactComponent type to import.meta.glob
    resolve: name => resolvePageComponent(`../ts/Pages/${ name }.tsx`, import.meta.glob<ReactComponent>('../ts/Pages/**/*.tsx')),

・・・

});

Please try it if you don't mind.
If you get good results, it would be helpful for everyone if you could let us know here.

@mt-u both of these functions are generic, so you have complete control over them. Without testing, i  would assume that the following would work without issue...
// Lazy globing...

resolve: (name) => resolvePageComponent(`./Pages/${name}.vue`, import.meta.glob<ReactComponent>('./Pages/**/*.vue')),

// Eager globing...

resolve: (name) => resolvePageComponent(`./Pages/${name}.vue`, import.meta.glob<Promise<ReactComponent>>('./Pages/**/*.vue', { eager: true })),

@timacdonald Yes. You are completely right.
And I think it would be helpful for all users if your solution was featured in the Laravel docs.
Alternatively, I would be happy if the resolvePageComponent type definition was changed to work with Typescript's type inference, similar to the eager version of import.meta.glob (I mean import.meta.globEager, not import.meta.glob('', { eager: true }))
// In this case you have to specify the type.
resolve: (name) => resolvePageComponent(`./Pages/${name}.vue`, import.meta.glob<Promise<DefineComponent>>('./Pages/**/*.vue', { eager: true })),

// Not necessary in this case (Type Inference works)
resolve: (name) => resolvePageComponent(`./Pages/${name}.vue`, import.meta.globEager('./Pages/**/*.vue')),

Thank you.

Laravel does not officially support Typescript, so we won't document that.
If you feel there is an improvement to be made, we would welcome a PR to improve the types.

@HaneulChile Hi. In the inertia-react type definition, options.resolve passed to createInertiaApp is described as follows.
//<EMAIL_ADDRESS>
・・・

export type ReactComponent = React.ReactNode;

export type PageResolver = (name: string) => ReactComponent|Promise<ReactComponent>|NodeRequire // TODO: When shipped, replace with: Inertia.PageResolver<ReactComponent>

・・・

export type BaseInertiaAppOptions = {
    title?: HeadManagerTitleCallback,
    resolve: PageResolver,
}

・・・

export type InertiaAppOptionsForSSR<SharedProps> = BaseInertiaAppOptions & {
    id?: undefined,
    page: Inertia.Page|string,
    render: typeof renderToString,
    setup(options: SetupOptions<null, SharedProps>): ReactInstance
}

export function createInertiaApp<SharedProps = Inertia.PageProps>(options: InertiaAppOptionsForCSR<SharedProps>): Promise<CreateInertiaAppSetupReturnType>
export function createInertiaApp<SharedProps = Inertia.PageProps>(options: InertiaAppOptionsForSSR<SharedProps>): Promise<CreateInertiaAppSSRContent>

・・・

And the PageResolver is a Promise that returns a ReactComponent or a ReactComponent itself , which is also defined in inertia-react. So if you use React, I'm not familiar with React so I haven't been able to try it, but I guess you can solve it by using ReactComponent. (Like Vue3's DefineComponent) Like this:
// app.ts

・・・

// import ReactComponent type
import { createInertiaApp, type ReactComponent } from '@inertiajs/inertia-react';

・・・

createInertiaApp({

・・・

    // specify the ReactComponent type to import.meta.glob
    resolve: name => resolvePageComponent(`../ts/Pages/${ name }.tsx`, import.meta.glob<ReactComponent>('../ts/Pages/**/*.tsx')),

・・・

});

Please try it if you don't mind. If you get good results, it would be helpful for everyone if you could let us know here.

Worked perfect!!!. Thanks so much.


git2 holds open deleted packfiles, effectively leaking disk space

We have a long-running daemon that does some stuff with a repository.
We noticed that the disk backing the repository was "full":
thing@thing-0:/var/cache/thing$ du -chs .
7.3G	.
7.3G	total
thing@thing-0:/var/cache/thing$ df -h .
Filesystem      Size  Used Avail Use% Mounted on
/dev/sdb         16G   16G     0 100% /var/cache/thing

That might see illogical, but that's a hint that deleted files are being held open. If we ls our daemon's open files, that confirms it:
0 -> /dev/null
1 -> 'pipe:[16731674]'
10 -> 'socket:[16737720]'
11 -> 'anon_inode:[eventpoll]'
12 -> 'anon_inode:[eventfd]'
13 -> 'anon_inode:[eventpoll]'
14 -> 'anon_inode:[eventfd]'
15 -> 'anon_inode:[eventpoll]'
16 -> 'socket:[16729776]'
17 -> 'anon_inode:[eventpoll]'
18 -> 'anon_inode:[eventfd]'
19 -> 'anon_inode:[eventpoll]'
2 -> 'pipe:[16731675]'
20 -> 'socket:[16729776]'
21 -> '/var/cache/thing/monorepo/.git/objects/pack/pack-a02eda73e77509f55e734b4f80f0e49c60093184.pack (deleted)'
22 -> '/var/cache/thing/environments/.git/objects/pack/pack-be204a28036cbc4cd01c2248a6ee2168a1989a96.pack (deleted)'
23 -> '/var/cache/thing/environments/.git/objects/pack/pack-6e09f3d300e72f084b05ba7bc1ccce40c10f582c.pack (deleted)'
24 -> '/var/cache/thing/monorepo/.git/objects/pack/pack-daec94cd70f17f2b1d3a638ce87cb4c086ce4629.pack (deleted)'
25 -> '/var/cache/thing/environments/.git/objects/pack/pack-2ad6f4562cd68fbf4ecc4d5af17579aa590e7263.pack (deleted)'
26 -> '/var/cache/thing/environments/.git/objects/pack/pack-aed3c72e24e8ec9a5db6b11bc2a81f071b5596f6.pack (deleted)'
27 -> '/var/cache/thing/environments/.git/objects/pack/pack-d4e7ed286a0e2e79cd36688ac33c5f98948e6b6a.pack (deleted)'
28 -> '/var/cache/thing/environments/.git/objects/pack/pack-6ffb344d987435d1cca7934ed89e7ddf9d6d79a2.pack (deleted)'
29 -> '/var/cache/thing/environments/.git/objects/pack/pack-09e7c333cc2b2dbb5b30550298b31f30ea4439c7.pack (deleted)'
3 -> 'anon_inode:[eventpoll]'

So, we can see here that we're holding a bunch of packfiles, which explains why the used space is so much higher than the du total.
We keep the git2::Repository object around: I think I didn't know if it was expensive to create, and we only have two repos, so we just create the git2::Repository objects are startup, and then keep them as global state.
But that seems to cause a "disk" leak, over repeated fetches & repacks.

IIRC it’s not unbounded, but libgit2’s limits are a bit optimistic. You can set the MWINDOW_FILE_LIMIT via the raw bindings (libgit2-sys).

IIRC it’s not unbounded, but libgit2’s limits are a bit optimistic.

It was holding onto 53% of the total size of the volume, at which point the volume ran it out of space.

You can set the MWINDOW_FILE_LIMIT via the raw bindings

The only hit I get for MWINDOW_FILE_LIMIT on Google is a copy of this issue, and libgit2's doc's search doesn't recognize it.

https://github.com/libgit2/libgit2/pull/5396

For my reference, the docs page.
(The commit indicates that the default for that is unlimited.)


Altitude units?

Does anyone have any clue what units the altitude measurement is when you call player_status(player_id)? I could not correlate it to anything at all. I was getting values between 12000 and 18000 in Central Park NY (my units are configured to metric in game)

Hi,
have you figured it out?


Constants

In C, Macros are almost like constants which can be used anywhere. This is useful functionality which should be in Booleang:
#delay 10ms

circuit main {
    clock #delay { ... }
}

This might be best implemented in the lexer.


Need to set limits on critical resources: tree height

Related issue: https://github.com/ainblockchain/ain-blockchain/issues/299
Expected steps:

Add limit values to genesis params (maybe 'TREE_HEIGHT_LIMIT = 15' limit in 'resources' section?) => https://github.com/ainblockchain/ain-blockchain/blob/develop/genesis-configs/base/genesis_params.json
Check the updated value whenever a transaction is executed => https://github.com/ainblockchain/ain-blockchain/blob/develop/node/index.js#L305
Fail the transaction if it's over the limit

Fixed.


[ios] fix testComponentFactory

fix testComponentFactory

Current coverage is 88.44% (diff: 100%)

Merging #1747 into ios-feature-20161208 will not change coverage

@@           ios-feature-20161208      #1747   diff @@
======================================================
  Files                        41         41
  Lines                      2233       2233
  Methods                     354        354
  Messages                      0          0
  Branches                    500        500
======================================================
  Hits                       1975       1975
  Misses                      258        258
  Partials                      0          0

Powered by Codecov. Last update 59bc69e...99574bf


Fix the user stories & use cases according to Week 7 Tutorial

Hi @nguyendqminh
Can you take a look at the current user stories & use cases to see if there is anything to improve/fix according to the Week 7 Tutorial? Add more obvious and essential user stories?
@simonteozw can you help Minh verify this after he is done?
Thanks

Noted with thanks. I will finish it by tomorrow (Sat) 6pm.


Null-safety support

https://github.com/azhansy/flutter_hms_gms_availability

Project owner merged @azhansy changes. Issue closed.


Can Monkey cache be used in maui with .net 7

Hi looking for a caching solution for .net maui app and wondering if this has been successfully used in maui and .net 7?
Any suggestions
thanks

Hi looking for a caching solution for .net maui app and wondering if this has been successfully used in maui and .net 7?
Any suggestions thanks

works?

2.0 should be further optimized

@jamesmontemagno just for clarity is this .net maui ready ? should we wait for 2.0?
Thanks for your time and for providing this

will ship 2.0 into stable today


Add support for auto-registering options for a class

This would use the inspect module to get type annotations (can't auto generate without), args, and default values and register a config.

implemented.


Reproducibility Issue of Stablebaseline3 ( FinRL )

Hello everyone, I've been trying to work with FinRL ( Stablebaseline3), everything is fine except the reproducibility issue ( I'm getting inconsistent results every time I run my model ). I've seen a GitHub issue where community tried raising issue on the same, but it got closed without any conversation or solution.
Github Issue Link : https://github.com/AI4Finance-Foundation/FinRL/issues/190
It would be great if anyone can help me to fix this issue or suggest something that can help me to solve the issue.
Thanks in advance.

Stability and reproducibility are two common issues for the RL, since there are many local optima.
I suggest to try some skills, e.g., fix seeds, ensemble strategy, select the best one over multiple agents.

I'll agree, I've tried the seeding for my environment & for the algorithms also but they're not giving me the same results. Is it like even after trying the seeding it's unlikely to produce same results ? or I'm doing anything wrong because of which I'm getting inconsistent results ?

I do not know how do you set seeds.
I do not sure current RL lib such as SB3 has opened the seed setting.

Please check the below code snippet, this is what I've tried when you mean seeding.

@zhumingpassional help me if I'm doing something wrong or if there's something that can be tried on top of this.

we may not set seeds for SB3


Generalize sharpib

Description of the problem

The sharp immersed boundary solver was limited to spherical particles
In order to increase accuracy when using non sperical solids with immersed boundaries, generalization was required
A few simple shapes and the possibility to output them in the form of a signed distance function field were added

Description of the solution

A Shape base class with the relevant virtual functions were added
The Sharp GLS solver was adapted to use only the Shape functions, thus making the shape definition and the system assembly and solving orthogonal
A postprocessor for defined scalar functions was added to NS base, allowing to output this type of function from any solver; in this case, the signed distance function

How Has This Been Tested?

Unit tests for the Shape class were added
Applications tests using newly implemented shapes were added for gls_sharp_navier_stokes_3d and gls_sharp_navier_stokes_2d

Documentation

The Resolved CFD-DEM parameters page was updated
The sharp IB examples were adapted

Future changes

Add tests that verify the reaching of theorical orders of convergence with non spheres
Add the possibility to change the particle orientation in time by following a function (now only initial orientation can be defined)
Optimize the shape functions evaluation (reuse some intermediate values)
Add an example showing the new functionalities, possibly mixing with a sharp IB defined impeler
Add the option to not output the signed distance function field

@oguevremont I think your PR made one of the test unstable (all push seem to break)... could it be that the checkpoint is not compatible anymore?????

@Luckabarbeau once you are ok with it, i'm merging?


Any todo or list of features needs to add for this project

I saw the other two issue about the adding the button for stackoverflow and github and which is covered in #3 , which is pretty awesome and loved the current implementation of that.
Is there any other list of thing need to do for this project were contributions can be done . I would be happy to help in those. I guess adding a todo list anywhere in the readme or a separate pinned issue for that will work good.
Really liked the idea of this extension 👌👌👍👍

We'll merge that one pretty soon and I'm sure there are plenty of things that will need to be done.. #8 is one of them :)

I think now there quite a few items to pick from, so I'm closing this one.


Extract the log from a deployment and/or operation

Hi,
For now, the only way to have the log of the deployment is to read the console output, or think to redirect the output into a file.
the command tdp browse <id> only displays the first 20 characters.
It would be nice to be able to export the logs in a file.

Are you talking about the Ansible logs which result of the execution of an operation?
If yes, the appropriated command to see the whole log is tdp browse [id] [operation_name]. Is it what you were looking for?

Actually, when I wrote this issue, I didn"t know that there is a command to print the operation log of a given deployment.
This command is what I thought, thanks ^^


130 - Dataset citation thumbnail

What this PR does / why we need it:
This PR adds the dataset thumbnail logic to the UI. This is not yet connected to the API
Which issue(s) this PR closes:

Closes #130

Special notes for your reviewer:
This is not yet connected to the API, it will need an API extension to get the dataset thumbnail.
Suggestions on how to test this:

Install the necessary dependencies using npm install.
Build the design system with cd packages/design-system && npm run build.
Navigate back to the root directory with cd ../../.
Start Storybook with npm run storybook.
Visit the Dataset Citation section in the Storybook a new story has been added to show how the thumbnail image will be displayed in the Citation block.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Yes, this PR introduces a user interface change.

Is there a release notes update needed for this change?:
Dataset thumbnail added to the Dataset View Mode

coverage: 98.58% (+0.3%) from 98.265% when pulling 87e39071676bfefb6853a2e16b463b430732c23b on feature/130-dataset-citation-thumbnail into 35c157339c8dbea6d675763cd7c6980b8b943d48 on feature/160-display-actual-file-thumbnails.

Shouldn't a thumbnail image be displayed instead of the default image? @MellyGray
I'm on http://localhost:6006/?path=/story/sections-dataset-page-datasetcitation--with-thumbnail

@GPortas sorry, that was a last minute change and I mixed things 🙏 fixed now! I also fixed some merge conflicts

@MellyGray Can you please resolve the conflicts? Thanks!

Looks like we have another issue when building the app for beta testing: https://github.com/IQSS/dataverse-frontend/actions/runs/6732893099/job/18300451474
Can you fix it so we can stabilize develop as you did on the other PR?
Do you know how we can detect this before merging PRs? @MellyGray

@GPortas I fixed the merge conflicts
Also, both the storybook error and this one was caused by type errors so I added a new step to the Github action and to the precommit to check this type of errors


tidy poetry dev deps

constrained python version and added poetry dev deps

I salute you champion :shipit:


Python cleanup + tests

Summary of Changes
This PR cleans up some python code and adds unit test coverage for key notifications logic
Tests
Added tests to cover uncovered notifications logic.
Documentation
Added/modified docstrings for relevant methods
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes, including screenshots of any UI changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes all tests documented in the developer guide
[ ] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

Codecov Report

Merging #277 into amundsen-notifications will decrease coverage by 1.76%.
The diff coverage is n/a.

@@                    Coverage Diff                     @@
##           amundsen-notifications     #277      +/-   ##
==========================================================
- Coverage                    69.2%   67.43%   -1.77%
==========================================================
  Files                         139      113      -26
  Lines                        3500     2374    -1126
  Branches                      423      329      -94
==========================================================
- Hits                         2422     1601     -821
+ Misses                       1043      767     -276
+ Partials                       35        6      -29

Impacted Files
Coverage Δ

amundsen_application/api/metadata/v0.py

amundsen_application/log/action_log_callback.py

amundsen_application/api/__init__.py

amundsen_application/api/utils/request_utils.py

...n/base/examples/example_superset_preview_client.py

amundsen_application/base/base_preview_client.py

...ndsen_application/base/base_announcement_client.py

amundsen_application/wsgi.py

amundsen_application/log/action_log_model.py

amundsen_application/api/search/v0.py

... and 12 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 b1867a4...2f3fd97. Read the comment docs.


fix overhead estimates to be more accurate

Description
Update the aks estimation utilities so that overhead is calculated more accurately.
Related Issues
Checklist:

[X ] I have followed the contribution guidelines and code style for this project.
[X ] I have added tests covering my contributions.
[X ] I have updated the documentation accordingly.

we already merged this to staging, so I'm going to make the small changes and move this to push into staging


Generate disk image using hdiutil

As discussed in https://github.com/zchee/docker-machine-xhyve/issues/18. This removes the libguestfs dependency.
docker-machine -D create -d xhyve --xhyve-disk-size "2000" xhyve

docker@boot2docker:~$ df -h
Filesystem                Size      Used Available Use% Mounted on
tmpfs                   896.6M    123.7M    772.9M  14% /
tmpfs                   498.1M         0    498.1M   0% /dev/shm
/dev/vda1               996.5M      2.0M    926.3M   0% /mnt/vda1
cgroup                  498.1M         0    498.1M   0% /sys/fs/cgroup
/dev/vda1               996.5M      2.0M    926.3M   0% /mnt/vda1/var/lib/docker/aufs

docker info
Containers: 1
Images: 2
Server Version: 1.9.0
Storage Driver: aufs
 Root Dir: /mnt/vda1/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 4
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.1.12-boot2docker
Operating System: Boot2Docker 1.9.0 (TCL 6.4); master : 16e4a2a - Tue Nov  3 19:49:22 UTC 2015
CPUs: 1
Total Memory: 996.3 MiB
Name: boot2docker
ID: TZZC:VPCB:ZLL6:UCV5:HKZZ:4ZCI:XWVB:UHZG:HY5Q:EZER:ABYI:OXP6
Debug mode (server): true
 File Descriptors: 12
 Goroutines: 20
 System Time: 2015-11-22T11:21:38.821801822Z
 EventsListeners: 0
 Init SHA1:
 Init Path: /usr/local/bin/docker
 Docker Root Dir: /mnt/vda1/var/lib/docker
Labels:
 provider=xhyve

Output as expected 1GB (- 1GB swap created by b2d https://github.com/boot2docker/boot2docker/blob/master/rootfs/rootfs/etc/rc.d/automount#L28)
This is using now the same method as the bundled vbox and hyper-v drivers are using to bootstrap the disk.
I'll remove libguestfs, if you think this is :+1:

LGTM.
I understand all now! Sorry that I can not understand in the discus :(
Check your PR code, and I will merge later.
I wanted to develop the implementation of tuntap and xpc state, glad this issue is resolved.
Thanks PR, and long discus :)

@johanneswuerbach Let me ask you a small question.
Why dmg instead of img? The method, what more the performance will be better? Or another reason?
Either is fine. but I’m curious about this changed.

'dmg' is the hdiutil default, actually I planned to use sparse (a vbox like self growing image), but couldn't get it to work and will retry later. Haven't look at performance yet, but docker pull had roughly the same speed as my vbox machine.

@johanneswuerbach

I planned to use sparse (a vbox like self-growing image)

I do not know exactly sparseimage, but good idea.

but couldn't get it to work and will retry later

Can you tell me the details? It is very interested.

Haven't look at performance yet, but docker pull had roughly the same speed as my vbox machine.

Whether that impression is intuitive? or measurement?
I was easily measured img and vboxfs using the iozone previously, certainly slow results were obtained.

Thanks fixed english :)

xhyve
$ dd if=/dev/zero of=/mnt/vda1/tmp/out bs=8k count=200k; rm -f /mnt/vda1/tmp/out
204800+0 records in
204800+0 records out
1677721600 bytes (1.6GB) copied, 2.531537 seconds, 632.0MB/s

vbox
$ dd if=/dev/zero of=/mnt/sda1/tmp/out bs=8k count=200k; rm -f /mnt/sda1/tmp/out
204800+0 records in
204800+0 records out
1677721600 bytes (1.6GB) copied, 3.222129 seconds, 496.6MB/s

looks sane

Can you tell me the details? It is very interested.
it failed to format the disk, but I didn't spend much time to investigate why

I'll remove libguestfs, if you think this is :+1:

Done

@johanneswuerbach
Good!
The xhyve is better than vboxfs, it seems that was a mistake of my measurement.(?)

it failed to format the disk, but I didn't spend much time to investigate why

Ok. I keep in mind the method.
All LGTM.
Merged!
Thank you again :)


GoogleMap.getLatLong() obsolete?

Seems to me that com.lynden.gmapsfx.javascript.object.GoogleMap getLatLong() creates a ClassCastException and is obsolete anyway.

Will look at correcting this in the next release.


Randomize RAM (or set it to deterministic semi-random values) on Apu reset

from the no$ doc:

Upon power-up, APU RAM tends to contain a stable repeating 64-byte pattern: 32x00h, 32xFFh (that, for APUs with two Motorola MCM51L832F12 32Kx8 SRAM chips; other consoles may use different chips with different garbage/patterns). After Reset, the boot ROM changes [0000h..0001h]=Entrypoint, and [0002h..00EFh]=00h).

@kylophone I keep forgetting to say thanks for the info, so thanks for the info! :)


FIX: doc-string - langchain.vectorstores.dashvector.DashVector

Description: There are several mistakes in the sample code in the doc-string of DashVector class, and this pull request aims to correct them.
The correction code has been tested against latest version (at the time of creation of this pull request) of: langchain==0.0.336 dashvector==1.0.6 .
Issue: No issue is created for this.
Dependencies: No dependency is required for this change,

Twitter handle: zeyanglin

thank you @linzeyang!


is there a plan to update this?

the README says the last supported version is 0.7.2. is there a plan to update to 1.9 or eventually 2.x ?

This repo is dead. I suggest you to implement your own playback solution, it's not difficult as I've described in this article.


Get tests to build as part of pipeline

Pipeline wasn't building test project (so no tests to run when creating test pipeline).

This was getting too big. Going to submit this now and continue working on fixing more tests.

@bschnurr did you have any more comments?

Still working on getting all the tests to pass, but wanted to submit this first.


pnpm.fetchDeps: ensure consistent hashes by setting file permissions after fetching

For reasons not yet completely understood, pnpm might create dependency files with inconsistent file permissions. Since file permissions influence the resulting hash, this PR attempts a workaround by ensuring consistency after the fact.
NOTE: I am unsure that this should be merged prior to understanding what causes pnpm to generate inconsistent file permissions in the first place.
Maybe this PR already helps others facing the same issue.
More details:
We tried to build a small pnpm-based application locally versus within Github actions.
Here we observed build failures due to differing hashes for pnpmDeps. After investigation of the actual derivation contents we discovered that on a local, Ubuntu 24.04.1 LTS-based laptops with multi-user nix installs, all files in the derivation either had 444 or 555, the same derivation within the ubuntu-24.04-based runner (with single user install via nix-quick-install) had 644/755 as permissions and in different quantities.
To make things even more confusing: The ubuntu-22.04 agent runner showed the same behavior as our local environment.
Detailed statistics.

environment
permission
count

Ubuntu 24.04 LTS-based laptop/ubuntu-22.04 Github actions runner
444
27511

Ubuntu 24.04 LTS-based laptop/ubuntu-22.04 Github actions runner
555
108

ubuntu-24.04 Github actions runner
444
27497

ubuntu-24.04 Github actions runner
555
122

Since file permissions are stored in the NAR-archives used to derive the hash of a fixed output derivation, this leads to inconsistencies depending on where a derivation is built.
Hence, we ensure a consistent file permission schema:

All files with -exec suffix have 555.
All other files have 444.
All folders have 555.

This schema was chosen because it as already upheld in most environments we tested (i.e. local multi user installs on Ubuntu 24.04.1 LTS and single user install via nix-quick-install in ubuntu-22.04-based Github action runners).

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

A related bug in pnpm recently got fixed in https://github.com/pnpm/pnpm/pull/8625, and got released in pnpm 9.12.2 4 days ago.
This pnpm version bump recently got merged into nixpkgs, which made pnpmDeps hashes change, see https://github.com/NixOS/nixpkgs/pull/349093#issuecomment-2423167951.
Were you using the same pnpm version on the 22.04 and 24.04 ubuntu machines? Would you mind testing it with pnpm 9.12.2?
(The merge conflict was caused by a formatting change, nothing to worry about when rebasing)

Thanks for the hint!

Were you using the same pnpm version on the 22.04 and 24.04 ubuntu machines? Would you mind testing it with pnpm 9.12.2?

Yes, we were using the same version in all cases, but it was 9.12.1.
I will check if the inconsistency still occurs with 9.12.2 and report back.

Finally got around to check: pnpm v9.12.2 does not fix the issue, making this permissions fix still necessary to ensure compatibility between Github actions and local development.


Add error check to decipher

Description

add error check for decipher
move decipher to own package
update makefile

Issues to fix
Please link issues this PR will fix:
#75
if no relevant issue, but this will fix something important for reference
, please free to open an issue.
Reminding
Something you can do before PR to reduce time to merge

run "make build" to build the code
run "make format" to reformat the code
run "make lint" if you are using unix system
run "make test-integration" to pass all tests

LGTM, thanks a lot!


[feature] Support setting drop target or allowing draggable to be nested children.

Is your feature request related to a problem? Please describe.
When using sortable.js with two tables, drag and drop work between the tables, but the drop target for the row, is tiny if there are no elements in the second table. This is setting the parent to a . If I set the table as the parent, it thinks the entire table is the draggable item. using .draggable or .item doesn't seem to work.
Describe the solution you'd like
Would like to set a selector for the drop (e.g. the table or a div that contains the table):
new Sortable(example3Left, example3Right, {
    group: {
        name: 'shared',
        pull: 'clone' // To clone: set pull to 'clone'
    },
    animation: 150,
    dropTarget: "#container"
});

For an example markup like:

        Name
        Phone
        Is Customer?

        Foo
        404040404
        Yes

        Name
        Phone
        Is Customer?

        Foo
        404040404
        Yes

Describe alternatives you've considered
I am using emptyInsertThreshold which helps, but isn't actually what I want.

Note, using a tbody:empty css rule seems to make this better, but still not exactly what I think we need. E.g.:
tbody:empty {
  display: block;
  padding-bottom; 40px;
}


ENH: Reliability analyis, scale-dependent roughness parameters, and refactored averaging/resampling of data points

This PR implements:

Reliability analysis for scanning probe measurements (see arXiv:2106.16103)
Scale-dependent roughness parameters (also arXiv:2106.16103)
Resampling/averaging of data has been moved to SurfaceTopography

Closes #696, #687, #586, #83

Hi @mcrot - this draft PR shows you what I am presently working on.
See also https://github.com/ContactEngineering/SurfaceTopography/pull/138 for required changes on the SurfaceTopography end

This looks ok overall, but I'm just trying to analyze the surface from #724 with this branch and a worker throws this message:
[2021-09-22 18:07:30,366: ERROR/ForkPoolWorker-7] Task topobank.taskapp.tasks.perform_analysis[dc485d57-eb86-4108-a6ef-01fc9d057fcf] raised unexpected: TypeError("power_spectrum_from_profile() got an unexpected keyword argument 'kwargs'")
Traceback (most recent call last):
  File "/home/michael/work/imtek/topobank/venv/lib/python3.8/site-packages/celery/app/trace.py", line 450, in trace_task
    R = retval = fun(*args, **kwargs)
  File "/home/michael/work/imtek/topobank/venv/lib/python3.8/site-packages/celery/app/trace.py", line 731, in __protected_call__
    return self.run(*args, **kwargs)
  File "/home/michael/work/imtek/topobank/topobank/taskapp/tasks.py", line 129, in perform_analysis
    result = analysis.function.eval(subject, **kwargs)
  File "/home/michael/work/imtek/topobank/topobank/analysis/models.py", line 288, in eval
    return self.get_implementation(subject_type).eval(subject, **kwargs)
  File "/home/michael/work/imtek/topobank/topobank/analysis/models.py", line 340, in eval
    return pyfunc(subject, **kwargs)
  File "/home/michael/work/imtek/topobank/topobank/analysis/functions.py", line 576, in power_spectrum_for_surface
    return analysis_function_for_surface(surface,
  File "/home/michael/work/imtek/topobank/topobank/analysis/functions.py", line 527, in analysis_function_for_surface
    r, A = log_average(topographies, funcname_profile, unit, progress_callback=progress_callback, **kwargs)
  File "/home/michael/work/imtek/topobank/venv/lib/python3.8/site-packages/SurfaceTopography/Container/Averaging.py", line 70, in log_average
    x, y = func(reliable=reliable, resampling_method=None, **kwargs)
  File "/home/michael/work/imtek/topobank/venv/lib/python3.8/site-packages/SurfaceTopography/HeightContainer.py", line 89, in func
    return self._functions[name](self, *args, **kwargs)
TypeError: power_spectrum_from_profile() got an unexpected keyword argument 'kwargs'

I'll have to continue tomorrow, but it seems there is still some problem. Maybe just adding kwargs fixes it..

I can also check tomorrow. I've seen this today but rerunning the calculations actually worked.

I'm having a look now.

I was trying to simply pass on all keyword parameters to the analysis function: Django may be confused by this. If this does not work, reintroduce the window parameter for power spectrum analysis and all analysis methods should probably have an nb_points_per_decade parameter that is passed through to SurfaceTopography.

I think the problem is, that you've added **kwargs arguments to the new functions. Through introspection they
are saved in the database as analysis arguments and passed as extra keyword argument to the analyses:

Do we need those? They cannot be set from outside I think, because when the analysis function is called by the worker, all arguments are taken from the pickled dict in the database.

We don't need those. Let me fix this now.

Hi @mcrot - the issue should be fixed. Can you double check?

Thanks, yes I'm checking with the surface of #724, there's some problem on my side.
When that's okay, I will also merge the current develop here first before I merge it back to develop. I'll let you know.

Sounds good!


L.Marker.setIcon() in click handler shifts popup anchor

Hi
I have maker bind a popup,but I use setIcon() function ,find popup offset was changed,question at Layer.Popup.js, line:30,why not add judge.

Sorry, @BKGiser, but I don't understand the question.

var marker = new L.Marker({lat:40,lng:40});
marker.addTo(map);
marker .on("click", function (e) {
e.target.setIcon(this.overStyle);
}, context)
Click on the marker,popup window's offset will be changed,my expression may not be good.I so sorry too.

OK, I think I understand. Just changing the marker's icon moves the popup's anchor, but this only happens if the icon is changed on the click event handler.
This can be worked out by manually adding the popup to the map in the click event handler instead of using bindPopup, I guess.

I found the bug cause at Layer._popupAnchor(), it adds new offset to original, not replace!
So, if you want to change icon, just change it but don't set popupAnchor again, or you will get a new offset by (old popupAnchor + new popupAnchor)

For reference, these are currently lines 30 and 147 of Layer.Popup.js, introduced by commit https://github.com/Leaflet/Leaflet/commit/6bf0f98796abecba05943e5dc7c5c2896d611568.
Still not sure what was the original intended behaviour of that code?

@yohanboniface was this fixed by the recent popup improvements by chance?

Hi,
Looks like this bug no longer occurs in Leaflet 1.0.0-rc.2: http://plnkr.co/edit/roYxw2ts8yTufltTnS4T?p=preview
The "faulty" code has been removed indeed, which also removed the offset option of L.Popup.
Still do not know what was the objective of that option though…

Cool, thanks for checking with rc2 :)
Closing then.

I'm using 1.7.1 but I have same issue, when I use setIcon, new icon change position, if I use same comand to restore previus icon cursor return to correct position.

@nemo3000 can you please create a demo: leafletjs.com/edit.html