select2 filter for booleans

I have setup a select2 filter for a boolean/tinyint(1) DB column:
$this->crud->addFilter([ // select2 filter
  'name' => 'is_online',
  'type' => 'select2',
  'label'=> 'Network Status'
], function() {
	return [
		0 => 'Offline',
		1 => 'Online',
		];
}, function($value) { // if the filter is active
	$this->crud->addClause('where', 'is_online', $value);
});

When used it works fine for selecting "Online" rows, but "Offline" shows all rows. How can one tweak this filter to show only the "Offline" rows (is_online == 0) ? I'd prefer not to use 2 simple type filters for each case.
Thank you for any advice on this! :)

Hmm this might be because 0 is resolving to false, so it doesn't detect a filter.
Could you possibly set your offline value to 2 and see if that works?
If it works as expected then there's a bug which needs fixing.
Thanks

Hi OwenMelbz,
Thanks for checking into this.
I set it to 2, and it does work. I would rather not keep this change as it breaks the "check" type Column in the list.

Oh sure, wouldn't expect you to :) but it's just confirming where the issues lays!
One last test, could you try changing your filter definition from 1/0 to true/false please and let us know what happens?
Thanks

@pushamaku I've put in a PR for @tabacitu to review and merge when he has time, I believe he's a bit sick at the moment so you might have to hold out! (maybe when he see's this/gets time he can also merge the table/video/address fixes in one go!)
https://github.com/Laravel-Backpack/CRUD/pull/292

Great thank you!


Fix quotes in report-results.py

With Python 3.10.12 there is a parsing error on that line. To fix the parsing error in the code, I would replace the double quotes around "unknown" with single quotes

Thank you!!! 👍


export Config from JsonDB

This is just a small QoL change that allows Config to be imported from JsonDB. It doesn't break anyone's existing projects.
Now you can just do import { JsonDB, Config } from 'node-json-db'.

@all-contributors please add @adamhl8 for code and doc


fixing support for verbose option ignoreTimestamp

In reference to PR #21 - I noticed due to the joi schema validation we're not really supporting the verbose --ignore-timestamp option but only the short syntax -i.
This commit fixes it by cleanly normalizing -i to --ignoreTimestamp.
Fixed and tested locally.

Coverage remained the same at 97.75% when pulling 2d2880b018864af1e3bdbe25659e4b779be9e612 on lirantal:fix/ignoreTimestamp-verbose-argument into 20d0c1b0ca32da574dc14ca2ef8cbf2c7516b9a4 on vinicius0026:master.

no problems @vinicius0026, thanks! :-)


General improvments to the documentation - in particular describing the model motivating the procedure

The definitions of some parameters in the workflow are missing. See essreflectometry/amor/typing.py.
Model motivating the footprint correction coefficient is missing.
Model motivating the supermirror calibration is missing.

Is this about adding the information (mathematical expressions) from the old notebook in the ess repo?
If so, a couple are missing from the list:

For the resolution function, there are references to an article, but we could also add the math for our docs
There is also a link to a reference for the conversion to theta, but the mathematical expression could be included

There is some discussion in #44 that might be useful for this issue.


IP address for Postgress

I've been trying connect to Postgres with PGAdmin using your Docker-Compose but won't connect to an IP address or Localhost. When I type in the command "docker ps", there's no address. When I inspect the config file (docker inspect name | grep "IPAdress") there's no address there.... in case there's something besides localhost or <IP_ADDRESS>.

You just need to map ports to your host or alternatively create a docker network and use that network to communicate between containers.


Initial delay before running script

Hello,
I'm trying to run some example scripts as desribed in GPIOZERO documentation. Is it normal to have a quite long (few sec) initial delay when executing a python script with some example code (for Motion Sensor, Distance Sensor ...)?

Yes - there's a lot of setup required, working out which pin factory to load, and that kind of thing, I'm afraid.


just this errrrrrrrrrrrrrrrrrrrrrr

just this errrrrrrrrrrrrrrrrrrrrrr

you have a stage channel in your server and the old version of discord.js this uses doesn't support them
might upgrade discord.js but i made this as a meme so i don't know if i will


Compiling Error with Texture2D on dev branch sometimes

I've used the 1.16 version without problems, but the 1.19 dev version fails to load with the errors
ERROR: 0:254: 'texture2D' : no matching overloaded function found (using implicit conversion)
ERROR: 0:254: 'texture2D' : function is not known
ERROR: 0:254: 'r' : field selection requires structure, vector, or matrix on left hand side

'

I've also gotten the texture2D errors on DartCat's fabulous shaders, but I dismissed them until I see this pack is in development still.
I assume this isn't a problem for everybody because otherwise you would have been unable to do other development on the pack.

pretty sure this is because your hardware doesn't support older versions of glsl.
i think the older version is being used because of gl_NormalMatrix? It seems the 1.19 dev rp isn't using core shaders, which i believe are neccesary to pass in that data in newer version since that gl built in doesnt exist anymore

I see a core shaders folder whose contents get modified in the dev rp. I did dismiss the other case as being from an old pack. I suppose that's an interesting change.
I thought I pasted it in for the original error report, but I got a message along the lines of forward compatibility (matching your theory), I'll try to find and paste that now. On the bright side, with the shader reload mod I get a cool entity effect (I assume with data) from before it's able to be passed into the in-that-case nonexistent post shaders.

'function' : is removed in Forward Compatible context texture2D

ah that's my bad. I am still in the process of upgrading to core 150. there are still some texture2D calls lying around. Nvidia is pretty lax when it comes to this.

should be resolved as of 9e2f49c. let me know if it is fixed and I will close.

@blank3times

Time to crash my computerrrrrrrrrrrrrrrrrrrrrr

Thank you! It loaded in the menu. I think I can close it if it loads in a world as well?

It's beautiful and I'm surprised I have more than 2 fps since I'm not using sodium for obvious reasons. This stuff is magic, thank you for fixing!

@bradleyq I don't know if you can see this since I haven't properly used github in the past, but I'm curious about version 330 and what's new (and/or optimized) about it (besides that mac won't support it), because I saw in that commit large deletions and large additions besides just changing the number. =]
Thanks again for removing Texture2D, it was a blast exploring the area I was in with the pack on!

@blank3times 330 itself does not give too much of an improvement. Maybe 1fps. I needed to upgrade for new GLSL api func  floatBitsToUint. This lets me store depth buffer info into a texture to sidestep this bug: https://bugs.mojang.com/browse/MC-254044
I also needed to reshuffle and add some passes to fix this, hence the deletions.

That's a helpful change. Thanks for the info! =] For optimization I thought maybe just making the code easier to write not playing fps, but I'm glad you can circumvent the bug which I would vote for if I had Mojira. Glad I'm not a mac user, heh heh.


Fluent-bit: Add log_to_metrics filter plugin

Is your feature request related to a problem? Please describe.
Really useful plugin to extract metrics from logs:
https://docs.fluentbit.io/manual/pipeline/filters/log_to_metrics
Describe the solution you'd like
It could be used with RemoteWrite plugin or other output plugins.
Thanks
Additional context
No response

@jcdauchy-moodys https://github.com/fluent/fluent-operator/blob/0f632a794feb743b890ba1cde13145ccb1781dd3/docs/best-practice/custom-plugin.md this might be helpful for your case
@cosmo0920 will it be useful to add the interface for this plugin?

To use fluent bit plugins, we might need to create a map (or interface?) to use actual parameters for plugins.
For log_to_metrics filter plugin, we defined the config map: https://github.com/fluent/fluent-bit/blob/master/plugins/filter_log_to_metrics/log_to_metrics.c#L965-L1052
For adding to build metrics type of pipelines, this plugin would be useful because it's only one plugin to bypass/translate logs type of pipelines into metrics type of pipelines.

Oh ok
Then i will go ahead and work on this then


Unsupported ARIA attribute

Hey, thanks for raising this. I'll get this sorted for the next release.

The PR for this is now in staging, this will be addressed in the next release


test: enable more linter rules in tests

Description of changes
This PR:

enables batch of linter rules, see changes in .eslintrc.js for the list.
fixes violations in test codebase

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

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

Do we need to run e2e on this PR?
Yes, it's in progress. If we touch product logic OR e2e tests it's advised to validate change. Or face consequences of breaking dev.


Prepare next v12 release

Bors merge bot cheat sheet
We are using bors-ng to automate merging of our pull requests. The following table provides a summary of commands that are available to reviewers (members of this repository with push access) and delegates (in case of bors delegate+ or bors delegate=[list]).

Syntax
Description

bors merge
Run the test suite and push to master if it passes. Short for "reviewed: looks good."

bors merge-
Cancel an r+, r=, merge, or merge=

bors try
Run the test suite without pushing to master.

bors try-
Cancel a try

bors delegate+
Allow the pull request author to merge their changes.

bors delegate=[list]
Allow the listed users to r+ this pull request's changes.

bors retry
Run the previous command a second time.

This is a short collection of opinionated commands. For a full list of the commands read the bors reference.

Let's add https://github.com/xing/hops/pull/1095 too and merge / release this after

bors retry


Add Tests for Five 1s and Five 5s in about_scoring_project.py

Makes sure five 1s is equal to 1200 (three 1s is 1000, then a 1 that is not part of a set is 100 (x2)).
Makes sure five 5s is equal to 600 (three 5s is 500, then a 5 that is not part of a set is 50 (x2)).

Thanks for submitting a PR!
I like what you're doing here, but do these tests add anything that would benefit the student though? I feel like we already have a lot of tests and some of them already covering the 5 element possibilities.

Thanks for the comment! It just adds a few more thorough tests (for a set of 3 then the singular values of 2 more single digits, if that makes sense). Admittedly though it doesn't really add anything.


Updates from Wed 22 Apr

Thanks for doing this! Mind rebasing on top of #975 to make sure objc tests work?

import


Add progname argument to PlaceOS::LogBackend

Add an argument to the initializer of PlaceOS::LogBackend that set the progname.
Additionally, ensure that progname is rendered in the logs.

Not necessary, progname is set as a class property of Log


Coindexing the gap of a bare relative

We now say that a Mod can be sister to a noun (without requiring a unary Nom layer). Does that include relative clauses? If so, how to coindex the gap in "a person you know"? On the N directly?

Yes, I think that's CGEL's position. "co-indexed with razor-blade, the head of the nominal containing the relative clause as modifier." p. 59

OK. Revised as follows:

Also elaborated on §4.3.


fix issue 14920 - [REG2.067.0] SList.insertAfter on uninitialized list triggers assertion in _first

fix https://issues.dlang.org/show_bug.cgi?id=14920

add a call to initialize() (similar to other insert* functions)
add unit test

Since this is a regression, can you please target the stable branch? Other than that, this is pretty simple, LGTM.

According to http://stackoverflow.com/questions/24159036/how-to-modify-a-pull-request-on-github-to-change-target-branch-to-merge-into it is impossible to edit the target branch.
Therefore I will create a new PR and close this one.
A section about which branch a PR should target in http://wiki.dlang.org/Pull_Requests would be nice. "If your changes are approved, they will be merged into the master branch." sounds like all PR should target the master branch...

A section about which branch a PR should target in http://wiki.dlang.org/Pull_Requests would be nice

I will look at doing that, good idea.


test(uplink): add unit test for local_storage_service

Description

Type of Change

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

@drepram 🔨


Update nix/cardano-node-service.nix with tracing arguments

Temporary fix until https://github.com/input-output-hk/cardano-node/issues/443 is implemented.

how can this be tested?

I'm not sure, the running the output of  nix-build -A scripts.staging.node on master gives the following error:
CallStack (from HasCallStack):
  error, called at src/Cardano/Node/Run.hs:155:14 in cardano-node-1.2.0-LOpCdC5zsAm4JmQZiPafiR:Cardano.Node.Run%

The output does look correct however.
The scripts successfully passing in hydra should confirm that I didn't break anything.

bors r+


Error when make build

Hello, I'm always getting error when try to run make build. this is the detail :
go build -ldflags="-s -w" -buildmode c-shared -o .build/uplink.so. runtime/cgo cgo-builtin-prolog:1: stddef.h: No such file or directory make: *** [build] Error 2
Can you help me how to solve this?

Hello @Nafidinara,
Can you provide details of your environment (OS version, etc.), please?

Also, could you give us an output of make -v command?

OS : Windows 10
Go version : go1.18.2 windows/amd64
and this when I run make -v
GNU Make 3.82 Built for i686-pc-mingw32 Copyright (C) 2010  Free Software Foundation, Inc. License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html> This is free software: you are free to change and redistribute it. There is NO WARRANTY, to the extent permitted by law.

I wonder if you may need to install the Windows SDK (https://developer.microsoft.com/en-us/windows/downloads/windows-sdk/)

For github.com/TopperDEL/uplink.net our friend installs MSYS2 and within that installs the package group mingw-w64-x86_64-toolchain. This also supplies stddef.h.

I've found that https://jmeubank.github.io/tdm-gcc/ is usually the easiest way to get C compiling to work with Go. This assumes it doesn't need to link to external libs (e.g. portaudio) -- in other words, it should work fine with uplink-c.

I'm wondering what we have to do to close this issue.
Should we document something in the README about compiling it in Windows?


Fix some tests

Basically just returns some of the language to match that of the test suite. 9 fails down to 4.

It appears we are fighting quite a bit of race conditions and "capybara" waits
Running Firefox on fast machine, certainly slower but at least it's not lying.
...........................

Finished in 3 minutes 1.7 seconds (files took 3.9 seconds to load)
27 examples, 0 failures

@rposborne definetely race conditions.
We're currently using the capybara-angular gem which is supposed to wait until there are no http requests/digests left. We may need to add some code that checks that in a more reliable way.

Yea, I tend to always battle capybara for front end JS.   I have had good luck using "within" blocks to scope to JS id that is not present but it always seem to be temperamental.  I think this can be merged as is.  I really just wanted to fix the scoping on various text fields we broken doing translatins.

Thanks @rposborne!


feat(cdk): Deploy component scaffolding

User Story
As a Lacework CDK developer,
I would like to have a skeleton of a new component,
So I don't have to start from scratch and some boilerplate code already exist for me to follow
Summary
Introducing Lacework CDK component scaffoldings.
It helps developers create new components fast by autogenerating a skeleton with boilerplate
code. Currently we only support Golang but we will work on Python components next.
Golang Scaffolding
https://user-images.githubusercontent.com/5712253/210906926-c1bd901e-eadf-42b6-99a1-a0b2482a20b2.mp4
Python Scaffolding

How did you test this change?
Added tests and ran the dev-mode command multiple times.
Issue
Documentation: https://lacework.atlassian.net/l/cp/u18EHxx1
Signed-off-by: Salim Afiune Maya<EMAIL_ADDRESS>

Check windows 🪟

Check windows again 😭

Trying to troubleshoot why the pipeline doesn't run the scaffolding component 🤔- locally it works... 👎🏽

check windows

check windows

check windows


Unable to set file download behavior on PDF to Execute in browser upon scratch org creation

Summary
Unable to set file download behavior on PDF to Execute in browser upon scratch org creation. I assume I most likely have a syntax error in my setting below, however, the error messages and documentation suggest to me that it is actually a bug.
Steps To Reproduce:

Add the following setting to project-scratch-def.json:

"fileUploadAndDownloadSecuritySettings": {
	"dispositions": [
		{
			"behavior": "EXECUTE",
			"filetype": "PDF"
		}
	]
}

As per documentation: https://developer.salesforce.com/docs/atlas.en-us.api_meta.meta/api_meta/meta_fileuploadanddownloadsecuritysettings.htm

Create a scratch org using sfdx force:org:create

Expected result
Scratch org is successfully created with file download behavior on PDF set to "Execute in browser" in File Upload and Download Security Settings
Actual result

ERROR running force:org:create:  A scratch org was created with username<EMAIL_ADDRESS>but the settings failed to deploy due to:
Error parsing file: 'EXECUTE' is not a valid value for the enum 'FileDownloadBehavior'

If I change the behavior value to e.g. HYBRID, I get the following error instead:
ERROR running force:org:create:  A scratch org was created with username<EMAIL_ADDRESS>but the settings failed to deploy due to:
Error parsing file: Element {}filetype invalid at this location in type FileTypeDispositionAssignmentBean

System Information
{
  "cliVersion": "sfdx-cli/7.181.1",
  "architecture": "darwin-x64",
  "nodeVersion": "node-v18.12.1",
  "pluginVersions": [
    "@dxatscale/sfpowerscripts 20.2.11 (user)",
    "@oclif/plugin-autocomplete 1.3.6 (core)",
    "@oclif/plugin-commands 2.2.1 (core)",
    "@oclif/plugin-help 5.1.19 (core)",
    "@oclif/plugin-not-found 2.3.9 (core)",
    "@oclif/plugin-plugins 2.1.7 (core)",
    "@oclif/plugin-search 0.0.4 (core)",
    "@oclif/plugin-update 3.0.7 (core)",
    "@oclif/plugin-version 1.1.3 (core)",
    "@oclif/plugin-warn-if-update-available 2.0.14 (core)",
    "@oclif/plugin-which 2.2.2 (core)",
    "1commerce 0.0.10 (user)",
    "alias 2.1.11 (core)",
    "apex 1.3.0 (core)",
    "auth 2.3.1 (core)",
    "community 2.0.18 (core)",
    "config 1.4.22 (core)",
    "custom-metadata 2.0.5 (core)",
    "data 2.1.15 (core)",
    "generator 2.0.12 (core)",
    "info 2.2.11 (core)",
    "limits 2.1.3 (core)",
    "org 2.2.18 (core)",
    "packaging 1.10.10 (core)",
    "schema 2.1.18 (core)",
    "signups 1.2.12 (core)",
    "source 2.3.7 (core)",
    "telemetry 2.0.5 (core)",
    "templates 55.1.0 (core)",
    "trust 2.2.1 (core)",
    "user 2.1.22 (core)",
    "@salesforce/sfdx-plugin-lwc-test 1.0.1 (core)",
    "salesforce-alm 54.8.4 (core)",
    "sfdmu 4.18.2 (user)",
    "sfpowerkit 6.0.1 (user)"
  ],
  "osVersion": "Darwin 21.6.0",
  "shell": "zsh",
  "rootPath": "/Users/Username/.local/share/sfdx/client/7.181.1-05b8719"
}

Additional information
I have tried many different syntaxes for this setting, but they all fail. I believe the syntax provided above should be in accordance to the documentation

@petter-eikeland Sorry the trouble with this.
I was able to get this to work with this structure.
        "fileUploadAndDownloadSecuritySettings": {
            "dispositions": [
                {
                    "behavior": "EXECUTE_IN_BROWSER",
                    "fileType": "PDF",
                    "securityRiskFileType": false
                }
            ],
            "noHtmlUploadAsAttachment": true
        }

The noHtmlUploadAsAttachment is a required field in this setting, so use with caution. The second error you saw with filetype is just a typo, should be fieldType. This feature belongs to another team, but I will log a bug so they might address the differences between public docs and what it took to get this to work.

Thank you so much for a swift resolution to my issue @peternhale! I will create a case for it and the attached Work Item.
I will keep the Issue open as you attached a Work Item, but feel free to close it.

Update from Salesforce Support:

As I could see, this issue has been identified as a bug at our end and would be fixed soon in the upcoming patches/releases (Safe Harbor).


Precompiled lib for nvidia jetson nano

I am trying to run porcupine on an nvidia jetson nano (arm linux), but none of the precompiled libraries will link.  I tried armlinux/a9-neon and all of the pis.

we don't support jetson currently. is this a commercial application?

it is not... yet.  I will close the ticket and look elsewhere, thanks.


API documentation navigation / style / source code linking issues

One thing I like about SilverStripe was the (old) api documentation, where the text are properly sized and there's a link to the source code.
The new api documentation is a step backwards - http://api.silverstripe.org/3/DataObject.html
It's nearly impossible to manoeuvre:

Awkward heading sizes
Page is too long in my opinion
v3 points to which version? v3.1? or v3.6?
No link to source code
No text color
Method summary & method elaboration shows no major differences (might as well link me to the source code like the old doc)
Don't know what at line 3756 is supposed to tell. Might as well use this as a link to the source code like before.
Too much repeating information in one page.

Consider reverting it and if re-styling is really necessary, do it properly.
If you're stuck with the new API documentation generator, consider re-styling it so that the UI looks like the old api doc.

It's been raised that we might need a "hide inherited members" feature, maybe on by default, so that you can limit the page to only members at the current inheritance. That should address 2.
Also having non-massive classes is probably the real "fix" for this issue; We shouldn't have 4000 line classes.
@clarkepaul can you provide design advice on 1. and 5. and see if there are some style improvements we can make?
When we add back links to source code, that will cover 4. / 7.
Regarding 3., we no longer maintain separate API docs for specific minor versions. We may re-introduce this at some future time, for our expectation is that users should be able to upgrade minor versions on projects under active development without breakages.
On 6. the major difference is the documentation on method arguments becomes visible.
Maybe regarding 6. / 8. we could restructure the summary / elaboration... clicking the summary "expands" the elaboration directly below, similar to an accordion. @clarkepaul what do you think about this?
E.g.

Expands into

We could have an "expand all" button somewhere, similar to the "show inherited members" action.

Related https://github.com/silverstripe/api.silverstripe.org/issues/56

Add to design backlog, no timeframes yet as to when we can look into it.

@pxwee5 in answer to your other question, reversion is not possible due to https://github.com/silverstripe/api.silverstripe.org/issues/45 - hence the rush to get the new documentation available in any form at all.
@clarkepaul would be good to know when it can be prioritised - there has been a lot of "constructive criticism" from the community on this one 😃

While most of this feedback sounds reasonable, I'd like to point out that SAMI is used by possibly the largest PHP library around: http://api.symfony.com/.
I've broken up this ticket in to relevant bits where appropriate:

Awkward heading sizes
No text color
https://github.com/silverstripe/api.silverstripe.org/issues/61+

No link to source code
https://github.com/silverstripe/api.silverstripe.org/issues/59

Page is too long
https://github.com/silverstripe/api.silverstripe.org/issues/60

Method summary
Too much repeated info
https://github.com/silverstripe/api.silverstripe.org/issues/62

On the notion of "doing it properly", just a friendly reminder that everything about this effort is open source, including this repository - you don't even need to know any SilverStripe to get cracking ;)
Closing this ticket, since it has multiple issues which hope are not captured elsewhere - with their own impact ratings, and separate priorities for the backlog.


bug: No linux/arm64 wheel in 0.3.37 release

Are you use the envd server?

[ ] Yes, I am using the envd server.
[X] No, I am not using the envd server.

Describe the bug
Since there's no arm wheel for the 0.3.37 release, when try to install envd on an arm Linux box pip tries to build the wheel locally, but that's failing because it requires go to be installed locally.
It's pretty common to use the official python slim-buster images as base images for python projects, unfortunately that image doesn't come with go preinstalled, which ends up being a problem to install that version of envd.
To Reproduce
> docker run  --rm -it python:3.11-slim-buster sh -c "apt-get update; apt-get install build-essential -y; pip install envd==0.3.37"
...
Collecting envd==0.3.37
  Downloading envd-0.3.37.tar.gz (328 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 328.6/328.6 kB 20.1 MB/s eta 0:00:00
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [26 lines of output]
      bash: go: command not found
      bash: git: command not found
      bash: git: command not found
      bash: git: command not found
      bash: line 1: go: command not found
      make: *** [Makefile:104: build-release] Error 127
      Traceback (most recent call last):
        File "/usr/local/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/usr/local/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/usr/local/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-wh2n0bxq/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 341, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-wh2n0bxq/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 323, in _get_build_requires
          self.run_setup()
        File "/tmp/pip-build-env-wh2n0bxq/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 338, in run_setup
          exec(code, locals())
        File "<string>", line 103, in <module>
        File "<string>", line 78, in get_version
        File "<string>", line 44, in build_envd_if_not_found
      AssertionError: Failed to build envd
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

Expected behavior
pip install envd=0.3.37 on arm linux boxes should work.
The docker info output
N/A
The envd version output
N/A
Additional context
No response

Should be possible if we enable the goarch in goreleaser. WDYT @gaocegege

Yeah, it should work, but I am not sure if envd build support ARM now.

@gaocegege , any estimates of when this might be fixed?


Improvement: Animate Badge when label changes.

It would be great if the badge will be animated when the label changes. E.g., the label can increase its size and decrease it afterwards.

You can do this by putting badgeView.layoutIfNeeded() inside of an animation.


epic-merge fails to run

Doesn't include a shebang line. If this is done (e.g. copying the way it is done in the epic program) - then I can't seem to avoid this error:
File "/home/n/Apps/anaconda2/bin/epic-merge", line 73, in
from math import gcd
ImportError: cannot import name gcd

I suspect this is a Python 2 related bug. I’ll start fixing it now :)

Thanks for reporting btw!

github automatically closed this as I wrote "fix #66" in the commit message (cool feature). Please tell me whether it worked or not :)


Pointers to structure elements

Support for pointers to elements in a structure. For example:
Given
struct foo{  int a, b, c; };
Allow a type of (struct foo.b*)

This is very close to two things:

Adjusted pointers #2461
Pointers to data members (c++) which are not handled at all by our type system and are honestly not very useful for analysis: you get a handle to (int T::*) which is a pointer offset to an int somewhere in T, not anywhere in particular.

This seems to be worded like the former, so this issue is blocking on that, and will track supporting those types through the type parser.

Implemented more generally in dev build 3751 -- not limited to pointing to struct members, but general offset pointer support.


Removed link to missing 404 page

The link titled ‘Fonts catalogue’ points to a 404 page: https://fontsource.org/fonts
I have edited it to point to the main page which still has the catalogue search filter.

What kind of changes does this PR include?

Fixed broken link

Minor content fixes (broken links, typos, etc.)
New or updated content
Translated content
Changes to the docs site code
Something else!

Description

Closes #
What does this PR change? Give us a brief description.
Did you change something visual? A before/after screenshot can be helpful.

Great catch, @SavPhill! Looks like fontsource updated their site. This is absolutely the link we want there, and we appreciate your help.  :muscle:
Thanks for taking the time to contribute, and welcome to Team Docs!  :partying_face:


Allow user to choose YouTube channel

Default channel is 'yogawithadriene'. Allow user to enter optional parameter to supply different channel name.

Waiting for support for @handles on YouTube Data API. Handles do not always directly correspond with channel names and when this is the case, it makes it hard for the user to identify their chosen channel. https://developers.google.com/youtube/v3/revision_history

Alternative channel name can now be specified as a command line argument. However, the fact remains that channel handles are not reliably the same as channel names, and where they differ it is difficult for a user to identify the channel name. Will open a separate issue to add functionality for handles.


feat: middleware implementation

Looks like our server processes are never closing when running puppeteer tests.

Fixed the e2e issues, so tests should pass now

Should shortCircuit just be a response? Like if I wanted to just short circuit the middleware chain, I could just pass a new response to next?

Is there a way to modify a response thats been handled by an internal middleware?

We should just expose requestHandler as middleware itself


Missing apiwin32

Hi thank you for sharing this awesome tool,
i am trying to use autoplotter on collab, however it is asking for missing module apiwin32
I installed pypiwin32 but also an error is occurring as showing below:
"Error cannot find a version that satisfies the requirement"
I also uploaded pywin32_postinstall.py after installing it is displaying the same error.
I tried to install via python2.7 and Python3 same problem any feedback/ solution would be appreciated.
Thank you 😊

Try
pip install pywin32

I have the same issue. pip install pywin32 does not remediate.

I will update the package ASAP.
It might because you have linux machine.

I am having the same issue on a Mac

Thank you @ersaurabhverma I will be awaiting for the updated packages cheers for your that 🙌🏻

+1 for linux support.
I think a lot of people and online notebook are running on linux.

Please make it compatible for linux. Thank you. Let me know if I can help in any way.

the idea: "i want to get the browser width with an call to an win32 (or in future linux/macos) api on the serverside" is just wrong.
thats not how it could work in any possible way.
just set the WIDTH to "100%" in app.py and you are done.
no platform depended code.

pull requests: https://github.com/ersaurabhverma/autoplotter/pull/4

if you need a quickfix
pip uninstall autoplotter
pip install git+git://github.com/thelittlebug/autoplotter

I have updated the package and removed the dependency on win32api.


Issue #1216 Update ~/.kube/config with crc contexts

Fixes: Issue #1216
Using the same strategy as minishift (https://github.com/minishift/minishift/pull/2881/files), update kubeconfig with crc-developer and crc-admin context.
It vendors k8s code and a little part of the openshift/oc tokencmd.RequestToken. It implies a large list of replace in go.mod but most of them are only used for dependency resolution and are not included in the final vendor directory. This list is taken from https://github.com/openshift/oc/blob/release-4.5/go.mod#L78 (I tried to trim it without success).
It also allows us to remove this kind of struct https://github.com/code-ready/crc/blob/master/pkg/crc/cluster/clusteroperator.go#L15

One thing to have in mind is that this doubles build times (from a few seconds to 2*a few seconds) and binary size (from 10MB to 20MB).

/retest

@guillaumerose Overall, this looks good to me, able to test it also. I think we should have it as part of single commit instead of vendoring as part of separate commit. I know that @cfergeau suggested it as part of review but to me it doesn't make sense to have vendor part as different commit then using actual module/package.

I merged the 2 commits. We should be good now!
The default context is admin. I chose it because otherwise, oc get all and kubectl get all return lot of errors and admin has already access to default project. Is it ok?

@guillaumerose looks like in case of proxy it is failing in the CI.
Command stderr: level=error msg="Cannot update kubeconfig: proxyconnect tcp: dial tcp <IP_ADDRESS>:3128: connect: connection refused"
Cannot update kubeconfig: proxyconnect tcp: dial tcp <IP_ADDRESS>:3128: connect: connection refused

Yes I will look at it. It also explains why some users might have issue with crc if oc talks to the proxy!

Yes I will look at it. It also explains why some users might have issue with crc if oc talks to the proxy!

This should be handled by oc-env
$ ./out/linux-amd64/crc oc-env
export PATH="/home/teuf/.crc/bin/oc:$PATH"
export HTTP_PROXY="http://squid.redhat.com:3128"
export HTTPS_PROXY="http://squid.redhat.com:3128"
export NO_PROXY="<IP_ADDRESS>,localhost,.testing"
# Run this command to configure your shell:
# eval $(crc oc-env)

Why do we want the oc command to use a proxy ? Maybe there are some remote interactions (not only the local server?).

Why do we want the oc command to use a proxy ? Maybe there are some remote interactions (not only the local server?).

@guillaumerose it is mostly because noproxy otherwise .testing also route from proxy which we want to avoid.

/retest

/retest

/retest this please

/retest this please

/retest this please


can i port saves?

i really want to port my save so i dont have to do everything again, is there a way to do it yet?

There is! If you use any save dumping app and navigate to where it stores them on your Switch's SD card, you will find the exact same save structure. You should be able to drop your PC save in there and reload the save using the same save dumping app.

that worked! thanks alot.
erhm, another concern, is there a way to mod audio? i modified the audio in the data.win but theres no sound in game, how do i fix that?

If you modify the sound files in the game.win then the changes should appear in game. What are you trying to modify exactly? Are you sure the game is actually using your new .win file?

yeap, because only the songs i modified are muted

Weird... I wouldn't know how to help with that, sorry :(


Visual Studio on build runner image windows-latest soon to expire.

Description
from windows-latest in one of my repos.

Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
Runner Image
  Image: windows-2022
  Version: 20241201.2.0
  Included Software: https://github.com/actions/runner-images/blob/win22/20241201.2/images/windows/Windows2022-Readme.md
  Image Release: https://github.com/actions/runner-images/releases/tag/win22%2F20241201.2

Is it regression?
not yet
Expected behavior
visual studio on a build image would not expire
Actual behavior
visual studio on the build image will soon expire
Repro steps

load .Net framework repo and build it with devenv.
devenv startup reports upcoming expiry
see, warning.....report it.

Thank you for bringing this issue to our attention. We are looking into this issue and will update you on this issue after investigating
cc @acnicholls

Hi @acnicholls ,
I request you , could you please check now . the new version is available with the latest rollout.  thank you !.

Hi @acnicholls , Could you please confirm on this ?

I can confirm that a workflow with windows-latest still has this issue.


[SPARK-42750][SQL] Support Insert By Name statement

What changes were proposed in this pull request?
In some use cases, users have incoming dataframes with fixed column names which might differ from the canonical order. Currently there's no way to handle this easily through the INSERT INTO API - the user has to make sure the columns are in the right order as they would when inserting a tuple. We should add an optional BY NAME clause, such that:
INSERT INTO tgt BY NAME <query>
takes each column of  and inserts it into the column in tgt which has the same name according to the configured resolver logic.
Some definitions need to be clarified:

BY NAME and specified column insertion (INSERT INTO t1 (a,b)... ) is a mutually exclusive operation
But it supports to define partition while using BY NAME: INSERT INTO t PARTITION(a=1) BY NAME <query>

At now don't support INSERT OVERWRITE BY NAME (I will supported in follow up)

Why are the changes needed?
Add new feature INSERT INTO BY NAME

Does this PR introduce any user-facing change?
No

How was this patch tested?
Add new test.

cc @cloud-fan

@cloud-fan @MaxGekk @hvanhovell Hi, PTAL. Thanks!

We should be very careful when inventing new SQL syntaxes. Do other database systems have similar syntaxes?

Do other database systems have similar syntaxes?

No, but I think this is a useful feature when insert into a table with many columns. cc @jose-torres

The GA is known to be unstable due to OOM. I'm merging this PR as the changed test can pass locally and this new parser feature should not break any existing tests.

thanks, merging to master!

Thanks @cloud-fan


Update enable-bitlocker.md

added an important Note

Docs Build status updates of commit 8ce1dea:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/bitlocker/enable-bitlocker.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dariomws306 @get-itips @tseward
Peer review needed for this PR. Thanks.

I think this note is not needed.

I agree with @dariomws306. Instead, this information should go into a general PSReadLine documentation as this will impact any inputted password when using ConvertTo-SecureString. Perhaps in the NOTE section of the doc at https://docs.microsoft.com/en-us/powershell/module/psreadline/about/about_psreadline and have ConvertTo-SecureString reference that note.

agree with @dariomws306 and @tseward

Should we apply this to 5.0, 5.1 and 7 versions?

The Change applies to all versions. I have created a new PR https://github.com/msbemba/PowerShell-Docs/pull/1 to reference converto-securestring

I see that @sdwheeler improve it and applied it only to 6 and 7 version. Does this not apply to version 5/5.1 too?

Closing as https://github.com/MicrosoftDocs/PowerShell-Docs/pull/4831 addresses the original issue. Thanks.


Copying registration-service cli jar in MVD CI workflow.

Description
MVD CI build registration service cli jar and then usage mvn dependency copy command to copy this jar to different location in order to register MVD connectors with registration service.
There was some feedback about improving this jar copy flow in CI during PR #31.
Here is the discussion from above mention PR: https://github.com/eclipse-dataspaceconnector/MinimumViableDataspace/pull/31#discussion_r932240457
Acceptance Criteria

Discuss with @paullatzelsperger to further this flow.
Update CI flow for cloud and local deployments.

@zeier Create an upstream issue based on the discussion in previous PR https://github.com/eclipse-dataspaceconnector/MinimumViableDataspace/pull/31#discussion_r932240457


String Literal Detection Issue When String Contains Keywords

Hey all!
I found an issue using Treesitter python bindings with this PowerShell library.
Example PowerShell Script:
$groups = "ABC123"
if (-not ($groups -Contains "Domain Admins")) {
    exit
}

This results in a tree that has ERRORs around the the string "Domain Admins" as it trys to parse the in within Domain as if that was the keyword in.

The full tree output when running treesitter parse on the example script:
(program [0, 0] - [3, 1]
  (statement_list [0, 0] - [3, 1]
    (pipeline [0, 0] - [0, 18]
      (assignment_expression [0, 0] - [0, 18]
        (left_assignment_expression [0, 0] - [0, 7]
          (logical_expression [0, 0] - [0, 7]
            (bitwise_expression [0, 0] - [0, 7]
              (comparison_expression [0, 0] - [0, 7]
                (additive_expression [0, 0] - [0, 7]
                  (multiplicative_expression [0, 0] - [0, 7]
                    (format_expression [0, 0] - [0, 7]
                      (range_expression [0, 0] - [0, 7]
                        (array_literal_expression [0, 0] - [0, 7]
                          (unary_expression [0, 0] - [0, 7]
                            (variable [0, 0] - [0, 7])))))))))))
        (assignement_operator [0, 8] - [0, 9])
        value: (pipeline [0, 10] - [0, 18]
          (logical_expression [0, 10] - [0, 18]
            (bitwise_expression [0, 10] - [0, 18]
              (comparison_expression [0, 10] - [0, 18]
                (additive_expression [0, 10] - [0, 18]
                  (multiplicative_expression [0, 10] - [0, 18]
                    (format_expression [0, 10] - [0, 18]
                      (range_expression [0, 10] - [0, 18]
                        (array_literal_expression [0, 10] - [0, 18]
                          (unary_expression [0, 10] - [0, 18]
                            (string_literal [0, 10] - [0, 18]
                              (expandable_string_literal [0, 10] - [0, 18]))))))))))))))
    (if_statement [1, 0] - [3, 1]
      condition: (pipeline [1, 4] - [1, 44]
        (logical_expression [1, 4] - [1, 44]
          (bitwise_expression [1, 4] - [1, 44]
            (comparison_expression [1, 4] - [1, 44]
              (additive_expression [1, 4] - [1, 44]
                (multiplicative_expression [1, 4] - [1, 44]
                  (format_expression [1, 4] - [1, 44]
                    (range_expression [1, 4] - [1, 44]
                      (array_literal_expression [1, 4] - [1, 44]
                        (unary_expression [1, 4] - [1, 44]
                          (expression_with_unary_operator [1, 4] - [1, 44]
                            (unary_expression [1, 9] - [1, 44]
                              (parenthesized_expression [1, 9] - [1, 44]
                                (pipeline [1, 10] - [1, 17]
                                  (logical_expression [1, 10] - [1, 17]
                                    (bitwise_expression [1, 10] - [1, 17]
                                      (comparison_expression [1, 10] - [1, 17]
                                        (additive_expression [1, 10] - [1, 17]
                                          (multiplicative_expression [1, 10] - [1, 17]
                                            (format_expression [1, 10] - [1, 17]
                                              (range_expression [1, 10] - [1, 17]
                                                (array_literal_expression [1, 10] - [1, 17]
                                                  (unary_expression [1, 10] - [1, 17]
                                                    (variable [1, 10] - [1, 17])))))))))))
                                (ERROR [1, 18] - [1, 43]
                                  (simple_name [1, 19] - [1, 27])
                                  (ERROR [1, 29] - [1, 33])
                                  (ERROR [1, 35] - [1, 39])
                                  (ERROR [1, 41] - [1, 42])))))))))))))))
      (statement_block [1, 46] - [3, 1]
        statement_list: (statement_list [2, 4] - [2, 8]
          (flow_control_statement [2, 4] - [2, 8]))))))

Hello @jconnor0426
I was not able to reproduce your error with the last version of the parser. Could you please retry please?


add motion sensor / rewrite ffmpeg binary sensor

Description:
The next of my ffmpeg sensor is a motion sensor. FFmpeg don't have a motion detection filter so I use a scene filter to detect a new scene/motion. In fact you can set how big a object or size of image they need change to detect a motion. The option 'changes' is the percent value of change between frames. You can add a denoice filter to video if you want a realy small value for 'changes'. On my cameras is the value 4 realy good opinium. On my raspberry it need 5-7% CPU. 'reset' handle how long don't detect a motion to reset to normal state.
A other feature, they is default deactivate, is a motion filter. With 'repeat' and 'repeat_time' can you set how many motion need to detection in a time that change the sensor to motion on.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):
binary_sensor:
 - platform: ffmpeg
   tool: motion
   input: FFMPEG_INPUT
   # optional
   name: ffmpeg motion
   ffmpeg_bin: /usr/bin/ffmpeg
   changes: 1.5
   reset: 30
   # feature to group motion
   repeat: 2
   repeat_time: 3

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

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

If code communicates with devices, web services, or a:

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

I've change to const. For sensor_class, I've use multible and not the same... And the first sugestion don't work with this class designe. Thanks

TOOL_TO_CLASS = {
  FFMPEG_SENSOR_NOISE: (FFmpegNoise, SensorNoise),
  FFMPEG_SENSOR_MOTION: (FFmpegMotion, SensorMotion),
  ...
}
INDEX_FFMPEG_CLASS = 0
INDEX_SENSOR_CLASS = 1
...

if not config.get(CONF_TOOL) in TOOL_TO_CLASS:
  return

tool = TOOL_TO_CLASS.get(config.get(CONF_TOOL))
entity = tool[INDEX_FFMPEG_CLASS](tool[INDEX_SENSOR_CLASS], config)

Could be cleaned up some but this would allow basically one line changes for future tools

Yes, that's true, but I can't import FFmpeg... Object from haffmpeg on top of platform. Thats the opinium why I do that with this.


[WIP] Generic functions proposal

Hey guys,
This is just a proposal, I'd love to hear some feedback.
Cheers,
Cleiton
var r []person
From(listPerson).WhereT(func(i person) bool {
      return i.Age > 18
}).ToSlice(&r)

It seems golint stopped support Go1.5

The external dependency is <100 lines, maybe we can embed it here and still keep it in vanilla go.

I would suggest the same idea @ahmetalpbalkan 😄   I can do that for sure. What about the new "generic" method names? Can I still call them "MethodT" as "WhereT"?

I basically like the idea of naming with "MethodT" pattern. It looks mainly like C#'s generic methods (and the original linq is written in C#). On the other hand, since there is no such thing as a generic method in Go it may look ugly to some developers that are not familiar with C#. What do you think?
Anyway it is usually better if you have choice. So I'll suggest some other possible namings. First, how about "MethodTyped"?  Secondly, there is always an option to leave method names as they are now, but put them in another type (e.g. TypedQuery)

I think I solved the design puzzle to fit both implementations in the same function. This is the current benchmark:
BenchmarkWhere-4                            	 1000000	      1342 ns/op	     616 B/op	      13 allocs/op
BenchmarkWhereT_UsingTypedPredicate-4       	  300000	      4819 ns/op	    1552 B/op	      47 allocs/op
BenchmarkWhereT_UsingInterfacePredicate-4   	 1000000	      1310 ns/op	     616 B/op	      13 allocs/op
PASS
ok  	github.com/ahmetalpbalkan/go-linq	4.243s

I like the idea of replacing Method with MethodT. go-linq is done mainly for convenience. For those who need performance I have an idea to generate more efficient loops with 'go generate'

@kalaninja @ahmetalpbalkan Please review. 😃

Totally agree @kalaninja! Changes done.

@ahmetalpbalkan  what do you think?

Sorry folks I have not had a chance to follow this patch closely. Pardon my questions in advance. @kalaninja thanks for reviewing this first of all. I am a little bit concered that the new method signatures are not self-explanatory enough:

It appears like changed sort of self-documenting func types to interface{}s without updating the godoc for these methods. For somebody who is new to the package, isn’t this going to be harder to understand? (To me, it looks like that.)
I think we should provide examples on the method comments (or as form of godoc examples). I would say, let's revisit the examples we have and extend them for complicated methods like I quoted above, otherwise this patch will be harder to use.
Also, regarding the benchmark results above, it appears like one form is faster/less-allocs over another. I am not clear which one is which (and without adding a note to README, users will not be sure either).
As a last question, is this a backwards-compatible change? Do we release this as a major or minor version? Thanks in advance.

For somebody who is new to the package, isn’t this going to be harder to understand?

Yes, I also thought about it. Seeing something like func (q Query) Select(selector interface{}) doesn't give you any info on what selector is supposed to be. Providing examples on each method seems to be a solution.

Also, regarding the benchmark results above, it appears like one form is faster/less-allocs over another. I am not clear which one is which (and without adding a note to README, users will not be sure either).

What do you mean? Do you mean that we should add a note that this version is slower?

is this a backwards-compatible change?

I am a bit doubtful about its versioning. On the one hand it is backward compatible with the current version, but on the other hand all api methods' signatures had changed, so it might be treated as a major update too.

@kalaninja sorry I could not really closely follow the changes. Are we always significantly slower now, or depending on the func signature that the user provides, are we in the same ballpark as v2.0 with this change? It would be great if someone summarizes the perf change with this patch.
About versioning, I am okay with releasing this as v2.1 if it is backwards compatible.
Should we add examples in this PR or in a separate patch? If the patch author enables contributions to the PR, maybe we can add the examples to godocs.

@ahmetalpbalkan Yes, we need to improve documentation. I was looking to the original Linq documentation and started to think in a way to document better the functions.
Please take a look at the GroupJoin godoc and review as a proposal. I also invited both to collaborate with my repo, I will summarize the benchmarks soon.

@ahmetalpbalkan Yes, we are always gonna be about 3 times slower, according to benchmark. Am i correct @cleitonmarx ?

@ahmetalpbalkan Yes, we are always gonna be about 3 times slower, according to benchmark.

I assumed we'd be around the same speed if the old method signature (e.g. func (i interface{}) bool) is passed. If this is not the case and the patch is always slowing it 3x down, I think we should keep the old methods and add new ones like WhereT?  With v2.0, we gained really good perf, it would be sad to lose it.
Sorry if it's too much to ask. I couldn't closely watch the decision early on.

I assumed we'd be around the same speed if the old method signature (e.g. func (i interface{}) bool) is passed. If this is not the case and the patch is always slowing it 3x down, I think we should keep the old methods and add new ones like WhereT? With v2.0, we gained really good perf, it would be sad to lose it.

You're right @ahmetalpbalkan. All the functions are optimized to use the old methods signatures (e.g. func (i interface{}) bool), the performance with this signatures didn't change.

Yes, we are always gonna be about 3 times slower, according to benchmark. Am i correct @cleitonmarx ?

@kalaninja, In the Where benchmark using a "generic function" is 3 times slower than the old signature, but for the GroupJoin function can be higher because GroupJoin haves 3 functions to parse and the Where just one. We will have more information about it when I finish the benchmarks.

@cleitonmarx we most definitely need to document the performance impact based on method signatures and document how to keep getting the current performance. Would you be up to updating README and adding examples to each method on how to use?
Also do you think if it's too much to go back and split this into Method & MethodT for all APIs? Is it too much to maintain? The way I see, splitting Method & MethodT have advantages:

easy to document: it will be clear that "Method" is how the input func is supposed to look like and "MethodT" would just accept the generic version of it.
we can say Method is faster than MethodT

and disadvantages would be:

bigger API surface (but I don't think it's too bad)
more maintenance/testing cost
more work for you

I'd say let's figure out whether we really need to merge or split Method/MethodT. I appreciate your thoughts.

@ahmetalpbalkan I took the liberty to execute the same benchmark code that @kalaninja created to compare 0.9 and go2linq (2.0) and I adapted to run with v2.0 and v"3.0" using generic functions (I think this change will be a major although it still fully backward compatible, the type and name of the parameters changed). Here are the results:
BenchmarkSelectWhereFirst_V2.0-4   5000000	       370 ns/op	     224 B/op	      10 allocs/op
BenchmarkSelectWhereFirst_V3.0-4   1000000	      2120 ns/op	     968 B/op	      35 allocs/op
BenchmarkSelectWhereFirst_V0.9-4   	 	 3	 363537164 ns/op   120546042 B/op	 2000052 allocs/op

BenchmarkSum-V2.0-4                 	30	  48693440 ns/op	 8000169 B/op	 1000006 allocs/op
BenchmarkSum_V3.0-4                	 	 2	 292160640 ns/op	88000632 B/op	 4000016 allocs/op
BenchmarkSum_V0.9-4                		 5	 253731714 ns/op	69161392 B/op	 1000053 allocs/op

BenchmarkZipSkipTake_V2.0-4       10000000	       217 ns/op	     192 B/op	       6 allocs/op
BenchmarkZipSkipTake_V3.0-4        1000000	      1271 ns/op	     824 B/op	      27 allocs/op
BenchmarkZipSkipTake_V0.9-4              2   672403213 ns/op   144520824 B/op    3000075 allocs/op

IMHO, I think we show keep both implementations in the same methods, it will bring much  convenience to use go-linq. As the results showed, using generics functions is not so bad about performance. I believe that we can create a good documentation explaining how to use both implementations with examples. The new validation error messages will guide the developer indicating the right implementation for each parameter function:
=== RUN   TestGroupBy_PanicWhenKeyFunctionParameterInTypeDoesntMatch
--- PASS: TestGroupBy_PanicWhenKeyFunctionParameterInTypeDoesntMatch (0.00s)
	groupby_test.go:63: GroupBy: parameter [keySelectorFn] has a invalid function signature. Expected: 'func(T)T', actual: 'func(int,int)bool'

=== RUN   TestGroupBy_PanicWhenElementSelectorFunctionParameterInTypeDoesntMatch
--- PASS: TestGroupBy_PanicWhenElementSelectorFunctionParameterInTypeDoesntMatch (0.00s)
	groupby_test.go:80: GroupBy: parameter [elementSelectorFn] has a invalid function signature. Expected: 'func(T)T', actual: 'func(int,int)int'

=== RUN   TestOrderByDescending_PanicWhenFunctionIsNotValid
--- PASS: TestOrderByDescending_PanicWhenFunctionIsNotValid (0.00s)
	orderby_test.go:65: OrderByDescending: parameter [selectorFn] has a invalid function signature. Expected: 'func(T)T', actual: 'func(int,int)int'

I can work on the documentation, but it will depend on my availability. I gave permissions to both of you in my repository if you have availability to help me with. Here is a try in how document the functions signatures.

Sorry folks I'm a bit busy nowadays, couldn't spend enough time on this. I realized the "v3.0" has around 4x more allocs and around 5x-10x slow down compared to "v2.0". Is there a way to drive these down or are the completely unavoidable?
In my earlier comment I said, if we can't avoid the perf penalties (which is apparently 5x-10x) with generic functions, let's keep the tabs separate and have Method and MethodT separate. This way:

no documentation hell, since it would be clearer when there are is an example Method with what the funcs look like (btw I liked your example docs, but having a separate Method and MethodT will be easier for users to grasp it).
little bit more duplicate code to maintain and test (I don't think this is a big problem as the code is mostly trivial)
we keep the perf/memory gains of v2.0, which IMO is great.
we would have a way to allow people write cleaner code with MethodTs.

I think the course of action I'd approve here would be to separate things back to Method and MethodT.

Totally agree with @ahmetalpbalkan Though I initially wanted to combine Method and MethodT into a single method, now it seems to result in really big issues with docs. It is absolutely unclear how to let end users know about the correct forms of needed predicates.

Ok guys, based on your last comments follow the new changes. Please let me know if you agreed with this approach, so I can finish the documentation.

@cleitonmarx thank YOU SO MUCH for going through this. 💙  I will give 2 pieces of feedback.
1. "Parameters" paragraph is not very necessary
Instead of this

func (q Query) All(predicate func(interface{}) bool) bool
All determines whether all elements of a collection satisfy a condition.
Parameters
predicate: A function to test each element for a condition.

you can just do:

func (q Query) All(predicate func(interface{}) bool) bool
All determines whether all elements of a collection satisfy a condition.

because you already have (1) an example and (2) the predicate is self-explanatory (3) there is more docs at AllT.
For this:

I don't think you need a separate parameters section. You can just say this in a paragraph:

predicateFn is of type "func(TSource) bool".

and your examples are sufficient. That said, we can keep this for complicated functions like GroupJoinT...
2. Rebase issues
It appears like this PR's Commits tab contains some of the old commits we pushed to the repo. This looks like a rebase issue you had locally I believe. Can you create a new branch to cherry-pick your commits (and ideally squash the final version of your patch into 1 single commit or something) and force-push to your branch? (It should look like this.)

@cleitonmarx just wanted to check if you would like to move forward with this sometime. The code sounds good, I commented on some cosmetics issues in my comment above. This would be really cool to have!

Hey guys, sorry for the lack of communication. I was too busy these last weeks. Now that I have some free time, let's keep moving this PR. 😄 . I added more examples, please review the new changes. I will try fix the rebase problems in the weekend.

@ahmetalpbalkan @kalaninja Hey guys, PR ready to merge. Let me know your thoughts about it.

This generally looks good to me. I wonder if we can reduce duplication in the tests for typed functions vs what we have today. I see some panic handling code etc. I bet in the long term, we can make tests shorter by using go-check or testify/require|assert packages.
Maybe we should merge this to a dev-v3 branch or something and polish it for a bit. I'd like to do a refinement on the godoc, API, and tests sometime before we merge it into master.  This would mean I have to just close the PR and push your commit to a new branch manually or you'd reopen a PR to the new branch. @kalaninja what do you think?

I just changed the base branch to v3-dev from master so that we can merge this and iterate on it a bit further before breaking people in master... @kalaninja if this also looks good to you, let's merge and polish later.

@ahmetalpbalkan looks good to me.
@cleitonmarx really good job!
Guys, what about putting corresponding T methods into the same files with their originals?  Having separate *typed.go files will double their amount and there are many files already in the repo.

@kalaninja Yeah we can address that later. @cleitonmarx would you like to git commit --amend --edit your comment and give it a nice title like Generic functions support for future reference?

@kalaninja Thanks man, I'm glad to have the opportunity to contribute with something. 😄
@ahmetalpbalkan Done!

🎉  !
@kalaninja I think we should make Cleiton a maintainer as well.

sorry guys, I was ill and couldn't cooperate.
@ahmetalpbalkan I am totally agree, @cleitonmarx made a great effort.

Tks guys, will be a pleasure.


Option to weight hit as crit for item comparison

the upgrade rankings on gear appears to use an unsituational weighting for hit as superior to other secondary stats, causing gear with hit almost always score higher than gear without hit. in situations where reaching hit cap is accomplished regardless of choice in the specific slot, inaccurate gear rankings are being provided. replacing the score of hit with the score of crit will allow better item comparison.

You can set the EP value of hit to the same value that crit has, but I don't understand how that is supposed to solve anything.

What prompted this suggestion is me plugging in my gear and checking
upgrades and it ranks loop of the agile below darkstone ring, even though
this is incorrect. The reason is due to the weighting of hit and the
comparison not knowing you arent necessarily giving up hit when you switch
out a hit item for a non hit item.
On Mon, Mar 6, 2023, 2:04 PM Tharre @.***> wrote:

You can set the EP value of hit to the same value that crit has, but I
don't understand how that is supposed to solve anything.
—
Reply to this email directly, view it on GitHub
https://github.com/wowsims/wotlk/issues/2747#issuecomment-1457077908,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6KBTFCM6KAEXOFJXQ3Q52TW2ZNOXANCNFSM6AAAAAAVRRXRLE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Anyway, thank you, i hadnt found that function.
On Mon, Mar 6, 2023, 2:27 PM Dan G. @.***> wrote:

What prompted this suggestion is me plugging in my gear and checking
upgrades and it ranks loop of the agile below darkstone ring, even though
this is incorrect. The reason is due to the weighting of hit and the
comparison not knowing you arent necessarily giving up hit when you switch
out a hit item for a non hit item.
On Mon, Mar 6, 2023, 2:04 PM Tharre @.***> wrote:

You can set the EP value of hit to the same value that crit has, but I
don't understand how that is supposed to solve anything.
—
Reply to this email directly, view it on GitHub
https://github.com/wowsims/wotlk/issues/2747#issuecomment-1457077908,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6KBTFCM6KAEXOFJXQ3Q52TW2ZNOXANCNFSM6AAAAAAVRRXRLE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Solution here would be as you found, using the 'stat weights window' and changing them as you see fit or recalc ep's for given gear


Update or replace django-classy-tags

We use the package django-classy-tags in edx/django-wiki, edx/edx-platform. It hasn't yet added support for Django3.2. Please follow the guidance in https://openedx.atlassian.net/wiki/spaces/AC/pages/3036972032/Handling+Outdated+Dependencies to resolve the problem this poses for the Open edX Django 3.2 upgrade.

I'll be looking into this one! :100:

Hey! Sorry for the delay.
I searched for classytags on edx-platform -here are the results- and didn't find anything. So are we sure this is being used?

I left a comment earlier saying that this package was not being used. I was wrong. It's a dependency of django-sekizai, here is the proof.
For Django-classy-tags to support Django 3.2, we could modify their CI adding tests for that version. The problem is that the package depends on Django-cms support for Django 3.2, and it hasn't been released yet. So I'm gonna ask Django-cms's maintainers when they expect to make the release :)

@mariajgrimaldi Thank you so much for an update, I appreciate.

It's merged! I'll be posting when they make the new release.

Great, thanks for an update, @mariajgrimaldi, I will be waiting for release news from you.

Hey there. I've got good news! django-classy-tags has a new release! v3.0.0

@mariajgrimaldi  Thank you! Closing this issue. CC @Jawayria @awais786


[BD-46] fix: remove props spreading in DataTable components

Description
Removes redundant props spreading in DataTable subcomponents that cause a lot of console errors during development, more details in
Deploy Preview
Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[ ] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[ ] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @viktorrusakov!
When this pull request is ready, tag your edX technical lead.

@viktorrusakov Even though your pull request wasn’t merged, please take a moment  to answer a two question survey so we can improve your experience in the future.


CNDB-9827: Online sorting and trimming of CQL result rows

When using ORDER BY or ANN, SELECT queries collect all the rows found in each replica into a ResultSet, then order them, and finally trim the first LIMIT results. This can be seen here.
For index-based sorting, it means that the ResultSet holds up to num_replicas*LIMIT in memory. If we were also using an OFFSET, it would hold num_replicas*(LIMIT+OFFSET) rows.
This PR uses online sorting in the ResultSetBuilder to limit its memory usage to just LIMIT rows, or LIMIT+OFFSET rows if we use an offset.

Thanks for the review.


Not compatible with angular-legacy-sortablejs when update form 1.9.0 to 1.10.0

Problem:
in version 1.9.0, i use angular-legacy-sortablejs by setting this on index.js (with webpack)
 require('angular-legacy-sortablejs-maintained/angular-legacy-sortable.js');
after i update to 1.10.0 there is a error on console:
TypeError: Sortable.create is not a function

Then i find 1.10.0 is completely reconstructed.
in 1.9.0 :

(function sortableModule(factory) {
	"use strict";

	if (typeof define === "function" && define.amd) {
		define(factory);
	}
	else if (typeof module != "undefined" && typeof module.exports != "undefined") {
		module.exports = factory();
	}
	else {
		/* jshint sub:true */
		window["Sortable"] = factory();
	}
})(...)

in 1.10.0 :
(function (global, factory) {
  typeof exports === 'object' && typeof module !== 'undefined' ? module.exports = factory() :
  typeof define === 'function' && define.amd ? define(factory) :
  (global = global || self, global.Sortable = factory());
}(this, function (){...})

and angular-legacy-sortablejs  get sortablejs by this way:
(function (factory) {
	'use strict';

	if (typeof define === 'function' && define.amd) {
		define(['angular', 'sortablejs'], factory);
	}
	else if (typeof require === 'function' && typeof exports === 'object' && typeof module === 'object') {
		require('angular');
		factory(angular, require('sortablejs'));
		module.exports = 'ng-sortable';
	}
	else if (window.angular && window.Sortable) {
		factory(angular, Sortable);
	}
})(function (angular, Sortable) {...}

So can you think about adding some conditions so that sortablejs can continue to be used by angular-legacy-sortablejs
thank you !

IMO the issue is not how angular-sortablejs import it, but a semantic versionning bug on this module.
Since breaking changes were introduced, a new major version should have been issued, which would have prevented npm update from breaking all framework bindings which depend on this module.
(Nevertheless, this module is super convenient, thanks for the good work!)


Possible type for template tag

On line #74 the following code is present.

Hide the template element in IE 8/9/11, Safari, and Firefox < 22.

Is this supposed to be
-Hide the `template` element in IE 8/9/11, Safari, and Firefox < 22.
+Hide the `template` element in IE 8/9/10, Safari, and Firefox < 22.

I don't understand the question

I think the question is: Is it "IE 8/9/10" or "IE 8/9/11"?

ah, thanks. fixed

Sorry for the ambiguity. That was the question. Maybe I will just send a PR next time


Add test cases for handling overflows on dequantization and fix size of count7x7

Unfortunately Lepton doesn't check to make sure that coefficients * quantization table aren't greater than 16 bits. This can cause all kinds of weird behavior so we need to make sure that this behavior doesn't change.
Tests showed found a regression in #67 that count7x7 is too small by one and would cause an assertion decoding some images.
Explicitly import rand_chacha instead of using the standard library for generating random numbers. Since the standard library can change implementations, we want it to be fixed so that our hashes don't change.

Hmm, in principle yes, the quantization table can be of (2^16-1), but Lepton does not support non-8-bit-depth JPEGs, and citing JPEG standard, sec. B.2.4.1 "Quantization table-specification syntax": "An 8-bit DCT-based process shall not use a 16-bit precision quantization table." So in principle standard-conforming JPEG able to be coded by Lepton should have 8-bit quantization table.

You need to update cargo files, it fails in CI.

Hmm, in principle yes, the quantization table can be of (2^16-1), but Lepton does not support non-8-bit-depth JPEGs, and citing JPEG standard, sec. B.2.4.1 "Quantization table-specification syntax": "An 8-bit DCT-based process shall not use a 16-bit precision quantization table." So in principle standard-conforming JPEG able to be coded by Lepton should have 8-bit quantization table. Dequantization in DB Lepton, however, always done in 32 bits accuracy, and support for 16-bit tables was present there.

Yeah... the problem is that the corpus that we've already compressed with the C++ version might include weird non-standard conforming JPEGs.


proxy for non-dockerized service

Hi
I am posting this here too, as if there is a solution here via a "fake" container, it will automatically work with the letsencrypt docker :-)
I am using docker-letsencrypt-nginx-proxy-companion for proxying multiple docker containers, but would like to use it also for an application that is running on another machine, and not in a local docker container.
So for the nginx-proxy part, i can add a custom.conf in the /etc/nginx/conf.d/ directory and the service will be reachable via HTTP. But I couldn't find a way to use it together with the letsencrypt functionality?
Or do you have an idea how i could make a simple "fake" docker container that would force nginx-proxy, and so docker-letsencrypt-nginx-proxy-companion, to proxy to another host?
thanks
KoS
see https://github.com/JrCs/docker-letsencrypt-nginx-proxy-companion/issues/127

Hi, i guess you could have a container that is just a tcp proxy

@wader thank you very much, i must have been blind :-)
i used a simple socat proxy container: https://github.com/silasdavis/socat-proxy-docker


cmake(enhance):Enhance romfs so that RAWS files can be added in any location

Summary
1.make the generation sequence of etc romfs no longer bound to the board
unify all add_raws calls after traversing all directories.
# nuttx/CMakeLists.txt
process_all_directory_romfs()

2.RCRAWS RCSRCS can be added from any directory
3.enable dynamic files, files generated during the compilation process,
and ensure the correct time order
add_dynamic_rcraws(SRCS dyn_file DEPENDS gen_dyn_file_target)

Impact
In order to be compatible with the old nuttx_add_romfs, we retain nuttx_add_romfs.
If you use this enhancement method, please do not call nuttx_add_romfs
Use add_board_rcsrcs and add_board_rcraws instead
Testing
custom board build pass

[Experimental Bot, please feedback here]
Yes, this PR appears to meet the basic NuttX requirements, but the provided information is insufficient for a thorough review.  Here's a breakdown of what's missing and how to improve it:
Missing/Insufficient Information:

Summary - Why?:  While the what is explained, the why is missing.  Explain the problem the current ROMFS generation process creates.  Why is decoupling from the board necessary?  Why is dynamic file generation important? What specific issues does this solve?
Summary - Related Issues: Are there any related NuttX issues this PR addresses?  Provide links.
Impact - User: Will users need to change existing build scripts or configurations? If migrating from nuttx_add_romfs, what specific steps are needed?  Be explicit.
Impact - Build: The impact is mentioned, but not fully described. How does the build process change specifically?  New CMake functions?  Changes to existing ones?  New variables? Provide concrete details.
Impact - Hardware:  Does this affect any specific architectures, boards, or drivers? Even if the answer is no, explicitly state it.
Impact - Documentation: Does this require documentation updates? If so, are they included in the PR? If not, explain why not and when they will be provided.
Impact - Security/Compatibility/Other:  Explicitly state "NO" for each if there's no impact. Don't leave them blank.
Testing: This is the weakest part. "Custom board build pass" is not enough.

Specify the host:  e.g., "Linux, Ubuntu 22.04, x86_64, GCC 11.3.0"
Specify the target: e.g., "sim:nsh", "stm32f4discovery:hello"
Provide relevant log snippets: Don't just dump entire logs. Show the output related to ROMFS generation before and after the change. Demonstrate that the dynamic files are included correctly and that the order is as expected.  Show that the decoupling from the board works as intended.  Ideally, show a failing case before the change and a passing case after.

Example of Improved Testing Section:
Testing

I confirm that changes are verified on local setup and works as intended:

* Build Host(s): Linux, Ubuntu 22.04, x86_64, GCC 11.3.0
* Target(s): sim:nsh, stm32f4discovery:hello

Testing logs before change (sim:nsh - demonstrating missing dynamic file):

ls /etc
config.ini  (static file)

Testing logs after change (sim:nsh - demonstrating dynamic file present):

ls /etc
config.ini (static file)
dynamic.cfg (dynamic file)

Testing logs demonstrating correct build order dependency (showing dynamic.cfg is generated *before* being added to ROMFS):

[... build output showing gen_dyn_file_target completing before ROMFS creation ...]

Testing logs on stm32f4discovery (demonstrating board decoupling -  showing same ROMFS content regardless of board selection):
...

By providing more specific and complete information, you'll make it much easier for reviewers to understand, evaluate, and ultimately merge your PR.  Address the points above to significantly improve the quality of your submission.


ArchaiusAutoConfiguration do not change ConfigurableEnvironmentConfiguration when context changed between tests

Describe the bug
Seems ArchaiusAutoConfiguration do not do it's work, when new application context is configured by SpringBootTestContextBootstrapper in multi tests mode and if all tests have different with different context (different test property sources for example). As a result, we have wrong Archaius config list, that contains stale spring environment object (from first test executed).
We could workaraound here - AtomicBoolean initialized variable in ArchaiusAutoConfiguration could be set to false, if we destroy context (for example use @DirtiesContext annotation). But we would like to save our context for optimization purposes
Can we do something here, except destroying previous contexts?
Thanks for advice
Sample
@TestInstance(TestInstance.Lifecycle.PER_CLASS)
@SpringBootTest(
		properties = {
				"prop1 = false",
})
public class Sample1Test {
     //some tests here
}

@TestInstance(TestInstance.Lifecycle.PER_CLASS)
@SpringBootTest(
		properties = {
				"prop1 = true",
})
public class Sample2Test {
     //some tests here
}

expected in Sample2Test:
DynamicProperty.getInstance("prop1") is true, but false returned, cause spring env from Sample1Test live in archaius config-lists

This module has entered maintenance mode. This means that the Spring Cloud team will no longer be adding new features to the module. We will fix blocker bugs (bugs that don't have a work around like you do) and security issues, and we will also consider and review small pull requests from the community.


plotting error with skip layer models

Hi Marcus,
With the code below, if you set the size = 1+, plotting works fine. But if you change it to 0 with skip connections enabled, I get the error instead:
Error in data.frame(wts, row_nms) :
arguments imply differing number of rows: 3, 13)
library(nnet)

ird <- data.frame(rbind(iris3[,,1], iris3[,,2], iris3[,,3]),
                  species = factor(c(rep("s",50), rep("c", 50), rep("v", 50))))

ir.nn2 <- nnet(species ~ ., data = ird, size = 0, rang = 0.1,
               decay = 5e-4, maxit = 200, skip = TRUE)

summary(ir.nn2)

library(NeuralNetTools)

plotnet(ir.nn2, nid = TRUE, all_out = TRUE, all_in = TRUE,
        wts_only = FALSE, rel_rsc = 5, circle_cex = 5, node_labs = TRUE,
        var_labs = TRUE, x_lab = NULL, y_lab = NULL, line_stag = NULL,
        cex_val = 1, alpha_val = 1, circle_col = "lightblue",
        pos_col = "black", neg_col = "lightgrey", bord_col = "black",
        max_sp = TRUE)

Ok, this is not a bug but  more of a feature request.  I'd have to modify the plot code significantly for the plotnet.nnet method... stay tuned.

No didn't think it was a bug - rather was just wondering if you had considered to include support for it.
I will be using the package a bit more in the next few weeks - will try to think of any other features that could improve it.
Regards,
Max

From: Marcus W Beck<EMAIL_ADDRESS>Sent: 21 August 2015 15:40
To: fawda123/NeuralNetTools
Subject: Re: [NeuralNetTools] plotting error with skip layer models (#12)
Ok, this is not a bug but more of a feature request. I'd have to modify the plot code significantly for the plotnet.nnet method... stay tuned.

Reply to this email directly or view it on GitHubhttps://github.com/fawda123/NeuralNetTools/issues/12#issuecomment-133448214.

Hi @mngreene, I think I've got a working version that can plot skip layer connections.  The plotting function now includes an option to plot the skip layer connections separate from the primary network.  This is the default if size = 0 in your example.  Here's a snippet with some examples:
# devtools::install_github('fawda123/NeuralNetTools')
library(NeuralNetTools)
library(nnet)

ird <- data.frame(rbind(iris3[,,1], iris3[,,2], iris3[,,3]),
                  species = factor(c(rep("s",50), rep("c", 50), rep("v", 50))))

# five nodes in hidden layer, no skip
ir.nn1 <- nnet(species ~ ., data = ird, size = 5, rang = 0.1,
               decay = 5e-4, maxit = 200, skip = FALSE)

# five nodes in hidden layer, skip
ir.nn2 <- nnet(species ~ ., data = ird, size = 5, rang = 0.1,
               decay = 5e-4, maxit = 200, skip = TRUE)

# only a skip layer
ir.nn3 <- nnet(species ~ ., data = ird, size = 0, rang = 0.1,
               decay = 5e-4, maxit = 200, skip = TRUE)

# plots
plotnet(ir.nn1)
plotnet(ir.nn2)
plotnet(ir.nn2, skip = TRUE)
plotnet(ir.nn3)

Let me know if you have any problems.
-Marcus


idmapper fails when used with old proximity files

ValueErrorTraceback (most recent call last)
 in ()
1 with gzip.open(os.path.join(data_dir, proximity_data_filename), 'r') as f:
----> 2     idmap = ob.preprocessing.id_to_member_mapping(f, time_bins_size, tz=time_zone)
/home/orenled/git/openbadge-analysis/openbadge_analysis/preprocessing/metadata.pyc in id_to_member_mapping(mapper, time_bins_size, tz, fill_gaps)
121         return pd.DataFrame.from_dict(idmap, orient='index')[0].rename('member')
122     else:
--> 123         raise ValueError("You must provide either a fileobject or metadata dataframe as the mapper.")
124
125
ValueError: You must provide either a fileobject or metadata dataframe as the mapper.

Solved in 0.4.1


How to ignore files?

If tiled is pointed at a user's working data directory, there may exist various files from which which tiled should not attempt to serve data.  (for example, IgorPro, SPEC, or MatLab code).
Learn how tiled can ignore such files without reporting them as uninterpretable on startup.

Or, develop a new handler for such files that serves empty data.
@danielballan : What do you think about a read_ignore(filename) function that is mapped to application/x-ignore?  Not sure I like it since it will identify the presence of certain non-data files.  But perhaps the mimetype could be handled internal to tiled so that such content is not served at all?


More mobile (and tablet) responsive updates

Shorten tab descriptions (to just "Read" and "Write")
Fix display width in tablet
Hide attachment controls in phone size (480px and below)
Always show edit/delete comment items in mobile/tablet view
@donjo: note I changed to display: none and display: inline-block for the comment controls as that is a more preferred and robust method

eregs/notice-and-comment#145

This looks nice, but can we avoid the !importants? This adds six to the file (one was already present)

DERP. I was confused why specificity was being weird, just realized that it's the ordering of the media queries so I'll change that up.

Reordered the media queries correctly and pushed over the left margin for bottom section nav on mobile.

Excellent


How to merge nuc to desktop?

I just have one machine. How can I run all the code in my PC without NUC.

It depends on if your PC is only for control purpose or if you need to use GPUs. If you need to use GPUs, merging is not a solution because the real-time kernel is not compatible with Nvidia drivers. If you do not need GPUs, you can just install real time kernel on your PC and install both c++ and python code on the same machine.


bors-stats.sh: Change ignore pattern "try" to "try\nBuild"

Issue Number

Overview

[x] Change ignore pattern "try" to "try\nBuild". Comments containing the phrase "retry" anywhere are no longer incorrectly filtered out.

Comments
Now:

On master:

bors r+

When I run this script I get these errors (gnu bash 4.4.23):

Hm 🤔 . Both are instances of '' multiline strings. But not sure how to fix.

Do you ever get the feeling that this script is pushing the limits of bash and jq?

Yes. Originally I doubted I'd get this far using only bash and jq, but then I got used to it, and it seemed to work decently. I don't think the initial GraphQL response processing would be easier in say Haskell.

When I run this script I get these errors (gnu bash 4.4.23):

Ok, I managed to reproduce using:
-#!/usr/bin/env bash
+#! /usr/bin/env nix-shell
+#! nix-shell -i bash -p nix coreutils gnugrep gnused jq curl

Will have a look some time. Thanks for raising.


Delete artifact refresh

@antgonza noticed that the page doesn't automatically refresh when you're deleting an artifact. This has been fixed here. GIF showing the result of the changes:

Codecov Report

Merging #2443 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #2443      +/-   ##
==========================================
+ Coverage   94.15%   94.15%   +<.01%
==========================================
  Files         164      164
  Lines       19442    19445       +3
==========================================
+ Hits        18305    18308       +3
  Misses       1137     1137

Impacted Files
Coverage Δ

...ta_pet/handlers/artifact_handlers/base_handlers.py
92.81% <100%> (+0.08%)
:arrow_up:

...lers/artifact_handlers/tests/test_base_handlers.py
95.32% <100%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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


[fix bug 1153578] Filter nominees against 'Rep' group.

Coverage remained the same at 91.99% when pulling bbcd2da45466da6bb4bc8390a3a231eee480bbc6 on akatsoulas:voting-urls into 871520dd0489ef73796d73fad6b12de99d400836 on mozilla:master.

r+


Request for Waymo ReID Dataset

Hi, is Waymo ReID dataset also open for free access? I found your research interesting and want to test the evaluation on WaymoReID as well! (I succeed to test on the NuScenese ReID dataset following your instructions)

Hi, I would be happy to help you get a version of Waymo ReID. I'm afraid it's slightly more complicated with Waymo due to the licensing restrictions placed on the dataset. Please send me an email about this at benjtherien at gmail dot com


Rate Limit for Arachni Scans

As a user I would like to reduce the number of requests per second during my scans so that they won't affect the performance of my application and the scanner won't be blocked by a WAF.
Arachni provides the options to set the browser-cluster-pool-size and the http-request-concurrency. Additionally, the rate_limiter plugin provides the option requests_per_second to set a maximum for the requests per second.
In my target definition (used for Arachni Scans) I would like to configure something like this:
[
    {
        "name": "Arachni BodgeIt Scan",
        "location": "http://bodgeit:8080/bodgeit/",
        "attributes": {
            "ARACHNI_RATE_LIMITS": {
                browserClusterPoolSize: 1,
                httpRequestConcurrency: 2,
                requestsPerSecond: 3
             }
        }
    }
]

 Archni Scanner um ein RATE-Limit Feature erweitern

This issue has been implemented.


Update pypi-deps-db in flake inputs

The current version of pypi-deps-db in the flake.lock file is over 5 months old.
This should be super quick, nix flake update or just manually update the lock file.

Maybe submit a PR? I wonder if this would help with some dependency issues I've been having.

@blissdev I made the pull request, but in the meantime if you are having issues with out of date dependencies, here is a sample flake input that should fix the problem.
  inputs = {
    mach-nix.url = "github:DavHau/mach-nix";
    pypi.url = "github:DavHau/pypi-deps-db";
    pypi.flake = false;
    mach-nix.inputs.pypi-deps-db.follows = "pypi";
  };

This just adds the pypi-deps-db input and makes mach-nix follow it.

Thanks!


Blending algorithm

I don't really know the details, but mspaint allowed you to do gradients like this: http://www.nerdparadise.com/mspaint/lineargradient
which isn't possible in jspaint.

@Mikejmoffitt points out selections are scaled with nearest neighbor sampling whereas the canvas is scaled with bilinear interpolation.

Looks like gradients work in the Windows XP version, but not Windows 98.
JS Paint is primarily targeting parity with Windows 98. I'm not ruling out this feature, because it seems fun to support various old hacks, but I'm not interested in working on it at this point.
MS Paint might use "box sampling" so that all pixels contribute to the output.


addresses/multiuprn not available in docker based version

In the docker-compose based version I don't see the addresses/multiuprn route available in the /routes

The API image is three years old.
  address-index-api:     image: "thornsj/address-index-api:190420"

thanks. I don't see any newer docker images. Do you plan to publish a newer one anytime soon, otherwise we will build the project on our side.


rename waitForTransaction to waitForStateChange

to better reflect its intent: waiting for the state change after a tx is submitted
can also add some docs for why this is important to do and not just waitForTransactionReceipt (which could lead to UI jank)

decided against this (see #3210) because txs won't always mutate state and I can't think of a better name to reflect "wait for storage adapter to have seen this transaction"


bug(forge): vm.transact cheatcode not working

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (80df71f 2023-09-27T00:24:52.309432000Z)
What command(s) is the bug in?
No response
Operating System
macOS (Apple Silicon)
Describe the bug
I ran the vm.transact code example from the Foundry Book in a forge test, the test failed with reason "backend: failed committing transaction: Transaction(NonceTooLow { tx: 1, state: 2 })".
For MAINNET_RPC_URL, I was using "https://rpc.ankr.com/eth".

Am having a similar problem (same OS, same version of forge). It seems like the vm.transact is not working as expected.
Example to replicate:
    function test_transact_not_working() public {
        vm.createSelectFork("eth-main", 15625301);
        // NOT WORKING
        // https://etherscan.io/tx/0x96a129768ec66fd7d65114bf182f4e173bf0b73a44219adaf71f01381a3d0143
        vm.transact(hex"96a129768ec66fd7d65114bf182f4e173bf0b73a44219adaf71f01381a3d0143");

        // WORKING
        address from = address(0x66C3c5D24BaE428A69c72117fd2f84B69F697Aed);
        address to = address(0x90F765F63E7DC5aE97d6c576BF693FB6AF41C129);
        uint256 value = 0;
        bytes
            memory data = hex"0a5cb529000000000000000000000000430a35baa51ddeaccf89092a5edbda47aaae78e400000000000000000000000000000000000000000000000000000000000000e000000000000000000000000039aa39c021dfbae8fac545936693ac917d5e7563000000000000000000000000000000000000000000000000000000396a23a87f000000000000000000000000a0b86991c6218b36c1d19d4a2e9eb0ce3606eb480000000000000000000000000000000000000000000000000000000000003887000000000000000000000000000000000000000000000000000000000000012000000000000000000000000000000000000000000000000000000000000000125a65726f4578417069416461707465725634000000000000000000000000000000000000000000000000000000000000000000000000000000000000000003c87a1eb1b900000000000000000000000039aa39c021dfbae8fac545936693ac917d5e7563000000000000000000000000a0b86991c6218b36c1d19d4a2e9eb0ce3606eb4800000000000000000000000000000000000000000000000000000000000000a0000000000000000000000000000000000000000000000000001f6064abb01a13000000000000000000000000000000000000000000000000000000001f35739e0000000000000000000000000000000000000000000000000000000000000002000000000000000000000000000000000000000000000000000000000000004000000000000000000000000000000000000000000000000000000000000001800000000000000000000000000000000000000000000000000000000000000003000000000000000000000000000000000000000000000000001cf6844fb63f74000000000000000000000000000000000000000000000000000000000000006000000000000000000000000000000000000000000000000000000000000000c000000000000000000000000000000000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000300000000000000000000000039aa39c021dfbae8fac545936693ac917d5e7563000000000000000000000000c02aaa39b223fe8d0a0e5c4f27ead9083c756cc2000000000000000000000000a0b86991c6218b36c1d19d4a2e9eb0ce3606eb480000000000000000000000000000000000000000000000000000000000000003000000000000000000000000000000000000000000000000000269e05bf9da9f000000000000000000000000000000000000000000000000000000000000006000000000000000000000000000000000000000000000000000000000000000c000000000000000000000000000000000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000300000000000000000000000039aa39c021dfbae8fac545936693ac917d5e7563000000000000000000000000dac17f958d2ee523a2206206994597c13d831ec7000000000000000000000000a0b86991c6218b36c1d19d4a2e9eb0ce3606eb48869584cd00000000000000000000000010000000000000000000000000000000000000110000000000000000000000000000000000000000000000c26f2d71f363330926000000000000000000000000000000000000000000000000";
        vm.prank(from, from);

        (bool success, ) = to.call{value: value}(data);
        require(success);
    }

Same issue here with
forge 0.2.0 (440ec52 2024-04-11T00:22:09.585245000Z)

What does the error mean?


Missing progressive response support

Basically I needed the requestId in order to post my progressive response to the amazon api endpoint. But when I was debugging I looked at the alexa-app source code and realise that the request.requestId filed was missing. Could this be added ?
See for further info : https://developer.amazon.com/docs/custom-skills/send-the-user-a-progressive-response.html

Nevermind I found it. Everything else is in the data attribute

Yes, to be clear, inside alexa-app, request is a custom object. The data attribute of the request contains the raw request exactly as sent from Alexa to the skill.


I want to increase frame of border line of Calendar

Hi,
I want in changes height of horizontal and width of vertical borderline.

What do you mean for borderline? Could you give me a screenshot?

Yes Border line

and do not want alpha between hours divider .

Can you show me a picture? It is really hard to get what you want.


Model instance aggregation "Schema Based Metadata" does not use consistent styling

The styling can be a bit hard to follow in the "Schema Based Metadata" section - notably, the placement of buttons and whitespace.  In the attached example, the "Collapse" and "Object Properties" for "SWAT Model..." wrap in a somewhat unintuitive way.  On very large width metadata windows, it looks great, however, the default sizing (or similar - up to about 2x the default width) has a lot of wrapping and associated whitespace/styling detials that can be hard to follow.
Possible resolution by opening a separate modal/popup for the Schema Based Metadata, otherwise forcing a wide metadata side panel, or using different styling.

@horsburgh and @goodalljl I believe this falls under Resource Model -> Model Instance aggregation. Labeling as an enhancement as it pertains to styling.

The text on the green button has been changed to indicate that the same button can be used for hiding or showing schema contents. This change is currently on prod.

@horsburgh Is this change is sufficient to close this issue?


Upgrade to Boost 1.69.0

Description
Math should use Boost 1.69.0 because
https://github.com/boostorg/math/issues/134
fixes a build error with integrate_1d on Solaris.
Example
Try to use integrate_1d on Solaris
Expected Output
A value of the integral
Additional Information
None
Current Math Version
v2.18.0

closed via #1082


AttributeError: module 'tensorboard.plugins.pr_curve.summary' has no attribute 'pb'

I am using Pycharm.
Python = 3.6.8
I uninstalled tensorflow-tensorboard and then installed tensorboard as @zainozzaini mentioned in some other issue. But it is not working for me. Please help me with this issue. Thanks in advance

It is working now. Thank you.


Fix 32 bits support in ios.cmake

32 bits support was dropped starting from iOS 11.0, but it is still possible, with the iOS 11.x SDK, to build for 32 bits architectures and actually you should do that if the deployment target is < iOS 11.
In other words, it is possible to use the iOS 11.x SDK and target iOS < 11: if that is the case, it is most likely necessary to support 32 bits.

Closing inactive pull request.