python interpreter not recognizing in VS Code Jupyter notebook via conda environment

Version: 1.55.0 (user setup)
Commit: c185983a683d14c396952dd432459097bc7f757f
Date: 2021-03-30T16:01:55.261Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.21292

It appears VS Code does not add one of the paths for Python interpreters in the Conda environment for some strange reason.
Running python --version from CMD prompt (with Conda env activated) shows Python 3.6.12 :: Anaconda, Inc.
But running python --version from within a Jupyter notebook in VS Code shows Python 3.8.3
So I suspect there's an issue with the VS Code not correctly mapping the Python path from the conda environment to the Jupyter notebook?
Here's the output of where python in my CMD prompt (within the Conda virtual env of course):
(my_conda_env) D:\projects>where python
C:\ProgramData\Anaconda3\envs\my_env\python.exe
C:\Python38\python.exe
C:\Users\username\AppData\Local\Microsoft\WindowsApps\python.exe

But here's the output of where python when called within VS Code Jupyter notebook (also with the Conda environment activated):
C:\Python38\python.exe
C:\Users\username\AppData\Local\Microsoft\WindowsApps\python.exe

Why does Visual Studio Code not correctly locate the 1st python path?

Certainly @rchiodo:
Screenshot of the Python interpreters (I have many versions of Python installed, this):

And the list of notebook kernels (again, I have multiple environments installed so need to scroll down to see all of them; these are just the first being displayed for you).

Which kernel did you pick for the notebook?

This was on VS Code Insiders so that may have affected this issue.
But a simple restart of VS Code, selecting the desired Conda environment (lower left corner of the IDE), and selecting the correspondingly named Jupyter kernel (upper right corner of the IDE) resolved this.
Closing as I believe this is resolved.  Much appreciated for the help.


add core API style guide

@Mzack9999 and I had a really great conversation about creating a "style guide" for the core API.

Note: Nothing is finalized yet, but I think it would be really great to help new developers join working on the code base when they know what they are expected to do. Creating documentation can help ease in new developers by having resources they can rely on.

Example
If I am making a new source, should I create a newSource() function that returns a *Source or do you just call &Source{}? Currently, I've been enforcing the more direct call to &Source{} which reduces the needed boilerplate to write a new source and directly uses Go's own initialization patterns instead of having a more traditional object orientated pattern with an explicit initializer function.
Examples and explanations can/should be curated to cultivate a unified style other developers can use.

Some work has been done on this in the README for sources: https://github.com/subfinder/research/blob/master/core/sources/README.md

This docs will need to be updated once https://github.com/subfinder/research/pull/59 is merged into master.


NSH ifconfig and Multiple Network Interfaces

The ifconfig command will not behave correctly if an interface is provided and there are multiple interfaces.  It should only show status for the single interface on the command line; it will still show status for all interfaces.

Fixed by https://github.com/apache/nuttx-apps/pull/1519.


Delay is not long enough

I tried out the script yesterday, it worked like a breeze apart from the fact it violates the fair use policy of ticketswap. I think it's due to the delay not being long enough.

I suspect Ticketswap is actively detecting the script by now. I have heard from users that were banned after using the script.


Reporting Infringement link broken

[Enter feedback here]
Reporting Infringement link broken

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

ID: c6bdbaf0-1a39-c369-deca-d22e0d4e6ec0
Version Independent ID: 7ec272a7-8291-5d33-f320-ef7f1bcd9648
Content: Contact Microsoft Edge extensions support - Microsoft Edge Development
Content Source: microsoft-edge/extensions-chromium/publish/contact-extensions-team.md
Product: microsoft-edge
Technology: extensions
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

I just tested the link now and it worked for me.
On the page, the link is https://www.microsoft.com/concern/dmca and I then get redirected to https://www.microsoft.com/en-us/concern/dmca (the locale part of the path gets added).
Can you tell me which URL you end up on exactly?


AttributeError: 'TracebackException' object has no attribute 'exc_traceback'

WARNING - root - Changed type of config entry "max_steps" from int to NoneType
WARNING - ViLT - No observers have been added to this run
INFO - ViLT - Running command 'main'
INFO - ViLT - Started
Global seed set to 0
INFO - lightning - Global seed set to 0
ERROR - ViLT - Failed after 0:00:03!
Traceback (most recent call last):
File "D:\Anaconda3\lib\site-packages\sacred\experiment.py", line 312, in run_commandline
return self.run(
File "D:\Anaconda3\lib\site-packages\sacred\experiment.py", line 276, in run
run()
File "D:\Anaconda3\lib\site-packages\sacred\run.py", line 238, in call
self.result = self.main_function(*args)
File "D:\Anaconda3\lib\site-packages\sacred\config\captured_function.py", line 42, in captured_function
result = wrapped(*args, **kwargs)
File "run.py", line 42, in main
grad_steps = _config["batch_size"] // (
ZeroDivisionError: integer division or modulo by zero
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "run.py", line 11, in
def main(_config):
File "D:\Anaconda3\lib\site-packages\sacred\experiment.py", line 190, in automain
self.run_commandline()
print_filtered_stacktrace()
File "D:\Anaconda3\lib\site-packages\sacred\utils.py", line 493, in print_filtered_stacktrace
print(format_filtered_stacktrace(filter_traceback), file=sys.stderr)
File "D:\Anaconda3\lib\site-packages\sacred\utils.py", line 528, in format_filtered_stacktrace
return "".join(filtered_traceback_format(tb_exception))
File "D:\Anaconda3\lib\site-packages\sacred\utils.py", line 568, in filtered_traceback_format
current_tb = tb_exception.exc_traceback
AttributeError: 'TracebackException' object has no attribute 'exc_traceback'

did you solve this question??? please help!!!

I have also encountered this problem, what is the reason? How to solve it?

python>=3.9


Fix pipelines and unblock repo

Description
Changes proposed in this pull request:

Improve make targets to work with the latest cli version

this PR duplicates this one
https://github.com/kyma-project/serverless-manager/pull/54


Make email box clickable/touchable

It should be possible to edit the email in the email box by clicking or touching the email box. This should then take the email box into it's 'edit' state

fixed by @aissshah


add et-EE.csv language files to core and default theme

Related issues
none
Short description and why it's useful
Add et-EE.csv language files to core/i18n/resource/i18n/et-EE.csv and src/themes/default/resource/i18n/et-EE.csv based on en-US.csv files in that directories.
Screenshots of visual changes before/after (if there are any)
none
Which environment this relates to

[x] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)

Contribution and currently important rules acceptance

[x] I read and followed contribution rules

@alphpkeemik hey, thanks for the PR! Please change base to release/v1.11 we are going to add this in the upcoming release.

@andrzejewsky base changed to release/v1.11.

@alphpkeemik I think there are some changes that aren't yours, it's because sometimes changing the base on githib breaks the history, could you use rebase command to solve that? or you can create this PR again starting from release/v1.11 - sorry.

@andrzejewsky sry forgot to change pull request base


Qubes packages (Its alive!)

Description of changes
You can initialize qubes database files using qubes-create from qubes-core-admin, and then you need to install some virtual machines - this can be done by running qvm-backup on QubesOS system, and qvm-backup-restore in NixOS.
After that, basic functionality will work.
Be careful with netvms, as dom0 updates will not work with them. Some loosening up of qubes security needs to be implemented, at least having "qvm-dom0-network-via-netvm" script (Which was present in QubesOS before, and now needs to be written from scratch?) would be a good start.
At this point, you're welcome for testing, but be aware that it is not finished yet.

Cc: @SigmaSquadron
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Pushed other batch of qubes fixes, now you can actually somehow manage VMs using qubes-manager, given that you have installed templates somehow (E.g you can copy VMs from other qubes installation). Standard qvm-template doesn't work, because it uses RPM to install such packages.
@SigmaSquadron xen needs python module, do you think it is possible to split it from xen package?
At least separate output will do, but ideally it needs to be built fully independently, since xen is a large package.
Also, another dependency cycle has been formed, this time it can't be resolved as with qrexec.

@SigmaSquadron xen needs python module, do you think it is possible to split it from xen package?
At least separate output will do, but ideally it needs to be built fully independently, since xen is a large package. Also, another dependency cycle has been formed, this time it can't be resolved as with qrexec.

I'm all for splitting the Python stuff from the main package, but I have no idea how to do that or how circular dependencies are even supposed to work in the first place.

Forgot to cherry-pick some changes from my workstation nixpkgs. Sorry for the dirty git history, I will rebase and clean it up after more polishing.

Feel free to dump everything in the draft PR. We can always beat the git history into a shape later.

how circular dependencies are even supposed to work in the first place.

Well, they don't, derivation circular dependencies needs to be resolved somehow. Some can be merged together, some can be updated to be loosely coupled (E.g use PATH or specific runtime environment variables), but they need to be fixed, derivation can't depend on itself, because derivation hash = hash of its inputs.
Right now, both of discovered dependency cycles are resolved using absolute system installation paths, and I'm not sure of a proper solution.

Some more information for the module:
Qubes' boot parameters:
virtualisation.xen.bootParams = [ "smt=off" "gnttab_max_frames=2048" "gnttab_max_maptrack_frames=4096" ];

Use the C-based XenStore (as Qubes doesn't use oxenstored due to the lack of security support for untrusted driver domains):
virtualisation.xen.store.path = "${pkgs.qubes-vmm-xen}/bin/xenstored";

Also, any systemd services brought in by the module need to have their Install sections duplicated in Nix code. (notably, wantedBy.)

Yeah, of course, this is just a rough draft of how it should look, nothing work in this nixos module yet.
BTW, qubesd has non-critical crash on service stop when using python 3.11 due to new python features being used (https://github.com/QubesOS/qubes-core-admin/commit/23be00ccf4c8c39fdaef908170aa37f2e3c3790a), and I can't upgrade it to 3.12 because of meson regression in Xen, do you have idea on how to fix that? https://github.com/NixOS/nixpkgs/issues/253751
Should have specified that this problem only affects Xen, qubes packages are building with python 3.12 just fine by itself.

and I can't upgrade it to 3.12 because of meson regression in Xen, do you have idea on how to fix that? #253751 Should have specified that this problem only affects Xen, qubes packages are building with python 3.12 just fine by itself.

The hacky solution is to python = (python312Packages.python.withPackages (p: [ p.setuptools ])); in Xen.

Thrown even more broken code into the pile, now it is able to generate menu entries for VMs, and recognize PCI device names.
Nixos module is still broken, but now it might start most of the required things by itself.

The hacky solution is to python = (python312Packages.python.withPackages (p: [ p.setuptools ])); in Xen.

This hacky solution sounds good enough for now, I don't know how many of the bugs are caused by python version older than upstream wants.

Latest changes make it run in KDE Wayland session.

There are still many bugs, but basic functionality works perfectly fine.

This branch is a little bit outdated, I'm debugging issues occured after nixos rebase.
For some reason, installation of StandaloneVM stopped working, qemu window appears, but it is always empty.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixos-xen-project-reviving-the-xen-project-hypervisor-on-nixpkgs/52768/7

Rebased&Cleaned up the whole history.
StandaloneVM installation is still broken, I believe this is caused by some upstream (qubes or nixos) changes, as I don't see any relevant changes between rebased and not rebased version of this PR.

Figured out StandaloneVM - this is caused by changes that were not included in the previous version of this PR, but in my private nixpkgs fork
    ++ lib.lists.optional withSeaBIOS "--with-system-seabios=${seabios}/share/seabios"

This is incorrect, as --with-system-seabios accepts full path to seabios binary, after replacing path with the correct one StandaloneVM installation works again.
TODO: Full path needs to be overrideable too, as we want to use qubes-seabios here, and it has different binary names inside.
@SigmaSquadron PTAL

god bless you seriously :bow:

Another rebase, this time StandaloneVM display is broken again?
This time seabios path is passed correctly as I can see, is it broken due to different reasons, or something in seabios/xen build has been broken in the meantime?

I have made some progress on QubesOS builders and building stubdom for NixOS-Qubes, however I believe it would be much easier to handle those tasks after base QubesOS packages are already merged in NixOS, even if they have suboptimal implementation (Looking at stubdom, which is extracted from QubesOS RPMs) sometimes.
Another way to simplify management of this package set would be extracting them to separate flake... Thoughts?
For now, I have updated all of packages to latest versions, rebased to latest master, and populated meta attributes as much as possible, to mark it ready for review.
Please ignore formatting deficiencies for now (Some of patches not present here might break on reformatting, and I would like to avoid too much rebase conflicts), there is enough of crimes against humanity here without that.

StandaloneVM seems to be fixed again, but I would expect that it might break again due to unknown reasons, I believe this might be fixed after unification of package versions between dom0 and stubdom, which will only happen in the future, as building our own stubdom is too complicated for now.

this is awesome work @CertainLach!
I've been working on Qubes packaging as well, coming at it from the angle of just creating a NixOS template vm that can be used under vanilla Qubes. I love the idea of using NixOS in dom0 as well.
as I'm fairly new to Nix I packaged it as a standalone flake to get feedback on how to best upstream it:
https://github.com/evq/qubes-nixos-template
if what I've created is useful it'd be great to collaborate on merging the two.


1.3.2-beta+fabric.1.20.6 is listed as compatible with 1.20.2-1.20.6 but requires Java 21

The Modrinth listing for 1.3.2-beta+fabric.1.20.6 specifies the compatibility range as 1.20.2-1.20.6. However, because it was compiled against Java 21 it cannot be run on 1.20.2-1.20.4 instances which use Java 17.
This can be fixed on the user-side by modifying the instance to use Java 21, but in my opinion that should not be required.

Yeah that was a mistake. The 1.3.2 version is currently only available for 1.20.6. Thanks for pointing this out, it has been corrected.


NoMethodError: undefined method `external_accounts' for #Stripe::Account:0x007fc131029168

For Stripe Managed Accounts, they now have the associated object "external accounts" that contain bank routing and bank account numbers.
https://stripe.com/docs/api#account_object-external_accounts
I would love to see external accounts supported by stripe-ruby-mock!

It seems not to be only external accounts but all managed accounts. There is no verification field either.

Any thoughts on how to handle this?

If you are testing web hooks, you can create a json file containing a sample response with the name of your web hook (i.e. account.external_account.created) in spec/fixtures/stripe_web_hook. It will be automatically loaded by the gem.
I you want to test basics calls (such as Stripe::Customer.create), I do not think it is implemented yet. I will maybe contribute to the gem to fix it this week end if I can find some time. :smile:

Actually, I'm trying to create an bank account.
  1) StripeService Customer should update a managed account
     Failure/Error: account.external_accounts.create(:external_account => options)

     NoMethodError:
       undefined method `external_accounts' for #<Stripe::Account:0x0055ed8c2a17d0>
     # /usr/local/bundle/gems/stripe-1.50.0/lib/stripe/stripe_object.rb:271:in `method_missing'

I tried to redefine the external_accounts inside my test, but I was not able to.
      module StripeMock
        def external_accounts=
        end

        def external_accounts
        end
      end
      Stripe::Account.extend StripeMock

Any thoughts on how to monkey patch this until there be an official support?
Thanks!

Well, I made a PR that was just merged to add external_accounts to account's hash and external_accounts webhooks.
Adding the whole external accounts logic will take a little more time considering the fact that I need to dive deeper into the code to fully understand the behaviour of the gem. I will start working on it tomorrow and I will notify you here when I'll be done with it. :)


Review - Feature importance

Once you use all variables + derived variables you do not comment on feature importance. What did you learn from them? Were they all needed? How do you establish feature importance? How is the subsequent work affected by this?

Although the project is already succesfully finished, for completeness sake here is additional information: Due to colinearity only a part of the features was needed. E.g. large-scale precipitation and total column water vapour had a high correlation (linearly of course), which is why we only used the former.
More comprehensive comments can be seen in the notebooks inside the /notebooks/2_preprocessing/ folder, as well as for the best performing model (TDNN) in this notebook: /notebooks/3_model_tests/3.05_Time-delay_neural-net.ipynb.


Move to Github Actions?

Hello
I saw few recent OSS projects that move to Github Actions. I did a very basic try with rspec-core and only with MRI at the moment.
https://github.com/benoittgt/rspec-core/commit/70abad7b1d3fbdd21d9270e09a07a1fbc397df7c/checks?check_suite_id=392910092
For the moment I do not see a big improvement. Also I had to do ugly things to make the build green quickly.
The config for rspec-rails will be more complex.
To be honest I think it is maybe too early to move. I do no see a gain in execution time and debugging for the moment.
This issue is not a real one. I just open a discussion. I am also curious if you think we should switch to Github Actions.
Closing because it is only a discussion.

I have different behavior when I use bin/rspec instead of https://github.com/benoittgt/rspec-core/blob/c25a2df53db05a107ec7b5dd428e0455757bd4c8/script/functions.sh#L70-L77
It is failing with shell commands, but not with bin/rspec

$ ruby -e 'puts Dir["spec/**{,/*/**}/*_spec.rb"]' | wc -l
     152
$ find spec -iname '*_spec.rb' | wc -l
     76

Why we are not using bin/rspec?

I'm not sure I understand what you're asking?

Sorry Jon. After digging, I found the answer. I was worried we were running multiple time bin/rspec with the same files.  This doesn't seems to be the case.

I am still playing with Github Actions. I found that running
require "rspec"
require "rspec/core/rake_task"
RSpec::Core::RakeTask.new.run_task(false)

On Github actions produce this warning:
/opt/hostedtoolcache/Ruby/2.6.3/x64/lib/ruby/gems/2.6.0/gems/rspec-core-3.9.1/lib/rspec/core/rake_task.rb:97: warning: Insecure world writable dir /opt/hostedtoolcache/Ruby/2.6.3/x64/bin in PATH, mode 040777
No examples found.

Finished in 0.00021 seconds (files took 0.05324 seconds to load)
0 examples, 0 failures

Ok, the warning is produced by RUBY from:
require 'rake'
require 'rake/tasklib'

system("#{Rake::TaskLib::RUBY} -e 'p :foo'")

Ok we gonna to move this subject a little bit faster if TravisCI drop OSS. ATM I have issues on basic rspec-core project. Here is the config file. https://github.com/benoittgt/actions_ruby_testing/blob/master/.github/workflows/ruby.yml
And a CI result https://github.com/benoittgt/actions_ruby_testing/runs/1343435673

It seems we can migration to travis-ci.com and use the free 10000 credits, but I'm not sure how much that would give us, as travis-ci.org was taking hours and hours during the release process for 3.10 I started investigating over here:
https://github.com/rspec/rspec-core/tree/experimental-github-actions
I got rspec-core passing on 2.5 2.6 and 2.7 but none of the earlier versions, so I started looking at how to deal with older Ruby versions.

Just checked rubocop-rspec's CircleCI plan page:

Plan Free
1 concurrent job with access to Linux and Windows builds on Small and Medium sizes

Credits
You have used 9,516 of 2,500 credits
9,516 used -7,016 available

At least you can go to the red zone ;D
Circle is out from my point of view.

For more ruby version I think we should switch to https://github.com/ruby/setup-ruby for example https://github.com/rspec/rspec-core/runs/1343995732
Ruby switch seems good... but I still have the resolve issue of
     rspec-xxxx was resolved to 3.10.0.pre, which depends on
      rspec-yyyy (= 3.10.0.pre)

 rspec-xxxx was resolved to 3.10.0.pre, which depends on
      rspec-yyyy (= 3.10.0.pre)

There is no 3.10.0pre, you have stale repos.

You may find this handy if you don't already have this open :D https://docs.github.com/en/free-pro-team@latest/actions/learn-github-actions/migrating-from-circleci-to-github-actions

GA claim they allow for up to 20 concurrent jobs on a Free plan.

I have MRI ruby running for rspec-core https://github.com/rspec/rspec-core/actions/runs/343419327

Yeaah. I spend 3h yesterday trying to understand why ubuntu-latest was failing. I also read the discussion https://github.com/ruby/setup-ruby/issues/100
Do we keep MRI preview on Travis or do we switch to head and allow failures?

We will keep MRI preview on travis or an alternative unless we can convince them to support released preview versions. We will no doubt have the same problem with rc versions, so it might be worth investing in a fork that supports any ruby version.
We also run head anyway and I have already configured allow failures.

TravisCI is very slow to start. I started to work on the Github Action config for rspec-rails: https://github.com/rspec/rspec-rails/tree/github-actions
As always feel free to push on my branch if you want to contribute to what I have started. :)

Its hugely slow due to their "migration" e.g. they are resource starving it.

According to https://www.traviscistatus.com/#month

Some background: https://travis-ci.community/t/builds-hang-in-queued-state/10250/5

we move infrastructure over from .org to .com

https://travis-ci.community/t/org-to-com-migration-deadline/10260

travis-ci.org 1 will be officially closed down completely on December 31st, 2020, allowing us to focus all our efforts on bringing new features and fixes to travis-ci.com 3

https://travis-ci.com/plans

Free for open source. We love the Open Source Community, and to show how much we love it, upon validated request placed with our Support Team you may receive free OSS credits for your public builds.

So I guess we'll have to wait for the migration to happen and re-evaluate the idea of moving to GHA basing on effect/effort.
With GHA I can foresee some problems with e.g. checking out other repos, but they have an interesting approach to this https://github.com/marketplace/actions/github-action-build-chain-cross-repo-builds

With GHA I can foresee some problems with e.g. checking out other repos, but they have an interesting approach to this https://github.com/marketplace/actions/github-action-build-chain-cross-repo-builds

In my experimental branch we have no difficulty checking out other repos, it just works, I have Ruby 2.3 - 3.0 working, just missing JRuby and legacy Rubies. I'm going to be opening PRs over the weekend to move the majority of our builds across.

For rspec-rails I have a weird loading issue ATM. https://github.com/rspec/rspec-rails/runs/1350217878

You might want to try latest ruby/setup-ruby (there were some fixes related to bundler-cache:), trying bundler-cache: false or forcing bundler: 1 (TravisCI pretty much forced Bundler 1 IIRC).
Maybe the --standalone flag also causes that, but I guess it was there before already?

Wondering if this line may cause this error:
if is_mri; then
  export RUBYOPT="--disable=gem"
fi

Locally:
$ ruby --disable=gem -e 'puts Gem'
Traceback (most recent call last):
-e:1:in `<main>': uninitialized constant Gem (NameError)
$ ruby -e 'puts Gem'
Gem

Thanks. Both of you for your answer. It is much better now without RUBYOPT="--disable=gem"

Ok part of the CI is now green. https://github.com/rspec/rspec-rails/runs/1403770340?check_suite_focus=true Still work to do with JRuby and an issue with mailer loading + a small fix with Ruby 3.
I have the same issue on the WIP branch for rails 6.1 https://github.com/rspec/rspec-rails/pull/2400

JRuby is failing on GHA for core too

Todo:

[ ] move rspec-rails over
[ ] move over jruby for main gems
[ ] docker images for legacy builds (pre 2.1)
[ ] investigate how to reduce build duplication (run pr builds but push only for certain branches
[ ] cron jobs

rspec-rails for MRI is nearly ok. Moving to jruby brings to many issues ATM. Few of them related to mail gem and ActionMailer.
https://github.com/rspec/rspec-rails/actions/runs/371123305

I am wondering if we should keep Github Actions for RSpec 4. Do not work on something that we will drop or do not use that much?

We do use it so much though, it's taking hours for Travis to run, delaying PRs immensely. I'm not sure how much effort I'll put in but I'm keen to ditch Travis ASAP.

Sorry my message was not clear. I am wondering how much effort we should invest on "docker images for legacy builds (pre 2.1)" ?

I'm replying to that part...

Closing because this is mostly done 🎉


Crash while importing rive file to unreal

Getting this crash every time I import a certain Rive file into unreal:
Running Windows 11, Unreal Engine 5.4.4 with the latest plugin from the github.
Unhandled Exception: EXCEPTION_ACCESS_VIOLATION 0x0000000300000000
UnrealEditor_Rive!rive::LinearGradient::makeGradient() [D:\rive-runtime\src\shapes\paint\linear_gradient.cpp:140]
UnrealEditor_Rive!rive::LinearGradient::applyTo() [D:\rive-runtime\src\shapes\paint\linear_gradient.cpp:130]
UnrealEditor_Rive!rive::Artboard::advanceInternal() [D:\rive-runtime\src\artboard.cpp:786]
UnrealEditor_Rive!URiveArtboard::Initialize_Internal()
[E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\Rive\Private\Rive\RiveArtboard.cpp:872]
UnrealEditor_Rive!URiveArtboard::Initialize() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\Rive\Private\Rive\RiveArtboard.cpp:657]
UnrealEditor_Rive!URiveArtboard::Reinitialize() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\Rive\Private\Rive\RiveArtboard.cpp:672]
UnrealEditor_Rive!TBaseUObjectMethodDelegateInstance<0,URiveArtboard,void __cdecl(bool),FDefaultDelegateUserPolicy>::ExecuteIfSafe() [E:\Epic Games\UE_5.4\Engine\Source\Runtime\Core\Public\Delegates\DelegateInstancesImpl.h:667]
UnrealEditor_Rive!TMulticastDelegate<void __cdecl(bool),FDefaultDelegateUserPolicy>::Broadcast() [E:\Epic Games\UE_5.4\Engine\Source\Runtime\Core\Public\Delegates\DelegateSignatureImpl.inl:956]
UnrealEditor_Rive!URiveFile::BroadcastInitializationResult() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\Rive\Private\Rive\RiveFile.cpp:173]
UnrealEditor_Rive!`URiveFile::Initialize'::`2'::<lambda_3>::operator()() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\Rive\Private\Rive\RiveFile.cpp:156]
UnrealEditor_RiveRenderer!FRiveRenderer::CallOrRegister_OnInitialized() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\RiveRenderer\Private\RiveRenderer.cpp:73]
UnrealEditor_Rive!URiveFile::Initialize() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\Rive\Private\Rive\RiveFile.cpp:100]
UnrealEditor_Rive!TBaseUObjectMethodDelegateInstance<0,URiveFile,void __cdecl(void),FDefaultDelegateUserPolicy>::Execute() [E:\Epic Games\UE_5.4\Engine\Source\Runtime\Core\Public\Delegates\DelegateInstancesImpl.h:650]
UnrealEditor_RiveRenderer!FRiveRendererModule::CallOrRegister_OnRendererInitialized() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\RiveRenderer\Private\RiveRendererModule.cpp:68]
UnrealEditor_Rive!URiveFile::PostLoad() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\Rive\Private\Rive\RiveFile.cpp:41]
UnrealEditor_CoreUObject
UnrealEditor_Rive!URiveFile::EditorImport() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\Rive\Private\Rive\RiveFile.cpp:243]
UnrealEditor_RiveEditor!URiveFileFactory::FactoryCreateFile() [E:\CW46\ColdWar\Plugins\Rive_0.2.1\Source\RiveEditor\Private\Factories\RiveFileFactory.cpp:64]

Is this plugin version 0.2.1? You say "latest" but jsut want to make sure.

Thanks for verifying. I'm not able to reproduce the crash so I'll need your .rev file.
Can you share your .rev file via a ticket in our Portal?

Thanks for verifying. I'm not able to reproduce the crash so I'll need your .rev file.
Can you share your .rev file via a ticket in our Portal?

Sent the file, thanks :)

To close the loop here: the file had gotten into an invalid state. We are writing in extra checks to make sure this can't happen again and to prevent the crash. Exporting the .rev (backup) file and re-importing it fixed the file.


updated .gitignore

added .zip to .gitignore

Can you provide an example under which circumstances there will be .zip files in a latex project?


📝 Create runbook for core-shared-services-production account

User Story
As a Modernisation Platform Engineer
I want to write a runbook
So that I can recreate resources for the Modernisation Platform account if the need arises
Value / Purpose
This backlog issue identifies key platform components that may need to be recreated.
For the core-shared-services-production account we may need to recreate any or all of the following:

The AWS account
Resources inside the account used by customers

VPCs for shared infrastructure
AMI builder & resources (eg, S3 bucket)
Instance Scheduler
Shared KMS keys

Useful Contacts
No response
Additional Information
No response
Proposal / Unknowns
No response
Definition of Done

[ ] Runbook written
[ ] Another team member has reviewed

Reviewed the steps and they clearly show how to recreate the account in the event of deletion/corruption


Confused logit callback

    Takes the logit predictions of a model and when the probabilities of two classes are very close, the model
    doesn't have high certainty that it should pick one vs the other class.

    This callback shows how the input would have to change to swing the model from one label prediction
    to the other.

Hello @williamFalcon! Thanks for opening this PR.

In the file pl_bolts/callbacks/vision/confused_logit.py:

Line 19:77: W605 invalid escape sequence '\p'
Line 19:91: W605 invalid escape sequence '\p'
Line 20:18: W605 invalid escape sequence '\i'
Line 20:22: W605 invalid escape sequence '{'
Line 20:28: W605 invalid escape sequence '}'
Line 54:9: E741 ambiguous variable name 'l'
Line 88:13: E741 ambiguous variable name 'l'

Do see the Hitchhiker's guide to code style


Update and move Pipet to Gemini

Description of the feature request:
Take the Pipet demo, convert to use Gemini and move to /demos/gemini/

Done now


Why ZenScript?

@shelby3 wrote:

@skaller some where you wrote that you don't know why anyone would run JavaScript on the server instead of something more performant or better designed (presumably with static checking). Some reasons that come to mind (@keean may have others?):

Node.JS and npm are an ecosystem
Reuse of the same code on the server and the browser
C++ is a pita (1500 page manual), Haskell is too obtuse, Python is slower, Scala too many corner issues, Java too verbose no damn unsigned int any way. Some of them don't have GC by default nor typeclasses.
 Bucklescript and Purescript maybe exceptions, but we didn't feel those languages give us what we or the mainstream want.

In short, there really isn't a great general purpose language for modern programming. That is probably why we are doing this, even though both of us probably didn't want to take time away from other important work we also need to do.

Before y'all get too attached the name ZenScript, please note I want to change it to Type-fu.
Seems I better do something about this soon, before this name gets adopted into everyone's consciousness by default.

@skaller replied:

There is a great language for modern programming: Felix.

And frankly I would expect even otherwise Felix would still get very low adoption by the JavaScript/Java/Python mainstream because of Felix's syntax choices and making the language too powerful.
I could be wrong about that though. I am getting old (51) so I may be out-of-touch.
Felix may be a great powerful language though. I don't know it well yet.

But JS is useless for anything serious.  Google agrees, that's why they invented Go, to write servers.

Swift doesn't have typeclasses. It has design pattern which can sort of give some of the benefits of typeclasses, but the compiler isn't doing the heavy lifting afaik.
Also I remember something about Swift using the upper bits of a 64-bit value for something on Android. I forgot the details. Seemed really hokey when I was reading about it.

Given that it has to wrap Objective C, it is pretty cool: variants, pattern matching, HOFs, closures, etc. They say it will be open sourced eventually.
In any case, I have no problem with a language that targets javascript. Zen is certain to be useful. I'm all in favour!

Please continue that discussion here. It doesn't apply to Compiler Implementation. That is why I made this thread.

 I'm 61 :)
Actually, Swift DOES have type classes. It doesn't call them that but it does have them.
I don't know quite how powerful they are but Swift is not static, they're upgrading it.
It's not perfect by any means, but at least the developers knew some dang computer science!

JS is useless for anything serious
I would seriously fight you on that. Unless you can show me a useful language (as in, not some esoteric language no one uses) that has prototypal inheritance, JavaScript is actually one of the most useful languages out there. Also, it's the most used language in the world and it used by massive servers all the time.

Optional typing through TypeScript massively improves the language of course (which is why Google is probably going to make a superset of JavaScript that is somewhat compatible with TypeScript, called SoundScript).
JavaScript is based on Scheme and Self, two of my favorite languages. I wish it had some more features of Scheme of course, but overall, JavaScript is fast, powerful, available on all platforms (there's embedded systems running JS for christ's sake) and it has the best ecosystem by a long shot.

JavaScript's asynchronous concurrency model is a better first step than multi-threading

I think this is important too, I talked about it before, but JavaScript's setTimeout model is pure genius. I'm not even sure which other language even has it? It's a very good concurrency model. It's simple, powerful and as you said @shelby3, a really good first step to build more useful abstractions on top of.

So Swift can do typeclasses. The part I missed before when I linked to this:

There is also another way of expressing the same idea: see this article on how to do it using Swift 1.2 (spoiler alert: <C: Ceasarable where C.T == C>)

So you could do multiple associated type parameter typeclasses with the linked method.
So I would say let's transpile to Swift (or just use Swift!), except:

there is no JS compiler, and if there ever is, it will likely be some bloated LLVM -> asm.js.
ARC instead of GC
no inferred anonymous unions
uses brackets instead of block indenting
no everything-is-an-expression

Other reasons to create ZenScript instead of hoping to advance with Swift over time, is that syntax of Swift isn't ideal from my (and probably @keean's) perspective such as lacking a data type syntax.

@shelby3 I can't see any evidence Swift can do multi-parameter type classes from following the link.

@keean I think you'd write that instead <A, B, C: Ceasarable where C.A == A, C.B == B> assuming Swift supports that intersection syntax on a where clause.

I wrote:

And @evancz the creator of Elm realizes that it’s not designed for all those use cases. I agree with his point that “alternatives make each community stronger”.

The Organizational Structure of Programming Languages.


Use Completion Suggester for Autocomplete

Currently the web application is running off of an Autocomplete JS library, but it only works for one term. A better way to get this feature would be to use Completion Suggester found in Elasticsearch:
https://www.elastic.co/guide/en/elasticsearch/reference/current/search-suggesters-completion.html

match_phrase_query is another alternative

autocomplete is now functional


Re-write to use pandas and add argument parsing

This is a rewrite to use Pandas for cleaner ID extraction and also addresses #244

Codecov Report

Merging #247 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #247   +/-   ##
=======================================
  Coverage   25.25%   25.25%
=======================================
  Files          20       20
  Lines         681      681
=======================================
  Hits          172      172
  Misses        509      509

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 c2360cc...25ab64b. Read the comment docs.

Thank you @thisisjaid!


Users can scroll trough several blocks of content with fixed buttons or mouse scrolling

Description
The design as shown in the attached pdf is for a one-page website in which the users scrolls between the several blocks of content.
220628 mockup sprint fuik.pdf
The static blocks that should be developed so far are:

Landing page
Section start
Call to action

To discuss

Do we want to use some sort of snapping of blocks during scrolling or not? Yes
Do we want some table of contents to quickly move between blocks? Nice to have
How difficult is it to add the background color of the blocks including the diagonal parts? Nice to have

Acceptance criteria

[ ] Developers can add standard blocks with all scroll and anchor functionality attached
[ ] Developers can link to blocks with link buttons
[ ] Blocks automatically scale to the height of the window
[ ] Users can scroll trough blocks with the height fixed to the browser window
[ ] Users can scroll smoothly to a specific block by clicking on a link

Do we want to use some sort of snapping of blocks during scrolling or not? Yes

If you wish to use Tailwind for this, here are the docs for scroll snapping:

https://tailwindcss.com/docs/scroll-snap-align
https://tailwindcss.com/docs/scroll-snap-stop
https://tailwindcss.com/docs/scroll-snap-type

It's well-supported by all modern browsers.

Blocks automatically scale to the height of the window

It also has utility classes which should help with this:

https://tailwindcss.com/docs/min-height

Sorry if I'm stating the obvious. 😇

@antw Thanks for the research and help, @pascalalferink @MariaBee3 please look at these tailwind classes for the implementation of this story.


Use userns-remap=buildkite-agent unconditionally, conditionally delete config at boot time

Needs testing, userns-remap is causing issues with pre-cached Docker images being unavailable once the userns-remap config is applied.

Test procedure:

Build off the resulting Linux image and cache some docker images
Deploy stack with the built AMI that has EnableDockerUserNamespaceRemap: true

[ ] Verify cached images are present

Deploy stack with the built AMI that has EnableDockerUserNamespaceRemap: false

[ ] Verify cache images are present

Initial test shows this hasn’t worked.
With the EnableDockerUserNamespaceRemap: true left on the cached images are present:
$ trap 'kill -- $' INT TERM QUIT; cat /etc/docker/daemon.json
--
  | docker images
  | {
  | "storage-driver": "overlay2",
  | "userns-remap": "buildkite-agent"
  | }
  | REPOSITORY    TAG       IMAGE ID       CREATED       SIZE
  | ruby          latest    090e42145ec2   5 days ago    881MB
  | python        latest    c05c608cfa20   5 days ago    915MB
  | golang        latest    9f8b89ee4475   5 days ago    941MB
  | hello-world   latest    feb5d9fea6a5   3 weeks ago   13.3kB

But with EnableDockerUserNamespaceRemap: false causing the userns-remap configuration to be removed the images are not present:
$ trap 'kill -- $' INT TERM QUIT; cat /etc/docker/daemon.json
--
  | docker images
  | {
  | "storage-driver": "overlay2"
  | }
  | REPOSITORY   TAG       IMAGE ID   CREATED   SIZE

Will need to dig into this some more.


sfdx force:apex:test:run process can't be stopped with ctrl+c

Summary
After running sfdx force:apex:test:run without specifying the list of tests I decided to stop the process, but something did not allow me to do it. Im pressing ctrl+c to exit the process and the terminal does not let me do it.

Steps To Reproduce:
1 - run sfdx force:apex:test:run -o TARGET_ORG -c -r human -d .
2 - press ctrl+c to exit the process
3 - verify that something is not allowing the process to exit
Expected result
pressing ctrl+c must exit the process.
obs: im not saying killing the process should stop the tests. I just want to regain control of my terminal.
Actual result
pressing ctrl+c does not exit the process
System Information

Which shell/terminal are you using?  zsh

{
  "cliVersion": "sfdx-cli/7.194.1",
  "architecture": "linux-x64",
  "nodeVersion": "node-v14.21.3",
  "osVersion": "Linux 5.10.0-21-cloud-amd64",
  "shell": "bash",
  "rootPath": "/home/admin/.nvm/versions/node/v14.21.3/lib/node_modules/sfdx-cli",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 2.1.6 (core)",
    "@oclif/plugin-commands 2.2.11 (core)",
    "@oclif/plugin-help 5.2.8 (core)",
    "@oclif/plugin-not-found 2.3.22 (core)",
    "@oclif/plugin-plugins 2.4.3 (core)",
    "@oclif/plugin-search 0.0.14 (core)",
    "@oclif/plugin-update 3.1.8 (core)",
    "@oclif/plugin-version 1.3.1 (core)",
    "@oclif/plugin-warn-if-update-available 2.0.32 (core)",
    "@oclif/plugin-which 2.2.17 (core)",
    "apex 2.2.7 (core)",
    "auth 2.7.10 (core)",
    "community 2.2.8 (core)",
    "custom-metadata 2.1.9 (core)",
    "data 2.3.8 (core)",
    "info 2.6.2 (core)",
    "limits 2.3.9 (core)",
    "org 2.6.2 (core)",
    "packaging 1.16.2 (core)",
    "schema 2.3.4 (core)",
    "settings 1.4.2 (core)",
    "signups 1.4.8 (core)",
    "source 2.8.0 (core)",
    "telemetry 2.1.3 (core)",
    "templates 55.4.4 (core)",
    "trust 2.4.5 (core)",
    "user 2.3.5 (core)",
    "@salesforce/sfdx-plugin-lwc-test 1.0.1 (core)",
    "sfdx-cli 7.194.1 (core)",
    "sfdx-git-delta 5.17.1 (user)"
  ]
}

Additional information
N/A

Hey @AllanOricil thanks for bringing this up - we just reverted behavior on that command to run asynchronously by default, you can test with the nightly or latest-rc tag
hopefully this change nullifies this issue

 ➜  sfdx force apex test run -c -r human -d .
Warning: The "force apex test run" command has been deprecated. Use "apex run test" instead.
^CError (1): Cancelled

 ➜  sfdx -v
sfdx-cli/7.196.6 darwin-arm64 node-v18.7.0

I was able to cancel it, were you?

Cool - I'll close this issue

when will u guys give me the "most valuable tester" trophy ? 🥺


Design a custom Zettelcorn project logo.

unDraw is great, but having a custom project logo would be even better! Preferably, the art style would be a "flat modern" look as seen on the unDraw website. The design should riff on the ideas of "kettle corn" and "knowledge base". If you are interested in helping, just leave a comment below!

While nice to have a custom logo is unnecessary at this time for this project.


adds a toast when adding products to cart

Prova att lägga till saker i kundvagnen

The toasts cover the buttons right now, maybe could but them a bit lower?

Yes, was looking to do errors with this (like if adding to cart failed) but thought it looked pretty good with something visual when adding stuff. But yeah I can lower them if we want to keep them.


PackedByteArray copied and converted to Array without error when passed to a function with typed arguments

Tested versions
4.2.1.stable.arch_linux
System information
Godot v4.2.1.stable unknown - Manjaro Linux #1 SMP PREEMPT_DYNAMIC Fri Mar 15 21:05:45 UTC 2024 - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4060 (nvidia; 550.67) - Intel(R) Core(TM) i7-9700F CPU @ 3.00GHz (8 Threads)
Issue description
If a function has an Array-typed param, passing a PackedByteArray to it will not cause an error, and will silently convert the PBA to a normal Array (and making it effectively passed by value).
extends Node2D

func _ready():
    var val := PackedByteArray()
    val.resize(10)
    val.fill(0)
    print(val)  	# [0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
    f1(val) 		# No error!
    print(val)	# [0, 0, 0, 0, 0, 0, 0, 0, 0, 0]

func f1(param: Array):
    param[5] = 10
    print(param) 	# [0, 0, 0, 0, 0, 10, 0, 0, 0, 0]

Steps to reproduce
Run the above code - there is no type error and val is not updated in _ready
Minimal reproduction project (MRP)
N/A

Related #40249


Combine top hash caching with bucket chaining

Reintroduces bucket chaining, so that the map can cope with poor quality hash functions. This is a required preliminary step for #46.

Codecov Report
Base: 93.10% // Head: 94.69% // Increases project coverage by +1.59% :tada:

Coverage data is based on head (64f7b60) compared to base (c340e98).
Patch coverage: 97.01% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
+ Coverage   93.10%   94.69%   +1.59%
==========================================
  Files           6        6
  Lines         725      830     +105
==========================================
+ Hits          675      786     +111
+ Misses         40       35       -5
+ Partials       10        9       -1

Impacted Files
Coverage Δ

map.go
93.65% <96.75%> (+1.55%)
:arrow_up:

mapof.go
94.16% <97.35%> (+3.02%)
:arrow_up:

mpmcqueue.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Store workflow execution trace in user's storage account

Problem:
We collect information about TesTask and store it as json in PostgreSQL database. This information might be useful for debugging, workflow performance tuning, etc, but it is not accessible for user when it's stored in database.
Solution:
Store TesTask information in user's storage account.
At this moment for each workflow run we generate 3 files in container 'outputs': metadata.json, outputs.data, timing.html. We might create one more file trace.json that will contain all TesTasks for that workflow run.
In Scheduler.cs (OrchestrateTesTasksOnBatch) when tesTask changes state to terminal state (completed, failed) we append tesTask to trace.json as JSONL.
Additional context
This work overlaps with the work required for having TES as a self-contained executable.  Requires discussion of implementation details to fit into future work on self-contained executable.

TES doesn't have scope for workflow-level features. This looks like an issue that should instead be in the microsoft/CromwellOnAzure repo instead (TriggerService, specifically, which creates those three files mentioned and has access to the TesTasks in the PostgreSQL db).

Possible solution: #360

More task information is presented to cromwell (so workflows failed by failing tasks now have execution failure metadata provided both to cromwell's caller(s) as well as in cromwell's workflow logs. This information includes links (without SAS tokens) to trace logs from task executions that are today stored in the user's storage account.


Only one global node

Because our global view represents Erlang cluster, there are situations when we have only one node. With this setup our global view is empty. Is there any way we can show only one node on global view?

This is definitely a bug; I'll dig into why it's not showing if there's only one node.
However, if you have only one node, do you need the global view?

You're right. The answer is yes and no. Right now we have region view only for one node in cluster so global view shows communication between nodes. Our goal is to give user access to select inspected node in global view in Vizceral, and showing this one node when only one node is available will make thinks consistent and explicit.

Are you trying to basically start at the region view level?  Because, I'm very interested in being able to do that as well.  Right, now create a completely made-up Internet and Region node just so I can go into the region view which is what I actually want.  So I don't actually need the Global view either.
I'd be interested in some pointers or guidance on the where and what to look for to "skip" the global level.

You just have to setView to ["INTERNET"] if you want region view for that node

I've found this is present when this one node has no connections. Temporary solution could be creating void connection to none existing node.

Here is line causing this issue: https://github.com/Netflix/vizceral/blob/master/src/base/trafficGraph.js#L666
Question is, how should we treat orphan nodes. For my use case it's necessary to have at least entry orphan node present. This could be good solution overall to add isEntryNode check for that.
I'll make PR and wait for your response @jrsquared

Because it's related to #55 I'm adding this to #59 PR

@Baransu,
I think we have some terminology mis-match.  I tried the setView(['INTERNET']) like you suggested, but came to realize that what I'm calling "Region view", isn't what you're calling "Region view".
To me, there are 3 "zoom" levels

Global (where you see the region nodes connected to Internet)
Region (where you see all the "services" contained in a region graphed out... ltrTreeLayout)
Detail Node (where you double-clicked a "service" and now see just that node and any nodes connected to the double-clicked node).

What do you call the "zoom" levels?
@jrsquared, what about you?  What's the "official" level/view "names"?

@aSqrd-eSqrd That is what we call the zoom levels internally too, glad to see I'm not the only one!
If you do not have a global view at all, you can just create a graph that only has a region view and it should just work. You don't need to fake a global view and then set the view to the region view. The example has a global view and region views just because that's how we built it and use it internally, but any graphs or subgraphs are possible.

For us :

cluster
node (in cluster)
process (Erlang VM process)

but that doesn't matter, it's the same as you.
I know I don't have to fake global view but for us it's important to show one node in global view and allow interaction with that node as well.

@jrsquared,
<<I'm duplicating this entire comment plus necessary context as a new issue (which it is), see issue #60.>>
I was really excited to see that a global node isn't required and immediately tried it out.  It is a change I need, but haven't had the guts to take on yet.
It does kind of work, but there is something crazy happens with event attachments or something.  I took my usual test JSON file which consist of one region, an Internet node and a single connection between them at the global, while the region is has about 30+ nodes interconnected.   I deleted the global render chunk so that only the region node and its children remained.
It graphed just like expected (ltrTree), but none of the child nodes can be zoomed into.  In fact, when I lathered the whole thing up with breakpoints I only get a breakpoint on the single click which opens the details panel, but nothing at all for the double-click that should zoom to the detail level.

Fixed with #59


Use container-based infrastructure

Mostly by dropping the use of sudo and installing on directories under $HOME.
I also enabled luajit-2.1 in the matrix (and luajit-2.0, which I just realized is redundant with luajit).

I also enabled luajit-2.1 in the matrix (and luajit-2.0, which I just realized is redundant with luajit).

luajit use last stable version (2.0.3) but luajit-2.0 uses current master.

Ah, thanks. Didn't realize that. Will submit a PR to clarify that in the docs.


Increase badge size slightly so that "PATCH" method fits inside better

This pull request fixes an issue where the "PATCH" method does not fit inside the badge very nicely at the cost of some horizontal space for the summary.
Before:

After:

Coverage remained the same at 82.137% when pulling 2d77880cd605bbd817ef908b6053e8dd2370866b on fredriklengstrand:badge-size into 822d0381908eb172d951c4534fbc13469d236074 on Rebilly:master.

Hi @fredriklengstrand, thanks for the PR.
I don't think increasing badge size is a good choice. This badge takes too much space even now.
I would prefer shortening PATCH to something like PTCH (similar to how it is done for DELETE -> DEL) or making PATCH font size a bit smaller.
What do you think?

I tried both shortening PATCH to something like PTCH and making PATCH font size smaller and both look not so good.
Looks like making badge size bigger is the best option here.


Architecture: where are we storing our movies?

We need some kind of data storage for the movies that we pull from the Netflix data set, like in the process described in #2
We could use a relation database:

MySQL/MariaDB
PostgreSQL

Or maybe a no-SQL database:

MongoDB

And maybe some kind of vector database?

I'm studying for a system design interview right now I feel like I should be able to answer this, but idk.
Like I'd imagine storing the media data itself, it would be a large amount but probably wouldn't increase exponentially or anything (would be limited by the rate at which new things come out), so not sure if it would need to be highly scalable (which I believe NoSQL would be better for). However, there could theoretically be a large number of users or something else that might need to scale significantly at some point, though I'm not sure that's an immediate concern.
Depending on what media types there are, I'm not sure what the schema would look like. I would think a lot of things would be the same regardless (title, rating, creator(s), genre, etc). But with so many different types there may be different media specific attributes that a dynamic schema would be better suited for. So kinda leaning towards NoSQL for this, but unsure.
And then I'm not sure ACID properties are necessarily super important, like eventual consistency is fine. Performance is more important I would say. So probably don't need SQL for that. But there's probably a lot of complex queries/ relationships between the data that SQL might be beneficial for.
Overall, I'd lean towards NoSQL, although I like Postgres. But there could also be a scenario where SQL could be beneficial as well. However this could change depending on what the vector database is used for. I'm only vaguely familiar with what they are, but don't know enough to evaluate tradeoffs etc. But from what I understand that could definitely be useful for this for sure, and plus just want to learn more about them, so definitely down to use one of those as well.

One thing we also discussed briefly in the meeting on Thursday was that we might have multiple databases:

Our "media store" where we have entries for every movie, book, etc
The "application database" where we store user profiles and information related to serving the frontend/website

I think for the media store, MongoDB makes a lot of sense. There is only one "table" in this database: the items/media. So there's nothing to JOIN against, no need for relational tables. It will be easier to stash and retrieve Python dictionaries to and from Mongo than writing SQL. And like you pointed out, having a flexible/dynamic schema could be very beneficial (no schema migrations).

Yea I totally agree, I think that makes sense. What would the vector database potentially be used for? Was that like an alternative option for the media store or application database, or something else? And when what would we use for the application one?

A vector database I believe is used for AI processes like Retrieval Augmented Generation (RAG). Not clear if we need one at this point, though I've also read that Mongo can act as one as well.


Godot try to use script even if it is removed from tree

Godot version
3.4stable
System information
Arch
Issue description
After modification of scene tree with renaming and movement script name . In moment of next start play godot try to use old script resource and show message :
 modules/gdscript/gdscript.cpp:775 - Condition "err" is true. Returned: err
 Cannot load source code from file 'res://Enemies/EnemyEngine.gd'.
 Failed loading resource: res://Enemies/EnemyEngine.gd. Make sure resources have been imported by opening the project in the editor at least once.

In same moment new script works .  Feels like something with caching of resources.
Steps to reproduce

create Scene
add script to res://script.gd
bind script to node
Play
stop
create folder res://scripts and move script to res://scripts/newscript.gd
unbind script res://script.gd from node and bind res://scripts/newscript.gd
start play and see error

PS: Issue disappear only in case if completely restart Godot.
Minimal reproduction project
No response

@jpVm5jYYRE1VIKL Can you reproduce this in Godot 3.3.4?

Cannot recreate this issue on 3.3.4 . And now cannot recreate on 3.4 as well. Feels like have random nature. I ll keep it on eye and case if get again will return to this topic.


socket: Respect IPv6 flowinfo and scope_id in InetAddr::from_std

Fixes #329

Looks great! Can't really test it as build is failing on nightly (nightly is required because of rust-lang/rust#32424).

@misuzu it builds locally at 8422bfa on my machine. I think the CI is more strict with raw_pointer_derive. If you have time, could you check out that commit and see if it works?

Actually, if you have a test case we can add, that would be even better!

@kamalmarhubi std::net:SocketAddrV6 is broken on stable so test also will be broken. That stuff fixed in nightly 2016-03-25+
Seems like there is some bug in rustc.
$ rustc -V
rustc 1.9.0-nightly (b678600ac 2016-03-29)
$ cargo build
   Compiling nix v0.5.1-pre (file:///home/misuzu/Documents/Rust/nix-rust)
src/sys/signal.rs:157:9: 157:27 warning: lint raw_pointer_derive has been removed: using derive with raw pointers is ok, #[warn(renamed_and_removed_lints)] on by default
src/sys/signal.rs:157 #[allow(raw_pointer_derive)]
                              ^~~~~~~~~~~~~~~~~~
src/sys/signal.rs:157:9: 157:27 warning: lint raw_pointer_derive has been removed: using derive with raw pointers is ok, #[warn(renamed_and_removed_lints)] on by default
src/sys/signal.rs:157 #[allow(raw_pointer_derive)]
                              ^~~~~~~~~~~~~~~~~~
src/sys/signal.rs:158:23: 158:32 note: in this expansion of #[derive_PartialEq] (defined in src/sys/signal.rs)
src/sys/signal.rs:157:9: 157:27 warning: lint raw_pointer_derive has been removed: using derive with raw pointers is ok, #[warn(renamed_and_removed_lints)] on by default
src/sys/signal.rs:157 #[allow(raw_pointer_derive)]
                              ^~~~~~~~~~~~~~~~~~
src/sys/signal.rs:158:17: 158:21 note: in this expansion of #[derive_Copy] (defined in src/sys/signal.rs)
src/sys/signal.rs:157:9: 157:27 warning: lint raw_pointer_derive has been removed: using derive with raw pointers is ok, #[warn(renamed_and_removed_lints)] on by default
src/sys/signal.rs:157 #[allow(raw_pointer_derive)]
                              ^~~~~~~~~~~~~~~~~~
src/sys/signal.rs:158:10: 158:15 note: in this expansion of #[derive_Clone] (defined in src/sys/signal.rs)
src/sched.rs:208:20: 208:34 warning: `extern "C" fn(*mut Box<core::ops::FnMut() -> isize>) -> i32 {sched::clone::callback}` is now zero-sized and has to be cast to a pointer before transmuting to `*const extern "C" fn(*const Box<core::ops::FnMut() -> isize>) -> i32`, #[warn(transmute_from_fn_item_types)] on by default
src/sched.rs:208         ffi::clone(mem::transmute(callback), ptr as *mut c_void, flags.bits(), &mut cb)
                                    ^~~~~~~~~~~~~~
src/sched.rs:208:20: 208:34 warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
src/sched.rs:208:20: 208:34 note: for more information, see issue #19925 <https://github.com/rust-lang/rust/issues/19925>
rustc: /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/llvm/include/llvm/Support/Casting.h:237: typename llvm::cast_retty<X, Y*>::ret_type llvm::cast(Y*) [with X = llvm::Function; Y = llvm::Value; typename llvm::cast_retty<X, Y*>::ret_type = llvm::Function*]: Assertion `isa<X>(Val) && "cast<Ty>() argument of incompatible type!"' failed.
error: Could not compile `nix`.

To learn more, run the command again with --verbose.

I was wrong, this fix doesn't need rust-lang/rust#32424
You can add this test to test/sys/test_socket.rs
#[test]
pub fn test_inetv6_addr_to_sock_addr() {
    let port: u16 = 3000;
    let flowinfo: u32 = 1;
    let scope_id: u32 = 2;
    let actual = net::SocketAddr::V6(net::SocketAddrV6::new("fe80::1".parse().unwrap(), port, flowinfo, scope_id));
    let addr = InetAddr::from_std(&actual);

    match addr {
        InetAddr::V6(addr) => {
            assert_eq!(addr.sin6_port, port.to_be());
            assert_eq!(addr.sin6_flowinfo, flowinfo);
            assert_eq!(addr.sin6_scope_id, scope_id);
        }
        _ => panic!("nope"),
    }

    assert_eq!(actual, addr.to_std());
}

Added the test, thanks!

r? @fiveop

@homu r+

:pushpin: Commit 6d01a26 has been approved by fiveop

:zap: Test exempted - status


allow node modules to be used when calling npm_run

this also means if:

a local NPM is configured in gradle
a shrinkwrap is included
the npm package is not included in the shrinkwrap/package.json

then the build should fail with a helpful message.

sorry, updated this to correctly fix the bug + included a helpful error message when the user mis-configures their build

womp. i'm using a bad shrinkwrap. fixing that now :(

Thanks. Nice work. Just get the build to go green and I will be happy to pull it :-)

Coverage increased (+0.01%) to 99.571% when pulling 8540609aa17de9a2225d8fd1f42ce7c9ba757338 on jmcampanini:jc/npm-setup-before-npm-run-tasks into 697e3ff2008ea4f73c7f2ed6535b6aecbf271cb3 on srs:master.

fixed. it does increase the build time by a bit, since it actually npm installs things :cry: maybe a future commit can pick smaller packages to cut this down.

how long does the appveyor queue usually take? :-\

Appveyor takes a long time as you have noticed. But it's critical for testing on windows.

cool. it passed :-D let me know if there's anything else i can do for this PR.
when do you think a new release (0.13?) can be cut?

Not sure when 0.13 will be out, but will try to get it out right after the weekend. Next version after 0.13 will probably be numbered 1.0 since I will merge node/grunt/gulp plugins.

sounds good. thank you! :)

ping

Sorry about this. Been too occupied with other stuff, but picking this up again now.


Needs an update to work with latest mekanism

Whenever a player uses the module, they get kicked, with this error on server side.
NF: 21.0.151
Mek: 10.6.7
GravUnit: 5.2

Fixed, Thanks for report.
Update to 5.3 version please.


NativeIntegerTypeDecoder+ErrorTypeException warnings

Every solution I pass to HtmlGenerator.exe in https://www.nuget.org/packages/SourceBrowser/1.0.34 produces hundreds of these warnings:

First chance exception while processing assembly: SomeAssemblyName
Microsoft.CodeAnalysis.CSharp.Symbols.Metadata.PE.NativeIntegerTypeDecoder+ErrorTypeException: Exception of type 'Microsoft.CodeAnalysis.CSharp.Symbols.Metadata.PE.NativeIntegerTypeDecoder+ErrorTypeException' was thrown.
at Microsoft.CodeAnalysis.CSharp.Symbols.Metadata.PE.NativeIntegerTypeDecoder.TransformType(TypeSymbol type)

Versions of Visual Studio installed: 16.9.3, 16.10.P1
Would it help if I attached a repro project?
What does this warning mean?

This is probably benign, it's just a first-chance exception deep in the bowels of Roslyn, it's being caught I'm guessing.
I guess I should try bumping Roslyn to latest? But it might not be fixed. Also it might be a real bug in Roslyn that's masked.

Haha would you look at that:
https://github.com/dotnet/roslyn/issues/49823

The 1.0.33 NuGet package doesn't have this issue, if that helps. I see the only difference is updating Roslyn to 3.9.0-2.final`.
Closing since this is upstream and probably benign. If you wanted to track something, feel free to reopen.


Add features from containers integrations to datadog-agent provisioners

What does this PR do?
Add some features to datadog-agent provisioners. That will be required to migrate containers e2e tests to provisioners.
Update provisioners to use new ECS component created in https://github.com/DataDog/test-infra-definitions/pull/1193
Motivation
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge

/merge

/merge


Crop icons when uploading

In the case of non-square icons, users may wish to be able to upload them and crop them on the site using JavaScript. It may be hard to integrate this UI with the current upload interface, and it may also be hard to create the interface (allowing a modified image to be created and then saved).
This was possible on Dreamwidth, at least (especially?) for images that are over 100px × 100px.

Warning for it (if it doesn't already?) and letting the user edit on their end would likely be easier and could work at least as a short term fix?

Images over 400x400 resized to less than that in #1174, I assume this does something sane with non-square icons


Redo Context Benchmark

Pull Request Checklist

[ ] Skip results with context added or removed
[ ] Generate a summary
[ ] Add to benchmark/summary results
[ ] Add auto-context precision/recall to BenchmarkResultSummary
[ ] Documentation has been updated, or this change doesn't require that

Replaced by #549


UofT-DSI | Deep Learning - Labs 1-6

What changes are you trying to make?
I have completed six labs covering fundamental deep learning concepts and implementations:

Lab 1: Implemented neural networks using Keras, exploring basic architectures and training procedures
Lab 2: Built backpropagation from scratch to understand gradient flow in multilayer networks
Lab 3: Created collaborative filtering and content-based recommender systems
Lab 4: Implemented convolution operations and CNN architectures for image processing
Lab 5: Built object detection models using pre-trained networks and transfer learning
Lab 6: Developed text classification models using neural network architectures

Each lab includes detailed code implementations, analysis, and documentation.
What did you learn from the changes you have made?
Key learnings include:

Understanding neural network fundamentals through hands-on Keras implementation
Deep comprehension of backpropagation mechanics by coding it from scratch
Practical experience with recommendation algorithms and evaluation metrics
Knowledge of CNN architectures and convolution operations
Experience with transfer learning and object detection frameworks
Skills in processing text data and building neural networks for NLP tasks

Was there another approach you were thinking about making?
Alternative approaches considered:

Using PyTorch instead of Keras for neural network implementations
Implementing different backpropagation optimization algorithms
Exploring matrix factorization techniques for recommender systems
Testing various pre-trained models for transfer learning
Trying different text preprocessing approaches

Were there any challenges?
Main challenges included:

Managing computational resources for large datasets
Tuning hyperparameters across different model architectures
Handling imbalanced datasets in text classification
Working with pre-trained models and transfer learning

How were these changes tested?
Testing was performed through:

Validation on held-out test sets
Comparison with baseline models
Visual inspection of training curves
Performance metric evaluation

Checklist

[x] I can confirm that my changes are working as intended

Labs 1-6 complete


Remove absolute pathnames to binaries and rely on PATH

Fixes issues with distributions that use non-standard binary paths (e.g., in
isolated build environments). Scripts rely on PATH for other programs, anyway.

Yes, I do. It is called NixOS. The reasoning behind this is to have a system in which all runtime dependencies are statically resolved for all binaries. Depending on context this might mean different things. It involves, though, that programs don't assume to find other programs in fixed locations with two exceptions (to fix the bootstrapping problem): /bin/sh, and /usr/bin/env.
This allows you to guarantee that a particular build of a program only ever calls a particular build of, e.g., pwd, regardless of other versions of pwd existing on the system.

On the topic of portability, sh isn’t guaranteed to have the local builtin.

redo's sh selector (t/shelltest.od) will reject any sh that doesn't have
'local'. I still haven't found one that doesn't, so that's not a big
problem, even if it isn't technically in POSIX.
ᐧ
On Tue, Oct 27, 2020 at 3:19 PM periish<EMAIL_ADDRESS>wrote:

On the topic of portability, sh isn’t guaranteed to have the local
builtin.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/apenwarr/redo/pull/29#issuecomment-717478837, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAFA4G6PWCC27NVPPWI7JTSM4MMRANCNFSM4IIOR5ZA
.

--
Avery Pennarun // CEO @ Tailscale

How should we proceed? Shall I keep the path to pwd in a variable?

Why not $PWD? It’s part of POSIX.

On 11 Nov 2020, at 12:30, spacefrogg<EMAIL_ADDRESS>wrote:

@spacefrogg commented on this pull request.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

This particular chunk of code is trying to defeat the unhelpful
physical-logical distinction in $PWD. If you don't know the difference
between pwd -P and pwd -L, go read about those first :)
But there was some other obscure reason I couldn't simply use the
sh-internal pwd -P, which I've now forgotten. I do remember it was
extra double plus arcane.
On Wed, Nov 11, 2020 at 7:38 PM periish<EMAIL_ADDRESS>wrote:

Why not $PWD? It’s part of POSIX.

On 11 Nov 2020, at 12:30, spacefrogg<EMAIL_ADDRESS>wrote:

@spacefrogg commented on this pull request.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

--
Avery Pennarun // CEO @ Tailscale

I incorporated all changes. I am a bit puzzled, now. @apenwarr, do you accept this PR or does it need further work?

@apenwarr As this PR hasn't seen any attention for the last year now, I want to reach out to you again concerning the time you are willing to invest into this topic.
Apart from this particular PR, I have spent some effort brushing up redoconf. The current implementation has several bugs, some deficiencies and lacked modularity/extensibility in certain respects (as it was very focussed on compiling C code). I kept its function completely compatible with the libssl/hello world example from the documentation.
I also added builders and feature detectors for building TeX documents.
Before publishing my extended version of redoconf, I wanted to hear your opinion on the way forward. I don't want to distract you from anything, which is why I would be willing to publish and push forward redoconf separately from redo.


width not divisible by 2

Describe the bug
#0:0 (h264 (native) -> h264 (libx264)) Press [q] to stop, [?] for help [libx264 @ 0x557aed4d1d20] width not divisible by 2 (577x324) Error initializing output stream 0:0 -- Error while opening encoder for output stream #0:0 - maybe incorrect parameters such as bit_rate, rate, width or height Conversion failed!
To Reproduce
Steps to reproduce the behavior:

I used ->HLS() and ->autoGenerateRepresentations([]) on a video that had a width or height that wasn't an even number (in my case, 577x324).

Expected behavior
Properly encode video.
Desktop/Server (please complete the following information):

ffmpeg version 3.4.6-0ubuntu<IP_ADDRESS>

Additional context
I fixed the problem with the following code, which feels a little bit weird. Maybe you will come up with a better solution. Also note that I fixed a typo in the phpdoc:
/**
 * @param $k_bitrate
 * @param $width
 * @param $height
 * @return Representation
 * @throws InvalidArgumentException
 */
private function addRep($k_bitrate, $width, $height): Representation {
    $evenWidth = $this->roundUpToNextEvenNumber($width);
    $evenHeight = $this->roundUpToNextEvenNumber($height);
    return (new Representation)->setKiloBitrate($k_bitrate)->setResize($evenWidth, $evenHeight);
}

protected function roundUpToNextEvenNumber(float $input): int {//https://stackoverflow.com/a/34363542/
    $output = ceil($input); // Round up decimals to an integer
    if ($output % 2 == 1) {
        $output++; // If odd, add one
    }
    return $output;
}

https://www.google.com/search?q=ffmpeg+"width+not+divisible+by+2" shows lots of people with this problem, but I didn't like any of the solutions I saw.

Thank you so much for taking the time and effort in finding bugs(typo in the phpdoc and this one). I am not able to spend time on this project at the moment but I will try to find a solution in the next release.

I just fixed the bug and now you can update the package.
composer update aminyazdanpanah/php-ffmpeg-video-streaming

@aminyazdanpanah Thanks!


Dynamic FormBuilderTextField delete method always unsubscribe last added

Hi,
I have a simple form with 5 rows made by a FormBuilderTextField with a "remove" button each, as in the code below.
When I push the "remove" button on the first row, the Field object is correctly removed from field list, FormBuilderState.unregisterField unregisters always the last added TextField and assert(_fields.containsKey(name)) fails with the following error lines:
`======== Exception caught by widgets library =====================================================
The following assertion was thrown while finalizing the widget tree:
'package:flutter_form_builder/src/form_builder.dart': Failed assertion: line 135 pos 12: '_fields.containsKey(name)': is not true.
When the exception was thrown, this was the stack:
#2      FormBuilderState.unregisterField (package:flutter_form_builder/src/form_builder.dart:135:12)
#3      FormBuilderFieldState.dispose (package:flutter_form_builder/src/form_builder_field.dart:134:24)
#4      _FormBuilderTextFieldState.dispose (package:flutter_form_builder/src/fields/form_builder_text_field.dart:475:11)
#5      StatefulElement.unmount (package:flutter/src/widgets/framework.dart:4854:12)
#6      _InactiveElements._unmount (package:flutter/src/widgets/framework.dart:1998:13)
...
====================================================================================================`
I'm using flutter 1.22.6 stable channel and flutter_form_builder 4.2.0.
`import 'package:flutter/material.dart';
import 'package:flutter_form_builder/flutter_form_builder.dart';
void main() => runApp(MyFormExample());
class MyFormExample extends StatefulWidget {
@override
_MyFormExampleState createState() => _MyFormExampleState();
}
class Field {
final String name;
String value;
Field(this.name, [this.value]);
}
class _MyFormExampleState extends State {
final GlobalKey _formKey = GlobalKey();
List fields = List.generate(5, (index) => Field(index.toString()));
List controllers;
@override
void initState() {
super.initState();
controllers = List.generate(5, (index) {
TextEditingController controller = TextEditingController();
controller.text="";
return controller;
});
}
@override
Widget build(BuildContext context) {
return MaterialApp(
home: Scaffold(
appBar: AppBar(
title: const Text('MyForm Example'),
),
body: FormBuilder(
key: _formKey,
child: ListView.builder(
itemCount: fields.length,
itemBuilder: (BuildContext context, int position) {
return Row(
children: [
Expanded(
child: FormBuilderTextField(
name: fields[position].name,
controller: controllers[position],
// focusNode: _focusNodes[position],
decoration: InputDecoration(labelText: "Field ${fields[position].name}"),
),
),
MaterialButton(
child: Icon(Icons.remove_circle_outline),
onPressed: () => setState( () => fields.removeAt(position)),
)
],
);
}
),
),
),
);
}
}
`

encountered  the same problem, can anybody help?

Any updates?

Add a key to the FormBuilderTextField as follows
FormBuilderTextField(                                 key: UniqueKey(), // rest of the attributes )
as well as a key for Scaffold
final GlobalKey<ScaffoldState> _scaffoldKey = GlobalKey<ScaffoldState>(); Scaffold(           key: _scaffoldKey,)
so in total you should have 3 different keys declared , one for Scaffold , one for the FormBuilder and UniqueKey() for FormBuilderTextField

Add a key to the FormBuilderTextField as follows FormBuilderTextField( key: UniqueKey(), // rest of the attributes )
as well as a key for Scaffold
final GlobalKey<ScaffoldState> _scaffoldKey = GlobalKey<ScaffoldState>(); Scaffold( key: _scaffoldKey,)
so in total you should have 3 different keys declared , one for Scaffold , one for the FormBuilder and UniqueKey() for FormBuilderTextField

Great, this seems to works fine!
There is only an annoying warning saying:
I/flutter ( 4034): Warning! Replacing duplicate Field for 1 -- this is OK to ignore as long as the field was intentionally replaced
Thanks!

FormBuilderTextField( key: UniqueKey(), // rest of the attributes ) is, maybe, the right way to use FormBuilderTextField or just a workaround.
Anyway, it works


Add versioning and schema migration for master catalog

We need to allow upgrading existing Myria clusters in-place, including when the catalog schema has changed. This would require adding a version field to the catalog (probably in a singleton table), and adding code accompanying any schema change to upgrade the catalog schema from the previous schema version. I don't think we should support upgrading from versions older than the immediate predecessor.
This should fix issues like https://github.com/uwescience/myria/issues/841. @dhutchis @jingjingwang @jortiz16 please comment.

In case it's helpful, this is the command I executed in SQLite to upgrade
ALTER TABLE relation_schema ADD is_indexed INTEGER NOT NULL DEFAULT 0;
ALTER TABLE relation_schema ADD is_ascending_index INTEGER DEFAULT 0;


Mobile menu hides when scrolling on mobile

On mobile, when the mobile menu is open and you touch the screen and begin to scroll when you release your finger the sidebar-menu gets display none. The JS could use some attention and refactoring.

I cannot reproduce this on my Android phone (5.1.1.), both Chrome and built-in browser works fine.
What mobile gadget/browser do you experience this issue on?
I'm not a "master of frontend", so would love for others to chime in with a fix ...

Hi @bep i'm using a Samsung Galaxy s5 with Android version 6.0.1 and Chrome version 50.0.2661.89
I'd be happy to debug when I have additional time but wanted to submit an issue to see if anyone else is experiencing the same.

I'm also not able to reproduce the issue with Android 5.1.1 and Chrome.

This belongs to the repo of the Hugo Theme https://github.com/gohugoio/gohugoioTheme/
Also there is an issue about the mobile menu which has other problems as well
Feel free to continue the discussion in https://github.com/gohugoio/gohugoioTheme/issues/141


Shader compilation fails on Safari due to luminance function being undefined.

Describe the bug
After updating to the latest version, the console shows the following error when trying to render any scene using the path tracer. This did not happen with version 8.
Safari throws an exception during shader compilation, as seen below. Chrome throws no exception, but just doesn't render.
Expected behavior
The luminance function should be defined and no error should occur.
Screenshots and Repro Model

Platform:

Device: Desktop
OS: MacOS
GPU: Radeon
Browser: Safari
Three.js version: 148
Library version: 11

Interesting. The function was changed here: https://github.com/gkjohnson/three-gpu-pathtracer/commit/bd8f9df0fff30378fc39390910bf05154609f585.

I can get past the failure if I change out this line of code in my app:
const rgbeLoader = new RGBELoader().setDataType(HalfFloatType)

And use the async version as shown in the examples here.
const texture = await new RGBELoader().loadAsync(`/static/environment/${envMapPath}`)

But then I get an all black rendering.

Thanks for the report! I'm not able to reproduce on my M1 machine - so a couple questions:

Which Mac are you on?
Is Safari up to date?
Have you tried the latest development version in the repo after #350 has been merged?

Interesting. The function was changed here: https://github.com/gkjohnson/three-gpu-pathtracer/commit/bd8f9df0fff30378fc39390910bf05154609f585. I can't find where luminance is defined.

The luminance function is defined in three.js' "common" shader chunk here. If readding the original implementation and removing the common chunk fixes this then I'm happy to do that. Otherwise it might be necessary to git bisect until we find which commit broke things or reporting to Webkit.

2017 Macbook Pro, Radeon Pro 555 GPU (I'm overdue for an upgrade :) )
After the initial post, I updated three, which may have resolved the initial error that it couldn't find the luminance function. The second error remains.  I'll try with the latest bits and see if that improves things.

Tried with the latest bits from master. Same problem on Safari.
Without path tracing using only HDRI + lights:

With path tracing:

Closing this as it seems related to something we're doing with materials. Will re-open if I can find a meaningful root cause.


add semicolon to end of inline style

Lines 21 and 26

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

This PR was closed because user's repo was deleted.


nomad: 1.4.6 -> 1.5.6

Description of changes
Using Nomad 1.5 as the "standard" Nomad version for nixos-unstable
Things done
No testing by me was done - this just changes the "default" to a package/version that was already defined (and hopefully) tested previously.

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

@fyi astro

The commit message is a bit confusing, it should probably be nomad: nomad_1_4 -> nomad_1_5

Learned something new! Thank you. Updated the commit message.


[REQ] Improvements to the Wear & Seed selection system

Would it be possible to implement the option for the user to enter the Wear and Seed they would like to put on?
Leave the one you currently have, but the fields are editable.

I noticed that the quantity system cannot be typed either, it would be interesting to include this feature where you can type it too.

Suggestion noted.

Addressed on eb044a209cada3933ca72fcc05f7a904571bc1f1


Add resources to the Resonate instance level.

Codecov Report
Attention: Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.17%. Comparing base (57af298) to head (dcad940).

Files with missing lines
Patch %
Lines

lib/resonate.ts
91.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #145      +/-   ##
==========================================
+ Coverage   81.09%   81.17%   +0.08%
==========================================
  Files          17       17
  Lines        1095     1105      +10
  Branches      290      292       +2
==========================================
+ Hits          888      897       +9
- Misses        207      208       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Failed to load template for single-posttype.twig (v1.0)

Expected behavior
In version 0.22.5 I could load a template for all custom posttypes via function.php and the following settings:
(single.php)
$post = Timber::get_post('TimberPost');
(...)
Timber::render(array('single-' . $post->post_type . '.twig', 'single.twig'), $data);

Actual behavior
With version 1.0, I get with my custom posttype unternehmen such as in this link the following error:
PHP Fatal error:  Uncaught Twig_Error_Loader: Unable to find template "single-.twig"
What version of WordPress, PHP and Timber are you using?
Wordpress 4.5.1, Timber 0.22.5 (no error), Timber 1.0 (breaks custom posttype twig loader)

Hi @minthemiddle - this is probably due to us removing some functionality to do with 'guessing' what class you want. Could you please try the following:
$post = Timber::get_post(array(), 'TimberPost');

Ok, let me test this locally.

Any luck @minthemiddle ?

Actually try, TimberPost(false, 'TimberPost') - I am not sure what you are trying to do with this code however? The first parameter should be your query, and the second parameter is already \Timber\Post by default. So this doesn't seem to be doing anything?

You could actually just remove your call completely and it will work, as $post will already be defined by wordpress in single.php
I'm going to close this as the Timber::get_post call you are doing isn't valid


Allow disabling CardField/CardForm

Is your feature request related to a problem? Please describe.
When we're sending the payment details to Stripe API and then when we wait for our API to receive webhook and return that payment has succeeded, we're blocking the submit button. In web we're setting the options={{ disabled: }} on Stripe Elements. Unfortunately there's no such option for CardField or CardForm
Describe the solution you'd like
Add disabled prop to CardField and CardForm components that blocks the inputs from being changed while the user is waiting to proceed further.
Describe alternatives you've considered
We can work this around by applying a semi-transparent View on top of the form element but it's far from good UX and may not work in all cases.

Would appreciate it too!

great ask! thanks for this, adding in https://github.com/stripe/stripe-react-native/pull/1403


Addresses DOCS-2033, correct some small typos and tidy up grammar

Description
https://jira.mesosphere.com/browse/DOCS-2033
Urgency

[ ] Blocker
[ ] High
[ x] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

Addresses DOCS-2033, also fix some further numbering typos, and tidy up grammar. Make commands tested on MacOS and Ubuntu Linux for correctness, markdown tested locally in MacDown.

Thank you for the corrections, @mattj-io ! I redid the numbering -- markdown will automatically create correct numbering if every item on the list is numbered 1., so if you add new steps later, you don't need to go through and renumber.

 But it’s because the code snippets and stuff aren’t indented. I love markdown.

I'll have a fiddle with it tomorrow and see if I can get it to render correctly.

I fixed it already.


deps: update osbuild/images to v0.1.0

We could have waited for dependabot to pick this up now that we have tagged releases, but we've had requests to get Fedora 40 in IB so I'm making this one manually again.
Also, it needed some minor fixes in the playground samples.

I think the failure might be related to services needing to be set enabled via presets instead of just enabling them during the build.

I think the failure might be related to services needing to be set enabled via presets instead of just enabling them during the build.

Might be. The same test passed in RHEL 9 and CS9. Something might be changed  in systemd, I think.

The change that caused this is that in the Fedora IoT raw image, we set ignition to perform some first-boot tasks.  The manifest diff for the Fedora 38 IoT raw image:
diff -Naur /tmp/tmp.OmotHyVE4L/a0af3a7776e060d99df0a3f16757975c9a2750be/fedora_38-aarch64-iot_raw_image-boot.json /tmp/tmp.OmotHyVE4L/PR/fedora_38-aarch64-iot_raw_image-boot.json
--- /tmp/tmp.OmotHyVE4L/a0af3a7776e060d99df0a3f16757975c9a2750be/fedora_38-aarch64-iot_raw_image-boot.json	2023-08-17 17:09:51.618501377 +0000
+++ /tmp/tmp.OmotHyVE4L/PR/fedora_38-aarch64-iot_raw_image-boot.json	2023-08-17 16:46:23.479383154 +0000
@@ -1924,7 +1924,10 @@
               },
               "kernel_opts": [
                 "modprobe.blacklist=vc4",
-                "rw"
+                "rw",
+                "coreos.no_persist_ip",
+                "ignition.platform.id=metal",
+                "$ignition_firstboot"
               ]
             }
           },
@@ -2022,6 +2025,15 @@
             ]
           },
           {
+            "type": "org.osbuild.ignition",
+            "options": {
+              "network": [
+                "systemd.firstboot=off",
+                "systemd.condition-first-boot=true"
+              ]
+            }
+          },
+          {
             "type": "org.osbuild.users",
             "options": {
               "users": {
@@ -2087,7 +2099,7 @@
             "options": {
               "root_fs_uuid": "6e4ff95f-f662-45ee-a82a-bdf44a2d0b75",
               "boot_fs_uuid": "0194fdc2-fa2f-4cc0-81d3-ff12045b73c8",
-              "kernel_opts": "modprobe.blacklist=vc4 rw",
+              "kernel_opts": "modprobe.blacklist=vc4 rw coreos.no_persist_ip ignition.platform.id=metal $ignition_firstboot",
               "uefi": {
                 "vendor": "fedora",
                 "install": true,
@@ -2101,7 +2113,8 @@
                   "console"
                 ],
                 "timeout": 1
-              }
+              },
+              "ignition": true
             },
             "mounts": [
               {

When ignition is run on first boot, it resets the machine-id and triggers all the regular first-boot tasks, which includes running (the equivalent of) systemd preset-all.
So to enable or disable units on systems where we have ignition configured, we should use presets instead of enabling/disabling the services.
The ability to write presets was added in osbuild in this PR: https://github.com/osbuild/osbuild/pull/1269

@achilleas-k What can we do in the meantime?

@achilleas-k What can we do in the meantime?

We need to fix how services are enabled / disabled on Fedora when ignition is used.


Added landline error handler

BACKEND PULL REQUEST
Related Issue
Fixes #3723
Changes Proposed

Implement an error handler for events with code Twilio 30006 (landline number).
Add a conditional in the Twilio event controller (AKA SmsCallbackController) that checks for error codes and when 30006 error is detected it calls the landline error handler.
When a phone number is detected as landline we looks for those records in the phone number table and retrieve them is they are mistakenly marked as mobile.
We then update the type for all the retrieved records and push the landline update.

Additional Information
Text coverage was also added as part of this PR.
Testing
Make Twilio to send an error event to the SimpleReport callback with landline error code 30006 and check that the phone number record type has properly been updated to LANDLINE.
Checklist for Primary Reviewer

[ ] Any large-scale changes have been deployed to test, dev, or pentest and smoke tested
[X] Changes comply with the SimpleReport Style Guide

@zdeveloper  I saw you wrote some unit test using mockito so it would be awesome if you could check on the testing :D

This looks great Johanna, thank you!
Were you able to verify that after attempting to send the text to a landline, the patient profile also silently updated the phone type as landline? (No reason to think it wouldn't, I just want to double check!)

In general it will not matter how the text message was sent to the person because the way that twilio reports these errors to SimpleReport is async. When SimpleReport asks Twilio to send a message it just "queues" it in Twilio messaging service and only receives an OK(200) acknowledging that the messages was received by Twilio and will be sent, but at this point the message is not actually sent to the end user.
Afterwards Twilio will process the requests in its messaging service and will notify SimpleReport of errors, warnings and messages being processed by communicating to the /api/pxp/callback endpoint. Is in this callback that the new logic has been implemented. At this point we detect the error code 3006 and proceed to scan the phone number table and adjust the phone type for the landline number that triggered the Twilio error.
Not sure if that answered the question. I would need more details on this patient profile scenario. Is it a specific table? or a specific flow?.

This looks great Johanna, thank you!
Were you able to verify that after attempting to send the text to a landline, the patient profile also silently updated the phone type as landline? (No reason to think it wouldn't, I just want to double check!)

In general it will not matter how the text message was sent to the person because the way that twilio reports these errors to SimpleReport is async. When SimpleReport asks Twilio to send a message it just "queues" it in Twilio messaging service and only receives an OK(200) acknowledging that the messages was received by Twilio and will be sent, but at this point the message is not actually sent to the end user.
Afterwards Twilio will process the requests in its messaging service and will notify SimpleReport of errors, warnings and messages being processed by communicating to the /api/pxp/callback endpoint. Is in this callback that the new logic has been implemented. At this point we detect the error code 3006 and proceed to scan the phone number table and adjust the phone type for the landline number that triggered the Twilio error.
Not sure if that answered the question. I would need more details on this patient profile scenario. Is it a specific table? or a specific flow?.

Ah I'm sorry, I should've been more specific! The patient profile is the same as the add/edit patient page. It's where end users can change phone numbers, phone types, and preferences.

By saving the new type in the database through the async Twilio flow, this page should automatically populate with landline the next time someone opens up the patient profile after the async change has been made. There's no reason to believe that it wouldn't do this with the code you have written, I just like to double-check.

Yeah, next time that phone number is retrieve it should display as Landline for any person