Message Bug

Issue
I'm having a issue I can't seem to figure out. Consider the following scenario. I have 2 clients. Client A and client B. Client A says "foo". Client A and client B see "foo" for 1 second then it disappears. Then Client B says "bar" and Client A and Client B see "bar" on client B but they also see "foo" on client A. The problem is we're seeing messages that we're suppose to be erased. So for e.g. if there were 20 clients connected and every client said at least one message and 1 second of time passed and all the messages disappeared, then one client decided to say something, every client would not just see their message but all the previous messages on all the other clients as well.
server.js https://github.com/valkyrienyanko/io-game-v2/blob/master/server.js
Lines of interest: line 73
client.js https://github.com/valkyrienyanko/io-game-v2/blob/master/public/scripts/client.js
Lines of interest: line 149 and 153
player.js https://github.com/valkyrienyanko/io-game-v2/blob/master/public/scripts/classes/player.js
Lines of interest: lines 29 - 38
Steps to Reproduce
Follow the setup to create at least 2 clients "A" and "B". Have clients side by side so you can see what's going on in both clients at the same time. In client A, press  to open up the chat and type "foo", press  again to send. Do the same for client B but this time say "bar". You should see the problem now.

Solved. Take a look at the commit --> https://github.com/valkyrienyanko/io-game-v2/commit/9bf8a95f4148f8879e194f27edf66f43a141fe17


Update documentation for external authentication SSO Backend to get a proper install

Environment

Python version: 3.8.5
Nautobot version: 1.02b

Steps to Reproduce

Follow steps to install SAML SSO backend in https://nautobot.readthedocs.io/en/latest/configuration/authentication/sso/
Run into several issues due to incomplete documentation of dependencies to install

Expected Behavior
Following the doc should get me to the intended next step

Observed Behavior
Following the doc led me to an error
Fix
These are the proper steps for SAML dependencies for external authentication:
Using sudo outside the venv

sudo apt install -y libxml2-dev libxmlsec1-dev libxmlsec1-openssl pkg-config

Switching to user nautobot and installing within the venv.
2. sudo -iu nautobot
3. source bin/activate
4. pip install testresources
5. pip install --upgrade pip setuptools wheel
6. pip install python3-saml
This is tested on ubuntu 20.04 with the base nautobot install using the nautobot install docs.

Thanks @syyna for the report! 👀

This is related to issue #149


Global styles don't work under 1.15

The following trivial example should produce a single red box:
specification {
  element thing
}

model {
  thing1 = thing
}

global {
    style make_red * {
      color red
    }
}

views {
  view thing1view of thing1 {
    include *

    global style make_red
  }
}

It does not. The box remains the default color (blue).
This was tested in the Playground but the same behavior is present in VSCode and the LikeC4 dev server (both v1.15.0)

Hey @hubertmis, can you take a look?
I checked in playground - indeed globals are not applied, and local styles are working.

I've reproduced the issue locally. Creating a fix. I don't know how to check if my patch fixes the playground as well.

Thanks @davydkov @hubertmis. FWIW I've validated the fix both on Playground and in VSCode.


Document the setup and compile workflow (using makefile) in the README

This is being addressed at #18


MLFlow Integration

Is your feature request related to a problem? Please describe.
Being able to utilise Mlflow (https://docs.microsoft.com/en-gb/azure/databricks/applications/mlflow/quick-start) in Databricks when using SynapseML would be helpful for storing models and metrics. I did see this feat: https://github.com/microsoft/SynapseML/pull/1277/files but wasn't sure if there is any documentation on using Mlflow with SynapseML?
Describe the solution you'd like
Documentation or examples of integrating mlflow with SynapseML

Thanks for raising this @hoemblem, adding the MLFlow lead @serena-ruan  for her comments

Thanks! Will update our website with some MLFlow examples.

That's awesome - thank you!

https://microsoft.github.io/SynapseML/docs/next/mlflow/examples/
For others looking for examples they have just been published here!


chore: run ginkgo in parallel execution mode

fixes #880
See https://onsi.github.io/ginkgo/#spec-parallelization

Looking at the failures it becomes clear that the current e2e Test suite has a few flaws that prohibit us from easily incorporating parallel execution:

The device classes are setup centrally and not per test so its impossible to decouple checks for available capacity in a parallel fashion (think of one test just succeeding while a second test is creating a lv which influences the capacity)
The setup is written in a way where namespaces are created statically and then multiple tests are run within a namespace

Ideally we need to rewrite the tests in a way where the deviceClasses are dynamically allocated per test and we have a test that runs exactly once on that deviceClass. Then we can truly parallelize the tests.
I think this may be going a bit far so I want to collect some ideas/feedback here

Looking deeper into the E2E suite, it might make sense to rewrite some of these tests from scratch and properly set them up (also not use things like kubectl for applying the resources, but the kube client)

It seems you are right. One of my colleagues told me that some tests expect multiple nodes, while others delete the nodes. Major code changes would be required to make this PR work.

I think looking at this we should decide wether to rewrite the Test suite or to scrap the Issue. I would like to see the rewrite happen but I understand if we think that thats too big of an impact.

I'd like to suggest closing this issue for now. I think we can add the e2e tests for btrfs without much slowdown if we design the tests carefully. Parallelizing the tests is attractive, but we can deprioritize it.


Unify all the implementations of HasOnlySupportedRefs()

I have been trying to coalesce all the copies of the method HasOnlySupportedRefs() but I don't know how to deal with it:

The semantics of the method are vague.  What does it mean exactly to have or to be a "supported ref"?  One comment seems to imply that it's for loads and stores.  But the predicates in the implementations of this method also seem to check other random bits (like decorations or access chains).
There are 4 different implementations of this method: LocalAccessChainConvertPass::HasOnlySupportedRefs, LocalSingleStoreElimPass::HasOnlySupportedRefs, LocalSingleBlockLoadStoreElimPass::HasOnlySupportedRefs, and, MemPass::HasOnlySupportedRefs.
The first 3 implementations seem identical.  The implementation in MemPass is slightly different.

Ideally, we would have a single implementation in Pass or MemPass.  Or perhaps somewhere else like in class CFG?  Though, I think it makes the most sense inside Pass.
@greg-lunarg, could you take a look?  Thanks.

Yes.


ci: use local version for testing

this enables proper testing of a change from PR 🦩
atm any PR is not really testing code that is changed but the code from the master...
cc: @garciasdos @ArnaudRinquin @piraka9011

@garciasdos @ArnaudRinquin @piraka9011, any chance you could approve CI run? :chipmunk:

@Borda I'm a contributor, not a member of the org :)
See instead https://github.com/orgs/sliteteam/people

See instead github.com/orgs/sliteteam/people

ok, thank you, just seems that others are busy...

Hello 👋 Sorry I got a bit too conservative with notifications. Thanks for the contribution @Borda!


Query: Process Cast to interface/object in Navigation expansion

Resolves #18087
Navigation expansion prunes Cast to implemented interface or object
Navigation expansion puts cast to non-implemented interface

cc: @Pilchie


audacity: 2.4.1 -> 2.4.2

Motivation for this change
A few other things:

use cmake instead of (the super slow) autotools
we drop the runtime linking with lame as audacity is picking it up on
its own
on linux, add a number of missing dependencies.

Cc: @lheckemann
Things done

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

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

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

UI looks a bit weird compared to version from from nixos-20.30. It's quite hard to switch devices now.

It's different sure, but what exactly is weird? Please note that gtk2 is dead and everything that supports gtk3 will be moved over.

@peterhoeg the dropdowns in the toolbar line beginning with "ALSA" seem to be truncated.

Ah, that's an upstream issue: https://github.com/audacity/audacity/issues/638

It's different sure, but what exactly is weird? Please note that gtk2 is dead and everything that supports gtk3 will be moved over.
@peterhoeg the dropdowns in the toolbar line beginning with "ALSA" seem to be truncated.

Not only truncated. Widgets and corresponding areas reacting on mouse clicks seem to be misaligned. That's not fatal, but feels very confusing.

Too many problems with the move to gtk3 (works OK here under KDE), so revert to gtk2 as recommended by upstream:
https://www.audacityteam.org/download/source/

building this and running it would cause a segfault for me:
$ nix-shell -I nixpkgs=/home/voidless/Files/nixpgks_alt/nixpkgs -p audacity --run audacity
Attempting to connect to Audacity failed...retrying
Fontconfig warning: "/etc/fonts/fonts.conf", line 86: unknown element "blank"
ALSA lib pcm_dsnoop.c:642:(snd_pcm_dsnoop_open) unable to open slave
ALSA lib pcm_dmix.c:1090:(snd_pcm_dmix_open) unable to open slave
ALSA lib pcm.c:2731:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear
ALSA lib pcm.c:2731:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe
ALSA lib pcm.c:2731:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side
ALSA lib pcm_dmix.c:1090:(snd_pcm_dmix_open) unable to open slave
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
Expression 'ret' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/alsa/pa_linux_alsa.c', line: 1736
Expression 'AlsaOpen( &alsaApi->baseHostApiRep, params, streamDir, &self->pcm )' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/alsa/pa_linux_alsa.c', line: 1904
Expression 'PaAlsaStreamComponent_Initialize( &self->capture, alsaApi, inParams, StreamDirection_In, NULL != callback )' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/alsa/pa_linux_alsa.c', line: 2171
Expression 'PaAlsaStream_Initialize( stream, alsaHostApi, inputParameters, outputParameters, sampleRate, framesPerBuffer, callback, streamFlags, userData )' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/alsa/pa_linux_alsa.c', line: 2839
Expression 'stream->playback.pcm' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/alsa/pa_linux_alsa.c', line: 4628
Expression '*idev = open( idevName, flags )' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/oss/pa_unix_oss.c', line: 832
Expression 'OpenDevices( idevName, odevName, &idev, &odev )' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/oss/pa_unix_oss.c', line: 878
Expression 'PaOssStream_Initialize( stream, inputParameters, outputParameters, streamCallback, userData, streamFlags, ossHostApi )' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/oss/pa_unix_oss.c', line: 1249
/tmp/nix-shell-13155-0/rc: line 1: 13173 Segmentation fault      (core dumped) audacity

The Program build localy. Maybe im holding it wrong.

oh nvm it destroys my users fontconfig I guess. after running audacity I have to delete ~/.cache in order to use my window system again. I think this is something which was allready discussed in another issue?

https://github.com/NixOS/nixpkgs/issues/97418

When I start it, I'm getting

Fontconfig warning: "/etc/fonts/fonts.conf", line 86: unknown element "blank"
ALSA lib pcm_dsnoop.c:642:(snd_pcm_dsnoop_open) unable to open slave
ALSA lib pcm_dmix.c:1090:(snd_pcm_dmix_open) unable to open slave
ALSA lib pcm.c:2731:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear
ALSA lib pcm.c:2731:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe
ALSA lib pcm.c:2731:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side
ALSA lib pcm_dmix.c:1090:(snd_pcm_dmix_open) unable to open slave
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
Expression 'stream->playback.pcm' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/alsa/pa_linux_alsa.c', line: 4628
Expression 'stream->playback.pcm' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/alsa/pa_linux_alsa.c', line: 4628
Expression 'stream->playback.pcm' failed in '/build/source/lib-src/portaudio-v19/src/hostapi/alsa/pa_linux_alsa.c', line: 4628
(audacity:13315): Pango-CRITICAL **: 10:09:25.200: pango_font_map_load_fontset: assertion 'fontmap != NULL' failed
(audacity:13315): GLib-GObject-CRITICAL **: 10:09:25.200: g_object_get_qdata: assertion 'G_IS_OBJECT (object)' failed
(audacity:13315): GLib-GObject-CRITICAL **: 10:09:25.200: g_object_replace_qdata: assertion 'G_IS_OBJECT (object)' failed

with the last pair of errors repeating very fast.  The UI doesn't react.  On one start it got further (shown a dialog) and segfaulted immediately.

So we're in a bit of a pickle here it seems.

upstream recommends gtk2 as there are some cosmetic issues with gtk3
upstream recommends wxgtk 3.1.x
wxgtk 3.1.x recommends gtk3

I was running audacity with gtk3 and it was fine (apart from a few cosmetic glitches) - I'm guessing you were using the default gtk2 version @vcunat ?

Yes, I just used the plain audacity attribute.

Do you mind trying this one:
(callPackage ../applications/audio/audacity { wxGTK = wxGTK31-gtk3; };)
Assuming this works, we have the following options:

keep 2.4.1 with gtk2 as that seems to work, or
update to 2.4.2 with gtk3 and accept the visual inconsistencies

In case of 1, we then just wait until upstream releases something that fully works with gtk3.

Appears to work OK for me.  (though I almost don't use audacity)

@peterhoeg did you get around to do more testing? I'll mark the PR WIP until I hear back from you.

Now we have both a gtk2 and a gtk3 variant. Still default to gtk2 until upstream gives the go-ahead.

Now we have both a gtk2 and a gtk3 variant. Still default to gtk2 until upstream gives the go-ahead.

Note that changes in pname break nix-env quite a bit due to "audacity" and "audacity-gtk2" not being considered the same package, so I think it's better to minimize them.  (Perhaps the default variant shouldn't have the suffix?)  Based on your usage, you'll either miss the update or try to have both at once (which usually results into collision).

Note that changes in pname break nix-env quite a bit due to "audacity" and "audacity-gtk2" not being considered the same package, so I think it's better to minimize them.  (Perhaps the default variant shouldn't have the suffix?)  Based on your usage, you'll either miss the update or try to have both at once (which usually results into collision).

Thank you. Better drop it then as we at some point would want people to upgrade from the gtk2 version to the gtk3 version.

Thank you. Better drop it then as we at some point would want people to upgrade from the gtk2 version to the gtk3 version.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 97370 run on x86_64-linux 1

2 packages built:

audacity (audacity-gtk2)
audacity-gtk3

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 97370 run on x86_64-linux 1

2 packages built:

audacity (audacity-gtk2)
audacity-gtk3


Move the long term seasonal to the data.py module

Move it out of the plot function and have it exist on its own
it will probably be its own function
include optional arguments to also add it to the generate_dataframes functions as one of its 'aggregates'
Clean up the plot_seasonal function (maybe remove it completely and just use plot)

When dealing with the long term seasonal you might want to have multiple dataframes for mean, min, max, median

custom df should also have an array of years as the input allowing it to skip years unlike the subset that selects everything

Just a question Fuad, For the long-term seasonal aggregation, is the only means of aggregation going to be the mean or do I provide a way to ask the user to add how they want to aggregate?

@UchechukwuUdenze  , as disscused you can make it optional for users, min, max, mean, median, sum, and quantiles. For the example plots, line plot for median/average, along with uncertainity band, min, max, and the quantiles like q1, and q3

Noted

Completed the fix in the data manipulation section

modify the title arg to be a string or a list of strings
modify the naming convention of the plot savings to include the title as well as what it currently is.
modify the plot seasonal func to use line, upper and lower as args all of which are lists of Dataframes where line is compulsory and the bounds are optional
naming convention of the plot

Include a transparency input to the function
when dealing with multiple bounds use multiples of the transparency input
modify 'line' to be a single input that can have "multiple upper and lower bounds in lists"


Fix: improve redis caching edge cases

Overview
Addresses ticket #237738
This changes redis caching from being done by the application layer, to being done on the repository instead. This way it lives in just two places and covers all contentful calls.
It also fixes two bugs in dependency management
Changes
Major

All cache methods moved into the Contentful repository class and removed from elsewhere
Updates deployment pipeline to clear the cache upon release (see historic flush redis cache workflow in actions history as tests that this works)

Minor

For whole content collections, the key is always <contentType>s
For content queried with options, the key is <contentType>_<serialisedOptions>
Amends existing dependency management to check whether there are no dependencies:

If so, store a dependency of Missing -> key

Upon cache invalidation for a particular id

Anything under Dependency:<componentId> is removed as usual
Everything under Missing is removed
This removes more than is strictly required, but storing empty content should be in the minority, and removing any empty content from the cache is less complex and more failsafe than trying to do pattern matching on the keys (though shout if you have another idea)

How to review the PR
Review the implementation and whether it will cover the edge cases we're currently aware of
Cases to test
Note: Make sure your locally running instance of plan tech is recieving the webhook messages for cache invalidation, best to put a breakpoint on it, keep editing it in contentful after trivial changes to ensure your local instance picks up the message
Fixes collections not being invalidated

Go to self assessment page and verify navigationLinks has appeared in the cache
Create a new navigation link
Verify that the navigationLinks cache key has been invalidated

Fixes not found pages stuck in the cache due to no dependencies

Try to navigate to a page that does not exist (e.g. /fake-page
verify that page is in the cache (something like page_{"fields":"slug":"fake-page"}....)
Now create fake-page in contentful
Verify that the fake-page in the cache is removed

Release requirements
The cache will need fully clearing as this changes all the keys.
Checklist

[x] Title uses Angular commit convention
[x] PR targets development branch
[x] Unit tests have been added/updated
[x] GitHub workflows/actions have been added/updated
[ ] Documentation has been updated where relevant

Should I be seeing new nav links showing in my footer without needing to flush the cache? Because I'm not currently with my local redis

Great spot, thank you, I forgot to edit the webhook processor when I changed all the casing, it shouldn't have been uppercasing the first character - fixed


Workshop with Cloud Architect.

The 3 and 4 des. 2018. A cloud architect will visit me at my office to discuss the architecture of the first prof of concept.

Great sessions and discussions, thanks for that @gjersvik . As we agreed, I started with making diagrams to capture key ideas and areas. I will add it to designdocs folder.

Yes was a very useful workshop thanks @pazdedav


eclipse-4.5 importing err

How do I fix it ?

Right-click on plugin no covered error...Quick fix.  Then choose ignore.

ignore...
OK . _

Did you figure it out?

@brettwooldridge  Yes. Do it by ignoring. I read the code except package the project.


Infra: Refactor TwoDomainLoadBalancer class

Currently TwoDomainLoadBalancer test class has method  testDeployAppAndInstallIngressControllers()  which install all three lb ingress  and  deploy the clusterView application.
When one of the loadbalancers fails to install, all the test fail, as each test uses the cluster view application to verify inter-server communication.  If ngnix installation fails, traefik and apache tests also fails.
So we need to  de associate each test methods to be independent of all type of  lb igrress installation.
Jenkin Run(s)
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/7137/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/71334

I'd not recommend removing the ordering because if we do so, we need to move installation of three LBers to BeforeAll. Then we go back to where we have this change -- when one installation of LBer failed, entire tests fail

You are right that all LBers related tests have to be done after order(1) and order(2). However, in order to remove the ordering, we have to install all three LBers before all tests (except 1 & 2) start, that strategy bring back the issue of one LBer failed to install, no test will run. It doesn't matter whether 5 depends on 4, I just didn't touch the original ordering. If you don't like it, I can re-order to group same LBer test together.

I changed NGINX namespace to force installing NGINX fail,
nginxHelmParams = installAndVerifyNginx(nginxNamespace + "123", 0, 0);
The results are expected, only NIGNX related tests failed all other tests passed
[INFO]
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR]   ItTwoDomainsLoadBalancers.testNginxTLSPathRoutingAdminServer:356->installIngressController:1928 [Test NGINX installation succeeds] NGINX installation is failed
[ERROR] Errors:
[ERROR]   ItTwoDomainsLoadBalancers.testNginxHttpHostRoutingAcrossDomains:513->getNginxLbNodePort:1551 NullPointer
[ERROR]   ItTwoDomainsLoadBalancers.testNginxHttpsHostRoutingAcrossDomains:531->getNginxLbNodePort:1551 NullPointer
[ERROR]   ItTwoDomainsLoadBalancers.testNginxPathRoutingAcrossDomains:547->getNginxLbNodePort:1551 NullPointer
[ERROR]   ItTwoDomainsLoadBalancers.testNginxTLSPathRoutingAcrossDomains:382->getNginxLbNodePort:1551 NullPointer
[INFO]
[ERROR] Tests run: 15, Failures: 1, Errors: 4, Skipped: 0
[INFO]


Do not opt users out of the insiders program if they have a stable version installed

For #14090

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #14091 into main will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main   #14091      +/-   ##
==========================================
- Coverage   59.77%   59.73%   -0.04%
==========================================
  Files         702      702
  Lines       38868    38859       -9
  Branches     5618     5615       -3
==========================================
- Hits        23232    23214      -18
- Misses      14426    14432       +6
- Partials     1210     1213       +3

Impacted Files
Coverage Δ

...t/common/insidersBuild/insidersExtensionService.ts
96.77% <100.00%> (-0.41%)
:arrow_down:

...very/locators/services/virtualenvwrapperLocator.ts
76.92% <0.00%> (-15.39%)
:arrow_down:

src/client/common/utils/platform.ts
60.00% <0.00%> (-12.00%)
:arrow_down:

.../pythonEnvironments/common/externalDependencies.ts
66.66% <0.00%> (-4.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

Continue to review full report at Codecov.

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


Can jshint warn when using deprecated options?

For both cli and local package usage.
Currently users hardly notice if there is any option deprecated and jshint does not check against those options. It confuses users and if there is any problem with the deprecated option in the code, users might not know. Say user enables smart tabs but jshint does not throw any error and there might be mixed tabs and spaces.

+1

duplicate of #1346


Update StringConversions for Windows shell escaping

We don't need to escape backslashes on Windows, and multiple double-quotes seem to be standard (https://stackoverflow.com/a/15262019) for escaping other characters. Note that single-quote escaping does not work on Windows shells.

@swift-ci please test

@compnerd could you please help review this since you worked on this lately


Update xmake.lua

I've updated the error message and force-pushed. Please check if it makes sense. I've also dropped the -g commit.

thanks!


Allow dropdowns to be fixed

Currently the dropdown panel is positioned with absolute for x-float. However when using a dropdown inside of a dropdown, this causes the nested dropdown panel to get positioned based on a parent container instead of the viewport. This causes the position to be off in certain instances:
Example 1: The nested panel flips up even though there is viewport space below, becuase the table "parent" is smaller than the panel.

This is even a bigger issue when the panel is taller than the viewport as the navbar covers it up.

Example 2: The nested panel shifts to the left because it's tied with the table. This pushes the panel offscreen causing scrolling.

This PR allows a dropdown to be positioned with fixed instead of absolute. The default continues to be absolute to avoid breaking changes. This is one of the recommended strategies by floating-ui: https://floating-ui.com/docs/misc#clipping
With fixed the above issues are resolved:

[ ] Changes have been thoroughly tested to not break existing functionality.
[ ] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Thanks for bringing this up, @archilex.
Isn't this what teleport is for though?
Did the issues you outlined occur with or without teleport on the dropdown?

@zepfietje Teleport works perfectly and fixes both of the above problems. Didn't know about that. Thanks!

Cool!


If the laptop is powered off in one mode, then booted on another mode, the brithness level remains in the incorrect level.

Fixed with 884e1ac7050a680cdce992d4530bbefb45629ae5.


feat: @effect/vitest custom testers

The MVP of https://github.com/Effect-TS/effect/pull/2629

I wonder if it would be better to add a tester that checks for Equal.isEqual and uses Equal.equals if true.
I'm exploring this option, the problem is the fallback on strict equality for objects not implementing Equal, e.g. chunk or list containing arbitrary objects.

Wondering if we can make equal act structurally for plain objects with a regional flag

While making the equals a little bit smarter by moving closer to the comparison by value absolutely feels like a good move, IMO we should still not break the equality logic of the vitest. Vitest (and I assume generally any testing lib) can have its own comparison logic and most importantly a mechanism for injecting a custom testers (users of Vitest can introduce their own equality for specific cases), therefore the best way to approach this seems to be to introduce an equality tester checking effect's data types "structurally" and than giving the control back to the equality logic of the testing lib for other nested values.


Feature/issue46

Adding in the option for a 12 inch HPD photodector. This is necessary to begin the design studies for an alternative configuration in the HK detector. Except for the name, radius, and expose height, the 12 inch HPD is identical to the entry for the 20 inch HPD. The expose height is assumed to be the same as the 12 inch PMT.

I didn't create a detector configuration entry for this HPD yet since I assume whoever is doing the design study will want to do this on their own with the appropriate radius, length, etc.

This looks fine to me.
As Erin said in her comment there is currently no detector defined that uses these tubes, but they are there for studies if you need them.


chore: bdd-spring-1575591930 to 0.0.1

chore: Promote bdd-spring-1575591930 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


go/oasis-test-runner: Try using abstract sockets

Not much gain in doing this for now, and this was more a proof of concept than anything.


Add Process Id to logfiles name

By adding the process id we make sure the log file is going to be unique per process.
If two or more commands are executed at the same second the log results would be intermixed.
closes #1075

[ ] Did you write/update appropriate tests
[x] Release notes updated (if appropriate)
[ ] Appropriate logging output
[ ] Issue linked
[ ] Docs updated (or issue created)
[ ] New package licenses are added to ThirdPartyNotices.txt (if applicable)

Closing in favour of #1078 (I won't have to do this next time, as @dylan-smith taught me how to run E2E tests for non-maintainer PRs!)


KYC Match - API Flow

Problem description
We need agreed KPI Match Flows for our first version API documentation.
I woud like to propose it as attached.   It is very straightforward, highlevel and generic flow, which could apply to any use cases for KYC Match.
Any comments are welcome.
Note: It was created by PowerPoint and captured as a picture, because I am not familiar with tools for drawing flows.  Please let us know if you are familiar with tools.

Many thanks,
Toshi

Thanks @ToshiWakayama-KDDI for your work. I usually use DrawIO for UML flows.

I've tried to put your proposition in a drawio schema. To be reviewed
KYC Fill-in flows

KYC Match flows

Hi @GillesInnov35, Thank you for your drawing.
The application provisioning sequence and the 3 legged authentication sequence are common to other APIs. Therefore, for the purpose of simplicity, why not omit these and draw only the KYC Fill-in and the KYC Match sequences themselves?
BR
Tet, KDDI

hi @tcchiba , thanks for your comment. Yes you're right. As Toshi mentionned the authentication flows in the previous schema he has proposed I thought also it was useful to show which credentials were used (client id/client secret) and the user consent.
I'll remove the authentication and authorization sequence if decision is that it is not useful.
BR
Gilles

@tcchiba, in fact the proposition made by @ToshiWakayama-KDDI was good and simple. We might keep this schema.
BR
Gilles

Closed as agreed.


Relectively invoking composables pt 2 - KSP processor

Relectively invoking composables pt 2 - KSP processor
Adjust
More rigid handling
Non-nullable
Fix
Adjust for ignoring previews

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#126
#125
#124  👈
#123
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rbro112 and the rest of your teammates on  Graphite


Kde sound applet can't find devices

Describe the bug
The kde sound applet no longer has any controls in it
Steps To Reproduce
Steps to reproduce the behavior:
I'm not really sure but I updated to the latest unstable release

update from cfd6b5fc90b15709b780a5a1619695a88505a176 to 5672bc9dbf9d88246ddab5ac454e82318d094bb8
i.e https://github.com/BCNelson/nix-config/commit/b20c768d086e3691101233ed24aef8b6b53023af

Expected behavior
KDE applet has no detected devices
Screenshots

Additional context
Add any other context about the problem here.
Notify maintainers
@K900 It looks like you made changes to pipewire and kde in the last few days #304310 and #304551
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
bcnelson@sierra-2 ~> nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.27, NixOS, 24.05 (Uakari), 24.05.20240416.5672bc9`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(bcnelson): `""`
 - channels(root): `"home-manager-23.05.tar.gz, nixos-23.05, unstable"`
 - nixpkgs: `/nix/store/3kwj19dbdfxnjbcns4hw307ylhz3wgrm-source`

Add a :+1: reaction to issues you find important.

Nevermind it was an issue on my config


Fix #552 - Allow stopClass for PropertyUtilsBean

Fix https://issues.apache.org/jira/projects/BEANUTILS/issues/BEANUTILS-552

Add new method PropertyUtilsBean#getPropertyDescriptors(beanClass, stopClass)
As per https://issues.apache.org/jira/projects/BEANUTILS/issues/BEANUTILS-520, it seems class property should be excluded by default. That's why there are still a few failed TUs (waiting for feedbacks before I do more)
Add ctor for PropertyUtilsBean

@Rapster I have this fork of Commons BeanUtils2 so I could release on Maven Central since there has not been a release in 5 years if you want the submit the patch on my fork and I can push to Maven Central.
https://github.com/melloware/commons-beanutils2

So what's the future of this repo then? I can see recent changes on this, but indeed no release so far...

So what's the future of this repo then? I can see recent changes on this, but indeed no release so far...

The future is the same as for any Apache Commons component: We are volunteering our time and that is a limited resource. Someone needs to take the time and work on it. Since the next release is a major release, extra care must be taken that the API is what we really want it to be as it will be set in stone.

Let me rephrase it as it's not what I meant, as a contributor myself I can only understand you're doing this on your free time, and appreciate very much the quality of this work.
As I just submitted a PR, and @melloware mentionned his own fork but I see you've been comitting things recently, understand that I was a bit confused...
I understand now a major release is on his way, and this repo is the one to look at. Look forward to see the result.
Keep up the good work 😉

@Rapster the last release of BeanUtils was 2016.  There was no release in sight and @garydgregory is extremely busy supporting all the Apache libs.  So there is no timeframe on a release.  My company got tired of waiting since it allowed me to remove Commons Collections 3 dependency and add 5 years worth of enhancments that is why I released it to Maven Central.


Autosave does not save as expected when losing network connectivity and then regaining it

Describe the bug
The editor does not gracefully handle saving data when resuming network connectivity.
To Reproduce
Steps to reproduce the behavior:

Open an existing game from Your Games in the editor.
Add one space, forcing an auto-save - observe save be successful
Disconnect your network connection (turn off wifi, for instance)
Add one space.  Observe a failure to save.
Resume your network connection (turn on wifi).

Expected behavior
The new space should be successfully saved shortly after resuming network connectivity.
Actual behavior
"Error saving" popup repeatedly flashes and never saves successfully (even after a full minute or two from what I've observed - until you make another change).
Screenshots
https://github.com/hackclub/sprig/assets/43505167/84d29503-fd75-4b9f-add6-979d81d1a91b

fixed since #1255


Long value in Activity drop down extends past the field itself

Create an account
Login as the newly created account
Select Enron in the Volunteer for Disaster drop down
Expand Activity field

Expected Result: Values are displayed properly
Actual Result: The value "&&&&&&&&&&&&&&&&&&&&" extends past the Activity field (looks terrible)

tested in IE, chrome, and firefox. Didn't see it extend past field.  looks okay.

Also verified. Closing.


Add a method to transform Time instance to a different time zone while preserving wall clock

When transforming a Time instance to a different time zone (Time::Location), there can be two different outcomes:

The new Time instance refers to the same instance, just expressed in a different time zone. This means the wall clock changes by the difference between time zone offsets.
The new Time instance shows the same wall clock, but in a different time zone. This means the instant changes by the offset difference.

The first variant is implemented as Time#in(location : Location). That's probably the most common usage, to express absolute instances in a local time zone.
The second is not so common but still there are valid use cases. This is useful to reference floating time instances in a local time zone. For example, the next turn of year is at 2019-01-01 00:00:00 but the exact instance depends on the time zone.
It is currently possible to construct such a Time instance but it's not easily possible to convert an existing instance. There are workarounds, but there should be a more direct way to do this:
local_time = Time.new(time.year, time.month, time.day, time.hour, time.minute, time.second, nanosecond: time.nanosecond, location: location)

I'm still missing good ideas for a method name, so this is just using a placeholder to show how it should work:
local_time = time.to_same_wall_clock(location)

Java's Datetime API has the method names withZoneSameLocal and withZoneSameInstant which are good descriptive but very bulky.

A useful application of this is described in https://github.com/crystal-lang/crystal/issues/6522#issuecomment-413462971


Ticket/03 radio button

Hi @libertadpozos!
I have created a RadioButton component which has its own radio button and its colors and this component I use in RadioButtonsList to generate the 3 options.
I also changed the color of one of the squares in the css that was wrongly placed.

Hey Irene,
I have checked your PR, there is a conflict that must be taken care of. Otherwise, I don't have more comments. Great work.

Hey Irene,I have checked your PR, there is a conflict that must be taken care of. Otherwise, I don't have more comments. Great work.

Done! I've already resolved the conflicts :)


chore: update firebase dependencies

This PR updates all Firebase dependencies to the latest versions.

This is deployed and is working fine. I'm going to merge this. 🚀


update changes for 4.5.0

Update with the following changes:

add BCM KNET support
Fix AS4630-54PE 1G ports on PHY2/3
Fix Makefile for AS7726-32X

Signed-off-by: Jonas Gorski<EMAIL_ADDRESS>

lgtm


[move-prover] Finishing pragma implementation and introducing pragma verify

This finishes the implementation of pragmas and pipes them through from the parser to the prover's environment.
Also introduces a pragma property verify as well a command line flag --verify to control the scope of verification:

on the command line one can specifiy which funtions are verified by default using --verify=public|all|none. public means only public functions are verified standalone (the _verify boogie function entry is generated), all public and private, and none none. public is the default (NOTE that is a breaking change in the previous behavior of the prover, which verified all functions).
the command line behavior can be overriden in spec blocks using spec fun foo { pragma verify=true'; .. }. The pragma can also be provided on module level, spec module { pragma verify=true; .. }, in which case it provides the default for all functions in the module which do not have their own pragma.

Motivation
Finer control over verification.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Added a test.
Related PRs
NA
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

@bors-libra r=DavidLDill

:pushpin: Commit e7bc4c5 has been approved by DavidLDill

:hourglass: Testing commit e7bc4c514f2b14dbf48bbbedab53bb39a6abcb54 with merge 72423d18120e2eb1bb01a4eaed90c7ad7001ee41...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit e7bc4c514f2b14dbf48bbbedab53bb39a6abcb54 with merge 5316450d60afc0d3ee1913d31f26b9eff8d7ecc5...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra r=DavidLDill

:pushpin: Commit 9dd77a6 has been approved by DavidLDill

:hourglass: Testing commit 9dd77a68bdf64625901cbbd9ad0c24963d2bd4e0 with merge c7bd7fc82233bdaecb93a7138e0520d2481b44ab...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: DavidLDill
Pushing c7bd7fc82233bdaecb93a7138e0520d2481b44ab to master...


GLIBC_2.34

I'm getting this error
Failed (/lib/i386-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by plugins/textdraw-streamer.so))

What is the Linux version you are using?

Qual é a versão do Linux que você está usando?

To be honest, I don't know, my computer is Windows, the host I use is Linux.  How can I see it?

Type hostnamectl in the terminal.

Type hostnamectl in the terminal.

Translation: 'hostnamectl' is not recognized as an internal command
or external, an operable program or a batch file.

You need to use this command on Linux.  If you are using Windows, use textdraw-streamer.dll.

Você precisa usar este comando no Linux. Se você estiver usando Windows, use textdraw-streamer.dll

My SA:MP hosting is Linux, there is no way to use Windows Plugins.

My SA:MP hosting is Linux, there is no way to use Windows Plugins.

libc.so.6

I found a similar problem with another plugin: https://github.com/bmisiak/samp-precise-timers/issues/9

My SA:MP hosting is Linux, there is no way to use Windows Plugins.

I found a similar problem with another plugin: https://github.com/bmisiak/samp-precise-timers/issues/9

Are you sure you are using the hot fix version?

Yes, I sent you a friend request on Discord, if you want I can show you better there.
Name: Calasans
User: walkerxinho7

The problem is related to the hosting company you use. Because it was compiled with GLIBC 2.28. The error you mentioned indicates version 2.34.

I understand. I will test with other hosting companies. I currently use TextDraw-Streamer version 2.0.2.

okay


fix: expand description button style

What are the relevant tickets?
Related to https://github.com/mitodl/ocw-hugo-themes/pull/1371 and https://github.com/mitodl/hq/issues/3858
Description (What does it do?)
In https://github.com/mitodl/ocw-hugo-themes/pull/1371 we fixed a markdown rendering issue. The fix changes the HTML layout of the collapsed-description div.
We used to have this layout

  text text text <button></button>

The fix changes it to

text text text

  <button></button>

This introduces a line break that changes how the "See more" button is placed.

What it should look like

What it looks like on RC right now

Screenshots (if appropriate):

How can this be tested?

Grab contents of this course.
Navigate to your local ocw-hugo-themes setup.
Runyarn start course sts.050-spring-2016

Open http://localhost:3000/
Observe the homepage.
Expect to see the description (collapses and expanded) rendered correctly. Please see the screenshot for reference.

Is there a way to add a test to this? Is a visual test necessary (like Percy) or can this be done with playwright?

Is there a way to add a test to this? Is a visual test necessary (like Percy) or can this be done with playwright?

Yes, Playwright has visual comparison tests that we can use. I'll write a quick test.
I think Percy would be better but I don't have any experience using it. @pdpinch, do we have Percy set up for this project?

@ibrahimjaved12, I'm going to try adding some visual tests. I'll let you know when it's ready for review again.

@ibrahimjaved12, @pdpinch, I experimented with adding Playwright visual tests. We require a more thorough plan to use visual tests effectively.
To create reliable visual tests, we need consistent environments. Playwright tests are run on three environments local, GitHub Actions, and Concourse, and we always use the latest browsers for these tests. Under these conditions, visual tests are not robust and tend to fail in one environment or the other.
@pdpinch, should I write an issue to set up visual testing for this project?
I read the Percy starter docs and I think Percy would be easier to integrate and maintain than playwright visual tests.

For now, I have added a test to validate CSS for this change (though this is not as reliable as a visual test would be).


boards: add separate u54/u74 targets for hifive_{unleashed,unmatched} boards

This PR adds support for U54 and U74 targets for hifive_unleashed and hifive_unmatched, respectively. The following changes are applied:

hifive_unmatched: switched from ram0 to l2lim in zephyr, sram in board DTS
added a Kconfig option that informs whether the FPU state tracking is imprecise, which for FU540 and FU740 it is (FPU state is reported dirty where, theoretically, it's not)
created the following new targets: hifive_unleashed//e51, hifive_unleashed//u54, hifive_unmatched//s7, hifive_unmatched//u74

Hey @kartben, could you please take a look at this PR? Thanks :)

please update the documentation pages of the boards, they are essentially containing non-working build instructions now :)
❯  west build -b hifive_unleashed samples/hello_world -p auto
-- west build: generating a build system
Loading Zephyr default modules (Zephyr base).
-- Application: C:/Users/kartben/zephyrproject/zephyr/samples/hello_world
-- CMake version: 3.27.0-rc4
-- Found Python3: C:/Python312/python.exe (found suitable version "3.12.3", minimum required is "3.10") found components: Interpreter
-- Cache files will be written to: C:/Users/kartben/zephyrproject/zephyr/.cache
-- Zephyr version: 4.0.0-rc3 (C:/Users/kartben/zephyrproject/zephyr)
-- Found west (found suitable version "1.2.0", minimum required is "0.14.0")
CMake Error at C:/Users/kartben/zephyrproject/zephyr/cmake/modules/boards.cmake:304 (message):
  Board qualifiers `/fu540` for board `hifive_unleashed` not found.  Please
  specify a valid board target.

  Valid board targets for hifive_unleashed are:

  hifive_unleashed/fu540/e51

  hifive_unleashed/fu540/u54

Ooops, thanks for spotting that. Should be fixed now.


Can anyone report successful parameters for A2C playing Pong or another Atari game?

I'm trying to run A2C, but the default parameters, including nsteps=5, aren't letting me train Pong. What parameters to the model can I pass that would let me play a simple Atari game like Pong?

@slerman12  Show the output of your training log, the command you ran, and how long you trained for. Hard to say without seeing these details.

I used this tutorial code based on Baselines: https://github.com/simoninithomas/Deep_reinforcement_learning_Course/tree/master/A2C with Sonic the Hedgehog
agent.py runs the code. I just changed the sonic environments to 8 copies of Pong. His model.py also divided up the training update into mini batches since his Runner was collecting a lot more data for Sonic than could be processed all at once; I changed that back to 1 minibatch since these 8 Pong environments don't demand quite so much data. I also brought his nsteps down to different values, including 5 which is the default in the Baselines code here. No matter what, the model did not seem able to learn to play Pong. The explained variance went up, but the average reward returned per training episode continued to be negative after two days of training.

Hello,
If you are looking for training script + working hyperparameters, I recommend you taking a look at the rl zoo (there are also pre-trained agent there).
Disclaimer: this is using stable-baselines, a fork of OpenAI Baselines, but the underlying implementation is the same.


Persist raw model data

I am not so sure about the change of behavior for compare() which now always returns a cc. I somehow appreciate the consistency, but to me, it was quite meaningful before: if compare one obs then I get one comparer, if I compare multiple obss then I get a collection. Makes sense. Let's discuss tomorrow :-)

The main reason I changed compare to return a ComparerCollection, is that multiple return types causes editors like VS Code to misinterpret the returned object, which you can expect writing a script with no active session, but also in a notebook when it should be able to inspect the actual object☹. Since both the Comparer and the ComparerCollection has a save method, VS Code presents the wrong auto-completion and documentation, which is not helpful.
The compare function should be the main entry-point for people starting out with the library and multiple return types is confusing.

But related to save/load functionality, I guess my only comment is: is "raw_" a special enough name that we will not get in trouble if people accidentally named their station "raw_stn1" or similar? I would suggest "raw" or something else that starts with an underscore to indicate that is an internal structure.

I'll add an underscore at the beginning, but we could also use NetCDF groups and split the raw data into separate datasets.
The structure of this file is a fairly important decision, if we want to be able to support loading files across versions of modelskill.

I guess I also agree with @rywm-dhi on the overwrite functionality...
It seems like it could be useful, but after inspecting some other methods writing files in Python, checking if the file exists behavior seem to be very unusual, so I guess it is better to leave this up to the end user, they can add a path.exists(), since the default would be to overwrite anyway.


Coordinate System Questions

Hello,
I have been going through the web sdk documentation and had a question about the coordinate system  in Resonance. The documentation says that the room dimensions are 'width, height, and depth' and then provides the ability to set the source and listener positions in (x,y,z). However, I can't seem to find any reference (written or visual) to the coordinate system that Resonance is using.
From tinkering it appears to be a Right-handed coordinate system with x+ pointing to in the right direction, y+ pointing in the up direction, and z+ pointing towards the user. This would mean that x would map to moving along the width of the room, y in the height, and z in depth. Is this the case?

Hi Darren,
that is correct!!
Cheers,
Drew
On Thu, Jun 14, 2018 at 5:08 PM DarrenG<EMAIL_ADDRESS>wrote:

Hello,
I have been going through the web sdk documentation and had a question
about the coordinate system in Resonance. The documentation says that the
room dimensions are 'width, height, and depth' and then provides the
ability to set the source and listener positions in (x,y,z). However, I
can't seem to find any reference (written or visual) to the coordinate
system that Resonance is using.
From tinkering it appears to be a Right-handed coordinate system with x+
pointing to in the right direction, y+ pointing in the up direction, and z+
pointing towards the user. This would mean that x would map to moving along
the width of the room, y in the height, and z in depth. Is this the case?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/resonance-audio/resonance-audio-web-sdk/issues/13,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGDsNoE7GHLP-9LH35-33qv9mm1bqDBGks5t8vrvgaJpZM4Uo1iL
.

Awesome thank you for clearing that up. One more minor question, when calling setListenerPosition(), does this effectively place the listener's ears at the location specified, or set the bottom of the head (with some specified height) to a location. This would mean I would need to account for this height.  I am 90% sure the ears are placed at this level, but just want to confirm.

Hello, to add something to this question.
How does sound source spread work in the webaudio version of Resonance? I see docs for it for Unity and Unreal but nothing for the web, is this not supported?
I just want to this source.sourceWidth=2 and have a sound that is spread by 2 meters instead of the default...
Thank you!


chore: Improve maintainability of at_app, add dependabot for pub

- What I did

Converted each app template into an individual dart project

Renamed each template.yaml to pubspec.yaml (6 renames with added headers to make them valid pubspec.yamls)
29 unchanged renames by moving templates/lib/ to templates/

Replaced the auto_clean_cache action - I don't believe it was working as intended

Flutter cache is now done by commit SHA

Added pub to dependabot to help maintain the templates

- How I did it
Local changes
- How to verify it
See the build workflow
- Description for the changelog
chore: Improve maintainability of at_app, add dependabot for pub

The new repo permissions have caused checks to fail.

awaiting #149 to test the new build


Feature request: beforeLoopFix event

Great swiper!
A beforeLoopFix event would be nice.

Nice, thx!


Do not use docker to run native tests, fixes #332

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/334/


hiding drag icon

Is there any way by which I can hide the drag icon as I am using Non-dismissible example.

<Drawer.Content className="[&>div:first-child]:!hidden">
this worked for me.

Providing a proper proper for it will be better


fix build with musl C library

With musl C library, we get following build error:
/home/buildroot/autobuild/run/instance-3/output/build/let-me-create-v1.5.2/src/core/spi.c: In function 'spi_transfer':
/home/buildroot/autobuild/run/instance-3/output/build/let-me-create-v1.5.2/src/core/spi.c:170:19: error: '_IOC_SIZEBITS' undeclared (first use in this function)
if (ioctl(fd, SPI_IOC_MESSAGE(1), &tr) < 0) {
^
/home/buildroot/autobuild/run/instance-3/output/build/let-me-create-v1.5.2/src/core/spi.c:170:19: note: each undeclared identifier is reported only once for each function it appears in
Include <linux/ioctl.h> for musl C library compatibility.
This build issue is detected by Buildroot autobuilder:
http://autobuild.buildroot.net/results/af9/af946fa6fe05ee265e4ac97742b15afeb0cea1ab/
Signed-off-by: Rahul Bedarkar<EMAIL_ADDRESS>

Thanks rahul !
Unfortunately, I cannot merge your PR. I will try to find someone.


TPM module emulation

Will be nice to have this feature to use windows 11 without limitations.

Perhaps this swtpm can be incorporated
https://github.com/stefanberger/swtpm

TPM seems to be already in QEMU:
In case an Arm virt machine is emulated, use the following command line:
qemu-system-aarch64 -machine virt,gic-version=3,accel=kvm
-cpu host -m 4G
-nographic -no-acpi
-chardev socket,id=chrtpm,path=/tmp/mytpm1/swtpm-sock
-tpmdev emulator,id=tpm0,chardev=chrtpm
-device tpm-tis-device,tpmdev=tpm0
-device virtio-blk-pci,drive=drv0
-drive format=qcow2,file=hda.qcow2,if=none,id=drv0
-drive if=pflash,format=raw,file=flash0.img,readonly=on
-drive if=pflash,format=raw,file=flash1.img

How apply this to utm vm machine?

This is the latest as far as I can tell
https://github.com/stefanberger/swtpm/issues/493

Anybody know what I'm supposed to do about this?
qemu-aarch64-softmmu: -chardev socket,id=chrtpm,path=/tmp/win11tpm/swtpm-sock: Failed to connect to '/tmp/win11tpm/swtpm-sock': Operation not permitted

FYI, this may be useful to you.
brew tap spikespaz/jacob
brew install swtpm

how to use this?

24 dec. 2021 kl. 07:34 skrev Jacob Birkett @.***>:
Anybody know what I'm supposed to do about this?
qemu-aarch64-softmmu: -chardev socket,id=chrtpm,path=/tmp/win11tpm/swtpm-sock: Failed to connect to '/tmp/win11tpm/swtpm-sock': Operation not permitted
FYI, this may be useful to you.
brew tap spikespaz/jacob
brew install swtpm
—
Reply to this email directly, view it on GitHub https://github.com/utmapp/UTM/issues/3082#issuecomment-1000680962, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGEUP5IQP26EKTG3JKOX3W3USQH7RANCNFSM5DOP7ZIA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.

I have the same issue (Failed to connect to). Got swtpm running, and socket present. Since UTM runs in a sandbox, I even tried to create the socket inside the sandbox (~/Library/Containers/com.utmapp.UTM/Data/Documents/...), with same result.
I can initiate connection to the socket from my terminal though, and swtpm replies.

I can add some more details. I've started swtpm with:
/opt/homebrew/bin/swtpm socket --tpmstate dir=/Users/myuser/Documents/UTM-VMs/Windows11.utm/Images --ctrl type=unixio,path=/Users/myuser/Documents/UTM-VMs/Windows11.utm/Images/swtpm-sock --log file=/Users/myuser/swtpm.log,level=20 --tpm2
(I moved the Windows 11 VM to the Documents folder, and I am using a space-free name)
When starting QEMU with the switches:
-chardev socket,id=chrtpm,path=/Users/myuser/Documents/UTM-VMs/Windows11.utm/Images/swtpm-sock
-tpmdev emulator,id=tpm0,chardev=chrtpm
-device tpm-tis,tpmdev=tpm0
...I get the same error as @dxlr8r (I suppose):

Neither logging on the swtpm nor on the qemu side yields any result, so I investigated a bit more in the system logs (Console.app with admin privileges), and I found out that...
Sandbox: QEMULauncher(58414) deny(1) network-outbound /Users/myuser/Documents/UTM-VMs/Windows11.utm/Images/swtpm-sock
...and this probably causes...
QEMUHelper: Could not open() the item: [102: Operation not supported on socket]
UTM: nw_socket_handle_socket_event <private> Socket SO_ERROR [61: Connection refused]
I hope this can be of use to solve the issue, let me know if you want me to attempt something else.

Hi All,
I have followed TPM 2.0 module installation and configuration with QEmu according to this site
Software TPM Emulator For QEMU.
I setup swtpm startup procedure in terminal
Createf folder inside my UTM image
mkdir /Users/dariusz/Library/Containers/com.utmapp.UTM/Data/Documents/Windows_11_ARM.utm/mytpm0

Create TPM start file (with content below) and started
touch tpm
nano tpm
chmod +x tpm
./tpm

Content of tpm file:
swtpm socket --tpm2 --tpmstate dir=/Users/dariusz/Library/Containers/com.utmapp.UTM/Data/Documents/Windows_11_ARM.utm/mytpm0 --ctrl type=unixio,path=/Users/dariusz/Library/Containers/com.utmapp.UTM/Data/Documents/Windows_11_ARM.utm/mytpm0/swtpm-sock --log file=/Users/dariusz/tpm.log,level=20

then added the following in UTM/QEmu:
-chardev
socket,id=chrtpm,path=/Users/dariusz/Library/Containers/com.utmapp.UTM/Data/Documents/Windows_11_ARM.utm/mytpm0/swtpm-sock
-tpmdev
emulator,id=tpm0,chardev=chrtpm
-device
tpm-tis,tpmdev=tpm0

I need to rename my system in UTM to be without spaces, as in another case I got information about operations not permitted as posted above.

When running Windows in UTM it starts spinning on the Gray screen. This is expected and as advised that will take a minute to generate keys and load Windows further. However in my instance after a short while, I received An error has occurred instead.
There is some progress but not sure where to go from there. The terminal did not show any logs and the specified log file do not show anything.

Change the path for swtpm-sock to "com.utmapp.QEMUHelper" instead and create the mytpm0 folder and the tpm file(with the new path) there:
/Users//Library/Containers/com.utmapp.QEMUHelper/Data/Documents/mytpm0
the QEMULauncher executes the socket-sock file inside its own container and no more sandbox network-outbound error; now the VM boots OK and Windows11 recognizes the TPM 2.0 (you can verify it under the device manager)

However,  the Windows driver or some missing config could not start the device...yet

Whats your setting in QEMU?
as mine
-chardev
socket,id=chrtpm,path=/Users/dariusz/Library/Containers/com.utmapp.UTM/Data/Documents/Windows_11_ARM.utm/mytpm0/swtpm-sock
-tpmdev
emulator,id=tpm0,chardev=chrtpm
-device
tpm-tis,tpmdev=tpm0

with
swtpm socket --tpm2 --tpmstate dir=/Users/dariusz/Library/Containers/com.utmapp.QEMUHelper/Data/Documents/mytpm0 --ctrl type=unixio,path=/Users/dariusz/Library/Containers/com.utmapp.QEMUHelper/Data/Documents/mytpm0/swtpm-sock --log file=/Users/dariusz/tpm.log,level=5

resulting an Error message when starting UTM machine

qemu-aarch64-softmmu: -device tpm-tis,tpmdev=tpm0: 'tpm-tis' is not a valid device model name

I can confirm that @idarek and @gvaldezd parameters are working on my setup, too. I'm now stuck at the same point you are (TPM detected by Win11, but not started).

Ok, doing some more tests:
If the --tpm2 option is removed (TPM 1.2 by default is used) from the swtpm setup, the tpm.log files shows a lot of activity (like everything is running fine before Windows boots), then under Device manager, listed under Memory Connections the TPM platform is OK but is not shown under Security Devices, then if you run CMD tmp.msc shows not valid TPM platform.

tpm_1_2.log
If we add the option --tpm2, the TPM2.0 device is not working (previous error) and the log shows less activity.

tpm_2_0.log
It seems there's something wrong in how Windows11Arm64 is identifying the TPM2.0 platform and the driver identified.

Would be great if there where an "out of the box" solution integrated into UTM or  or a summarized "step by step" solution for Mac users. At the moment I don't really know how to realize what has been written, because it's quite messy.

Hi, just to let you know, here's the TPM emulation using VW Fusion Tech Preview for OS (with M1), is working correctly under device manager, using the same MS 2006 Driver (default) with Windows11 ARM64, don't know what the differences are with swtpm.

Because they developed and provided their driver solution for it. Behind VM Fusion is a big company with larger funds. Behind UTM there is no so much. Unless we all work together to get this working, there is no golden solution for that.

Looks like the issue with TPM2.0 might be a UEFI firmware issue, and the OVMF image edk2-aarch64-code.fd from upstream QEMU might need to be rebuilt with TPM2_ENABLE. https://groups.google.com/g/linux.debian.bugs.dist/c/KIARY7bL5RI
It seems the upstream EDK2 defaults for the ArmVirtQemu.dsc that AFAICT is the basis for the firmware in upstream QEMU does not include TPM2.0 support: https://github.com/tianocore/edk2/blob/b92298a/ArmVirtPkg/ArmVirtQemu.dsc#L32
It looks somewhat straightforward to build a custom UEFI image following Gerd Hoffman's instructions here: https://www.kraxel.org/blog/2022/05/edk2-virt-quickstart/

Are there any limitations in Windows11 without TPM2.0?  I'm using windows11 2021H2 and wanna to upgrade to 2022H2 but failed because of missing TPM2.0 and SecureBoot. I don't want a clean setup, so  I tried Rufus(https://rufus.ie/en/) and it has to create bootable u-disk from iso, and it has option to remove TPM2.0 and secure boot limitation of windows setup program. And then I used setup.exe in u-disk to upgrade windows. It works.

@ehntoo Thanks for your research! Just wondered if you've been able to get it work after all?

Since we can't update Windows 11 Arm automatically without TMP 2.0, does anyone know of another method that does not require wiping and reinstalling?
 WTF?

OK all. I found an article that explains how to force windows 11 arm into the dev channel. Tested and it is updating as I type.
https://winaero.com/how-to-enable-windows-11-dev-channel-on-unsupported-device/

does anyone know of another method that does not require wiping and reinstalling

@phirestalker Have you come up with an answer?

@phirestalker Have you come up with an answer?

I gave up and think I will wait for a real release of Windows 11 arm. I am just using my windows box with RDP for now.

Testing with the following:
-chardev
socket,id=chrtpm,path=/Users/dariusz/Library/Containers/com.utmapp.QEMUHelper/Data/Documents/mytpm0/swtpm-sock
-tpmdev
emulator,id=tpm0,chardev=chrtpm
-device
tpm-tis-device,tpmdev=tpm0

Starting TPM prior to UTM with
swtpm socket --tpm2 --tpmstate dir=/Users/dariusz/Library/Containers/com.utmapp.QEMUHelper/Data/Documents/mytpm0 --ctrl type=unixio,path=/Users/dariusz/Library/Containers/com.utmapp.QEMUHelper/Data/Documents/mytpm0/swtpm-sock --log file=/Users/dariusz/tpm.log,level=5

Can confirm that is working. Great! Now the matter of drivers to sort out :)

Relace /dariusz/ with your username

Where do I put this?


LocalStorage fails in Safari private mode

Looks like Safari has a pretty strange behavior with local/sessionStorage when being in private mode, it just throws an unrelated error: https://stackoverflow.com/a/14555361/758728
Would you mind adding this localStorage support check?

Is it not a support check in index.js from line 9 ?
function canWriteStorage(storage) {
  try {
    storage.setItem('@@', 1);
    storage.removeItem('@@');
    return true;
  } catch (e) {}

  return false;
}

Right, didn't see that. Looks like the return statement is outside the catch scope, is it normal? Here's the code from StackOverflow answer:
try
    {
        storage.setItem(testKey, '1');
        storage.removeItem(testKey);
        return localStorageName in win && win[localStorageName];
    }
    catch (error)
    {
        return false;
    }

Here's the error given in iOS 10.3 in private mode:

It is ok to have the return statement outside the catch, the catch does nothing in this case. And it seems that the check is doing its work correctly as you can see on line 52. You need to build some functionality around it yourself I am afraid to handle private visitors.


Append agreeSimilarTransaction

I try to resolve #298.
If you find any flaws in the implementation, please feel free to close it.
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you read and signed the automated Contributor's License Agreement?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

[x] Does your submission pass tests?
[x] Have you lint your code locally prior to submission?

Changes to Core Features:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your core changes, as applicable?
[x] Have you successfully ran tests with your changes locally?

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

Thank you for your submission.
We are going to solve #298 in a slightly different way, in #331 , to be more consistent with another one of PayPay's SDKs.
So, I'm going to close this pull request in favor of #331 , which should be released to NPM within the next few days.

@curtisfennerpaypay Thank you for update.
When do you plan to next release (when is the next few days)?


JsonSyntaxException crash on empty course list

This issue was faced by a user not registered in any courses. He circumvented the issue by enrolling via the website, after which the app functioned normally. The attached logs were provided by him.
As per the relevant code in CourseRequestHandler, in the case of no courses being registered, the responseString should be [] (an empty JSON Array) and is parsed correctly.  Bug could not be replicated with another account, which was not registered in any courses. Unclear why it might have happened to him.
Should be fixed if the problem becomes clearer. Replicating the request which causes this wrong response to be returned, and then printing it, would be an ideal way to go about this.

Closed as it is a duplicate to issue #37. Relevant details have been copied over.


Can I put scrollBar instead of dots?

Hello,
Is it possible to put scroll bar instead of dots at the bottom of the scroller?

May be you can use slickGoTo method to implement this usecase
https://react-slick.neostack.com/docs/example/slick-go-to


Errors with music assistant, and I can't remove music assistant as well

What version of Music Assistant has the issue?
v2023.2.1
The problem
I haven't been using Music Assistant for a while since I've deleted our spotify account. I've disabled the integration when I deleted the account. Today I was browsing my logs and I saw that Music assistant is giving errors. Then I've tried to delete music assistant, but it won't let me. It says that I need to delete the configuration first and sends me to my integration page. On that page I see that Music assistant isn't loading and I can't delete it over there as well. What can I do to solve this issue?
I've been running the following Home assistant version:
Home Assistant 2023.5.2
Supervisor 2023.04.1
Operating System 10.1
Frontend-versie: 20230503.3 - latest

How to reproduce
I've tried a lot, but I'm afraid to remove folders or lines from the configuration without consulting this first.
Relevant log output
Failed to parse message (b'8184016a') on cluster 32799, because Data is too short to contain 1 bytes
14:03:27 – (FOUT) runner.py
Setup failed for custom integration mass: Requirements for mass not found: ['music-assistant==<IP_ADDRESS>'].
14:03:26 – (FOUT) setup.py
Unable to install package music-assistant==<IP_ADDRESS>: ERROR: Cannot install music-assistant because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts [notice] A new release of pip is available: 23.1 -> 23.1.2 [notice] To update, run: pip install --upgrade pip
14:03:26 – (FOUT) util/package.py - bericht kwam voor het eerst om 14:03:16 en verschijnt 3 keer
Additional information
No response
What version of Home Assistant Core are your running
2023.5.2
What type of installation are you running?
Home Assistant Supervised
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Hi @Bart-van-Gorp
This discussion thread and post should help you solve your issue:
https://github.com/music-assistant/hass-music-assistant/issues/1143#issuecomment-1467928564
We hope it helps.


fix(volume-confirmation): Rename shared security group and changes in …

…confirmation

@ksendart yes, Russian dialog must be changed as well. @HeyRoach please complete the task


Fix view with notes in cooperation details

https://github.com/ita-social-projects/SpaceToStudy-Client/assets/118522791/2a5c2fde-04ec-48e2-a10f-11720fd4a1fc

Don't forget to link issue.


Python wrapper for starSpace

Hi, I have wrote a python wrapper for starSpace. It uses pybind11 can be installed with conan package manager (conan is required in the dev. env.). To build wrapper library, CMake is required.
build.sh inside python directory will build the wrapper and run test script.

Thanks for the work, FWIW I had the following minor issues:

Making the starspace binary dependent on libstarspace.so means you have to set LD_LIBRARY_PATH or some such, maybe it should be statically linked?

Maybe the default makefile target should remain the main starspace binary?

It would be nice if the bindings included e.g. getNgramVector, getDocVector and maybe predictOne, though I guess this would also require a numpy binding for the matrix stuff so isn't just a matter of adding a line for each extra function

Maybe the default makefile target should remain the main starspace binary?

@jamesjrg That's a good suggestion. We can keep a separate makefile for those who needed python binding. Right?

It would be nice if the bindings included e.g. getNgramVector, getDocVector and maybe predictOne, though I guess this would also require a numpy binding for the matrix stuff so isn't just a matter of adding a line for each extra function

I'll try to do it. And you are welcome if you could help.

Looks like numpy binding is done with the def_buffer function:
https://pybind11.readthedocs.io/en/stable/advanced/pycpp/numpy.html
https://github.com/facebookresearch/fastText/blob/master/python/fastText/pybind/fasttext_pybind.cc
If noone else has done it in like a month's time I might well take a look, but a bit busy right now and quite likely someone else will get there first!

UPDATE: Incorporated @jamesjrg 's first two suggestions

Reduced dependency from two to one. Only starwrap.so is needed.
added separate makefile for python bindings (makefile_py), reverted original makefile.

@ledw Update: Added comments in build script, Added README for python, added reference from root README.
@rnditdev I'm sorry for the late reply, suddenly got into unbalanced mode. predictOne has also got typedefs as params, that I have no idea on how to bind.

Could we simplify the interface of PredictOne, so we facilitate the creation of bindings?

Thank you for the python wrapper again. But I am wondering whether it's possible to return the responses as objects rather than just print it on the screen. For example, when calling sp.nearestNeighbor('some text',10) in the test.py file, I think it's better we can assign the response to a variable neighbors=sp.nearestNeighbor('some text',10) so that we can do better downstream analysis. Similarly, it's better we can record the training history into a history object.
Sorry that I am not quite familiar with c++ projects and may not able to help, but I do wish someone can help make the python wrapper more user-friendly.


Switch to a better maintained PyPDF Fork

Last commit to https://github.com/mstamy2/PyPDF2 was in 2018.
See https://github.com/mstamy2/PyPDF2/issues/571 and https://github.com/hellerbarde/stapler/issues/78 as well
I'd propose switching to https://github.com/sfneal/PyPDF3 which seems to be better maintained with a few releases this year.

Hi! Given that the original pypdf2 upstream seems  defunct and I might need xhtml2pdf for rst2pdf (I'm packaging this for Arch Linux), it would be awesome to see some direction on this, as I would like to circumvent packaging pypdf2 if it is in that shape.

If the project agrees with the PyPDF3 choice, I am happy to start work on a PR.

I agree, go a head

Filed 3 PRs:

Switch to PyPDF3. This drops support for Python <3.6
Remove debug logging from tests. This uses unittest.assertLogs which needs Python 3.
Add GitHub Actions. This still needs some work, and can be merged after the above 2 are finalized.

If there is a chance to get a release with the recent changes, that would be fabulous! <3

Yes, I will be a new release soon (one or 2 weeks), but first we need to work more in test, the new release has a lot of changes and dependencies updates.   thanks @captn3m0  for your great work.

Stuff that I'd like to try to do before the next release:

[ ] Test improvements for Mac/Windows. Appveyor is very slow, and I'd like to consolidate everything under GitHub Actions
[ ] Fix coveralls integration. @luisza if you could add
[ ] Publishing setup for the docs (also broken currently)
[ ] A few general test setup improvements
[ ] Fix render-tests across all platforms, if possible.

None of these are critical as such, so maybe we can push out a beta release in the interim?


Got T.nilable when T::Boolean expected

Input
→ View on sorbet.run
# typed: true

class SrbTest
  extend T::Sig

  sig { returns(T::Boolean) }
  def call
    [true, false].sample
  end
end

Observed output
editor.rb:9: Returning value that does not conform to method result type https://srb.help/7005
     9 |    [true, false].sample
            ^^^^^^^^^^^^^^^^^^^^
  Expected T::Boolean
    editor.rb:8: Method call has return type T::Boolean
     8 |  def call
          ^^^^^^^^
  Got T.nilable(T.any(TrueClass, FalseClass, T::Array[T::Boolean])) originating from:
    editor.rb:9:
     9 |    [true, false].sample
            ^^^^^^^^^^^^^^^^^^^^
Errors: 1

Expected behavior
No errors.
The example call function expectations are strictly true/false why the sorbet shows a nilable (T.nilable(T::Array[T::Boolean]))?

This is the expected behaviour of Sorbet, since the definition of Array#sample is overloaded. Array#sample can return nil, a single value or an array of values depending on the parameters supplied to the sample method:
> [].sample
=> nil
> [1, 2, 3].sample
=> 1
> [1, 2, 3].sample(2)
=> [3, 1]

There is no way for Sorbet to statically know which one of these types will be returned, so the return result is the most general it can be.
The documentation and signature for the method can be seen here: https://github.com/sorbet/sorbet/blob/master/rbi/core/array.rbi/#L2184-L2212


Custom template with DisplayTemplate and full path not working

First thank you very much for your nice component :)
In my project i need to specify the full path of my template but when i use a path like :
~/Areas/SiteWeb/Views/Shared/_MyTemplate.cshtml it is not working because in your component in Pager.cs you force the template to be in the Displaytemplates folder like this :
  var templatePath = string.Format("DisplayTemplates/{0}", this.pagerOptions.DisplayTemplate);
I suggest to check if this.pagerOptions.DisplayTemplate start with ~  in order to use full path like this for example :
if (!String.IsNullOrEmpty(this.pagerOptions.DisplayTemplate))
{
	var templatePath = String.empty;
	if(this.pagerOptions.DisplayTemplate.StartsWith("~")){
		templatePath = this.pagerOptions.DisplayTemplate;
	}
	else{
		templatePath = string.Format("DisplayTemplates/{0}", this.pagerOptions.DisplayTemplate);
	}
	return htmlHelper.Partial(templatePath, model).ToHtmlString();
}

It will be more flexible for people who have my needs.
Thank you in advance

Good idea. Could you perhaps create a pull request for this?


Split keyword docuymentation to sections (by tags or otherwise) to make it easier for user to find what they are looking for

Requires upstream modifications to be able to modify the generated libdoc documentation

I agree also, closing issue.


Swift allows & calls @available(*, unavailable) deinit

For classes, Swift should probably make @available(*, unavailable) deinit { ... } illegal syntax because it's impossible to uphold the unavailability.
For @noncopyable types, I think it should make the user call a consumeing and forgetful function.
class Foo {
    @available(*, unavailable)
    deinit {
        print("Isn't it ironic.")
    }
}

func foo() {
    _ = Foo()
}

foo()

$ swift test.swift
Isn't it ironic.

rdar://106409012

Addressed for 5.9 with #64844. Note that the PR does not address @noncopyable, types, though since I think the proposed behavior would need to be incorporated into an accepted Swift evolution proposal before it is implemented.


Update AboutUs and create skeletal PPP

Update my roles and responsibilities in AboutUs and create skeletal PPP.

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (4d71b90) compared to base (ec2c13b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #30   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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.

I think Rezwan and I are using our own photos

I saw Rezwan's photo but I didn't see yours.

Cos its still in the PR haha

This should work then.


No detection of printers in "reject" state

Split from issue #224.
cups-connector detects when a printer is paused (with cupsdisable) but not when it is rejecting jobs (with cupsreject). It needs to do both.

https://bugs.chromium.org/p/chromium/issues/detail?id=619428

We should be acting on the printer-is-accepting-jobs boolean.
https://tools.ietf.org/html/rfc2911#section-4.4.23


chore: sanitize http.url attribute values

Which problem is this PR solving?
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
I added a map of semantic attributes to a value sanitizing function. also made this work for one off values.
Errata: I'm not sure if you want to classify this as a breaking change or not. Also, I'm not sure why my tests are failing. Any vscode+mocha tips appreciated!
Fixes #2000
Short description of the changes
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] added unit tests

Checklist:

[x] Followed the style guidelines of this project
[x] Unit tests have been added
[] Documentation has been updated

i'm not sure what i need to do - i accepted as an individual contributor for the cla, but it says it is not enabled for this project.

Are you signing the CLA with the email used to commit?

thanks @osherv for the review - i'm working on your feedback.
do you have any tips on how that i can debug in vscode?

@daniel-white sure! unfortunately i'm working in Webstorm, i can help you debug with Webstorm 😅

i know my tests/lint are failing. however, i'd like to confirm that this is the pattern you all would like to go with


Full API reference link is down.

The link (http://nabovyan.xyz/telegram-node-bot/) is down. You can still access throw the internet archive wayback machine, but I think that I would be better if the docs here on github didn't list to a broken link.
Link for docs in wayback: https://web.archive.org/web/20180402035304/http://nabovyan.xyz/telegram-node-bot/#toc18__anchor

I agree with you. I have been trying to get to the main website to see me documentations but I noticed it was broken. Thanks for the wayback link )


Missing guidance for maintainers

Dear @jochenklar
as discussed in the train on Friday, I'm opening an issue for this...
After being enlighted by the workshop about the correct usage of the catalog, etc. I want to rise an issue and complain about missing documentation for maintainers of an RDMO instance.
Currently, there is a lot of technical documentation available, but the different pitfalls while trying to setup an instance for production use is missing a lot of details.
I am willing to contribute here as we go on within FZJ. (Mentioning @bronger)
Cheers,
Oliver

Hi @poikilotherm, sorry I did not reply sooner... We are still not sure what to use for this kind of documentation. The technical documentation ist done with sphinx. We also have material on http://www.forschungsdaten.org/index.php/RDMO. Maybe gitbook is an option.


Ensure garbage is collected in Python tests

C++ destructors may not run on time if Python garbage collection doesn't run soon enough, this is problematic if we fork the process causing state from previous tests to leak into the new processes which may fail due to lack of state owned by the parent process (e.g., CUDA context). Additionally, forking after the process has started the worker progress thread and hasn't stopped is problematic as it will be a shared state in the new process.
Fixes https://github.com/rapidsai/ucxx/issues/138 .

I think there's no reason not to get this merged even with #142 being in, though I think it would be useful to wait a couple of days before merging to see how nightly builds look like after #142.

I think there's no reason not to get this merged even with #142 being in, though I think it would be useful to wait a couple of days before merging to see how nightly builds look like after #142.

Agreed

We only got branch-0.36 nightlies actually building about 4-5 days ago, but the original issue from #138 hasn't happened since so #142 must have fixed the issue. In any case as discussed previously there's no harm in merging this PR, so I'm going ahead with that now.

/merge


Add module name and fix docstring, add datatype dict and some other changes

I need an explanation for CHANGELOG.md if you can write an explanation about your pull request and send it here.

I need an explanation for CHANGELOG.md if you can write an explanation about your pull request and send it here.

Sure. i'll do this later.

Done.

Thank you for P.R and help :heart:


⚠ Bofrost.de has degraded performance

In 2479d8b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1863 ms

Resolved: Bofrost.de performance has improved in 3c0c7d9 after 16 minutes.


Update apache-client, kinesis to 2.20.153

About this PR
📦 Updates

software.amazon.awssdk:apache-client
software.amazon.awssdk:kinesis

from 2.20.118 to 2.20.153
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #675.


Implement bare-bones blob listing

Update FakeStorageRPC to (barely) implement list(bucket).
Wire the NIO provider to take advantage of it so we can list the contents of folders.
Add the corresponding test case.

I notice my IDE helpfully changed some of the files: it added an explicit java version (I'm using 1.8 features, hopefully that's OK?), and it added some wildcard imports.
Let me know which of those (if any) I should revert!
Also I'm not entirely sure of the contract that list(String bucket, Map<Option, ?> options) is supposed to follow, beyond "Lists the bucket's blobs". I filled it in with my best guess but if there's a spec somewhere for me to follow, pointers would be appreciated!

OK, looks like Java 8 is a no-go, I'll change things to the old way.

Changed to Java 1.7, PTAL.

This should address all the comments so far.

mmh, the extra line is still there even though I thought I deleted it. Let me try to delete more and see what happens. Alternatively, we could decide we're OK with two blank lines at the end of a file.

The mystery deepens. On my disk there clearly is only a single newline at the end (which is what you want to see), and git believes that my files match what's been pushed.
gcloud-java$ xxd gcloud-java-contrib/gcloud-java-nio/src/main/java/com/google/gcloud/storage/contrib/nio/CloudStorageFileSystemProvider.java  | tail -n 4
00005960: 7265 7475 726e 206e 6577 2049 4f45 7863  return new IOExc
00005970: 6570 7469 6f6e 286f 6f70 732e 6765 744d  eption(oops.getM
00005980: 6573 7361 6765 2829 2c20 6f6f 7073 293b  essage(), oops);
00005990: 0a20 207d 0a7d 0a                        .  }.}.
gcloud-java$ git diff
gcloud-java$ git commit
nothing added to commit but untracked files present

We can come back to this later.

The mystery deepens. On my disk there clearly is only a single newline at the end (which is what you want to see), and git believes that my files match what's been pushed.

I cloned your branch and I can see the 2 empty lines. If I remove one git diff shows:
--- a/gcloud-java-contrib/gcloud-java-nio/src/main/java/com/google/gcloud/storage/contrib/nio/CloudStorageFileSystemProvider.java
+++ b/gcloud-java-contrib/gcloud-java-nio/src/main/java/com/google/gcloud/storage/contrib/nio/CloudStorageFileSystemProvider.java
@@ -109,7 +109,6 @@ public final class CloudStorageFileSystemProvider extends FileSystemProvider {
     }
   }

-
   private final Storage storage;

   // used only when we create a new instance of CloudStorageFileSystemProvider.

I don't know what to say. Does git lie to us when it says that my branch's up-to-date? It looks like what's on my machine isn't the same as what's on the server. I'm going to push with the other fixes, let's see if after that the state on my machine matches the state on the server.

FWIW, when I look at the file on GitHub's viewer, it shows no empty line at all at the end of the file, so line 637 is the last line. When I look at it in my IDE it shows one more line (638), empty.
The hex dump still shows a single 0x0a at the end of the file. The diff for this commit indicates no change to this last line, nor does the previous one. Yet we seem to be seeing something different.
When you clone my branch, and before you open the file in any editor, what does the hex dump show?

The extra line seems to be gone now, am I wrong?
This looks good to me feel free to merge after squashing the commits.

Squashed. Can we agree not to worry about extra blank lines next time?


restrict upper limit of build process to the number of CPUs

This PR restrict the upper limit of build process for mkldnn to the number of CPUs.
Without this change, it is hard to build mlkdnn on my machine (intel nuc, which has 16GB ram), because of a lot of gcc processes.
This PR is ported from https://github.com/intel/chainer/pull/7, since ideep is now developed in this repository.

@tkng Leave a comment

@tkng

@tkng

Do you mean when build with "make -j" on machine with 8 logic CPU. The build system spawn more than 8 task for parallel build?

make -j executes as many processes as possible. I couldn't count the actual number of processes, but current build scripts seems executing at least tens of build processes (here, I mean gcc/g++ by build process) in parallel.
The problem is, each gcc/g++ process requires several hundreds of memory. As a result, I can't finish the build because of memory slashing.
On the other hand, parallel build itself is very attractive, since most of users have multiple CPU cores nowadays. Hence this PR intend to limit the maximum number of parallel build processes to the number of CPU cores by make -j {number of cpu cores}.

Thanks for the PR and explain.


Fee processing changes and fixes to Dev

Add new ENV variables to foi-flow web and requestmanagementapi
Also make sure the master data tables like NotificationTypes, DocumentTypes and CommentTypes are not having duplicate records.
deploy below WFs to bpm (to dev automatic deployment should happen)

foi-rawrequest-processing.bpmn
foi-request-processing.bpmn
foi-initial-payment-submission.bpmn

Migrate all WF instances to the latest version
Update the formsflowai forms if needed (already done for dev-marshal)
upload email template to dev S3 bucket (already done for dev-marshal)


SDC-10501 : Option to enable/disable Kafka auto commit Offsets

JIRA: SDC-10501 Kafka Multi-Topic Consumer not checking delivery guarantee
Toggle Option for Kafka enable.auto.commit.

The consumer can either automatically commit offsets periodically, default enable.auto.commit=true, with auto.commit.interval.ms=1000 interval.
Or it can choose to control this committed position manually by calling one of the commit commitSync after finishing processing messages.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Sandish Kumar seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

This looks good. I wonder if an upgrader is needed?

I also wonder if instead of making this an option, we just remove this entirely. At the pipeline level you can pick at least once or at most once.

This looks good. I wonder if an upgrader is needed?
https://github.com/streamsets/datacollector/blob/master/kafka_target-protolib/src/main/java/com/streamsets/pipeline/stage/destination/kafka/KafkaTargetUpgrader.java

Thanks for the response, KafkaTargetUpgrader.java looks like it related to Kafka Producer. But changes are related to kafka_multisource.

I also wonder if instead of making this an option, we just remove this entirely. At the pipeline level you can pick at least once or at most once.

Yeah, this makes sense. I will make changes.

Thanks for the submission @SandishKumarHN . I will need to also ask you to write an integration test for such a change - for the time being we keep the integration tests in a separate repository https://github.com/streamsets/datacollector-tests

Do you need any help with the integration tests or is all progressing well @SandishKumarHN ?

Do you need any help with the integration tests or is all progressing well @SandishKumarHN ?

Sorry, didn't have much time to work on integration tests, planning to work finish this weekend.

How are things going @SandishKumarHN , anything I can do to help?

Cool, let me know if there is anything that I can help with then.

Any luck @SandishKumarHN ? I'm happy to help.

Any luck @SandishKumarHN ? I'm happy to help.

Sorry for late reply, was stuck with some issues. will have something this weekend.

I'm thinking that the proper test will be to have pipeline reading from Kafka, through delay processor terminating with scripting processor that throws exception depending on pipeline parameter. We can run the pipeline once, wait for it to fail, then second time, wait for it succeed and ensure that we read all the records from Kafka and nothing was lost. What do you think @SandishKumarHN ?

I took the liberty of jumping in and help writing the test case described above @SandishKumarHN . It actually uncovered a small issue in the patch here, so let me upload an updated variant.
Can I ask you to sign our NDA so that I can merge the fix?

@jarcec Sorry for making delay on test cases was quite busy with work. and I have already signed NDA and this is my second PR for datacollector project.

No worries, we all have day jobs @SandishKumarHN  - This patch has been merged, thank you for your contribution!


Allow ö,ü,ä and numbers in subject and test names

Hi @DragonCat4012
Can you explain the issue here. Would like to take a look.

"^[a-zA-Z_ ]*$"
This regex doesn't allow ä,ö,ü


Fixes 2842 credit @merriam.

Essentially Fixes #2842. Also improves documentation of ColorPicker.
From @merriam.
Color picker bug not reaching pure colors.
This was part of #2842.   Also, some doc.
Complies with make style.

Looks good, thanks both!


feat: add biome

Hey 👋,
This PR adds biome, as discussed in #244.

Closes #244

I believe the tests are failing because the biome CLI currently exits with 1 when running biome --help or biome --version, which is something we fixed on main, but not in the current release.

Thanks for the PR!

If there is no way around that, perhaps we should wait until the next release.

You can also handle biome as a special case here (by adding biome) rx "${tool_bin}" --version || true ;;).

Thank you, @taiki-e, works like a charm!

Published in 2.20.0.

Thanks for merging and your help @taiki-e !

Thanks for picking this up @nhedger !