Fixed 009 - Moved all Word documents to mark down

In depth review please.

Links to challenge MD files from the main TOC are broken.  (no hyphen in the links)
Same problem with links in Coach Guide TOC.
Why are the two root readmes all caps?  I know we're not consistent, but the majority seem to be lowercase.  I think that should be our standard for new readme files going forward.
You created the /Student/Resources folder.   But in Coach, there is no "Solutions" folder.  Consider putting the /Coach/Code/ and /Coach/Scripts folder in a new /Coach/Solutions/ folder?  (I assume this would make more links have to be updated. However, this would be a low priority fix IMO.
On the hack homepage, you left the "(Optional)" label on "Repository Contents" header!

Looks great overall!!!  Thanks for the updates!

Here's another comment.  Does this count as a review?

@jrzyshr The standard the world over for readme files is all CAPS.... just look at what you get when you create a new repo in GitHub. Comes from an ancient standard in Unix and C programming having "README" files (no extension) for calling people's attention.
We should flag all PRs that submit readme's that aren't NOT all caps

@jrzyshr Look at the latest two commits for your fixes.


module load exception too generic

Rasa NLU version: 0.13.7
Operating system (windows, osx, ...): macOS High Sierra (10.13.4)
Content of model configuration file:
language: en
pipeline:
- name: "path.to.my.EntityRecognizer"
- name: "intent_classifier_sklearn"

Same path was specified in the class itself.
Issue:
I get
Exception: Failed to find component class for 'path.to.my.EntityRecognizer'. Unknown component name. Check your configured pipeline and make sure the mentioned component is not misspelled. If you are creating your own component, make sure it is either listed as part of the `component_classes` in `rasa_nlu.registry.py` or is a proper name of a class in a module.

while running
python -m rasa_nlu.train --config nlu_config.yml --data training_data/ --path ./
This error message is not correct however and is misleading as I could validate that in my case the correct path is actually being picked up by rasa, but this message is being shown because exception catch in line 111 in rasa_nlu.registry is too generic so it picks up other exceptions happening on load of module.

Thanks for raising this issue, @wochinge will get back to you about it soon.

@yousrahmoh
I tried it out myself and the class loading form the string works fine.
Can you provide your directory structure, so that I can check whether your path is right?
Note that your current pipeline makes no sense in general, as you first have to apply a featurizer before you can apply all the other components (see the description part of https://rasa.com/docs/nlu/components/#intent-classifier-sklearn)

@wochinge
The issue happens when your code has a problem on import of other modules. The issue is emanating from the use of except Exception which is used on line 111 in rasa_nlu.registry. This is generally discouraged in python as the above issue will happen; This exception will fire on behalf of all other failures in your code and the real problem is lost in the misleading error message telling you to double check the path to your custom component. Exceptions need to be more specific.
On your other note, I currently have the pipeline working fine. The EntityRecognizer is doing the featurization within.

@yousrahmoh
I tried it out myself and the class loading form the string works fine.
Can you provide your directory structure, so that I can check whether your path is right?
Note that your current pipeline makes no sense in general, as you first have to apply a featurizer before you can apply all the other components (see the description part of https://rasa.com/docs/nlu/components/#intent-classifier-sklearn)

Seems to me like the more specific exceptions should be KeyError for when file is not found in globals and attribute error in case it's a path.

@yousrahmoh You are right! I see now what you mean! We will improve this (you are also welcome to contribute a pull request for that 🙂 )

@erohmensing what? :D can you move it back to maintenance in the sprint board

Hahah sorry I accidentally closed this when I was having sprint board issues 😅didn't realize it wouldn't go back to maintenance when i re-opened it!

I want to work on this

Awesome @hsm207 . Let us know if you need feedback / help 🚀

@wochinge There is no unit test to this functionality of get_component_class, is there?

@wochinge There is no unit test to test this functionality of get_component_class, is there?

Check out test_components. There are tests (e.g. test_builder_create_by_module_path) which test create_component with custom modules, which then should call the get_component_class

@wochinge I've made PR #4166. Please let me know if I should change something.


Read options from a file

Currently the only way to pass options (including password) to librespot is via command line. This means that username and password show up in a process list, which makes my eyes bleed.
Simple feature such as reading options from a file is much appreciated.

If you define a cache folder to be used, librespot would write out credentials to a credentials.json file.


Milan - Sending IB email for translation

Should, "P" in Forex Micro pairs be uppercase?

Content is written as small and the website is showing as well Micro pairs, so I would say that it has to be small.
Email have passed review and no one said that it has to be capital
Thank you for pointing on this
Sharp eye


feat: config Loro text style automatically based on ProseMirror schema

This PR will create text style from the ProseMirror schema and apply it to the given Loro document. Only ProseMirror's mark.spec.inclusive is used, which is mapped to Loro's expand.
Closes https://github.com/loro-dev/loro-prosemirror/issues/8.
Also fix a small CI issue.

Note that this PR is based on #12, which makes the Git diff hard to read before #12 is getting merged.

Great! Thanks!


problems with matplotlib 3.0.0

What command was run:
pmag_gui.py -h
What happened:
/Users/nebula/Python/PmagPy/programs/pmag_gui.py:11: UserWarning: matplotlib.pyplot as already been imported, this call will have no effect.
  matplotlib.use('WXAgg')

What I expected to happen:
Backend should have been set to WXAgg, instead of being stuck as MacOSX.  Problem is solved by downgrading to 2.3.3.  Possibly this is a new matplotlib bug?  It is of concern if new users download matplotlib 3, though.
Data file(s):
Screenshot(s):
PmagPy version:

https://github.com/matplotlib/matplotlib/issues/12362

This problem is present in 3.0.1, but should be solved in 3.0.2 (see matplotlib issue referenced above).

I know you told me somewhere how to downgrade, but I can't find the info at the moment.
conda install matplotlib=2.3.3 does not work as it looks like anaconda has removed it from their repos.

It looks like this won't be resolved until 3.1.

Relevant issue has been closed in matplotlib, fix should be available in 3.1: https://github.com/matplotlib/matplotlib/issues/13096

All tests work except for pmagpy_tests.test_pmagplotlib.TestPlotMagMap.test_success in the Linux test environment.  This test does work on my Linux VM, though.  Possibly the version of wx is the problem?

Nope, having wxPython 4.0.4 instead of 4.0.6 does NOT fix the above error message :(

So, in general matplotlib 3.1 works just fine, for Windows, MacOS, and Linux.  It fails only in one case in the test environment.  I haven't yet updated the Cookbook instructions to reflect this, but PmagPy is currently compatible with either matplotlib 2.2.3 or 3.1.  Yay!

So, it looks like 3.0.3 and only 3.0.3 will work without crashing on Linux.  We're going to need to address the deprecation warnings and the future warnings.  3.0.3 and 3.1.0 work on OSX.

Further testing -- 3.0.3 and 3.1.0 both work on linux.  It's just the testing environment that is broken.  So I'm going to leave this alone for now.  We do still need to get those deprecation warnings and future warnings, though.


Status of gitlab 8.1 support

I was wondering about the support of gitlab 8.1. Are there any plans supporting it? Maybe it simply works already?

Hey @sotte !
Unfortunately with a change of job and the stupid decisions from https://github.com/gitlabhq on breaking the API on every minor release, I had no time to update or take care of testing the lib against every release nor add new features and try to maintain backwards compatibility.
I may work but it will probably behave wrongly. They probably changed the api AGAIN for some strange reason.
Regards,
Itxaka

@Itxaka, thanks for the quick response! That's bad news :( I'll give it a try. Maybe the basic stuff (creating users and project) works. Thanks!


chest: always accept items, unless limited by config

It would be nice to have all chests accept anything and send it to the item network.
This would allow me to be lazy and not configure each chest as a "provider" with the right item(s).
If an item is already in the chest then it is sent to the network, but the chest won't accept items.
The factory status says "output full" and the inserter status says "waiting for space in destination".
I looked at update_network_chest(), but I'm not sure what is preventing items from being inserted. Maybe that set_bar(), assuming that closes all the slots.

It's a combination of .set_bar() and .set_filter that prevent unconfigured items from being inserted. This is intentional.

Yes, that makes sense.
set_bar() closes unused slots and set_filter() restricts a slot to a particular item.
The set_filter() documentation isn't too elaborate.

Set the filter for a slot in an inserter, loader, or logistic storage container.

On each call to update_network_chest(), the code gathers the contents, clears the inventory, does the exchange with the global network and then locks each slot to a particular item, restricted by stack size and rebuilds the inventory.
What I don't understand is how the count can be restricted to a value below the stack size. For example, if I set up a network chest to provide stone with (5, 5) and then have a inserter load it with stone, it stops at 5. If you only close and filter slots, then I don't get how you do that. I looked through the LuaInventory documentation and the code, but didn't see anything that could do that.

That being said, having chests that by default send all items to the network doesn't seem useful because it never backs up and creates back-pressure. This means that upstream assemblers will produce items forever and take resources from other parts of the factory.

I agree. A "provide anything" chest would only be useful if the item count was globally limited (#38).
Thanks for you time.

What I don't understand is how the count can be restricted to a value below the stack size

This was fun to implement. The code checks for items with more than their limit, sorts them to appear last in the chest, and uses set_bar to prevent those items from being inserted into the chest.
A side effect is that you can exceed the buffer limit if you insert items really quickly or if there are a ton of chests in the map.


Remote Michrophone, Camera and Fun menu please??

can you add Remote michrophone - remote camera and Fun menu?
So i can prank my friends, and the software will be lot useful for me
Thanks :)

bruh
#1117


Multiline descriptions are not well shown by the tasks command

I normally use multine description for my settings and tasks, for having good documentation.
With help myTask, inspect myTask the description is shown ok.
But with settings or tasks, the formatting is bad:
> tasks
...
  myTask                      Line 1
Line 2
Line 3
  publish                     Publishes artifacts to a repository.
  publishLocal                Publishes artifacts to the local Ivy repository.
  publishM2                   Publishes artifacts to the local Maven repository.

...

I suggest to keep just the 1st line.

+1

steps
define a new task or setting:
val myTask = taskKey[String](
  """Line 1
    |Line 2
    |Line 3
  """.stripMargin.trim
)

myTask := "1"

problem
look up the task/setting with tasks/settings:
> tasks -v
...
  myTask                          Line 1
Line 2
Line 3
  package                         Produces the main artifact, such as a binary jar.  This is typically an alias for the task that actually does the packaging.
  packageBin                      Produces a main artifact, such as a binary jar.

expectation
the formatting isn't that bad, perhaps just keep the 1st line.
notes
sbt version: 1.1.4
keys with multi-line description aren't that common, but this issue is nicely self-contained for any interested first time contributor.
tasks and settings are commands in sbt and are defined here and here, and taskStrings is where the key description is extracted.
one possible solution is to split the string into lines and use only the first one.
I don't think we have any tests for these two commands, so I think the best way to test it is to manually test by building sbt from source. an alternative is the sbtOn custom command, but how to use that is not documented.
if you get stuck on this at all, or if I can help you in any way please feel free to ping me (@dwijnand) here or in the sbt/sbt-contrib Gitter room.

I will try and look at this :)


Setup Visualization

Developed in branch - Feature/#103-Setup-Monitoring

A grafana image is created in the docker-compose.yaml file

The server is hosted at port 3000

I think that it is smart that we have a local development Grafana and Prometheus. so that we can test and work with them locally. Just like we must have a local Postgres database at some point. But the username/password for local development is okay being admin/admin. It is not exposed to the www.


Clear migration process

Document clearly how to migrate data from the old CantusDB to the new CantusDB.
Write tests to check that two databases are identical.

The data migration process has been documented in the wiki of this repo.


VS Code Freeze

Issue Type: Performance Issue
Hi,
VS Code is freezing at random moments... well, not so random, it seems it happens when I have compilation errors (code underlined with red).
It happens more and more often recently. At first, I thought it was Angular compilation/hot reloading that made my laptop freeze but even when I don't use Angular CLI at all (no ng serve) it still freezing.
Can you look into it? Thanks for this amazing editor by the way. Never had any problems  until a week ago...
VS Code version: Code 1.41.0 (9579eda04fdb3a9bba2750f15193e5fafe16b959, 2019-12-11T18:32:17.711Z)
OS version: Linux x64 4.15.0-72-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz (4 x 3100)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
25, 38, 20

Memory (System)
7.70GB (4.91GB free)

Process Argv
--no-sandbox --unity-launch

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    1	    79	  2721	code main
    0	     0	  2724	   zygote
    0	    47	  2752	   gpu-process
    0	    32	  2763	   window (equipmentTypeService.ts (Working Tree) - gim-ws - Visual Studio Code)
    0	     8	  3195	     extensionHost
    0	     0	  3492	       electron_node tsserver.js
    0	     8	  3493	       electron_node tsserver.js
    0	     0	  3526	         electron_node typingsInstaller.js typesMap.js
    0	     0	  3511	       electron_node tslintServer.js
    0	     0	  3823	       /usr/share/code/code /usr/share/code/resources/app/extensions/json-language-features/server/dist/jsonServerMain --node-ipc --clientProcessId=3195
    0	     0	  3196	     /usr/bin/zsh
    0	     0	  3455	     watcherService
    0	     0	 15762	     /usr/bin/zsh
    0	    63	  3469	   shared-process
    0	     0	 10629	     /bin/sh -c /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    0	     0	 10630	       /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    0	   323	  9998	   window (● table-factory.service.ts - gim-ui - Visual Studio Code)
    0	   173	 10019	     extensionHost
    0	    79	 10314	       electron_node tsserver.js
    0	   236	 10315	       electron_node tsserver.js
    0	    71	 10356	         electron_node typingsInstaller.js typesMap.js
    0	    87	 10332	       /usr/share/code/code /usr/share/code/resources/app/extensions/html-language-features/server/dist/htmlServerMain --node-ipc --clientProcessId=10019
    0	   102	 10333	       electron_node tslintServer.js
    0	    55	 10448	       /usr/share/code/code /usr/share/code/resources/app/extensions/json-language-features/server/dist/jsonServerMain --node-ipc --clientProcessId=10019
    0	     0	 10020	     /usr/bin/zsh
    0	   102	 10193	     watcherService
    0	    63	 10424	     searchService
    1	   102	 10606	   window (Issue Reporter)

Workspace Info
|  Window (● table-factory.service.ts - gim-ui - Visual Studio Code)
|  Window (equipmentTypeService.ts (Working Tree) - gim-ws - Visual Studio Code)
|    Folder (gim-ws): 169 files
|      File types: js(64) ts(64) json(6) log(3) gitignore(1) prettierrc(1)
|                  md(1) jpg(1) gitkeep(1) env(1)
|      Conf files: package.json(1) tsconfig.json(1) tslint.json(1)
|    Folder (gim-ui): 3458 files
|      File types: svg(2708) scss(259) ts(116) js(37) jpg(34) html(30) png(22)
|                  css(19) eot(10) ttf(10)
|      Conf files: tsconfig.json(2) package.json(1) tslint.json(1);

Extensions (23)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.3.0

gitignore
cod
0.6.0

gitlens
eam
10.2.0

tslint
eg2
1.0.44

vscode-npm-script
eg2
0.3.10

vscode-great-icons
emm
2.1.47

prettier-vscode
esb
3.18.0

Angular2
joh
8.3.0

dotenv
mik
1.0.1

vscode-pugbeautify
mrm
1.0.2

vscode-scss
mrm
0.8.1

vscode-docker
ms-
0.9.0

python
ms-
2019.11.50794

Go
ms-
0.11.9

vscode-typescript-tslint-plugin
ms-
1.2.3

peep
nwa
0.0.5

vetur
oct
0.22.6

material-icon-theme
PKi
3.9.2

vscode-yaml
red
0.6.1

code-settings-sync
Sha
3.4.3

gitflow
vec
1.2.1

vscode-icons
vsc
9.6.0

vscode-todo-highlight
way
1.0.4

(2 theme extensions excluded)

Does this reproduce in the latest VS Code insiders build with all extensions disabled?
/needsMoreInfo

Hi @mjbvz,
So it seems that the freeze was related to the TSLint plug-in I used. I disabled it and so far so good.
Thanks!

@Blockost
Thanks.
Confirmed TSLint plug-in problem.


premake script does not look for CUDA_PATH on Linux

Premake configuration is incorrect for NVIDIA Linux setups, OpenCL headers are not plugged in by default. Need to fix that.

Need to manually define CUDA_PATH


Broken link

First link under Related Resources is broken.
URL: https://github.com/dotnet/dotnet-docker-samples/README.md

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

ID: 72ef1481-3b6f-30bf-330d-45400c0a349c
Version Independent ID: 03a156f7-b178-4d54-503d-26d229f89dcd
Content: Learn Docker Basics with .NET Core
Content Source: docs/core/docker/docker-basics-dotnet-core.md
Product: dotnet-core
GitHub Login: @JRAlexander
Microsoft Alias: johalex

Thanks for reporting the issue @boskjoett. I've opened a PR to fix this.


feat: watsonx assistant module

Description
This PR addresses the internal Issue to create watsonx Assistant module.
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[x] Major release (X.x.x)

Release notes content

created terraform module to provision IBM watsonx Assistant instance.

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[x] If relevant, a test for the change is included or updated with this PR.
[x] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@imprateeksh There i no code yet in main branch - so upgrade test will not pass. Just skip it for initial PR

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline


Change ssh to another port (obscurity)

https://www.hostinger.my/tutorials/how-to-change-ssh-port-vps#:~:text=Therefore%2C we recommend changing the,ranging from 1024 to 65535.


value_cast is canonical over number_cast

Instead of teaching number_cast for scalars we can just use value_cast. Thus remove number_cast from the public APIs, and just call the implementation from value_to.

Unfortunately we need number_cast for the error code


fix: Drop Direct3DInteropSample

What does the pull request do?

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

Fixes #9666

You can test this PR using the following package version. 11.0.999-cibuild0027482-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


feat: allow  and  to get any metadata-* attrs set

Related to #500
Keeping this as a draft rn to answer this question:

Is there any value to keeping the well-defined list of 3 "special" attributes? Attributes.METADATA_VIDEO_ID, etc? Autocomplete tooling, typescript flagging in editors?
For mux-player, does it makes sense to use the connectedCallback in order to access the shadow DOMs muxVideo.metadata= prop?

todo

[x] resolve outstanding questions
[x] tests

mux-video
This solves it for <mux-video> so someone could do:
<mux-video
  <!-- other attrs -->
  metadata-sub-property-id="2434"
></mux-video>

[x] Also we'd need to do this for <mux-audio> (same code as this).

mux-player
<mux-player
  <!-- other attrs -->
  metadata-sub-property-id="2434"
></mux-player>

Makes a small internal change to how metadata is passed from <mux-player> -> <mux-video>. Now we only use the muxVideo.metadata= prop for assigning metadata. The attrs syntax is removed from the template

Update:

going to try to add some tests
examples have been updated so we can test the functionality on the deploy preview -- will revert those example updates before merging
will want to squash & merge this one with a conventional commit message like:

feat: allow arbitrary Mux Data metadata-* attributes syntax (previously only supported this syntax for metadata-video-title, metadata-video-id and metadata-viewer-user-id

@cjpillsbury if I understand correctly there may be some edge-case breaking change if a user tries to set the same metadata fields in 2 different ways:
<mux-player video-title="Title AB"></mux-player>

AND
document.querySelector('mux-player').metadata = { video_title: 'XY' }

The potential change in behavior would be which one gets precedence?
If that's the case, I think we can live with that. Folks really shouldn't be setting the same metadata fields with different values in 2 different ways.

Verified the behavior on esm-vanilla deploy preview, reverted the changes to examples/, will squash merge after the CI builds

@cjpillsbury if I understand correctly there may be some edge-case breaking change if a user tries to set the same metadata fields in 2 different ways:
<mux-player video-title="Title AB"></mux-player>

AND
document.querySelector('mux-player').metadata = { video_title: 'Title XY' }

The potential change in behavior would be which one gets precedence?
If that's the case, I think we can live with that. Folks really shouldn't be setting the same metadata fields with different values in 2 different ways.

Kind of... before, neither ended up getting precedence because of the way the metadata attrs" flowed to" the underlying mux-video instance + the fact that the the getter for metadata always merged the set prop plus the attrs. Since we now never pass the mux-player attributes down via attributes, we loose this "automatic merging" behavior.


Extension fails to run in WSL environment; only partial operation in windows environment when running WSL folders.

Describe the bug
Essentially the bug is precisely as described in the title. When running the extension in WSL2, it doesn't function: The default folding functionality remains. The most that will happen is an ellipsis will appear next to the line being folded. This appears to affect every attempted language (Tested in HTML, JS, CSS, and JSON files).
When a WSL file (That is, a file that is stored within the WSL2 directories) is opened from VSCode, while VSCode is running on Windows, the extension will mostly work, except for that trailing brackets remain. Ie, The description of a function's contents therein will appear next to the line being folded, but the trailing brackets remain.
Also, while in WSL2, VSCode runs the extension in the Remote Extension Host, giving the message: "This extension is running in the remote extension host because it prefers to run there."
To Reproduce
Steps to reproduce the behavior:

Launch a folder VSCode in WSL2
Ensure Extension is installed. Open any .js file
Attempt to collapse any function, object, array, etc.
See error - failure to produce desired folding

Expected behavior
The folding described in the extension about page therein.
Screenshots
Pre Fold (in Express App):
https://i.imgur.com/whtiLMN.png
Post Fold:
https://i.imgur.com/7NalNcd.png
Pre Fold:
https://i.imgur.com/KTeffxo.png
Post Fold:
https://i.imgur.com/fYMkEWO.png
Desktop (please complete the following information):

OS: Windows 10 Pro 19044.2486
-- Version 21H2
WSL2 OS :  Ubuntu-20.04

Additional context
This probably isn't relevant as I haven't been able to investigate the extension's code deeper than just a barely surface level, but at a brief glance through the code I see the use of __dirname. I've run into problems when building apps in WSL2 with __dirname - having instead to use a custom __dirname absolute path such that:
const __dirname = url.fileURLToPath(new URL('.', import.meta.url));
Again this probably isn't relevant to the operation of your extension, but such an inconsistency like that between WSL and other operating environments (one which necessitates a workaround to things like absolute file paths) may be worth investigating.
Thanks so much for the hard work and the wonderful extension. You're seriously doing the work that Microsoft hasn't been able to do in almost an entire decade - excellent project.

Hey thanks for the detailed report, always a joy to see those.
Would you mind clean install  this  version and let me know if it works?

Gave it a try - no such luck unfortunately.
That version does however work quite well when I open the directory solely in Windows. Seems to just be the WSL2 Ubuntu-20.04 environment in which it doesn't work.
Appreciate the hard work, Good luck!
(It really is long past time I drop WSL. Been clinging to it on my home machine out of sheer force of habit, if nothing else).

Oh sad, I'll look more into it.


Inline "[author], [last updated]" messages display before every flow type definition

GitLens Version: 8.3.4
VSCode Version: 1.24.0
OS Version: macOS 10.13.4

Steps to Reproduce:

Have a git committed file /w the following contents:

// @flow

type Something = { name: string, age: number };
type AnotherThing = { whatever: string };

Notice the [author], [last updated] message shows up before every time definition (gitlens.debug enabled for screenshot)

Ref: https://twitter.com/eamodio/status/1004802247614783488
❤ this extension btw, thank you so much for making it!
Edit: Attaching another screenshot in case that'll help:

@eamodie Yes, this file has a .js extension.

@t1mmen sorry for the delay in getting back to you. Can you try adding the following to the gitlens.codeLens.scopesByLanguage setting?
        {
            "language": "javascript",
            "symbolScopes": [
                "!Interface"
            ]
        },

That should resolve the issue for you. Hope that helps.

I am closing this issue because it needs more information and has not had recent activity.  Please re-open this issue when more details can be provided.
Thanks!


One part of the interface to avoid so much confusion and this repeated question.

I'm having these guys' problem:
https://discuss.atom.io/t/connecting-to-github-in-atom-1-18/45655
https://discuss.atom.io/t/connecting-github-with-atom/50963
As well as the first commenter here:
https://www.youtube.com/watch?v=HqrkEVWBifw
I still haven't figured out how to do this even though I've been a developer for 20 years.
But look you can avoid these questions that you keep seeing crop up if you just change the interface like this:
The Github tab just says "No Remotes This repository dos not have any remotes hosted at gitgub.com".  Just make it say anything remotely helpful on how to connect Atom to github.  Because right now absolutely no part of Atom has anything useful on how to do that.
Even at the bottom you could have the "No remote" button be clickable and bring up some kind of explanation.
I'm sorry but this seems so obvious to put into the interface right?
If it has to be done on the CLI, ok sure! But tell people that.
Atom 1.41.0 on Ubuntu 18.04.2

“No remote” mean that there is no remote git repo (on a git server). You can create/upload the repo to GitHub and thus GitHub will be hosting your repo


Enable CI/CD with ARM template

Is it possible to enable CI/CD within an ARM template?

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

ID: a053a774-e0e7-e7c7-8a08-7244cb0f058d
Version Independent ID: 7acef172-4c69-6502-d506-c8c3ef13a1c1
Content: Continuous deployment from a Docker container registry with Web App for Containers - Azure
Content Source: articles/app-service/containers/app-service-linux-ci-cd.md
Service: app-service
GitHub Login: @ahmedelnably
Microsoft Alias: aelnably;msangapu

@szinck1 Thanks for the feedback! We are currently investigating and will update you shortly.

Figured it out. Under appSettings, set the variable DOCKER_ENABLE_CI to true


Force file download

Documentation says (https://pocketbase.io/docs/files-handling/):

Additionally, to instruct the browser to always download the file instead of showing a preview when accessed directly, you can append the ?download query parameter to the file url.

But looking at https://github.com/pocketbase/pocketbase/blob/master/apis/file.go the only QueryParam that are being used are token and thumb.
Was this a feature that has been removed or something that has never been implemented yet?

The download query param is part of the serve handling - https://github.com/pocketbase/pocketbase/blob/b3f09ff045cfb915aea8a8334d0f4f17d236952d/tools/filesystem/filesystem.go#L326

@ganigeorgiev I didn't noticed that, thanks.
However it has to be specified it has to be true and not just being set. Took me a while to figure it out.
Docs could be improved here.
❯ curl -i "http://<IP_ADDRESS>:8090/api/files/i37o5ykupb5voix/ipqdssqswp3n8ns/example_TCfBbTqfhn.png"
HTTP/1.1 200 OK
Accept-Ranges: bytes
Cache-Control: max-age=2592000, stale-while-revalidate=86400
Content-Disposition: inline; filename=example_TCfBbTqfhn.png
Content-Length: 42240
Content-Security-Policy: default-src 'none'; media-src 'self'; style-src 'unsafe-inline'; sandbox
Content-Type: image/png
Last-Modified: Mon, 31 Jul 2023 13:45:16 GMT
Vary: Origin
X-Content-Type-Options: nosniff
X-Xss-Protection: 1; mode=block

curl -i "http://<IP_ADDRESS>:8090/api/files/i37o5ykupb5voix/ipqdssqswp3n8ns/example_TCfBbTqfhn.png?download"
HTTP/1.1 200 OK
Accept-Ranges: bytes
Cache-Control: max-age=2592000, stale-while-revalidate=86400
Content-Disposition: inline; filename=example_TCfBbTqfhn.png
Content-Length: 42240
Content-Security-Policy: default-src 'none'; media-src 'self'; style-src 'unsafe-inline'; sandbox
Content-Type: image/png
Last-Modified: Mon, 31 Jul 2023 13:45:16 GMT
Vary: Origin
X-Content-Type-Options: nosniff
X-Xss-Protection: 1; mode=block

❯ curl -i "http://<IP_ADDRESS>:8090/api/files/i37o5ykupb5voix/ipqdssqswp3n8ns/example_TCfBbTqfhn.png?download=true"
HTTP/1.1 200 OK
Accept-Ranges: bytes
Cache-Control: max-age=2592000, stale-while-revalidate=86400
Content-Disposition: attachment; filename=example_TCfBbTqfhn.png
Content-Length: 42240
Content-Security-Policy: default-src 'none'; media-src 'self'; style-src 'unsafe-inline'; sandbox
Content-Type: image/png
Last-Modified: Mon, 31 Jul 2023 13:45:16 GMT
Vary: Origin
X-Content-Type-Options: nosniff
X-Xss-Protection: 1; mode=block

Yes, the parameter must have a "truthy" value.
I'll consider updating the documentation sometime later today.

@ganigeorgiev Just noticed it's also missing in the Web APIs Reference (https://pocketbase.io/docs/api-files/#download-fetch-file)
Thanks for the quick response and update!


update dependencies

Coverage remained the same at 46.468% when pulling d749f58ef76302c293c23031c0d27782f1883063 on update-dependencies into 1aed12c4cb391f9a392655845018966241fd23c3 on master.


Can't even require the module

$ cat run.js
var tape = require("tape");
var tapes = require("tapes");
$ node run.js
.\node_modules\tapes\index.js:81
test.Test = tape.Test;
            ^
ReferenceError: tape is not defined
    at Object.<anonymous> (.\node_modules\tapes\index.js:81:13)
   ...

$ node --version
v4.3.2
$ which node
/cygdrive/c/Program Files (x86)/nodejs/node
$ npm --version
3.9.2
$ cat package.json
{
  "name": "test",
  "version": "1.0.0",
  "description": "",
  "main": "test-table.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "azure": "^0.10.6",
    "nconf": "^0.8.4"
  },
  "devDependencies": {
    "tape": "^4.6.0",
    "tapes": "^4.0.0"
  }
}

The first 3 lines of your file should be:
var tape = require('tape');
var tapes = require('tapes');
var test = tapes(tape);

Are you remembering to instantiate test by passing tape into tapes?

var tape = require('tape');
var tapes = require('tapes');
var test = tapes(tape);
$ node run.js
.\node_modules\tapes\index.js:81
test.Test = tape.Test;
            ^

ReferenceError: tape is not defined

Yep.. clearly line 81 expects tape to be in the global scope but it's not there. If it's not already obvious my environment is cygwin (actually babun) on Windows. That may have some bearing on the issue.

I don't think it's an issue with your terminal emulator - replicated here

Same problem with me too. i get the same error
ReferenceError: tape is not defined
    at Object.<anonymous> (...../node_modules/tapes/index.js:81:13)

I'm running on a mac.

I'm out on vacation. I'm open for a PR to fix if anyone has time in the next couple days!

Mistake on my part! Sorry all. Fixed in 4.0.1


Added DialogflowApp#getIntentName to retrieve DialogFlow intent

The existing method #getIntent currently fetches the action name from DialogFlow. In order to not break existing functionality the method to retrieve the actual intent has been called #getIntentName

CLA signed.

Is there a use where you may want this?

Yes, for example when routing by intent instead of by action.

The Intent name(s) can also contain space in between words and there is no restriction. You may want to consider adding it in the API document. Also, matter of fact the intent based action mapping would be fairly complicated handling the restrictions.

I also am not sure if getIntentName is the ideal API as that may cause confusion with the existing getIntent

I totally agree. However, I would argue that the original getIntent method already suffers from this problem and forced me to pick a suboptimal name for the new method. I would've suggested calling getIntent getAction instead to use getIntent for the new method. But that might break to much.
It's a bit weird using something like DialogflowApp#getDialogflowIntent
Any suggestions for the method name?

I won't speak for the library owners, but I like using label to refer to the display name of a thing and not the underlying name. getIntentLabel().

I'm fine with getIntentLabelas well, sounds better.

Agreed that the naming issue falls on the original getIntent() method, so we should keep this as getIntentName() to reflect Dialogflow's UI


Add None as possible parameter value to isnull/isna/notnull/notna

[X] Closes #318
[X] Tests added: Please use assert_type() to assert the type of any return value

I wrote exactly the same changes as you and was going to create a PR when I saw this 🤣 good job!

One additional thought here - is there any way we can make these functions a TypeGuard so they can be used to narrow types? Right now it will not narrow an Optional or a Union[np.nan, T], but it should.


Ingress-nginx could use k8s native sysctl

https://kubernetes.io/docs/tasks/administer-cluster/sysctl-cluster/#setting-sysctls-for-a-pod

Seems like net.ipv4.ip_local_port_range is considered a safe sysctl and allowed by default, but tuning net.core.somaxconn would require kubelet --allowed-unsafe-sysctls net.core.somaxconn.
The default net.core.somaxconn = 128 could cause slowdowns with very high rates of new connections (faster than nginx is able to accept them), but TCP should handle retries in that case (assuming default net.ipv4.tcp_abort_on_overflow=0): https://serverfault.com/questions/518862/will-increasing-net-core-somaxconn-make-a-difference

...except that the ingress-nginx pod runs with networkMode: host, so those sysctls affect the entire node.


rsqrt: account for AMD vs. Intel inconsistency in RCPPS relative error

:wave:
This PR fixes #757 by setting up the rsqrt test with the documented relative error from Intel/AMD.
(For future reference: see https://news.ycombinator.com/item?id=23891810)
Fixes #757

Thanks for this. The goal of this test is mostly to check AP consistency, it's a bit clumsy for inaccurate functions like rsqrt


Official OpenAPI / swagger bindings

Environment

Python version: N/A
Nautobot version: N/A

Proposed Functionality
Officially maintained bindings (Generated from swagger) for golang / others
There is an existing community maintained version for Netbox here https://github.com/netbox-community/go-netbox
However due to various swagger definition problems in Netbox the generated client generally needed modification in areas.
Issues: 4808, 4804, 4803 among others
Most of the existing issues can be resolved by fixing up the swagger definitions and by adding tests.

Use Case
Enhanced usability in external applications for automation tooling.

Database Changes
N/A

External Dependencies
N/A

Just to clarify, you're effectively asking for an OpenAPI 2.0-compliant go-nautobot fork?

Just to clarify, you're effectively asking for an OpenAPI 2.0-compliant go-nautobot fork?

That would be my personal use case here, however maintaining the openapi swagger definitions will allow for usable bindings to be generated for other languages.

Okay, I'm with you. So far we've maintained whatever was already part of NetBox core and extended it slightly for the changes we've made.
Discussions are ongoing right now as to continue to support OpenAPI 2.0 support, or move to OpenAPI 3.0 (w/ drf-spectacular) for the new API work we'll be doing. As I understand it drf-yasg, which is what is being used to define the v2.0 schema, is end-of-life in this regard.
Longer term, we 100% want to be able to support auto-enumerating clients for other languages, so it sounds like that's what you're looking for. When that time comes it's almost certainly going to be OpenAPI 3.0 but no final decisions have been made.

Aha I wasn't aware that OpenAPI 3 was a thing,
I am not too worried about what form it takes, as long as its maintained and has at least the same functionality as the current one.
Nice to hear its being planned 👍

I'd suggest looking at polishing up the graphql api for the long-term 'mother of all nautabot interfacing'.
With a good gql backend thats the only access for the UI and external clients a lot of duplicate effort in maintaining duplicate code/logic can be avoided. For the clients depending on REST style api a gql2rest mapper could be packaged.

+1 on @sdktr sdktr's comment.  It is difficult to keep the OpenAPI bindings 100% accurate with the core API, and the core API isn't always well-behaved since it is written in a dynamic language.  Is it an int, a str, nullable, all-of-the-above?  It may be a better option long-term to refer users of statically-typed languages to the GraphQL API and let them figure out how to build their own structs based on crafted responses.  Golang has tooling for this, and I'm sure the same can be done in other languages.  go-netbox is a sprawl of boilerplate with practically no tests and several bugs (nature of the beast, no offense to the project or idea).  I would be reluctant to replicate that for Nautobot.

+1 on @sdktr sdktr's comment. It is difficult to keep the OpenAPI bindings 100% accurate with the core API, and the core API isn't always well-behaved since it is written in a dynamic language. Is it an int, a str, nullable, all-of-the-above? It may be a better option long-term to refer users of statically-typed languages to the GraphQL API and let them figure out how to build their own structs based on crafted responses. Golang has tooling for this, and I'm sure the same can be done in other languages. go-netbox is a sprawl of boilerplate with practically no tests and several bugs (nature of the beast, no offense to the project or idea). I would be reluctant to replicate that for Nautobot.

Isn't it a bad thing that the API can return undefined results that don't match the defined API in the first place?
Making the API itself use static types should be seen as a good thing as you know for certain what to expect from it, regardless of the clients language and weather or not it may be statically typed.
Json2struct is not a great solution in general to what could be automatically generated fairly easily with the templates being maintained, and which will likely lead to a more stable API going forward in general.
Personally as a heavy automation user of the current netbox-go integration I would like to see the OpenAPI solution upgraded as per @jathanism's comment.
I do not believe that GraphQL should replace a good rest API.

After several internal discussions, I have submitted a proposal togo all-in on OpenAPI 3.0. See: https://github.com/nautobot/nautobot/issues/595

From the experience we had with go-nautobot it looks like we have had success in providing these, anything missing here?

From the experience we had with go-nautobot it looks like we have had success in providing these, anything missing here?

Is that a private repo?, I can't access it

Oh, spoiler alert, sorry! We're working on getting that published, not quite there yet.

Oh, spoiler alert, sorry! We're working on getting that published, not quite there yet.

Is there an ETA on getting this published?

Oh, spoiler alert, sorry! We're working on getting that published, not quite there yet.

Is there an ETA on getting this published?

Also hoping for an ETA, any updates?

Any updates?

@UnityDG @mristok @Letsu apologies for the delayed answer, the repository is now public and in beta!


Removes write version from StorableAccountsWithHashesAndWriteVersions

Problem
Now that we no longer use the write version when storing accounts (see #476), and StorableAccounts no longer contains anything about write version either (see #542), StorableAccountsWithHashesAndWriteVersions does not need to care either.
Summary of Changes
Remove write version from StorableAccountsWithHashesAndWriteVersions.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.8%. Comparing base (0168e0a) to head (ac1858f).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master     #561     +/-   ##
=========================================
- Coverage    81.8%    81.8%   -0.1%
=========================================
  Files         847      847
  Lines      229180   229144     -36
=========================================
- Hits       187600   187502     -98
- Misses      41580    41642     +62


Uncaught BufferedProcessError after installing ternjs

I use Atom version 1.0.5 (it just got updated) and this problem arises when I install/enable atom-ternjs (currently on version 0.7.1). Disable it, and the error gows away. I've tried several combinations aof enabling and disabling extensions, but ternjs is the only one that causes trouble. So it seems.
Uncaught BufferedProcessError: Failed to spawn command `/Applications/Atom.app/Contents
/Frameworks/Atom Helper.app/Contents/MacOS/Atom Helper`.
Make sure `/Applications/Atom.app/Contents/Frameworks/Atom Helper.app
/Contents/MacOS/Atom Helper` is installed and on your PATH

Also mentioned here [https://github.com/atom/atom/issues/7594]

I've also faced this.

Same. It's happening to everyone.

Same here.

Yep, running into it here on Atom 1.0.7and atom-ternjs 0.7.1
Running Atom Helper from the CL gives me:
Error: Cannot find module '/Applications/Atom.app/Contents/Frameworks/Atom Helper.app/Contents/Resources/atom.asar/browser/lib/init.js'     at Function.Module._resolveFilename (module.js:328:15)     at Function.Module._load (module.js:270:25)     at Function.Module.runMain (module.js:493:10)     at startup (node.js:120:16)     at node.js:877:3
Running ./Atom --safe quiets down the bug.

Atom 1.0.11, atom-ternjs 0.7.2, got same problem:
Failed to spawn command /Applications/Atom.app/Contents/Frameworks/Atom Helper.app/Contents/MacOS/Atom Helper. Make sure /Applications/Atom.app/Contents/Frameworks/Atom Helper.app/Contents/MacOS/Atom Helper is installed and on your PATH

Same problem on Atom 1.0.19

Atom 1.0.19 / atom-ternjs 0.7.3 / Node 4.1.1 / OS X 10.10.4
I was getting this error but closing all my open projects, quitting Atom and restarting it seemed to fix it!

Having this problem as well.

I get this error when opening preferences with the ternjs plugin enabled.

Same problem here with Windows 10 and atom 1.0.19

Sorry, referenced the wrong PR. #144 fixes this (probably), #145 does not.


Unitie logo in safari

unitie logo on safari when on brand page, INITIALLY does not come out properly. eventually returns to normal when you go onto another page and return to brand pages. Then goes back to its 'not normal form' when form is completed. Ive sent tormod a pic on whatsapp.
@thegsi @tormod17

problem in safari v 6.0.5 but not in 6.1.6.

@officialunitie is this issue fixed now ?

@tormod17 @thegsi  now, on the brand page, it initially comes out fine. but when sign up form is filled and submitted it returns to its old non-normal form

still not fixed, just checked. @tormod17 @thegsi
not trying to be a pain, i am just very aware that many people will have previous versions of browsers, so its v.important that the sites functionality and aesthetics is of standard across the board

@officialunitie We have just made some changes. Should work now.


uibModal service not being injected?

Having the same issue as [this user] (http://stackoverflow.com/questions/33605162/inject-provider-into-directive-not-working).
This is the console error
Unknown provider: $uibModalProvider <- $uibModal <- Lightbox <- Lightbox

ui.boostrap is being required correctly and used on other elements in the same controller
var accountApp = angular.module('accountApp', [
  'ngRoute',
  'ngResource',
  'messageModule',
  'accountControllers',
  'ui.bootstrap',
  'Devise',
  'braintree-angular',
  'ngFileUpload',
  'bootstrapLightbox',
]);

Which version of UI Bootstrap? I think this happens with v0.13.x and below.

Yep that was it.  I was using ui-bootstrap 0.13.4x and upgrading to 0.14.3x fixed it.
Thank you.


feat: added closers that will be called sequentially

Added *Seq functions help to ensure that closers are executed in order without specifying order.

What is better to do when one of the errors in the sequence is non-nil?:

Return the error (which will be logged at the end)
Log the error and call close for the remaining closers in the sequence


Develop Grid-Diag Wrapper

Describe the New Feature
The Grid-Diag tool released in METv9.0 needs a new wrapper for several projects and DTC T&E activity
Acceptance Testing
Input test data:  Data used for tool development
Output test data:  Output used for tool development
Need consistency with other METplus wrappers for functionality and availability of all config settings that will be changed routinely
Time Estimate
Estimate the amount of work required here.
Issues should represent approximately 1 to 3 days of work.
Sub-Issues
Consider breaking the new feature down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
6/15/2020
Funding Source
7790971
Define the Metadata
Assignee

[x] Select engineer(s) or no engineer required
[x] Select scientist(s) or no scientist required  - Weiwei Li and Michelle Harrold

Labels

[x] Select component(s)
[x] Select priority
[x] Select requestor(s)

Projects and Milestone

[x] Review projects and select relevant Repository and Organization ones
[x] Select milestone

Define Related Issue(s)
Consider the impact to the other METplus components.

[x] METplus, MET, METdb, METviewer, METexpress, METcalcpy, METplotpy

New Feature Checklist
See the METplus Workflow for details.

[x] Complete the issue definition above.
[x] Fork this repository or create a branch of develop.
Branch name:  feature_<Issue Number>_<Description>
[x] Complete the development and test your changes.
[x] Add/update unit tests.
[x] Add/update documentation.
[x] Push local changes to GitHub.
[x] Submit a pull request to merge into develop.
Pull request: feature <Issue Number> <Description>
[x] Iterate until the reviewer(s) accept and merge your changes.
[x] Delete your fork or branch.
[x] Close this issue.

Work completed and merged into develop from PR: https://github.com/NCAR/METplus/pull/516


license

Thanks for making this public!  Would you mind adding a license to it

Want to encourage a license also. MIT would be compatible with https://github.com/pion/webrtc
which is already using this.
This is great package. A license should encourage more users.

Done

Thank you very much.


update http-proxy-middleware usage to reflect 1.0.0 api

Per the release notes, the default export has been replaced with a named export
https://github.com/chimurai/http-proxy-middleware/releases/tag/v1.0.0

Dupe of #8515. Thanks!


Weed "minimal" docker image

I think the weed docker image which build from the given Dockerfile is too large, so I made this minimal version and it's only about 19.6 MB.

Is it possible to have 2 socket files?

Yep actually we can. Maybe something like this?
`-- project
     |-- minimal
     |    |-- Dockerfile
     `-- Dockerfile

or minimal Dockerfile in the root? Which one do you prefer?

I like the minimal docker file as the default. The original one can be called something like "Dockerfile.go_build".

I just pushed an update :)
thanks

Thanks. One more thing, can you please also adjust the documentation page "doc/gettingstarted.rst", in order to make it consistent with the change?

No problem! I'm gonna have a look on that.

I did some changes on the gettingstarted.rst

Thanks!
I think the "--rm -t 'cydev/weed' " can be removed from this?
docker build --no-cache --rm -t 'cydev/weed' .
Also, the "docker run" in "In production" section needs change.

Yeah --rm is not necessary but -t, it's for the repository name.
I named it "cydev/weed" so that people can start up a container with same command line (Doesn't matter Local or Docker Hub image)
docker run --name weed -d -p 9333:9333 -p 8080:8080 \
	-v /opt/weedfs/data:/data cydev/weed server -dir="/data"

Sorry but could you tell me what to change in the "In production" section, as I see there's nothing needs to be changed.

thanks a lot!


Unable to deploy DefectDojo version 2.0.0/2.2.x

I am trying to deploy DefectDojo latest version 2.2.1.
I even tried versions like 2.2.0 and 2.0.0, but I always have the below error while deploying DefectDojo.
The logs of defect-dojo-defectdojo-django pod are something like this:
ERROR:django.request:Internal Server Error: /login
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/django/db/backends/base/base.py", line 219, in ensure_connection
    self.connect()
  File "/usr/local/lib/python3.8/site-packages/django/utils/asyncio.py", line 26, in inner
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/django/db/backends/base/base.py", line 200, in connect
    self.connection = self.get_new_connection(conn_params)
  File "/usr/local/lib/python3.8/site-packages/django/utils/asyncio.py", line 26, in inner
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/django/db/backends/postgresql/base.py", line 187, in get_new_connection
    connection = Database.connect(**conn_params)
  File "/usr/local/lib/python3.8/site-packages/psycopg2/__init__.py", line 122, in connect
    conn = _connect(dsn, connection_factory=connection_factory, **kwasync)
psycopg2.OperationalError: FATAL:  password authentication failed for user "defectdojo"

Which of your two tickets do you want to progress?
Did you follow the steps fromhttps://github.com/DefectDojo/django-DefectDojo/blob/master/KUBERNETES.md to create the necessary secrets?

Which of your two tickets do you want to progress?
Did you follow the steps fromhttps://github.com/DefectDojo/django-DefectDojo/blob/master/KUBERNETES.md to create the necessary secrets?

Hi, I want to keep this issue, the previous one is resolved for now.
And yeah, I have created all the necessary secrets except for Redis and MySQL as I am using RabbitMQ and PostgreSQL.

I did few customizations in my helm chart, but then followed the official one. After doing that I was really wierd logs of nginx container in django pod.

The above nginx pod error is fine, but still I have 500 Server error while accessing Defect Dojo URL.
1 - [11/Sep/2021:16:34:38 +0000] "GET / HTTP/1.1" 302 0 "https://login.microsoftonline.com/" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/93.0.4577.63 Safari/537.36" "<IP_ADDRESS>, <IP_ADDRESS>"
<IP_ADDRESS> - [11/Sep/2021:16:34:38 +0000] "GET /login?next=/ HTTP/1.1" 500 145 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/93.0.4577.63 Safari/537.36" "<IP_ADDRESS>, <IP_ADDRESS>"
<IP_ADDRESS> - [11/Sep/2021:16:34:39 +0000] "GET /favicon.ico HTTP/1.1" 302 0 "https://defect-dojo-url/login?next=/" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/93.0.4577.63 Safari/537.36" "<IP_ADDRESS>, <IP_ADDRESS>"
<IP_ADDRESS> - [11/Sep/2021:16:34:39 +0000] "GET /login?next=/favicon.ico HTTP/1.1" 500 145 "https://defect-dojo-url/login?next=/" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/93.0.4577.63 Safari/537.36" "<IP_ADDRESS>, <IP_ADDRESS>"

Here is the error message of uwsgi pod, same as before
django.db.utils.OperationalError: FATAL:  password authentication failed for user "defectdojo"

Any update on this ?


fix: renamed one multitenancy feature flag to CONNECTOR_ENABLE_UNASSIGNED_CONNECTORS

Description
Renamed secret and template option to CONNECTOR_ENABLE_UNASSIGNED_CONNECTORS.
Verification Steps
CI passes without errors.
Checklist (Definition of Done)

[X] All acceptance criteria specified in JIRA have been completed
[X] Documentation added for the feature
[x] CI and all relevant tests are passing
[x] Code Review completed
[x] Verified independently by reviewer

ahoukd be ready to be merged right ?

Done!


HF Spaces demo not working ?

Hi, I've been trying to use the demo for inference on some examples. The demo loads properly, and I'm able to add videos to the demo but the inference always returns an Error. I even tried out each of the default examples provided on the website but the same issue persists. I've attached a sample screenshot of the problem :

Could you please help me out on this? Thanks in advance!

Hi, thanks for your interest in our project. We have fixed the HF demo bug, you may try it again.

Hi, HF demo isnt working.


test: Test missing description

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 99.275%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
137

Relevant Lines:
138

💛  - Coveralls


testing: cleanup temporary directory

MkDir: https://github.com/go-check/check/blob/v1/check.go#L170
Signed-off-by: Li Zhijian<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

ping


Bug in changeAddress

Sometimes when user input change address and send tx, the change is sending to ledger address and block it.

Resolved in #129


BPM Change in scoreTxt.text

Describe your problem here.
I'm making a mod where it shows the BPM in the Score Text, the one in PlayState.hx and where it says scoreTxt.text = . Right now I have SONG.bpm + ' BPM', but that only shows the starting BPM and when the BPM changes, the text doesn't. Is there a way to  change the text with the BPM (with source only)?
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

SONG.bpm would only display what the BPM is in the json file, even if it's changed mid song it won't change. Conductor.bpm will always display the current bpm.
Although since you are setting this to score text, it won't update until a note is hit/missed.

Ah ok, thank you!

Ah ok, thank you! Also, do event notes count as the hit or missed thing?

event notes aren't real notes

scoreText.text = 'BPM: ${Conductor.bpm}';


Support backup to new text-based database dump format

Note: Leaving out the RPC dumpwallet format because it stores an encrypted wallet as unencrypted. If that's desirable to support at all, it can be a followup PR (perhaps with a strong warning).

What use cases do you see for such a feature?

Personal preference? Dunno, @achow101, what use cases did you have in mind for this dump format?
(I'm not adding the new format here, mind you, simply making it available to GUI users)

I'm not sure how useful this is. There were two primary reasons for adding the dump and createfromdump commands: 1) To make it easier to debug low level database record issues and 2) to make it possible to change the database storage type without changing the wallet level logic. In the second case, this is not something that we really want people to do but I wanted to be able to do it at least just for testing.

🐙 This pull request conflicts with the target branch and needs rebase.
Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".


change sign-up.html page add some CSS and made the buttons much better

…r and added some hover effects
🛠 Fixes Issue
The buttons were not looking good and there was no such CSS there. I have added all the styles and now it's looking cool.
👨💻 Changes proposed

I have added new CSS in the html page
I have added cool animations and effects in the website.

✔ Check List (Check all the applicable boxes)

[ ✔ ] My code follows the code style of this project.
[ ✔ ] This PR does not contain plagiarized content.
[ ✔ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

@SauravMukherjee44 please check the codes and merge it.
Thank you!!


The event handler should be declared as nullable

Type of issue
Outdated article
Description
This sample was written pre-nullable reference types. It should be updated to handle NRT without warnings.
Page URL
https://learn.microsoft.com/en-us/dotnet/csharp/programming-guide/events/how-to-publish-events-that-conform-to-net-framework-guidelines
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/programming-guide/events/how-to-publish-events-that-conform-to-net-framework-guidelines.md
Document Version Independent Id
1fd093e9-f47c-ffdd-91e9-5039c3ccde61
Article author
@BillWagner
Metadata

ID: 38e8321f-5516-e451-5429-750d49f0ebed
Service: dotnet-csharp
Sub-service: fundamentals

I can update the sample if this is up for grabs


Implement interface for event relevant views

npm i react-calendar
use this command to install calendar component

Looks awesome <3 @YufXiao @Pasafa  Should we do a meeting today or tomorrow to integrate and merge?


fix: update GrpcStorageImpl handling to be aware of quotaProjectId

Add integration tests to verify desired precedence when resolving userProject options.
Fixes #1736

Depends on test fixture changes in https://github.com/googleapis/java-storage/pull/1876


Add API 34 emulator

Testing whether this is less flaky than the API30 emulator.

What about API 35? 😃


Performance improvement of npairs_s_mu_engine.pyx

I noticed that s_mu_tpcf from from halotools.mock_observables is extremely slow and found that npairs_s_mu_engine.pyx had a very inefficient loop structure. I made small changes to the engine. s_mu_tpcf is now ~3 times faster when using 20 bins in s and mu. Previously, it took ~30 sec for a zheng07 mock (threshold -19) in bolplanck and now takes ~10 sec.
I have checked that the engine (and npairs_s_mu_engine.pyx) give identical results for this particular, non-trivial use case. I did not perform any other tests.

Not sure I follow this at a glance, @johannesulf. Do you know why your changes to the logic only apply to npairs_s_mu, and not, for example, npairs_rp_pi?

@aphearin Sorry, I made an error with the time it took with the original code. The performance improvement is not nearly as dramatic as originally described. But the new code still makes it faster by a factor of a ~3 for my use case.
The same logic also improves npairs_rp_pi. It does crucially depend on the number of bins, of course. For 40 bins in rp and pi I also get an improvement of order ~2 for tpcf_rp_pi.

Ok, well this is passing all tests so I'll go ahead with the merge. Nice work with the performance enhancement. I'll raise an issue to adapt this for rp_pi purposes.

Just to update what I did wrong: I originally compared the performance to halotools v0.3. That did not include commit 2e1caba9e2b65980f7922308ad79d8d107e4e520. The speed increase of ~100 came largely from that commit.


NH-21693: Convert Cluster Status composite metrics to k8s Agent metrics

Hold on with this, I created https://swicloud.atlassian.net/browse/NH-22038 to update HealthScore calculation based on last discussion with Vish. We are going to remove conditions from HealthScore calculation


clicking search on people tab should open the search screen on the people tab

https://github.com/vector-im/riot-android/commit/7fc19401b994d3ddc0810030486e657b2c47a30c


Incorrect bindings

Using the default bindings, all the buttons are messed up, but the axes are fine. I would fix them in the config files but I don't know the syntax. Using Gentoo Linux

whoops, same as #125 I probably should have looked a bit more. Funny it didn't come up on google


WIP: Some small optimizations

I hope you aren't getting too tired of these optimization PRs. It appears there is still some juice left to squeeze in this thing. Getting rid of the #[inline(always)] and implementing the two event iterators separately yielded the largest speedup. Then there's a short circuit for html scanning and some other small scanner optimizations.
Master:
crdt_total              time:   [384.45 us 384.82 us 385.25 us]

PR:
crdt_total              time:   [356.17 us 357.33 us 358.61 us]

In total, this produces a ~7.5% speedup.

crdt_total              time:   [350.39 us 350.70 us 351.05 us]

Not tired of the improvements at all! But am dealing with overload atm, might not be able to get to these in detail very soon.

Yes!! Indeed, as_bytes is a no op! But it's the string subslicing that's expensive apparently!

All the char boundary checking.. I'm not quite done yet, but you should give this PR a spin. It flies dude!

Current benchmark time on my machine:
crdt_total              time:   [338.86 us 339.12 us 339.41 us]

or about 13.4% higher throughput!

On mine (13" MBP 2017), with target-cpu=native and simd feature:
crdt_total              time:   [258.98 us 259.69 us 260.59 us]

![Neo whoa gif]

Thanks for the review!
In total these changes amount to a ~18% throughput increase on my machine.


feature/93-circleci-codecov-fixes into develop

This migrates CodeCov in the CircleCI script to use a new uploader instead of the deprecated bash uploader.
This also fixes minor issues in writing the secrets file by using echo in combination with env vars.
Finally, this also split lint and test into separate CI workflows to more easily see which one failed, if any.  This does NOT make test contingent on lint passing since it is rare that line failures are causing major test issues.
Closes #93.

This is going to remain as failing lint and test in this PR.  This is due to changes in the latest pylint as well as changes to Alpaca API returns.  This will be fixed immediately in a follow up issue.


leaflet marker image not showing

The leaflet marker image is being replaced by the default no image icon. This only happens on windows, and only after the web folks rolled back Enbridge.
https://stackoverflow.com/questions/60174040/marker-icon-isnt-showing-in-leaflet

Blue icon fixed with 20de287
Still need to add marker shadow, but not a big deal


ROX-12620: Deprecate Policy and VulnerabilityReports resources

Description
With the introduction of the WorkflowAdministration resource, the long-term goal is to replace both Policy and VulnerabilityReports resources in favor of it.
This PR will mark the Policy and VulnerabilityReports resources as deprecated.
Marking them as deprecated will have the following effect:

There will be a "replacing" resources assigned to it (in this case WorkflowAdministration). SAC will give access when the context has access to either resources.
There will be a banner within the access control page, which contains a note about the deprecation as well as the version in which it will be removed.

Checklist

[ ] Investigated and inspected CI test results
[X] Unit test and regression tests added
[X] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed

see CI.

Current dependencies on/for this PR:

master

PR #4108   👈

This comment was auto-generated by Graphite.

/retest

/retest


MIDs Revisions

No issue, More like a question
Hi Enrique,
first, thanks for this amazing interpreter, it is very helpful, i wish to ask a question please:
i am currently working with Revision 5 of the interpreter and I'm trying to use MID 0061 on my massage, which is revision 1.
from looking at the interpreter code it looks like it always uses the massage as if it was Revision 8, is that correct? if it is, can you help me understand how can i properly read my Revision 1 message?
thanks, Asi.

Hello @asidadon,
When creating/sending a mid , by default the library will always assume you're using the highest revision if you're using default constructor. So, if you want to specify another revision you should specify it at the constructor. e.g.: new Mid0061(1).
When parsing it will considers the revision sent by controller/integrator.
However, something tells me this wont answer your doubt.
When you say "properly read", you mean which properties to use ?

Hello @asidadon,
When creating/sending a mid , by default the library will always assume you're using the highest revision if you're using default constructor. So, if you want to specify another revision you should specify it at the constructor. e.g.: new Mid0061(1).
When parsing it will considers the revision sent by controller/integrator.
However, something tells me this wont answer your doubt.
When you say "properly read", you mean which properties to use ?

hi @Rickedb,
thanks for the quick reply, actually you have answered my question.
my tightening result messages are revision 1 and in the documentation it looks like revision 1 is different from 2 and on, so i thought it might cause wrong parsing of the message string.
anyway i have successfully applied your explanation and it worked good, thanks.
Best Regards

Just clearing things: Yes, revisions are different, usually they have additional information as far as revisions are increasing.
Take sometime to download read the PDF documentation here it will explain to you how it works in dept.


How to replace existing release tag instead of add more files?

Is it possible to replace files under release for existing release tag instead of adding more file ?
Currently if the tag is exists it will add more files  to them, I prefer to delete the existing release and tag and then add a new one
btw I'm building workflow for releasing develop branch on commit, the tag is always dev/latest

I can do this manually by running this command before this github action:
git fetch --tags
hub release delete <TAG>
git push --delete origin <TAG>

Would be nice if this can be done inside with, maybe something like replace_existing: true (default to false)


Codeunit 5814 "Undo Return Shipment Line" - function UpdateOrderLine

Dear ALAppExtensions Team,
We would like to request the following event publisher in codeunit 5814 - function UpdateOrderLine
procedure UpdateOrderLine(ReturnShptLine: Record "Return Shipment Line")
var
    PurchLine: Record "Purchase Line";
begin
    with ReturnShptLine do begin
        PurchLine.Get(PurchLine."Document Type"::"Return Order", "Return Order No.", "Return Order Line No.");
        // DINNL <<
        DINOnUpdateOrderLineOnBeforeUpdatePurchLine(ReturnShptLine);
        // DINNL >>
        UndoPostingMgt.UpdatePurchLine(PurchLine, Quantity, "Quantity (Base)", TempGlobalItemLedgEntry);
        OnAfterUpdatePurchLine(PurchLine, ReturnShptLine);
    end;
end;

local procedure DINOnUpdateOrderLineOnBeforeUpdatePurchLine(var ReturnShptLine: Record "Return Shipment Line")

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Ty/better page api

changing this around a bit so that the Page now has an agent binded to it + optional agent args

Wait, Browser takes an agent now? What makes it different from AgentBrowser? We could just do it all in the Browser class.


TPA-4502: SDK 0.7.0 Be able to parse accessToken out of query string.…

… FixCredentials support for migrating login_problem.

Front-end changes necessary bump to 0.7.0
I need to deploy this for the main chunk of changes.


No clear Linux desktop installation for the application

Please add it if possible~

Basically, Its only require
pip install cupy chainer
pip install opencv-python


Missing toml configs for claim-config.xml properties

"<Claim>
   <ClaimURI>http://wso2.org/claims/username</ClaimURI>
   <DisplayName>Username</DisplayName>
   <AttributeID>mail</AttributeID>
   <Description>Username</Description>
</Claim>"

Reference doc:
https://docs.wso2.com/display/IS580/Using+Email+Address+as+the+Username

This file will not be templated as this is a data resource and will on be used at initial server startup


EventEngine: fix callers of Run() and RunAfter() to create ExecCtx

See discussion in #31041.
Given the number of places we missed this, I think we may want to consider adding some sort of test to ensure that all callers of Run() or RunAt() will create their own ExecCtx.
CC @ctiller @veblush

Interesting. I was hoping that any missing, necessary exec_ctx instances would be ... less subtle than this. We had discussed trying to avoid a blanket policy to add them to every callback. I'm not sure they're needed in all of these cases, but we aren't paying a huge price to add them, so I'm ok with it.

I don't think we can assume that the ExecCtx is not needed in any case that calls EventEngine::Run() or EventEngine::RunAt().  There are too many subtle ways that callback ordering can cause bugs.
I don't think we can start removing ExecCtx instances until we remove all the code that actually uses the ExecCtx.


New Belize phone numbers not validating

This Belize phone number is a working phone number that you can call successfully, but the libphone is not validating it yet: +5016519885
This number is also reachable +5016529885, so I believe the +501 65x prefix is valid. Can this prefix please be added to the library?

Country/region affected: BZ (Belize)
Example number(s) affected: +5016519885, +5016529885
The phone number range(s) to which the issue applies: +501 65X XXXX
The type of the number(s) ("fixed-line", "mobile", "short code", etc.): mobile
The cost, if applicable ("toll-free", "premium rate", "shared cost"): n/a

@rzaryan
Thank you for taking the time to improve libphonenumber!
We've accepted this issue and will fix this in an upcoming release.
Can you just confirm what carrier you expect this mobile range 65X to be for and could you provide any evidence for it?

Fixed in release 7.3.0
http://libphonenumber.appspot.com/phonenumberparser?number=%2B5016519885


Footer: moving Twitter, Slack and GitHub icons to the footer

A possible solution to this problem: Twitter, Slack and GitHub icons don't appear in mobile version.

Holy guacamole, this is awesome! Thanks.

Thanks @apandichi !! I think it's the best comment I've seen. I love "holy guacamole" :sweat_smile: and I think I'll use it as well :dancer:


Change banner logging to stderr

Hello,
I saw that https://github.com/ffuf/ffuf/pull/210 wasn't moving much.
I took the opportunity to make the change in code.
As discussed there, redirecting banner logging to stderr, nothing much.

A tiny change request still: I prefer to keep the CONTRIBUTORS.md sorted alphabetically :)


Move price per time calculations into the rust

Since variable penalties can affect time, exit movements that have price per time components will need to have variable prices (especially for #198 adding a movement field that would change which AV2 mist movement we'd use based on the additional conditions).

Instead of performing the calculations in the solver, we could simply precalculate the difference in costs to associate with each penalty. However, this won't work very well for more complicated cases, such as a penalty that changes the movement type and a penalty that adds time (and mostly doesn't work at all for penalties that calculate time).


Incorrect date format in IPTT filter dropdowns

Currently show "Jan 2017" (English only), needs to show appropriate localized month/year format

I don't think this is working as expected.

Dropdown dates are now in ISO format.


Clarified the use of the AIP client in coexistence with built-in labels (added paragraphs in the right order this time)

…ls (in the right order this time)
The vast majority of customers interviewed didn't know this critical scenario was an option, and as a result most of them were not using built-in labels in Office.

Hi @JohanFreelancer9, apologies for my ignorance, I committed the changes, but do I need to close the pull request? When I do it tells me the branch has uncommitted changes.


Theming color tokens for inputs

Refactor - ZInputs - theming color tokens

Motivation and Context
Semantic tokens were applied to allow inputs to adapt to themes.

Priority

[ ] 1 - Highest
[ ] 2 - High
[x] 3 - Medium
[ ] 4 - Low
[ ] 5 - Lowest
[ ] 6 - Not a Priority

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] Feature (non-breaking change which adds functionality)
[ ] Component (add a Component as approved dy Design System)
[ ] Docs (add documentation)
[x] Chore (changes that adds small enhancement)
[ ] Breaking (fix or feature that would cause existing functionality to not work as expected)

Abstract
Abstract link

Component and Fix approval flow to Master branch
A Pull Request to be merged must have two approvals, one from a member of the product team who developed it and another from a member of the dst dev team other than the team representative.

Hi Dario! it seems that some test are failing, could you check if is just me?

yarn test.e2e is still failing


Create a function to upload and scan receipts.

** User Story
As a developer, I want to obtain an API that can provide probable recipes based on the items scanned from a receipt or listed in user data so that I can offer recipe suggestions to users based on their available ingredients.
** Acceptance Criteria

Fetch Recipes Based on Items: The API should have a function to fetch recipes based on items that are listed, scanned, or enrolled in the user's data.
The function should accept a list of items (names or IDs) as input.
The function should return a list of recipes that use these items as ingredients.
The response should include recipe details such as title, ingredients, instructions, and any necessary metadata (e.g., cooking time, servings).
UI Update: The API should provide a mechanism to update the UI with the fetched recipes.
The UI should display the recipe titles and a brief summary initially.
Detailed views of recipes should be available, showing ingredients, steps, and other details.
Testing Functionality: Developers should be able to test the API integration to ensure recipes are displayed correctly.
The API should include test endpoints or a sandbox environment for testing.
The API should provide detailed logs or feedback on the fetching process and any errors encountered.


Badge is killed on close event

Hi, this is not really bug but wanted your help/thoughts on how to solve one issue I'm facing.
It'a interesting one, my client requested to override windows closing mechanism to mimic mac OSX behaviour (on X press don't close app - just minimize it). After I done that on first minimize/restore event'a badge stops working since inside lib mainWindow it's set to null on close event.
Some dirty workaround it to recreate it on restore event but I'm not happy with it.
Do you guys have in plan to support this use case? Maybe we can provide one flag within opts which will be responsible to setting this.win to null or not?
let me know what do you think.

I'm having the exactly same problem. Wouldn't it be better to set it to null on 'closed' event?

Yup same like skype when you click the X button it will hide only the app.. unless you quit on the taskbar.. What i did is temporary remove the code below under index.js

// this.win.on('close', () => { this.win = null; });

Fixed https://github.com/viktor-shmigol/electron-windows-badge/pull/9


Support for Kubernetes v1.31

How to categorize this PR?

/area open-source
/kind enhancement
What this PR does / why we need it:
Support for Kubernetes v1.31.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/10286
Part of https://github.com/gardener/gardener/issues/5325
Special notes for your reviewer:

I didn't vendor new k8s.io/* Golang sources, this item is usually tracked and worked on separately.
⚠ This PR only contains the Gardener part for supporting 1.31 - there will/must be follow-up PRs for each extension repository where individual support gets added.
Code changes have been tested in the following scenarios:

Create/Delete a new (normal/workerless) Shoot cluster with 1.31.0
Upgrade an existing (normal/workerless) Shoot cluster to 1.31.0

Release note:

Gardener can now support clusters with Kubernetes version 1.31. To allow creation/update of 1.31 clusters you will have to update the version of your provider extension(s) to a version that supports 1.31 as well. Please consult the respective releases and notes in the provider extension's repository.

Gardener can now support clusters with Kubernetes version 1.31. Extension developers have to prepare individual extensions as well to work with 1.31.

For Kubernetes 1.31+ Shoot clusters, the kubelet and containerd cgroup driver is set to `systemd`. Previously, the used cgroup driver was `cgroupfs`. Find more details in the [cgroup driver section](https://github.com/gardener/gardener/blob/v1.105.0/docs/extensions/operatingsystemconfig.md#cgroup-driver).

/assign

/test pull-gardener-integration

/test pull-gardener-integration

The last failure of pull-gardener-integration is a known flake: https://github.com/gardener/gardener/issues/10576

/test pull-gardener-integration

/assign

/lgtm

/approve
/retest


Feature: extend Dendron: Doctor -> createMissingLinkedNotes / findBrokenLinks to main vault links

Please select if your request is either something new or an enhancement

[X] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[X] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[ ] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[ ] Publish - External Site Publish
[ ] Markdown - Markdown features, such as syntax support and features

Is your feature request related to a problem? Please describe
If you use Dendron: Doctor -> createMissingLinkedNotes or  findBrokenLinks only works for links to other vaults [[Dendron://vault/some.note]] for example...
Describe the solution you'd like
IMHO this also must apply to links inside main vault [[some.note]] for eample
Describe alternatives you've considered
n/a
Additional context
n/a

could you submit a repro case? are you saying that missing linked notes don't work in the current vault?

this is Dendron Doctor -> findBrokenLinks

same for createMissingLinkedNotes
but maybe i misunderstood the feature as i expect it to list linked inexistant notes or autocreate them


Cursor Bug

My Cursor stops existing when i start the game with Vanilla Fix on pls help fast

How did you solve it?

Same issue. Menu basically doesn't work with the addon enabled.

are you guys not on patch 13? this mod broke when patch 13 came out, and the fix i did probably means it doesnt work on older versions anymore

okay. i had dgarroDC test this on 1.1.10 and he had the same problem. update your game to the latest version for vanillafix to work.
if updating to the latest version (1.1.13) doesnt actually work, please post a message here and ill work with you from there.

Didn't even realize I was on 1.1.10. Thanks for letting us know. It works on 1.1.13.


Show node help in a popup

Description
This adds the node 'help text' to a popup instead of always showing it. This makes the node list a lot more compact.
I've also added it to the node menu.
Demo

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Additional details and impacted files

:loudspeaker: Thoughts on this report? Let us know!


BUG: Text not visible in dark mode of about us section

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
text is not visible in dark mode of about us section specially when we hover over it

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anuragverma108
I have fixed this issue and created a pull request for the same . Please assign this issue to me and merge my pull request under gssoc label .


Add java-agent to /app/.java-agent

Pull java-agent into slugs so people can use it as needed.

Not sure which java-agent is referenced here. Heroku's regular metrics agent is nowadays managed by the https://github.com/heroku/heroku-buildpack-jvm-common/ buildpack. I'll close this as it seems outdated.


Add support for TiledJSON with base64 encoding

Tiled 0.13.0 added support for layer data compression when exporting as LUA or JSON. This means that any .tmx stored unsing base64 encoding will start exporting layer data as a base64 encoded string rather than a native array.
Phaser would try to load the level anyway without emitting any errors, resulting in a corrupted level while playing.
This PR adds detection and support for base64 encoded levels as long as they don't use compression.
Tiled commit adding base64 encoding: https://github.com/bjorn/tiled/commit/26a4675929e1622331a9b316291ee9283fc52a65

Nice, thanks for this. I guess we'll need a window.atob polyfill for IE9 though.


Abstract object implementation chekcs only for function name

Description:
The concrete implementation of a abstract clients only checks for function names when referring an abstract object . They are not validated against the input parameters and the return types of the functions. Same issue exists with the abstract objects as well.
The below code doesn't give an compiler error, but it should as the function contains different input parameters and return types.
public type AbstractSQLClient abstract client object {
    public remote function update(string queryStr, int count) returns int|error;
};

public type Client client object {
    *AbstractSQLClient;

    public remote function update(string sqlQuery) returns string|error {
        return "test";
    }
};

Same issue is there with abstrct objects as well.
public type AbstractSQLObj abstract object {
    public function update(string queryStr, int count) returns int|error;
};

public type Obj object {
    *AbstractSQLObj;

    public function update(string sqlQuery) returns string|error {
        return "test";
    }
};

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/16002


translator extension

I was wondering if the bundle supports the translator extension
http://knplabs.fr/blog/I(blah...blah...blah)n
It appears it has more power and better form support than the original translatable extension.
The extension is part of the doctrine extensions package ( i checked in github)
If the bundle also has support , please add that to documentation.

@l3pp4rd is there something needed for the Translator extension ? It does not use a listener.
I would like to figure it before merging #96 to make it part of the 1.1 release too (i.e. the version compatible with the extensions 2.2)

no, it does not use, translator uses a proxy to populate collection with translations, its very simple really. Translatable now is able to have direct collections, so symfony2 forms could use it directly also there are many features now possible to hook into domain objects

ok, so the only thing to support it is to add it in the doc ? Does it require some mapping or no ?

and regarding your mapping, you should not create entities again, you could use the mapping of extensions. and directly add it into the driver chain used by doctrine bundle. you can add additional paths to the annotation driver initialized by doctrine bundle. at least for those entities which are abstract mapped superclasses

there is new AbstractPersonalTranslation

I wanted to avoid messing with the mapping drivers of the bundles by adding stuff in a compiler pass in a hidden way. This would make things hard to maintain when changes are done in the bundles.
Another solution would be to require people to add the mapping explicitly for the gedmo extensions. The doctrine bundles support it but it will require more configuration for the end-user than just registering the bundle and activating the extensions.

well hooking the mapping paths in the same way like doctrine bundle does, should not be harmfull. because now you do something like this which would force me to make such methods like getTranslationClass now getPersonalTranslationClass do you think it looks good? :)

The issue is that the handling of the mapping driver is done in the DI extension for the Doctrine bundles. If I want to register the mapping for the library behind the scene, I have to do it after that in a compiler pass, which removes the possibility for the user to map it differently. And if something changes in one of the Doctrine bundles, it will require changing the compiler pass to keep them working.
I know the way to replace the class does not look good at all currently but doing it with a compiler pass does not seem clean either. The cleaner solution would be register the mapping through the doctrine config, but this is clearly not user-friendly


Always display the last visible line number

Before:

After:

This bug is not visible when using a font that triggers rendering path 1 (namely, Monaco, Courier and Consolas).
Basically, the problem is that -[NSString drawInRect:withAttributes:] clips every line that does not fit entirely in the given NSRect.

Thanks!


Participation | Hacktoberfest 2022

Would you mind registering @toss/slash as a Hacktoberfest participant?

 Maintainers need to add the hacktoberfest topic to the repository, or add the hacktoberfest-accepted label to your PR/MR.For more information about how exactly we validate PR/MRs for contributors, check the details on our participation page.

We have just added the hacktoberfest topic to our repository. Can you have a look?

Awesome! It is now working. Thank you for your help!


rescale.m

rescale.m called by esfit.m does have the same name as Matlab function "rescale" introduced in Matlab R2017b, in result, esfit fails when FitOpt.Scaling is defined

This is an unfortunate consequence of MATLAB's design where all functions share the same namespace.
One workaround is to make sure that the EasySpin folder easyspin-x.y.z/easyspin is on the MATLAB path before the path to the other function (toolbox/matlab/datafun I believe). In this case, MATLAB's rescale will be shadowed, and esfit will call EasySpin's rescale.
For EasySpin version 6, we can consider renaming the function to avoid this clash.

When pressing F1 on rescale in the editor, the documentation page for MATLAB's rescale function appears, even though it is shadowed:
>> which -all rescale
C:\easyspin\easyspin\rescale.m
C:\Program<EMAIL_ADDRESS>% tall method
C:\Program Files\MATLAB\R2021a\toolbox\matlab\datafun\rescale.m        % Shadowed

Tested on R2021a.
It looks like we might need to rename the EasySpin function.

Thank you, it is probably easier to rename the function that most of the users of EasySpin do not use directly instead of searching for workarounds.

I filed a service request with MathWorks regarding this. Response is that they won't fix this behavior. So EasySpin has to rename the function.


Ensure beacon node cleanly exits

Describe the bug
When starting the chain without a genesis state, not all modules are started, since BeaconChain blocks on start until a genesis state has been created.
If the node is stopped during this time, the node is not stopped properly, because other modules' stop method assumes that all modules have been started.
Expected behavior
Beacon node should exit cleanly, even if all modules have not started.
Steps to Reproduce
start lodecli, using eth1 to trigger genesis, eg: using https://hackmd.io/2wNvrO77RQ6BWc2ZA7M0Cg
Once the node has started, hit Ctrl+C
Instead of exiting, an unhandled promise rejection is thrown

I'd be happy to dig into this, what do you think?

Oh yea, go for it

Hey, this might be a larger discussion better had elsewhere, but I noticed that strictNullChecks is set to false in the main tsconfig. What's the reasoning there? This issue would have been caught already if strictNullChecks were true

There are a couple ways to go about this from what I can tell. There seem to be a lot of places where we aren't checking if variables hold values that are defined, leading to this situation and probably many others that are currently hidden. I could just put in some definition checks and call it good, but I'm running into a bit of a problem with the libp2p-gossipsub library.
If you try to call the stop method of a GossipSub instance, it will actually throw an error because this._heartbeatTimer is not defined. So it looks like GossipSub believes it is invalid to call stop if start hasn't been called. I'm not sure how to get around this, is there some public state in a GossipSub instance that I can use to determine if stop needs to be called?
This problem leads me to another potential solution (not sure if it would be a full solution). Instead of just doing these definition checks, what if each of these modules kept a started state within them. When a module/service starts, we can set started to true. If you call stop and started hasn't been called, then stop will do nothing. This could help

re: strictNullChecks, not sure there's a good reasoning for it being false. Definitely think we should rip the bandaid off at some point.
I think tracking the started state in each module would help, we already do this in a few modules (eth1 and sync).

re: strictNullChecks, not sure there's a good reasoning for it being false. Definitely think we should rip the bandaid off at some point.

When do you think we should rip the bandaid off? Should I open an issue for this? Will probably take some significant tedious effort across the entire codebase

Yeah, please do. :) I think we should do this as soon as feasible, as it won't be getting any easier the longer we wait. Agreed, it'll be a tedious kind of thing.


sfpowerkit:project:diff  project package filter not working correctly

Describe the bug
Given a project directory structure:
- sfdx-src
---env-src
-------dev
-------devops

and  sfdx-project.json package has the sfdx-src/env-src/dev path for a package
When  a change is made in sfdx-src/env-src/devops
Then the package build is not excluding the change.
To Reproduce
Steps to reproduce the behavior:

create the above project structure
commit a change in the devops folder
run command sfdx sfpowerkit:project:diff -r HEAD~1 -d powerkitdif

Expected behavior
No package diff to be built as the change is made in the devops path which is not a path in sfdx-project.json
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS Big Sur
Node Version
Version of sfpowerkit[e.g 1.45.0]  - 3.2.3
Salesforce Org Version - NA

Additional context
Add any other context about the problem here.

@jacobpeterson2012  is sfdc-src in the project path as well?

@azlam-abdulsalam no it's not a project path.


Update width and height the gif in general

Not sure why you're suggesting this change. Please provide more context or keep it in your own fork.


Add debug option to runtests

Running runtests with debug=true will cause mocha to run in debug mode and break on the first line, enabling one to connect a debugger to the running tests and step through a test at their leisure.

:+1:

:+1:


Add morgsdaly's file

created a branch, created a file and made a commit, and opened a pull request

good job.


There is no feedback when provisioning a new device

Type of issue

[ ] Bug
[ ] New feature
[x] Improvement

Description
When provisioning a new device and clicking the "Apply" button, the UI doesn't give any hint to the user, about the operation being complete or not, successful or not, leaving you wondering if you have to retry or what...

There is a currently open issue that should address this https://github.com/Azure/pcs-remote-monitoring-webui/issues/468 So this issue seems like a dup.

@spryor the issue is a little different. #468 is about showing the Connection String, which one needs to connect the device to ioT Hub.  The issue here is about having a visual clue that the user action was completed.

Can someone assign this to me please?


Accuracy of ct clamp figures

The ct clamp figures not be converted correctly. For example:
CT 1 Internal Load 0W
CT 2 Grid 122W
CT 3 Generation -25W
CT 4 None 0W
CT 5 None 0W
CT 6 None 0W
Supply voltage: 231.3V frequency: 49.94Hz
Power:
Grid      : 115W
Generated : -25W
at that time the generation figure was 2.5kw, house consumtiption was 2.5kw and grid 0.1kw.

Re-open if still an issue, probably just misconfigured though?


Addition of Resources for learning Cybersecurity

Related Issue

Closes: #98
Description of Changes

Added a new card for Cybersecurity Resources.
Also added 5 books resources and 2 Youtube videos and 1 full cybersecurity course.

Checklist:

[x ] My code adheres to the established style guidelines of this project.
[x ] I have conducted a self-review of my code.
[x ] I have included comments in areas that may be difficult to understand.
[x ] I have made corresponding updates to the project documentation.
[x ] My changes have not introduced any new warnings.

Screenshots

@kartikkc  @Astrodevil  @bhartik021  Please review it . Thank You

@kartikkc @Astrodevil Please review it . Thank you


Upgrade prettier to latest

The current version of prettier (1.7.0) had a bug that was breaking the pre-commit hook so I've upgraded it to the latest.

Released in 0.24.2 🎉