Nord\Lumen\Cors\CorsService::__construct(): Argument #1 ($config) must be of type array, null given

What did you do?
Trying the package out
What did you expect to happen?
To work fine
What actually happened?
Throws an error
What versions of PHP, Lumen and this module are you using?
PHP 8.0, Lumen 9
Please include code that reproduces the issue. The best reproductions are self-contained scripts with minimal dependencies.
[2022-03-27 21:01:13] local.ERROR: Nord\Lumen\Cors\CorsService::__construct(): Argument #1 ($config) must be of type array, null given, called in PROJECT_DIR/vendor/nordsoftware/lumen-cors/src/CorsServiceProvider.php on line 33 {"exception":"[object] (TypeError(code: 0): Nord\\Lumen\\Cors\\CorsService::__construct(): Argument #1 ($config) must be of type array, null given, called in PROJECT_DIR/vendor/nordsoftware/lumen-cors/src/CorsServiceProvider.php on line 33 at PROJECT_DIR/vendor/nordsoftware/lumen-cors/src/CorsService.php:59)
[stacktrace]
#0 PROJECT_DIR/vendor/nordsoftware/lumen-cors/src/CorsServiceProvider.php(33): Nord\\Lumen\\Cors\\CorsService->__construct()
#1 PROJECT_DIR/vendor/illuminate/container/Container.php(873): Nord\\Lumen\\Cors\\CorsServiceProvider->Nord\\Lumen\\Cors\\{closure}()
#2 PROJECT_DIR/vendor/illuminate/container/Container.php(758): Illuminate\\Container\\Container->build()
#3 PROJECT_DIR/vendor/illuminate/container/Container.php(694): Illuminate\\Container\\Container->resolve()
#4 PROJECT_DIR/vendor/laravel/lumen-framework/src/Application.php(300): Illuminate\\Container\\Container->make()
#5 PROJECT_DIR/vendor/illuminate/container/Container.php(1027): Laravel\\Lumen\\Application->make()
#6 PROJECT_DIR/vendor/illuminate/container/Container.php(947): Illuminate\\Container\\Container->resolveClass()
#7 PROJECT_DIR/vendor/illuminate/container/Container.php(908): Illuminate\\Container\\Container->resolveDependencies()
#8 PROJECT_DIR/vendor/illuminate/container/Container.php(758): Illuminate\\Container\\Container->build()
#9 PROJECT_DIR/vendor/illuminate/container/Container.php(694): Illuminate\\Container\\Container->resolve()
#10 PROJECT_DIR/vendor/laravel/lumen-framework/src/Application.php(300): Illuminate\\Container\\Container->make()
#11 PROJECT_DIR/vendor/illuminate/pipeline/Pipeline.php(169): Laravel\\Lumen\\Application->make()
#12 PROJECT_DIR/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(30): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#13 PROJECT_DIR/vendor/illuminate/pipeline/Pipeline.php(116): Laravel\\Lumen\\Routing\\Pipeline->Laravel\\Lumen\\Routing\\{closure}()
#14 PROJECT_DIR/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(426): Illuminate\\Pipeline\\Pipeline->then()
#15 PROJECT_DIR/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(175): Laravel\\Lumen\\Application->sendThroughPipeline()
#16 PROJECT_DIR/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(112): Laravel\\Lumen\\Application->dispatch()
#17 PROJECT_DIR/public/index.php(28): Laravel\\Lumen\\Application->run()
#18 {main}
"}

Did you perhaps miss this step:

Copy the configuration template in config/cors.php to your application's config directory and modify according to your needs

Did you perhaps miss this step:

Copy the configuration template in config/cors.php to your application's config directory and modify according to your needs

I did copy it

Are you sure? The error happens if you don't have the configuration file in place.

Fixed, thanks for your time.

I've got the same exception from CorsService
2022-05-11 13:08:45.923 EEST[11-May-2022 10:08:45] WARNING: [pool www] child 11 said into stderr: "[2022-05-11 10:08:45] local.ERROR: Argument 1 passed to Nord\Lumen\Cors\CorsService::__construct() must be of the type array, null given, called in /var/www/html/vendor/nordsoftware/lumen-cors/src/CorsServiceProvider.php on line 33 {"exception":"[object] (TypeError(code: 0): Argument 1 passed to Nord\\Lumen\\Cors\\CorsService::__construct() must be of the type array, null given, called in /var/www/html/vendor/nordsoftware/lumen-cors/src/CorsServiceProvider.php on line 33 at /var/www/html/vendor/nordsoftware/lumen-cors/src/CorsService.php:59)"

I have added config file into correct place config\cors.php but for some reason it doesn't see it 🤔


Support embedding binary files

Raw bytes list
Base64 encoded string

Ah, a lot of works, thanks so much!
I'll check it out this weekend.

@NicolaVerbeeck
Thanks for the contribution! Left one small comment before we can land this!

@NicolaVerbeeck
Published in v1.2.0


Support for get_auth_code_manually to work with generic channels

I have been using this commit since you implemented it: https://github.com/z-Wind/schwab_api/commit/62db905ffaf6275352bc29c58ff8d300a31d9644#diff-95eb5afe64e2d10a8f9a34145dbb1fe9d99ca910471f4325e2fc3635d92a89edR115 and have found it extremely helpful, especially when working on a remote machine and the callback doesn't work seamlessly with what's registered in Schwab.
I have reviewed https://github.com/z-Wind/schwab_api/issues/1, and I think there is still a helpful use-case to get some kind of channel based manual auth. If you're okay with it, I'd like to work on this and open a pull request to support this functionality.
Namely, instead of explicitly relying on stdout to provide the authorization link and stdin to provide the callback URL containing the token, I'd like to make introduce another function that uses a generic tx and rx channel, where the default implementation could still utilize stdout and stdin. This way, there is no change in functionality for users. But by exposing a generic trait, it could open up usecases where authentication could be push-based (e.g., tx to sms, email, or some other webhook / mfa mechanism, and then rx to provide the callback). This would allow users to authenticate without an explicit TTY session.
Please let me know what you think.

Thank you for the detailed explanation. I think this is a great suggestion, and it would definitely enhance flexibility for various use cases. Adding a generic tx and rx channel-based manual authentication sounds like a valuable improvement, especially for scenarios without an explicit TTY session. I’m happy for you to work on this and open a pull request. Looking forward to seeing your implementation!

@z-Wind , as I'm going through this, I realize this will probably have to be a breaking change. My current draft has a new set of traits that are generic on the Authorizer, which would mean the generics also need to be specified on the TokenChecker because it stores the Authorizer on as an attribute. This would unfortunately would be a breaking change for other useres. Is this still something you'd be interested in?

@roykim98 , I believe this change would significantly improve the crate, and given its current usage, introducing a breaking change shouldn’t be a major concern. Please feel free to move forward with it—I'm looking forward to seeing the updates!

@z-Wind Here's the initial draft for the PR: https://github.com/z-Wind/schwab_api/pull/8. It's a bit larger than I had anticipated and wanted, but I managed to get it to work without any real breaking changes to the main user interfaces by using dynamic dispatch instead of generics. The main benefit of this is that any users currently using the crate can pull the latest and have it be a no-op. It makes the code a little harder to read and a bit less performant (theoretically) but I think this will be better than for an unexpected change to the main interfaces by having to specify some generics.
Aside from that, it's a first pass, so the PR's still a bit messy, but most of the basic checks are passing and should be in a good enough state for some general feedback regarding the approach.


3d audio support

Hello,
A small question. This wrapper supports the 3d features of the openalsoft? Including the hrtf functions?
thanks :3

This only supports basic 3D position.


Add BitCoinForum

For future reference, take a look at the wiki for help on adding new sites.  The claimed username example in your pull request was not actually claimed.  So, the tests failed.
I corrected this during the merge.

Sorry, i copied and edited the entry above, and forgot to change that line.


Implement list-to-tuple comprehension unpacking

This is a part of #827. I tried using libcst_native to create the new item, but the SetComp items uses curly brackets. Since this is a really simple fix I decided to use the replace method. However, if you want this changed let me know.

FYI there’s no such thing as a “tuple comprehension”. (fn(x) for x in items) returns a generator object and is called a generator expression.

Thanks, sorry, I'm tired!

Thanks for the catch!


Fixing a bug in the required validity check

When you manually set the required status it causes quite a few ui oddities, This approach just overrides the isEmpty method (As is illustrated in the docs) in order to use the existing ngRequired structures

I just rebased to fix the conflicts, any chance on getting this pulled in?

Released as 1.6.1


[Bug] Sort Code: prompt does not agree with behavior

Description
The Mask for Sort code is "AA-AA-AA" and when used, a numeric keyboard is given to do the entry with.
Everywhere else that a Numeric keyboard is used, the prompt/mask uses XXXX...
I think to be consistent, For Sort code, either the Mask should be changed to "XX-XX-XX" or the Keyboard should be changed to Text or Default.
Steps to Reproduce

Expected Behavior
Actual Behavior
Basic Information

Version with issue:
Last known good version:
IDE:
Platform Target Frameworks:

iOS:
Android:
UWP:

Android Support Library Version:
Nuget Packages:
Affected Devices:

Workaround

Reproduction imagery

Reproduction Link

@WillAutioItrax could you please build a sample showing what are you talking about?)

Like Andrei I'm not really sure what you mean, so could you please clarify :)

@WillAutioItrax the pattern can use any character. The default character is 'X'.
The sample just shows that you can override the masked character, i.e. in the sample it's overridden to 'A'
The sample shows the defualt and overriden version -
<Entry Keyboard="Numeric">
    <Entry.Behaviors>
        <xct:MaskedBehavior Mask="XX-XX-XX" />
    </Entry.Behaviors>
</Entry>
<Label Padding="10,0,10,50" FontSize="Small" Text="e.g. Sort code 'XX-XX-XX'" />

<Entry Keyboard="Numeric">
    <Entry.Behaviors>
        <xct:MaskedBehavior Mask="AA-AA-AA" UnMaskedCharacter="A" />
    </Entry.Behaviors>
</Entry>
<Label Padding="10,0,10,50" FontSize="Small" Text="e.g. Sort code 'AA-AA-AA'" />

There is no issue here.

I find it confusing that sometimes that mask is "X" and sometimes (once) it is "A" and mostly there is a Numeric keyboard. I've been in places where "X" would mean any character can go here; "A" for alpha; "N" for numeric. I get that the "A" is a placeholder. It seemed odd that there are all these Behaviors with "X" and then one with "A". That's my 2 cents.
BTW, I appreciate the work that you all put into this package.  Thanks.

@WillAutioItrax can this issue be closed then? :)

@AndreiMisiukevich Yeah - sure.


See through header components on navigate

When you pop / push with StackNavigator the left component is see through. Watch closely the back button.

Setting background/underlay color for the back button component should change the behavior.

I believe @skevy is working on this

Close the issue once it's done, not when someone is allegedly working on it, then we can reference the PR @spencercarli. Until then it should be open.

Pinging OP @ferologics since this issue is quite old and related to an old version of the lib. If no answer I'll close in 7 days.

Hi there @ferologics ,
In an effort to cleanup this project and prioritize a bit, since this issue had no follow up since my last comment I'm going to close it.
If you are still having the issue (especially if it's a bug report) please open a new issue that uses the new Issue Template to provide some more details to help us solve it.


Introduce foreman Host (via connection.host)

Change foreman connection to reduce the number of methods available.
This is pulling out these changes for #1783

removed foreman methods that were simply rest wrappers
introduce host object

/cc @gmcculloug @brandondunne

:+1: Looks good, I think this is really close.

How is this different from #1783 ?

@Fryguy Thanks - looks like a commit from here got over there.
this pr (1819) changes the connection and introduces the host object. There is a bunch of changes around that and I even introduced a few commits.
#1783 introduces the provider object and is relatively straight forward.

@brandondunne updated host_object to be called host.
@Fryguy let me know if you have any further questions

:+1: nice


Feature Request/Question: zero-allocation support

Thanks for your great library; I am excited to finally see proper promises in C++ :D
Now to my question: Would it be possible to support environments in which allocation is not an option? Under which conditions are allocations mandatory at the moment?

Hey, I'm glad that you like the library.
Continuable does an allocation in following two circumstances:

When using type erased objects like continuable<...> or promise<...>
When using a connection between continuables (when_all...)

Probably it would be possible to store the promises of an unknown type in a preallocated space,
so you could ellide the memory allocation completely when deferring the promise resolution.
Avoiding the memory allocation for the shared state in connections is currently not possible nor planned.

Thanks for your quick response :)
I am still trying to wrap my head around the library…
In my use case I just wouldn't use type erasure, so that would be fine, but the connections using allocation would be a problem in an allocation-free application…
Thanks for the infos anyways :)


Add agent_id in authtoken

Closes #23

Assign the PR to yourself
Add labels
Attach the PR to dopagent project


The currently opened project name is not recognized as Main module.

To reproduce:

Open any project and create a new method in its Main module (by, for example, collapsing nodes).
(perhaps needed: rename the project).
Open Component browser and pick this function (it's in "Local Scope" section).The inserted node has code <current project name>.<function name> e.g. Unnamed1.func1

Expected:
The node evaluates properly
Actual:
The node has error, the message says that the project name is not recognized.

Not needed for initial release.


Snippet-bot complains about files in owl-bot-postprocessor/synthtool/gcp

https://github.com/googleapis/repo-automation-bots/issues/4697

obsolete


Feature: target individual corners when rounding

Hey guys,
Looks like there are no options yet to target individual corners with rounded.
Would it help if I make a PR to add the following?:
.rounded-tl - top left
.rounded-tr - top right
.rounded-bl - bottom left
.rounded-br - bottom right

There's already a PR for this: #188 🙂
This will be available in 0.2 🙂

Hey there @egyptik! We've actually got a PR (#188) in the works already to add this to the next (breaking release). It's part of a PR to change "rounded" to "radius", which we are actually still debating. 😀
Should be out within the next few days. 👍

Ah okay missed that, sorry. Great! 😄


Add a public API to process a customized LiterateDocument.

Now that Literate if internal my fix for #251 doesn't work anymore :(.
See also https://github.com/tpetricek/FSharp.Formatting/commit/dd03c61e726af20867b1c824e34ab7a6f484fa64.
Do you like another optional parameter for that? Alternatively we could use a03fcfb.

Oh, hmm - I guess we don't need to make it internal. I did that just to cleanup the public API reference, but we can just [omit] the module.
For individual document, you should be able to use ParseScriptFile, then do the transform and then call WriteHtml. Or is there some reason why this wouldn't work?
Now, I see that this cannot be done for ProcessDirectory. I'm not a very big fan of complex callbacks (see this blog post), so I'd probably prefer an API where you can call ParseDirectory (which returns a list of files with parsed documents) and then WriteHtml taking the list and writing all documents. Would that let you solve the problem?

WriteHtml is very limited and it doesn't do the same processing that https://github.com/tpetricek/FSharp.Formatting/blob/master/src/FSharp.Literate/Formatting.fs#L88 does.
Sure there are a lot of ways to solve this, I think the callback is reasonably simple though (basically LiterateDocument -> LiterateDocument).
Besides that I first experimented with a03fcfb (an alternative solution) which would also solve my problem (and is a unification of WriteHtml and WriteLatex actually). What do you think about that API?

The ProcessDocument looks like a nice addition to me.
If that solves your problem too, I'd be much happier to accept that (and yes, you're right - the WriteHtml and WriteLatex functions wouldn't work because they omit the transformations...). I think adding ProcessDocument would be good even if we had the callback, actually...


Fix/improve api

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Class 'MongoDB\BSON\ObjectId' not found

I am using Ubuntu 18.04.2 LTS and my server is nginx. I am getting below error.
FatalThrowableError in ReportFileController.php line 906: Class 'MongoDB\BSON\ObjectId' not found
906 line is:
'$id' => new ObjectId(request('patient_id'))
FYI,I have installed php-mongodb and added extenstion=mongodb.so in the /cli/php.ini file and restarted nginx.
Can anyone please help me?

@kajem, Check that extension was installed for php-fpm

You can see on this issue

@Smolevich, I have checked and founds php-fpm is installed and active.

@kajem, check that installed mongodb

@kajem, let's go to slack community. Link here

You need to insert the extenstion=mongodb.so on php.ini of php-fpm, you only putted on cli.


YETUS-263 Personality for Apache Accumulo

Here's an initial personality. Limited testing of test-patch with the personality locally appears to be functioning as intended.
Two changes I've run into so far to get test-patch working as desired for Accumulo:

We want to run all unit tests, not just the ones in modules changed by the patch
Our checkstyle rules are defined in configuration only available in the check checkstyle-plugin goal

Feedback is humbly and appreciatively welcomed.

This was committed in https://git1-us-west.apache.org/repos/asf?p=yetus.git;a=commit;h=e89c0843876a836f5713495e273a68e34b71012e


Diagnostics placement is sporadic and displays no data

Does this affect the legacy HoloToolkit (master) or the Mixed Reality Toolkit (mrtk_release)?
Mixed Reality Toolkit
Describe the bug
Diagnostics view is meant to display diagnostic information in the scene.  However, when enabled, the "box" is not in the view (placement of the mesh is odd) and there is no content displayed.
To Reproduce
Steps to reproduce the behavior:

Enable diagnostics in the configuration profile
Set Visible to True in the diagnostics profile.
Create a new Scene
Click Mixed Reality Toolkit > Configure
Enter play mode.

Expected behavior
Diagnostics view should be in the FoV of the headset/camera with data displayed
Actual behavior
Placement of window is sporadic and no data is displayed
Screenshots
If applicable, add screenshots to help explain your problem.

This "window" was placed below and to the left of the player, single sided texture so it's only visible if the player steps back.
Unity Editor Version
2018.2.13f1
Mixed Reality Toolkit Release Version
2018.9 beta
Additional Notes
Tested using Seated / standing mode with no boundary

i'll take this one. I am investigating adding additional data

As part of this, the diagnostic window should be parented to the camera (or parent).
The intent weas to be headlocked so the data was always visible.

The intent weas to be headlocked so the data was always visible.

Headlocked sounds terrible lol. Can we make that configurable at least?

Headlocked sounds terrible lol. Can we make that configurable at least?

We can, though the greatest utility of this feature is to give you information on the portion of the scene that is in view (ex: as you move through the scene, the data is relevant to what you are seeing)

I noticed that this also doesn't really put anything in the headset, but only prints the values to the editor window


Check version skew in local connect

E.g.:
$ signadot local connect
Error: The connection type ControlPlaneProxy requires operator > v0.15.0

I suppose we are not going to do the same thing for tcp:// in signadot local proxy? might be better to include that.

But that has nothing to do with the fix in the operator (the fix was only related for service tunnel-proxy). In other words, if someone tries connecting a TCP service inside the cluster (and they have Istio), what will determine if it works or not, is:

the protocol characteristics (is it client first or server first, how verbose it is, etc)
or if the target K8S service has an appProtocol defined or the port is named following the Istio recommendations.

I suppose we are not going to do the same thing for tcp:// in signadot local proxy? might be better to include that.

But that has nothing to do with the fix in the operator (the fix was only related for service tunnel-proxy). In other words, if someone tries connecting a TCP service inside the cluster (and they have Istio), what will determine if it works or not, is:

the protocol characteristics (is it client first or server first, how verbose it is, etc)
or if the target K8S service has an appProtocol defined or the port is named following the Istio recommendations.

Oh, right, thanks. Probably worth documenting somewhere.


"OSError: CUDA_HOME environment variable is not set" when installing submodules/depth-diff-gaussian-rasterization

I'm trying to install all the requirements, but both of the submodules show the same error message when trying to install the dependencies using the given pip command in the readme file.
When trying to execute pip install -e submodules/depth-diff-gaussian-rasterization,  the following error message appears:
Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [14 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "Path\to\4DGaussians\submodules\depth-diff-gaussian-rasterization\setup.py", line 21, in <module>
          CUDAExtension(
        File "Path\to\anaconda3\Lib\site-packages\torch\utils\cpp_extension.py", line 1074, in CUDAExtension
          library_dirs += library_paths(cuda=True)
                          ^^^^^^^^^^^^^^^^^^^^^^^^
        File "Path\to\anaconda3\Lib\site-packages\torch\utils\cpp_extension.py", line 1208, in library_paths
          paths.append(_join_cuda_home(lib_dir))
                       ^^^^^^^^^^^^^^^^^^^^^^^^
        File "Path\to\anaconda3\Lib\site-packages\torch\utils\cpp_extension.py", line 2407, in _join_cuda_home
          raise OSError('CUDA_HOME environment variable is not set. '
      OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root.
      [end of output]

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

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

even though is set it through the windows environment variable window as well as the set CUDA_HOME="..." command. It is also found by my system in the command prompt, powershell and in my conda environment. When executing echo %CUDA_HOME%, the right path appears. It just can't be found when launching the setup.py script.
I'm working on Windows 11 using conda, I tried it using different python versions and many different torch versions.
Thanks for your time.

Hi,
Do you install torch with the correct versions?

And when using conda in order to install torch (as recommended on pytorch website) using conda install pytorch==1.13.1 torchvision==0.14.1 torchaudio==0.13.1 pytorch-cuda=11.6 -c pytorch -c nvidia I get another error during the build of diff_gaussian_rasterization extension :
  Preparing metadata (setup.py) ... done
Installing collected packages: diff-gaussian-rasterization
  Running setup.py develop for diff-gaussian-rasterization
    error: subprocess-exited-with-error

    × python setup.py develop did not run successfully.
    │ exit code: 1
    ╰─> [96 lines of output]
        running develop
        running egg_info
        writing diff_gaussian_rasterization.egg-info\PKG-INFO
        writing dependency_links to diff_gaussian_rasterization.egg-info\dependency_links.txt
        writing top-level names to diff_gaussian_rasterization.egg-info\top_level.txt
        reading manifest file 'diff_gaussian_rasterization.egg-info\SOURCES.txt'
        adding license file 'LICENSE.md'
        writing manifest file 'diff_gaussian_rasterization.egg-info\SOURCES.txt'
        running build_ext
        building 'diff_gaussian_rasterization._C' extension
        Path\to\anaconda3\envs\Gaussians4D\lib\site-packages\setuptools\command\easy_install.py:147: EasyInstallDeprecationWarning: easy_install command is deprecated. Use build and pip and other standards-based tools.
          EasyInstallDeprecationWarning,
        Path\to\anaconda3\envs\Gaussians4D\lib\site-packages\setuptools\command\install.py:37: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
          setuptools.SetuptoolsDeprecationWarning,
        Path\to\anaconda3\envs\Gaussians4D\lib\site-packages\torch\utils\cpp_extension.py:358: UserWarning: Error checking compiler version for cl: [WinError 2] Le fichier spécifié est introuvable
          warnings.warn(f'Error checking compiler version for {compiler}: {error}')
        Emitting ninja build file Path\to\repos\4DGaussians\submodules\depth-diff-gaussian-rasterization\build\temp.win-amd64-cpython-37\Release\build.ninja...
        Compiling objects...
        Allowing ninja to set a default number of workers... (overridable by setting the environment variable MAX_JOBS=N)
        [1/1] C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.6\bin\nvcc --generate-dependencies-with-compile --dependency-output

        [......]

had the wrong graphics card, everything is working now as i upgraded to a RTX 3060

If someone encounters the same problem, check if you have installed PyTorch incorrectly. I made a mistake when looking for previous versions at https://pytorch.org/get-started/previous-versions/ and habitually copied the pip command without paying attention, which resulted in copying the Wheel OSX command. That was silly of me. I uninstalled the OSX package and re-ran the command under "Linux and Windows," which solved the issue.


Is it possible to set custom bounds constraints?

I have a multi image layout in OSD. What I was wondering is can I set dynamic viewport constraints? As shown on the image, let's say I want to show the user only the second image so he cannot pan over those limits - the bounds could technically also be something else, like two images at once or only parts of the image.

Technically I could use a single image approach but would like to keep the multi layout approach for other reasons.

There isn't really at the moment. You might be able to make it work like so:
const newBounds = viewer.world.getItemAt(1).getBounds();
viewer.viewport._setContentBounds(newBounds, viewer.world.getContentFactor());

...But _setContentBounds is a private function, so it may change or be removed in future versions of OSD. Also there may be unintended consequences.
It seems like we need to bring setHomeBounds back to cover this scenario (and possibly also for  https://github.com/openseadragon/openseadragon/issues/2527). Let me know if you'd be interested in working on that.


Injection Error

Hello. Sometimes get that error on text injection

Is that something i can avoid?

This assertion is firing because the script line it is trying to ingest contains a > (and end-ruby control code) without the associated open code. These inserted lines look like gibberish -

D&ed mq >qmbm pq&jm edv+ jb&fj&.

Is this really the text you are trying to inject?

So, i need to change all ">"   Yes, text. Our alphabet has slightly more characters than yours. So we had to resort to replacing some characters to display correctly.

Ah, I see. Yes, I would advise against using the <|>@ characters, since these are actually used by the game engine. The first three are used to display ruby text (the small text on top of words, e.g. <bottom text|top text>, and the @ symbol is used to toggle between fonts using the @g control code (arc d3).


id confilicting in widgets

What steps will reproduce the problem?
when rendering widgets via ajax request there is probability that new loaded widget has conflict id with already existed widgets.
What is the expected result?
conflict in js based on this id is inevitable for example in ActiveForm validation.
What do you get instead?
use unique mechanism for generating Widget::autoIdPrefix
Additional info

Q
A

Yii version
2.0.20

PHP version
7.1.22

Operating system
OS X

Won't be changed in 2.0. You can set it manually for a widget so it won't conflict.

I think that we should eliminate the automatic generation of id and that it be configured manually with the id that the developer likes.

Yes, that sounds OK.

Seems to be done.


Convert to GitHub Actions

This converts the build, test, and publish CI workflows from CircleCI to GitHub Actions.
Notable differences from the CircleCI implementation:

We can now trigger this manually via workflow_dispatch if needed
It no longer runs on weekends to prevent surprises when nobody is around
The sequencing is a little different - in Circle, the example tests for heroku-22 might run before the build of heroku-20 completes. In GitHub Actions, we're not starting testing until all the builders are built. This simplified the workflow dependencies a bit.

W-11830543

@joshwlewis It looks like the publish job is being skipped on main too? Not sure why at first glance.
eg https://github.com/heroku/builder/actions/runs/3176302715/jobs/5175517309

Ah, the github context doesn't have a branch attribute, but it does have a ref attribute:
https://docs.github.com/en/actions/learn-github-actions/contexts#github-context
So something like this should work:
if: github.ref == 'refs/heads/main'

xref:
https://github.com/orgs/community/discussions/26349
https://stackoverflow.com/q/58139406
However it also seems that the default if for a job with a needs is success() and so by defining our own, it might be overriding the behaviour that skips the job if the previous jobs failed. See:
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#example-not-requiring-successful-dependent-jobs
https://docs.github.com/en/actions/learn-github-actions/expressions#status-check-functions
That is, I wonder if we really need to use this:
if: success() && github.ref == 'refs/heads/main'


Specify the type/version of App from the list "Aproved app"

It seems not be clear what is the type of application  that are supported by the Approve client App.  For example : Microsoft Power BI if is the access through a browser Edge app( in the list) is not supported.

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

ID: 0073f5a8-daf6-7f14-127f-e0c9d80af78d
Version Independent ID: d6d0afdf-3619-ca03-718b-a40bfd07bab9
Content: Azure Active Directory Conditional Access settings reference
Content Source: articles/active-directory/conditional-access/technical-reference.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

Updated.

#please-close


Use default preset instead of having the user creating a presets

With default presets automatically activated (see https://github.com/bbatsov/rubocop/blob/master/config/enabled.yml), giving the possibility the script to run without making additional configuration files.

Sure! I've planned this as well. Hanum will have a directory named presets which store every contributed presets. Suppose that I've created a presets for checking the OSM attributes data type. I will store it in presets directory. Then, I will update the configuration of Hanum. Well, I still thinking which way is the best to do that. Let see in this first Hanum release. 👍


Improve test coverage of core unit tests

What does this PR do?
Add missing UT on some core module classes, and improve some UT for maintainability

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

Project coverage is 70.36%. Comparing base (4a7586e) to head (810e356).

Files with missing lines
Patch %
Lines

.../android/core/internal/data/upload/UploadStatus.kt
95.83%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2294      +/-   ##
===========================================
+ Coverage    69.97%   70.36%   +0.39%
===========================================
  Files          733      730       -3
  Lines        27236    27264      +28
  Branches      4576     4584       +8
===========================================
+ Hits         19058    19183     +125
+ Misses        6894     6822      -72
+ Partials      1284     1259      -25

Files with missing lines
Coverage Δ

...d/core/internal/data/upload/RotatingDnsResolver.kt
95.83% <ø> (ø)

...ernal/persistence/datastore/DataStoreFileHelper.kt
93.75% <100.00%> (+78.75%)
:arrow_up:

...ernal/persistence/datastore/DatastoreFileReader.kt
95.56% <100.00%> (ø)

...ernal/persistence/datastore/DatastoreFileWriter.kt
84.75% <100.00%> (ø)

...rsistence/file/batch/PlainBatchFileReaderWriter.kt
75.68% <ø> (ø)

...ternal/persistence/tlvformat/TLVBlockFileReader.kt
100.00% <ø> (ø)

...atadog/android/core/internal/utils/ByteArrayExt.kt
94.64% <100.00%> (+23.21%)
:arrow_up:

...m/datadog/android/core/internal/utils/NumberExt.kt
50.00% <100.00%> (+50.00%)
:arrow_up:

.../android/core/internal/data/upload/UploadStatus.kt
98.68% <95.83%> (+10.22%)
:arrow_up:

... and 43 files with indirect coverage changes


Data preprocessing

Hi, @JunMa11,
Thanks for your great work. I have a question about data preprocessing. As you've provided the detailed splits for the datasets. Could you further provide the code for liver CT dataset processing, so we can compare our results with yours directly?
Best,
Xi

Hi @xifang001 ,
We use the default data preprocessing in nnU-Net.
for nnU-Net V1 Resuls, run
python experiment_planning/plan_and_preprocess_task.py -t TaskXX_MY_DATASET -pl Y -pf Z

Currently, we are re-evaluating the popular plug-and-play losses with nnU-Net  V2. The preprocessing command is
nnUNet_plan_and_preprocess -t XXX

Thanks for your reply. That sounds great. I'll try that one.

Hi @xifang001 ,
You can download the dataset here
https://pan.baidu.com/s/1reGPEiQico1SGaONMDSv8Q
PW：2021

Hi, @JunMa11,
Pretty useful. Thanks so much!

Happy to help:)


Angular 4.0.0

It seems that with the recent release of Angular 4, this broke. As far as I can tell, this issue provides an explanation of what needs to be done. It seems that BrowserAnimationsModule from @angular/platform-browser/animations needs to be included as a dependency. This was apparently moved out of core with the Angular 4 release so the synthetic attribute @ng2ElementState is be used.

Hey there @xogeny . Thanks for reporting this, I knew about the upcoming changes with 4.0.0. I will wait for the ng-conf to hear all about it and then I think more can be done than just making it work again with 4.0.0. Compare https://github.com/bergben/ng2-page-transition/issues/6

I have applied the update to import BrowserAnimationsModule but the current code tries to import AnimationEntryMetadata which unlike state, trigger etc is NOT in BrowserAnimationsModule and therefore causes transpilation errors. :-S

Alright guys looks like Angular 4.1 will support route animations finally!
The Angular 4.1 beta.0 is out already so I don't think it makes sense to rebuild this module for Angular 4.0
Check out the talk of Matias at the ng-conf:
https://www.youtube.com/watch?v=Oh9wj-1p2BM&t=12m20s


Pre-trained model

Thanks very much for your excellent work.
Can you provide the pre-trained model? I want to apply the model on my data without label for downstream task.
Much appreciate.

Hi @yyyzzzhao,
The pre-trained model is included within my Docker image which can be pulled from Docker Hub. You can use the model included in this Docker image for applying to your unlabeled data for your downstream tasks. The Docker image also ensures the correct environment and dependencies for the model to work properly.
To get the Docker image, you can follow the instructions on the official MICCAI FLARE22 website.

Thank you for your patient reply.


How to force EF to check Concurrency Token even if entity is not modified?

I need something like a shared lock to guarantee consistency. EF checks ConcurrencyToken of an entity only when it was modified. How can I force EF to check the concurrency token?

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

ID: 649fbab8-b960-6873-14bf-188d14757fee
Version Independent ID: f5ee3135-54c8-689c-e28b-b203b376bd6f
Content: Handling Concurrency Conflicts - EF Core
Content Source: entity-framework/core/saving/concurrency.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @rowanmiller
Microsoft Alias: avickers

See https://github.com/aspnet/EntityFrameworkCore/issues/18536#issuecomment-546514593


fix(Rx 4 and under): support routes returning older Rx observables

also refactors logic to convert values to observables

adds support for Symbol.observable observables (Most.js, etc.)

makes get, set and call return an Observable (Rx5) that may be subscribed to in the "old rx" format. That means you can subscribe with an { onNext(x) { } }-style observer, and you can unsubscribe with either dispose or unsubscribe.

related #193

cc/ @jhusain @lrowe


docs components input -> type is Password Reveal second click eye display error

Yep. I can confirm that. I will change this ASAP. Sorry about that.

Already fixed. Will be released on the very next fix version.
PS.: Thanks for this gif. Helped me a lot! :)


Fix spelling mistakes/typos found by CodeSpell 2.1.0

CodeSpell 2.1.0 found some spelling mistakes/typos, which I have corrected in this pull request.  They are all changes to comments.
CodeSpell is a free, open source tool for finding common English misspellings and typographic errors in source code. Information about it can be found here: CodeSpell on GitHub

Thanks @warmsocks! The spelling mistakes in the vendor files might reappear though when I'm merging their upstream files.


Wordy incorrect test

I'm making this an issue because I'm not sure if this was intentional or not, but the wordy_test.rb has the test:
def test_add_then_multiply
  question = 'What is -3 plus 7 multiplied by -2?'
  assert_equal(-8, WordProblem.new(question).answer)
end

my complaint is the following:
-3 + 7 * -2 = -17
(-3 + 7) * -2 = -8
if this was intentionally built this way, can something be put as a comment to the test, or in the readme?
otherwise change to -17?

Huh, yeah, that's no good. Let's change the answer to -17.

I'd love an explanation of this in the readme/test

Yeah, it should probably go in the README. I'll start a discussion about it with the track maintainers first, since other people may have other thoughts about this.

Is this a spoken word problem or a written word problem?  The spoken word problem can be manipulated by pausing for an answer at the first operation, which provides the parenthesis which enforces order of operations.  If it is written, then it is up to the reader to apply the order of operations (PEMDAS) as is appropriate and expected.

I see this as a spoken word problem.

Another bump, as the 'fix' is simple, but sometimes connecting with people is hard.

Should we ask forgiveness?
Update the readme/test comment, and if it is something that fundamentally
breaks the thinking, then there will be noise.

The README has been updated to reflect this. If there's noise we can discuss further.

Thanks, wanted to submit it both ways.

Ah, I missed that. I think of this as a word problem, so my bias made me think it was fixed :)

You said 'verbal' not wprd. O thought... ok, now we are both confused. :)  I submitted it both ways as in a test message for failure, but also the README solution.  Cover both bases.  I am pretty sure I communicated in both places that it was a verbal word problem.

LOL. Yes. Verbal. I'm so sorry. My brain was not functioning very well.


Dependency Conflict When Installing Unsloth with Torch 2.2.0

I'm using the following setup in my project:

PyTorch: 2.2.0
Python: 3.10
CUDA: 12.1.1

When running the command:
pip install "unsloth[cu121-ampere-torch220] @ git+https://github.com/unslothai/unsloth.git
I encountered the following error:
ERROR: Cannot install unsloth because these package versions have conflicting dependencies.

The conflict is caused by:
    bitsandbytes 0.43.3 depends on torch
    flash-attn 2.6.3 depends on torch
    xformers 0.0.24 depends on torch==2.2.0
    accelerate 0.34.1 depends on torch>=1.10.0
    peft 0.7.1 depends on torch>=1.13.0
    trl 0.7.9 depends on torch>=1.4.0
    unsloth-zoo 2024.10.1 depends on torch>=2.4.0
    bitsandbytes 0.43.3 depends on torch
    flash-attn 2.6.3 depends on torch
    xformers 0.0.24 depends on torch==2.2.0
    accelerate 0.34.1 depends on torch>=1.10.0
    peft 0.7.1 depends on torch>=1.13.0
    trl 0.7.9 depends on torch>=1.4.0
    unsloth-zoo 2024.10.0 depends on torch>=2.4.0

It seems like the issue started with this commit.
Issue
The root cause of the conflict is an inconsistency between the required versions of torch across various dependencies:
Some packages, like xformers 0.0.24, require a specific version of torch (e.g., torch==2.2.0).
Other packages, like unsloth-zoo 2024.10.1, require torch>=2.4.0, which is incompatible with the version 2.2.0 required by xformers.
This creates a version mismatch that prevents the dependencies from being resolved correctly during installation.
Workaround
I was able to resolve the conflict by installing the package using the following command:
pip install "unsloth[cu121-ampere-torch220] @ git+https://github.com/unslothai/unsloth.git@a2f4c9793ecf829ede2cb64f2ca7a909ce3b0884"
Please, let me know if there’s a better approach or if this can be addressed in a future update.

Oh would
pip install --no-deps "xformers<0.0.26" trl peft accelerate bitsandbytes
pip uninstall unsloth -y
pip install --upgrade --no-cache-dir "unsloth[colab-new] @ git+https://github.com/unslothai/unsloth.git"

work?

It doesn't work.
When I run the command:
pip install --upgrade --no-cache-dir "unsloth[colab-new] @ git+https://github.com/unslothai/unsloth.git"
I get this error:
Installing collected packages: sentencepiece, pytz, xxhash, unsloth, tzdata, triton, tqdm, shtab, safetensors, requests, regex, pyarrow, protobuf, propcache, nvidia-nccl-cu12, nvidia-cudnn-cu12, multidict, mdurl, hf-transfer, frozenlist, docstring-parser, dill, async-timeout, aiohappyeyeballs, yarl, pandas, multiprocess, markdown-it-py, huggingface-hub, aiosignal, torch, tokenizers, rich, aiohttp, tyro, transformers, datasets, trl, unsloth-zoo
Attempting uninstall: triton
Found existing installation: triton 2.2.0
Uninstalling triton-2.2.0:
Successfully uninstalled triton-2.2.0
Attempting uninstall: requests
Found existing installation: requests 2.31.0
Uninstalling requests-2.31.0:
Successfully uninstalled requests-2.31.0
Attempting uninstall: nvidia-nccl-cu12
Found existing installation: nvidia-nccl-cu12 2.19.3
Uninstalling nvidia-nccl-cu12-2.19.3:
Successfully uninstalled nvidia-nccl-cu12-2.19.3
Attempting uninstall: nvidia-cudnn-cu12
Found existing installation: nvidia-cudnn-cu12 <IP_ADDRESS>
Uninstalling nvidia-cudnn-cu12-<IP_ADDRESS>:
Successfully uninstalled nvidia-cudnn-cu12-<IP_ADDRESS>
Attempting uninstall: torch
Found existing installation: torch 2.2.0
Uninstalling torch-2.2.0:
Successfully uninstalled torch-2.2.0
Attempting uninstall: trl
Found existing installation: trl 0.11.4
Uninstalling trl-0.11.4:
Successfully uninstalled trl-0.11.4
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
torchaudio 2.2.0 requires torch==2.2.0, but you have torch 2.4.1 which is incompatible.
torchvision 0.17.0 requires torch==2.2.0, but you have torch 2.4.1 which is incompatible.
xformers 0.0.25.post1 requires torch==2.2.2, but you have torch 2.4.1 which is incompatible.
Successfully installed aiohappyeyeballs-2.4.3 aiohttp-3.10.10 aiosignal-1.3.1 async-timeout-4.0.3 datasets-3.0.1 dill-0.3.8 docstring-parser-0.16 frozenlist-1.4.1 hf-transfer-0.1.8 huggingface-hub-0.25.2 markdown-it-py-3.0.0 mdurl-0.1.2 multidict-6.1.0 multiprocess-0.70.16 nvidia-cudnn-cu12-<IP_ADDRESS> nvidia-nccl-cu12-2.20.5 pandas-2.2.3 propcache-0.2.0 protobuf-3.20.3 pyarrow-17.0.0 pytz-2024.2 regex-2024.9.11 requests-2.32.3 rich-13.9.2 safetensors-0.4.5 sentencepiece-0.2.0 shtab-1.7.1 tokenizers-0.19.1 torch-2.4.1 tqdm-4.66.5 transformers-4.44.2 triton-3.0.0 trl-0.11.1 tyro-0.8.12 tzdata-2024.2 unsloth-2024.10.0 unsloth-zoo-2024.10.1 xxhash-3.5.0 yarl-1.15.3

Same issues unfortunately

Tried many different variations on runpod, and none work

ERROR: pip's dependency resolver is not actually an error -- it's more of a warning about version mismatches. The broader issue here is that you have installed torch 2.4.1 (i.e., ...but you have torch 2.4.1...). Try doing this to fix it in that environment now:
pip install torch==2.2.0
In principle though, it might still work with 2.4.1 despite the "error."

Also from the readme :
https://github.com/unslothai/unsloth/blob/main/README.md#pip-installation
You can try:
pip install "unsloth[cu121-torch240] @ git+https://github.com/unslothai/unsloth.git"

Wait for torch 2.4 you'll need to use
pip install --no-deps xformers trl peft accelerate bitsandbytes
pip install unsloth
pip install --upgrade --no-cache-dir "unsloth[colab-new] @ git+https://github.com/unslothai/unsloth.git"

There isn't anything unsloth can do that you can't do by some other means. Unsloth is just another unnecessary wrapper/layer.  No magic beans no free lunch. Just look at the code.

There isn't anything unsloth can do that you can't do by some other means. Unsloth is just another unnecessary wrapper/layer. No magic beans no free lunch. Just look at the code.

Lol sure, good luck with that, tell us how it worked out for you

what dependencies are required for local use

@Ayush2006128 Doing pip install unsloth should auto install all dependencies

@Ayush2006128 Doing pip install unsloth should auto install all dependencies

I keep getting stuck in dependency hell.
The conflict is caused by:
unsloth-zoo 2024.12.1 depends on cut_cross_entropy
unsloth-zoo 2024.11.8 depends on cut_cross_entropy


Wiki Link Broken

Describe the problem
Here i read the new Wiki and the Link for the explaining of the Setting config.ini is missing
Driver version >= v1.0.0
Copy the values from /data/etc/dbus-serialbattery/config.default.ini to /data/etc/dbus-serialbattery/config.ini you want to change. All options can also be copied from [here]
Site does not exists: (https://github.com/Louisvdw/dbus-serialbattery/blob/jkbms_ble/etc/dbus-serialbattery/config.default.ini).
Driver version
Wiki
Venus OS device type
Cerbo GX
Venus OS version
v2.92
BMS type
Daly Smart BMS
Cell count
8s
Connection type
Serial USB adapter to RS485
Config file
Wiki Link Broken

Relevant log output
Wiki Link Broken

Any other information that may be helpful
No response

Thanks, I fixed it!


SAK-43472 SWITCH Assignments : use permissions webcomponent

This is the first on a series of jiras that will replace the original permissions screen with the webcomponents one. For that reason, I've created a common vm file/flow that can be resused by all the velocity tools. I've renamed/retagged the property files/keys instead of modifying the current behavior of the permissions webcomponent.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- velocity/tool/src/java/org/sakaiproject/cheftool/VelocityPortletPaneledAction.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- velocity/tool/src/java/org/sakaiproject/cheftool/VelocityPortletPaneledAction.java  1

See the complete overview on Codacy


We need instructions how to do a Custom signing key

There are no instructions linked how to do a custom signing key so we can test the claims transformations. The issue was closed over a year ago and needs to be revisited.

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

ID: 195c79a3-ccc5-8c6a-233e-95e344a9b4df
Version Independent ID: 401c7557-5817-bbd3-5bd8-8c97ef5a28cc
Content: Customize claims for an Azure AD tenant app (Public Preview) - Microsoft identity platform
Content Source: articles/active-directory/develop/active-directory-claims-mapping.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@mahoekst

By using new-azureadapplicationkeycredential you can do custom key signing by creating a symmetric signing key: https://docs.microsoft.com/en-us/powershell/module/Azuread/New-AzureADApplicationKeyCredential?view=azureadps-2.0
In addition to that, currently the MSAL library doesn't have any documentation on how to add the appID to the URL as a query parameter.
You should be able to use the  .WithExtraQueryParameter, see the wiki here on this info : https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/wiki/Client-Applications
This parameter will add the query to every request.
In the future there are plans to add more information to the wiki for this claims mapping policy. However as it is in preview this feature is not fully supported yet.
I will now be closing out this github issue,

we will go in and make those changes to the docs accordingly as well.

Is there meanwhile a documentation?


Update uris for formatting

Spaces in URLs is causing URI format validation errors in #2253

We need to update machine so that it decodes these URLs before sending them to the FTP client code we use.

I can do this.


Remove Upload And Replace Member Action

THe command is shown because of
{
"command": "vscode-ibmi-projectexplorer.uploadAndReplace",
"when": "view == projectExplorer && viewItem =~ /^member*/",
"group": "5_transfer@1"
},
in package.json
Not sure if I can make the "when" clause sensitive to whether we are in Merlin
@irfanshar Did we always want to remove this or only if we are in Merlin?

Not sure if I can make the "when" clause sensitive to whether we are in Merlin

@edmundreinhardt FYI, this is possible in fact by setting a custom context value when the extension is started. If this is needed, let me know and I can add the logic for this.

@SanjulaGanepola that would be very helpful.  The current action prompts for files from the Merlin container rather that workstation and so is not useful in Merlin


delete old/unused branches

I want to help! Perhaps we can delete branches from PRs that have merged.
What do you think? @saikat

Yeah, no reason not to I suppose, but I don't think their existence hurts us too much.  Are you in the Slack by the way?  I'm going to try to post some projects that we need help with in the next week in there!

(Going to close this issue though, but really glad you want to help and welcome aboard!)


Fix crash in invite collaborators

Verified that @varungu has signed the CLA. Thanks for the pull request!


No IIS bindings with host names were found

Hi, I am using an IIS server (windows server 2012R2) inside my LAN behind a fritzbox that uses myfritz.net (a dynDNS service) and redirects server.xyz.myfritz.net:7001 to my server.
As the site is an owncloud instance the certificate errors get very annoying. So I wanted to create a certificate but when I run the letsencrypt.exe I get
Scanning IIS Site Bindings for Hosts  No IIS bindings with host names were found ...
My bindings in the IIS settings for the site are:
https                                      443  *
https  server.xyz.myfritz.net  7001 *
How can I get this to work?

Since you haven't yet set up an SSL certificate you first need an unsecured host instance to work with.
So instead of using https you need to set http first. Though how it will cope with not using port 80 i'm not sure. I would hazard a guess that you might need to have a firewall rule setup to redirect port 7001 to port 80 for IIS.
Hopefully that should get it for you.

Ah thank you, once I added a binding for port 80 it poped up in the program. When I select that one though I get an error message that the ACME server was unable to reach the file. And sure if I open that url in the browser I get a 403 error.
Any idea how to fix that? Or how to use another port instead of 80? I did already add a port forwarding rule in my router for port 80 to 7001 on the server

#203 might resolve this without requiring an http host.


Configure ghost zone t-intersections

Is your feature request related to a problem?
This SRE issue indicates a performance bottleneck in AMR databases when creating ghost zones. Gunther reveled that the issue is related to ghost zones for t-intersections to produce crack-free isosurfaces. This appears to be showing up even if there is just one level in the AMR database.
Is your feature request specific to a data set?
The dataset has to be AMR data.
Describe the solution you'd like.
There is currently a command-line flag -disable-ghosts-for-t-intersections that turns this off and resolves the performance bottleneck. Gunther suggested an option in the Visit GUI menu (and possible revert to old behavior by default). The AMRStitchCell operator could detect the setting and print a warning message.
Basically, a user who wants crack-free isosurfaces have to explicitly set it.
Describe alternatives you've considered.
As described above, using the command-line flag is a path forward.

We should disable ghosts for t intersections, remove -disable-ghosts-for-t-intersections and add a new option -enable-ghosts-for-t-intersections. As far as adding it as a GUI option, we should see where the work is done. I believe it is at file open, so the user would probably have to reopen the file to have the change take effect.

We should disable ghosts for t intersections, remove -disable-ghosts-for-t-intersections and add a new option -enable-ghosts-for-t-intersections. As far as adding it as a GUI option, we should see where the work is done. I believe it is at file open, so the user would probably have to reopen the file to have the change take effect.


[2476] Refactor content status to service

Context
The course model is currently over 600 lines and has become a bit of a god class
Changes proposed in this pull request

refactor content status into a service object

Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Look at the course model to see what can be refactored into a service or concern


alt+number doesn't work in ncdc

To reproduce

install and run ncdc: https://dev.yorhel.nl/ncdc
/open some_ip_address. This puts you in another tab
press alt+1 to go back to the first tab

Expected behaviour
ncdc goes back to the first tab
Actual behaviour
ncdc does not go back to the first tab

This is identical behavior to both Xfce and Gnome terminal, both of which bind alt+N by default and require unbinding to get ncdc to work.

Both Xfce and Gnome terminal sends \e<number> when pressing Alt+<number> if you only have one tab open (and Xfce terminal sends it as long as the number doesn't match an existing tab).

I also tested xterm and xterm is converting alt+N to a Unicode character. I'm not sure how to disable this but ncdc also doesn't work by default in xterm.

xterm is an outlier here. It requires setting the metaSendsEscape option to have it send Alt+<key> as \e<key>. As far as I know, all other modern terminal emulators does this by default (at least on Linux).
Would you reconsider changing Ghosttys default behavior here? Perhaps to match Xfce or Gnome terminal. Another option could be using Ctrl+<number> to switch tab instead of Alt+<number>, since Ctrl+<number> can't be used by terminal applications unless using an alternative keyboard protocol.

This doesn't just affect ncdc btw, but also programs like WeeChat and Irssi.


Unhandled Exception: Exception: videoQuality cannot be empty

I am getting this error since yesterday.
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Exception: videoQuality cannot be empty
#0      PodPlayerController._checkAndWaitTillInitialized (package:pod_player/src/controllers/pod_player_controller.dart:73:9)
#1      PodPlayerController._checkAndWaitTillInitialized (package:pod_player/src/controllers/pod_player_controller.dart:82:11)
<asynchronous suspension>
#2      PodPlayerController.initialise (package:pod_player/src/controllers/pod_player_controller.dart:61:5)
<asynchronous suspension>

Flutter 3.22.3
Dart 3.4.4
pod_player: ^0.2.2
youtube_explode_dart: ^2.2.3
affected os both iOS and Android

I am currently experiencing an issue. Is anyone else facing the same problem or found a solution?

I also have this problem..

Same problem here

In youtube_explode_dart 2.2.2 it stopped working returning video's, probably YouTube changing something in their API. In the new version, youtube explode for some reason can't return "muxed" videos, therefore there are only video only and audio only links, and thus pod player not working.

Same issue here :(

Confirmed from youtube explode that the muxed videos are not avaible through youtubes api with the current stream client; this may or may not be fixed with youtube explode depending on a working stream client.
Pod player itself can change their player to handle youtube videos differently, with iframe example.
Currently there are 2 possible solutions:

Continue to use youtube explode and retrieve the video_only and audio_only files, combine them with FFMPEG and display them using a videoplayer (you can alter pod_player to play local video's)
Use https://pub.dev/packages/youtube_player_flutter, very easy to implement and works okay

Any solutions

Any solutions

Yes, actually I provided 2 above...

May i know what is muxed videos?

May i know what is muxed videos?

Muxed videos are complete videos, with audio and video combined.

the same error

Changed this. It's working for me.
`Stream _getStreams(VideoId videoId, {required bool fullManifest}) async* {
try {
  if (fullManifest) {
    await for (final stream
        in _getStream(videoId, VideoController.androidClient)) {
      yield stream;
    }
  }else{
    // Use await for instead of yield* to catch exceptions
    await for (final stream
    in _getStream(videoId, VideoController.iosClient)) {
      yield stream;
    }
  }
} on VideoUnplayableException catch (e) {
  if (e is VideoRequiresPurchaseException) {
    rethrow;
  }
  yield* _getCipherStream(videoId);
}

}
`
How to use
final manifest = await yt.videos.streamsClient.getManifest(youtubeIdOrUrl, fullManifest: true);
I have updated my app, you can check it out.
WeTube:Video&Music

For me it's work after:
flutter clean
flutter pub cache clean
flutter pub get

And :
flutter pub outdated
flutter pub upgrade

i'm also getting this error

i've found the solution changing the youtube client using YoutubeApiClient.androidVr or YoutubeApiClient.android is working

i've found the solution changing the youtube client using YoutubeApiClient.androidVr or YoutubeApiClient.android is working

Where do we change this??

i've found the solution changing the youtube client using YoutubeApiClient.androidVr or YoutubeApiClient.android is working

Where do we change this??

You can select which client to use when calling the stream.


Fix 2.6.x failing to compile on gcc 13.1.1 because of missing headers

I was getting this error until I added this header: /home/dogunbound/SFML/SFML/src/SFML/Audio/SoundFileReaderMp3.cpp:68:23: error: ‘uint64_t’ is not a member of ‘std’;
This is a fix
Thanks a lot for making a contribution to SFML! 🙂
Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs to ticked, but the more the better!)

[ ] Has this change been discussed on the forum or in an issue before?
[x] Does the code follow the SFML Code Style Guide?
[x] Have you provided some example/test code for your changes?
[ ] If you have additional steps which need to be performed list them as tasks!

Description
There is a bug where compile sfml 2.6.x fails because of missing stdint library. This pull request fixes this bug. This is probably a GCC v13 issue and probably why some have yet to encounter it.
Fix is just to include the header in this file.
https://github.com/SFML/SFML/issues/2552
Tasks

[x] Tested on Linux
[ ] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
Run this command to compile SFML 2.6.x:
cmake -B build -DCMAKE_BUILD_TYPE=Release
cmake --build build -j20
sudo cmake --install build

SFMLBuildMaster: Build this please.


Transaction filtered by an undefined payment_processor fails

File "/home/krotz/workspace/silver/silver/api/views.py", line 894, in get_queryset
    if not payment_method.payment_processor.transaction_class:
AttributeError: 'unicode' object has no attribute 'transaction_class'

Next time please provide the steps to reproduce as well, because the code changes in time.

This was probably fixed with https://github.com/PressLabs/silver/pull/423, as in it works now. But the parameter name needs to be changed from payment_method to payment_processor, because it's named wrong in the TransactionFilter.


[Forwardport] v2prov certificate rotation does not follow correct order of operations

Forwardport: https://github.com/rancher/rancher/issues/46034

Ticket #46434 - Test Results - ✅
Verified on Rancher v2.10-7f7897bcce2089c720c7a60846dd9fa4a7eb1d96-head:

Scenario
Test Case
Result

1.
Execute cert-rotation 3 times on downstream RKE2 w/ 3 etcd, 2 cp, 3 wkr
✅

2 .
Execute cert-rotation 3 times on downstream RKE2 w/ 1 etcd, 1 cp, 1 wkr
✅

3.
Execute cert-rotation 3 times on downstream K3s w/ 3 etcd, 2 cp, 3 wkr
✅

4 .
Execute cert-rotation 3 times on downstream K3s w/ 1 etcd, 1 cp, 1 wkr
✅

Scenario 1 - ✅

Fresh install of Rancher v2.10-head
Provision a downstream RKE2 cluster w/ k8s v1.30.4+rke2r1 - [3 etcd, 2 cp, 3 wkr]
Initiate cert-rotation + monitor logs
Verified - cert-rotation executed etcd pool => cp pool => wkr pool, and nodes were brought back up etcd pool => cp pool => wkr pool
Initiate cert-rotation 2 more times + monitor logs
Verified - cert-rotation order as expected each time; 3/3 pass

Scenario 2 - ✅

Fresh install of Rancher v2.10-head
Provision a downstream RKE2 cluster w/ k8s v1.30.4+rke2r1 - [1 etcd, 1 cp, 1 wkr]
Initiate cert-rotation + monitor logs
Verified - cert-rotation executed etcd pool => cp pool => wkr pool, and nodes were brought back up etcd pool => cp pool => wkr pool
Initiate cert-rotation 2 more times + monitor logs
Verified - cert-rotation order as expected each time; 3/3 pass

Scenario 3 - ✅

Fresh install of Rancher v2.10-head
Provision a downstream K3s cluster w/ k8s v1.30.4+k3s1 - [3 etcd, 2 cp, 3 wkr]
Initiate cert-rotation + monitor logs
Verified - cert-rotation executed etcd pool => cp pool => wkr pool, and nodes were brought back up etcd pool => cp pool => wkr pool
Initiate cert-rotation 2 more times + monitor logs
Verified - cert-rotation order as expected each time; 3/3 pass

⚠  Note: the following issue was encountered during my testing, https://github.com/rancher/rancher/issues/46992, but the correct order of operations were seen consistently, so validating this issue as resolved now 👍

Scenario 4 - ✅

Fresh install of Rancher v2.10-head
Provision a downstream K3s cluster w/ k8s v1.30.4+k3s1 - [1 etcd, 1 cp, 1 wkr]
Initiate cert-rotation + monitor logs
Verified - cert-rotation executed etcd pool => cp pool => wkr pool, and nodes were brought back up etcd pool => cp pool => wkr pool
Initiate cert-rotation 2 more times + monitor logs
Verified - cert-rotation order as expected each time; 3/3 pass


Cannot start platform, if etcd uses SSL

Due to the Security considerations  in our CoreOS cluster I configured etcd to use SSL and client certificates, it works fine, but I cannot make deisctl to start platform.
deisctl --etcd-cafile=/path/to/ca.pem --etcd-certfile=/path/to/cert.pem --etcd-keyfile=/path/to/key.pem --endpoint=https://<IP_ADDRESS>:4001 start platform
Configuring, listing, installing worked, but starting did not.
I have checked logs and the Unit Files (eg: deis-store-monitor.service), and I think it hangs at ExecStartPre=/bin/sh -c "etcdctl set /deis/store/hosts/$COREOS_PRIVATE_IPV4 hostname >/dev/null" because etcdctl cannot access etcd, and I have no idea how to tell etcdctl to use client certificate.
I would appreciate any idea, thanks.

I think you'd have to find any instance of etcdctl in any of our scripts and add the ca-file, cert-file, and key-file flags. Because many of our components use etcdctl inside their /bin/boot scripts, I'd think that you'd need to custom-build the components themselves and somehow pass through your cert and key, maybe bind-mounting the file in the container.
Simply put, it's a large undertaking. Thinking off the top of my head, a reasonable approach would be to propose a PR that makes all uses of etcdctl use an environment variable called $ETCD_OPTS or something, which could be passed into the running container. So, for example, the publish loop in a component changes from:
until etcdctl --no-sync -C $ETCD ls >/dev/null 2>&1; do

to
until etcdctl $ETCD_OPTS --no-sync -C $ETCD ls >/dev/null 2>&1; do

It's fine if it's blank, but users could add additional options.

Thanks for the reply, I will think it over next week.
As I see, there are three parts of the task.

UnitFiles and Dockerfiles are supposed to use newer version of etcdctl which already knows the cert options (0.4.6 does not know these options).
Implement $ETCD_OPTS you advised
This is the most difficult to solve simply: the cacert, the cert and the key must be accessible by the containers. Bind-mounting, or generate certs in the Dockerfiles. I will examine the possibilities. If it works I will make a PR.

This is the most difficult to solve simply: the cacert, the cert and the key must be accessible by the containers.

Agreed. I'm not sure how best to achieve this. Maybe we bind-mount /etc/certs or some path on the host into the container, but this starts to go down the route of specific functionality that most Deis users probably wouldn't use.

UnitFiles and Dockerfiles are supposed to use newer version of etcdctl which already knows the cert options (0.4.6 does not know these options).

I think using etcd version differs from the main version used by CoreOS hosts or by Deis containers cannot be maintainable, so a correct and useful PR could be proposed after upgrade of etcd at least v2.0 in the main entities. If this upgrade happens, I will make a PR. Do you think, if it happens in this spring?

but this starts to go down the route of specific functionality that most Deis users probably wouldn't use.

I think, in production enviroment the access to the etcd must be restricted. Using client certificates may be easier than maintain iptables continuously.

Do you think, if it happens in this spring?

We have no idea when CoreOS will release etcd 2.0 into the main release channels.

I think, in production enviroment the access to the etcd must be restricted

We already do that. etcd is currently binding to the private address space on boot such that users external to the cluster cannot access etcd through the node's public IP address, which is why @carmstrong thinks that most users probably wouldn't use this unless they were somehow exposing etcd to the world.

It also depends on how paranoid you are about your cloud provider's internal security. Case in point: Google now HTTPS encrypts traffic all the way through:
http://techcrunch.com/2014/03/20/gmail-traffic-between-google-servers-now-encrypted-to-thwart-nsa-snooping/
It really depends on how much you trust your cloud provider and the security of their network topologies.

I think it is not about the cloud provider's network topologies. For example: I run a wordpress container. WP has a security hole, the bot comes and uploads a php file with a system call inside: etcdctl ls --recursive. The container can reach the private network, so the attacker can read the whole etcd, can't he? Or do I misunderstand something?

that loophole will get closed up via #986, but essentially yes, the container can reach the private network so long as the attacker knows of a way to get the container's host IP address.

Arguably, the attacker only needs to know <IP_ADDRESS>

there is no such thing as a "private" network, unless you're talking about privately monitored networks. I monitor my own subnets but I don't monitor beyond that. Just because you monitor it, doesn't mean you can trust it. All your hosts can and will be compromised. Chat with etsy, they're much better at explaining ramifications of monitoring vs protection than I.

@wenzowski, the mention of <IP_ADDRESS> was with regard to the default docker0 interface IP address on all CoreOS hosts. Also, that etcd by default binds to INADDR_ANY (<IP_ADDRESS>) port 4001, and is reachable from that <IP_ADDRESS>/16 network that is used by all docker containers on a CoreOS host. Meaning, after you've gotten a toehold into your docker container that is your app (via the HTTP traffic that is routed to it), you have carte blanche to <IP_ADDRESS>:4001, which is where the OP of this issue comes into play: using SSL certs restricts access to the etcd instances over the network in addition to enabling encryption of the traffic between them.

Yep exactly @ianblenke thanks for spelling that out. All it takes is one toehold. My point is that even with #986 this will still be a concern. Given this, encrypting etcd traffic seems very important: we should all probably be doing what OP is.

I think there might be an easier (and maybe a better) option: #3572

etcd will be going away in v2 and we won't be able to get to this for the LTS release so I'm going to close this, but thank you very much for posting this :)

It is fine, I am looking forward v2 ;)


Red Wine Quality Prediction

Have you read the Contributing Guidelines ?
Yes
Description
I have added the notebook which predicts the quality of red wine.
Fixes #820

Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
I have tried my best to do my work. Please review it.

@Kushal997-das I have added the project in basic folder. Please review it.


[zh-cn] sync cheatsheet.md and extensible-admission-controllers.md and abac.md

sync cheatsheet.md and extensible-admission-controllers.md and abac.md

/approve


Vip weight regression

JIRA: https://jira.d2iq.com/browse/COPS-6002
DCOS PR: https://github.com/dcos/dcos/pull/7186

It looks good to me, but I haven't seen the part that sets the weight to 0. Is this coming from another component?

So for this branch (1.13.x) it was already merged in https://github.com/dcos/dcos-net/pull/194 and for master in https://github.com/dcos/dcos-net/pull/193. It never made it into 2.0.x, I just reused that PR to also add the fix ;)


2 stems 22khz dont work but 16 kHz vers good

Hi @expectopatronum
Thanks for your questions. I'll try to answer them:

First of all, why use 41000 in the config (for loading/training?) when the model ins only trained to 11k?

This is two different things. The spectrogram model is trained until 11kHz meaning input spectrograms are cut at 11kHz, but separation can be performed until 22kHz, either by increasing the value of F at test time (this is exactly what spleeter:stems-16kHz does) up to 2048 or by extending the mask averaging (using the "average" option for "mask_extension" as you mentioned). Note that models were trained with 44.1KHz, and changing the sampling rate at test time may lead to lower separation performances.

In the FAQs you mention different configurations, but separator = Separator('spleeter:5stems-16khz') fails with spleeter.SpleeterError: No embedded configuration 5stems-16khz found. Should this work or does it only work with the command line interface?

separator = Separator('spleeter:5stems-16khz') should work but if you have a quite old version of Spleeter. I've just tested it with the most recent version and it works on my side.

Could you add an example of how to use spleeter in Python? It took me a while to figure out how to load the data to work with separator.separate(...) (and maybe it's still wrong - see below)

To perform separation in python, either you do it at the file level using separator.separate_to_file (that behaves quite similarly to the command line) or you process directly waveforms as numpy arrays with separator.separate. The wiki provides sample code for separator.separate usage.

Is the following the correct way of doing it? (Apparently not, but I don't know what else to change)

See 3.

Using "mask_extension":"average", you should have separated signals that sum up to the mix. For older versions of spleeter (i.e. <=1.5.3), you may have differences at the very beginning or the very end of the signal due to the way STFT/iSTFT were managed, but that should be fixed from 1.5.4. Let us know if you still have troubles with recent versions.
Originally posted by @romi1502 in https://github.com/deezer/spleeter/issues/503#issuecomment-706243455

Where we can get version 1.5.4 please

Where we can get version 1.5.4 please

Hi @lesecs,
Note that the last version of spleeter (2.0.1) should not suffer from the reported issue.
If you need to install 1.5.4, you can install it with pip (provided you already have ffmpeg installed):
pip install spleeter==1.5.4

Note that, 1.5.4 is only compatible with python 3.6 and 3.7 (not 3.8).

Hi
When i try seaprate at 22khz, no folder is found. 16 kHz, is very good. I dont Knowl why
Tx

Hi
When i try seaprate at 22khz, no folder is found. 16 kHz, is very good. I dont Knowl why
Tx

I havé ffmpeg installer but pip installer spleeter==1.5.4 canot install


Use block_in_place for synchronous Rocksdb operations

This PR wraps all Rocksdb I/O operations in fedimint_api::task::block_in_place so that the executor thread is not blocked on I/O. Rocksdb does not do I/O during writes (it always writes to memory or it fails), but insert/remove operations return the previous value, so they have a chance of doing I/O while reading for the previous value.
Issue: https://github.com/fedimint/fedimint/issues/1528

@dpc what are your thoughts on how to validate that this unblocks the executor? I'm not sure how much overhead block_in_place adds and if the channel implementation that you mentioned here is superior.

I'm not sure how much overhead block_in_place adds and if the channel implementation that you mentioned https://github.com/fedimint/fedimint/issues/1528 is superior.

Under the hood block_in_place uses a threadpool and sends it stuff over channel. So it is probably the same. If everything works and there is no lifetime issues, go ahead.


Add --wait switch on riff function create

Fixes https://github.com/projectriff/riff/issues/682

If this works out, we can do the same for other places where --verbose is supported.

Merged as 491d1094


💡 [FEATURE] - <BACK TO TOP BUTTON>

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
Adding a back to  top button on bottom of the web page
Explain your solution
In an effort to improve user experience on our website,  "Back to Top" button. This feature will enhance navigation, especially on pages with extensive content, by allowing users to quickly return to the top of the page with a single click.
Any alternative approaches/features
Back to top button
Additional Context
It will increase user experience

Hello, I'm interested in working on this issue. Please assign it to me.

duplicate issue #699


Remove "CCleaner" tag from CCEnhancer

Currently, if a user attempts to run "winget install ccleaner", they will be blocked from installing CCleaner as a result of CCEnhancer having the CCleaner tag. This PR removes the CCleaner tag from CCEnhancer, thus resolving the issue.
Microsoft Reviewers: Open in CodeFlow

I am not in favor of this change
same issue is described for common packages like Git:

see https://github.com/microsoft/winget-cli/issues/292 and matching spec Package matching bug fix
like relayed https://github.com/microsoft/winget-cli/issues/292#issuecomment-638831069, it is programed for next milestone of WinGet = Package Manager v0.2 (due by June 30, 2020)

and here, for the CCEnhancer package, the "ccleaner" tag is clearly good to be present


The nutri-score v2 moderator only knowledge panel is not visible anymore

What

The nutri-score v2 moderator only knowledge panel is not visible anymore

@teolemon Probably linked to the fact that the cookie is null.

@stephanegigandet We expect the cookie in headers?['set-cookie']  after a call to auth.pl.
Something has changed recently?

it's weird, because I believe nothing has changed server side, this API is superrecent.
@monsieurtanuki did you ever manage to see the new Nutri-Score on products ?

it's weird, because I believe nothing has changed server side, this API is superrecent.

Could be a merge issue in off-dart. In my latest PR, I rebuilt the generated files as needed, and found out that the "user" generated file changed, unexpectedly. I'll have a look at it today.

@monsieurtanuki did you ever manage to see the new Nutri-Score on products ?

Never, but as I currently struggle to even run the app due to "macbook/android emulator/api 34" BS I strongly limit my use of the app.

@teolemon @stephanegigandet We're supposed to get a "cookie" somewhere, but that's not there.
Response body:
{
  "status":1,
  "status_verbose":"user signed-in",
  "user":{
    "admin":0,
    "country":"",
    "moderator":0,
    "name":"the user name",
    "preferred_language":"en"
  },
  "user_id":"grumpf"
}

Response header:
{
  connection: keep-alive,
  access-control-allow-origin: *,
  transfer-encoding: chunked,
  date: Mon, 06 May 2024 15:40:35 GMT,
  strict-transport-security: max-age=63072000,
  access-control-expose-headers: Content-Length,Content-Range,
  content-type: application/json; charset=utf-8,
  x-xss-protection: 1; mode=block,
  server: nginx/1.18.0,
  x-request-id: 5gj3d4KDRYZYrtyV,
  access-control-allow-headers: DNT,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type,Range,If-None-Match,Authorization,
  access-control-allow-methods: HEAD, GET, PATCH, POST, PUT, OPTIONS,
  x-download-options: noopen,
  x-content-type-options: nosniff,nosniff
}

Our input is a POST to cgi/auth.pl, with

header: {Accept: application/json, User-Agent:  - off-dart integration tests, From: grumpf}
body: {body: 1, comment: dart API test, user_id: grumpf, password: ****, app_name: off-dart integration tests}

Discussed yesterday evening with @stephanegigandet , and nothing has changed server side

@teolemon I've just had a look at @g123k's related PRs, and the files are 99% untouched:

https://github.com/openfoodfacts/openfoodfacts-dart/pull/914: ok
https://github.com/openfoodfacts/openfoodfacts-dart/pull/916: ok
https://github.com/openfoodfacts/openfoodfacts-dart/pull/911: slightly different for generated file user.g.dart, because the json related build tool was not run after changes to user.dart

I've never seen the new nutriscore logos in the app.
Could it be that the cookie feature was only tested on emulators, and not on devices?

I'm sorry I'm abroad and don't have access to a computer.
But as a reminder, the server was not deployed and switching to DEV was mandatory

@monsieurtanuki @g123k DEV does not improve the shebang: it's already been deployed to .org if you're logged in: https://fr.openfoodfacts.org/produit/4613448779566/chocapic#

@teolemon @stephanegigandet Regardless of the new nutriscore being deployed in DEV, TEST or PROD, our problem is that cookies are NOT returned by the PROD server through auth.pl.
But they are by the TEST server, something like:
set-cookie: session=user_id&openfoodfacts-dart&user_session&NYF1KHwKQe71Wce7EvcJepGqN9lm1cQJzSP52OcnUj5i4n2hnTm6kHJsN0C4hnkW; domain=.openfoodfacts.net; path=/; SameSite=Lax
So as long as we don't get cookie, we won't send them back, and we won't get new nutriscore depending on the sent cookie.
If it's not a matter of code difference between PROD and TEST, maybe it's a server parameter. Worst case scenario: a server parameter that cannot be set to this value in production.

@monsieurtanuki the auth.pl cookie is not in prod in .org yet, it will be deployed early this week.

It's ok for me after a sign-out/log-in:

Could you confirm this works for you?

I don't know if I'm supposed to see it, but anyway I don't see the new formula.

@monsieurtanuki are you using .net, and is your country set to Germany ?

(you are a moderator anyway)

(but @stephanegigandet made the Nutri-Score generally available outside France on .net)

I've just tested successfully with .net, Andorra and English:

Ok, so perfect.
So I guess we can close this issue, isn't it?

Indeed


fix: pipeline beam search error when k=1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 96.099%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
468

Relevant Lines:
487

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 96.715%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
471

Relevant Lines:
487

💛  - Coveralls


Data().Scenario() ends immediately when populating DataTable with async data

What are you trying to achieve?
Data driven scenario that uses randomly generated test data from an asynchronous call to a third party (Mockaroo).
What do you get instead?
Data(...).Scenario(...) ends immediately.
> codeceptjs run --steps

CodeceptJS v1.3.2
Using test root "C:\path\to\repo"

  OK  | 0 passed   // 0ms
{ id: 1,
  FirstName: 'Graig',
  LastName: 'Wellfare',
  Email<EMAIL_ADDRESS>  Password: 'P@$$w0rd!',
  PhoneType: 'Work',
  PhoneNumber: '425-555-1212',
  EmailOptedIn: false,
  TextOptedIn: false,
  CreatedIn: 'dev-us',
  AddressLine1: '2 Valley Edge Pass',
  City: 'Long Beach',
  State: 'California',
  PostalCode: '90847',
  PetName: 'Arnoldo',
  Birthday: '01/21/1975',
  AdoptionDay: '09/30/2011',
  Species: 'dog',
  Gender: 'Male',
  SpayedNeutered: false,
  isAdopted: false,
  PrimaryBreed: 'Japanese Chin',
  isMixedBreed: true,
  Weight: 199,
  Color: 'Mahogany' }
PS C:\path\to\repo>

Provide test source code if related

/// <reference path="./steps.d.ts" />

var Mockaroo = require('mockaroo');

var accounts = new DataTable([
    'id',
    'FirstName',
    'LastName',
    'Email',
    'Password',
    'PhoneType',
    'PhoneNumber',
    'EmailOptedIn',
    'TextOptedIn',
    'CreatedIn',
    'AddressLine1',
    'AddressLine2',
    'City',
    'State',
    'PostalCode',
    'PetName',
    'Birthday',
    'AdoptionDay',
    'Species',
    'Gender',
    'SpayedNeutered',
    'isAdopted',
    'PrimaryBreed',
    'isMixedBreed',
    'Weight',
    'Color'
]);

var client = new Mockaroo.Client({
    apiKey: '1a96a8a0'
});

client.generate({
    count: 1,
    schema: 'my-saved-schema'
}).then(function(records) {
    console.log(records);

    for (var i = 0; i < records.length; i++) {
        accounts.add([
            records[i].id,
            records[i].FirstName,
            records[i].LastName,
            records[i].Email,
            records[i].Password,
            records[i].PhoneType,
            records[i].PhoneNumber,
            records[i].EmailOptedIn,
            records[i].TextOptedIn,
            records[i].CreatedIn,
            records[i].AddressLine1,
            records[i].AddressLine2,
            records[i].City,
            records[i].State,
            records[i].PostalCode,
            records[i].PetName,
            records[i].Birthday,
            records[i].AdoptionDay,
            records[i].Species,
            records[i].Gender,
            records[i].SpayedNeutered,
            records[i].isAdopted,
            records[i].PrimaryBreed,
            records[i].isMixedBreed,
            records[i].Weight,
            records[i].Color
        ]);
    }
});

function sleep(ms) {
    return new Promise(resolve => {
        return setTimeout(resolve, ms);
    });
}

Feature('Account');

BeforeSuite(async(I) => {
    await sleep(5000);
});

// This ends immediately...
Data(accounts)
.Scenario('test something', (I,current) => {
// ... but comment out previous 2 lines and uncomment next line and delay works...
// Scenario('test something', (I) => {
    I.amOnPage('https://www.google.com');  // <-- starts after 5 seconds for Scenario() but not for Data().Scenario()
});

Details

CodeceptJS version: "^1.3.2"
NodeJS Version: v10.6.0
Operating System:

Edition         Windows 10 Home
Version         1803
Installed on    5/17/2018
OS build        17134.165

Configuration file:

{
  "name": "codeceptjs-mockaroo-test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "postinstall": "selenium-standalone install",
    "start": "selenium-standalone start",
    "test": ".\\node_modules\\.bin\\codeceptjs run --steps"
  },
  "keywords": [],
  "author": "Random QA Tester<EMAIL_ADDRESS>  "license": "MIT",
  "dependencies": {
    "codeceptjs": "^1.3.2",
    "mockaroo": "^0.1.7",
    "selenium-standalone": "^6.15.1"
  }
}

@DavertMik - Here is the issue you requested I create.

var Mockaroo = require('mockaroo')

var accounts = new DataTable([
    'id',
    'FirstName',
    'LastName',
    'Email',
    'Password',
    'PhoneType',
    'PhoneNumber',
    'EmailOptedIn',
    'TextOptedIn',
    'CreatedIn',
    'AddressLine1',
    // 'AddressLine2',
    'City',
    'State',
    'PostalCode',
    'PetName',
    'Birthday',
    'AdoptionDay',
    'Species',
    'Gender',
    'SpayedNeutered',
    'isAdopted',
    'PrimaryBreed',
    'isMixedBreed',
    'Weight',
    'Color'
])
var client = new Mockaroo.Client({
    apiKey: '1a96a8a0'
})

var XMLHttpRequest = require("xmlhttprequest").XMLHttpRequest;
var request = new XMLHttpRequest();
request.open("POST", `https://mockaroo.com/api/generate.json?client=node&key=1a96a8a0&count=3&schema=my-saved-schema`, false)
request.send()
let records = JSON.parse(request.responseText)

for (var i = 0; i < records.length; i++) {
    accounts.add([
        records[i].id,
        records[i].FirstName,
        records[i].LastName,
        records[i].Email,
        records[i].Password,
        records[i].PhoneType,
        records[i].PhoneNumber,
        records[i].EmailOptedIn,
        records[i].TextOptedIn,
        records[i].CreatedIn,
        records[i].AddressLine1,
        // records[i].AddressLine2,
        records[i].City,
        records[i].State,
        records[i].PostalCode,
        records[i].PetName,
        records[i].Birthday,
        records[i].AdoptionDay,
        records[i].Species,
        records[i].Gender,
        records[i].SpayedNeutered,
        records[i].isAdopted,
        records[i].PrimaryBreed,
        records[i].isMixedBreed,
        records[i].Weight,
        records[i].Color
    ])
}
Data(accounts)
    .Scenario('use async data', async (I, current) => {
        console.log("current.id =" + current.id);
    })

npm i  xmlhttprequest -D

It's a workaround for that specific use case for not a general solution for async data driven tests. Any news on this?

I also have the same problem. I can't manage to use async test data into the Data() provider. Any solution would be greatly appreciated.

I am trying to do the same thing, I'd love to feed Data() using the DbHelper but I can't seem to get it to handle async properly.

Hi! I'm a security researcher @TruffleSecurity. I found a valid API key leaked in a comment in this Issue. I highly recommend rotating the API key immediately (not just deleting the comment). Here's a blog post I wrote on remediating these types of vulnerabilities: https://trufflesecurity.com/blog/remediate-leaked-api-keys-with-key-rotation.If you can't find the leaked API key, you can use the following TruffleHog command to automatically scan for it:trufflehog github --repo https://github.com/<your_username>/<your_reponame> --issue-comments --only-verifiedIf it's not clear where/what the security issue is, please let me know. I'd be happy to help. Please note that the API key might be in the past edit of a comment.(cc @wackydude)


Please add 'Backup power SOC' as control parameter

What would you like to be added?
Please add 'Backup power SOC' as parameter to read and write
Why is this needed?
In systems, where a battery and a backup box are installed, the 'Backup power SOC' defines down to what battery minimum level the consumption is limited in case of normal usage. The 'Backup power SOC' is hence the reserve left in the battery in case the system switches to backup power because of a power outage.
Being able to set this parameter allows the user to define for their own usage scenario how much of the battery is reserve. As this might depend on weather forecast etc, it is a parameter that will be set more frequently when optimising the battery usage.

Hi Max,
Do you mean the register address 47102?

Hi, yes, indeed, this is what I mean.

Hi Max, dein issue ist umgesetzt.
LG Stephan

Hi Stephan, vielen Dank! Gestestet, funktioniert.
LG Max


Mention the PATH where dotnet is installed.

I manage my $PATH, so maybe that is why I see "-bash: dotnet: command not found". Docs should provide info about the implicit path used.

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

ID: d49bc05d-7c8e-5f89-aac5-d133225da4d5
Version Independent ID: 2f831fc9-dfe6-15db-4ec6-00f59c2dacc2
Content: .NET Core Guide
Content Source: docs/core/index.md
Product: dotnet-core
GitHub Login: @richlander
Microsoft Alias: mairaw

I found some wisdom: https://wilsonmar.github.io/powershell-on-mac/

Hi!
I don't think this article is the appropriate one to document this. However, I think the root Getting Started article would be better. Would you like me to move your issue to that article?

If we document this, I'd suggest adding this to the main dotnet topic instead: https://docs.microsoft.com/en-gb/dotnet/core/tools/dotnet
This is not a Getting started info.


Icons for auto-detected files disappear after reloading

Housekeeping

[X] I'm sure this issue is not a duplicate

Information:
As you can see in the gif, the .clang-format file is always displayed correctly while the .clang-tidy and .cmake-format files (also YAML) are given the right icons only after opening (i.e., after language detection). But when you reload the window, the icons disappear again.

Potentially related: #3303
I've noticed with .module.ts icons if I restart VSCode they'll display properly (they're supposed to show special NestJS icons), but after some amount of time, or maybe relaunching a project they revert back to plain .ts icons.

Do your icons fix if you restart VSCode?
I tried switching my icon theme around thinking maybe that could fix it, but it does not. Neither does restarting the extension host.

I just ran across this documentation which seems odd and dated, maybe forgotten:

https://github.com/vscode-icons/vscode-icons/wiki/Pad

I just ran from the command palette: Icons: Reset Project Detection Defaults and now my icons are displaying properly. Maybe some code needs to get re-ran upon window reloads or something? 🤔  Or maybe this feature is off by default and needs to be turned on, but seems to maybe initialize at some point?

The Icons: Reset Project Detection Defaults command doesn't help, nor does restarting or reloading.


Internet Explorer Playlist Items

Legacy IE is not rendering the expected playlist items based on the data-youplay-playlist-item template.

Resolved with commit 16229a3.


Wrong timestamp for last recorded frame

Required Info

Camera Model
D435 / L515

Firmware Version
<IP_ADDRESS> / <IP_ADDRESS>

Operating System & Version
Linux (Ubuntu 18.04) / MacOS 10.15.7

Kernel Version (Linux Only)
<IP_ADDRESS>

Platform
PC

SDK Version
2.40.0

Language
C++

Segment

I'm recording to a bag file with the rs2::pipeline API. The last frame gets recorded with a wrong  timestamp and this creates an incorrect duration. While reading the bag file, the reader code (including realsense-viewer) waits for ever or crashes. This happens on both platforms listed above (macOS with L515 and Ubuntu 18.04 with D435).

The debug logs created with LRS_LOG_LEVEL=DEBUG are attached. There is a big jump in the timestamp for the last frame in the recording logs.
Playback_2020-12-18-15_19_17.log.zip
Recording_2020-12-18-15_11_26.log.zip
Is there any way to mitigate this, such as not writing the last frame, or fixing the last frame timestamp? Here is what hasn't worked:

Pausing recording some time before stopping the pipeline.
Re-starting and re-stopping the pipeline after stopping it.

This error happens if pause() recording is called very soon after pipeline::start().

Hi @ssheorey
In looking at this further, I think what is happening is the pause function is pausing the recording but not stopping the streams from continuing which could lead to a change in timestamp when recording is resumed.
Took a look at recording via the rs-recorder as well as the recording function inside realsense-viewer and I don't see this issue.
*If we don’t hear from you in 7 days, this issue will be closed.

Hi @ssheorey
In looking at this further, I think what is happening is the pause function is pausing the recording but not stopping the streams from continuing which could lead to a change in timestamp when recording is resumed.
Took a look at recording via the rs-recorder as well as the recording function inside realsense-viewer and I don't see this issue.
*If we don’t hear from you in 7 days, this issue will be closed.

Thanks for looking into this. My earlier code called pipeline::start() immediately followed by pause() to wait for the user to request start recording. I wanted to retrieve frames for display while recording was paused. The timestamp issue only affects the last frame recorded, which is strange, and the last timestamp actually looks to be a junk value (e.g. 5 years in the future), not what you would get if the clock had just continued running. This pattern isn't used in rs-recorder so the issues doesn't appear.
After a lot of trial and error, my work around was to create a new pipeline when I needed to start recording, instead of keeping recording paused().

Thanks for looking into this. My earlier code called pipeline::start() immediately followed by pause() to wait for the user to request start recording. I wanted to retrieve frames for display while recording was paused. The timestamp issue only affects the last frame recorded, which is strange, and the last timestamp actually looks to be a junk value (e.g. 5 years in the future), not what you would get if the clock had just continued running. This pattern isn't used in rs-recorder so the issues doesn't appear.
After a lot of trial and error, my work around was to create a new pipeline when I needed to start recording, instead of keeping recording paused().

Thanks and glad that you were able to determine a workaround for this.
Is there anything else you need here?
*If we don’t hear from you in 7 days, this issue will be closed.

Thanks and glad that you were able to determine a workaround for this.
Is there anything else you need here?
*If we don’t hear from you in 7 days, this issue will be closed.

hello @ssheorey
I tried to reproduce the issue with the flow you described without success , can you please attach a code snippet from your application that reproduce this issue?


added elasticIP to services instance

Hey, thanks a lot for the PR! I think this likely makes sense, but there might be one or two other things that need to be tweaked to accommodate it. At a minium, https://github.com/circleci/enterprise-setup/blob/master/circleci.tf#L437 should refer to the EIP. I'll try to review more completely soon!

@bellkev fair enough, change made. :-)

this a really helpful PR that we should incorporate into master if we can, or just add elastic IP to the services box ourselves & close this PR