ckan 1.16.1 -> 1.28.0

Please review this for MacOS, I'm not sure if my change breaks that.
Motivation for this change
CKAN is heavily out-of-date
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

[ ] 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 nixpkgs-review --run "nixpkgs-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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks for doing this! A while ago I spent a few hours trying to get their new build system to work and gave up.

Can this please be merged? I've been running this out of a manual nix-build from the branch.

I don't know enough about .NET packaging or mono. Perhaps @jonringer ?

I had some time to re-review my own request. As of CKAN 1.29.2, it uses Cake to build. As I can't find anything about this in nixpkgs or the documentation, I guess it has to be set up first. Not really worth the hassle if you ask me considering there is a fine EXE in the releases that definitely works fine.
How shall we continue?

I had some time to re-review my own request. As of CKAN 1.29.2, it uses Cake to build. As I can't find anything about this in nixpkgs or the documentation, I guess it has to be set up first. Not really worth the hassle if you ask me considering there is a fine EXE in the releases that definitely works fine.
How shall we continue?

I had some time to re-review my own request. As of CKAN 1.29.2, it uses Cake to build. As I can't find anything about this in nixpkgs or the documentation, I guess it has to be set up first. Not really worth the hassle if you ask me considering there is a fine EXE in the releases that definitely works fine.
How shall we continue?

we can use that and patch it if you prefer. The C# support in nix is not that great at this point tbh.

I had some time to re-review my own request. As of CKAN 1.29.2, it uses Cake to build. As I can't find anything about this in nixpkgs or the documentation, I guess it has to be set up first. Not really worth the hassle if you ask me considering there is a fine EXE in the releases that definitely works fine.
How shall we continue?

we can use that and patch it if you prefer. The C# support in nix is not that great at this point tbh.

Ok, I've pushed a new commit. We're now going straight to ckan 1.29.2 and using the provided exe. Please check if it was done properly.
Also, @Baughn please check if you still want to be a maintainer for this package or if I shall remove you from the list.

Ok, I've pushed a new commit. We're now going straight to ckan 1.29.2 and using the provided exe. Please check if it was done properly.
Also, @Baughn please check if you still want to be a maintainer for this package or if I shall remove you from the list.

I don't know enough about .NET packaging or mono. Perhaps @jonringer ?

We don't really have any Cake support, patching the distributables is fine for now.

I don't know enough about .NET packaging or mono. Perhaps @jonringer ?

We don't really have any Cake support, patching the distributables is fine for now.


Is there a way to revoke a shared link and delete related data permanently?

Hi,
I accidentally shared a conversation with private information, is there a way to undo it?
BTW, Thanks for your awesome extension, I really like it.

+1!

ShareGPT introduced deletes but you can delete the conversation within 30 seconds of sharing.


Mina beta2 bugs with mina-unicorn

Maybe it's time to update mina-unicorn to use with latest mina version 1 ?
https://github.com/mina-deploy/mina/issues/420

@hieuhlc Done!
Could you or anybody else test it with their setup and report back here?
It works fine in my apps, but I'm hesitant to push it to rubygems without any external test runs.

Thanks for doing this!
I had some issues with the trailing semicolons, not sure if it's something specific with my setup.
The lines end up being joined with a  &&\n which causes a syntax error.
With the semicolon the resulting code looks like this:
if true; then
  echo 1;
fi; &&
  echo 2

Causing:
test.sh: line 3: syntax error near unexpected token `&&'
test.sh: line 3: `fi; &&'

Without:
if true; then
  echo 1;
fi &&
  echo 2

Works:
1
2

@twe4ked There's only 2 places where && is used in the source code, and there's no if near them.
Could you provide the relevant chunk of the failing shell lines?

I mean that mina joins the commands together with &&.
       bash: -c: line 163: syntax error near unexpected token `&&'
       bash: -c: line 163: `  fi; &&'
       Connection to [snip].com closed.

 !
In my case it wraps the commands into () to avoid such errors:
  if [ -e /unicorn_test/current/tmp/pids/unicorn.pid.oldbin ] &&  kill -0 `cat /unicorn_test/current/tmp/pids/unicorn.pid.oldbin` > /dev/null 2>&1; then
     kill -s QUIT `cat /unicorn_test/current/tmp/pids/unicorn.pid.oldbin`;
  fi;
) &&

# ============================
# === Complete & unlock
(
rm -f "deploy.lock"

But you're right, it'll look better without trailing semicolons.
Thank you for pointing this out.

Mina-unicorn v1.0.1 has been released and published to rubygems with mina 1.0 support.


Support see changes for columns added during init

Hello, today I tried to move some our signals to the hooks and realised that if column was added into model init then it is shown as not changed.
Ex:
class Reservation(DirtyFieldsMixin, LifecycleModelMixin, models.Model):
  ...

In [15]: r = Reservation(edit_comment='test')

In [16]: r.has_changed('edit_comment')
Out[16]: False

In [17]: r.get_dirty_fields(check_relationship=True, verbose=True).get('edit_comment')
Out[17]: {'saved': None, 'current': 'test'}

In [18]: r = Reservation()

In [19]: r.edit_comment = 'test'

In [20]: r.has_changed('edit_comment')
Out[20]: True

So, if column is changed AFTER init - it is shown as changed, if into init - not.
Is it possible to cover this case too?

I'm not sure about this. If it's passed in init, then it's the initial value so it doesn't seem like it's really changed. There is no prior value.
What about adding new hookable moments for "before_init" and "after_init"? These could mirror the built-in pre_init and post_init signals: https://docs.djangoproject.com/en/3.2/ref/signals/


duplicate images in dataset

First let me thank you for your great work!
I downloaded the high-res version of your dataset and found that this image below shows up in the dataset for thousands of times with masks seem identical to me. Is there any specific reason for doing so? I trained a shadow removal model and found it overfitting on this image.

Thanks for your feedback. We have updated the OneDrive link.


Updating from 1.7.0 to 1.8.1/1.9.0 breaks

Since updating hubot-test-helper from 1.7.0 to 1.8.1 or 1.9.0, my project has been experiencing an odd issue with the sinon stubs we're creating for our hubot scripts. Essentially, we're creating an instance of aws's elasticbeanstalk and verifying a method was called a certain number of times
For example
beforeEach(() => {
      describeEnvironmentsStub = sinon.stub(Object.getPrototypeOf(new aws.ElasticBeanstalk()),
        'describeEnvironments').returns('some environment');
    });

    it('asks for environment information from various elastic beanstalks', async () => {
      await room.user.say('someUser', '@hubot list versions');
      expect(describeEnvironmentsStub.callCount).to.equal(1);
    });

Where the elasticbeanstalk instance is created in the script file outside the module.exports
let devElasticbeanstalk = new aws.ElasticBeanstalk();

module.exports = (robot) => {
  robot.respond(/list versions/, msg => {
    ...where we eventually call devElasticbeanstalk.describeEnvironments.
  });
}

In 1.8.1+, the stub fails to be created and the describeEnvironments function is undefined on the devElasticbeanstalk instance. This isn't the case in 1.7.0.
Any clue into what could be breaking it would be super helpful. Thanks!

After further investigation, it seems upgrading to 1.8.1 or above executes hubot's respond listener after the test cases have been run, so my stubs will never capture the callCount in the script files. Downgrading to 1.7.0 reverses this order and calls my implementation code before my test cases.
I'm looking through the 1.8.1 commits and it's hard to pinpoint where exactly this implementation changed since the Helper class was simply converted from coffee to regular javascript.
I tried following the examples on the README using the co package and I'm getting the same results :\


Minor bug. Faucet Balance shows 0

This is a minor bug and not breaking anything. When you start the faucet in production mode it says it has no balance. In fact it really has.

If you query the balance you get the real amount:
{
"jsonrpc": "2.0",
"result": {
"balance": "45000000000000000"
},
"id": 2
}

This issue should be solved with the latest C-Chain updates.


Removed the deprecatd class "TrackUtils" used in StartRecording.java

Technical Debt Description
The "TrackUtils" class is marked as deprecated, which means it is no longer recommended for use and may be removed in future releases. Ignoring deprecation warnings can lead to potential issues and reduced code maintainability.

Proposed Solution
I Replaced the usage of the deprecated "TrackUtils" class with the ContentProviderUtils.java. I Updated the code to use the new approach to ensure compatibility, adherence to best practices, and maintainability.

Commit ID : https://github.com/Vaibhav2399/OpenTracks-Winter-SOEN-6431_2024/pull/14/commits/509f7833ea1107f3dfcfbb604b3001796ef6276b

Done successfully.


documentation issue for @inherits

[EDIT by guardrex to fix the code blocks. Triple backticks above and below for a block, singles for inline]
Good day, for
@inherits CustomRazorPage<TModel>

where
public abstract class CustomRazorPage<TModel> : RazorPage<TModel>

this will raise an error:

The name 'PageContext' does not exist in the current context

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

ID: b87067b2-347c-072d-26c8-5a8c7c9c860b
Version Independent ID: a0b3b70a-3b39-1b5f-a4ce-c41bd2d6b044
Content: razor syntax reference for ASP.NET Core
Content Source: aspnetcore/mvc/views/razor.md
Product: aspnet-core
Technology: aspnetcore-mvc
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@hoksource  were you able to update the project for 3.1?
@pranavkm @NTaylorMullen  what's the 3.1 version of MvcRazorTemplateEngine Class ?

yes, im using 3.1. although calling
inherit PageBase
works. for razor pages

@hoksource  can you post a basic sample on GitHub and I'll get engineer to review it.

below does not work
Test.cshtml
`
@@page
@@inherits Microsoft.AspNetCore.Mvc.Razor.RazorPage
`
below works
Test1.cshtml
`
@@page
@@inherits Microsoft.AspNetCore.Mvc.RazorPages.PageBase
`

@HDONGDeveloper from https://github.com/dotnet/aspnetcore/issues/21013 and @hoksource  remind me where you get the 3.1 version of RazorTemplateEngine
I upgraded the sample from 2.0 to 3.1 but need the replacement for RazorTemplateEngine
https://github.com/Rick-Anderson/RazorTemplateEngine

I found https://github.com/dotnet/aspnetcore/issues/5079 where RazorTemplateEngine was removed.
So that needs to be removed from the sample.

this will raise an error:

The name 'PageContext' does not exist in the current context

@hoksource  all the pages with
@inherits CustomRazorPage<TModel>

work fine.

@hoksource  if you can put the simplest possible complete sample on GitHub and *Explain why you need this, I'll look into it
below does not work
Test.cshtml
@page
@inherits Microsoft.AspNetCore.Mvc.Razor.RazorPage<int>

below works
Test1.cshtml
@page
@inherits Microsoft.AspNetCore.Mvc.RazorPages.PageBase

Thank you for the info @Rick-Anderson . I was trying to see if i can work with razor pages with just the .cshtml no PageModels. using .cshtml only is not well documented for razor so i'm not sure how to add property and binding models to the property with .cshtml razorpage only. So I ended up using PageModels.


Update api-hello-world.md

Added URL - Documentation link to the PS script. It needs to be a complete link as it is hashed

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/10173
@JohanFreelancer9 Please copy review

Learn Build status updates of commit ada27b0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/api-hello-world.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 77032a9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/api-hello-world.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@mjcaparas Please merge this PR. Thanks

Learn Build status updates of commit 7582c8a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/api-hello-world.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 14a3a50:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/api-hello-world.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Horizon: account streaming should send an event when account is created/merged

When streaming a non-existing account the connection is currently closed immediately with a 404. It'd be nice if the endpoint could keep the connection alive and send an event when the account is created (same for when it's removed/merged). Currently we need to poll the account endpoint for that which is not ideal.

One way around this might be to stream for effects and filter out the account id that you are interested in

Is this the StreamEffects function in effect_request.go? I'm having trouble finding where the code related to this ticket is.

Can I  work on this issue


Snapshot Space missing

from Adam: I think there might be an issue for this already? Think that the text is there it's just the same color as the input box.

I added a snapshot space but it doesn't show here, and i wanted to remove it
Browser metadata
Path:      /settings?dao=sep:0x329d13a13d82d1fb5d3A7D14C5a2417AFf525768
Browser:   Chrome <IP_ADDRESS> on Mac OS 10.15.7
Viewport:  1544 x 1020 @2x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved
Originally posted by @azadikoff in https://github.com/decentdao/fractal-interface/issues/1578#issuecomment-2101007374

Believe this is a duplicate of https://github.com/decentdao/fractal-interface/issues/1749.
@azadikoff can you please share your Sepolia DAO address?

0xb69FB5068AF1d3b1Ae1BC8CC8D37E12F83FEFAe4

0xb69FB5068AF1d3b1Ae1BC8CC8D37E12F83FEFAe4

@azadikoff this doesn't look like a DAO address.. is this your EOA address? I meant, what is the address of the DAO you deployed / were posting transactions to on wendesday.

Confirmed that the snapshot space is set for this DAO, it's just not "visible" in the settings because the text is the same color as the input background.
https://deploy-preview-1578.app.dev.fractalframework.xyz/settings?dao=sep:0x329d13a13d82d1fb5d3A7D14C5a2417AFf525768

Closing this issue as a duplicate of #1749


Route params

When validating params route params were not taken into account which
caused them to be deleted from the params hash.
This required us to write code like this
api_method :get, '/transfers/:id/:security_hash' do |id, security_hash|
    params[:id] = id
    params[:security_hash] = security_hash
    action_result Actions::ShowTransfer
end

Does this handle Sinatra block arguments? I.e. is a typecast done for the values yielded to the block defining the action after routing?
api_method :post, '/thing/:number' do |id| # <-- what is the type of "id" here?
  'Total success'
end

also the route param changes should be probably rendered with the docs, there is an environment variable when you run the specs that opens the generated documentation in the browser


request help: How to coordinate work between multiple ingress-controller pods.

Issue description
When we tested, a cluster deployed 2 apisix and 2 ingress pods, found that 1 ingress did not work, and stopped 1 ingress pod, the other ingress pod still did not work, the relationship between these 2 ingress pods is The master-slave relationship still works at the same time.
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long): 1.14
your Kubernetes cluster version (output of kubectl version): 1.21
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):

Hi
Your description confuses me. Please describe more clearly, including steps to reproduce.
In addition, we have not released v1.14 version, please describe it accurately.


Make useFsEvents as default strategy for the watching

Now that we have gotten better at handling inode movement etc and many people have tried using useFsEvents as default for watching files, make it default. This should give more agile callbacks compared to polling especially if it involves lots of files. It should also be better on CPU.
One catch on systems which have limits on fs watchers like linux, it would still fallback to polling but that was anyways done for the recursive directory watching so files is no different, but quantity varies so something to keep in mind.
This change also ensures that fsWatches are cached and reused with multiple callbacks instead of creating different fs watcher.
Most of the changes are refactoring on better baseline test infra update
10a7b4e actually has change to make fsEvents as default for file watching
61a99bf has change to cache the watches per name even when using fsEvents

@typescript-bot pack this

I wonder if you can see a change TSServer perf tests.
@typescript-bot perf test this

@amcasey any other feedback or is this good to go.. I would like to get this in sooner than later. Thanks

Just didnt think about it.. i can change that to UseFsEventsWithFallbackDynamicPolling

I think it depends whether you think one is "riskier" than the other. Fewer people opt themselves into the dynamic polling behavior, right? But we can still get it in in time for the beta if you think it would be the right course of action.

feels like relying on native implementation seems like better option but our dynamic polling is good.. But we dont have experience with its usage in fallbacksetting (eg how does reponse time look with very few files in polling queue etc) so safer bet seems to be useFsEvents. i feel like people have not complained about this much given for directories default is useFsEvents.

That's fair. I do think we should keep an eye out and revisit. My impression is that Windows and macOS, I assume this almost never makes a difference; but on Linux, it will be pretty common to hit the file watching limit, so we may still end up installing a lot of file-watchers and it would be good to improve the situation. Let me know if that's an accurate assessment.

end up installing a lot of file-watchers and it would be good to improve the situation. Let me know if that's an accurate assessment.

Yes thats true.. also many users just increase limit of native file watchers instead of opting for fallback but then this was before things were not easily configurable.
Definitely can revisit this at later point.

@sheetalkamat have to say I thought the blog post was interesting! I just did a deep dive on file-watching in Go for Pulumi and we ended up using a Rust binary to ship the feature.
Does this implementation of file-watchers use ignore files at all, to reduce the number of system calls needed to monitor a directory containing, e.g., node_modules?

Love to find the community of people banging their heads against cross-platform file-watching!
During review of the blog post @sheetalkamat mentioned there's a few specific things in editor scenarios where we watch the node_modules directory instead of individual files within node_modules. But I might have misunderstood and she can fill in with more context.

As daniel mentioned, we watch node_modules instead of watching each file in editor scenarios. But apart from that when watching node_modules recursively, we have been ignoring files/directories starting with "." in node_modules. https://github.com/microsoft/TypeScript/blob/main/src/compiler/sys.ts#L487


{webpubsub} HTML tag isn't allowed in example syntax

Describe the bug
The source code for webpubsub is using HTML tags for examples
Instance 1: https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/latest/docs-ref-autogen/webpubsub.yml#L198
Example: --remove property.list <indexToRemove> OR --remove propertyToRemove.'
Instance 2: https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/latest/docs-ref-autogen/webpubsub.yml#L205
Example: --set property1.property2=<value>.'
Modify these instances to use some other character like a backtick or escape character.
Contributor's Guide guideance:
https://review.learn.microsoft.com/en-us/help/platform/validation-ref/disallowed-html-tag?branch=main
Instance 1 YML syntax looks like this:
summary: 'Remove a property or an element from a list.  Example: --remove property.list <indexToRemove> OR --remove propertyToRemove.'
Instance 1 published content renders like this:

Instance 1 example correction:

This impacts 2017-03-08-profile, 2018-03-01-hybrid and latest

https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/2017-03-09-profile/docs-ref-autogen/webpubsub.yml
https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/2018-03-01-hybrid/docs-ref-autogen/webpubsub.yml
https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/latest/docs-ref-autogen/webpubsub.yml

Related command
az webpubsub update --remove
az webpubsub update --set
Errors
MicrosoftDocs/azure-docs-cli build warning:

Issue script & Debug output
See "Describe the bug" section above.
Expected behavior
See "Describe the bug" section above.
Environment Summary
See "Describe the bug" section above.
Additional context
See "Describe the bug" section above.

@yonzhan, do you know if this issue made it to anyone's backlog to get fixed?

webpubsub related CLI commands are owned by service team.

This is the first reported issue so I will use this one as the centralized tracker.
The <indexToRemove> string comes from the help message of generic update argument --remove:
https://github.com/Azure/azure-cli/blob/261c6d53a3ce205291ef08e85cf487ae1b97bcbb/src/azure-cli-core/azure/cli/core/commands/arm.py#L391
https://github.com/Azure/azure-cli/blob/5eddcd4ff7cfce21c0b42b89435cf28cc6e094cf/src/azure-cli-core/azure/cli/core/aaz/_arg.py#L644
<indexToRemove> is actually a placeholder, instead of an HTML tag.
This happens not only to --remove, but to --set as well: https://learn.microsoft.com/en-us/cli/azure/vm?view=azure-cli-latest#az-vm-update()

It should have been --set property1.property2=<value>.
https://github.com/Azure/azure-cli/blob/261c6d53a3ce205291ef08e85cf487ae1b97bcbb/src/azure-cli-core/azure/cli/core/commands/arm.py#L390
Possible solutions:

Mark it as code with `: `--remove property.list <indexToRemove>` OR `--remove propertyToRemove`
Use curly brackets {}: {indexToRemove}
Escape <> with \: \<indexToRemove\>

@zackliu please help take a look

@jiasli I think the docs is auto generated so what we need to fix is the source code in azure-cli? However, these issues come from common commands, cli team will fix it at large?

I will fix this issue for webpubsub in #27752.

The reference doc of az webpubsub update has now been fixed. <> placeholders in --add and --remove's help messages are displayed correctly:
https://learn.microsoft.com/en-us/cli/azure/webpubsub?view=azure-cli-latest#az-webpubsub-update


Make resolution of the distortion mesh a config-file parameter

Added in pull request https://github.com/sensics/OSVR-RenderManager/pull/320


fix(dotenv): updates to latest version

:tada: This PR is included in version 1.32.13 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Validation failure responses in profile editor break with MFA settings field present

Steps to reproduce:

Visit the profile admin UI
Trigger a validation error (enter a single character in one of the password change fields)
Submit, observe Internal Server Error

The cause of this is that the MFA Settings field is not being handed its value (the Member object, which is required for it to generate its schema and operate) - seemingly the Form logic takes a different path for rendering when a validation error is present, which doesn't include instantiating the fields via getCMSFields.

I would imagine this is due to values loading from the submitted data (POST values), rather than the initial values (as per getCMSFields), and the React based form field has no value submitted, as it is not a real form field.
I will investigate.

Fixed by #231 (I think)


Handling photos

To test:
/images to view uploaded images
/success for success message on uploading images
/image_upload to upload an image
Details:
Created functionality to upload an image to the database and later view that same image.
No capabilities yet for deleting images or previewing uploads.

 Implement photo uploading functionality to find out how it works.


xSPInstallPrereqs does not trigger a reboot each time it is needed

xSPInstallPrereqs does not trigger a reboot each time it is needed
In complement to existing code in Set-TargetResource why not to add at the end of the procedure:
return (((Get-Item 'HKLM:\Software\Microsoft\Windows\CurrentVersion\Component Based Servicing\RebootPending' -ErrorAction SilentlyContinue) -ne $null) -or ((Get-Item 'HKLM:\Software\Microsoft\Windows\CurrentVersion\WindowsUpdate\Auto Update\RebootRequired' -ErrorAction SilentlyContinue) -ne $null) -or ((Get-Item 'HKLM:\SYSTEM\CurrentControlSet\Control\Session Manager' | Get-ItemProperty).PendingFileRenameOperations.count -gt 0))
Asking so to DSC to trigger a reboot when a reboot pending is detected

@BrunoReboul Great idea, will get something like that implemented in it shortly.

This is in pull request #246 - ill close this issue off once it is merged in to dev

Merged this in to dev, closing this issue


Add linter.yml again

Signed-off-by: Kirill Batyshchev<EMAIL_ADDRESS>

Finally this works :)


Performance Issue on Android

I am getting some performance issues while using it in device.
While sliding the page freezes for a moment before getting to the next page but when I click on the tab, it doesn't.
What could be the cause for this?
Here is a link explaining the issue:
https://drive.google.com/open?id=0B6mManAqlfpSRWNZanpwbTEzOTg
I am not using any lifecycle events in my code.
Ionic Info:
cli packages: (/home/jash/Desktop/ionicSud/insaniyat/node_modules)
@ionic/cli-utils  : 1.9.2
ionic (Ionic CLI) : 3.9.2

global packages:
Cordova CLI : 7.0.1

local packages:
@ionic/app-scripts : 2.1.3
Cordova Platforms  : android 6.2.3
Ionic Framework    : ionic-angular 3.6.0

System:
Android SDK Tools : 25.2.5
Node              : v6.11.2
npm               : 3.10.10
OS                : Linux 4.4

Thanks.

@lincolnthree Does that mean we have to live with that kind of lag, or is there anything we could do about?
This just doesn't feel smooth on a real device. Feels sluggish.


adding experiments to the yaml file

You can try them out with the following commands:
build/sbt assembly

$SPARK_HOME/bin/spark-shell --jars /Users/tjhunter/work/spark-sql-perf/target/scala-2.10/spark-sql-perf-assembly-0.4.10-SNAPSHOT.jar

and then in Spark:
com.databricks.spark.sql.perf.mllib.MLLib.run(yamlFile="/Users/tjhunter/work/spark-sql-perf/src/main/scala/configs/mllib-small.yaml")

I will make a separate PR to run the content of this file as a unit test.

Thanks!


Pressing . on github.com repo fails to setup editor

Pressing . on github repo fails to load the edit with the message "Error loading webview: Error: Could not register service workers: SecurityError: The operation is insecure.." (see image below).

Version: 1.64.2
Commit: f80445acd5a3dadef24aa209168452a3d97cc326
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:96.0) Gecko/20100101 Firefox/96.0
Embedder: github.dev

@paul90 could you try in Chrome? I think this is a Firefox-only issue, so I'd appreciate if you could test it 🙏.

I see much the same error in Chrome 98.0.4758.80

I notice in Firefox's console it is complaining that Error: Could not register service workers: SecurityError: The operation is insecure.. ![Screenshot 2022-02-11 at 11 17 53](https://user-images.githubusercontent.com/1552489/153585684-18764ab6-9003-4b61-b4d3-a779edaa5424.png)

Likely a duplicate of https://bugzilla.mozilla.org/show_bug.cgi?id=1320796
Are you using private mode or have you enabled any additional security settings?

Not using private mode, or enabled any additional security - at least as far as I can remember.

Is this the official Firefox build?

Yes

Same issue, so the question is how to configure firefox so that the CORS error and connection to api.github.com succeeds?
Seems to be a long standing firefox bug though: https://bugzilla.mozilla.org/show_bug.cgi?id=1552376 (no service workers allowed if 'delete cookies & site data when FF closes' is checked)

I don't see errors in Firefox console, and Markdown preview doesn't work either, but it works fine in Chrome.

And it's on Firefox 101.0.1

I have a similar error on Brave ([Version 1.40.107 Chromium: 103.0.5060.53 (Official Build) (64-bit)]). I see the same error as @paul90 (https://github.com/microsoft/vscode/issues/142813#issuecomment-1036122610) Additionally those errors are shown in the script console:

The core of the issue appears to be the "3p" (3rd party) property of the Worker's domain.
Browsers may have disabled 3p cookies (and storage requests) by default.
    [ See also: https://bugzilla.mozilla.org/show_bug.cgi?id=1413615#c5 ]

[ Quote mjbvz @ CE 2023-09-25 16:47:46 UTC:
https://github.com/microsoft/vscode/issues/194002#issuecomment-1734120563
    We use a third party domain to keep webviews isolated from the main VS Code window ]
<^>    Would you mind explaining the rationale of such design?
    The related functionalities don't seem to accept the attack vectors that would cause the security problems.


Updated XML helper to support storing and parsing of Guid

Updated XML helper to support storing and parsing of Guid.

@dotnet-policy-service agree company="LectureLinx"

Excellent addition, I would also include this in the 1.10.x branch.


Map Generation Block Diagram

Go here for a preview.

I'm getting ready to merge and updating the change log - please @me if any other changes are pushed.


Add midway worker bug to FAQ

We should try to give some hints in the FAQ that would help someone encountering #633.

While we should go ahead and document troubleshooting these issues, we still need confirmation on whether this was a networking issue on midway. I'm waiting on a midway support ticket on this.


Elementum configuration menu isn't displayed property

Elementum configuration menu isn't displayed property
Device: raspberry pi 4b osmc
Kodi log: https://paste.kodi.tv/dawozopeko

@FKOV You can try version 0.1.77, maybe it will help you.

Closing. Not getting response.


http://localhost:5006/simulate_imdb_perceptron is a blank page

When I visit http://localhost:5006/simulate_imdb_perceptron, the page is blank.
Is this normal? If not, how can I fix it?

Logs are as flowing:
(decai-simulation) root@f8d0a0404b4d:~/workspace/0xDeCA10B/simulation# bokeh serve decai/simulation/simulate_imdb_perceptron.py
2022-08-09 11:33:25,711 Starting Bokeh server version 2.4.3 (running on Tornado 6.2)
2022-08-09 11:33:25,719 User authentication hooks NOT provided (default user enabled)
2022-08-09 11:33:25,733 Bokeh app running at: http://localhost:5006/simulate_imdb_perceptron
2022-08-09 11:33:25,734 Starting Bokeh server with process id: 40
2022-08-09 11:33:52,321 WebSocket connection opened
2022-08-09 11:33:52,329 ServerConnection created
2022-08-09 11:33:52,364 404 GET /favicon.ico (<IP_ADDRESS>) 2.53ms

Thanks for trying the project! Sorry for taking so long to get back to you. I meant to try it myself but got busy. Until I get the chance to try it, some more details would help. Please share a screenshot of what you see in the browser on the page. At first I was suspecting that it wasn't connecting, maybe a Docker port forwarding issue but I see the GET /favicon.ico request went through, so I suspect it's something else. Also, is there anything in the browser's DevTools Console?

Same issue
bokeh serve decai/simulation/simulate_imdb_perceptron.py
2023-03-03 23:20:27,217 Starting Bokeh server version 3.0.3 (running on Tornado 6.2)
2023-03-03 23:20:27,417 User authentication hooks NOT provided (default user enabled)
2023-03-03 23:20:27,420 Bokeh app running at: http://localhost:5006/simulate_imdb_perceptron
2023-03-03 23:20:27,420 Starting Bokeh server with process id: 120302
2023-03-03 23:20:31.599605: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  SSE4.1 SSE4.2 AVX AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-03-03 23:20:33,283 WebSocket connection opened
2023-03-03 23:20:33,285 ServerConnection created

Same issue :
bokeh serve decai/simulation/simulate_imdb_perceptron.py
2023-03-03 23:20:27,217 Starting Bokeh server version 3.0.3 (running on Tornado 6.2)
2023-03-03 23:20:27,417 User authentication hooks NOT provided (default user enabled)
2023-03-03 23:20:27,420 Bokeh app running at: http://localhost:5006/simulate_imdb_perceptron
2023-03-03 23:20:27,420 Starting Bokeh server with process id: 120302
2023-03-03 23:20:31.599605: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  SSE4.1 SSE4.2 AVX AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-03-03 23:20:33,283 WebSocket connection opened
2023-03-03 23:20:33,285 ServerConnection created

@ali-alhwayzee were you using the Docker container?
https://github.com/microsoft/0xDeCA10B/tree/main/simulation#docker-setup
I suspect it could be an issue with the versions of dependencies.

@juharris thanks
added / updated specs:
- bokeh
- mkl
- mkl-service
- numpy
- pandas
- phantomjs
- python=3.8
- scikit-learn
- scipy
- tensorflow
The following packages will be downloaded:
package                    |            build
---------------------------|-----------------
hdf5-1.14.0                |nompi_h5231ba7_103         3.4 MB  conda-forge
lcms2-2.15                 |       hfd0df8a_0         235 KB  conda-forge
platformdirs-3.1.0         |     pyhd8ed1ab_0          17 KB  conda-forge
setuptools-67.5.1          |     pyhd8ed1ab_0         566 KB  conda-forge
------------------------------------------------------------
                                       Total:         4.2 MB

Downloading and Extracting Packages
hdf5-1.14.0          | 3.4 MB    | ###################################### | 100%
lcms2-2.15           | 235 KB    | ###################################### | 100%
setuptools-67.5.1    | 566 KB    | ###################################### | 100%
platformdirs-3.1.0   | 17 KB     | ###################################### | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: done

To activate this environment, use

$ conda activate decai-simulation

To deactivate an active environment, use

$ conda deactivate
update: conda update -n base -c defaults conda
Retrieving notices: ...working... done
Downloading and Extracting Packages
conda-repo-cli-1.0.2 | 114 KB    | ###################################### | 100%
zstandard-0.19.0     | 474 KB    | ###################################### | 100%
ruamel.yaml.clib-0.2 | 140 KB    | ###################################### | 100%
conda-package-stream | 26 KB     | ###################################### | 100%
conda-package-handli | 267 KB    | ###################################### | 100%
conda-23.1.0         | 942 KB    | ###################################### | 100%
ruamel.yaml-0.17.21  | 178 KB    | ###################################### | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: done
Retrieving notices: ...working... done
sudo docker run --rm -it -p 5006:5006 -v ${PWD}:/root/workspace/0xDeCA10B/simulation --name decai-simulation mcr.microsoft.com/samples/blockchain-ai/0xdeca10b-simulation bash
(decai-simulation) root@c903d9566f68:~/workspace/0xDeCA10B/simulation# bokeh serve decai/simulation/simulate_imdb_perceptron.py
2023-03-09 09:43:21,998 Starting Bokeh server version 3.0.3 (running on Tornado 6.2)
2023-03-09 09:43:22,314 User authentication hooks NOT provided (default user enabled)
2023-03-09 09:43:22,319 Bokeh app running at: http://localhost:5006/simulate_imdb_perceptron
2023-03-09 09:43:22,319 Starting Bokeh server with process id: 17
2023-03-09 09:43:32,554 WebSocket connection opened
2023-03-09 09:43:32,555 ServerConnection created
2023-03-09 09:44:02,040 WebSocket connection closed: code=1001, reason=None
2023-03-09 09:44:02,240 WebSocket connection opened
2023-03-09 09:44:02,241 ServerConnection created
2023-03-09 09:44:19,126 WebSocket connection closed: code=1001, reason=None
2023-03-09 09:44:19,488 WebSocket connection opened
2023-03-09 09:44:19,488 ServerConnection created

with Same problem blank page

Hey @juharris ,
I am also having trouble running the code provided. I get the above errors as well. Can the problem have something to do with the version of bokeh? If not, do you have any other recommendations? Thanks.

Network tab returning 200, check the server response.


How to make an object heading in the direction it falls

I wonder how I can make a grenade (in my case this is just a rectangular physic thing) point in the direction it falls. At the moment it seems it just points in the direction I fired it in the first instance. Somehow I have to make the "head" of it a bit more heavy or so. What is the best or easiest way to achieve that?

To contrast the real world with dyn4j, there are two things that dyn4j does not simulate:

Non-uniform mass (per body)
Fluid resistance (air, water, etc.)

In a vacuum, a body with non-uniform mass would experience a torque applied because part of the object is experiencing a larger force.  In the presence of a fluid its even more complicated because the object can experience torques based on its shape.
In short, you need to find a way to apply a torque to the body (or just set the rotation) that simulates the real world behavior (if that's what you are going for).  For example:
// run this every time step or maybe every 10 steps
Body b = //
// the initial launch direction
Vector2 launchv = //
// get the change in position from last frame
Vector2 v = b.getChangeInPosition();
// note v could be zero if the body didn't move
double a = v.getAngleBetween(launchv);
// set the rotation of the body
b.getTransform().setRotation(a);

This approach isn't the best.  The best approach would be to apply a torque every frame to keep the body pointed in the correct position instead of setting the rotation.  If you aren't creating a lot of these you might be able to achieve this with a WeldJoint:
Body grenade = //
grenade.addFixture(Geometry.createCircle(1.0));
grenade.translate(0, 0);

// put it at the top
Body heavierPart = //
heavierPart.addFixture(Geometry.createCircle(0.25));
heavierPart.translate(0, 0.35);

Vector2 anchorPoint = new Vector2(0, 0.35);
WeldJoint j = new WeldJoint(grenade, heavierPart, anchorPoint);

world.addBody(grenade);
world.addBody(heavierPart);
world.addJoint(j);

(full disclosure - I have not tested this) This might give you the effect you want, but you wouldn't want to do this if everything in your scene needed this behavior.  Alternatively, you could create your own process to apply a torque to the grenade body that keeps the rotation of the body in line with it's ballistic motion.

Adding torque every frame did the trick. Just had to fiddle a bit with the amount of torque until it looks fine. I think if the grenade falls into a hole or so it will overturn. But in my case, it would be out of the screen or already detonate.
thanks a lot again for your fast reply. I wonder why I don't see your answer here?

TLDR: You are correct you need to apply a torque to the body to cause it to rotate about its center of mass.  Another option might be to compute the torque based on projectile motion equation(s).

thanks a lot again for your fast reply. I wonder why I don't see your answer here?

Because I deleted it...  It was a terrible answer filled with incorrect information.  I was trying to complete it before I had to be somewhere and didn't think it through.  Thinking about it more while driving, I realized how bad the answer was and decided to just delete it before it did too much damage.  To amend the previous answer:

I wrongly described gravity.  It's an acceleration applied uniformly to all bodies regardless of their mass or density distribution, therefore, it won't help here.  Recall the hammer/feather experiment on the moon.
dyn4j does support non-uniform mass via multi-fixture bodies.  But again - that doesn't help here.

The general premise was correct - to apply a torque - but the information around it was wrong - apologies.  Glad you solved it on your own despite me misleading you. :)

appreciate that! I'm glad this library is alive and well maintained. I love the improvements. And it is so super simple to integrate it into my entity component system (I use Zay-Es for that).
Keep to good work up!

Appreciate it!


Linear Evaluation is too slow

Describe the bug
Not sure what the issue is. It is really slow though.
To Reproduce
import torch
import torch.nn as nn
from pytorch_lightning import Trainer
from pytorch_lightning.loggers import WandbLogger
from pytorch_lightning.callbacks import LearningRateMonitor
from torchvision.models import resnet18
from omegaconf import OmegaConf
from solo.methods.linear import LinearModel  # imports the linear eval class
from solo.data.classification_dataloader import prepare_data
from solo.utils.checkpointer import Checkpointer
#from pytorch_lightning.plugins.training_type import DDPPlugin
from pytorch_lightning.plugins import DDPPlugin
#from pytorch_lightning.strategies import DDPStrategy as DDPPlugin
import glob
from solo.methods.base_simple import model
kwargs = OmegaConf.create({
"name":"TestingSetUp",
"num_classes": 10,
"cifar": True, #??
"max_epochs": 100,
"optimizer": "sgd",
"precision": 16, #??
"lars": False, #??
"lr": 0.1,
"exclude_bias_n_norm_lars": False,  #??
"gpus": "0", #??
"weight_decay": 0,
"extra_optimizer_args": {"momentum": 0.9},
"scheduler": "step",
"lr_decay_steps": [60, 80],
"batch_size": 128,
"num_workers": 4,
"pretrained_feature_extractor": "SoloLearn-Weights/Weights-CIFAR10-SoloLearn/BT/barlow-cifar10-otu5cw89-ep=999.ckpt"
})
conf=OmegaConf.create({
"data":
{
"num_classes":10
},
    "max_epochs":100,

    "optimizer":
    {
        "name":"sgd",
        "batch_size":128,
        "lr":0.1,
        "weight_decay":0,
        "kwargs" :{},
        "exclude_bias_n_norm_wd":False,
        "exclude_bias_n_norm_lars":False,
        "layer_decay":0.0,
        "lars":False,
        "precision":16,
        "num_workers":4
    },

    "scheduler":
    {
        "name" :"step",
        "min_lr" : 0.0,
        "warmup_start_lr" : 3e-5,
        "warmup_epochs" : 10,
        "lr_decay_steps" : [60,80],
        "interval" : "step",
    },

    "finetune" : False,

    "performance":
    {
        "disable_channel_last":False
    },

    #loss_func:,
    "accumulate_grad_batches":1,
    "pretrained_feature_extractor": "SoloLearn-Weights/Weights-CIFAR10-SoloLearn/BT/barlow-cifar10-otu5cw89-ep=999.ckpt"
}

)
backbone=model()
model = LinearModel(backbone, conf)
model=model.cuda()
train_loader, val_loader = prepare_data(
"cifar10",
train_data_path=None,
val_data_path=None,
batch_size = conf.optimizer.batch_size,
num_workers = conf.optimizer.num_workers,
)
wandb_logger = WandbLogger(
name="linear-cifar10-Barlow",  # name of the experiment
project="self-supervised",  # name of the wandb project
entity=None,
offline=False,
)
wandb_logger.watch(model, log="gradients", log_freq=100)
callbacks = []
automatically log our learning rate
lr_monitor = LearningRateMonitor(logging_interval="epoch")
callbacks.append(lr_monitor)
checkpointer can automatically log your parameters,
but we need to wrap them in a Namespace object
from argparse import Namespace
#args = Namespace(kwargs)
args = kwargs
saves the checkout after every epoch
ckpt = Checkpointer(
args,
logdir="checkpoints/linear",
frequency=1,
)
callbacks.append(ckpt)
trainer = Trainer.from_argparse_args(
args,
logger=wandb_logger,# if args.wandb else None,
callbacks=callbacks,
plugins=DDPPlugin(find_unused_parameters=False),
checkpoint_callback=False,
terminate_on_nan=True,
max_epochs=100
)
trainer.fit(model, train_loader, val_loader)
Versions
Pre-Trained with pl-lighting 2.0.2, evaluating with pl-lighting 1.6

Hey. I can't properly understand what is going on in your question. Can you reformat it? Also, what does too slow mean? Did you run the code before and saw a decrease in performance?

I'm closing because there's no proper info. Feel free to re-open.

I'm sorry for not getting back to you sooner. I was busy. Here is the reformatted code.
Earlier Runs: It was way faster, and i could get my results within an hour.
Now, it takes forever to run. It takes up to 20secs/iteration.
I do have to say that there were issues with pl-lighting 2.0.2 in evaluation and I had to make a separate environment with pl-lighting 1.6.0--ish.


unexpected logical expression result

Hello
|| and && operators does not seems to work as expected
    expression, _ := govaluate.NewEvaluableExpression("foo == true || bar == true")
	parameters := make(map[string]interface{}, 8)
	parameters["foo"] = true
	parameters["bar"] = false
	result, _ := expression.Evaluate(parameters)
	log.Printf("actual %t, expected : true \n", result)

	expression, _ = govaluate.NewEvaluableExpression("foo > 10  && bar > 10")
	parameters = make(map[string]interface{}, 8)
	parameters["foo"] = 1
	parameters["bar"] = 11
	result, _ = expression.Evaluate(parameters)
	log.Printf("actual %t, expected : false \n", result)

this code will evaluate false for the first result, and true for the second one.

Thanks for the report, especially the sample! There was a gap in my tests, and it made me think I could be clever and get by without doing a type assertion when evaluating logical operators. For whatever reason, nobody's hit this case until today.
Ought to be fixed as of dc6cef2046fa304157beca38bcc8635e63e183e1, I added test cases for your examples and everything ought to work as expected. Let me know if you have any problems with the master branch.

It works !  Thanks for being so quick to fix it !
Nice piece of code !


Exits as soon as it has started

The containers exit immediately after starting with ./docker-samba launch
The log output is:
smbd version 4.3.9-Ubuntu started.
Copyright Andrew Tridgell and the Samba Team 1992-2015
STATUS=daemon 'smbd' finished starting up and ready to serve connections

nmbd version 4.3.9-Ubuntu started.
Copyright Andrew Tridgell and the Samba Team 1992-2015
STATUS=daemon 'nmbd' finished starting up and ready to serve connections
EOF on stdin
Got SIGTERM: going down...

Running the container manually not in daemon mode works docker run --rm -it -v $PWD/etc/samba:/etc/samba ewindisch/samba-nmbd $@ but this is less than ideal.

+1

Smbd exits when it receives an EOF,. Dockers puts EOF on the stdin when it starts a daemonized container. Use a wrapper to avoid smbd receiving EOF.
CMD /usr/sbin/smbd < /dev/null

@hzsolt94 great, can you make that a PR?

@ewindisch Sorry, but I'm not using your repository, so I can't properly test this out to make a PR.
Something like that will do:
samba-smbd/Dockerfile
@@ -6,5 +6,4 @@
 expose 445
 expose 139
 expose 135

-entrypoint ["/usr/sbin/smbd"]
-cmd ["-F", "-S"]
+cmd /usr/sbin/smbd -FS < /dev/null

@episage  the latter works for me, took me a min to figure out wtf was going on then I realized how counter intuitive it was to comma separate parameters if you don't have to
#CMD [ "/usr/sbin/smbd", "-F", "-S", "-s", "/etc/samba/smb.conf", "-d3", "<", "/dev/null"]
CMD /usr/sbin/smbd -FS -d3 < /dev/null

Funny thing. I found myself facing exactly the same problem 3 years later haha
still doesn't work for me haha

Funny thing. I found myself facing exactly the same problem 3 years later haha
still doesn't work for me haha

FYI --no-process-group solves the problem
smbd -FS --no-process-group

FYI --no-process-group solves the problem
smbd -FS --no-process-group


Can not do a function waveform of mic or audio

Is it can do a function like this:
if waveform is low then hide image, else show image.
repeating function on when playing audio
Example:

Hey, this is not a main use case of Pizzicato. However, you can likely achieve it by attaching an analyserNode to the Pz.Sound object and checking the values returned by the analyserNode to trigger the show/hide of the image.
You have an example of attaching an analyser node to a Sound object here: https://github.com/alemangui/pizzicato#connecting-nodes-to-a-pizzicatosound-object


fix(core): correctly space full width checkbox and radio

Purpose
Fixes Checkbox inside containers that force its width to be larger than its content.

Approach
Make the checkbox indicator use only its own width (auto) and the label use the rest of available space (1fr).
Testing
Can't reproduce on Storybook 🤔
But essentially a display: grid parent and a Checkbox inside.
Risks
None.

Don't need it right now, but might be good to release a patch including other fixes soon.
I'll update the commit message to what you suggested when merging.

We have v1.1 scheduled to be released next Friday, however we can do it also earlier if all current form components are good to go.


Error: Expected Doctrine\ORM\Query\Lexer::T_FROM, got  EntityName  since upgrade to 2.6.0

I have the following builder request
          return $builder
              ->select('e')
              ->from('AppBundle:Event', 'e')
              ->addSelect('MIN(o.startTime) AS HIDDEN')
              ->leftJoin('e.occurences', 'o')
              ->where('o.startTime BETWEEN CURRENT_DATE() AND :date ')
              ->andWhere('e.published = true')
              ->andWhere('e.place = :place')
              ->groupBy('e.id')
              ->setMaxResults(10)
              ->setParameter(':date', $date)
              ->setParameter(':place', $place)
              ->getQuery()
              ->getResult()
          ;

it used to work fine,  but since upgrade to 2.6.0 I got the error
[Syntax Error] line 0, col 42: Error: Expected Doctrine\ORM\Query\Lexer::T_FROM, got 'AppBundle:Event'

It seems to work if I add  FROM like this :
              ->from(' FROM AppBundle:Event', 'e')

is it a known issue ? (or something wrong in my original code ?)

What's the generated DQL string for that query builder?

how do i get it ?

Instead of ->getQuery() do a ->getDQL()

On master, QB generates following DQL:
SELECT e, MIN(o.startTime) AS HIDDEN FROM AppBundle:Event e LEFT JOIN e.occurences o WHERE (o.startTime BETWEEN CURRENT_DATE() AND :date ) AND e.published = true AND e.place = :place GROUP BY e.id

So there is obviously missing alias for MIN() function.

"HIDDEN" is a reserved DQL keyword
On 21 Dec 2017 17:35, "Michael Moravec"<EMAIL_ADDRESS>wrote:

On master, QB generates following DQL:
SELECT e, MIN(o.startTime) AS HIDDEN FROM AppBundle:Event e LEFT JOIN e.occurences o WHERE (o.startTime BETWEEN CURRENT_DATE() AND :date ) AND e.published = true AND e.place = :place GROUP BY e.id
So there is obviously missing alias for MIN() function.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/issues/6932#issuecomment-353396548,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakMI1jPG4A6MrbXMtaOrZSgPm1YZwks5tCojhgaJpZM4RJ-e3
.

"HIDDEN" is a reserved DQL keyword

Yes, but the query is valid according to 2.5 EBNF:
SelectExpression        ::= (IdentificationVariable | ScalarExpression | AggregateExpression | FunctionDeclaration | PartialObjectExpression | "(" Subselect ")" | CaseExpression | NewObjectExpression) [["AS"] ["HIDDEN"] AliasResultVariable]

"AS HIDDEN" is fine and the alias should be optional as well (which sounds a bit strange though).
This is not a QueryBuilder bug which generates same DQL as on 2.5, but a parser bug.
LanguageRecognitionTest test case:
    public function testOmittingAliasVariableInSelectExpression() : void
    {
        $this->assertValidDQL('SELECT g, LENGTH(g.from) AS HIDDEN FROM ' . DQLKeywordsModelGroup::class . ' g');
    }

Works with 2.5, fails with 2.6.
Bisected to b7bd42638dcf8aa049698f01466688ff7762953b so it's related to #6928.

Not, the EBNF has  [["AS"] ["HIDDEN"] AliasResultVariable]. so AS HIDDEN is not valid, as you miss the AliasResultVariable

Fixed by https://github.com/doctrine/orm/pull/7077


Support cross compile to sbt 0.13.16 and 1.1.0

Changes

support of sbt 1.1.0
cross compile to sbt "0.13.16", "1.1.0"

Sweet.

Can we merge this PR? We could really use it in our projects.

@piotrglazar I dont have the force to merge so you have to poke some of the people of hmrc: https://github.com/orgs/hmrc/people

@tomasz-rosiek can we merge this PR? Thanks!

Bump! Also, sbt is now on 1.3.3.
@chotai @fayimora @konradwudkowski @LGoldfinch2 @michaelmeasures @robertbutacu @seane

Thanks for the PR, the plugin is now cross-built for sbt 1.3.4 as of version 6.0.0


distinct_keys doesn't actually remove duplicates

The following code removes duplicates, but doesn't shrink the vector to actually remove the duplicates that std::unique moved to its end:
https://github.com/jermp/pthash/blob/2607873d28c5467205b9962c89b30680f5b788b4/src/util.hpp#L92
The line should probably be:
keys.erase(std::unique(keys.begin(), keys.end()), keys.end());

Thank you @Kristine1975,
you're right and indeed I usually do what you suggested as keys.resize(std::distance(keys.begin(), std::unique(keys.begin(), keys.end())));. However, the subsequent push_back will cause doubling of the vector which is better to avoid.
Also, I noted that pushing consecutive values may cause overflow and actually insertion of duplicate keys.
So I changed strategy. See latest commit.
Let me know if everything works you.


fix-bug windows run error

Use os.sep.path replace '/',  windows sep is ''

Thanks for mentioning it. I have not tested the lib on the Windows machine and it is great for this PR to make a fix. Appreciate it!


Quick fixes

-fixed icon alignment for buttons on how to contribute page

Too many new css classes.


Custom secret phrase:QR/Link issue

If I chose Custom secret phrase from Advanced but didn't enter anything in that and just close that modal and Click on the Next button.Then I try to share with Link and also with QR.But the QR did not load and the link was not created.

https://user-images.githubusercontent.com/62699947/147364987-4bdfcdbf-1077-4797-a8c8-2d0e884d69d4.mp4

Now it does not proceed without entering the secret phrase and hint.It is showing error toast.

Verified this issue on dev app v2.0.68(410)


[Bug Report] GetLore() misuse

Discussed in https://github.com/Crazy-Crew/CrazyEnchantments/discussions/695

Originally posted by Koozu1 August 10, 2022
Describe the bug
Huge performance issue!
Versions (please complete the following information):

Minecraft Server Version: 1.19
Plugin Version: 1.9.4

Issue
Huge performance loss from converting new lores to legacy strings when using getLore method. the method Lore should be used instead because getLore is deprecated and slow. And in the second link, the getLore check is useless anyway if hasLore is also checked! 35% ticktime to this on 50 players
https://github.com/Crazy-Crew/Crazy-Enchantments/blob/b8c3bc5cea71a076b234c3bbab1495982395788a/src/main/java/com/badbones69/crazyenchantments/api/CrazyManager.java#L554-L570
https://github.com/Crazy-Crew/Crazy-Enchantments/blob/b8c3bc5cea71a076b234c3bbab1495982395788a/src/main/java/com/badbones69/crazyenchantments/Methods.java#L388-L390
Screenshots

I already fixed it on a private build.

Have this exact same issue indeed. However it is using a lot more of the tick count causing the tps to drop to 2-6. Only happens when 1 player is mining. Does not happen with everyone and most likely involves some way to send more packets, but still a huge performance issue. Are there any updates on this so far?

@linsaftw Do you have a dev build or something that fixes this that we can use? This performance bug makes the server unplayable with 2 tps. Ram and CPU usage are normal

 Have this exact same issue indeed. However it is using a lot more of the tick count causing the tps to drop to 2-6. Only happens when 1 player is mining. Does not happen with everyone and most likely involves some way to send more packets, but still a huge performance issue. Are there any updates on this so far?

@linsaftw Do you have a dev build or something that fixes this that we can use? This performance bug makes the server unplayable with 2 tps. Ram and CPU usage are normal

Never had the problem before nor am I able to reproduce it, so...

Make Sure you are using the latest build for PaperMC and that you have the latest plugin version (from Modrinth).
Make sure that you are on 1.19.* when you do it and send the exact version and build that you are on.

Create a new test server.
Reproduce the problem.
Send a step-by-step on how to reproduce it here.

You need thousands of players online for the issue to appear.

None of us have a thousand players to reproduce it also no one should have that many players on a server ( just shooting yourself in the foot ) but that's besides the point.
You got involved with the issue in the first place to say you fixed it in a private build and said something vague about how to fix it but you could've just made a pull request like any other sane person would've.
It doesn't matter though anymore, We're just removing lore checks etc in the next few months.

The conversation is locked since it's just here for us to close when we remove lore/name checks.


feat: add contributing md

feat: add contributing md

@hanyuancheung Are you planning to follow up on this PR?

@hanyuancheung Are you planning to follow up on this PR?

Sure, I'll fix the comment above.

I've fixed, PTAL again, thanks. @pellared


Swift compiler can't determine the type of a closure set in a ternary operator

Describe the bug
When assigning a property to a closure using a ternary operator, the compiler complains that the "Type of expression is ambiguous without more context", even when the type is explicitly set.
Steps To Reproduce
Steps to reproduce the behavior:
       var onTapChange: (@MainActor () -> Void)?

        let action: (() -> Void)? = { [weak self] in
            guard let self = self else { return }
            self.delegate?.doSomething()
        }

        onTapChange = options.count < 2 ? nil : action      <--- Type of expression is ambiguous without more context

Expected behavior
The compiler should be satisfied because all of the types are explicitly declared
Screenshots
N/A
Environment (please fill out the following information)

OS: MacOS 12.6
Xcode Version/Tag/Branch: Xcode 14.1 beta 3

Additional context
Above is an attempt to be more explicit with the types. Our previous code, which has no issues in 14.0, creates and sets the closure inline in the ternary operator.
       var onTapChange: (@MainActor () -> Void)?

        onTapChange = options.count < 2 ? nil : { [weak self] in      <--- Type of expression is ambiguous without more context
            guard let self = self else { return }
            self.delegate?.doSomething()
        }

Reduced:
let x: @MainActor () -> Void

let y: () -> Void = {}

x = true ? y : y


Ubuntu who command

Environment
Windows build number: Microsoft Windows [Version 10.0.19042.685]
Your Distribution version: 20.04
Whether the issue is on WSL 2 and/or WSL 1: WSL 2

Steps to reproduce

WSL logs:
Expected behavior
These command should be showing me Username of Ubuntu
Actual behavior
who and logname command is not working on WSL 2 Ubuntu

I can confirm that above commands don't work.
But id and whoami seem to work correctly.

I can confirm that above commands don't work.
But id and whoami seem to work correctly.

/dupe #913 ➡ #573

/dupe #913 ➡ #573


[tests] Add test for new oraclize call back structure to deal with gas checks

it('gas used by Oraclize callback is within specified limit', async function() {
    const marketContractExpirationInTenSeconds = Math.floor(Date.now() / 1000) + 10;
    const priceFloor = 1;
    const priceCap = 1000000; // 10k USD
    gasLimit = 6500000; // gas limit for development network
    let block = web3.eth.getBlock('latest');
    if (block.gasLimit > 7000000) {
      // coverage network
      gasLimit = block.gasLimit;
    }
    let txReceipt = null; // Oraclize callback transaction receipt

    MarketContractOraclize.new(
      'ETHUSD-10',
      [ accountMaker, marketToken.address, collateralToken.address, collateralPool.address],
      [priceFloor, priceCap, 2, 1, marketContractExpirationInTenSeconds],
      'URL',
      'json(https://api.kraken.com/0/public/Ticker?pair=ETHUSD).result.XETHZUSD.c.0',
      { gas: gasLimit, accountMaker }
    ).then(async function(deployedMarketContract) {
      let oraclizeCallbackGasCost = await deployedMarketContract.QUERY_CALLBACK_GAS.call();
      let contractSettledEvent = deployedMarketContract.ContractSettled();
      let updatedLastPriceEvent = deployedMarketContract.UpdatedLastPrice();
      marketContractInstance = deployedMarketContract;

      contractSettledEvent.watch(async (err, response) => {
        if (err) {
          console.log(err);
        }
        assert.equal(response.event, 'ContractSettled');
        txReceipt = await web3.eth.getTransactionReceipt(response.transactionHash);
        console.log('Oraclize callback gas used : ' + txReceipt.gasUsed);
        assert.isBelow(
          txReceipt.gasUsed,
          oraclizeCallbackGasCost.toNumber(),
          'Callback tx claims to have used more gas than allowed!'
        );
        contractSettledEvent.stopWatching();
      });

      updatedLastPriceEvent.watch(async (err, response) => {
        if (err) {
          console.log(err);
        }
        assert.equal(response.event, 'UpdatedLastPrice');
        await web3.eth.getTransactionReceipt(response.transactionHash);
        updatedLastPriceEvent.stopWatching();
      });
    });

    let waitForContractSettledEvent = ms =>
      new Promise((r, j) => {
        var check = () => {
          if (txReceipt) r();
          else if ((ms -= 1000) < 0) j(new Error('Oraclize time out!'));
          else {
            setTimeout(check, 1000);
          }
        };
        setTimeout(check, 1000);
      });

    await waitForContractSettledEvent(60000);
    assert.notEqual(
      txReceipt,
      null,
      'Oraclize callback did not arrive. Please increase QUERY_CALLBACK_GAS!'
    );
  });

@pelsasser about this issue. Should the code in the description be added to the codebase? Or perhaps you should update the issue description.

@perfectmak sorry yea not clear.  I was moving fast when I created this issue.
The goal here is to take this old test that @cryptomental wrote and adapt it to the new OracleHub to ensure everything works there as expected with regards to the amount of gas needed for the callback.
make sense?

Hmmm... Okay. Make sense.


Automate your network rule creation - Powershell Example

Section #--automate-your-network-rule-creation would benefit from a PowerShell cmdlet example:
Add-AzStorageAccountNetworkRule -ResourceGroupName <ResourceGroupName> -Name <StorageAccountName> -VirtualNetworkResourceId <ResourceID[]>
where <ResourceID[]> is the list of network identities copied from the NCC -> Default Rules -> Network identities (View all) by pressing the "Copy subnets" button.
Add-AzStorageAccountNetworkRule -ResourceGroupName mystorage-rg -Name myaccount -VirtualNetworkResourceId /subscriptions/8453a5d5-9e9e-40c7-87a4-0ab4cc197f48/resourceGroups/prod-azure-eastusc3-nephos2/providers/Microsoft.Network/virtualNetworks/kaas-vnet/subnets/worker-subnet, /subscriptions/8453a5d5-9e9e-40c7-87a4-0ab4cc197f48/resourceGroups/prod-azure-eastusc3-nephos3/providers/Microsoft.Network/virtualNetworks/kaas-vnet/subnets/worker-subnet

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

ID: 7ff8d539-1320-7205-e2a0-867545a73217
Version Independent ID: 7ff8d539-1320-7205-e2a0-867545a73217
Content: Configure a firewall for serverless compute access - Azure Databricks
Content Source: databricks/security/network/serverless-network-security/serverless-firewall.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

Hello @ian-barlow
Thanks for your contribution. Please add your ideas in below link, so our production team can review it and update the same.
Ideas · Community (azure.com)

Thanks for providing feedback that helps improve our documentation. We've created an internal work item (DOC-13580) to address your feedback. The timeline for resolution varies based on resourcing.
#please-close


Add Ubuntu 24 to tested/supported versions

Add Ubuntu 24 numbat to tested and documented-supported versions.
Remove EOL OS versions from readme.

https://endoflife.date/ubuntu
https://endoflife.date/debian

Ensure versions meet lint requirements of
https://github.com/ansible/ansible-lint/blob/main/src/ansiblelint/schemas/meta.json#L1102
Requires update of plugin version
https://github.com/ansible/ansible-lint/releases

Confirming that CI is passing: https://github.com/s01ipsist/ansible-dokku/actions/runs/11790959668


Related objects with no ids

Yes - I will also remove 2.7, but I just wanted to get rid of 3.4 for now, since it was causing the Travis build to fail


[CI] PHP CS Fixer it up

hm, tests started failing because of the PHPStan... not sure why :/

composer up does not help, seems the phar file conflicts with the package, not sure how to overcome it at the first sight

There's something weird with that PHPStan test, I just skipped it in 6b580f4 until a better time. In short, as far as I understand, PHPUnit (which is installed via Symfony PHPUnit Bridge) has some dependencies that conflict with the ones installed with PHPStan - it loads the incorrect file from the PHPUnit deps instead. I'm not sure about a possible solution. Probably we should get rid of the Symfony PHPUnit bridge in favor of direct PHPUnit dependency in the require-dev? But that's a bummer to go this way, but I don't see other ways so far. If anyone could give it a try - please, PR is warmly welcome!


Route behavior in high fan-out

Setup: server A has a route to server B.
B has thousands of subscriptions.
When A sends a message to B, B in turn matches all subscriptions and send. This may result in a bit of time spent doing this. While doing so, socket between A and B is not drained. A may continue to send to B. When socket buffer is full, A will disconnect B as a slow consumer.
If all subscriptions were to be on a single server, and each individual send is not blocking for more than the write deadline, everything would work fine.

As an update I have been digging into this quite heavily. Hope to have something soon.


Fixes LGTM: Command line string concatenation

Contains
Fixes the LGTM "Command line is built with string concatenation" error described in #3510.
Fix as per rule described here.
How to test
Create a game with the Advanced mode. Select the "Module Details" screen and hit the "Open in browser" button. This seems to be the only usage of UIButtonWebBrowser so far.

This pull request fixes 3 alerts when merging 72e6e1cde503ea488458b459044a01be838497ed into 9118708ad3f34f0bc05572fe54ad47538c3164bb - view on LGTM.com
fixed alerts:

3 for Building a command line with string concatenation

Comment posted by LGTM.com

Github is working weirdly for me. I can't see this pull request on Github
anymore. Anyway, I closed this because it has already been fixed in #3512.
Please beat me to closing #3515 and #3516 if you can.
Regards,
Nihal Singh
On Mon 22 Oct, 2018, 1:48 PM Rasmus Praestholm<EMAIL_ADDRESS>wrote:

This pull request fixes 3 alerts when merging 72e6e1c
https://github.com/MovingBlocks/Terasology/commit/72e6e1cde503ea488458b459044a01be838497ed
into 9118708
https://github.com/MovingBlocks/Terasology/commit/9118708ad3f34f0bc05572fe54ad47538c3164bb

view on LGTM.com
https://lgtm.com/projects/g/MovingBlocks/Terasology/rev/pr-7d81d1ec814944421f4b7148429cc71cce40042c

fixed alerts:

3 for Building a command line with string concatenation

Comment posted by LGTM.com https://lgtm.com
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/MovingBlocks/Terasology/pull/3516#issuecomment-431765484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANRZYRU5sWm_oTTO2Eutbp4cnAIwxXW_ks5unX9QgaJpZM4Xy9Ji
.

Github has been having issues all day...


[Feature Request] Eloquent Api Resource Pagination

We should also support pagination for api resources
https://laravel.com/docs/master/eloquent-resources#pagination
Should we add another annotation like @apiResourcePagination
There are 2 methods for pagination ( https://laravel.com/docs/5.8/pagination )
simple and default
So
@apiResourcePaginate resolves to Model::paginate(15);
@apiResourcePaginate simple resolves to Model::simplePaginate(15);
What do you guys think about?

The same could be said for @transformerCollection with @transformerModel, if you were to use code like this in your controller:
$transformer = new Collection($categories, new CategoryTransformer(), 'data');
$transformer->setPaginator(new IlluminatePaginatorAdapter($categories));

return $fractal->createData($tranformer)->toArray();

The response should be similar to {"data": [ {category}, {category} ], "meta": ...}, but there's no way to configure pagination rules for Fractal transformers right now.

+1

This is now supported in Scribe 🙂.

Docs on paginating API Resources
Docs on paginating Transformers

@shalvah could you please update the new documentation for this paginate feature? thanks!

@mehrancodes Did you read the last few comments? This package is no longer maintained.

@mehrancodes Did you read the last few comments?

Whoops, I see those links are broken. You can just go to scribe.rtfd.io and search for "API Resources".


Run port scans in parallel subprocesses.

Suggestion:
Most scanned ports are closed. Scanning of a closed port scan takes the TWA_TIMEOUT seconds, default 5 seconds.
Therefore, the most time used by the port scan is wait time.
Running the port scans in sub-processes takes a bit more resources (memory) but reduces the run-time of the script remarkable, typical 50 seconds.

Pros: reduces run-time significant
Cons: sub-processes are not a tiny tool (but only a tiny &, because bash does the work behind)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


botamusique: use same ffmpeg as yt-dlp

...to save the runtime closure
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

I wish there was an easy way to implement automatic checks that we're not using multiple variants of the same things in the closure


Event-ish refactor: suspendable listeners and this. namespaced events - RFC

Description of the pull request:
This does a few different-yet-related things:
Silence-able observers
There's been an open issue for a while requesting the ability to temporarily stop observers from firing  (#1182), so this adds silence(), isSilenced(), and resume() methods to the handle returned from ractive.observe().
This could conceivably be extended to cork-like functionality where changes are stashed until uncorking (probably with resume too) or a different form of silence where the observer ignores notification, and thus, keeps the old value in place for future resume. All of that seems a bit YAGNI for now though, whereas built-in locking for observers would be handy in a number of pretty common scenarios. Opinions?
Silence-able events
Similar to observers, there's been a request around for a while to temporarily stop events from firing (#2323), so this adds the same silence(), isSilenced(), and resume() methods to the handle returned from ractive.on(). Again, I went with silence because nothing is torn down while the listener(s) is silenced; they just don't fire. It also mirrors the observer version nicely.
Event listener other stuff
This also fixes a bug with multiple events subscribed with the same listener at the same time e.g. this.on('foo bar baz') wherein the returned handle's cancel() method wouldn't actually cancel anything.
It also refactors the on() method to be a bit less branchy, like the recent observe() refactor.
Events with an initial implicit this namespace
One of the things that seems to be a bit counter-intuitive when first dealing with event bubbling is that ractive.on('foo') and ractive.on('*.foo') won't both be triggered by ractive.fire('foo') (#2566). It seems that the general assumption is that *.foo means all the foos, which makes sense to me too. So this adds an implicit this. namespace to all intial events, meaning that ractive.fire('foo') will trigger any of these listeners: foo * this.foo *.foo this.*. It notably won't fire *.* because that would mean that any lifecycle events would start triggering *.* listeners. That may not be a bad thing, but it breaks tests. I'm not sure that it's intuitive behavior, and I can't really see how all-wild listeners are particularly useful anyways. Opinions?
I really wasn't aware of this before I started poking around this code, but you could already fire namespaced events on your own by using names with .s in them. Given that, I considered adding ** flavors of listeners that would "consume" multiple path segments in the ** portions, but for now that seems a bit YAGNI too. Again, opinions?
This refactor also (kinda) removes one of the vestiges of versions < 0.8, - the star map. The star map helpers were only used by events as of 0.8, as the Model family took over the matching of wildcard keypaths. The star map helpers basically create a cartesian product of a keypath and an equal-length all-star keypath ( a.b.c <-> *.*.* ), which is still needed for finding matching event listeners. So there's now a slightly simplified, still-caching (since they're not exactly cheap to create and lifecycle events mean that they're not exactly rare - I'm looking at you change) version of star maps moved into the event firing code.
Fixes the following issues:
#1182 #2323 #2566
Is breaking:
Kinda sorta. If you were relying on foo and *.foo listeners catching only this instance events and only child component events respectively, this is breaking. Otherwise, which seems to be the more intuitive assumption based on issues and SO questions, this would actually be a bugfix. If *.foo must only be triggered by child events, you can now check the source ractive against this to be sure they're different.
Reviewers:
Yes, please.

It seems that subscribing multiple events as object map fixes also https://github.com/ractivejs/ractive/issues/1458

@mefjuu it does now :smile:.
I've added changes to resolve #1458:
Events hash
With this, you can specify an on hash in your options that takes key/value pairs. The keys may be anything that's valid to pass to ractive.on(), so you can stick the same handler on multiple events with a space separated list e.g. 'foo bar baz.bat'. The values may be functions or an object with a handler function and other options. At this point "other options" for events is just once: true, which subscribes the event using ractive.once() instead of ractive.on().
Components/extended instances add to their parent event hash, so
const cmp = Ractive.extend({ on: { foo() { ... } } });
const r = new cmp({ on: { foo() { ... } } });
r.fire('foo');

will trigger the foo methods from both configs. The reasoning behind this is that the cmp component set the handler up for a reason, and as such, it should not be overridden accidentally by an extension or instance. Handlers are installed from root component to instance, so the root handlers are called first. If necessary, we could specify that one of these listeners returning false would halt the subscriber notification, but I don't think that will really come up.
Observers hash
As with events, you can specify an observe hash in your options that takes key/value pairs. Like events the keys can be any string that can be passed to ractive.observe(). Also like events, the values may be a function or an object with a handler function and additional options. Here's an example with each:
new Ractive({
  observe: {
    foo() { /* I will fire at init as the observer initializes */ },
    'bar baz': {
      handler() { /* I will not fire at init, and I will not fire until dom manipulation is done */ },
      init: false,
      defer: true
    },
    bat: {
      handler() { /* I will only fire once when bat first changes */ },
      once: true
    }
  }
});

As with events, the observer hash is additive, so base component observers and instance observers are all subscribed, with the base observers subscribing first.

Yasss! Events hash! I remember requesting that one. Few questions:

I assume events and observers are taken names already? Would be more intuitive though.

Can lifecycle events be put there? Or are those specifically for user-defined events?

I encountered this._super() while reading the components docs. It appears to be used by lifecycle events to call overridden parent lifecycle events. Would it make sense to do the same for regular events?

And a few other things:

A warning as to why *.* is not supported might be good though.

Starmap... the code that generates all possible combinations of keypaths given a starred keypath? 😁

events is taken. I went with on and observe because they match the method name, and it seemed fitting as they are just shortcuts for calling on and observe after the config event fires.
Any old event can be here, including lifecycle and wildcard.
I think the _super() is only for dealing with the onwhatever method flavor of event handling. With this, that stuff could actually go away. Even if you don't call this._super() in an onwhatever method, the event will still trigger an listeners that super may have installed.
Warn about not supporting it, or just don't special-case it and note it as a breaking change?
Well sort of :smile:. It's all possible matching starred keypaths for a given absolute keypath. The star map for a.b.c is [ 'a.b.c', '*.b.c', 'a.*.c', 'a.b.*', '*.*.c', '*.b.*', 'a.*.*', '*.*.*' ] - or the cartesian product of a.b.c and *.*.*. It's used when a namespaced event fires, say foo.bar, to find any listeners that could match, so when you have ractive.on('*.bar', ...), the foo.bar event will call that listener.

Addenda:

I also went with on and observe to keep them consistent because events is taken.
Any old event, except things that take place on or before onconfig :smile:. I think I could move the subscription point to just before onconstruct, but I would probably have to split events and observers up. Should I do that?


[C14877] object null is not iterable (cannot read property Symbol(Symbol.iterator))

@seveibar tsx cli/main.ts convert -i C14877 --output-format tsx seems to be working fine for me

@andrii-balitskyi i think i fixed this one when i fixed a different issue- thanks for confirming!

@andrii-balitskyi also there is now a script in the scripts directory to automatically download and generate a test for a part number- it's pretty cool!

@seveibar, that's awesome! Thanks for creating the script!


CI improvements

@tancredi there's no matrix.os in lint

Also, you could probably remove the cache fallback line ${{ matrix.os }}-node-v${{ env.NODE }}-

Oopsy - https://github.com/tancredi/fantasticon/pull/112
TBH I'd almost remove fallbacks entirely - i'd expect cache keys to be either correct or not, but it kinda just makes the thing more unreadable in a way. What do you think?

I'd say just remove the aforementioned fallback and keep the exact match. Better have a cache miss that cache growing, assuming you it's not a lot slower.

But why that one specifically and not others?

The other one is not a fallback. It's needed so that the cache action knows what to restore AFAICT.

Meh - still not totally convinced as I think caching here is far less critical than maintainability and deffo don't see the point in a fallback key (I'd either hit exact key or not hit anything ay all), for the time being just removed that line :)

Like I said, the only fallback there is is that line wherever repeated. The other line is needed for caching. If you don't want caching, that's another thing, but it definitely speeds things up.


Linux Executables with .exe suffix are killed running in Window Subsystem for Linux

In configurator.ml, in the compile_c_prog function, there is the line:
let exe_fname = base ^ ".exe" in
which gives the executable a ".exe" suffix even if running on linux. When the linux is running as part of Windows Subsystem for Linux, Windows is apparently trying to run the program as a windows program and the result is that the program is "killed" on startup before reaching the main program.
I'm running the beta version of WSL that comes with the Creator's Update, (windows version 1703) which is running ubuntu 16.04.
To test this out, I took the failing executable and renamed it to a name not ending with .exe and it executes file.
I suggest the following change:
type t on line 16 already has a field "ext_obj" for the object file extension. A "ext_exe" field should be added as well (the ocaml config already has this) and it should be used instead of the hardwired ".exe".

This looks terrifying to me as Dune uses the .exe extensions for all executables during the build. Do you know if this crash is a bug of the beta version of WSL? I can't imagine WSL would forbid all linux programs with the .exe extension.
/cc @rgrinberg @dra27
BTW, this repository will soon be completely deprecated as we integrated configurator directly into Dune.

I don't know whether this is a WSL beta bug or not. I've not tested this particular mode of failure on a non-beta WSL (we have not upgraded to the version of Windows that has the supported WSL). I would think that WSL could tell by looking at a file whether it was a Windows .exe file vs. a linux executable.

The failure also seems to depend on the location of the executable. For example, when I tried it in the Windows part of the file-system it worked ok. But when the file is located in the linux /tmp directory it fails. Of course since configurator puts its little test programs below /tmp this is a problem.
Here is the test program that was causing the problem:
#include <stdio.h>
#include <caml/config.h>
int main()
{
#if defined(ARCH_SIXTYFOUR)
printf("ARCH_SIXTYFOUR=b:true\n");
#else
printf("ARCH_SIXTYFOUR=b:false\n");
#endif
return 0;
}
The test was compiled (if I recall properly) using
ocamlfind ocamlc -o test.exe -package base,stdio,configurator -linkpkg
I had done a byte-code compile in order to try to diagnose were it was failing, but it actually failed before getting to the first instruction.

Ok, so if it's only when the executable is in /tmp then it's weird but not that bad. The version of configurator that is in Dune should be safe then because it no longer need to actually run the executable.

WSL is supposed to determine whether to run an executable using Windows based on the content of the file, not the extension (I can't find a reference to the announcement, but it's related to how #! work in the kernel IIRC). If this is still an issue, worth opening on issue on Microsoft's WSL tracker.


'Refused to set unsafe header' - interceptor installed

I need to send requests including restricted headers like Cookie, Host. I've seen http://www.getpostman.com/docs/requests on restricted headers. Interceptor is installed (but not on, as per doc?) 0.2.8 according to Chrome. Using Postman packaged app 1.0.7.
Chrome dev tools console explicitly complains about the headers:

Not sure if relevant, but the only other error in console is:

For completeness, I also reported an issue with the Send button event handler - I'm using the fix mentioned in this thread, to re-attach the handler:
https://github.com/a85/POSTMan-Chrome-Extension/issues/712

You need to click on the interceptor icon in Postman (traffic light icon in the top right) to route requests through the interceptor.

Oh, OK, misunderstood the doc - it says you should toggle 'again' but presumably that's if you've just installed it, to turn it on. Not to turn it back off again. Sorry.


Use WinGet Releaser to package to WinGet

Hi, I would suggest using WinGet Releaser to submit future packages for this project to winget-pkgs.
It is a GitHub actions workflow that will run on every release, retrieve the installer from the GitHub release assets and automatically create a PR to winget-pkgs with the new update (from your account with a token). I saw that with 1.3.2 you tried to overwrite the previous version on winget-pkgs - I'm not sure whether this is intentional but WinGet Releaser will allow you to do this if you do only wish to have the latest version on winget-pkgs.
It would look like this:
name: Publish to WinGet
on:
  release:
    types: [released]

jobs:
  publish:
    runs-on: windows-latest
    steps:
      - uses<EMAIL_ADDRESS>        with:
          identifier: Xanthus58.ValorantRandomizer
          installers-regex: '\.exe$'
          delete-previous-version: 'true' # Optional
          token: ${{ secrets.WINGET_TOKEN }}

I'm happy to make a PR for this change to this repo :)

Hey i love this idea if you're happy to make a PR I'd be more then thankful

PR Incoming! 🚀


Want to restore my wallet, but says "This is a wallet, but is not a 1wallet address"

Hi, I would like to restore my wallet, but when I enter my address in the filed, I have a pop up that says "This is a wallet, but is not a 1wallet address".
I'm 100% sure this is my 1wallet address, I already restored in in the past.

Can you post the address here?

Any update? Closing for now


fix resolve path for windows

This is the fix mentioned in the comments of #5329

I signed it!

I signed it!

I signed it!


Support arbitrary Java types in base-jdbc

This is necessary to handle LongTimestampType in base-jdbc.

CI failed -- https://github.com/prestosql/presto/issues/3943


Refactor gossipsub message-id

Currently the gossipsub message-id is specified as:
message-id: base64(SHA256(message.data))

The base64 is there since in the earlier gossispub spec versions message-id had to be a string but this changed a while ago (see https://github.com/libp2p/specs/commit/8e5da407893d6e340b7ec68d849cb57b19449671#diff-b6f4ac67de1a41957e8f97f5728669b7), so now message-ids can be arbitrary bytes. I therefore propose changing the message-id to simply:
message-id: SHA256(message.data)

This will save us valuable bandwith (especially for IHAVE and IWANT messages that contain mostly only message ids). Furthermore, it is in my opinion cleaner than using the byte representation of a base64 string.
As we recently found out, until now Prysm used a different message id than defined in the spec (see https://github.com/prysmaticlabs/prysm/pull/7177) which was not really a problem for Medalla. We only found out about the inconsistency during testing gossipsub peer scoring. In fact if no peer scoring is active the only problem with inconsistent message id functions is that there will be more gossipping happening (since message ids in IHAVE will not coincide with the computed message ids of the received messages).
So I propose to do this change before most of the clients activate gossipsub scoring which would penalize inconsistent message id functions and would make it harder to switch.

Agree! I think all the clients can implement it asynchronously.

LGTM!

Looks good to me. Thanks!


Appdaemon Starting fail loop

Problem/Motivation

Unable to start Appdaemon

Log
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service base-addon-banner: starting
-----------------------------------------------------------
 Add-on: AppDaemon
 Python Apps and Dashboard using AppDaemon 4.x for Home Assistant
-----------------------------------------------------------
 Add-on version: 0.12.2
 You are running the latest version of this add-on.
 System: Home Assistant OS 10.1  (amd64 / generic-x86-64)
 Home Assistant Core: 2023.4.6
 Home Assistant Supervisor: 2023.04.1
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums or the Discord chat.
-----------------------------------------------------------
s6-rc: info: service base-addon-banner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service base-addon-log-level: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service base-addon-log-level successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service init-appdaemon: starting
s6-rc: info: service init-appdaemon successfully started
s6-rc: info: service appdaemon: starting
s6-rc: info: service appdaemon successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
[01:25:32] INFO: Starting AppDaemon...
2023-05-03 19:25:42.759454 INFO AppDaemon: AppDaemon Version 4.4.2 starting
2023-05-03 19:25:42.760987 INFO AppDaemon: Python version is 3.10.11
2023-05-03 19:25:42.762226 INFO AppDaemon: Configuration read from: /config/appdaemon/appdaemon.yaml
2023-05-03 19:25:42.763285 INFO AppDaemon: Added log: AppDaemon
2023-05-03 19:25:42.764355 INFO AppDaemon: Added log: Error
2023-05-03 19:25:42.765285 INFO AppDaemon: Added log: Access
2023-05-03 19:25:42.766282 INFO AppDaemon: Added log: Diag
2023-05-03 19:25:43.276447 INFO AppDaemon: Loading Plugin HASS using class HassPlugin from module hassplugin
2023-05-03 19:25:43.708868 INFO HASS: HASS Plugin Initializing
2023-05-03 19:25:43.709334 INFO HASS: HASS Plugin initialization complete
2023-05-03 19:25:43.709984 WARNING AppDaemon: ------------------------------------------------------------
2023-05-03 19:25:43.710421 WARNING AppDaemon: Unexpected error during run()
2023-05-03 19:25:43.710785 WARNING AppDaemon: ------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/appdaemon/__main__.py", line 131, in run
    self.AD = ad.AppDaemon(self.logging, loop, **appdaemon)
  File "/usr/lib/python3.10/site-packages/appdaemon/appdaemon.py", line 250, in __init__
    self.plugins = plugins.Plugins(self, args)
  File "/usr/lib/python3.10/site-packages/appdaemon/plugin_management.py", line 103, in __init__
    self.plugins[name]["refresh_delay"] = 600
TypeError: 'str' object does not support item assignment
2023-05-03 19:25:43.714877 INFO AppDaemon: Previous message repeated 1 times
2023-05-03 19:25:43.715212 INFO AppDaemon: AppDaemon Exited
[01:25:44] INFO: Service AppDaemon exited with code 0 (by signal 0)
[01:25:45] INFO: Starting AppDaemon...`

Expected behavior

Appdaemon should start

Actual behavior

Appdaemon start fail loop

Steps to reproduce

Uninstall Appdaemon
Install Appdaemon via Addon Store
Click Start

It is fixed after updating to Home Assistant Core 2023.5.0


feat: support float

Closes #11

Thanks @wesleimp for the PR! ✨ This looks good :) I approved the CI jobs so as soon as it's green we'll merge.


Handle props and URL params in MapsIndoorsMap component

What
Refactor the handling of props and query parameters to be the responsibility of the MapsIndoorsMap component.

There are still some places where we use hasURLParameters on the MapTemplate.jsx file. Do we still need that, or can we get rid of it completely on that file?

Do we need all the props defined on the MapsIndoorsMap.jsx component, or is it enough to have them on the MapTemplate.jsx?

Do we need all the props defined on the MapsIndoorsMap.jsx component, or is it enough to have them on the MapTemplate.jsx?

The MapsIndoorsMap is the "public facing" component (and web component), so it needs to be there and passed down to the child component, MapTemplate. Or am I misunderstanding? Can you elaborate?

Do we need all the props defined on the MapsIndoorsMap.jsx component, or is it enough to have them on the MapTemplate.jsx?

The MapsIndoorsMap is the "public facing" component (and web component), so it needs to be there and passed down to the child component, MapTemplate. Or am I misunderstanding? Can you elaborate?

The same props are defined on the MapTemplate and also on the MapsIndoorsMap files - and I thought they don't need to be present in both files, but it makes sense that they need to exist on the public facing component and web component too.

There are still some places where we use hasURLParameters on the MapTemplate.jsx file. Do we still need that, or can we get rid of it completely on that file?

It was my ambition to only have that in the MapsIndoorsMap component. I'll look into it.

@andreeaceachir142 I removed the unnecessary hasURLParameters from the MapTemplate component now.


Bi LSTM residual.

At https://github.com/smousavi05/EQTransformer/blob/01bf112a00c1a3bbb43adb2c7dba3e21b5fbcdb9/EQTransformer/core/EqT_utils.py#L2596
you claim it is a residual block, but it is not. What is correct here?

@Erik-BM this is a specific type of residual block called network-in-network block.

@smousavi05 ok, thank you for the clarification. Closing this.


⚠ GLDDNS Server (ns2.glddns.com) has degraded performance

In 3e5fa68, GLDDNS Server (ns2.glddns.com) (ns2.glddns.com) experienced degraded performance:

HTTP code: 200
Response time: 363 ms

Resolved: GLDDNS Server (ns2.glddns.com) performance has improved in 41cd23a after 7 minutes.


Imgui

Looks ready to merge ?
while(ShowCursor() > 0) {} result in an infinite hang and it seems to be working fine without it. I'm removing it for the time being.
I'm starting a new branch from here for OpenGL 4.3 compatibility (glDebugMessageCallback, and deprecation errors on old functions)

Looks ready to merge ?
while(ShowCursor() > 0) {} result in an infinite hang and it seems to be working fine without it. I'm removing it for the time being.

ah, this was an error, i forgot that this is not needed in the glfw backend. normally the game enables/disables the cursor quite often, so it has to be enabled again to use imgui. but this only happens if dinput is used. i dont know why it should hang though 😅