SQLMesh doesn't communicate impact on downstream models if a model is renamed

I think this can be repro by:

In the example project, run sql plan
In example_incremental_model.sql rename name to sqlmesh_example.example_incremental_model_renamed
Re-run sql plan

The output is this:
Summary of differences against `prod`:
├── Added Models:
│   └── sqlmesh_example.example_incremental_model_renamed
├── Removed Models:
│   └── sqlmesh_example.example_incremental_model
└── Indirectly Modified:
    └── sqlmesh_example.example_full_model

The dag is incremental -> full so renaming incremental_model breaks the dag. But because sqlmesh_example.example_full_model isn't rebuilt, there is no error thrown (even though there should be; the full model now has a broken upstream reference)

did you choose non-breaking? renaming a mode like this is a breaking change.

What do you mean by "choose non-breaking?" Where in the app do I make that choice?

I think I'm in auto mode.

@foundinblank thanks a lot for discovering this!
The reason execution fails is because even though the model with the old name was technically dropped, its name still resolves into a valid reference since the version of this removed model is till promoted on production. This happens because we backfill models before we update views for the target environment. Once you apply the change in prod, the subsequent evaluation of the full model will fail since the view name will have been removed at that point.
After some discussion we think that the best way forward would be checking whether a model that is about to be removed has any downstream references and fail if it does at the plan creation time. The only edge case we could think of where this approach fails is when a user decides to manage that table / view outside SQLMesh. But in that case it will have to be first demoted in production environment anyway.
Please let us know if you see any concerns with the suggested solution.

I think that sounds good!


config.json file Nested configuration Conversion exception problem

When I was developing the mirroring function, I found a configuration file parsing problem
When the config.json file is converted to hashtable, the keys, values and attributes will be lost
For example
The content of config.json is as follows
{
  "SCOOP_BRANCH": "master",
  "lastupdate": "2021-07-11T01:03:58.096058+08:00",
  "SCOOP_REPO": "https://github.com/lukesampson/scoop",
  "aria2-enabled": true,
  "list": "all",
  "test-map": {
    "k1":"v1",
    "k2":"v2",
    "k3":"v3"
  },
  "test-list": [
    "v1",
    "v2",
    "v3"
  ]
}

After adding this parameter

It seems that this parameter is
Supported only in 6.0  😭


Missing checksum files

I managed to get this to run - referring to the other issues. I am using python 3.11.2.
I have a problem whereby the script is ignoring the checksum files - in my case sha1 files. I added some blocks so that the checksums are also downloaded. The upload routine fails however with {"error":"Request contains an invalid argument."}. Any help?
Modified function:
def _java_artifacts(artifact_fullname, artifact_type, artifact_path, transfer_deps):
    return [
        {
            'name': artifact_fullname + '.' + artifact_type,
            'path': artifact_path,
            'target': True
        },
        {
            'name': artifact_fullname + '.pom',
            'path': artifact_path,
            'transfer_deps': transfer_deps,
            'target': False
        },
        {
            'name': artifact_fullname + '-tests.jar',
            'path': artifact_path,
            'target': False
        },
        {
            'name': artifact_fullname + '-sources.jar',
            'path': artifact_path,
            'target': False
        },
        {
            'name': artifact_fullname + '-javadoc.jar',
            'path': artifact_path,
            'target': False
        },
        ## Modifications from here
        {
            'name': artifact_fullname + '.jar.sha1',
            'path': artifact_path,
            'target': False
        },
        {
            'name': artifact_fullname + '.pom.sha1',
            'path': artifact_path,
            'target': False
        }
    ]

The reason the checksum files were not being uploaded was because they were being handled in the same way as jar files - i.e. binary multipart file uploads - as opposed to pom files - which are treated as text.


Popup UI "no button" option.

Is your feature request related to a problem? Please describe.
Alerts/Banners are limited in a few ways. First, they cannot be programmatically cleared when no longer useful. Second, they clearly aren't as flexible as the popup UI. Thus, an alternate mode offered by tools like jamfHelper is a dialog option for no buttons.
Describe the solution you'd like
A popup UI option to display no buttons.
Describe alternatives you've considered
Currently using jamfHelper to solve for my use case.
Additional context
See jamfHelper Screenshot for example use case.

Hi @Macjutsu, thanks for creating this FR. The workflow you described sounds like a background process on which the user doesn't really have control.
Pop-up UI without any button or live information doesn't really meet Apple HIG for Alerts, it give to the user no way to interact with the process and at the same time any info on what is going on in background.
IBM Notifier already provide multiple way to prompt the user with an informative pop-up UI that shows progresses or timed actions:

Determinate Progress Bar: used to show a determinate progress of what is happening in background. The pop-up will not show any destructive button until the end of the progress by default;

Indeterminate Progress Bar: used to show an indeterminate progress bar for an indeterminate process happening in background. Again the pop-up will not show any destructive button until the end of the progress by default;

https://user-images.githubusercontent.com/23209655/156996966-b07406fc-363a-4833-b66a-0f3026e792d2.mov

Timer: used to show a timed action with the relative timer. In this case the destructive buttons are showed to the user and they usually should allow the user to execute that action now or cancel it.

From your FR anyway I think that a good future feature for IBM Notifier could be a logic to clean it's user notifications from the Notification Center. If you agree I'll modify the scope of this FR to describe that logic and to track the future implementation.
Thanks!

I'll modify this FR to track the development of some logic to allow the admin to delete old IBM Notifier alerts/banners from Notification Center using IBM Notifier it self.
In the meantime for the workflow you described I suggest you to use a Pop-up UI with an indeterminate progress bar to expose to the user the "untracked" update going on in background. IBM Notifier execution doesn't interrupt the reboot.
Thanks.


Fix Web notification icon not updated once you read said notifications (go-gitea#29065)

Fix #29065
Remove status filtering from GetUIDsAndNotificationCounts sql.

Could you add some details about how it fixes the problem? TBH I haven't caught the point 🤣

I'm sorry, of course 🙂
When the number of unread notifications becomes 0, the original SQL query does not return the user's row.
As a result, there will not be a new notification-count event to update the icon. With this modification, the user's row will be included in the result with 0 count, and the event will be triggered.
The event generation :
uidCounts, err := activities_model.GetUIDsAndNotificationCounts(ctx, then, now)
if err != nil {
	log.Error("Unable to get UIDcounts: %v", err)
}
for _, uidCount := range uidCounts {
	m.SendMessage(uidCount.UserID, &Event{
		Name: "notification-count",
		Data: uidCount,
	})
}

I need to find out how to get the missing UIDs, but I am going to try.
Thanks for the idea!

Read the code again, it seems that it is impossible to figure out the non-existing IDs quickly.
So current method should be good enough. Let's try it.


Initial release

Setup pipeline with Travis to push to Bintray

version 0.1.3 released https://bintray.com/elyast/maven


bitwig-studio3: 3.1.3 -> 3.2.1

Motivation for this change
Update Bitwig Studio 3
Things done

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

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

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

@tasmo can you update with 3.2.2 and I'll merge

@Philipp-M @wmertens Thank you!


[CLOSED] Order document editions in mongoid

 Issue by james
Monday 01 December 2014 at 20:37 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/337

rather than in ruby. This dramatically speeds up listing of documents.
Note: requires a restart

james included the following code: https://github.com/alphagov/specialist-publisher/pull/337/commits

 Comment by kalleth
Monday 01 December 2014 at 20:38 GMT

:+1:


adding java folder and java code to add and subtract variables #244

Good work @ARPIT0077! Congratulations on your first open-source contribution. Keep learning and do help others contribute as well!


Incorrect Field Mapping - PAN Threat - User Field (mapped with http category - Sender)

We have observed the props enabled with comma separated defined fields and its mapped with incorrect values..
User field mapped with the value (music-low risk, private IP addressed) which is actually http category and its mapped to sender.
https://splunkbase.splunk.com/app/2757
EVAL-user = case(SourceUser!="null",'SourceUser',SourceUserName !="null",'SourceUserName',src_user!="null",'src_user',dest_user!="null",'dest_user',recipient!="null",'recipient',sender!="null",'sender',true(),"unknown")

Hi @dharmanr Could you please let me know what version of PANOS you are using?

Hi All,
We are on PANOS 10.1.5 on all of our Palos.
Thanks
Warm Regards,
Dharman.R
Director – Security Operations Center
M +91<PHONE_NUMBER>
From: Paul Nguyen @.>
Sent: Wednesday, June 7, 2023 10:06 AM
To: PaloAltoNetworks/Splunk-Apps @.>
Cc: Dharman R @.>; Mention @.>
Subject: [EXTERNAL] : Re: [PaloAltoNetworks/Splunk-Apps] Incorrect Field Mapping - PAN Threat - User Field (mapped with http category - Sender) (Issue #297)
CAUTION: This email originated from outside of the organisation. Do not click links or open attachments unless you recognise the sender and know the content is safe.
Hi @dharmanrhttps://github.com/dharmanr Could you please let me know what version of PANOS you are using?
—
Reply to this email directly, view it on GitHubhttps://github.com/PaloAltoNetworks/Splunk-Apps/issues/297#issuecomment-1579869086, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BACWXVNYRG4H6REQST6VVX3XKAALBANCNFSM6AAAAAAYOHNQEI.
You are receiving this because you were mentioned.Message ID: @.***>


stacked bar chart implementation

Solution description
Collaborated with the front end team to convert our existing data visualization from a line graph to a bar chart.
Covered unit test cases
Were unit test cases recorded for this fix?
Not yet.
Code is clear and maintainable:

[x] My code follows the style guidelines of this project
[x] My code is consistently formatted.
[x] No instance on duplicate code.

Tested this issue fix on all browser
[x] Chrome
[x] Safari
[x] Internet Explorer
[x] Firefox
Keep Pull Request Simple
Pull Request is relatively small. It does not contain multiple changes or solve multiple problems.
Reviewed by Team members
Pull Request is reviewed by one or more team members
Idiomatic Code
Code follows industry standards for language, framework, and libraries, using ESLint and Gulp to meet suggested guidelines.

Thanks for the assistance, gentlemen.


Use DiagnosticSource and Microsoft.Extensions.Logging.Abstractions

Microsoft.Extensions.Logging

[x] Migrate to Microsoft.Extensions.Logging
[x] Adapt Logger Providers for MT
[x] Migrate from Log4Net to Serilog
[ ] Omit static Loggers

I'm going to rework this, and I've started a branch where most of the heavy lifting is done.

if you need any help, just provide me a branch you are working on

the sample you've provided, you want to use such approach or don't?
I do not see any value of using Current approach, Activity is always nesting if there is a parent, same for logger. It seems to me as Anti-pattern (ServiceLocator). I really like how EF Core done that:
https://github.com/aspnet/EntityFrameworkCore/blob/master/src/EFCore/Diagnostics/IDiagnosticsLogger.cs

That's basically what ILogContext is, it has the methods for logging to ILogger and DiagSource.
There are places in the code where injecting an ILogContext is just not feasible, which is where the .Current approach is used. It also keeps the flow cleaner, for things like configuration, which previous used many different categoryNames (class name) for log messages - making it impossible to really filter with any level of usefulness.

hm, I have never seen it. Could you please provide me a link where I can read more about it?

I created it, so once I am happy with how I've put it together, I'll publish a branch/PR for review.


chromedp-stop headless server

i am using chormdp but i am face  issue.
after the screenshot i went to stop  chormedp headless server and  start again on new request
I start chormedp server on api hit when i hit my api again new data is not appear
Can someone help me on it
Thanks

Please provide an minimal runnable demo which can reproduce your issue.

@ZekeLu in below files you can find a code and the postman you can see if you change api data and hit again and again the picture save in local is save with different names because i save a new picture on new hit but the data is appear in picture is the data of first API hit
hello-world.zip
headless.json.zip

@afzalabbasi I haven't run your code, but I guess here is the problem:
go testServer(fmt.Sprintf(":%d", *flagPort),src)

You start an http server on every request with the same port number, so just the first one starts successfully (the rest fail because the port number has already been taken). All the requests are served by the first server with the same response, that's why the data in the screenshot are always the same.
It's too heavy to generate images like this. You should try something else.

@ZekeLu  i always open a new connection with same port
is it possible i just stop  server after screen-short is taken?

@afzalabbasi You are making simple things complicated. Please don't do that!
If you insist on using chromedp to convert html into image, you could at least take out those extra http servers.
There are at least tow ways to avoid those extra http servers:

use the server that listens on port 8083 to generate html from template.
use fetch.FulfillRequest to provide html to the headelss browser directly.

@ZekeLu  thanks  i will close the port at my end after successfully screen-short taken


chore: Copying world state binary to yarn project is on generate

We only build tsc on build, and use the generate step for moving build artifacts and generate code.

Ah, I think this can also go now that generate gets run by both bootstrap full and fast
https://github.com/AztecProtocol/aztec-packages/pull/9252


Tweak Alerts

As of today, I added the new component of alerts. Right now, it is a very rough component and can use a lot of work, in both the actual code of them and the design. It would be great if someone could take this over.
You can find the alerts component file in src/components/alerts.scss
Thanks!

I'll take this. @cpdevelops can you assign this to me? Secondly, is the alert a popup or inline? I do not see it added to index.html (only a scss file exists)

Thank you for wanting to take this over. I will go ahead and assign it to you.
It is an inline alert, and I attempted to add it to the demo site but there were too many merge conflicts so I decided to hold off on that.
Some thing to note would be a JS based close button would be great, and you can take any creative freedom you want wether its the color of the entire layout.
Thanks for starting to contribute and happy coding.

✅  Please refer https://github.com/cpdevelops/simple.css/pull/25

Thanks for the PR @bhagatparwinder. Closing this issue now


Create exampleis-not-a.dev

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description

Link to Website

git push -u origin patch-1


Use new SwiftPM flag to remove $ORIGIN from installed sourcekit-lsp ELF executable runpath

Also, move installation-only environment variable into get_swiftpm_environment_variables().
One of the latest trunk snapshot toolchains shows five executables built by SwiftPM that incorrectly have this runpath, including sourcekit-lsp from this repo:
> readelf -d swift-DEVELOPMENT-SNAPSHOT-2023-10-10-a-ubuntu20.04/usr/bin/*|ag "File:|runpath"|ag "ORIGIN[^/]" -B1
File: swift-DEVELOPMENT-SNAPSHOT-2023-10-10-a-ubuntu20.04/usr/bin/docc
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN:$ORIGIN/../lib/swift/linux]
File: swift-DEVELOPMENT-SNAPSHOT-2023-10-10-a-ubuntu20.04/usr/bin/sourcekit-lsp
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN:$ORIGIN/../lib/swift/linux]
File: swift-DEVELOPMENT-SNAPSHOT-2023-10-10-a-ubuntu20.04/usr/bin/swift-build-sdk-interfaces
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN:$ORIGIN/../lib/swift/linux]
File: swift-DEVELOPMENT-SNAPSHOT-2023-10-10-a-ubuntu20.04/usr/bin/swift-driver
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN:$ORIGIN/../lib/swift/linux]
File: swift-DEVELOPMENT-SNAPSHOT-2023-10-10-a-ubuntu20.04/usr/bin/swift-help
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN:$ORIGIN/../lib/swift/linux]

This will remove that unneeded runpath, as I just did for swift-package in apple/swift-package-manager#6947. I limited this to non-Darwin installs because I have no idea if this would be useful on Darwin too.

Ping, a CI run and this can go in? I'll submit for 5.10 next.

Ready for merge, will submit for 5.10 once this is in.


Cannot use snapshots without enabling the 'Saved Game' feature in the Play console

After setting everything as per the documents, I am always getting Cannot use snapshots without enabling the 'Saved Game' feature in the Play console  error when i try to open a snapshot.
Third step of 'Set up Play Games Services' on console always stays unchecked even tho i have added id and metadata in Manifest.
Setting screen: https://ibb.co/x2r245C
I have also enabled the 'Saved Game' option in Configurations screen.

please reopen if you followed the suggestion from comment above and it didn't resolved the issue.


zig 0.10.0 and osx

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

it seems that the error 137 is still a problem when compiling zig here :(
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=624995&view=logs&j=656edd35-690f-5c53-9ba3-09c10d0bea97&t=e5c8ab1d-8ff9-5cae-b332-e15ae582ed2d&l=425

I just saw there's a PR opened for the same thing. Sorry about that.

NP, thanks for contributing there :)
actually, after your PR I decided to check about that with the zig community and it seems that a PR that aimed to resolved this problem was merged. so I am checking the next steps about that: https://github.com/conda-forge/zig-feedstock/pull/5#issuecomment-1342909194


Revert "configs: Google example remove dns_lookup_family (#2844)"

This reverts commit 2c78cde82a85a2630d51f9a6c42e872209ee5bea.
This is still an issue per https://github.com/envoyproxy/envoy/issues/2853

@zuercher


spec: requiring identical tags for struct identity seems overly restrictive

The rules for struct identity require that two identical structs have identical tags.
This plays into the rules for assignments and conversions of structs in unfortunate ways:
There appears to be no way to assign or even convert a struct with tags to another struct with different or no tags, wether that other struct is named or unnamed:
http://play.golang.org/p/6FZcWSIc9w
I think we're overly restrictive here. For instance, if I wanted to convert a struct with json tags to a struct with tags for some other encoding (to convert from one encoding to another), I actually have to copy all the fields by hand.
This seems counter-intuitive. I would have expected that I could go via an unnamed struct, or at the very least use an explicit conversion.
I'm thinking that perhaps we should relax the rules here. One might be to remove again the requirement of identical tags for struct identity (simplest); or perhaps to relax the rules for assignments and conversions of structs.

Feedback from @ianlancetaylor (copied from e-mail):
I tend to agree.  FYI, the requirement that struct tags be identical
was added here: https://codereview.appspot.com/1698043 .  It seems to
me that this doesn't make much difference in practice.  You already
can not assign or implicitly convert between named struct types.
Unnamed struct types are rarely used, so I don't see them as an
issue.  If we reverted this CL, I think it would be possible to
explicitly convert between struct types as long as the fields had the
same names and types, because the underlying types would be identical.

PS: I vaguely recall that I voted for this change. But I also don't remember the details. It looks like a case of unintended consequences. Rolling back the change would be a backward-compatible change.

Duplicate of #6858.


Fixing typo in path /dev/null

Which issue this PR addresses:

N/A - informal
What this PR does / why we need it:

There is a typo in the paths, which results in a no such file or directory error that will always exit 1 and make these  two  IF statements always fail.
Test plan for issue:

I discovered this bug after following the deploy an int-like env. This PR is a partial fix to getting that to work.
Is there any documentation that needs to be updated for this PR?

I'm unsure, but I doubt it.

/azp run e2e

/azp run ci

/azp run ci

/azp run e2e

/azp run e2e

/azp run e2e


Updates karma to prevent cancelling tests because of timeout

This will set the DEFAULT_TIMEOUT_INTERVAL of jasmine up to 10000ms. Hopping, that this won't be the reason for failing tests anymore.

worth a try I suppose. I will restart the client testing a few times and we'll see whats happening


check config for defaults

What's in this PR?
Should address the comment on this thread: https://github.com/html-next/ember-hammertime/issues/31#issuecomment-355735187
I ran into this issue as well.  The README implies that by adding the EmberHammertime hash to your config/environment you can set defaults. However, the mixin does not look at the consuming app's config and overwrite the defaults.  This PR would allow the user to specify which defaults they want.

This LGTM! 🚀

@sbatson5 the config stuff should work. What issues are you experiencing?

Ah, I guess the issue is the config values work when you don't use the mixin, but the mixin does not?

the config stuff should work. What issues are you experiencing?

I think he wants to configure it from environment.js from within the app. If that's not it then not sure either...

I think this should work. Going to merge.

Thanks @sbatson5!

ps @sbatson5 - double thanks never hurts - nice job fixing this issue - its been outstanding for a good long while, appreciate it


Please get rid of the need to supply a mobile phone number when registering

Instead, have the number be sent to the email address that the user has specified.

They won't. It's presumably to stop people from mass creating accounts for use in botting or whatever.
Sure, it sucks for those of us without a phone number we're able to use, but thats life.

They can use captcha or something.

I agree. Phone number shouldn't be needed IMO. They can have different way to prevent bots.

This seems to have been fixed by a PR that implements CAPTCHA in place of phone verification (which has been merged). Now to wait for a new release.

Captcha verification is now live on web, will roll out to mobile later in the week.


u256 add operator

Description
This PR is part of https://github.com/FuelLabs/sway/pull/4794. It implements support for u256 add operators.
The IR generated is a vanilla add operator as any other integer.
script {
    entry fn main() -> u256, !1 {
        entry():
        v0 = const u256 0x0000000000000000000000000000000000000000000000000000000000000001, !2
        v1 = const u256 0x0000000000000000000000000000000000000000000000000000000000000002, !3
        v2 = call add_0(v0, v1), !4
        ret u256 v2
    }
}

This will then be transformed by miscdemotion pass to something like:
script {
    entry fn main() -> u256 {
         local u256 __wide_lhs
         local mut u256 __wide_result
         local u256 __wide_rhs

        entry():
         v0 = get_local ptr u256, __wide_lhs, !0
         v1 = const u256 0x0000000000000000000000000000000000000000000000000000000000000001, !0
         store v1 to v0, !0
         v2 = get_local ptr u256, __wide_rhs, !0
         v3 = const u256 0x0000000000000000000000000000000000000000000000000000000000000002, !0
         store v3 to v2, !0
         v4 = get_local ptr u256, __wide_result, !0
         wide add v0, v2 to v4, !0
         v5 = load v4, !0
         ret u256 v5
    }
}

Mind the wide add here. It is a Fuel specific operation, giving space to other targets to implement this differently.
Checklist

[x] I have linked to any relevant issues.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

A unit test framework already exists to test this, in sway-ir/tests/demote_misc. Any reason you aren't using that?

Never realised those tests exist. Moved everything there.


Add go and java pipeline for +Add flows

This adds pipelines for go and java based source.
Pipeline consist of build and deploy tasks
required for +Add flows in console
Signed-off-by: Pradeep Kumar<EMAIL_ADDRESS>

@nikhil-thomas @sthaha
This is an initial PR from a series to add pipeline template for most used languages.
Please let me know if we the directory structures and the version is handled properly.

/ok-to-test

/test e2e


Should pump calibration have near 0 intercept?

Put a prior on it
Just drop a bias from the linear regression

I mean, it should have a 0 intercept. I know that if I run the pump for 0 seconds, no volume is moved.

Yes, we force a zero intercept by design using simple_linear_regression_with_forced_nil_intercept.


How to deal with "name '_C' is not defined" on Windows?

A wonderful job! It's a pity that I haven't tried it yet, and I'm asking for your help.
I have looked through the relevent issues for this problem, but all of them are on Linux
(e.g.,
export CUDA_HOME=/path/to/cuda-11.3/
pip install ...
)
I want to know how it can be solved on Windows? I really appreciate this job, and I really want to try it on my device!!!!
Looking forward to your reply, thanks!

Hello, we did not test grounded-sam on Windows device before, maybe the community can help with this question I guess

Thanks for your reply! But I meet another problem...  Actually, I don't find the similar issue here，I want to know if it's because windows or something else:
python -m pip install -e GroundingDINO
Then I see:
Installing build dependencies ... /
I wait a lot of time and it gives me :
pip._vendor.urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='files.pythonhosted.org', port=443): Read timed out. This error originates from a subprocess, and is likely not a problem with pip.

LOL. I used your code python -m pip install -e GroundingDINO. It works, but rises another error following:
AssertionError: Torch not compiled with CUDA enabled

Thanks for your reply! But I meet another problem... Actually, I don't find the similar issue here，I want to know if it's because windows or something else:
python -m pip install -e GroundingDINO
Then I see: Installing build dependencies ... / I wait a lot of time and it gives me : pip._vendor.urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='files.pythonhosted.org', port=443): Read timed out. This error originates from a subprocess, and is likely not a problem with pip.
did you fix this error?

Thanks for your reply! But I meet another problem... Actually, I don't find the similar issue here，I want to know if it's because windows or something else:
python -m pip install -e GroundingDINO
Then I see: Installing build dependencies ... / I wait a lot of time and it gives me : pip._vendor.urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='files.pythonhosted.org', port=443): Read timed out. This error originates from a subprocess, and is likely not a problem with pip.

have you fixed this error?

I have fixed this error. It is quite simple, and what you need to do is just copy the groundingdino folder and past it to the same folder in Grounded-SAM. groundingdino


How stable is Sanic? Can we consider it as at least beta?

How stable is Sanic now? Is there serious bugs or plans to drastically change API?
Current status is Development Status :: 2 - Pre-Alpha, maybe it's a time to change it to at least Development Status :: 4 - Beta?

Closing this since we've updated it to beta status.


Disable back botton with one account

I think that the screen with the conversation list should be the 'main' screen. Especially when you have only one chat account, the back button at the left upper corner is not really useful because the position of the button suggest that it is NOT the 'main' screen.
My suggestion is to disable the back button on the conversation screen (the Android back button go back to the prev. screen) when you have only one account, because you can always add one via the settings menu.

was this fixed? when I click ← from the conversation screen, I get to the android home screen


OFA-CN pre-train dataset

Dear Author,
I am trying to pre-train the Chinese OFA model from scratch. Could you please tell me what dataset you used to pre-train the OFA-CN model? Is the Chinese dataset available? Looking forward to your answer.
Best wishes.

We use very similar multimodal dataset (mostly publicly available) to that in our recent work Chinese CLIP https://arxiv.org/abs/2211.01335. Use it for reference. For the plaintext, we use the internal dataset from M6 https://arxiv.org/abs/2103.00823, and thus I advise you to use other plain text datasets of similar sizes as alternatives.


[2277] fix: remove unneeded login step on workflow action

List of changes

Fix: failing github action

Checklist

related issue
[ ] My changes generate no new warnings
[ ] My code follows the style guidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the changelog
[ ] I have added tests that prove my fix is effective or that my feature works

dont merge, im testing the via another repo


ARCHIVED

DuckEngine is now discontinued. This was a fun side project to work on in which I have learned a lot along the way. I plan to make a new 2D game engine for the web called Forge2D.
Thanks

unarchieved


Typescript questions

View
A view is a function but the declaration is an interface. Why?
https://github.com/hyperapp/hyperapp/blob/master/hyperapp.d.ts#L73-L79
I'd expect it to like a app. What am I missing?
https://github.com/hyperapp/hyperapp/blob/master/hyperapp.d.ts#L90
Extra char
The prefixed pipe char seems strange and confused me. I see it gives nice alignment but I think should be dropped
https://github.com/hyperapp/hyperapp/blob/master/hyperapp.d.ts#L69
Components
The readme says a component only takes props and not wired to actions and state but the test example takes actions and state as props. This is confusing to say the least. I suggest the example is changed, count, up and down only! as props and not passing the global objects in.
https://github.com/hyperapp/hyperapp/blob/master/test/ts/Component.tsx#L12
You obviously need to use State and Props as a lazy component can return a View and a View needs the global State and Props.
Would it be better to have separate Component and LazyComponent?

View
OK so interface is equivalent to type. at least in this context.
I propose changing to type would be clearer - but might just be me :)

@SteveALee Perhaps a PR would be a better way to discuss the types and changes you are proposing? 🤔

@jorgebucaran Your are quite right! :) It sort of expanded as I was writing it.

@jorgebucaran see #739

@SteveALee Okay, let's close here then?

Thanks! 👍


Ensure Blackfire support is functional

Ensure Blackfire support is functional

Verified in pending https://github.com/svpernova09/EnterpriseSettler/releases


contour-next does not work

Hi,
if i call,
src/contour-next-usb/contour_next
i get no output,
server01 contour-next # src/contour-next-usb/contour_next

server01 contour-next #

but i get this issue in the kernel log:
[Sa Jun  9 16:26:46 2018] hid-generic 0003:1A79:7410.000B: usb_submit_urb(out) failed: -28
Any suggestions how to fix this?

Hi,
I didn't saw this error before. As far as i know usb_submit_urb(out) is defined within the kernel. From the kernel sources i know that "-28" means ENOSPC (include/uapi/asm-generic/errno-base.h). From the information found on https://mchehab.fedorapeople.org/kernel_docs/driver-api/usb/error-codes.html i assume that the error has something to do with our device. But i'm not sure.
Kind regards


Analysys
newPartitionRT
definition
go tool pprof -http :8088 -focus "newPartitionRT" heap.prof

Decision
### Tasks
- [ ] `appparts` should allocate `*partitionRT` with [pool](https://pkg.go.dev/sync#Pool) and return it to pool from `*partitionRT.Release()`

Results
Top heap objects
Top heap space
newPartitionRT is escaping from the list of the first 246 memory allocators


Handled sent_to_provider_at being nil

Context
This should only be happening in the Support Console
Raises Sentry error https://dfe-teacher-services.sentry.io/issues/5678398911/?referrer=slack&alert_rule_id=853774&alert_type=issue
Changes proposed in this pull request

Added safety operator to handle nil values

Guidance to review
N/A
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[ ] Inform data insights team due to database changes
[ ] Make sure all information from the Trello card is in here
[ ] Rebased main
[ ] Cleaned commit history
[ ] Tested by running locally
[ ] Add PR link to Trello card

 Sent to provider at can be nil


Upgrade test v2

Aim: Change the conceptual picture of upgrade tests to be simpler to understand.

Make the core of the test a method for performing a preview from a different (baseline) configuration.
Change the core concept of "recording directory" to be an optional cache.

The main method is PreviewUpgrade which has two steps:

SnapshotUp performs an "up" using the baselineProviderConfig and captures the resulting state.
PreviewUpdateFromSnapshot imports the state from the snapshot and performs a "preview" with the original pulumitest settings.

If a CacheDir is configured:

If there is a stack.json then SnapshotUp will be skipped and the cache used instead for running PreviewUpdateFromSnapshot.
If there is not yet a stack.json then SnapshotUp will be run and its result stored to the cache before running PreviewUpdateFromSnapshot.

TODO:

[ ] Expose all additional options when running the preview.
[ ] Helper to fix non-hardcoded stack names.
[ ] Use proper functional options
[ ] Nit: Make stack name customisable.

Superseded by https://github.com/pulumi/providertest/pull/51


Add internal REST proxy to the node

Allow transporting REST endpoint requests via the CAF protocol, indepentend of the web plugin.
There are alternatives options in how deep we want to integrate this into the node:

We could funnel all requests (ie. also those coming from the web plugin) through this handler in the node, and get rid of the need to spawn handlers ad-hoc at the price of one additional routing indirection.
We could also decouple this a bit more by running this internal proxy as a separate component in the node.

I went somewhere in the middle with this PR, but I don't really have good arguments either way.

Obsoleted by recent architecture changes; successor PR is #3073


Avoid rendering same slide vnode twice for small amount of slides in loop + virtual mode

This fix adds one extra logical check in order to avoid pushing more elements to slidesToRender that we have slides, i.e. we avoid duplicates.
Refers to #7557
Cases with 4+ slides work as before

Merged, thanks!


[No QA] Update expensify-common hash

@ctkochan22 since you have the expensify.cash one
Hash bump for https://github.com/Expensify/expensify-common/pull/360

Looks good. Didn't do anything weird in the lock file. @AndrewGable to you

Going to self merge since @AndrewGable was only added because pullerbear is marked as a code owner


Simplify simulate, cleanup after first MR

Also, the behaviour with the bundled example is exactly the same as before.

:+1: Merged.


Adds API only mode support for rswag-ui

Fixes this
#30

The underlying issue should be now be fixed by #121 (available in version 2.0.0 and above). Please give it a go and let me know if it works


Events: execution order is theoretically not defined

Event code uses for to iterate over object properties, like seen here: https://github.com/Leaflet/Leaflet/blob/master/src/core/Events.js#L194
This should work, but browser vendors could change the iteration order anytime. Also numerical properties are treated differently apparently: http://stackoverflow.com/a/280861/342947

What's a failing use case for this? Can't really see when this can become a problem.

It probably won't ever fail, as long as there are no event names which start with numbers.
It's technically not correct though to rely on Javascript property iteration order, because it's not defined. Chrome could start iterating properties from behind (which it won't do). If you want predictable iteration you are required to use arrays.

@fab1an Now that #4697 has been merged into master, is this still an issue?


projects

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other: <!-- Please describe: -->

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

SPAM: This fork adds an advertisement for Mr. Singh's services and an about page about himself.  Totally spam and self-serving.
I hope no one foolishly accepts this PR.


Add more function to multiworld client GUI

#2
This largely keeps the same gui files, but adds 2 buttons, one to join rooms and one to disconnect rooms. The Set-Up button now does nothing since room creation is not actually supported server-side yet.
The relevant fields are automatically populated on load with values from the current Config.
Pressing the 'Join' button will create a new thread that will join the server, and make a 'Disconnect' button show up. This disconnect button will request an interruption which will be caught by the main client loop in clientCommunication.client(), then properly disconnecting and ending the thread. The same will happen when the user attempts to close the window when connected.
There is also now a Qt Signal passed down from the main gui to the connection classes. Emitting this signal with a string will add said string to the QListWidget on the main gui, displaying a message to the user. This can be made a lot cleaner, but for now it works.
Note that this still is not loaded by main.py, it still is currently only implemented by testing.py. It simply is closer to being ready to be properly impmlemented into main.py.

All requested changes implemented


[BETA 3] Crash involving smelter vent facing

I was making a platform of stone bricks to place an infernal furnace. The nearby essentia smelter (with vents) was above me. When I looked up at the smelter the game crashed, with the following log:
https://pastebin.com/gcqJ0a3q

This is actually a crash with portablejim.bbw.core.WandWorker which is part of a mod which I am unfamiliar with.

That must be Better Builder's Wands. I can report it to portablejim.


Update links and remove the link to the Beginners' Guide

ref: https://wiki.archlinux.org/index.php?title=Talk:Beginners'_guide&oldid=443725#The_Great_Merge

@kyechou Good job! Thank you! 🙇


old warnings, release?

I'm getting warnings concerning fixes that are 16 months old:
https://github.com/fredrik-johansson/mpmath/commit/4990fb5cf3447b956e87bf060420a296b13cedb1#diff-550b6d03e9982886a0afe0a8e1e4010d
Perhaps a release is in order?

Yes. I'll try to find some time for working on mpmath.

I'd probably just try to get a shorter release pipeline. You already got tests in order, which is is great. If you get an idea of the coverage and if that's reasonably high, this can give you some confidency to rero.

1.2.1 released, https://pypi.org/project/mpmath/#history.


[Cosmos]: add Stride

Description
Similar to #3053 - adding stride
How to test
Run C++ unit tests, swift and kotlin

Just one suggestion: How about adding a test to TWAnyAddressTests.cpp with:
EXPECT_TRUE(TWAnyAddressIsValid(STRING("stride1gckvjxau7k56f8wg8c8xj80khyp83y8x8eqc94").get(), TWCoinTypeStride));
EXPECT_EQ(TWAnyAddressCreateWithPublicKeyDerivation(privKey, TWCoinTypeStride), "stride1gckvjxau7k56f8wg8c8xj80khyp83y8x8eqc94");

as soon we move cosmos to rust, let's have macro to gen such test for bech32 - will have a better overview after the transition


Contact Holding Fields Not Populating

When resource is created from form the following fields are not populating as expected - Submitter Email Holding, Submitter First Name Holding and Submitter Last Name Holding.

Added back access to those field on the Guest Site User assigned profile - Sprinty's Community Resources Profile - https://sprintyscommunityresources.my.salesforce.com/00e4w000001dlk1AAA.


Create VensStylePPTextures.ckan

It looks like you've created a great netkan file to collect all the info from Spacedock. If you submit that file instead, new versions will be automatically added to CKAN. .ckan files don't work in this repository.


[CLOSED] The executor CLI

 Issue by akutz
Wednesday Mar 23, 2016 at 16:07 GMT
Originally opened as https://github.com/emccode/libstorage/pull/15

This patch introduces the executor CLI that is built for each driver. To build executors use the following Make targets:

Target
Description

build-executor
builds the executor with the default driver (mock) for the system $GOOS_$GOARCH

build-executors
builds the executors for all drivers for the system $GOOS_$GOARCH

build-executors-all
builds the executors for all drivers for all $GOOS_$GOARCH combinations

build-executor-$GOOS_$GOARCH
builds the executor with the default driver (mock) for the system $GOOS_$GOARCH

It's also possible to build an executor for a non-default driver and/or non-system $GOOS_$GOARCH combination without resorting to the multi-build targets. Invoke the build-executor or build-executor-$GOOS_$GOARCH targets specifying different go tags. For example:
env GO_TAGS='ec2 driver' make build-executor

The above command will build the executor for the ec2 driver for the system $GOOS_$GOARCH. To build the ec2 driver for a 64-bit Linux system, the command would be:
env GOOS=linux GOARCH=amd64 GO_TAGS='ec2 driver' make build-executor

akutz included the following code: https://github.com/emccode/libstorage/pull/15/commits

 Comment by akutz
Wednesday Mar 23, 2016 at 16:09 GMT

Hi @vladimirvivien, @Oskoss,
I would like you each to take a brief-to-however-detailed-look-you-want at this for feedback. Thanks!
@clintonskitson, you're welcome to as well of course, but I'm trying to not throw too many things like this on your plate as we lead up to the event that shall not be named :)

 Comment by akutz
Wednesday Mar 23, 2016 at 16:10 GMT

Also, I realize the executor CLI isn't yet complete. It doesn't handle getting local devices or the next device ID. However, I wanted to show you the idea.

 Comment by codecov-io
Wednesday Mar 23, 2016 at 16:15 GMT

Current coverage is 36.44%

Merging #15 into master will not affect coverage as of 61f2880

@@            master     #15   diff @@
======================================
  Files            7       7
  Stmts          354     354
  Branches        12      12
  Methods          0       0
======================================
  Hit            129     129
  Partial         12      12
  Missed         213     213

Review entire Coverage Diff as of 61f2880

Powered by Codecov. Updated on successful CI builds.

 Comment by vladimirvivien
Wednesday Mar 23, 2016 at 17:31 GMT

@akutz what's the Make option for the build ?  It looks like you are passing go tags to make and gomk handling multi-arc builds, no ?

 Comment by akutz
Wednesday Mar 23, 2016 at 16:42 GMT

Hi All,
I've updated the PR commit with a more complete executor CLI -- it now expects one of three commands:

instanceID
nextDevice
localDevices

$ ./libstorage-mock-executor
usage: libstorage-mock-executor instanceID|nextDevice|localDevices

$ ./libstorage-mock-executor instanceid
{"id":"mockDriver1-InstanceID","metadata":{"max":10,"min":0,"rad":"cool","totally":"tubular"}}

 Comment by akutz
Thursday Mar 24, 2016 at 18:55 GMT

Hi All,
This PR has been updated to be even more useful. Now when executing simply make all executors for all drivers and all GOOS_GOARCH combos are built, then embedded into the server via ./api/server/executors/executors_generated.go and go-bindata. Give it a spin.

 Comment by akutz
Wednesday Mar 23, 2016 at 17:43 GMT

Hi @vladimirvivien,
I'm not sure I understand the first question. Are you asking what are all the Makefile targets? You can use bash completion (brew install bash-completion) and then type make <TAB> <TAB> to have it list available targets:
$ make
Makefile                         dist                             install                          libstorage_run_test.go-cyclo
all                              dist-all                         libstorage                       libstorage_run_test.go-fmt
api                              dist-clean                       libstorage-clean                 libstorage_run_test.go-lint
api-clean                        dist-darwin_amd64                libstorage-test                  libstorage_run_test.go-vet
build                            dist-linux_amd64                 libstorage.go                    libstorage_test.go
build-all                        drivers                          libstorage.go-cyclo              libstorage_test.go-cyclo
build-darwin_amd64               drivers-clean                    libstorage.go-fmt                libstorage_test.go-fmt
build-executor                   glide.lock                       libstorage.go-lint               libstorage_test.go-lint
build-executors                  glide.lock-clean                 libstorage.go-vet                libstorage_test.go-vet
build-executors-all              glide.yaml                       libstorage.json                  libstorage_tests_test.go
build-executors-darwin_amd64     gnix-cat                         libstorage.test                  libstorage_tests_test.go-cyclo
build-executors-linux_amd64      gnix-cp                          libstorage.test-clean            libstorage_tests_test.go-fmt
build-linux_amd64                gnix-curl                        libstorage_mock_test.go          libstorage_tests_test.go-lint
clean                            gnix-date                        libstorage_mock_test.go-cyclo    libstorage_tests_test.go-vet
clean-all                        gnix-env                         libstorage_mock_test.go-fmt      run
clean-darwin_amd64               gnix-grep                        libstorage_mock_test.go-lint     run-debug
clean-linux_amd64                gnix-mkdir                       libstorage_mock_test.go-vet      run-tls
cli                              gnix-mv                          libstorage_nomock_test.go        run-tls-debug
cli-clean                        gnix-printf                      libstorage_nomock_test.go-cyclo  test
clobber                          gnix-rm                          libstorage_nomock_test.go-fmt    test-build
cover                            gnix-sed                         libstorage_nomock_test.go-lint   test-clean
cover-clean                      gnix-tar                         libstorage_nomock_test.go-vet    vendor-clean
deps                             gnix-touch                       libstorage_run_test.go           version

As far as options, check out .gomk/config.mk for a list of configuration variables, their descriptions, and default values. For example:
# the build platforms for which to perform the cross-* goals
BUILD_PLATFORMS ?= Linux-x86_64 Darwin-x86_64

# an ordered, space-delimited list of the go package directories to build and
# install. the root package should be a '.', and all the remaining packages
# should be in the form of './subdir1', './subdir1/subdir1a', './subdir2'
#
# if this variable is empty then it will be populated automatically by using
# the find command to find all directories not beginning with a leading '.' or
# '_' and named 'vendor' or 'doc'.
GO_PKG_DIRS ?=

# an ordered, space-delimited list of directory patterns that should be ignored.
# for more information on the pattern matching scheme used, search for help on
# the makefile text functions, specifically the filter-out function.
GO_PKG_DIRS_IGNORE_PATTS ?= ./vendor% ./docs%

The only exception to this is that the variable DRIVERS is defined at the top of the root-level Makefile:
# the space-delimited list of drivers for which to build the libstorage
# server, client(s), and executor(s)
DRIVERS := mock

 Comment by akutz
Thursday Mar 24, 2016 at 23:14 GMT

Hi All,
There is no more time for review, so I'll just have to fix any issues that linger. Merging.


UpdateRemotePlugins command hangs

Hi, this looks very promising, however after I followed the instructions, I cannot find any evidence that it is working.
I have installed on Ubuntu using PPA,
NVIM 0.0.0-alpha (compiled Sep 28 2015 21:21:49)
Commit: ?
Build type: RelWithDebInfo
Compilation: /usr/bin/x86_64-linux-gnu-gcc -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -Wvla -std=gnu99 -fstack-protector --param ssp-buffer-size=4 -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/build/config -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/src -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/.deps/usr/include -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/.deps/usr/include -I/usr/include/luajit-2.0 -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/.deps/usr/include -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/.deps/usr/include -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/.deps/usr/include -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/.deps/usr/include -I/usr/include -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/build/src/nvim/auto -I/build/neovim-CFa3dS/neovim-0.0.0+git201509280033+1917+14~ubuntu14.04.1/build/include
Compiled by buildd@lgw01-50
I initally used Vundle to clone the node-host repo, then placed the example as example.js in .nvim/rplugin/node/example.js
When I executed the UpdateRemotePlugins command, nvim just hangs.
Any ideas why that happens? I did see an error in .nvimlog, but it may just be because I killed the terminal.. stock Gnome Terminal 3.6.2, btw.
2015/09/29 21:26:29 [info @ main_loop:559] 3047 - Starting Neovim main loop.
2015/09/29 21:26:53 [info @ main_loop:559] 3130 - Starting Neovim main loop.
2015/09/29 21:27:17 [info @ main_loop:559] 3183 - Starting Neovim main loop.
2015/09/29 21:28:15 [error @ call_set_error:743] 3183 - msgpack-rpc: Channel was closed by the client
2015/09/29 21:29:07 [info @ main_loop:559] 3310 - Starting Neovim main loop.
2015/09/29 21:30:20 [info @ main_loop:559] 3474 - Starting Neovim main loop.
2015/09/29 21:30:56 [error @ call_set_error:743] 3474 - msgpack-rpc: Channel was closed by the client
2015/09/29 21:32:16 [info @ main_loop:559] 3712 - Starting Neovim main loop.

@bigfish the problem is that this package depends on an updated version of the client that hasn't yet been published to NPM; sorry for the trouble. This should be addressed soon enough, but if you don't want to wait, in package.json change
-    "neovim-client": "^1.0.4",
+   "neovim-client": "neovim/node-client",

and then npm install.
@tarruda would you mind publishing [neovim/node-client] 1.0.5?

@nhynes published
Can I add you as a owner of the package? What is your npm username?

@tarruda Fantastic, thanks a lot. I'd be happy to help maintain the npm package--my username is "nhynes."

Ok, added you as owner

It works now, thanks for the quick fix, @nhynes !


Modal-on-Modal breaks scrolling of first modal

When we show a modal-on-modal and close the later modal, the first modal will not be able to scoll anymore. This is a known bootstrap weakness. See here:
https://stackoverflow.com/questions/28077066/bootstrap-modal-issue-scrolling-gets-disabled
We can reproduce this situation for example with

Open the "Create a new action item" dialog

Reduce browser window height that not the whole dialog fits onto the screen

Now enter a free-text responsible name that is NOT a valid mail address

Press enter, so this error confirmation dialog will show up:

Now Confirm the error message.

Back to the "Create new Action Item Dialog" user is not able to scroll anymore.
=> OK/Cancel button is not reachable. UI-Dead-End!

Solution would be to improve \client\helpers\confirmationDialog.js with a more clever closing behaviour, to keep the  style "modal-open". (see stackoverflow topic above)

Alternative solution: Reduce the usage of modals :smile:

Should be fixed with PR https://github.com/4minitz/4minitz/pull/474


Combining conventions with explicit use of ProducesResponseType

Conventions is a great idea and want to use it, but I still want to specify ProducesResponseType on the controller class to cater for methods not included in the conventions. When I do it, conventions don't seem to apply. How do I combine both? There surely must be a way, conventions unlikely to be used as the sole source of the metadata.

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

ID: 9d14d206-7998-2031-d1d7-f8c8dbecf54a
Version Independent ID: 3071c40e-f08b-555e-4fe7-dc3828f5828e
Content: Use web API conventions
Content Source: aspnetcore/web-api/advanced/conventions.md
Product: aspnet-core
Technology: aspnetcore-webapi
GitHub Login: @pranavkm
Microsoft Alias: scaddie

Per the doc:
Web API conventions are a substitute for decorating individual actions with [ProducesResponseType].

Per my question:
I still want to specify ProducesResponseType on the controller class to cater for methods not included in the conventions
Is it possible to define class-level conventions, or is there another way to get the same functionality as I get by applying [ProducesResponseType] on a class while using conventions for individual methods?


Return policy error on pull

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>When a pull failed due to a signature policy violation, the error returned was:
ERRO[0001] image "quay.io/tomsweeneyredhat/myalpine" not found in /etc/containers/registries.conf registries: image not known

The PolicyRequirementError that triggered the failure but itself was lost:
Running image docker://quay.io/tomsweeneyredhat/myalpine:latest is rejected by policy.

With this change the failure now looks like:
# buildah from quay.io/tomsweeneyredhat/myalpine
Source image rejected: Running image docker://quay.io/tomsweeneyredhat/myalpine:latest is rejected by policy.

I'm a little leery about doing a return of the error instead of the continues.  I thought about doing a lastError thing, but that go messy too.  However if @nalind thinks that's the better way to go, I'll convert.
This addresses at least in part: #849

looks like network flakes, going to rekick the tests

bot, retest this please.

bot's gone on vacation, going to close/reopen to retest.

bot, retest this please.

bot, retest this please

Oh my goodness, green test buttons finally.  @rhatdan, @umohnani8, @nalind PTAL.

LGTM

@rh-atomic-bot r+


Netio Switch platform support

Added support for Net4all Netio switches.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):
switch:
  - platform: netio
    host: netio-living
    outlets:
      1: "AppleTV"
      2: "Htpc"
      3: "Lampe Gauche"
      4: "Lampe Droite"
  - platform: netio
    host: <IP_ADDRESS>
    port: 1234
    username: user
    password: pwd
    outlets:
      1: "Nothing..."
      4: "Lampe du fer"

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If code communicates with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Flake8 linting ok.

Our dev branch was broken for 24 hours and that's the moment that you branched off :( Please rebase on latest dev so only the tests broken by your commits show up.

Closing and re-opening to trigger tests.

Unfortunately, the same tests are failing but are not related I think.

Looks good! :dolphin:

Great!
Thanks :)

On 10 Jun 2016, at 08:40, Paulus Schoutsen<EMAIL_ADDRESS>wrote:
Merged #2181 https://github.com/home-assistant/home-assistant/pull/2181.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/home-assistant/home-assistant/pull/2181#event-688185621, or mute the thread https://github.com/notifications/unsubscribe/AAgpIy_ypaUD_AkD0azL0JbDzNSve8HIks5qKQbagaJpZM4Ip3yf.


Approximate element pairing for sets

This tries to implement #66 in the following manner:

Introduces difference counts (one for the number of difference, and another for the number of ignored fields)
Abstracts pairing functions into PairingFn which can be equals-based or differ-based:

Equals-based functions work the same way as they used to
Differ-based pairing can be either approximate, using the default item differ or can rely on a user-specified differ instance. This is to allow for user customizability.
Differ-based pairing functions take an additional parameter for difference count threshold.

The difference count is compared against the threshold specified in the pairing function to then match similar elements.

I tried to approach this in as common-sense way as possible and made some assumptions when implementing this such as using the approximate pairing function by default and specifying difference threshold to be 1, but I am open to discussing/adjusting them as needed.

Thanks for the PR @spavikevik. Sorry I wasn't able to reply to your issue yesterday.
When I saw your message I was original thinking that we can changes things such that Differ[Set[A]] is available when there's an implicit ValueDiffer[A], otherwise the user has the specify the paring function manually.
However, I do like the user-experience here of providing a default a Set differ which tries to do some smart guessing. Let's explore further!


Aging

Hi,
Nice work you got here, just wondering, is there a way to figure out which are the outstanding AR/AP invoices/bills instead of just getting a total due_balance? Thank you.

unfortunately, invoices and bills are higher-level abstract (bookkeeping vs core accounting) than this package attempts to handle.


Update CI Node 20 version

At this time GitHub Actions default NodeJS is 20.6.0 which causes problems on CI with ShellJS dependency (https://github.com/shelljs/shelljs/issues/1133).
We temporarily changed the Node version to explicit 20.6.1 but should set it back to just 20 once it will be released to GH Actions node-versions repository.

https://github.com/o1-labs/zkapp-cli/pull/447/commits/a3e0c4eaa4f78b1ce58405bd2333194da2ed2dc1


Fetch time slips - results in 2 candles generated simultaneously

At least on my system the candle fetch / process / method loop takes just over 60 seconds (around 62), this means that the candle fetch time slips by 2 seconds a minute.
So what?
This means that every so often the process generates 2 consecutive candles simultaneously, as I am acting on 1 minute candles this can result in a significant delay in placing orders - Just now I had a fetch at 12:40:58 - so this generated the candle for 12:39, fetch at 12:42:00 which generated the candle for 12:40 and then a fetch at 12:43:02 which generated the candles for both 12:41 and 12:42...
Can we make the candle fetch interval <60 seconds (say around 50 secs) - if it is a prime number would it spread the fetches more evenly - how about 53?

this is fixed!


A few questions about Ccured and SAFECode

Hi,
Thanks for your impressive work again. May I ask a few more questions about some details on implementing Ccured and SAFECode? My initial goal here is to find a solution to extend DataGuard to support the classification of both heap and global objects as well.

About Ccured. I checked more about the current Ccured implementation, and I think here is the code handling heap objects (if I understand correctly): https://github.com/Lightninghkm/DataGuard/blob/master/CCured/NesCheckFast.cpp#L1241. Inside this if-branch, the isUnsafeStackPointer is assigned as false, and the branch will execute break later. Does this mean the current version of DataGuard treats all heap objects as safe objects? And for global objects (https://github.com/Lightninghkm/DataGuard/blob/master/CCured/NesCheckFast.cpp#L1294), are they also classified as safe? I raised the questions as I am not sure whether the current implementation of Ccured already supports classifying all (stack, heap, and global) objects or whether DataGuard just simply filters heap or global objects because the paper is only targeting the data on the stack. Thanks!

About SAFECode. I am sorry I didn't find any implementation details about SAFECode in the paper. Did you use the version here (https://safecode.cs.illinois.edu/) to support the pointers analysis? Since the supported LLVM versions are quite old (used on LLVM-3.x), I am not sure how did you use it in DataGuard (used on LLVM-10). Could you please let me know it? Also, I am also concerned about whether SAFECode supports classifying the pointers in the heap or global. Do you have any ideas about it? Thanks!

About challenges for classifying the pointers from the heap. Currently, I am confused about what are the difference between classifying heap objects and stack objects, because the pointers whether point to the stack or heap is a value. The locations of that value may not matter a lot, so I don't see any differences/challenges to classify the heap objects. Did I miss anything? Do you have any ideas on what could be the specific challenges to identify the unsafe pointer operations from the heap or global instead of the stack? Thanks!

Thank you so much if can give me some hints on the above questions. Have a nice week ahead!
Best regards,
Haoxin

Hi Haoxin,
Thanks for your interest.
The DataGuard work targets the stack, so we only analyze the cases where "pointers are on the stack and point to the stack".
Stack pointers that point to heap and globals are filtered out in the CCured analysis stage to exempt them from further static analyses (e.g., value range analysis). This means CCured can handle them but we filtered them out. They are not classified as safe but directly classified as unsafe. They are not counted as unsafe stack pointers since our additional analyses require the pointer points to the stack.
We did not use SAFECode in DataGuard, but I tested SAFECode in our follow-up works for comparison, I used the same repo pointed out by you.
There are plenty of differences between analyzing stack and heap objects, at implementation-wise, you don't see heap/global objects represented as SSA variables in LLVM IR, but as memory locations, which is different from stack objects. Pointer-wise, they are pretty much the same (e.g., ptrarith and cast are the same), so the current CCured classification can already handle them.
We have a new work currently under submission for heap protection using a similar spirit, please stay tuned.
Best,
Kaiming.

Hi Kaiming,
Thank you so much for your super speedy responses. Your explanations cleared my confusions.
Wish the best for your new submission and hope I can read it soon!
Best,
Haoxin


1546c82 doesn't work in fish

When using fish shell, the fzf previewer throws an error because it uses bash syntax to set variables. Replacing with a call to "env" should fix the issue.

Actually since it's so quick I just implemented it, seems to work fine, let me know if you're still having issues and we can reopen this.


lua: 5.2.3 -> 5.2.4

Motivation for this change
Lua 5.2.4 is the EOL point release of the 5.2 series.
Things done

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

[x] NixOS
[x] macOS
[x] other Linux distributions (Ubuntu)

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

@GrahamcOfBorg build hydra awesome vlc

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: hydra, awesome, vlc
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

@GrahamcOfBorg build wireshark

Success on x86_64-linux (full log)
Attempted: hydra, awesome, vlc
Partial log (click to expand)
/nix/store/nxzki0r4dncm0hpx1p3116q7sf7lri05-vlc-3.0.3/share/vlc/utils/audio-vlc-default.sh: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/nxzki0r4dncm0hpx1p3116q7sf7lri05-vlc-3.0.3/bin/nvlc: interpreter directive changed from " /bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/nxzki0r4dncm0hpx1p3116q7sf7lri05-vlc-3.0.3/bin/rvlc: interpreter directive changed from " /bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/nxzki0r4dncm0hpx1p3116q7sf7lri05-vlc-3.0.3/bin/qvlc: interpreter directive changed from " /bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/nxzki0r4dncm0hpx1p3116q7sf7lri05-vlc-3.0.3/bin/cvlc: interpreter directive changed from " /bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/nxzki0r4dncm0hpx1p3116q7sf7lri05-vlc-3.0.3...
postPatchMkspecs
/nix/store/lz6b43lqlx9jf4pad4944sbcl343xk3f-hydra-2018-08-07
/nix/store/p5c34dm9w6pmnqpk74nwbrd1bglybknf-awesome-4.2
/nix/store/nxzki0r4dncm0hpx1p3116q7sf7lri05-vlc-3.0.3

@GrahamcOfBorg build lua

Success on x86_64-darwin (full log)
Attempted: wireshark
Partial log (click to expand)
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/lib/libwireshark.11.1.3.dylib: fixing dylib
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/Applications/Wireshark.app/Contents/Frameworks/libwiretap.8.0.3.dylib: fixing dylib
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/Applications/Wireshark.app/Contents/Frameworks/libwsutil.9.0.0.dylib: fixing dylib
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/Applications/Wireshark.app/Contents/Frameworks/libwscodecs.2.0.0.dylib: fixing dylib
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/Applications/Wireshark.app/Contents/Frameworks/libwireshark.11.1.3.dylib: fixing dylib
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/lib/libwiretap.8.0.3.dylib: fixing dylib
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/lib/libwsutil.9.0.0.dylib: fixing dylib
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/lib/libwscodecs.2.0.0.dylib: fixing dylib
/nix/store/686zri3sc4790j65xwj102k8h5d5i63f-wireshark-qt-2.6.3/lib/libwireshark.11.1.3.dylib: fixing dylib
postPatchMkspecs

Success on x86_64-linux (full log)
Attempted: lua
Partial log (click to expand)
/nix/store/3j8s75zamc7sicpawszdk6i5ljwl41s2-lua-5.2.4

Success on x86_64-darwin (full log)
Attempted: lua
Partial log (click to expand)
cd src && cp -d liblua.5.2.4.dylib /nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4/lib
cd doc && cp -d lua.1 luac.1 /nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4/share/man/man1
ln -s -f liblua.5.2.4.dylib /nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4/lib/liblua.5.2.dylib
ln -s -f liblua.5.2.dylib /nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4/lib/liblua.dylib
post-installation fixup
gzipping man pages under /nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4/share/man/
strip is /nix/store/df6k4mgdjxciy0f637lryp7c9ln7n1m3-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4/lib  /nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4/bin
patching script interpreter paths in /nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4
/nix/store/jn4cqfhqikdk0p71ickylaym2z60gc7h-lua-5.2.4

Success on aarch64-linux (full log)
Attempted: lua
Partial log (click to expand)
/nix/store/g5d2jki3nmpq2mksf8k5psnbs9f0mrfd-lua-5.2.4

Success on aarch64-linux (full log)
Attempted: wireshark
Partial log (click to expand)
shrinking /nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3/bin/sharkd
shrinking /nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3/bin/rawshark
shrinking /nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3/bin/tshark
gzipping man pages under /nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3/share/man/
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3/lib  /nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3/bin
patching script interpreter paths in /nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3
checking for references to /build in /nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3...
postPatchMkspecs
/nix/store/08dn80339hkjiizi26yg24g6hhcc66z8-wireshark-qt-2.6.3

vlc failure on aarch64 is unrelated

backported to 18.09: b831700b8f2

Success on x86_64-linux (full log)
Attempted: wireshark
Partial log (click to expand)
shrinking /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3/bin/tshark
shrinking /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3/bin/text2pcap
shrinking /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3/bin/editcap
shrinking /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3/bin/capinfos
gzipping man pages under /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3/share/man/
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3/lib  /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3/bin
patching script interpreter paths in /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3
checking for references to /build in /nix/store/vwkn7kkr6c7f1bsqjyy18plgks925bdi-wireshark-qt-2.6.3...
postPatchMkspecs


Add air gapped wg readme

Add initial air gapped wg readme with links and meeting info

/assign @AloisReitbauer


Add wildcard ACM certs back

It's less annoying for sub-domains (only need to verify one)
It's easy to hit the (default) ACM cert limit for your account
Still required for sub-sub domains, so we need some logic here to determine when a new cert is necessary
Make sure deleting the stack won't take other services with it...

This would be awesome. Now I'm finding myself not specifying stages at all to setup them later on api gateway manually. I thought about the problem and came up with the following idea how this could be implemented:
{
  "stages": {
    "production": {
      "domain": "api.gh-polls.com",
      "cert": "*.gh-polls.com"
    },
    // Or maybe specify the certificate arn?
    "staging": {
      "domain": "stage.gh-polls.com"
      "cert": "arn:aws:acm:us-east-1..."
    }
  }
}

This could be added without breaking changes and your requirements above could be met. What do you think?

I'd like to avoid specifying the cert, I think we can pretty easily determine when to use a wildcard programmatically. Stack deletion may be an issue though I have to double check that.

Would you accept a PR? Im pretty new to Golang but I want to do more with it. For the programmatic part. Would you do something like:

List certs and use existing wildcard cert that matches if possible.
If not 1., create wildcard cert

or something fancier? For the deletion I would need some hints

I'm not sure off-hand what AWS' behaviour is for the deletion. I believe it will refuse to delete certs used by other apps. If that's the case this might not be worth adding

I also think thats the case. I will double check it and if it refuses to delete used certs, just log it accordingly. Ok?

oh right, I forgot that we have certs outside of CloudFormation. Go for it :D. util.Domain  may be useful there.


Module import requires unsafe-eval in script content security policy directive

I'm trying to import the module in an Electron application with a recommended Content Security Policy of script-src 'self' but it prevents loading of the module with the error below.

It loads if I add 'unsafe-eval' to the policy, but I get reminded of the security risk.

Is there a way around this or should an Electron application use the dgraph-js-http package instead? I've read in this blog post that "For many use cases, the difference in performance is negligible".
Would that apply to an Electron application using a single Dgraph instance locally?
dgraph-js: 20.3.0

I would recommend you use to dgraph-js-http instead.
You can check out this sample electron app which makes use of dgraph-js-http client.
https://github.com/prashant-shahi/electron-todo

Thank you for the fast response and for providing an example I can work off of! :man_dancing:
I was hoping for the dgraph-js-http recommendation to avoid the electron-rebuild and<EMAIL_ADDRESS>onwards context aware requirement for grpc which I'd have ran head first into.


[POC]: Add an initial shift for coregistration

Context
The CNES wants to integrate the ability to instantiate an initial offset for the co-registration calculation. Because the dem_coregistration function allows for easy addition of this feature and we will recommend its use to CNES members, we propose simple modifications at this level.
Implementation

Using the translate function from Geoutils:
The translate function from Geoutils allows for translating a Digital Elevation Model (DEM). We will call this function in the dem_coregsitration method .

def dem_coregistration(
    src_dem_path: str | RasterType,
    ref_dem_path: str | RasterType,
    out_dem_path: str | None = None,
    coreg_method: Coreg | None = NuthKaab() + VerticalShift(),
    grid: str = "ref",
    resample: bool = False,
    resampling: rio.warp.Resampling | None = rio.warp.Resampling.bilinear,
    shp_list: list[str | gu.Vector] | tuple[str | gu.Vector] | tuple[()] = (),
    inout: list[int] | tuple[int] | tuple[()] = (),
    filtering: bool = True,
    dh_max: Number = None,
    nmad_factor: Number = 5,
    slope_lim: list[Number] | tuple[Number, Number] = (0.1, 40),
    plot: bool = False,
    out_fig: str = None,
+   estimated_initial_shift : list = [0, 0]
) -> tuple[DEM, Coreg, pd.DataFrame, NDArrayf]:
+	"""
+	explain estimated_initial_shift + affine only
+	"""

	    if isinstance(ref_dem_path, str):
	        if grid == "ref":
	            ref_dem, src_dem = gu.raster.load_multiple_rasters([ref_dem_path, src_dem_path], ref_grid=0)
	        elif grid == "src":
	            ref_dem, src_dem = gu.raster.load_multiple_rasters([ref_dem_path, src_dem_path], ref_grid=1)
	    else:
	        ref_dem = ref_dem_path
	        src_dem = src_dem_path
	        if grid == "ref":
	            src_dem = src_dem.reproject(ref_dem, silent=True)
	        elif grid == "src":
	            ref_dem = ref_dem.reproject(src_dem, silent=True)
+      if estimated_initial_shift != [0, 0] :
+          logging.warning("Initial shift in affine mode only")
+          estimated_initial_shift = [estimated_initial_shift[0] * pixel_resolution_x, estimated_initial_shift[0] * pixel_resolution_y]
+          src_dem.translate(estimated_initial_shift[0], estimated_initial_shift[1], inplace=True)
	.
	.
	.
    # Coregister to reference - Note: this will spread NaN
    coreg_method.fit(ref_dem, src_dem, inlier_mask)
    dem_coreg = coreg_method.apply(src_dem, resample=resample, resampling=resampling)
+   if estimated_initial_shift != [0, 0] :
+   	calculated_shift_x = dem_coreg.meta["shift_x"]
+       calculated_shift_y = dem_coreg.meta["shift_y"]

+		dem_coreg.meta["shift_x"] = calculated_shift_x + estimated_initial_shift[0]
+		dem_coreg.meta["shift_y"] = calculated_shift_x + estimated_initial_shift[1]

Tests

Set up a test for the dem_coregistration function with an initial offset value different from 0.
Here, a new paragraph should be added to test this updated version.

Documentation

Update the documentation to include examples on using the new estimated_initial_shift

/estimation 3d

Looks good! :slightly_smiling_face:
So the "explain affine only" would essentially be a check like coreg_method.is_affine for a Coreg object or all(c.is_affine for c in coreg_method) for a CoregPipeline object?
I'm not sure of the behaviour of the is_affine property on a pipeline right now (it might fail), we could modify it so it derives the proper attribute directly here: https://github.com/GlacioHack/xdem/blob/595acb9bf2ba150232cf161fc09f8747c84b5602/xdem/coreg/base.py#L1602 (either by finding a solution for it to run on all subclasses, or simply by overridding the property in CoregPipeline).


Hey! Sorry, I can't figure out how to setup connection mode

I had installed quasar and configured vuex-orm, but I can't get rid of nodes from queries
I keep getting this Connection Query Mode is 2 by config
This is my config of store
import Vue from 'vue'
import Vuex from 'vuex'
import VuexOrm from '@vuex-orm/core'
import VuexORMGraphQL from '@vuex-orm/plugin-graphql'

Vue.use(Vuex)

global.database = new VuexOrm.Database()
require('../models/autoloader')

VuexOrm.use(VuexORMGraphQL, {
  database,
  debug: true,
  connectionQueryMode: 'edges',
})

const store = new Vuex.Store({
  modules: {
  },

  plugins:[
    VuexOrm.install(database, {
      ConnectionMode: 3
    })
  ],

  // enable strict mode (adds overhead!)
  // for dev mode only
  strict: process.env.DEV
})

export default store

But any my changes to connectionQueryMode doesn't make any affects
packacge.json strings is
"@vuex-orm/core": "^0.31.12",
"@vuex-orm/plugin-graphql": "^1.0.0-rc.35",

That's because connectionQueryMode was deprecated with Release 1.0.0-rc.34.
You can change it now via the adapter pattern now.
However the documentation was not deployed for some reason. I've fixed that. Sorry for that! :)

In case someone is still wondering how to set up connection mode. https://vuex-orm.github.io/plugin-graphql/guide/adapters.html#basics
import { DefaultAdapter, ConnectionMode } from "@vuex-orm/plugin-graphql"

export default class CustomAdapter extends DefaultAdapter {
    constructor() {
        super()
    }

    getConnectionMode() {
        return ConnectionMode.EDGES
    }

    // Your code
}


add an integration test multiplexing multiple clients on one packet conn

Depends on #1407.

Codecov Report

Merging #1409 into master will decrease coverage by 0.46%.
The diff coverage is 92.17%.

@@            Coverage Diff             @@
##           master    #1409      +/-   ##
==========================================
- Coverage   90.01%   89.56%   -0.46%
==========================================
  Files         121      122       +1
  Lines        8412     8473      +61
==========================================
+ Hits         7572     7588      +16
- Misses        550      594      +44
- Partials      290      291       +1

Impacted Files
Coverage Δ

server_tls.go
76.51% <100%> (+0.32%)
:arrow_up:

server.go
90.79% <100%> (-0.08%)
:arrow_down:

packet_handler_map.go
100% <100%> (ø)

client_multiplexer.go
100% <100%> (ø)

session.go
72.56% <50%> (-5.71%)
:arrow_down:

client.go
88.55% <83.33%> (-0.77%)
:arrow_down:

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 a737aca...2f16f95. Read the comment docs.


button

All of the following criteria must be met

[X] All Requirements must be installed.
[X] Changed the title so that it doesn't just says "[Bug]: "
[X] I have searched the GitHub issues for my bug, even in the closed issues.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[X] Tried restarting PC.
[ ] Was able to see the Start Screen.

Describe the issue.
Even though I put the Btd6ModHelper.dll file in. There is no button. I'm using x86. (Another weird thing, I can't see the start screen nor a console.)
Did you attach your log file?
Select an option below

the game is only x64

I downloaded it from Steam so I use x86.


Fix tokenization of SE Asian and South Asian scripts

There are more scripts where we need to add exceptions to the Unicode word-boundary matcher (as I found in ConceptNet):

More spaceless scripts: Khmer, Lao, Myanmar
Scripts where the algorithm just doesn't seem to account for all the combinations of letters and marks that can come up: Devanagari, Gurmukhi, Sundanese, Tamil, Telugu, Tibetan

BTW, the comment at the top of this PR is out of date given the changes I made. I handled Devanagari and friends by changing the rules for 'marks' instead of by listing them all explicitly.


False positive on netty-tcnative

For #3865 only netty-tcnative-classes received a suppression for the false netty:netty CPE link. However also other libraries of the netty tcnative multimodule project are affected by the same FP as can be seen from the issue-comments:
It fails for us in 6.5.3 as well, did anyone still notice the issue on latest version?
[ERROR] Failed to execute goal org.owasp:dependency-check-maven:6.5.3:check (default-cli) on project zookeeper:
[ERROR]
[ERROR] One or more dependencies were identified with vulnerabilities that have a CVSS score greater than or equal to '0.0':
[ERROR]
[ERROR] netty-tcnative-2.0.48.Final.jar: CVE-2021-43797, CVE-2019-16869, CVE-2015-2156, CVE-2021-37136, CVE-2014-3488, CVE-2021-37137, CVE-2019-20445, CVE-2019-20444, CVE-2021-21295, CVE-2021-21409, CVE-2021-21290

For more info see discussion on the PR: https://github.com/apache/zookeeper/pull/1817
Originally posted by @nkalmar in https://github.com/jeremylong/DependencyCheck/issues/3865#issuecomment-1036191143

Was this supposed to be fixed in 7.0.0? Because if yes, then the error is still there
[ERROR] Failed to execute goal org.owasp:dependency-check-maven:7.0.0:check (default) on project ***REDACTED***:
[ERROR]
[ERROR] One or more dependencies were identified with vulnerabilities that have a CVSS score greater than or equal to '7.0':
[ERROR]
[ERROR] netty-tcnative-boringssl-static-2.0.48.Final-osx-x86_64.jar: CVE-2015-1346(7.5), CVE-2011-1797(9.3), CVE-2017-7000(8.8), CVE-2015-1205(7.5)
[ERROR]
[ERROR] See the dependency-check report for more details.
[ERROR] -> [Help 1]

@gmariotti Please open up a FP report for it.


Add tests for Documentation Live Preview

Most of the heavy lifting for Documentation Live Preview is done by SourceKit-LSP/SwiftDocC. However, there is some code that we can exercise within the extension.
These are effectively smoke tests that ensure the Web View is sent the appropriate notifications to display content generated by SourceKit-LSP.

These tests will be skipped until we have a nightly build of SourceKit-LSP that includes the textDocument/convertDocumentation request. We should wait to merge until they can be properly run in CI.


⚠ Bofrost.at has degraded performance

In 9c7b2c2, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2853 ms

Resolved: Bofrost.at performance has improved in 750378a after 6 minutes.


Restore support for duplicate Hue scene names

Description:
The Hue component requires a scene name and group name in order to activate a scene.  However, this fails for scenes with names that exist in multiple Hue groups (such as the default ones).  Logic to handle duplicate scene names existed prior to converting the Hue API to aiohue, but was removed during the transition.  This restores the original logic using the new API (aiohue 1.5.0)
Related issue (if applicable): fixes #13685
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Checklist:

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

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

[X] New dependencies have been added to the REQUIREMENTS variable (aiohue 1.5.0).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

Thanks 🎉

Please note #14739, it seems that scene_id needs a None default.
Also, I think it's odd that group.lights is sorted but scene.lights is not. But maybe that is not actually a problem.


Add definition for global variable

// lib
declare global {
  interface LuaGlobal {
    // empty. Use interface merging to extend.
  }
  const global: LuaGlobal
}

// usage
declare global {
  interface LuaGlobal {
    savedData: string
  }
}
global.savedData = 'foo'

I think it's much simpler to declare global yourself (as already described in the docs) than to use interface merging:
declare const global: {
  savedData: string
}


update to electron version 1.0

Steps to reproduce and a minimal demo of the problem
update electron-prebuilt to version 1.0.3
and run npm run start.desktop.windows

@fdu-axiometrics Is the current version causing you any trouble, just curious?

yeah. I actually I found a fix. the electron import changed. not sure if you have already fixed it.

Electron import? Is it incorrect on master currently? If you find it to be incorrect here, mind submitting a PR with your fix if so?

I need Electron upgraded as I need features in Chrome 50.  When I update the Electron version number to 1.2.1, it fails to start up with:
App threw an error during load
Error: Cannot find module 'app'
    at Module._resolveFilename (module.js:438:15)
    at Function.Module._resolveFilename (/Users/fstrecker/dev-sp/gp6/node_modules/electron-prebuilt/dist/Electron.app/Contents/Resources/electron.asar/common/reset-search-paths.js:47:12)
    at Function.Module._load (module.js:386:25)
    at Module.require (module.js:466:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/fstrecker/dev-sp/gp6/dist/dev/main.desktop.js:4:11)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:456:32)
    at tryModuleLoad (module.js:415:12)

The fix turned out to be super easy :)
One of these days, I will figure out how to do a pull request... until then, here is the solution to upgrade to Electron 1.2.1.
In main.desktop.ts, change this:
const app: any = require('app');
const Menu: any = require('menu');
const shell: any = require('shell');
const crashReporter: any = require('crash-reporter');
const BrowserWindow: any = require('browser-window');

to
const electron = require('electron');
const app = electron.app;
const Menu: any = electron.Menu;
const shell: any = electron.shell;
const {crashReporter} = require('electron');
const BrowserWindow = electron.BrowserWindow;

Thank you @friksa updated to 1.2.1 on master now 👍


Calls to recover that have no effect

I was testing a static analysis tool for go programs by running it on flynn/flynn. It highlighted that the calls to recover in ExitWithCode, Fatal and Fatalf in /pkg/shutdown/handler.go don't do anything. The reason is that calls to recover only have an effect if they are in a function that has been deferred [1], but ExitWithCode, Fatal and Fatalf aren't ever deferred. I'm not sure why exit wasn't also highlighted - either it is deferred somewhere, or it isn't called at all (or the tool isn't working correctly).
[1] "Recover is only useful inside deferred functions." https://blog.golang.org/defer-panic-and-recover

@owen-mc they may well be there in error, what do you propose?

I'd delete the lines ```
if serious != nil {
panic(serious)
}
in `exit` as the condition can never evaluate to true, and then get rid of the argument `serious`.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


Phantom Tiles happening when placing sprites with names longer than a character

Describe the bug
Creating a sprite with a name that starts with the name of another sprite creates phantom tiles
To Reproduce
Create a sprig project and create 2 items in the legend, w for winSpace, and wa for Wall
If I want to place the wa, instead of just placing the wall, it places a winSpace and a red square, the req square makes a new row or column of tiles which cannot be interacted with or placed on, generating phantom tiles
Expected behavior
Either WA should be placed just fine or an error should be given saying "Sprite names must be one character"
Actual behavior
Creates phantom tiles with no indication of something being wrong.
https://github.com/hackclub/sprig/assets/71194682/a33330bd-bf6d-4d4f-bf1e-7b2ac66e8500

I think that's just due to the nature of the map. Maps have a multi-line string of characters representing the bitmaps to be rendered. When the sprite name is longer than a character, "wa" gets added, but it gets parsed as "w" "a". I think having an error added is a good idea though

Yeah that is the issue, but it's not addressed which is the problem

theres already errors when you set a bitmap to more than one character. how else do you suggest this to be adressed?

It should probably just not let you assign a bitmap to a multi-character name in the first place, as that error only appears when the run button is pressed


suspicious login attempt

I'm getting a suspicious login notice in the Threads app every time I try to log in using this API. Any tips for how I might avoid this? Thanks!

The unidentified login notice is because the automatically generated deviceID so pass it but also include  userID, token and if you have an android that you already logged in with added to the device option
  const threadsAPI = new ThreadsAPI({
    verbose: true,
    device: {
      manufacturer: 'Your device manufacturer,
      model: 'Your device model',
      os_version: Your device android version,
      os_release: 'Your device manufacturer OS version',
    },
}


RandomChoiceMover

Small PR to keep it independent of other things:
To-Do:

[x] Rename MixedMover to RandomChoiceMover
[x] Add tests for RandomChoiceMover

Another really quick PR; this one is also ready to merge.


Program {program id} failed: custom program error: 0x1778

Thanks for putting this together!
I'd like to run it locally but am having an issue getting it to work.
I don't understand the error I received while running this multisig example test run:
Transaction simulation failed: Error processing Instruction 1: custom program error: 0x1778
    Program<PHONE_NUMBER>1111111111111111111111 invoke [1]
    Program<PHONE_NUMBER>1111111111111111111111 success
    Program 6tbPiQLgTU4ySYWyZGXbnVSAEzLc1uF8t5kJPXXgBmRP invoke [1]
    Program log: Instruction: CreateMultisig
    Program log: Custom program error: 0x1778
    Program 6tbPiQLgTU4ySYWyZGXbnVSAEzLc1uF8t5kJPXXgBmRP consumed 4765 of 200000 compute units
    Program 6tbPiQLgTU4ySYWyZGXbnVSAEzLc1uF8t5kJPXXgBmRP failed: custom program error: 0x1778

I followed the instructions on the README page.
If I'm doing something wrong please let me know.
Please advise, thanks.

Can you specify which instructions from README you are following?
Please describe the steps which lead you to mentioned error.

README does not have specific instructions except a command for running tests: anchor test. Do you get error on anchor test?

README does not have specific instructions except a command for running tests: anchor test. Do you get error on anchor test?

Yeah, that is where I got the error, thanks for clarifying.

Is this output that you see and from where your error is copied?
  multisig
    ✔ Tests the multisig program (1579ms)
Transaction simulation failed: Error processing Instruction 1: custom program error: 0x1778
    Program<PHONE_NUMBER>1111111111111111111111 invoke [1]
    Program<PHONE_NUMBER>1111111111111111111111 success
    Program 6tbPiQLgTU4ySYWyZGXbnVSAEzLc1uF8t5kJPXXgBmRP invoke [1]
    Program log: Instruction: CreateMultisig
    Program log: Custom program error: 0x1778
    Program 6tbPiQLgTU4ySYWyZGXbnVSAEzLc1uF8t5kJPXXgBmRP consumed 4487 of 200000 compute units
    Program 6tbPiQLgTU4ySYWyZGXbnVSAEzLc1uF8t5kJPXXgBmRP failed: custom program error: 0x1778
    ✔ Assert Unique Owners (65ms)

  2 passing (2s)


Exhentai tag autofill not working

Steps to reproduce

open exhentai filter sheet
type female: in the Tags box

Expected behavior
Autofill should appear
Actual behavior
Autofill doesn't appear
Crash logs
No response
Tachiyomi version
505
Android version
Android 13
Device
galaxy s21
Other details
No response
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.9.0.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

It works fine for me

Here's a recording
https://user-images.githubusercontent.com/83420659/232350558-8e771039-754d-449f-8ca4-bfa15023adb9.mp4

Here's a recording
https://user-images.githubusercontent.com/83420659/232350818-44eab32a-51c1-49d9-98b5-6eec7a49b4ca.mp4

I improved this in the latest build, it's not perfect yet but it's usable

It's gotten a bit better, yes. Hope it's fixed fully soon


support cheaper ignore in config.before_process

Would be nice if returning :ignore or something like that worked too so I don't have to raise and exception, which is kinda expensive and pollutes debug output

Thanks for the suggestion @grosser!

@grosser I'm not sure I understand. You want to be able to trigger ignoring by returning :ignore instead of having to raise Rollbar::Ignore?

yes, since that would be far cheaper (cpu/memory) than raising+catching an
exception
On Tue, Sep 25, 2018 at 8:12 AM Artur Moczulski<EMAIL_ADDRESS>wrote:

@grosser https://github.com/grosser I'm not sure I understand. You want
to be able to trigger ignoring by returning :ignore instead of having to
raise Rollbar::Ignore?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rollbar/rollbar-gem/issues/691#issuecomment-424382042,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAsZ729SlJkJ9JnNP4T6WkuNiUJ6TUvks5uekfZgaJpZM4SRqDe
.