Integration testing fails due to not handling new popup boxes

Basic Information
The integration test for the queue page fails due to not handling the new popup box
Other Information
Don't need to fix this, just including for future reference

close due to wontfix


🐛[BUG]: ModuleNotFoundError: No module named 'warp'

Version
[0.4.0] - 2023-11-xx
On which installation method(s) does this occur?
Pip, Source
Describe the issue
running darcy_fno model I got the error below:
Traceback (most recent call last):   File "/media/adminisatrator/SSD1/nvidia/self-peace/darcy_fno_modulus_launch/train_darcy_fno.py", line 12, in <module>     from modulus.datapipes.benchmarks.darcy import Darcy2D   File "/home/adminisatrator/.local/lib/python3.10/site-packages/modulus/datapipes/benchmarks/darcy.py", line 18, in <module>     import warp as wp ModuleNotFoundError: No module named 'warp'
Can anyone help to fix it?
Minimum reproducible example
No response
Relevant log output
No response
Environment details
No response

I missed package warp-lang as mentioned in readme.


--watch does not work for T.input

See also related issue #832 and it's fix 2b3fb0578233de00feea5f34a09a6736e0e456ce.

Created a PR #839 which tackles the issue for most common types: PathRef, Seq[PathRef], os.Path and Seq[os.Path].

A possible solution looks like this (copied from #839): We want to plumb the lambda that T.input captures and change the rest of the RunScript infrastructure that does watching/etc. to know how to handle either PathRefs or lambdas, instead of being hard-coded to PathRefs as it is now. We could even re-use the ammonite.interp.Watchable type, since that models exactly this and is used for exactly the same use case


Fix table name in dbStatements

Changed table name to "cpeEntry" to avoid SQL errors on MySQL (case sensitive)

Thanks!


Fixing the android build issue with docker

With the docker image, there were errors in fetching the libc++shared.so file from the ndk.
Adding android-33 platform in the docker file

looks like one of the jobs failed. I don't have permissions to view the details.
@anhappdev, can you share the failure logs please?


please update npm to version 0.1.28

OK. I will update npm package once I finish a new version.

@borisirota It has been updated to 0.1.29.  https://www.npmjs.com/package/spectre.css


Display all possible combination of attribute values

Currently just the values are listed, but not the proper combination.

See https://github.com/sphereio/sphere-sunrise/issues/199

That was already solved in v0.3.0


Add an option to clean() only one time at start/init (inside watch-mode)

Hey there,
would be nice to have this option to clean up  only on start my dev-server or every x-times.
Reason is that cleaning up my folders takes 0.5 - 1.0 sec :-)
thx!
Greetings
crazyx13th

It is already an option:
new CleanWebpackPlugin({
    cleanStaleWebpackAssets: false,
});

Will this option work correct in prod mode, or I should create two different options of the plugin for each build mode?


pathlib support bug - TypeError: can only concatenate str (not "WindowsPath") to str

https://github.com/takluyver/bookbook/blob/50678bfaef28f18ac51dfbaf79247fb0d79e9529/bookbook/latex.py#L79
Full Traceback:
Traceback (most recent call last):
  File "C:\Python37\lib\site-packages\bookbook\latex.py", line 77, in combine_notebooks
    combined_nb.cells.extend(add_sec_label(nb.cells[0], nbname))
  File "C:\Python37\lib\site-packages\bookbook\latex.py", line 53, in add_sec_label
    raise NoHeader
__main__.NoHeader

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Python37\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Python37\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Python37\lib\site-packages\bookbook\latex.py", line 163, in <module>
    main()
  File "C:\Python37\lib\site-packages\bookbook\latex.py", line 160, in main
    combine_and_convert(args.source_dir, args.output_file, args.pdf, args.template)
  File "C:\Python37\lib\site-packages\bookbook\latex.py", line 144, in combine_and_convert
    combined_nb = combine_notebooks(notebook_files)
  File "C:\Python37\lib\site-packages\bookbook\latex.py", line 79, in combine_notebooks
    raise NoHeader("Failed to find header in " + filename)
TypeError: can only concatenate str (not "WindowsPath") to str

Looks like the line linked above just needs str(filename) so that the NoHeader error gets raised correctly. This won't solve my real problem of something to do with a header, but it's a bug nonetheless.

Yup, I think you're right. Or it could use .format() to put the path into the message. Do you want to make a PR?


Fix broken image on GitHub Integration

Fix the broken image URL at the bottom of the page
Test plan

Visit https://deploy-preview-235--okteto-docs.netlify.app/docs/self-hosted/administration/github/#verifying-your-installation and the image should display correctly

Add the GitHub App information to your Helm configuration file
^ does not link to an anchor on the page.  looks like that section is missing

Reconfigure your Okteto instance
^ does not link to an anchor on the page.  looks like that section is missing

Add or Remove private repositories
^ does not link to an anchor on the page.  looks like that section is missing

from:
https://deploy-preview-235--okteto-docs.netlify.app/docs/self-hosted/administration/github/#adding-or-removing-private-repositories


Laravel development server failed to listen on <IP_ADDRESS>:8000

Laravel Version: 8.0.3
PHP Version: 7.3.9 with all necessary extensions enabled
Database Driver & Version: MySQL 8.0.17
OS: Windows 10

Description:
After running composer create-project --prefer-dist laravel/laravel blog to create a new project the php artisan serve command fails to start the server. The terminal output is:
Starting Laravel development server: http://<IP_ADDRESS>:8000
[Thu Sep 10 14:02:24 2020] Failed to listen on <IP_ADDRESS>:8000 (reason: ?)

It tries ten times before exiting. I have tried to start it on a different port but this also fails.
Other projects on the same machine running Laravel Framework 7.22.4 can all start the server.
Steps To Reproduce:
composer create-project --prefer-dist laravel/laravel blog
php artisan serve

We don't use Windows ourselves and we've had other reports that this works for people on Windows.
See https://github.com/laravel/framework/issues/34229

Why did you close this issue.
I have the same problem.

Laravel Version: 8.0.3
PHP Version: 7.3.9 with all necessary extensions enabled
Database Driver & Version: MySQL 8.0.17
OS: Windows 10

Description:
After running composer create-project --prefer-dist laravel/laravel blog to create a new project the php artisan serve command fails to start the server. The terminal output is:
Starting Laravel development server: http://<IP_ADDRESS>:8000
[Thu Sep 10 14:02:24 2020] Failed to listen on <IP_ADDRESS>:8000 (reason: ?)

It tries ten times before exiting. I have tried to start it on a different port but this also fails.
Other projects on the same machine running Laravel Framework 7.22.4 can all start the server.
Steps To Reproduce:
composer create-project --prefer-dist laravel/laravel blog
php artisan serve

I solved the problem by this link

change php.ini-development to be php.ini

i already have a php.ini it doesnt work for me im on windows 10

php -S <IP_ADDRESS>:80 -t public/ worked for me.

php -S <IP_ADDRESS>:80 -t public/ Works for me too


doc: base installed tools should not be wrapped

It is unreasonable amounts of work to add those packages to all packages that require them. The simpler and better to maintain solution for nixpkgs maintainers is to add those packages on systems that do not ship them.

And PATH has a size limit. It cannot be expanded indefinitely and if every program adds the same or similar entries without deduping them some badly written programs will crash.

Originally posted by @SuperSandro2000 in https://github.com/NixOS/nixpkgs/pull/208333#discussion_r1063508244
this notion of "not wrapping base tools" is novel, and does not appear documented in the manual or other nixpkgs usage. should we add something so that other maintainers are not also confused?

I don't agree. There are a number of situations where programs are run in nearly empty paths (notably systemd units), so wrapping even basic tools is useful.

as a meta point, maintainers have the option to also replace bare program names with /nix/store paths, as is done for shared objects; generally this is a lot of work, and wrapping is just easier, but for well described languages it is possible to do this automatically: bash has resholve, see docs.

Instead of wrapping by adding each individual item to PATH, how about building an environment using symlinkJoin or similar and than add that instead? That's what I've been doing in general to avoid exactly the issue of PATH length limitations.

interesting, it might be nice to have that as a turnkey function like:
"wrapProgram $out/bin/cmd --prefix PATH : ${symlinkBinPath [
  coreutils
  gnugrep
  git
]}"

makeBinPath = paths: symlinkJoin {
  name = "bin-path";
  inherit paths;
};

* They ignore users PATH priority and tool choice. If I have another compatible implementation in my PATH already or use a wrapper then they are overwritten. If there is no need to overlay a binary such fallback paths should be always suffixed. We had this problem with xdg-utils already and suffix is in most cases the correct choice.

I am pretty mixed on this one. On the one hand, sure some users may have configured ssh to their liking with builtin configuration options, but plenty of non-nixos systems also bundle fundamentally incompatible system utilities. I am concerned about assuming that an implementation is "compatible", simply because it is in PATH. Another situation is tools that both link against a shared object and shell out to an executable. In this case we pin to the nixpkgs version for the library, but defer to PATH for the binary.

* it makes delivering security updates for programs harder because they will trigger more rebuilds for more packages. See the recent openssh removal from rustPlatforms inner workings.

Mind giving a pointer to this?

* it will make using overlays harder because they would cause more rebuilds which are not justified.

IIUC, this issue is greater than just PATH wrapping, since updating a shared object dependency will trigger a similarly "unjustified build". If the old build could just reference the new, compatible library out of /usr/lib/, then a full rebuild would not be necessary.

  * maybe we can implement this in a way to avoid rebuilding packages completely.

Yeah, I got this working a while back by doing all the makeWrapper stuff in a separate derivation that consumed the main one, but was cautioned that this would close to double the number of derivations in nixpkgs. If we are already paying that cost with symlinkJoin trick, it seems reasonable to have that derivation also setup PATH instead of using the main derivation's installPhase. I would even wager we could write it up such that the mkDerviation api need not change too much.

  * Maybe saving phases to the nix store could help but that's in the work for a long time and will probably not ready in the next months.

Is there an thread tracking this? I would be keen to follow along.

* not every program works with wrapping and we are not there yet that the wrappers are completely hidden. Also we want to migrate to binary wrappers by default before starting with this.

Personal curiosity, but should we be using binary wrappers wherever possible today?

How about we expose the default path in top-level as some core package and advice people to install that on anything that is not a recent linux with gnu tools?

@roberth seemed amenable to even going so far as to offering pkgs install as part of the nix install process, over here: https://github.com/NixOS/nix/issues/7566#issuecomment-1374821066. Does somebody want to spawn that new issue, or should I?

Mind giving a pointer to this?

openssh a while ago, also currently git. If we want to deliver security updates faster we cannot wrap everything that might be required into a program.
Generally the more we wrap, the more rebuilds will be triggered unless we move the wrapping to an extra build steps or indicate required binaries in another way that does not trigger a rebuild.

Is there an thread tracking this? I would be keen to follow along.

I don't know the exact title of and couldn't find it.

Personal curiosity, but should we be using binary wrappers wherever possible today?

Yes because eg. on macos a script interpreter must be a binary and cannot be a shellscript. Also they are slightly faster.


Restrict access to Sidekiq admin interface to admin users only

Blocked by #37
Once we can track a user's group membership, we can restrict access to the Sidekiq interface based on UMG membership. The current scholarsphere admin group is up.dlt.scholarsphere-admin so any member of that group should have admin privileges.

We use a constraint block in the routes with https://github.com/psu-stewardship/scholarsphere/blob/master/config/initializers/resque_admin.rb


Publish on Maven Central

Any chance this could just be published on maven central? I'm having a lot of abnormal problems when using this library. Plus, it's also not nice that if I create another library which imports this library... Any app which depends on my library needs to also add the KHash repo to their settings.gradle.kts. It's really non-standard to have to do that for kotlin.

I would really like to avoid publishing to maven central. IMHO rather the problems resulting with publishing to the github repo should be fixed if there are any. I myself consume the lib via jitpack - unfortunately this does not work for multi platform.

I would really like to avoid publishing to maven central. IMHO rather the problems resulting with publishing to the github repo should be fixed if there are any. I myself consume the lib via jitpack - unfortunately this does not work for multi platform.

@ligi I think you may not have clicked this button


RDBES: RS_ReasonForNotSampling update of codes

Please see attached spreadsheet for the 11 code updates.
RS_ReasonForNotSampling updates.xlsx

@HenrikK-N  Explain the difference between No Answer and Unknown - since they mean the same thing (looking at the description for unknown) - Also RMG would like a clarification of Not Available also.
RMG suggest to have refusal in the description instead of decline e.g Industry refusal.

Another question here is if "Other" is necessary. Could "Other" be covered by "NoAnswer" or "Unknown"? An alternative is also to add new codes to the vocabulary if necessary. The reason why the RMG is bringing this up is because in other systems we avoid the code "Other", and instead prefer to add codes if necessary, since "Other" is not informative.

Agree with spelling out refusal when it is so, like Industry_refusal; we have a case not seen here in the file when a vessel master cannot embark an observer for security reason (vessel too small, no authorisation to embark a special staff, ...); this case is interesting to evaluate the target or sampled population; for your consideration.

-To modify to IndustryDecline, as Refusal sounds too harsh. Industry declines to allow sampling of fish or to allow an observer on board, e.g. for logistical reasons.
-ObserverDecline, Observer declines to collect the sample or go onboard the vessel e.g. for logistic reasons.
-Unknown- Not recorded in the paper work
-No answer- Contact attempted but no answer received.
-Other: required for the time being as many other codes might be needed- Covers all other reasons

I promised to write the list of codes for which the ‘, ref. SGPIDS report 2013’ should be added to the description (not long description). The codes are: NotAvailible, NoContactDetails, ObserverDecline and IndustryDecline. The codes for 'ObserverDecline' and 'IndustryDecline' should be updated with a 'd' and the end to 'ObserverDeclined'.


[Discussion] Can't seem to get the GPU working for training; maybe I'm to early in the pipeline to see it?

I've been trying for the past week to get GPU acceleration working and I don't think I'm getting it. I've tried a dedicated VM, the container, version 2.3.0 & 2.2.2.. I've lost count of all the different cuda, tensorflow configurations I've tried.. They all work when I test tensorflow directly for GPU access (I test pyTorch as well) and it seems to have acess.. I thought I got it to work once, I saw GPU memory go up to 20GB (half of the 40GB that was available) but after 6 hours the most I ever saw was the GPU spike up to 15% for 1 sec. Next time I tried the same job on a different config the memory never went above 300MB and GPU usage never went above 0%. When I run nvidia-smi I see the python job listed.
Now I'm not sure if this is because I haven't figured out how to get the GPU setup properly or if there is a very long and costly loading phase (I have about 450k wavs). I see the wavs being loaded in the STDOUT logging, they are getting cached to disk.. But should I be using GPU during this phase or do all the files need to be converted before training really starts?
Can someone please provide a brief explanation of what to expect when the training is working properly.. I can't tell if I'm doing something wrong or not..

Hi @dustyny,
The spleeter data pipeline for training uses audio features (spectrograms) caching, and the spectrogram computation is done on CPU. If you have a quite big database, spectrogram computation can take quite long and during this time, GPU won't be much used. To check that the GPU are correctly used, I would recommend to try with a fake test config with a very small dataset : for such a config, spectrogram computation and caching should be quite fast and the GPU should start to get used a lot quite soon (if correctly setup).

@romi1502 thanks for confirming my suspicion. Is there a way to split the spectrogram computing stage out? Right now I'm limited to 12 CPUs when using a GPU but if I can split the stages, I can use 96 CPUs to create the spectrograms and then switch to an instance with a GPU and run the training.

One more related question.. I'm trying to get a sense of how long caching is going to take. I have a 500GB training set, right now I see the cache is taking about 113GB.. But I don't know if the is a 1:1 relationship between the size of the wav file and the data generated in the cache, if it's less or more..
Should I expect to see a cache that's smaller, larger or the same size as the training set? If it's smaller or larger, roughly how big is that difference?
Thanks!


explicit call to commit after part.setValue in directive

Hi,
I'm currently trying lit-element/lit-html with firebase on a pet project.
I made a firebase directive, to automatically update content when a new data event fire.
Here is the directive
    //ref is a firebase reference, content and defaultContent lit-html template
    const onFirebaseData = (ref, content, defaultContent) => directive(part => {
        part.setValue(defaultContent);
        ref.on("value", snap => {
            let data = snap.val();
            if (data !== undefined) {
                part.setValue(content(data))
            }
        })
    });

After being busy for about one month, I came back to the project, upgraded to latest versions, and the directive stopped working.
After looking in the code, I managed to get it do it's job again by adding
part.commit() after the part.setValue()
new version
    //ref is a firebase reference, content and defaultContent lit-html template
    const onFirebaseData = (ref, content, defaultContent) => directive(part => {
        part.setValue(defaultContent);
        ref.on("value", snap => {
            let data = snap.val();
            if (data !== undefined) {
                part.setValue(content(data))
                part.commit()
            }
        })
    });

It seems strange to me to add this call for the same behavior and I don't see any advantages in this, so I'm not sure if I'm missing something, and using it wrong.
There is also no mention I could found in the documentation
directives
of this call to commit.
Can someone explain me what is going on here ?
Thanks in advance.

Yeah, I need to update the docs (cc @arthurevans ), but what's going on here is that there are two phases to updating DOM: the setValue phase, and the commit phase. DOM is only mutated in the commit phase. This split exists so that we don't update attributes or properties multiple times if they have multiple bindings.
Directives are invoked during the commit phase, but you can call setValue() in a directive and the new value will still be committed. However if you call setValue at a later time, when not already in a commit, such as in the event callback you have, you need to manually call commit, or the new value just sites there.

I believe the docs now cover when commit is required over just setValue: https://lit-html.polymer-project.org/guide/creating-directives#asynchronous-directives.
Closing as resolved.


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

7ea8a01: Added new Property Converter FunctionType as FunctionT

This pull request includes updates to the @ayu-sh-kr/dota-core package, specifically adding a new property converter and updating the package version.
Version updates:

package.json: Updated the version from 1.6.9 to 1.6.10.

Changelog updates:

CHANGELOG.md: Added a new entry for version 1.6.10 to document the addition of the new property converter FunctionType as FunctionT.

Documentation updates:

.changeset/many-days-dress.md: Removed the changeset entry for the new property converter function type.


Reduce minimum distance registering as location change to 5m

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
Reduce minimum distance registering as location change to 5m (currently 10m).
Describe alternatives you've considered
N/A
Additional context
N/A

Change made in https://github.com/StAResComp/sifids/commit/062d18a505859060bfdaa51bd1a04337723f3a4e


couldn't extract plugin archive: zip: not a valid zip file

When I execute the check plug-in command, docker run -- pull=always grafana/plugin-validator-cli http://xxx.zip Or docker run -- pull=always grafana/plugin-validator-cli - sourceCodeUri http://xxx http://xxxx/xx.zip There will be the following errors

Hi! Based on the error message, it couldn't extract the zip file https://github.com/grafana/plugin-validator/blob/6da5c1ca3bc71cb35be50264dfa8e1d3d3c2920e/pkg/cmd/plugincheck2/main.go#L60-L66
So worth checking, is it a real zip file? How are you packaging it?
Here's what I've used in my projects, which seems to work fine.
mv dist $MY-PLUGIN-ID
zip $MY-PLUGIN-ID.zip $MY-PLUGIN-ID -r

Hi @CloudWise-Lukemiao are you still having problems with this? I agree with @eh-am where it looks your zip file was corrupted or incorrectly packaged.


punctuation error: missing apostrophe

This string:

Azure AD Connect synchronizes a hash, of the hash, of a users password

should likely read:

Azure AD Connect synchronizes a hash, of the hash, of a user's password

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

ID: aad82713-d562-5778-fbf8-035e24f9c470
Version Independent ID: 6370cca0-c78e-4738-ba67-8fb14594b0f2
Content: What is password hash synchronization with Azure AD?
Content Source: articles/active-directory/hybrid/whatis-phs.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@cqueern
Thanks for your feedback! We will investigate and update as appropriate.

@cqueern
I have created a PR to update the documentation and will now close out this issue. Please allow some time for the changes to reflect and if you don't see the document update, please let me know.
I will be more than happy to re-open this issue and look into what happened.
Thank you for your time and patience!


add a favicon

we need a favicon
(not really but it's better then not having one at all imo)

how's this

it's alright.

addressed by 75904e3b7bf843a795f30ed44cfe5bfca96b6c82


Calibration values

Would you please give any comparable values for kinect2 calibration. I've succeed to obtain values without error. But what are the means of those values, coeffs,matrix etc ?.
Thank you
yselim

Any comment please?


Add Discussions to list of supported products for saved replies

Why:
Closes #21202

What's being changed (if available, include any code snippets, screenshots, or gifs):
Changes this:

Once you've added a saved reply, it can be used in both issues and pull requests.

to this:

Once you've added a saved reply, it can be used in issues, pull requests, and discussions.

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

و

[ ] و


dns log filter is buggy

utc handling looks wrong, as 'db' will be all UTC times, need to be more careful with the conversions here
if picking a date into the future, theres no way to get back without refreshing the page as the time filter disappears

unclear if date picker is supposed to be min date or max date or if its supposed to allow a range selection
another good feature would be to query multiple dns buckets

https://github.com/spr-networks/super/commit/aa4a70f270829c0c6d898a7d8a61fed76ba6237c
UTC part fix is up
Next -- why does date picker disappear?


Setup proxy in cloud-init boothook

Configure proxy during cloud-init boothook. This is the very first hook called by cloud-init and will allow to use the proxy also for update/upgrade action called by cloud-init itself.
This fixes https://github.com/aws/aws-parallelcluster/issues/1647
This patch removes /tmp/proxy.sh and setup proxy directly into /etc/environment. This change is paired with a patch in the cookbook code.
Signed-off-by: Luca Carrogu<EMAIL_ADDRESS>Please See Git Pull Request Instructions
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1672 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1672   +/-   ##
========================================
  Coverage    51.95%   51.95%
========================================
  Files           24       24
  Lines         3948     3948
========================================
  Hits          2051     2051
  Misses        1897     1897

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 a2b72c7...b0b3fbd. Read the comment docs.


Fix typo in README example

Here's a minor fix. Thanks for your excellent work. Setting up the sensor on my RPi was simple with your driver and python lib.

Thanks!


Move filtering elsewhere?

https://github.com/GeorgeVassilakis/SMPy/blob/e9d6f7e51a8167c2a00bf6cc3d8687a0d92d3a1a/SMPy/KaiserSquires/plot_kmap.py#L6
Future development may see the use of other smoothing kernels (some kind of wavelets approach, perhaps?). Moving the smoothing operation outside of the plotting function would make it easier to implement different smoothings.

An output converge FITS file could have 1 layer be raw convergence, and another be the smoothed convergence in the same FITS file. Note this for development.


Read out the remaining, still missing values from Modbus

Such as the outside temperature of the heat pump, external room temperature, etc. There is some certainly more.

With this I could remove the normal Modbus query from HA.

I had also been looking for a long time but that is the outside temperature of the heat pump.
Unfortunately, I cannot tell you why it is the same address as the boiler. But it works.

@Vaszago Alright can you try to add the boiler in the setup and check if the values match?

Unfortunately I don't know how to include this entry in the setup!

Currently, it is not possible I guess, as heat pump and bio mass boiler are exclusive, as currently implemented in the integration.
You would need to add another instance of the integration and configure it to only Bio mass boiler.

I've just checked. We would need to adapt the register layout for vampair for bio mass boiler.
I'll look at it later.

It actually seems to work already.
Is the precision the same for you ie, no 10th degree or so?

Also posdible to add the modbus features fir circulation pump? Adress e.g. 32003

Also posdible to add the modbus features fir circulation pump? Adress e.g. 32003

That is already available when the Boiler component is enabled. There is a button to trigger the circulation (button.solarfocus_boiler_1_circulation) and a sensor for reading the current state of the circulation (sensor.solarfocus_boiler_1_circulation)

Oh, Thanks, did not recognize this.
Now I can check that, Thank you :)

button.solarfocus_boiler_1_circulation

Is there also possibility to start stop the heating circulator pumps? i see only the binary sensor to check the state. and when i change heat mode, it fdoes not stop the circulator pumps...

button.solarfocus_boiler_1_circulation

Is there also possibility to start stop the heating circulator pumps? i see only the binary sensor to check the state. and when i change heat mode, it fdoes not stop the circulator pumps...

@bszattila Unfortunately, there is no control provided by Solarfocus currently. However, for me the behavior is as expected from you: if I turn of the heating the circulation pump of the heating also stops. So maybe contact the Solarfocus support why this is the case in your installation?

button.solarfocus_boiler_1_circulation

Is there also possibility to start stop the heating circulator pumps? i see only the binary sensor to check the state. and when i change heat mode, it fdoes not stop the circulator pumps...

@bszattila Unfortunately, there is no control provided by Solarfocus currently. However, for me the behavior is as expected from you: if I turn of the heating the circulation pump of the heating also stops. So maybe contact the Solarfocus support why this is the case in your installation?

Thank you! I contacted them, let's see what their answer ;) i share if it is modbus related, and not my pellet broken

Really strange - tried the button "request circulation" as well which works. However, not possible to stop the circulation pump viy TCP / Modbus.
That would be such a great, energy saving feature (e.g. for triggering the circulation via smart switch in bathroon and stop it after 30s).
I'll contact Solarfocus as well...

button.solarfocus_boiler_1_circulation

Is there also possibility to start stop the heating circulator pumps? i see only the binary sensor to check the state. and when i change heat mode, it fdoes not stop the circulator pumps...

@bszattila Unfortunately, there is no control provided by Solarfocus currently. However, for me the behavior is as expected from you: if I turn of the heating the circulation pump of the heating also stops. So maybe contact the Solarfocus support why this is the case in your installation?

Thank you! I contacted them, let's see what their answer ;) i share if it is modbus related, and not my pellet broken

Really strange - tried the button "request circulation" as well which works. However, not possible to stop the circulation pump viy TCP / Modbus.
That would be such a great, energy saving feature (e.g. for triggering the circulation via smart switch in bathroon and stop it after 30s).
I'll contact Solarfocus as well...

O, i already got feedback on my enquiry. They are saying:
that’s because the outside temperature is low, the frost protection mode is active.
If the outside temperature drops under this parameter the pump will run 24h.
If you decrease this parameter to minus temperatures it will switch off (attached a picture where You can setup the antifreeze temperature, which in our case is set to 5 degrees)
But in the worst case some heating water pipes can freeze if nobody watch the system and the boiler is switched off or has a fault.

Good Point - so checking heating state makes sense in addition. This sensor will show if e.g. frost protection is active.

It actually seems to work already. Is the precision the same for you ie, no 10th degree or so?

Is the outside temperature already integrated as sensor when installing the Integration? I cannot see it when Integrating the vampair with boiler... otherwise, one can use the integration in config.yaml from vaszago

Good Point - so checking heating state makes sense in addition. This sensor will show if e.g. frost protection is active.

I am recording the heating state on a graph, and i observed that it returns different values. is there any reason for it? sometimes it returns value 5,8,12, 17 and 0 of course, when heating stops. Do You know what is the difference between the values?

i made the update, and now i have this issue: %key:common::config_flow::error::cannot_connect%]

@bszattila the numbers refer to the Enumeration from the Specification cf. Status Heizkreis

@PhilippRettenmeier yes, the outdoor temperature sensor of the Vampair seems to be available as sensor of the Biomass Boiler. Therefore, to make use of this sensor, you need to add the Solarfocus integration twice:

Configured as VampAir heat pump + the respective components
Configured as Biomass Boiler Therminator. <-- afterwards you can delete all sensors but the outdoor temperature, as the other sensor values are invalid. Make sure you name the second instance differently

Great, works, thanks a lot


Bug: When I adjust the volume with the touchpad, the standard Windows version is displayed instead of the ModernFlyouts.

When I adjust the volume with the touchpad, the standard Windows version is displayed instead of the ModernFlyouts.

To Reproduce:
Steps to reproduce the behavior:

Use the touchpad to increase or decrease the volume.
Windows standard ones are displayed instead of ModernFlyouts.

OS Version:

Windows 10 Version: Windows 10 Version 20H2

ModernFlyouts Version:

Version: v<IP_ADDRESS>

Additional context:
This issue does not occur when using the volume control keys on the mouse or keyboard.

Hi, thanks for letting us know.
Will close this as it is a duplicate of #4. See #4 for more updates on thus issue.


Document new function translations in 7.0

[ ] https://github.com/dotnet/efcore/issues/25396
[ ] https://github.com/dotnet/efcore/issues/28078
[ ] https://github.com/dotnet/efcore/issues/28110
[ ] https://github.com/dotnet/efcore/issues/28115

May I ask, do we use string.Join normally or by some method defined here?

@amunim for string.Join you can just use that function as you normally would, e.g. within a Select following a GroupBy.

Thank you


Explore and implement options for FIT to add value for developers (e.g., provide tool use metrics, maintain builds, other development support).

Adding value might include providing support, such as collecting resources like this r open sci guide on what a package maintainer does. I like that this post's point about ownership.

Defining what the FIT provides developers (and other groups) was also brought up during our FIT scoping session.

Related to #58, we discussed at the FIT committee meeting on 6/25 notes. We only got to discussing how R universe could add value for developers. We discussed how:

R universe allows users to more easily install packages (a cran like experience) because they are precompiled, which may result in less installation questions for developers
tracks installs, provides a webpage that developers can use
You can easily see when builds fail, which may provide info for developers who have not set up automated r cmd check github actions.

Downside: All of this is only available for R packages and not other tool formats.
Action items:

[ ] Share the FIT R universe more widely on the FIT website
[ ] Let developers know about the FIT R universe
[ ] Include R universe in the onboarding process.

FIT R universe is now linked directly on the FIT homepage! There also is a blog post about R universe, including a section about what R universe provides to developers.
As for "let developers know about the FIT R universe", I added this to communication plan, so I think we can close out this ticket.


pygame-ce compat with weird missing dlls

Thank your for contributing to Nuitka!
!! Please check that you select the develop branch (details see below link) !!
Before submitting a PR, please review the guidelines:
Contributing Guidelines
What does this PR do?
pygame-ce is a fork of pygame that uses a custom SDL Image build. For whatever reason, libpng16-16.dll and libjpeg-62.dll don't get copied into the build. I'm hoping this is a temporary fix (I tested that this seems to still work fine with the original). We have our own issue here that I'm hoping we can eventually resolve on the library side, rather than Nuitka's side. But for now, this is the only solution I've come up with that doesn't require us to point every user to include our vendored in binaries for png and jpeg
Why was it initiated? Any relevant Issues?
pygame-ce issue linked above, no Nuitka issue
PR Checklist

[x] Correct base branch selected? Should be develop branch.
[x] Enabled commit hook or executed ./bin/autoformat-nuitka-source.
[ ] All tests still pass. Check the Developer Manual about Running the Tests. There are GitHub
Actions tests that cover the most important things however, and you are welcome to rely on those,
but they might not cover enough.
[ ] Ideally new features or fixed regressions ought to be covered via new tests.
[ ] Ideally new or changed features have documentation updates.

reproducer.zip
Here's the sample project I was working with
Build command:
nuitka --standalone main.py --include-data-files=./alien1.png=alien1.png --include-data-files=./alien1.jpg=alien1.jpg --include-data-files=./Roboto-Regular.ttf=Roboto-Regular.ttf --include-data-files=./boom.wav=boom.wav

It’s been pointed out to me that potentially more dlls will have issues, so I need to convert this to a draft

I recommend installing the commit hook; it autoformats your changes at commit time, which will address the CI error.

reproducer.zip
New version of the reproducer. I ultimately decided that pygame(-ce) needs to glob all dlls because they're runtime dynamic linking. It would be great to be able to have package-level entrypoints like pyinstaller does so that this doesn't have to belong in Nuitka, but I suppose this is how it must be for now.
Note that this version of the reproducer fails on both pygame and pygame-ce, it just so happens that libpng is fine on the original because it's not dynamically linked by their version of sdl-image like pygame-ce's is. I could list out all the runtime dll dependencies for one or the other here, but that would break the other version, so wildcard glob is the best choice here

custom SDL Image build

It’s not that it’s custom, it’s that it’s new. And it also doesn’t include webp and tiff DLLs. And also DLLs for loading audio formats.

the reproducer doesn't reproduce, i get this
❯ python main.py
pygame-ce 2.5.0 (SDL 2.30.3, Python 3.11.9)
Traceback (most recent call last):
  File "C:\Users\NuitkaDevOps\Desktop\Nuitka-Issues\pyg_ce\main.py", line 17, in <module>
    pygame.mixer.music.load("MIDI_sample.mid")
pygame.error: mixer not initialized

the reproducer doesn't reproduce, i get this
❯ python main.py
pygame-ce 2.5.0 (SDL 2.30.3, Python 3.11.9)
Traceback (most recent call last):
  File "C:\Users\NuitkaDevOps\Desktop\Nuitka-Issues\pyg_ce\main.py", line 17, in <module>
    pygame.mixer.music.load("MIDI_sample.mid")
pygame.error: mixer not initialized

the reproducer doesn't reproduce, i get this
❯ python main.py
pygame-ce 2.5.0 (SDL 2.30.3, Python 3.11.9)
Traceback (most recent call last):
  File "C:\Users\NuitkaDevOps\Desktop\Nuitka-Issues\pyg_ce\main.py", line 17, in <module>
    pygame.mixer.music.load("MIDI_sample.mid")
pygame.error: mixer not initialized

That definitely shouldn't be happening, line 5 of main.py initializes mixer in the call to pygame.init(). Here's what I get using unmodified nuitka
pygame-ce 2.5.0 (SDL 2.30.3, Python 3.12.0)
Traceback (most recent call last):
  File "C:\Users\andre\Projects\temp\MAIN~1.DIS\main.py", line 37, in <module>
  File "C:\Users\andre\Projects\temp\MAIN~1.DIS\main.py", line 23, in __init__
pygame.error: Failed loading libpng16-16.dll: The specified module could not be found.

Oh and here's the build command I used
nuitka --standalone main.py --include-data-files=./alien1.jpg=alien1.jpg --include-data-files=./Roboto-Regular.ttf=Roboto-Regular.ttf --include-data-files=./boom.wav=boom.wav --include-data-files=./boom.ogg=boom.ogg --include-data-files=./alien1.png=alien1.png --include-data-files=./bg.tiff=bg.tiff --include-data-files=./MIDI_sample.mid=MIDI_sample.mi

i'm not using nuitka at all there, simply doing python main.py
Administrator in Desktop\Nuitka-Issues\_pygrepro via 🐍 v3.11.9 (venv)
❯ type main.py
import pygame
import pygame.freetype
import pygame.midi

pygame.init()
pygame.freetype.init()

screen = pygame.display.set_mode((500, 500))
clock = pygame.time.Clock()

# bg = pygame.image.load("bg.tiff")
# bg = pygame.transform.scale(bg, screen.get_size())

# pygame.Font()
pygame.freetype.Font("Roboto-Regular.ttf")

pygame.mixer.music.load("MIDI_sample.mid")
pygame.mixer.music.play(-1)

class Player:
    def __init__(self, pos: pygame.Vector2) -> None:
        self.pos = pos.copy()
        self.image = pygame.image.load("alien1.png")
        other_image_for_test = pygame.image.load("alien1.jpg") #noqa
        self.speed = 200 # pixels per second

    def move(self, direction: pygame.Vector2, dt: float) -> None:
        if direction:
            direction.normalize_ip()

        self.pos += direction * self.speed * dt

    def draw(self, screen: pygame.Surface) -> None:
        screen.blit(self.image, self.pos)

player_pos = pygame.Vector2(screen.get_rect().center)
player = Player(player_pos)

running = True
while running:
    dt = clock.tick() / 1000

    for event in pygame.event.get():
        if event.type == pygame.QUIT:
            running = False

    direction = pygame.Vector2()
    keys = pygame.key.get_pressed()
    if keys[pygame.K_LEFT] or keys[pygame.K_a]:
        direction.x -= 1
    if keys[pygame.K_RIGHT] or keys[pygame.K_d]:
        direction.x += 1
    if keys[pygame.K_UP] or keys[pygame.K_w]:
        direction.y -= 1
    if keys[pygame.K_DOWN] or keys[pygame.K_s]:
        direction.y += 1

    player.move(direction, dt)

    screen.fill("black")
    # screen.blit(bg, (0, 0))
    player.draw(screen)
    pygame.display.flip()

this is the error that happens when i do the command
Administrator in Desktop\Nuitka-Issues\_pygrepro via 🐍 v3.11.9 (venv)
❯ python main.py
pygame-ce 2.5.0 (SDL 2.30.3, Python 3.11.9)
Traceback (most recent call last):
  File "C:\Users\NuitkaDevOps\Desktop\Nuitka-Issues\_pygrepro\main.py", line 17, in <module>
    pygame.mixer.music.load("MIDI_sample.mid")
pygame.error: mixer not initialized

my main.py is the same as your main.py

Alright, you can just comment out these lines, they're not important to the reproducer. That's super weird though
pygame.mixer.music.load("MIDI_sample.mid")
pygame.mixer.music.play(-1)

in which case i suggest using
- module-name: 'pygame'
  data-files:
    patterns:
      - 'freesansbold.ttf'
  dlls:
    - from_filenames:
        prefixes:
          - 'lib'
          - 'SDL2'
          - 'freetype'
          - 'portmidi'
  options:
    checks:
      - description: 'PyGame is a GUI framework'
        macos_bundle: 'recommend'

rather than the glob pattern, this also pulls in

webp and tiff DLLs. And also DLLs for loading audio formats.

doing a quick scan for DLLs in pygame lists all these DLLs
freetype.dll
libjpeg-62.dll
libogg-0.dll
libopus-0.dll
libopusfile-0.dll
libpng16-16.dll
libtiff-5.dll
libwavpack-1.dll
libwebp-7.dll
libwebpdemux-2.dll
libxmp.dll
portmidi.dll
SDL2.dll
SDL2_image.dll
SDL2_mixer.dll
SDL2_ttf.dll

all of which are pulled by the yaml config above

Doing a search on “lib” is pretty similar to just grabbing all executables. It’s not like it adds more protection or anything?
But if we or SDL change how they build those, the lib prefix may not be included in the future (it’s really more of a Linux thing), so this seems more liable to break.
pygame-ce won’t ship DLLs in our wheels unless they are needed.

in which case
- module-name: 'pygame'
  data-files:
    patterns:
      - 'freesansbold.ttf'
  dlls:
    - from_filenames:
        relative_path: '.'
        prefixes:
          - ''
  options:
    checks:
      - description: 'PyGame is a GUI framework'
        macos_bundle: 'recommend'

but if you look at all our DLL configurations, nuitka doesn't like using glob patterns for DLLs, even if supported, maybe we should make that more explicit, we prefer being explicit and only including the DLLs we know that are absolutely needed

but if you look at all our DLL configurations, nuitka doesn't like using glob patterns for DLLs, even if supported, maybe we should make that more explicit, we prefer being explicit and only including the DLLs we know that are absolutely needed

I would generally agree with this, and I hate wildcarding it, but I don't see much of a choice here that will satisfy everyone. The ideal solution IMO would be for nuitka to have entry points that 3rd party libraries can implement themselves where they can handle the specification of dynamic libraries that don't get caught by the dependency checker in nuitka. However, that isn't an option right now. We have the complicating consideration that we have to share a config between pygame and pygame-ce because they both install to the pygame directory. Sure, we can do explicit like you suggest, but that's got two big problems for us to consider:

we have absolutely no say in what pygame does or what dependencies they use, so if they end up with a dependency we don't have that doesn't happen to match any pattern, then their stuff will break. I don't want to end up with Nuitka being in the middle of any future drama between pygame and pygame-ce that may result from one or the other breaking the dependencies of the other
we have a lot of dependencies that are constantly changing, and soon we'll have to worry more about what SDL 3.0.0+ is doing, which is going to be a lot. I don't want to have to come make a pull request to nuitka every time SDL changes something about their dynamic dependencies.

but I don't see much of a choice here that will satisfy everyone.

the empty relative dir and prefix will pull in all DLLs within the module. that should sastify everyone i believe.

pygame-ce won’t ship DLLs in our wheels unless they are needed.
provided that this keeps being true, i think we will be happy with the above approach, we just don't like including bloat DLLs as this greatly increases the final binary size

the empty relative dir and prefix will pull in all DLLs within the module. that should sastify everyone i believe.

out of curiosity, what's the difference between this and what I have? I don't mind making the change as long as the behavior is the same

Alright, I've made the changes. Thanks @KRRT7!

Administrator in Desktop\Nuitka-Issues\_pygrepro via 🐍 v3.11.9 (venv)
❯ python main.py
pygame-ce 2.5.0 (SDL 2.30.3, Python 3.11.9)
Traceback (most recent call last):
  File "C:\Users\NuitkaDevOps\Desktop\Nuitka-Issues\_pygrepro\main.py", line 17, in <module>
    pygame.mixer.music.load("MIDI_sample.mid")
pygame.error: mixer not initialized

As for this, I'm slightly concerned about it. Was this run on a local machine, or was it run though something that doesn't have audio capability?
It's possible mixer failed to initialize and the return value of pygame.init() would have revealed at least one module failing to initialize, but I don't immediately know what else could have caused that. If it was local, could you provide the output of pygame.print_debug_info()? (that function only exists in pygame-ce). I'd at least like to be able to open an issue about it on our repo so that it can be investigated if it's a legitimate issue

was it run though something that doesn't have audio capability?
that'd do it, it's running on an Azure windows server.

pygame-ce 2.5.0 (SDL 2.30.3, Python 3.11.9)
Platform:               Windows-10-10.0.22631-SP0
System:                 Windows
System Version:         10.0.22631
Processor:              Intel64 Family 6 Model 85 Stepping 7, GenuineIntel
Architecture:           Bits: 64bit     Linkage: WindowsPE

Python:                 CPython 3.11.9 (tags/v3.11.9:de54cf5, Apr  2 2024, 10:12:12) [MSC v.1938 64 bit (AMD64)]
pygame version:         2.5.0
SDL versions:           Linked: 2.30.3  Compiled: 2.30.3
SDL Mixer versions:     Linked: 2.8.0   Compiled: 2.8.0
SDL Font versions:      Linked: 2.22.0  Compiled: 2.22.0
SDL Image versions:     Linked: 2.8.2   Compiled: 2.8.2
Freetype versions:      Linked: 2.11.1  Compiled: 2.11.1

Display Driver:         Display Not Initialized
Mixer Driver:           Mixer Not Initialized

Apologies, I missed one additional step in there. Can you call pygame.display.init() and pygame.mixer.init() before the print?

pygame-ce 2.5.0 (SDL 2.30.3, Python 3.11.9)
Traceback (most recent call last):
  File "C:\Users\NuitkaDevOps\Desktop\Nuitka-Issues\_pygrepro\main.py", line 6, in <module>
    pygame.mixer.init()
pygame.error: WASAPI can't find requested audio endpoint: Element not found.

Thanks. Let's move this back on topic lol. Lemme know if I should make any further changes

Yeah, the schema update is very much needed, I will do it for 2.4 really soon. The commit hook is designed to catch such violations, but once I make the change, you can do it. You can trick it with COMMIT_UNCHECKED=1 environment, but that's not the way it's supposed to be of course.
Thanks for your patience.

I'll definitely wait until after the schema update is made to push the extra data files. If I skip the pre-commit hook, then it'll still be blocked by the schema stuff unless you want every pull request between this one and the schema to have to skip the hook too

As far as I know, this should be good to go. Need a maintainer to run the CI though to see if I borked anything

I verified locally that it worked with my reproducer

This is going to be in 2.4.8, I made the invasive hotfix 2.4.6 to make that easy to do, now that's of course bound to happen, but I need to accumulate more important changes to actually push that out, but usually that happens in a week max.


Define Cameo integration scope

Define the cameo integration scope.
Note:

The goal is to define the extent of what a cameo integration will look like

What is the value prop of the proposed integration? How does it help the user?
What features are feasible/complimentary to AaC?

AC:

[ ] Define the scope and boundaries of a potential cameo integration

[ ] Provide an example integration scenario

In the spirit of Lean Startup, recommend we simply establish a means of referencing a Cameo model as a file location.  If necessary, supporting information may be provided for more precise model content reference.  A more robust technical integration with Cameo needs to be evaluated from a value proposition perspective prior to expending any energy.

I think this might have gotten addressed with this task


add script to dump candidates information from items.json into db

add script to dump candidates information from items.json into our sqlite3 database.

@MeowMeowMeowMeowMeowMeowMeowMeow and @tsai0606 can you please review and if good, go ahead and merge. Thanks


Ignore .python-version

This PR adds .python-version to our template .gitignore.

@Scotchester It probably should, but so should a lot of other things in our .gitignore files.


[tinygltf] update to v2.4.0

Library name: tinygltf
New version number: v2.4.0
Other information that may be useful (release notes, etc...)
The current viersion is v.2.2.0 so some critical codes are missing.
v2.4.0 Experimental RapidJSON support. Experimental C++14 support(C++14 may give better performance)
v2.3.0 Modified Material representation according to glTF 2.0 schema(and introduced TextureInfo class)
You can check more detail information at https://github.com/syoyo/tinygltf

@daemyung Version 2.4.0 has not been released yet, we will update it as soon as possible after release

@JonLiu1993 Then could you update to v2.3.0?

@daemyung Sorry, version 2.3.0 has not been released, the latest version is 2.2.0

@JonLiu1993 It's strange... because v2.4.2 is released and I can see on the code. I think vcpkg tracks the version from git tags. I have raised up the issue on tinygltf(https://github.com/syoyo/tinygltf/issues/282). Please update after this issue is done. Thanks for your help.

@JonLiu1993 I have talked with syoyo who is maintainer of tinygltf. tinygltf doesn't support vcpkg officially so you can update tinygltf in vcpkg by yourself. Please update it to v2.4.2.


Update terraform-render-bootkube version

Currently, we have only updated the commit hash of terraform-render-bootkube on packet alone in bootkube module. We should do same for other providers.

@surajssd why is this closed? Versions of bootkube are not aligned across providers.
~/repos/kinvolk/lokomotive-kubernetes (master)$ git grep render-bootkube *.tf | grep source | awk '{print $4}' | sort | uniq -c
      1 "git::https://github.com/kinvolk/terraform-render-bootkube.git?ref=d07243a9e7f6084cfe08b708731a79c26146badb"
      1 "git::https://github.com/kinvolk/terraform-render-bootkube//?ref=0d3951ff08eeaa617e5aa83da5db19ef2e12324f"
      2 "git::https://github.com/poseidon/terraform-render-bootkube.git?ref=b96d641f6d42cf5d9bf3ac36f557aa21cc157680"
      1 "github.com/kinvolk/terraform-render-bootkube?ref=0913331140747fd464946981c1d43a895d7a630b"
      2 "github.com/kinvolk/terraform-render-bootkube?ref=d07243a9e7f6084cfe08b708731a79c26146badb"

I think I closed it because Kosy's PR for the pod checkpointer PSP was merged!

The follow up issue for updating other providers, is still pending, hence the reason for opening this :)

Closing it as currently all providers are aligned to https://github.com/kinvolk/terraform-render-bootkube?ref=30b2abcb1266ce9cedc92ddb3c6d8a4bbcbe7da0.


Service postgres user

not a Bug but a question:
The  "ddev pgsql_import" is not working because the user postgres dosnt exists and the command psql ist unavailable  (the user and the commant exists inside the postgres container only)
if i fire up the command:
No passwd entry for user 'postgres'
Failed to run pgsql_import : exit status 1
did I miss something here?
importing the database inside the postgres container now :)

I think you probably put the pgsql_import command inside .ddev/commands/host instead of in .ddev/commands/postgres, because the user and pgsql certainly exist in the postgres container, as you confirm.  The ddev pgsql_import command runs inside the postgres container.

Yes your right, i copied the command files not the hole directory, my fault
thanks for fast response and clarify


please provide proper changelog for 3.0.1

the release 3.0.1 does not come with a proper changelog. Could you please add the change to the changelog (even it is a oneliner on master)

Which change are you referring to

Hi @julesjulian. I cannot find a changelog item found for version 3.0.1.
You marked the tag v3.0.1 with the text Go go gadget bugfix.. However in the commit I see more changes.
We are now unsure of what has been changed. We are using jsonschema in production and rely on jsonschema's for security and data persistence, so changelog is important to us.

Ah I see -- yeah obviously a reasonable request!
To be honest, I don't like manually maintaining CHANGELOG.rst (or any changelog). If you look in it you'll see in the past there also really wasn't entries for bugfix releases, I really only use it to call out major new features or changes in backwards compatibility.
If you really want to know what's in a release, that'd be what the git commit history would be for. In this case, it's #529 and #532. But yeah I don't want to be manually maintaining a changelog :/. If it's really necessary, we can potentially think about towncrier but it only supports py3, and still comes with some overhead...
So not sure if that answers the spirit of your question but maybe tells you what's actually in 3.0.1.

Closing, but obviously let me know if you have more thoughts!


chore(main): release 1.239.0

When this PR is merged, a new tagged release will be created with the notes below.
1.239.0 (2023-09-01)
Features

Add best effort to detect Ajax events before instantiation (#648) (2d3c9d3)
Add mode to enable agent to not harvest until user consent (#656) (9141a45)
Remove allow_bfcache flag (#652) (ec113af)
Removing old supportability metrics (#669) (c17d344)

Bug Fixes

Fixing issue with leaking event listeners (#668) (6cb8238)
Issues with types resolution (#670) (85336a4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/newrelic/newrelic-browser-agent/releases/tag/v1.239.0 :sunflower:


Function calls, arguments, async flag, etc

I didn't really finish up user-defined functions. We need to carry through the async flag, for one, and the BIR needs to have local variables for each declared argument, and this code needs to provide those arguments to execute_bir (which needs to accept them):
https://github.com/dada-lang/dada/blob/3e31f4aac2dd762e764ee8c41e84e5ae5d6aee48/components/dada-execute/src/data.rs#L171-L174
We also need to validate the labels, similar to what we do for intrinsics:
https://github.com/dada-lang/dada/blob/3e31f4aac2dd762e764ee8c41e84e5ae5d6aee48/components/dada-execute/src/data.rs#L178

I think I will tackle this.


[WIP] pjax form submit heroku example

I just add a form to the heroku example branch. I think it help someone want to try pjax form submit.
Current status is let it just work. To be merged there still some work needed.

[ ] Add a pjax form with a text input. Value of the input will display.
[x] Update jqeury.pjax to use $.pjax.submit.
[x] Replace old local jquery src with new CDN src. Because latest jquery.pjax require newer jquery.
[ ] Replace or remove old jquery file (if use of a CDN doesn't seem to be suited).
[ ] jquery.cookie.js contained in the branch does not work with new jquery. replace with new one, if available, or use an alternative, or .....

Thanks for the submission. What were you trying to do here?

What were you trying to do here?

Sorry, I added description to the first comment.
I just want to see $.pjax.submit works. But heroku example doesn't have form example so I added.
I want to share this with my team mates. I think creating PR is best to do. Also I think this PR will help someone.

Gotcha. Well, your pull request is based on a commit that's very old: from year 2012 it seems. Then you upgraded the library files in the commit "Update jquery and jquery.pjax.js". That wasn't the proper way to upgrade these files. A better way would be to base your changes on a newer commit, preferrably last commit on our master branch. A way to update this pull request would be:

git fetch origin master (assuming "origin" remote points to defunkt/jquery-pjax)
git checkout heroku-submit — switch to the branch where you're doing work on this PR
git rebase -i origin/master — interactive rebase
During the interactive rebase, only pick to keep the "pjax form submit" commit, and discard the "Update jquery…" commit.
git diff origin... should now contain only your form-related changes
git push -f — force-push the "heroku-submit" branch to your remote, and this pull request will automatically get updated with newest changes.

Well,
It seems that the heroku branch is an orphan like gh-pages.
See first commit of the branch https://github.com/defunkt/jquery-pjax/commit/88b7074a0735f930e5bbf725abd7c8c35ecdb625 has 0 parents.
And latest "update pjax" commit https://github.com/defunkt/jquery-pjax/commit/64ff5ceaadb7b4a03d3e8f15263f9d5b1719914e has 1 parent.  @defunkt just overwrote jquery.pjax.js from master. So I thought I do same thing...
Anyway I tried your instruction...
git fetch origin master
git checkout heroku-submit
git rebase -i origin/master
pick 88b7074 first draft
pick aa45e93 hook it up
pick 636006b home n whatnot
pick bf9c0f0 sweet
pick c2460f0 better explanation
pick 8044633 make errything work
pick 00a7f5a finishing touch
pick 914b638 favicon
pick e1e376e cleaner, better
pick 4e5454d readme
pick 1bf695f view
pick f00abaf no pjax for you :(
pick 23cb50b load
pick e2c656e update from master
pick 6ea1d31 update pjax
pick c2a7265 update pjax... again!
pick c7f80c5 iOS web apps <3 pjax
pick 64ff5ce update pjax
# pick 7b152cf Update jquery and jquery.pjax.js.
pick a0d7c26 pjax form submit.

# Rebase 56b950b..a0d7c26 onto 56b950b
#
...

error: could not apply 88b7074... first draft

When you have resolved this problem, run "git rebase --continue".
If you prefer to skip this patch, run "git rebase --skip" instead.
To check out the original branch and stop rebasing, run "git rebase --abort".
Could not apply 88b7074a0735f930e5bbf725abd7c8c35ecdb625... first draft

Hmm,
I think I cannot rebase branches without same origin. or I did something wrong way?

Oh I'm sorry, I provided you with wrong instructions. I thought that you opened a PR for our "master" branch, like most of our PRs. But you made a perfectly valid pull request for our "heroku" branch which was indeed slightly out of date because it only serves as a demo app. Ignore my above instructions.

@mislav Thanks for your review.
Yeah, currently my change are kind of quick hacks. I'll fix those where you point to be capable to be merged.

@mislav I updated most of codes where you pointed. Also updated https://pjax-form.herokuapp.com/.


Can we have a table with the Backup Retention rules for all environment types on this page?

Would be supper helpful to have a 'simple' grid view of instance type to backup retention policy on this page.
Currently one has to dig though several paragraphs of information regarding backup policies and there are notices of conditions scattered throughout the document.
thanks

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

ID: d9c58fb1-e6a2-5210-3774-4ea4bbff4572
Version Independent ID: 57c1780c-b097-af8c-82fa-47e38a63995c
Content: Back up and restore environments - Power Platform
Content Source: power-platform/admin/backup-restore-environments.md
Service: power-platform
Sub-service: admin
GitHub Login: @ChrisGarty
Microsoft Alias: cgarty

Thanks for your comment, @MattB-msft. This issue has been assigned to the doc author for review.

@MattB-msft
Thanks for the suggestion. Taking to internal thread for action.


Command 'elm.make' not found when pressing cmd+f5 in a non-Elm project

The command + f5 keybinding is used to run the project without debugging. vscode-elm has set the keybinding to run elm.make. The problem is that it does this even when you're not using Elm in your project.
I'm now forced to uninstall the Elm extension because VS Code doesn't have the ability to disable extensions yet.
This problem can probably be solved by using a when clause for the keybindings (https://code.visualstudio.com/docs/customization/keybindings#_when-clause-contexts).

Fixed in 0.11.1


"too many open files" when pushing a large number of modified files

TLDR; Files opened by drive for uploading because they were modified are not closed after upload.
I have just started using drive after discovering it yesterday and wasn't able to find anything about ran into an issue when pushing a large number of files.
The push scans for new/removed/modified files and prompts. After confirming that I would like the modifications pushed it begins uploading the files. After running for a while I begin receiving errors like:
/path/to/the/file/file.name: open /home/username/gdrive/path/to/the/file/file.name: too many open files
On my initial run I had 4738 files that were modified. It ran to 16.92% before complaining about open files.
Running drive again against the same path there were 3721 modified files remaining. This run went to 27.47%.
On the third run there were 2704 modified files remaining. This run went to 16.6%
I did some checking with lsof during the third run. During the scanning phase I saw at most 9 files open by the drive process (only counting the files that were being sync'd). When drive was waiting at the prompt to confirm the push changes no files to sync were open, only dependencies of drive. As soon as the sync began files being sync'd started being listed but were never closed. lsof -p [drive's PID] | wc -l grew quickly to 80 then 200 then 950 open files as files.
I tested doing a push with a large number of new files (15000) and did not run into this issue. It seems to be isolated to pushing modified files.

Additional info...
result of cat /proc/version: Linux version 3.11.0-12-generic (buildd@allspice) (gcc version 4.8.1 (Ubuntu/Linaro 4.8.1-10ubuntu7) ) #19-Ubuntu SMP Wed Oct 9 16:20:46 UTC 2013
Installed with go get -u github.com/odeke-em/drive/cmd/drive
result of go version: go version go1.1.2 linux/amd64
go was installed from a default repo. To install newer go I did:
go get gopkg.in/niemeyer/godeb.v1/cmd/godeb
sudo apt-get remove golang
sudo apt-get remove golang-go
godeb install 1.3.3

new result of go version: go version go1.3.3 linux/amd64
Re-ran my tests from above and still have the issue with drive push a/path/to/stuff not closing files when the files being uploaded were selected because they were modified.

After retrying a push with a large set of files (~15000) that did not exist remotely I received a fatal error from go. This lead me to believe there was an issue with my go environment after installing the newer version above. I did the following to cleanup:
godeb remove
sudo apt-get purge golang*
rm -rf  ~/.go
godeb install 1.2.2
go get github.com/rakyll/drive/cmd/drive
mv ~/gdrive ~/gdrive_old
drive init ~/gdrive
ln -s /actual/path/to/data/ ~/gdrive/path

After this it seems my issue is gone... Running a push with a large number of modified files works. Running a push with a large number of new files works.
Lesson learned... don't assume the version in the default repositories is remotely current.

Re-opened issue for a follow up that may be related to some of the install/re-install things done above. Now that drive push seems to be working, I no longer receive a summary of files to be updated at the confirmation prompt. Related to the above?
If not, feel free to re-close this.

Hello @pez252 sorry for the late reply, been swamped the whole morning and afternoon. Welcome to drive!
As per your issue disappearing:
go get github.com/rakyll/drive/cmd/drive

wrong installation please make sure you are doing installs from this repo
$ go get github.com/odeke-em/drive/cmd/drive

Also the last time I checked the number of open files is set by kernel and if you have way too many applications open e.g web browser tabs, terminals, pushes etc in this case you you might have tens of thousands of files open but your system has only few resources this trips things out. Please Google for ulimit and also how to find the max number of open files.
I can't reply much right now but when I get home I'll try to.
Thank you for reporting this.

I did indeed put the wrong github repo when re-installing. I ran some additional tests worried that the issue with files not being closed would come back after re-installing the correct version of drive. The issue has not recurred, and the change summary is now displayed.  It seems that the problem was with drive and 1.1.2 version of go. Since 1.2+ is listed as a dependency I am closing this issue. Thanks for the assistance, and thanks for the work on this great tool.

Thank you too for the analysis, using drive and for raising this issue. It is indeed something to watch out for when doing file uploads. Please don't hesitate to reach out in case of any issues or ideas.


Fixing some of the problems

Dockerfile improvement
Update ReadMe file
delete unnecessary files

Your pull request is greatly appreciated and I'm looking forward to seeing the results of your work.  Thank you!


"Călmățuiu", "Bujoru", "Caravaneți" and "Nicolae Bălcescu" are listed in the drop-down menu

Description:
"Călmățuiu", "Bujoru", "Caravaneți" and "Nicolae Bălcescu" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Călmățuiu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Călmățuiu, TELEORMAN (Călmățuiu)" and "Comuna Călmățuiu, TELEORMAN".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Bujoru" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Bujoru, TELEORMAN (Călmățuiu)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Caravaneți" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Caravaneți, TELEORMAN (Călmățuiu)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Nicolae Bălcescu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Nicolae Bălcescu, TELEORMAN (Călmățuiu)".


Change name and get different behaviour.

Hi,
I was wondering whether somebody might be able to explain what is going on in the code below:
grammar = """line: base
base: NAME date? STRIKER
NAME: /john|bob/
date: DAY_NUM MONTH_STR YEAR_NUM?
MONTH_STR: /jan|feb|mar|apr|may|jun|jul|aug|sep|oct|nov|dec/
DAY_NUM: /\d{1,2}/
YEAR_NUM:/\d{4}|\d{2}/
STRIKER: /\d{1,5}/
%import common.WS
%ignore WS
"""
parser = lark.Lark(grammar, start='line')
s = "john 4 may 10750"
x = parser.parse(s)
I get for x:
Tree(line, [Tree(base, [Token(NAME, 'john'), Tree(date, [Token(DAY_NUM, '4'), Token(MONTH_STR, 'may'), Token(YEAR_NUM, '1075')]), Token(STRIKER, '0')])])
However, if I change 'STRIKER' to 'S', I get this:
Tree(line, [Tree(base, [Token(NAME, 'john'), Tree(date, [Token(DAY_NUM, '4'), Token(MONTH_STR, 'may')]), Token(S, '10750')])])
You can see that the final token is different.  I am using the latest version of lark.

So first of, your grammar ambiguous. Both parses are valid. This means that which one is used is not well defined, and does probably in this case depend on the hash of the name of the string. This means that changing the name might change everything.
There are multiple ways to fix this:

rewrite your grammar to not be ambigous
uh se token priorities.
use ambiguity='explicit' and manually resolve it.

Hi MegaIng, Thanks for the advice!


fix: goroutine leak

#322
Concurrency exists between user shutdown and automatic reconnection.

Maybe there's a better modification.

Thanks for pointing it out, this change still has concurrency issue, I'll create another commit

Thanks for pointing it out! I think this change still has a timing issue, I'll create another commit

I have checked the code again and it doesn't have the timing issue, sorry for the error message. I think it would be better to put all the close code to the Close method, so create #332 to fix this.


feat(two-sum): add twoSum function to find an index that sum to the t…

My first time #hacktoberfest 😅
hope this is valid to be merged

Keep up your hard work


Transform exercises in Stack projects - Phase 6 - Cleaning up

Transform exercises in Stack projects #185.
Phase 6 - Cleaning up

Rewrite .travis.yml.
Remove _test and update config.json.
Remove Vagrantfile.

For this commit I think I only look at https://github.com/exercism/xhaskell/pull/203/commits/72e3af24b3ee727ce9385f1abfa6ebfad14044a4 and that looks good. Feels good to delete old code! :+1:


fix: update to use pyhive function to get compatible sasl

resolves #1094
docs dbt-labs/docs.getdbt.com/#

Problem

Current SASL factory for TSaslClientTransport using SASLClient from pure-sasl which is incompatible with pyhive
Solution

Pyhive provides a wrapper PureSASLClient which implement some missing methods to interact with thrift-sasl.
This MR update to use get_installed_sasl of pyhive to decide which sasl client should be use in current environment.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

cc @McKnight-42


Fix #678: removed hashcode implementation details from javadoc

This PR addresses GitHub issue: #678 .
Briefly describe the changes proposed in this PR:

javadoc for model objects (and BindingSet) cleaned up: implementation details of hashcode algorithms removed, added override annotations, reformatted where necessary.

I disagree with the removal on the principle that we don't track or convert the types of objects in most of our APIs (MemValueFactory and similar excepted), so we need hashcodes to be compatible.

@ansell I'm not sure I follow. The rationale behind removing the implementation detail was that it falsely leads users to believe that the hashcode will never change between releases.  Are you suggesting that we should in fact make that commitment? Can you elaborate on why you think this is necessary?

Users will virtually always be using Value objects from different ValueFactory's, both internally and from third-parties. If they add these Value objects to a HashSet or a LinkedHashModel, they want to be sure that they will obtain consistent results. Having the hashCode contract stated explicitly (whatever its contract turns out to be) is the only way to ensure that core promise of the Java Collections API (and out Model API) in the long term across different implementations. This partly relates to the removal of ValueFactory when we changed from Graph to Model, but users will expect to be able to use the hash-based Java Collections API in the same way.

It is possible that they may change, but the only possible stable point at which to change the hashcode contract would be during a major release IMO.


Unable to execute the aspect

Looks like fetch of one of pip packages fails, even though I am able to locate it in /var/cache/pip folder.
ERROR: An error occurred during the fetch of repository 'dwyu_py_deps_pcpp':
   Traceback (most recent call last):
        File "/bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/pip_repository.bzl", line 438, column 13, in _whl_library_impl
                fail("whl_library %s failed: %s (%s)" % (rctx.attr.name, result.stdout, result.stderr))
Error in fail: whl_library dwyu_py_deps_pcpp failed: Looking in links: /var/cache/pip
Collecting pcpp==1.30
  File was already downloaded /var/cache/pip/pcpp-1.30-py2.py3-none-any.whl
 (Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/parse_requirements_to_bzl/extract_single_wheel/__main__.py", line 4, in <module>
    main()
  File "/bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/parse_requirements_to_bzl/extract_single_wheel/__init__.py", line 68, in main
    whl = next(iter(glob.glob("*.whl")))
StopIteration
)
ERROR:<STRIPPED>/WORKSPACE:1474:18: fetching whl_library rule //external:dwyu_py_deps_pcpp: Traceback (most recent call last):
        File "/bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/pip_repository.bzl", line 438, column 13, in _whl_library_impl
                fail("whl_library %s failed: %s (%s)" % (rctx.attr.name, result.stdout, result.stderr))
Error in fail: whl_library dwyu_py_deps_pcpp failed: Looking in links: /var/cache/pip
Collecting pcpp==1.30
  File was already downloaded /var/cache/pip/pcpp-1.30-py2.py3-none-any.whl
 (Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/parse_requirements_to_bzl/extract_single_wheel/__main__.py", line 4, in <module>
    main()
  File "/bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/parse_requirements_to_bzl/extract_single_wheel/__init__.py", line 68, in main
    whl = next(iter(glob.glob("*.whl")))
StopIteration
)
ERROR: /bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/depend_on_what_you_use/src/analyze_includes/BUILD:4:11: @depend_on_what_you_use//src/analyze_includes:lib depends on @dwyu_py_deps_pcpp//:pkg in repository @dwyu_py_deps_pcpp which failed to fetch. no such package '@dwyu_py_deps_pcpp//': whl_library dwyu_py_deps_pcpp failed: Looking in links: /var/cache/pip
Collecting pcpp==1.30
  File was already downloaded /var/cache/pip/pcpp-1.30-py2.py3-none-any.whl
 (Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/parse_requirements_to_bzl/extract_single_wheel/__main__.py", line 4, in <module>
    main()
  File "/bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/parse_requirements_to_bzl/extract_single_wheel/__init__.py", line 68, in main
    whl = next(iter(glob.glob("*.whl")))
StopIteration
)
ERROR: Analysis of aspects '[//:aspect.bzl%your_dwyu_aspect] with parameters {} on <TARGET>' failed; build aborted:

bazel version: 6.3.2
dowyu version: 0.13

Okay, if I hack this script bazel_cache/output_user_root/61da6a75bcd751b8ee01d9c498363d41/external/rules_python/python/pip_install/parse_requirements_to_bzl/extract_single_wheel/init.py and just use direct path /var/cache/pip/pcpp-1.30-py2.py3-none-any.whl
instead of iterator it works

I am not sure if I will be able to help you. The error appears inside rules_python which is used to fetch the Python dependencies for DWYU. I did not implement this myself. Also I did not encounter this error yet.
Could you tell me the following, which maybe gives me more hints:

which OS are you using
which Python version are you using.
Are you using the host Python interpreter or a hermetic Python toolchain offered by rules_python?
Is your project settings its own rules_python version, or are you using the default version fetched by DWYU (which would be rules_python version 0.18.1 for DWYU 0.0.13).

rules_python 0.10.0 is indeed quite old.
Please try a recent version like 0.18.1.

closing due to no more feedback or help requests


custom invalid function not getting parameters

_.extend(Backbone.Model.prototype, Backbone.Validation.mixin);

var test = Backbone.Model.extend({
    validation: {
        value: {
            pattern: 'url'
         },
     },
     initialize: function(){
         this.bind('validated: invalid', this.invalid);
     },
     invalid: function(view, attr, error, selector){
         console.log(view, attr, error, selector);
    }
});

var t = new Test({value: 'hotdogs'});

When I run this, this is the result from the console:
>child [] undefined undefined
Where child is the model that was being validated, but all the other attributes are empty so I'm unaware as to what failed validation.
I'm trying to make it so if there's an invalid setting and there's a default set for that setting, it'll use the default to override the invalid value trying to be set.
Additionally, I must bind to validate:invalid in the model's initialization function, if I don't it will (nor the method I'm overwriting) get called at all.
Non-view model level validation is super important to us.

If I set a breakpoint at line 149 I get the following:
> model._isValid
false
> result.invalidAttrs
[]

I'm working my way back into validateObject now to see why invalidAttrs isn't receiving a value there.

Nothing I seem to able to do is allowing the invalidAttrs array to return with a list of the invalidAttrs.

OK, Seem to have narrowed it down a little.  It looks like validate:invalid is being called every time I set a value, even if what I set was valid.
It looks like validateObject() (line 147) never returns result.isValid as true.  It always returns as false.

Hi,
First, the signature of your invalid method is wrong. It should be:
invalid: function(model, invalidAttrs) {
}

Second, at the moment the invalidAttrs will only contain the name(s) of attributes that was set, causing the validation to trigger. However, I am thinking that this is probably not an ideal solution. All attributes should be validated at all times.
validated:invalid will trigger as long as there is something on the model that is not valid.
Will look more into this soon.

Yeah, I realized there were some errors in my code later, but didn't have time to update it.
Actually, after playing around for a few hours, I realized your approach and what we're looking for were incompatible (raw model-level validation regardless of views), so I made my own validation framework! (And borrowed your format() function with attribution in the documentation and the code).
http://github.com/toddself/Backbone.Validator
Thanks for the inspriration!


Enable Google Search Console and/or Google Analytics

Problem: We know little about how Google is indexing our pages, and how users are using our sites. “Search Console” (formerly “Webmaster Tools”) and “Google Analytics” are complementary. They pull information from different sources: for Search Console, we just need to demonstrate our ownership of a site; For analytics, we add JS to our pages, and so the browser is reporting back to Google directly.
For now, I think Search Console will be more than enough, and it is less invasive than Analytics.
From https://search.google.com/search-console/welcome:
Possible fix 1 (preferred):

Someone with access to DNS for opendp.org can demonstrate that we own the domain, and all subdomains (ie, “docs”)

Possible fix 2:

Demonstrate that we own a domain or subdomain by adding a file at a particular URL. Going through a docs redeploy for this would probably be more of a nuisance than just demonstrating DNS ownership.

Also explore analytics other than google that are more privacy focused


Fixed buggy regex in URL dispatcher docs

I think we need to escape - in [\w-_], otherwise it is interpreted as a range:
django.core.exceptions.ImproperlyConfigured: "^articles/(?P<year>[0-9]{4})/(?P<month>[0-9]{2})/(?P<slug>[\w-_]+)/$" is not a valid regular expression: bad character range \w-_ at position 59

Furthermore, \w already matches _, so we can just remove that.

Agreed about removal of _ due to it being handled by \w.
For a literal - it can be escaped or placed at the start or end of the character set to avoid accidental ranges. I generally prefer the start, i.e. [-\w], as new values tend to be tacked on the end.

(Oops, I didn't see Nick's comment before I merged this. Well, I don't think that regex is likely to change again and it's used in a couple tests.)


chore(package): update kronos-flow to version 2.5.36

Coverage remained the same at 14.035% when pulling 7ccc3653831940b7bc2f68fc6e58e7558bb2ef36 on greenkeeper/kronos-flow-2.5.36 into 875766bbabaa1731784c67a0fc765eff0606243c on master.


Investigate time out issues in PROD

Lizzy reported a time out in the Deposition App when creating submission (5f7dbe682b987d0001beccea).
It seems that the issue appeared when updating the publication status in SOLR.
Resolution:

restart SOLR in PROD
re-index all publications in PROD


[issue - 28] Security: Regular User is allowed to view and manage list of subscriptions belonging to a different user

fixed the logic normal user can not access the subscription of other users
ticket here
Fixes #28

Hello @sibasankarnayak,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Codecov Report

Merging #76 (d06b6fc) into master (806f094) will not change coverage.
The diff coverage is 16.66%.

@@           Coverage Diff           @@
##           master      #76   +/-   ##
=======================================
  Coverage   16.18%   16.18%
=======================================
  Files          13       13
  Lines        1853     1853
=======================================
  Hits          300      300
  Misses       1534     1534
  Partials       19       19

Impacted Files
Coverage Δ

server/command.go
6.79% <0.00%> (ø)

server/subscriptions.go
14.51% <28.57%> (ø)

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 806f094...d06b6fc. Read the comment docs.

According to https://github.com/mattermost/mattermost-plugin-bitbucket/issues/28#issuecomment-1089402081, this should not be implemented until the design is revisited

Closing this for now until the feature has been designed


Aether Farmland Issue

Was comparing farmland speeds of different mods compared to vanilla using small size auto-villager farms of 9x9 blocks and noticed that villagers wont plant on modded farmland like the aether farmland unless there's regular farmland in range

Requires a mixin/accessor to Rabbit#isValidTarget and VillagerProfession to fix this from looking into it


ERROR "The Input name is empty" is coming

Describe the bug
We created an m2kconfig.yaml and added private registry configuration and other configurations but when we run transform using -f option and give config yaml file, it gives  "The Input name is empty" error.Also it creates file with name -secret.yaml in deploy/yamls file which doesn't run using kubectl command as file name starts with hyphen.
To Reproduce
Steps to reproduce the behavior:

Create your own m2kconfig_test.yaml file and add private registry information
run transform with the above file

Expected behavior
Error should not come and secret file name should not start with hyphen
Screenshots

Desktop (please complete the following information):

OS: Ubuntu
Version v0.3.0+unreleased. move2kube version should return it.

Additional context
NA

With v0.3.0 rc.2, this issue is fixed.


[BUG] test_arrays_zip failures on nightly

Describe the bug
[2023-01-06T11:40:23.945Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Byte)] - py4j.protocol.Py4JJavaError: An error occurred while calling o304419.collec...
[2023-01-06T11:40:23.945Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Short)] - py4j.protocol.Py4JJavaError: An error occurred while calling o304598.collec...
[2023-01-06T11:40:23.945Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Integer)] - py4j.protocol.Py4JJavaError: An error occurred while calling o304777.collec...
[2023-01-06T11:40:23.945Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Long)] - py4j.protocol.Py4JJavaError: An error occurred while calling o304956.collec...
[2023-01-06T11:40:23.945Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Float)] - py4j.protocol.Py4JJavaError: An error occurred while calling o305135.collec...
[2023-01-06T11:40:23.945Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Double)] - py4j.protocol.Py4JJavaError: An error occurred while calling o305314.collec...
[2023-01-06T11:40:23.945Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(String)] - py4j.protocol.Py4JJavaError: An error occurred while calling o305493.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Boolean)] - py4j.protocol.Py4JJavaError: An error occurred while calling o305672.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Date)] - py4j.protocol.Py4JJavaError: An error occurred while calling o305851.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Timestamp)] - py4j.protocol.Py4JJavaError: An error occurred while calling o306038.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Null)] - py4j.protocol.Py4JJavaError: An error occurred while calling o306217.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Decimal(7,3))] - py4j.protocol.Py4JJavaError: An error occurred while calling o306396.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Decimal(12,2))] - py4j.protocol.Py4JJavaError: An error occurred while calling o306575.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Decimal(20,2))] - py4j.protocol.Py4JJavaError: An error occurred while calling o306754.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Array(Short))] - py4j.protocol.Py4JJavaError: An error occurred while calling o306933.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Array(String))] - py4j.protocol.Py4JJavaError: An error occurred while calling o307112.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Struct(['child0', Byte],['child1', String],['child2', Float]))] - py4j.protocol.Py4JJavaError: An error occurred while calling o307291.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Map(String(not_null),String))] - py4j.protocol.Py4JJavaError: An error occurred while calling o307470.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip[Array(Binary)] - py4j.protocol.Py4JJavaError: An error occurred while calling o307649.collec...
[2023-01-06T11:40:23.946Z] FAILED ../../src/main/python/array_test.py::test_arrays_zip_corner_cases - py4j.protocol.Py4JJavaError: An error occurred while calling o307828.collec...
[2023-01-06T11:40:23.946Z] = 20 failed, 15074 passed, 830 skipped, 572 xfailed, 235 xpassed, 911 warnings in 6812.06s (1:53:32) =

Errors:
[2023-01-06T11:40:23.930Z] E                   	at java.lang.Thread.run(Thread.java:750)

[2023-01-06T11:40:23.930Z] E                   Caused by: java.lang.IllegalArgumentException: All columns must have the same number of rows

[2023-01-06T11:40:23.930Z] E                   	at ai.rapids.cudf.ColumnView.makeStructView(ColumnView.java:3440)

[2023-01-06T11:40:23.930Z] E                   	at ai.rapids.cudf.ColumnView.makeStructView(ColumnView.java:3458)

[2023-01-06T11:40:23.930Z] E                   	at ai.rapids.cudf.ColumnVector.makeStruct(ColumnVector.java:395)

[2023-01-06T11:40:23.930Z] E                   	at org.apache.spark.sql.rapids.GpuArraysZip.$anonfun$zipArrays$2(collectionOperations.scala:825)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.Arm.withResource(Arm.scala:46)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.Arm.withResource$(Arm.scala:44)

[2023-01-06T11:40:23.931Z] E                   	at org.apache.spark.sql.rapids.GpuArraysZip.withResource(collectionOperations.scala:701)

[2023-01-06T11:40:23.931Z] E                   	at org.apache.spark.sql.rapids.GpuArraysZip.$anonfun$zipArrays$1(collectionOperations.scala:824)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.Arm.withResource(Arm.scala:28)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.Arm.withResource$(Arm.scala:26)

[2023-01-06T11:40:23.931Z] E                   	at org.apache.spark.sql.rapids.GpuArraysZip.withResource(collectionOperations.scala:701)

[2023-01-06T11:40:23.931Z] E                   	at org.apache.spark.sql.rapids.GpuArraysZip.zipArrays(collectionOperations.scala:823)

[2023-01-06T11:40:23.931Z] E                   	at org.apache.spark.sql.rapids.GpuArraysZip.$anonfun$columnarEval$5(collectionOperations.scala:782)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.Arm.withResource(Arm.scala:46)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.Arm.withResource$(Arm.scala:44)

[2023-01-06T11:40:23.931Z] E                   	at org.apache.spark.sql.rapids.GpuArraysZip.withResource(collectionOperations.scala:701)

[2023-01-06T11:40:23.931Z] E                   	at org.apache.spark.sql.rapids.GpuArraysZip.columnarEval(collectionOperations.scala:732)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.RapidsPluginImplicits$ReallyAGpuExpression.columnarEval(implicits.scala:34)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.GpuAlias.columnarEval(namedExpressions.scala:109)

[2023-01-06T11:40:23.931Z] E                   	at com.nvidia.spark.rapids.RapidsPluginImplicits$ReallyAGpuExpression.columnarEval(implicits.scala:34)

Actually this is not a bug in CUDF. It looks like we are relying on broken behavior from CUDF in these cases and https://github.com/rapidsai/cudf/pull/12370 fixed it.  Prior to the change if we did a segmented gather with a NULL input LIST, and a gather map of {0} the output would be a single NULL LIST row, but the child column would have one entry in it that was also marked as NULL. That is no longer true after this change.
I think the only way to fix this properly is to replace the null values with an empty list on input.  I could do it the cheap way and drop the validity and null count when making my own fake column view, but I think I want to do it the correct way for now and we can look into speeding that up if it shows up as a performance problem.


Cleanup

how about these small housekeeping commits?
@SebastianLuebke, hope this goes into the right direction for you. feedback & change requests are welcome.

Looks good, thank you  :)