Add fine-grained permission checks for the edit item page

Describe the bug
The work we did for #1047 made it clear that apart from edit metadata, which that ticket addressed, there are also a lot of other parts of the edit item page and its tabs that can use more fine grained feature checking
Expected behavior
After we've come to an agreement and have an implementation for https://github.com/DSpace/DSpace/issues/3201 we should go over the edit item page and disable the tabs and certain individual features unless you have permission to use each specific feature. Here's an overview of the tabs and the features that are relevant for the tab itself or the features inside

Status

canManageMappedItems  (proposed in https://github.com/DSpace/DSpace/issues/3201)
withdrawItem
reinstateItem
canManagePolicies
canMakePrivate
canDelete
canMove

Bitstreams

canManageBitstreams (proposed in https://github.com/DSpace/DSpace/issues/3201)
canReorderBitstreams
canCreateBundle
canCreateBitstream

Metadata

canEditMetadata (will be the case after the PR for #1047)

Relationships

canManageRelationShips (proposed in https://github.com/DSpace/DSpace/issues/3201)
canEditMetadata (will be the case after the PR for #1047)

VersionHistory

canManageVersions (proposed in https://github.com/DSpace/DSpace/issues/3201)
canSeeVersions
canCreateVersions

CollectionMapper

canManageMappedItems  (proposed in https://github.com/DSpace/DSpace/issues/3201)

This means that we'll also need a way to check more than one feature in a guard for route (e.g. for the status route we need to check if the user is authorized for at least one of the features mentioned). This is currently not the case.
Related work

#1047
https://github.com/DSpace/DSpace/issues/3201

We'll need to:

Add the ability to combine features for a guard in an OR query
Implement the guards for the routes
enable/disable the tabs and buttons on the edit item page itself based on authorization for these features.

This will take an estimated 16 hours

Thanks @artlowel .  I'll tentatively schedule this for 7.0 (to be built during/after Testathon), and assign back to your team.


Implement Python classes for SOMA arrays/groups

As of the initial 'ingest-only' version we have at present, we have a single SOMA class with lots of methods. Those methods should be moved to individual classes:

For parity with tiledbsc-r
To facilitate comparisons tiledbsc-py <-> tiledbsc-r
To make it easier to complete readback/outgest logic
To faciliate development of the nascent SOMA-query API

#78 is merged.


Pickle to json

changed the pickle file into json files in the following tests (related to issue #366 )-

test.accreditation.test_accred.TestAccred.test_accred_fitter
test.measurement_calibration.test_meas.TestMeasCal.test_tensored_meas_fitter_with_noise
test.quantum_volume.test_qv.TestQV.test_qv_fitter
test.rb.test_fitters.TestFitters.test_fitters
test.rb.test_fitters.TestFitters.test_interleaved_fitters
test.rb.test_fitters.TestFitters.test_cnotdihedral_fitters
test.rb.test_fitters.TestFitters.test_purity_fitters

in addition, removed the skipping of these tests (related to issue #402 )

note that in some of the tests, a data generation functions still need to be added (will be in separated PR, in order to remove the skipping of the tests first)

Details and comments
in the accreditation test folder there was a generation data file - so I changed it to save the generated data in json format.
there is no big change in the performance after the change to json files (checked run times only on my laptop), and the json files are little bigger than the pickle files, but only in some KB.
some numbers:
Qv:
Sizes (in KB):
Pickle: 21, 32
json: 20, 39
Run times (in seconds):
json : 0.082
Pickle: 0.072
Measurement:
Sizes (in KB):
Pickle: 3, 6
json:  4, 5
Run times (in seconds):
Pickle: 7.9
json: 7.8
Accreditation:
Sizes (in KB):
Pickle: 118, 118
json: 125, 126
Run times (in seconds):
Pickle: 0.3
json: 0.35

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

Any idea why license / cla is still pending?


Add logging override for 0.7

0.7 has a new logging infrastructure, which saves the output stream (e.g. Base.stderr) within itself.  We need to override that if we want to receive things like warn() or info().  This is not the most elegant way this could be done, but it does seem to work (and passes its own test suite on 0.7)

Hmmm, although things seem to be getting captured, they do not seem to be getting communicated to the browser.

Ah, found it.  String(d) was clobbering d, so we need to copy it first.

See also #26798.

Closing in favor of #671


Electron support - Question

Does the plugin support electron? If so, how can I configure it?

If Capacitor works on electron, I can't think of a reason why it wouldn't work. As for configuration, I know nothing about Electron and don't have time to find out, sorry.


[CAMEL-17310] Use absolute Path.

Thanks I have continued this work in another PR that fixes the issue with classpath vs file
https://github.com/apache/camel/pull/6532

I see, I'm closing this PR then, thanks


Trying to go to /admin/plugins/slack & get a error

When I go to /admin/plugins/slack it gives me a error that something went wrong.
From the console.
TypeError: Discourse.ajax is not a function

__exports__["default"] = Discourse.Route.extend({
      model: function () {
        return Discourse.ajax("/slack/list.json").then(function (result) {
          var final = result.slack;
          return final.map(function (v) {

Discourse Version:
1.7.0.beta6
This is also the first plugin I installed.

Fixed in  82dad24


Fixed parts list scrolling issue. Added CTRL+SPACE shortcut to open parts list

This PR adds two (in my opinion) improvements:

Added additional parts list shortcut (Ctrl+Space) in addition to the original one (Ctrl+P). This change makes opening parts list with one hand possible and massively improves workflow speed (because opening parts list is one of the most used actions).
Fixed scrolling the parts list. Previously scrolling had to be done by dragging a scrollbar instead of a mouse scroll. Now it works also with a mouse scroll without closing a parts list window.

Good improvement thanks!


chore: Cleanup claims slashing

I think we won't need to merge this PR; just update the umee version once we get this PR merged: https://github.com/umee-network/umee/pull/349

Do not merge. Old stuff


Got the bots working with 7.37d

The bots didn't work with 7.37d. Got these kind of errors in the console:
LoadScript: ...apps\workshop\content\570\3246316298\hero_selection.lua, scope name HeroSelectionScriptScope
[VScript] Script Runtime Error: ...eamapps\workshop\content\570\3246316298\FunLib\utils.lua:343: module 'bots.ts_libs.dota.index' not found:
    no field package.preload['bots.ts_libs.dota.index']Failed to find scripts\vscripts\bots/ts_libs/dota/index.lua
stack traceback:
    [C]: in function 'require'
    ...eamapps\workshop\content\570\3246316298\FunLib\utils.lua:343: in main chunk
    [C]: in function 'require'
    ...p\content\570\3246316298\FunLib\aba_global_overrides.lua:2: in main chunk
    [C]: in function 'require'
    ...mapps\workshop\content\570\3246316298\hero_selection.lua:1: in main chunk
Failed to run script '...apps\workshop\content\570\3246316298\hero_selection.lua'!

I changed these kind of lines:
local ____dota = require("bots.ts_libs.dota.index")
to
local ____dota = require(GetScriptDirectory().."/ts_libs/dota/index")
And the bots work again!

Thanks a lot for reporting this issue with the exact details needed and even with the fix you came up with. Appreciated.
It would probably take a much longer time to realize the normal game play isnt gonna work because of this issue. This is due to the fact we tried to introduce typescript to the code base and in typescript we use bots as the filepath to resolve the path for ts and thus this caused issue for the generated lua files.

Addressed this in commit https://github.com/forest0xia/dota2bot-OpenHyperAI/commit/d6307a31a56b5c3717d046483b3191d486f2c3fb

@mikkokeskinen please try to use the latest version available in Workshop see if that works. maybe re-subscribe to the script. I want to make sure the version there is working for ppl besides me.


PEP 519: Path protocol support

Uproot currently does not support pathlib, so something like this will fail:
from pathlib import Path
import uproot
p = Path('/data/file.root')
f = uproot.open(p) # Error

The path and similar functions should use name = os.fspath(name) if os has fspath (depends slightly on what sort of objects uproot accepts here). (This is the correct solution for Python 3.6+, and will also support other path objects in libraries like Plumbum; anything that provides a __fspath__() method. It won't fix the pathlib backport on Py2, or Python < 3.6.)
See https://www.python.org/dev/peps/pep-0519 .

Something new for me to learn about! I'll be doing an OS upgrade soon, which would update my personal Python from 3.5 to 3.6, where I'll learn about that language feature and add support for it in uproot. If delayed, I'll just use Miniconda for testing, which I think is 3.6. is this urgent? (It should be easy to do a pull request, which Travis would test against 3.6...)

No, not urgent; it can be bypassed by the user by converting to strings; it just something I'd like to avoid having to teach students to do in the future.
I could do a pull request but I'm not sure what other objects you might expect here. Can you pass in streams, for example?

It takes a string or an iterable of strings and expands each by glob pattern (if not a URL) into a list of strings. I'm not sure what pathlib does, but it's probably a strongly typed version of that.
Filename handling happens in two places: uproot.open and uproot.iterate, and I'm pretty sure they both call the same internal function.

Great, thanks! I'll give it a test within a day or so, ping me if I don't respond in that timeframe!

PS: This should also work with Pathlib2, for example, in Python 2.6+; I'd recommend that you not check for the exact name (and there are other path libraries). Can I suggest the folllowing change:
- if path.__class__.__module__ == "pathlib":
-          import pathlib
-          if isinstance(path, pathlib.Path):
-              path = str(path)
+ if hasattr(path, '__fspath__'):
+     path = path.__fspath__()

The Python 3 pathlib has no __fspath__(), so I shouldn't substitute it entirely. Should I check both?
By the way, I check path.__class__.__module__ == "pathlib" because I can't import it (it might not exist because I might be in Python 2) and this string check is exactly as safe as import pathlib when it does exist— that also checks for the module by name!

Tell me what you think of the above change. We can test for path.__fspath__ for pathlib libraries that put a __fspath__ method on the path object and then afterward check for a class in module "pathlib" that is a subclass of pathlib.Path.

At least it's not wrong. I'm closing this now so that I have an easier time refactoring for uproot 3 (that's why I was handling all of these issues, to clear the board).

I would use elif to save one check, but otherwise that looks fine. pathlib does have __fspath__, but not on older Python 3's (Pre 3.6, I think).

Because I wasn't sure, I didn't want to force the two checks to be mutually exclusive. Perhaps the output of __fspath__ can never be a pathlib.Path; using elif would always make that assumption. I guess you're saying it's a safe assumption; I can change that later.
Also, I decided that having an __fspath__ should trump being a pathlib.Path, since there could be third-party libraries satisfying that interface.
I can confirm that pre Python 3.6 (mine is 3.4), pathlib.Path doesn't have a __fspath__.
My reason for pushing this through is to not delay the uproot 3 refactoring. I'd really like to hear your feedback about my e-mail to Scikit-HEP Admins (you're on that, right?), since it involves hepvector. (If you're not on it, I'll forward you the e-mail.)

The output of __fspath__ is Union[str, bytes], so it cannot be a Path, so that saves a check.
Before Python 3.6 users pretty much were always required to wrap Path in str to use it, since it doesn't even work in the builtin functions like open() without adding str. That's why that PEP was written, and that's why I wasn't really that worried about Python < 3.6 pathlib; the behavior would at least be consistent. After 3.6, though, it is assumed that you don't have to add the str() anymore, since the __fspath__() method was added and supported throughout the standard library.
I'm on the email list, just haven't had time to write a proper response. Overall, it looks great, and I like the modularization of functionality. I'll reply tommorrow!

FYI: This is the way matplotlib is going to do it: https://github.com/matplotlib/matplotlib/pull/11307/files

I'm reopening the issue to remind myself to change the code this way. For the record, I'll be changing
if hasattr(path, "__fspath__"):
    path = path.__fspath__()
if path.__class__.__module__ == "pathlib":
    import pathlib
    if isinstance(path, pathlib.Path):
         path = str(path)

(in both places) to
if isinstance(fname, getattr(os, "PathLike", ())):
    fname = os.fspath(fname)
elif hasattr(path, "__fspath__"):
    path = path.__fspath__()
elif path.__class__.__module__ == "pathlib":
    import pathlib
    if isinstance(path, pathlib.Path):
         path = str(path)

I've tested Pythons 3.4 (Ubuntu 14.04) and 3.6 (Miniconda): 3.4 has no os.PathLike or os.fspath, so it would miss the first block. 3.4's pathlib.Path also lacks a __fspath__ method, so it would miss the second block as well. The third block only works for libraries named pathlib with a pathlib.Path superclass, which might not include the Pathlib2 you mentioned. This should maximize the coverage matrix of Python versions and standard vs. third party libraries enough to say that "uproot supports pathlib" and any exceptions can be reported as bugs and dealt with as additional cases.
Given the way things are going, I'll probably sneak this change in with other uproot 3.0 refactorings, without going through the usual branch-merge-deploy cycle.

By the way, for the record, check 1 and 2 are identical on Python 3.6+, PathLike is an Interface that simply requires that one method be implemented, __fspath__(). However, pre-Python 3.6, they will be different if you are using a separate path library like pathlib2, plumbum, etc.; so I don't think it hurts to have it.


Release/6.0.0

Checklist

[x] I've checked that all new and existing tests pass
[x] I've updated the documentation if necessary
[x] I've added an entry in the CHANGELOG to credit myself

Description
This branch contains the next release 6.0.0 of OHHTTPStubs.
This also closes #228 & #222.
Motivation and Context
This PR is to ensure the 6.0.0 release branch compiles correctly in all environments, by letting the CI matrix check them all, before officially making the release.

@Liquidsoul 👌
I've added one last commit to add more information for Carthage users. I think that:

given most people have migrated to Swift 3.0 already
and that people still using Swift 2.3 won't be in a hurry migrating to OHHTTPStubs 6.0.0 if they haven't migrated to Swift 3.0 yet and those people will still use 5.2.3
it's time-consuming to maintain a specific "swift-2.3" branch just for those users (which are likely not many, as projects in Swift 2.3 and not migrated to Swift 3.0 yet is rare by now)

Then I've decided to stop doing swift-2.3 specific branches just for so few people. Those people have alternatives anyway, and I don't want to lose time maintaining that old branch any longer for an obsoleted Swift version which is already not supported anymore in latest Xcode.


Error calling NCO command with tempfile helper

I am attempting to use the Python NCO bindings for calling an NCO command using the tempfile helper feature. For example:
nco = Nco()
temp_netcdf_file = nco.ncpdq(input=arguments.netcdf_precip,
                             options=["-a \"-lat,lon,time\""])

My expectation from the above is that I will receive a string value with the name of the temporary NetCDF file created as a result of the NCO command. (Or maybe it will be a file object? The return type for this usage is unclear from the documentation.)
When I run the above I get an error:
nco.nco.NCOException: (returncode:1) b''
When I step through the nco.py code I see the command appears to be properly parsed:
cmd: <class 'list'>: ['/home/james/miniconda3/envs/climate/bin/ncpdq', '-a', '"-lat,lon,time"']
However, when the command is called (line 263) it gets unexpected return values, resulting in the above-mentioned exception.
I can run the command in a terminal on Linux (Ubuntu 18.04) without issue, although in that case I am specifying an output file.
I have seen similar behavior the previous time I attempted to use this package, perhaps these two issues are related?
Thanks in advance for any help with this.

I think this usage should be returning a path to a temporary. I'm not actively working on this package but would be happy to help others fix any current problems.

Thanks, @jhamman . There are several failing tests that I have fixed, plus a few more cleanups (PEP8, unused variables, etc.) that I've worked in. I will submit a PR soon. I appreciate your help.

BTW while testing this with a Python2.7 virtual environment I am seeing some cases where the NCO executable path is being set from the OS environment's PATH, and in that case it doesn't find the NCO executables installed in the virtual environment but rather from the base OS location instead. This is problematic if NCO is installed into the OS but ncdump is not, i.e. when the NCO path is set it uses the location where ncks is located in the OS, such as /usr/bin, and when the test for ncdump -h is run it fails since ncdump isn't located under /usr/bin (file not found error). Maybe there's something amiss with how I have my PATH being set when running these tests under a 2.7 (Anaconda) virtual environment? I've tried this again using the previous virtual environment I was using before and seeing the same behavior. I'm not sure why this was working well before, i.e. it was locating the ncks in the virtual environment before the OS location and using the virtual environment's path to ncks as the NCO path, and since this matched with where ncdump was installed in the venv all was good. This behavior has changed since creating a new Python2.7 virtualenv using conda, and I assume that this has goobered up some PATH settings that were causing the venv path to be used, which I assume is what we want? I think the underlying issue for the test at play (test_operator_prints_out()) is that it tests ncdump using the path to ncks, and the two could be installed in separate locations since ncdump isn't part of NCO per se. Or am I missing something obvious?
TL;DR: The 2.7 support is a bit hosed, while trying to fix that I have run into issues with finding NCO path under the new 2.7 conda environment I've created for this. One test involving ncdump is failing since the path to ncdump isn't always the same as the path to ncks, especially in cases when NCO and/or ncdump is installed only within a virtual environment and not on the base OS.


Text generation issue with missing characters

Hello, I wanted to report an issue with non-English language output after a recent patch. I'm a Korean user and frequently utilize the Korean language model. Until recently, I had no problems, but when using the latest version of llama-cpp-python, I noticed that characters are occasionally missing within the generated text.(I'm using 0.1.55)
It seems to be related to the decoding process of tokens in the _create_completion function in llama.py, although I'm not entirely certain as I'm a beginner in coding. When I replace that function with an older version, the issue of missing characters disappears, leading me to suspect that it might be the cause. However, I haven't tested it with other foreign languages that have different character systems, such as Chinese or Japanese, so I'm not sure if it's specific to Korean. But when generating Korean text, the problem occurs consistently.
I would greatly appreciate it if you could look into resolving this issue. Many of my activities heavily rely on llama-cpp-python, so it would be quite inconvenient if I can't use the latest version. Thank you for your attention.
Example:
When inferring the Korean translation of 'Beautiful Woman', it should result in '아름다운 여자', but instead it produces '아다 여자'. All the intermediate '름' and '다's are missing.

I also confirm it happens in Japanese.
After doing some experiments, I figure out below.

it happens after 0.1.51
it only happens in streaming mode

I found there were huge updates in llama.py for decoding multi-byte characters at streaming mode, between 0.1.50 to 0.1.51.
I suppose this causes the issue.

@1980Dragon @nai-kon I'm really sorry about that, I had to re-factor some of the completion logic and the previous fix for multi-byte unicode sequences was incompabitble with the changes. I'll try to migrate it over to the new logic, could you please provide me with some examples of expected inputs and outputs? Much appreciated.

Thank you for your response. As a beginner in coding, I am not sure about the most effective way to present information. However, I have attempted to create a few examples below as a trial, but I am unsure if they are sufficient. If more examples are needed or if a different format is required, please let me know. Thank you once again.
WizardLM-7B-uncensored.ggmlv3.q5_1, using Text generation web UI.
[Format]
Question: Translate '{English Word}' to Korean.
Translation: {expected output} {current output}
Question: Translate 'Hello' to Korean.
Translation: 안녕하세요 (안하세요)
Question: Translate 'Computer' to Korean.
Translation: 컴퓨터 (터)
Question: Translate 'America' to Korean.
Translation: 아메리카 (아리)
Question: Translate 'Love' to Korean
Translation: 사랑 (사)
Question: Translate 'Morning' to Korean
Translation: 아침 (아)

Thank you for your reply.
Due to the LLM's reproducibility, I can't come up with the best provide way, but 1980Dragon's translation examples seems good idea.
Here is an example in Japanese.
Translate "newspaper" to Japanese.

expect:新聞
current:新

Again, Thank you for this great work!

Hello, I also confirmed same issue in Chinese.
And I figure out it happens since this commit: dc39cc0fa410f8b46954ad507b705052947da6bc
Hope it helps, thanks

@abetlen  Hi, I have a simple fix for this bug: #309.
Although it's just a draft, it should serve as a reference.
Thanks

@abetlen ping

Just tested @MeouSker77 's draft fix works for Korean/Japanese/Chinese

Great. Thank you, everyone. ❤

@1980Dragon btw the draft fix is not merged yet

@1980Dragon Please re open this issue. Don't close this until the PR merged.

Oh, sorry. I'm not familiar with the workflow on this GitHub. I've reopened the issue.

@abetlen Hi, I have a simple fix for this bug: #309. Although it's just a draft, it should serve as a reference.
Thanks

I tried this modification in oobabooga/text-generation-webui and it worked fine.

I hope to express through this message that the Pull Request (PR) submitted by MeouSker77 can be beneficial to me. This PR is extremely important for the output of Traditional Chinese characters as it can resolve the existing issue of missing characters.
While using the current version, we found that occasionally, there would be missing characters when outputting Traditional Chinese text. This not only affects the user experience but can also lead to the loss or misinterpretation of important information.
In MeouSker77's PR, they made some adjustments and optimizations to the code to ensure that no characters are missed when outputting Traditional Chinese text. We have conducted multiple tests on this PR and confirmed its effectiveness and stability.
Therefore, we strongly recommend that you consider this PR, as it not only solves a significant issue but also enhances the user experience.
Before:

After

#309 has been merged!

This issue still confused me much when I use the llama-cpp-python to load model ,anyone can fix the problem?
args.model_name_or_path='TheBloke/WizardCoder-33B-V1.1-GGUF'
llm = Llama.from_pretrained(
repo_id=args.model_name_or_path,
chat_format="llama-2",
        filename="wizardcoder-33b-v1.1.Q6_K.gguf",
        n_ctx=12000,
        tokenizer=tokenizer,
         n_gpu_layers=-1,

        verbose=True
    )
    output = llm.create_chat_completion(
        messages=[
            { "role": "system", "content": """You are an AI that follows instructions extremely well.
            Help as much as you can. Remember, response in Chinese.

"""},
{
"role": "user",
"content": user_prompt
}
],
stream=True,
max_tokens=12000,
repeat_penalty=1,
    )

注意，SQL语句中的字段名、表名都需要用反引号`包�起来，并且日期格式需要与数据库中的日期格式一致。 生成耗时：30.285314559936523,文字长度：353,每字耗时（sec/word）：0.08579409223778052


Broken Breadcrumb Links

@teddybradford - we have 12 breadcrumb links on http://welcomegrad.risd.edu/ that are broken and I believe they can only be fixed at the template level, as I checked the CMS: https://risd-graduate-welcome.risd.systems/cms and don't see where I can edit them.
Here is a spreadsheet that tells you what the broken links are, what pages they are located on, includes small screenshots of where they are on the page, and what urls the links should be fixed with. Can you please fix these? Thanks!
welcome_grad_broken_links_2_8_18.xlsx

Community link still is incorrect at http://welcomegrad.risd.edu/learning-environment/community/vernacular-spectacular/

@room421 I think you may have looked prior to the new change propagating to the live site. It seems to be working fine for me. Can you confirm that it's working for you now?

You're right. Totally good now. Thanks!

oh, wait. One last one.
http://welcomegrad.risd.edu/community/supporting-your-research/the-gravedigger/

Home link is wrong.

Squashed it!


How to programmatically save photos?

How can i programmatically create a photo with photologue. This is what i tried but i'm getting an error.
from django.core.files.base import ContentFile
import urllib2
from photologue.models import Photo

file = urllib2.urlopen('http://placehold.it/350x150')
photo = Photo(title='Photo Title', slug='photo-title', image=ContentFile(file))
photo.save()

Error i'm seeing:
No handlers could be found for logger "photologue.models"
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/photologue/models.py", line 555, in save
    super(Photo, self).save(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/photologue/models.py", line 506, in save
    self.pre_cache()
  File "/usr/local/lib/python2.7/dist-packages/photologue/models.py", line 474, in pre_cache
    self.create_size(photosize)
  File "/usr/local/lib/python2.7/dist-packages/photologue/models.py", line 413, in create_size
    if self.size_exists(photosize):
  File "/usr/local/lib/python2.7/dist-packages/photologue/models.py", line 366, in size_exists
    if self.image.storage.exists(func()):
  File "/usr/local/lib/python2.7/dist-packages/django/utils/functional.py", line 13, in _curried
    return _curried_func(*(args + moreargs), **dict(kwargs, **morekwargs))
  File "/usr/local/lib/python2.7/dist-packages/photologue/models.py", line 344, in _get_SIZE_filename
    return smart_str(os.path.join(self.cache_path(),
  File "/usr/local/lib/python2.7/dist-packages/photologue/models.py", line 309, in cache_path
    return os.path.join(os.path.dirname(self.image.name), "cache")
  File "/usr/lib/python2.7/posixpath.py", line 129, in dirname
    i = p.rfind('/') + 1
AttributeError: 'NoneType' object has no attribute 'rfind'

Hi,
I don't think that you can attach an image file when you create a model instance - the way I usually do it is to create the model instance & then save the image file to the instance. See here for an example.
But I'm happy to be proven wrong if someone suggests a quicker route!

thank you that did the trick.


Acceptance Testing: Google Maps

Google maps characters invalid.

Hi there, this issue seems to be isolated to Mac users only. Our team member who is using a Mac also has this issue.

Yup, this issue seems to be faced by Mac users only due to font incompatibility. We have tried changing the font in the browser panel to a compatible one but to no avail since the issue seems to stem from the Google Map search engine itself.


[Bug]: Unable to tick "Auto Cleanup"

Issue validation

[X] I checked the issue to prevent duplicate
[X] I checked my configurations files and the documentation
[X] My issue concern doesn't concern the Mkdocs template

Command used
No response
Plugin version
6.11.3
Describe the bug
When trying to tick "Auto Cleanup", nothing happens.
How to reproduce ?
No response
Minimal Reproducible Example
There's no example as it's part of the settings.

Configuration
{
  "github": {
    "branch": "main",
    "automaticallyMergePR": true,
    "tokenPath": ".obsidian/plugins/obsidian-mkdocs-publisher/env",
    "api": {
      "tiersForApi": "Github Free/Pro/Team (default)",
      "hostname": ""
    },
    "workflow": {
      "commitMessage": "[PUBLISHER] Merge",
      "name": ""
    },
    "verifiedRepo": true
  },
  "upload": {
    "behavior": "yaml",
    "defaultName": "",
    "rootFolder": "",
    "yamlFolderKey": "",
    "frontmatterTitle": {
      "enable": false,
      "key": "title"
    },
    "replaceTitle": [],
    "replacePath": [],
    "autoclean": {
      "enable": false,
      "excluded": []
    },
    "folderNote": {
      "enable": false,
      "rename": "index.md",
      "addTitle": {
        "enable": false,
        "key": "title"
      }
    },
    "metadataExtractorPath": ""
  },
  "conversion": {
    "hardbreak": false,
    "dataview": true,
    "censorText": [],
    "tags": {
      "inline": false,
      "exclude": [],
      "fields": []
    },
    "links": {
      "internal": true,
      "unshared": false,
      "wiki": true,
      "slugify": true
    }
  },
  "embed": {
    "attachments": true,
    "forcePushAttachments": [],
    "keySendFile": [],
    "notes": true,
    "folder": "Attachments",
    "convertEmbedToLinks": "bake",
    "charConvert": "->",
    "bake": {
      "textBefore": "",
      "textAfter": ""
    },
    "useObsidianFolder": true
  }
}

### Relevant log output

```bash session
Nothing happens in console when I try to do that.

OS
No response
Anything else?
No response
Obsidian information
SYSTEM INFO:
	Obsidian version: v1.4.16
	Installer version: v1.4.16
	Operating system: #38~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Thu Nov  2 18:01:13 UTC 2 6.2.0-37-generic
	Login status: not logged in
	Insider build toggle: off
	Live preview: on
	Legacy editor: off
	Base theme: dark
	Community theme: Minimal v7.3.5
	Snippets enabled: 1
	Restricted mode: off
	Plugins installed: 41
	Plugins enabled: 39
		1: Advanced Slides v1.20.0
		2: Advanced Tables v0.19.1
		3: Better Command Palette v0.17.1
		4: Better Export PDF v1.0.5
		5: Calendar v1.5.10
		6: Callout Manager v1.0.1
		7: Clear Unused Images v1.1.0
		8: Copy Image and URL context menu v1.5.1
		9: Creases v0.7.0
		10: Dataview v0.5.64
		11: DB Folder v3.5.0
		12: Footnote Shortcut v0.1.3
		13: Filename Heading Sync v1.9.0
		14: Github Publisher v6.11.3
		15: Heading Shifter v1.5.1
		16: Hide Sidebars on Window Resize v1.1.0
		17: Hot Reload v0.1.8
		18: Icon Shortcodes v0.9.7
		19: Iconize v2.8.0
		20: Icons v0.3.0
		21: Jump to link v0.4.4
		22: Linter v1.20.1
		23: Minimal Theme Settings v7.3.1
		24: Mousewheel Image zoom v1.0.19
		25: Number Headings v1.16.0
		26: Omnisearch v1.19.0
		27: Open vault in VSCode v1.2.1
		28: Outliner v4.8.0
		29: Paste URL into selection v1.7.0
		30: Periodic Notes v0.0.17
		31: Remember File State v1.1.1
		32: Smart Typography v1.0.18
		33: Style Settings v1.0.7
		34: Tab Switcher v1.4.0
		35: Workspaces Plus v0.3.3
		36: Zoom v1.1.2
		37: Obsidian Git v2.22.0
		38: Vertical Tabs View v2.4.5
		39: Imgur Plugin v2.4.0

RECOMMENDATIONS:
	Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
	Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.

Also thank you! This is an amazing plugin.

Please, read the docs before posting.
You need to set a root folder to enable this.


Initial setup -- copied from hello-world.g8

Copied setup from hello-world.g8.
The template still needs to be customized for twitter4s

LGTM


Flux Tool Redux style

I see the loader but no output selector? I see style applied on "depth" flow... is that how it was used?
There's no such flow just for Flux Redux?

Redux can be applied to every output. That’s why there is no individual output for it.
When you use Redux make sure you increase the „guidance” to 10+


Digit recognition(0-9) using Logistic-Regression

Hello again,
This is a multiclass classification problem

dataset Logistic-Regression: sklearn.dataset
There will be preprocessing and visualization of the data, and then tuning the model for the right accuracy
It will be achieved by changing various hyperparameters for the models,
at last accuracy with various metric like, MSE, R2, Confusion matrix,
@Jayshah6699 Please add the label and assign.

@Jayshah6699 is this issue open??

@Jayshah6699 is this issue open??

@Amit366 yes, this issue is open, I'm busy right now with the college examination, if you want to work upon this issue you are welcome.
Please keep a note of the followings:

Use handwritten digits, I was using from Kaggle check here, you are free to work on any dataset if you find
Use Logistic Classification Model for classifying,
You need to convert the images into the matrix to make DataFrame

Please confirm if you want to work on this.
@Jayshah6699 if @Amit366 wants, then assigned to him

@Amit366 yes, this issue is open, I'm busy right now with the college examination, if you want to work upon this issue you are welcome.
Please keep a note of the followings:

Use handwritten digits, I was using from Kaggle check here, you are free to work on any dataset if you find
Use Logistic Classification Model for classifying,
You need to convert the images into the matrix to make DataFrame

Please confirm if you want to work on this.
@Jayshah6699 if @Amit366 wants, then assigned to him

Yes sir I want to work on this.

Yes sir I want to work on this.

@Jayshah6699 I am getting this error when using git add .

@Jayshah6699 I am getting this error when using git add .

change a file name a little bit

change a file name a little bit


Add to entitlement managment update a connected organization

Add to the article in Azure AD entitlement management on connected organizations a section on how to update a connected organization.  Currently the article only covers how to create and delete them.

@markwahl-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Fix issue url

Description

Additional context

Related items

Testing

By submitting this pull request, I confirm that my contribution is made under the terms of BSD 3-Clause License and I agree to the terms of the LICENSE.

test


fix(remix-express): respect expressjs hostname property when behind reverse proxies

@remix-run/express creates request.url directly from the Host HTTP header, but when behind a reverse proxy with app.enable('trust proxy') set, the actual hostname is provided via Express's req.hostname property, which is a "proxy aware" property that parses either Host or X-Forwarded-Host headers depending on the app configuration.
This can cause unexpected behavior when a user requests (for example) https://remix-app.org.com/page, but some condition causes a redirect to be thrown that uses request.url for a returnTo parameter (for instance: auth failure due to a protected resource).
If the app is behind a reverse proxy, and utilizing an internal DNS name (like an AWS internal host name), then that host will be used to create request.url, and you'll get a response with a bad Location header, and will also be leaking internal hostnames externally.
For example:
// server.js
const app = express();
app.enable('trust proxy');
// ... etc ...

// ~/routes/page.tsx
export function loader({request}: LoaderArgs) {
  await authenticator.isAuthenticated(request, {
    failureRedirect: `/login?returnTo=${encodeURIComponent(request.url)}`
  });
}

Request:
GET /page HTTP/1.1
Host: remix-app.internal.us-east.aws.io
X-Forwarded-Host: remix-app.org.com

Response:
HTTP/1.1 302 Found
Location /login?returnTo=http%3A%2F%2Fremix-app.internal.us-east.aws.io%2F <- INCORRECT

Closes: #

[ ] Docs
[ ] Tests

Testing Strategy:

Made the change to<EMAIL_ADDRESS>and verified that request.url reflected the X-Forwarded-Host header when app.enable('trust proxy') was set in my Express app.

Oops, sorry this must have fallen off my radar.  Do you want to add a changeset and we can get this merged?


Rigbits: Mirror shape with custom names issue

reported in the forum http://forum.mgear-framework.com/t/bug-naming-rules/3389

Hi Miquel, I fixed this issue this weekend. Could you check this out? Thanks!


Some anime not loading, others stuck in infinite Loading, JSON Parsing Error (Desu-online extension)

Source information
desu-online 13.1 (Polish)
Source language
Polish
Steps to reproduce

Open the anime viewing application.
Select the "DesuOnline" source.
Try to watch different anime to observe the different behaviors.

Expected behavior
I should be able to successfully view and play all anime series.
Actual behavior

Some anime load and play correctly. (e.g. Bleach)
Some anime are stuck in an infinite loading state and never play. (e.g. Monster)
For some anime, I receive the following error: (e.g. Sono Bisque Doll wa Koi wo Suru)

Aniyomi version
Preview r6151
Android version
Android 13
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Resolved by #2039


github actions occasionally fail due to stalls/timout

On occasion github actions builds fail while waiting for a failed arm-sdk download, but does not retry, nor timeout in a timely manner.
I've not done this type of coding before, but after brief searching, i used a pre-made package @vercel/fetch-retry (https://github.com/vercel/fetch-retry). My initial idea was to add a retry and timeout, but found the pre-existing package, so i attempted to implement it.
What do you think of this: https://github.com/fiam/arm-none-eabi-gcc/compare/master...emuflight:retry_timeout ?
It did not fail, but i'm yet to discover if it provides the expected retries/timeout. i'm certain you would have better insight.
example failure 1+ hours:

my branch (unknown if solves issue):

i had not noticed some new retry logic here: https://github.com/fiam/arm-none-eabi-gcc/commit/abeabe73714f61c325d323a4247787efcdafe906
my retry_timeout may be defunct/obsolete.


Make libopae-c-ase.so the duplicate of libopae-c.so in order to make …

…the ASE plugin changes backward compatible for ASE applications

This fixes the original problem: links now work.
However, "with_ase" no longer works. I wonder whether the ordering problem someone suggested a while ago is happening now.
We don't really have to do a preload, now that ASE is actually loaded as a plugin. I looked through the plugin manager and didn't see a way to do it, but what if an environment variable could trigger loading a specific configuration file? If it would, "with_ase" could just set the right environment variable and then invoke the normal binary.

@lzhan55 - Let me know when you think it is ready to try again.

@michael-adler you can try again. thanks.

It works!
Can't init_ase.c be made much smaller now instead of duplicating the search code? It could either set the WITH_ASE environment variable in a constructor (not sure if that would work) or you could add some global boolean variable with the same side effect as WITH_ASE that init_ase.c could set and the code in init.c could read.

@michael-adler Does DCP regression working? Thanks. For now I would like to keep the search code duplicated. When WITH_ASE is set in the script, the application will search for opae_ase.cfg even if it is linked with libopae-c.so library. If libopae-c-ase.so is linked directly, it should search only for opae_ase.cfg. Either way ASE will be executed correctly.

@deepakcu84 said he would test when you are ready

@tswhison @nakulkorde Can you review the code change so that I can merge the changes into feature/ase_plugin3 so that Deepak can run regression on the feature branch some day? Thanks.

I like Michael's idea to remove the duplicated code.  If this goes into the feature branch as is, can we revisit that before it goes into master?

@tswhison Do you mean in init_ase.c, we only set the environment variables WITH_ASE, and call the fpgaInitialize with opae_ase.cfg file from libopae-c.so? I can try now.

You might experiment with adding both init.c and init_ase.c to libopae-c-ase.  The constructor in init_ase.c will fire first, because it has been prioritized.  I think it could then just set WITH_ASE as you suggest.

@tswhison There are two cases in ASE:

ASE application is linked with libopae-c-ase.so directly. So we duplicated the libopae-c.so to make it work
ASE application is invoked with with_ase.sh as "with_ase ./hello_fpga". In this case, the application will be linked directly to libopae-c.so.

In order to satisfy both use cases, I think we should keep the code as it is with some duplicating code of search for opae_ase.cfg. I don't see a reason to try something like making libopae-c-ase.so a proxy library. Please correct me if my understanding is wrong.

You already have the new init.c that responds to WITH_ASE linked into libopae-c-ase. The build in its current configuration guarantees that. The only thing now required in init_ase.c is setenv("WITH_ASE") in a constructor so that it runs before the init.c code.

@michael-adler your assumption is that the application is linked with both libopae-c-ase.so and libopae-c.so, right? But WITH_ASE case, ASE is linked to libopae-c.so, not libopae-c-ase.so.

All of libopae-c's plugin manager code is now inside libopae-c-ase. You did that yesterday.

@michael-adler I will sync up with you in the afternoon.

@tswhison is on the same track. You could just talk to him.

@michael-adler It is OK as long as it passed the checks.


feat: websockets

[ ] Websockets for
[ ] Seeding Users

Suggestion before merging Squash and Merge so this wont pile up the next PR.

Closing cause wrong branch

Suggestion before merging Squash and Merge so this wont pile up the next PR.

noted po, will comply for next PR, thank you.


Unable to upload

CLI, what a mess worst error reporting ever.  Trying to upload with command I get this lovely error.  File is there as when I use it in Open API it is able to upload
`App
Center
CLI
would
like
to
collect
data
about
how
users
use
CLI
commands
and
some
problems
they
encounter.
Participation
is
voluntary
and
when
you
choose
to
participate
your
device
automatically
sends
information
to
Microsoft
about
how
you
use
App
Center
CLI.
For
more
information,
please
see
our
privacy
policy
at
https://aka.ms/mobilecenterprivacy
? Enable telemetry?  (Y/n) ? Enable telemetry?  (Y/n) ? Enable telemetry?  (Y/n) | Loading prerequisites.../ Loading prerequisites...- Loading prerequisites...Error: failed to create release upload for /home/circleci/repo/Sprint.apk. Not found. Correlation ID: e982f78a-c3ee-4170-816d-a5f2c3a8231f
Exited with code exit status 3`
I have so many questions.  Using -app it requires me to add the userid from appcenter otherwise it fails.  Our app name has - dashes in it and this might be messing up.  on group I can have more than one group to set.  Why do I need userid in -app when I am using the appcenter token?  Supposedly if I have APPCENTER_API_TOKEN variable set on Circleci I don't need it in the command, well you do otherwise it errors.  We have a legacy group from Hockey called "Tag: our-qa", well guess what setting this in --group, it fails.  There is soooo much wrong with CLI, why are you wanting us to use this garbage when open API was sooooooo much better?
appcenter distribute release \
    --app "my_appcenter_id/${APP_NAME}" \
    --group Collaborators \
    --file "${FULL_APP_PATH}" \
    --token "${APPCENTER_API_TOKEN}" \
    --debug

Please fix this garbage

CLI is broken for our project it looks like to use the upload.  Not sure why but testing out in postman gives me a 404 error that it can't find the project even though I can upload it using the current API that you are killing.

CLI is broken for our project it looks like to use the upload.  Not sure why but testing out in postman gives me a 404 error that it can't find the project even though I can upload it using the current API that you are killing.

Hi, @grndvl1, sorry you are experiencing issues.
Regarding the telemetry question format: it's working fine for me, could you share some information about the environment (os, terminal, node, npm versions) where you're experiencing this issue?

Why do I need userid in-app when I am using the appcenter token?

This is likely because users can share access to their applications with other users.  In this case, appcenter apps list will show something like
yourUserId/AppName
otherUserId/AppName

Our app name has - dashes in it and this might be messing up.

Could you please describe the dash problem in more detail?

We have a legacy group from Hockey called "Tag: our-qa", well guess what setting this in --group, it fails.

Have you tried escaping space char (\  on macOS)?

Not sure why but testing out in postman gives me a 404 error that it can't find the project even though I can upload it using the current API that you are killing.

Can you please file a support ticket with us so we can look into your specific issues and logs? https://docs.microsoft.com/en-us/appcenter/general/support-center#contact-us

Hi, @grndvl1, sorry you are experiencing issues.
Regarding the telemetry question format: it's working fine for me, could you share some information about the environment (os, terminal, node, npm versions) where you're experiencing this issue?

Why do I need userid in-app when I am using the appcenter token?

This is likely because users can share access to their applications with other users.  In this case, appcenter apps list will show something like
yourUserId/AppName
otherUserId/AppName

Our app name has - dashes in it and this might be messing up.

Could you please describe the dash problem in more detail?

We have a legacy group from Hockey called "Tag: our-qa", well guess what setting this in --group, it fails.

Have you tried escaping space char (\  on macOS)?

Not sure why but testing out in postman gives me a 404 error that it can't find the project even though I can upload it using the current API that you are killing.

Can you please file a support ticket with us so we can look into your specific issues and logs? https://docs.microsoft.com/en-us/appcenter/general/support-center#contact-us

CLI is broken for our project, it looks like, to use the upload. Not sure why but testing out in postman gives me a 404 error that it can't find the project even though I can upload it using the current API that you are killing.

JFYI we have an updated instruction on the new API: https://docs.microsoft.com/en-us/appcenter/distribution/uploading#upload-new-release

CLI is broken for our project, it looks like, to use the upload. Not sure why but testing out in postman gives me a 404 error that it can't find the project even though I can upload it using the current API that you are killing.

JFYI we have an updated instruction on the new API: https://docs.microsoft.com/en-us/appcenter/distribution/uploading#upload-new-release

@russelarms I ran into this and the problem was that my app was part of a company.  And then I had 2 groups to distribute to.
LATEST_VERSION="$(appcenter distribute releases list --app $OrganizationName/$APP_NAME --token ${APPCENTER_API_TOKEN} | grep ID | head -1 | tr -s ' ' | cut -f2 -d ' ')"
echo "LATEST_VERSION2=$LATEST_VERSION"
appcenter distribute releases add-destination -a $OrganizationName/$APP_NAME -d "OurTesters" -t group -r $LATEST_VERSION --token ${APPCENTER_API_TOKEN} --debug --disable-telemetry

Hello, @grndvl1 !
I am trying to reproduce your issue but for me all work correctly. Could you please clarify some details:

Which version of App Center CLI are you use?
You can't upload release only to the Collaborators group or to any group? Group is public or private?
Did you try to perform command from your description without \ symbols?
It doesn't work only for a specific app or for all apps?
Could you please share logs (please make sure before sending logs that they don't contain personal information)? If you can't share logs here you can open a support ticket https://docs.microsoft.com/en-us/appcenter/general/support-center#contact-us.

Seems it was a hiccup in your system or CircleCI.  I think it was at a time when appcenter was having issues.  No longer an issue.
From: Anastasia Senyushina @.>
Sent: Tuesday, March 23, 2021 3:25 AM
To: microsoft/appcenter-cli @.>
Cc: John Matthews @.>; Mention @.>
Subject: Re: [microsoft/appcenter-cli] Unable to upload (#1217)
Hello, @grndvl1https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgrndvl1&data=04|01|jmatthews%40rightpoint.com|b88f62c5060b4a535fc408d8edd518b7|5fbbce2ac3e64b5ea51f222674fdb44d|1|0|637520846809192741|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Ex9Rus4A62fJFQu9DrSAsrOdzoSDh3E115j5Ho6CA1Y%3D&reserved=0 !
I am trying to reproduce your issue but for me all work correctly. Could you please clarify some details:

Which version of App Center CLI are you use?
You can't upload release only to the Collaborators group or to any group? Group is public or private?
Did you try to perform command from your description without \ symbols?
It doesn't work only for a specific app or for all apps?
Could you please share logs (please make sure before sending logs that they don't contain personal information)? If you can't share logs here you can open a support ticket https://docs.microsoft.com/en-us/appcenter/general/support-center#contact-us.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fappcenter-cli%2Fissues%2F1217%23issuecomment-804713017&data=04|01|jmatthews%40rightpoint.com|b88f62c5060b4a535fc408d8edd518b7|5fbbce2ac3e64b5ea51f222674fdb44d|1|0|637520846809197737|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=q1uZ74U7kTp62XqFT2bi7VnupDK6xLk1BV2QSnXOMGI%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAFUUFTA7XO4HMFL3AM4YDLTFBF4JANCNFSM4WEUKH3A&data=04|01|jmatthews%40rightpoint.com|b88f62c5060b4a535fc408d8edd518b7|5fbbce2ac3e64b5ea51f222674fdb44d|1|0|637520846809202717|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Dw%2F0jg2foImZ8PgRcpa%2Fvs87qhX7A391dU24PK8NQ5c%3D&reserved=0.

Glad to hear that your problem was resolved. I'm closing this issue but feel free to reopen this issue if you have additional questions.


Add draft placeholder for blog post on CNB and Paketo

Signed-off-by: David Espejo<EMAIL_ADDRESS>
Summary
As discussed during 11/1 WG Meeting, this is a place to collaboratively draft a blog post answering one of the FAQs:

What's the difference between Cloud Native Buildpacks and Paketo Buildpacks?

Use Cases

Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[ ] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Closing because we've updated the OSS workstream


0 score not being passed back

Hi,
When a student receive a 0 and I pass the score back to Canvas, nothing actually gets passed back, making it look like it is something that needs to be graded (Bug report posted here:  ).
As a possible fix, in https://github.com/IMSGlobal/lti-1-3-php-library/blob/master/src/lti/LTI_Grade.php:
$score_given = 0 + $this->score_given;
$grade_array = array_filter([
"scoreGiven" => $score_given,
"scoreMaximum" => 0 + $this->score_maximum,
"comment" => $this->comment,
"activityProgress" => $this->activity_progress,
"gradingProgress" => $this->grading_progress,
"timestamp" => $this->timestamp,
"userId" => $this->user_id,
"submissionReview" => $this->submission_review,
]);
if (!$score_given) {
$grade_array['scoreGiven'] = 0;
}

Use https://github.com/packbackbooks/lti-1-3-php-library instead, it fixes this and multiple other issues and this repo here is dead.

Thanks!


resholve: don't propagate, 0.4.0 -> 0.4.1

Motivation for this change
While using resholvePackage on a project with a python3.8 component, I noticed resholve's propagated dependencies were leaking into that build environment. There's a little more information about the error and the solution used here (😍 @jtojnar) in abathur/resholve#20.
Things done

[x] Tested in resholve's CI @ https://github.com/abathur/resholve/actions/runs/479039322
[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

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

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

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

1 package built:

resholve

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

1 package built:

resholve

/marvin opt-in
/status needs_reviewer

/marvin opt-in
/status needs_reviewer


4.0 alpha.3

Remove npm dependency on npm
global.rootPathJoin, global.__rootPath
Translations bugfix
Fully changeable modules directory
Reduced dependency on NPM
Hubot integration users

Approved


Error when building

Hi,
I build this project with Visual Studio Community 2019 but got these error:
...\B4DB4B3-RAT-main\B4B3-RAT\Builder\common.h(34,10): fatal error C1083: Cannot open include file: 'modes.h': No such file or directory
...\B4DB4B3-RAT-main\B4B3-RAT\Builder\common.h(34,10): fatal error C1083: Cannot open include file: 'modes.h': No such file or directory
And I use your buider but the rat cannot add to startup. Maybe you could help.
Thank you,

You need to build CryptoPP library


Playlist Manager

Why not remake the playlist manager? I think the function of the current playlist manager is too simple, if you can be modified like Br3tt's JS Smooth Playlist Manager. It will be more convenient to use! This is just a suggestion! tks!

Hi,
can you tell me what's the difference between the playlist manager in this theme and Br3tt's JS Smooth Playlist Manager?
All I can remember is the auto-playlist/presets...
-TT

basic list management functions, rename lists, sort lists and  auto-playlist/presets

You can all do that in the playlist manager, if you click on it or just hit CTRL+M.
Maybe in the future I'll add the auto-playlist feature, for now I need to do other important things...
-TT

I have added the ability to create your own custom auto playlists and some presets.
It is available in the latest commit, download and overwrite your existing files.
You can find and access the auto playlist feature in three different ways:

In the top of the playlist ( mouse hover ) header, then click on the font > Create new auto playlist
Right click for context menu in the playlist > Create new auto playlist.
In the top menu "Playlists" > Create new auto playlist

Now the playlist manager has the same functionality as Br3tt's JS Smooth Playlist Manager.
-TT


[PackageGraph] Bail out as early as possible when there are errors

rdar://problem/49527170

@swift-ci smoke test


Multiple bugs related to Special Characters

I'm sorry but this report is going to have few different bugs cause I see them all kinda relating to each other.
1.) Setting special char to "NA" literally sets it to "NA". I believe it's supposed to "unset" it, right?
So I've set "Character for Scheduled" to be "NA"
.logseq/settings/logseq-datenlp-plugin.json
{
  "semiAuto": true,
  "insertDateProperty": false,
  "lang": "en",
  "removeTime": true,
  "specialCharHeading": "",
  "dateChar": "@",
  "scheduledChar": "NA",
  "deadlineChar": "^",
  "disabled": false,
  "auto": false,
  "notifications": false,
  "defaultSnooze": 5,
  "useBlockAsAlarmTitle": true,
  "preferredDateFormat": "yyyy-MM-dd"
}

It becomes apparent especially in this example case:
- ## This is some block
  date:: [[2023-10-31]]
  tags:: [[AWS/NAT Gateway]]

Exiting that block will change it to:
- ## This is some block
  date:: [[2023-10-31]]
  tags:: [[AWS/NAT Gateway]]
  SCHEDULED: <2023-10-31 Tue>

2.) Scheduled blocks are rescheduled many times
At least now with these settings revisiting this block makes it to be:
- ## This is some block
  date:: [[2023-10-31]]
  tags:: [[AWS/NAT Gateway]]
  SCHEDULED: <2023-10-31 Tue>
  SCHEDULED: <2023-10-31 Tue>

And revisiting it again changes it to:
- ## This is some block
  date:: [[2023-10-31]]
  tags:: [[AWS/NAT Gateway]]
  SCHEDULED: <2023-10-31 Tue>
  SCHEDULED: <2023-10-31 Tue>
  SCHEDULED: <2023-10-31 Tue>
  SCHEDULED: <2023-10-31 Tue>

... and so on always doubling the amount of SCHEDULED entries.
3.) Disabling both Scheduled and Deadline is "overlapping inline parsing characters".
So setting both to "NA" (which I still assume should be disabled) makes the plugin error with the following message:

There are overlapping characters for the inline parsing character. Please re-check your settings

Then the plugin sets the settings back to default:
.logseq/settings/logseq-datenlp-plugin.json
{
  "semiAuto": true,
  "insertDateProperty": false,
  "lang": "en",
  "removeTime": true,
  "specialCharHeading": "",
  "dateChar": "@",
  "scheduledChar": "%",
  "deadlineChar": "^",
  "disabled": false,
  "auto": false,
  "notifications": false,
  "defaultSnooze": 5,
  "useBlockAsAlarmTitle": true,
  "preferredDateFormat": "yyyy-MM-dd"
}

I don't use DEADLINE and SCHEDULED at all and would like to disable parsing of those. Especially when the % character makes editing templates a bit hard since I have for example <% today %> there sometimes :)

I don't use DEADLINE and SCHEDULED at all and would like to disable parsing of those. Especially when the % character makes editing templates a bit hard since I have for example <% today %> there sometimes :)
Edit:
As a workaround for now I set up the characters for scheduled and deadline to be something I have no reason ever to type :D This is something that can't be done from the UI though but can be changed from the settins file.
.logseq/settings/logseq-datenlp-plugin.json
{
  "semiAuto": true,
  "insertDateProperty": false,
  "lang": "en",
  "removeTime": true,
  "specialCharHeading": "",
  "dateChar": "@",
  "scheduledChar": "ə",
  "deadlineChar": "ð",
  "disabled": false,
  "auto": false,
  "notifications": false,
  "defaultSnooze": 5,
  "useBlockAsAlarmTitle": true,
  "preferredDateFormat": "yyyy-MM-dd"
}

Thanks for the info. Also, it's possible to use two characters to better avoid false matches.
   "dateChar": "@",
   "scheduledChar": ":^",
   "deadlineChar": ":%",

Hi, will escape the character in code ?

Hi, will escape the character in code ? I wish to! But I found it didn't.

@hkgnp

Hi, will escape the character in code ? I wish to! But I found it didn't.

@hkgnp

Can you try the latest release?

The latest seemed to work! Thanks for your great work!


Hidden Comments

Hidden Comments
Hi often when i am making a presentation I like to have extra notes for myself which I don't want displayed in the notes at all.
I know that comments in markdown is a problem but perhaps something could be implemented in remark?
Am thinking an extra division within ??? that prevents text being displayed marking a possible slide be;

Slide
???
Note about the slide for people to read
Some extra separator like but not ---   or ??? here
Super top secret notes not displayed

@scotthewitt, if you are talking about in-slide notes, I have a simple solution: use the content class feature of remark:
<style type="text/css">
.invisible-ink {
    display: none;
}

.remark-container.remark-presenter-mode .invisible-ink {
    display: inline;
    color: red;
}
<style>

<!-- your code here -->

<textarea id="source">

# My Slide

Here's some text. .invisible-ink[Here's some text that only shows up in presenter mode.]

</textarea>

Obvious idea awesome thanks

This worked well for me. Maybe it should be in the FAQ?

Documented here, thanks!
https://github.com/gnab/remark/wiki/HTML-CSS-JS-samples


after running pip install Flask_WTF package reported successful python couldn't find package.

After running install package wasn't found in python3.
Just out of curiosity, I installed WTForms-SQLAlchemy and now flask_wtf and wtforms packages were found. Something wrong with the setup.py on Flask_WTF?

I can't reproduce this issue. Sounds like an issue with something else on your machine.
~
❯ pew mktmpenv
created virtual environment CPython3.9.6.final.0-64 in 114ms
  creator CPython3Posix(dest=/home/david/.local/share/virtualenvs/7741073bcf125ed, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=/home/david/.local/share/virtualenv)
    added seed packages: pip==21.1.3, setuptools==57.1.0, wheel==0.36.2
  activators BashActivator,CShellActivator,FishActivator,PowerShellActivator,PythonActivator,XonshActivator
This is a temporary environment. It will be deleted when you exit
Launching subshell in virtual environment. Type 'exit' or 'Ctrl+D' to return.

~ via 🐍 v3.9.6 (7741073bcf125ed)
❯ pip install flask-wtf
Collecting flask-wtf
  Downloading Flask_WTF-0.15.1-py2.py3-none-any.whl (13 kB)
Collecting itsdangerous
  Using cached itsdangerous-2.0.1-py3-none-any.whl (18 kB)
Collecting Flask
  Using cached Flask-2.0.1-py3-none-any.whl (94 kB)
Collecting WTForms
  Using cached WTForms-2.3.3-py2.py3-none-any.whl (169 kB)
Collecting click>=7.1.2
  Using cached click-8.0.1-py3-none-any.whl (97 kB)
Collecting Jinja2>=3.0
  Using cached Jinja2-3.0.1-py3-none-any.whl (133 kB)
Collecting Werkzeug>=2.0
  Using cached Werkzeug-2.0.1-py3-none-any.whl (288 kB)
Collecting MarkupSafe>=2.0
  Using cached MarkupSafe-2.0.1-cp39-cp39-manylinux2010_x86_64.whl (30 kB)
Installing collected packages: MarkupSafe, Werkzeug, Jinja2, itsdangerous, click, WTForms, Flask, flask-wtf
Successfully installed Flask-2.0.1 Jinja2-3.0.1 MarkupSafe-2.0.1 WTForms-2.3.3 Werkzeug-2.0.1 click-8.0.1 flask-wtf-0.15.1 itsdangerous-2.0.1

~ via 🐍 v3.9.6 (7741073bcf125ed)
❯ python
Python 3.9.6 (default, Jun 30 2021, 10:22:16)
[GCC 11.1.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import flask_wtf
>>> flask_wtf
<module 'flask_wtf' from '/home/david/.local/share/virtualenvs/7741073bcf125ed/lib/python3.9/site-packages/flask_wtf/__init__.py'>
>>>


FormatException when using AzureSearch 5.0.3 on MacOs Sierra

On MacOS Sierra, the following exception is thrown while initializing Application Insights, the cause seems to be the inclusion of AzureSearch 5.0.3

I also face the same error below is my code to initialize SearchServiceClient
var cred = new SearchCredentials(apiKey);
_client = new SearchServiceClient(serviceName, cred);

@shahabhijeet Per the stack trace above, this looks like an issue in ClientRuntime, not Microsoft.Azure.Search. It looks like a bug in how ServiceClient is handling the user agent string. Can you please take a look?

@imjustjoekingex @knnithyanand Are you able to repro this with Azure .NET libraries other than Microsoft.Azure.Search?

not so far, its only this specific version too
On Wed, Feb 13, 2019 at 2:03 PM Bruce Johnston<EMAIL_ADDRESS>wrote:

@imjustjoekingex https://github.com/imjustjoekingex @knnithyanand
https://github.com/knnithyanand Are you able to repro this with Azure
.NET libraries other than Microsoft.Azure.Search?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-net/issues/5184#issuecomment-463324533,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APSFCdPVBOW2R-4i30XrZxAiOAr7_7nNks5vNGGTgaJpZM4aXPC2
.

@imjustjoekingex can you provide the Microsoft.Rest.ClientRuntime version that you are using.
And if you can upgrade to the latest version and then let me know if you still encounter the issue.

@imjustjoekingex @knnithyanand A new Azure Search preview SDK is now available on NuGet that uses the latest version of the ClientRuntime library. A stable release that includes this change will be available within a week.
I'm going to close this issue for now. If you get a chance to use this new version, please re-open this issue if it doesn't solve the problem.


[FEAT] Update Center - Configuring Maintenance Configurations dynamic assignments

Community Note

Please vote on this issue by adding a :thumbsup: reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
When using the Update Center module you can't configure the maintenance configurations dynamic scope.
You can make assigments for the created maintenance configuration from the virtual machine module, but I think that it could be useful to set a dynamic assignment scope for the maintenance configuration.
I could submit a PR to implement this.
New or Affected Resource(s)/Data Source(s)
azurerm_maintenance_assignment_dynamic_scope
Potential Terraform Configuration
New variable
variable "mc_dynamic_scope_enabled" {
  description = "Enable Maintenance configuration dynamic scope for current subscription."
  type        = bool
}

New resource
resource "azurerm_maintenance_assignment_dynamic_scope" "mc_dynamic_scope" {
  for_each = var.mc_dynamic_scope_enabled ? { for config in var.maintenance_configurations : config.configuration_name => config } : {}
  name                         = "mcdc-${each.key}"
  maintenance_configuration_id = azurerm_maintenance_configuration.maintenance_configurations[each.key].id

  filter {
    locations       = var.location
    os_types        = ["Linux", "Windows"]
  }
}

References
No response

Hi @aolmosj ,
Nice suggestion, would be great if you can open a PR!


Update CLI to reflect Node version requirement

Hi @MatthewMcGlynn, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Matthew McGlynn). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Added status code check to getting SAT

fixes https://github.com/Comcast/codex-svalinn/issues/59

Codecov Report

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

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   41.39%   41.39%
=======================================
  Files           6        6
  Lines         372      372
=======================================
  Hits          154      154
  Misses        210      210
  Partials        8        8

Continue to review full report at Codecov.

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


Polish models

Closes #179.
EDIT: I outsourced most of the non-model changes to separate PRs.

All changes here have been merged already.


prettier-plugin-go-template: fix package declaration

Description of changes
I introduced dontNpmBuild = true; by mistake. Package wasn't working.
#328654
#325309

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4314

Please squash your commits into prettier-plugin-go-template: fix package declaration

Done.


Filtering

Thanks for the awesome plugin. I have a question about filtering.
Current filter matches All properties defined in filterProperties. Any idea how to set up for any property match?

Found my answer in Issue:12  Or works by default . filter Properties is for And. Thanks again!

cool no prob!  If you need OR functionality, you can use the expression function option.  See docs/tests for more info!  Thanks again!

oops just reread and you were looking for ALL which as you pointed out is the default (no filter properties) Thanks again!


test-pr haha just practice pull request ignore it

ignore it

do NOT use public repositories to learn how to use github


Is it possible to insert items at the beginning of the data set?

I'm trying to use this list in a chat scenario.
When I append new messages at the end of the list, things are ok: my display position stays the same, and all I have to do is scroll to the new one(s) I just added.
However, when I start by loading only the latest data initially, and want to load older messages as the user scrolls back up, I cannot figure out how to manage the scroll.
I would need to update the display position exactly at the same time I update my data array content, but the scrollTo and scrollAround method do not proide this.
How to deal with this?
Thanks!

Same as issue #50, sorry, closing.


Implement PMMLModel and PMMLModelInstance methods

@avan1235 let's sync on how to cope with the updates.


Data Filters

Rewrite filters from naive lambda/function style (still available in the dictset module) to a DNF style filter where fields being filtered on are explicit.
This is a precondition for filtering on Indexes.

GA in 0.4.40


Add sendMediaGroup method and InputMedia* types.

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

Project coverage is 100.00%. Comparing base (259cfe8) to head (63cf56a).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##              master       #31    +/-   ##
============================================
  Coverage     100.00%   100.00%
- Complexity       916       959    +43
============================================
  Files            208       215     +7
  Lines           3412      3600   +188
============================================
+ Hits            3412      3600   +188

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


Is it okay if feature, view, and other tests are too long?

Rubocop complains if you tests are more than five lines long. It seems find to me for feature tests to be long, given the setup time that it can take for each test. View tests, perhaps as well could benefit from this exemption as well?

Yes.


Update display to v0.0.33

Allow disabling rm2fb-client with the RM2FB_DISABLE environment variable

This appears to work on a rM2


Some fonts aren't loading on the docs

Hi, when using the docs, these fonts below don't seem to load, resulting in a bunch of ▢ characters, slightly confusing when they open a new window! 😄
http://ramdajs.com/fonts/glyphicons-halflings-regular.woff2 Failed to load resource: the server responded with a status of 404 (Not Found)
http://ramdajs.com/fonts/glyphicons-halflings-regular.woff Failed to load resource: the server responded with a status of 404 (Not Found)
http://ramdajs.com/fonts/glyphicons-halflings-regular.ttf Failed to load resource: the server responded with a status of 404 (Not Found)

yeah been aware of this one for a while, but haven't had bandwidth to fix it yet. PR would be most welcome

should have mentioned, this issue should be on ramda.github.io repo.

Ah sorry! Will move over there 😄

thanks


Inference time on CPU/GPU

It would be nice if you can add the latency results to the README as well. I am planning to use this for an industry application, but before experimenting, it would be nice to know if it's even a feasible option (since I have an SLA of like 1 sec per image).

Hi, @ajkdrag
Thx for your interest.
You can check the Table VIII for the inference time of CLIP4STR on an NVIDIA A100 40 GB GPU in the paper https://arxiv.org/pdf/2305.14014.pdf.
However, I am not sure what is the batch size the authors used for testing.
It is pretty straight-forward to figure out whether you can use CLIP4STR-B in your case. It is a standard 12-layer vision transformer + 2 extra transformer decoder layers.
There are many benchmarks about the inference speed of transformer on different devices like https://lambdalabs.com/gpu-benchmarks.
Hope this is helpful for you.

Close due to inactivity. Feel free to re-open.


Input Validation run separately for each dependent (depends not cached when input validation fails resulting in duplicated errors)

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Example
Here's a self-contained, minimal, reproducible, example with my use case:

import logging
from pydantic import BaseModel
from fastapi import FastAPI, Depends, Request

logger = logging.Logger(__name__)

class UserModel(BaseModel):
    username: str
    name: str = None

app = FastAPI()

@Depends
def get_user_model_data(model: UserModel):
    print("Running function get_user_model_data")
    return model.dict(exclude_unset=True)

@Depends
def log_new_users(request: Request, data: dict = get_user_model_data):
    logger.info(f'Got request from {request.client.host} with username {model.username}')

@app.post('/users', dependencies=[log_new_users])
def create_account(data: dict = get_user_model_data):
    return {
        "status": "OK",
        "data": data,
    }

Description
Above is a simple application with a /users endpoint using a pydantic model which requires a username in the post body.
when the model validation passes all is fine. But when model validation fails, the model is validated for each depends separately resulting in duplicated errors.
I would expect that it should be cached even when there are errors so it is validated only once.

Run the app
execute curl -X POST "http://<IP_ADDRESS>:8000/api/users" -H  "Content-Type: application/json" -d '{"username":null, "name":"string"}'
it will result in duplicate error as follows

{
  "detail": [
    {
      "loc": [
        "body",
        "username"
      ],
      "msg": "none is not an allowed value",
      "type": "type_error.none.not_allowed"
    },
    {
      "loc": [
        "body",
        "username"
      ],
      "msg": "none is not an allowed value",
      "type": "type_error.none.not_allowed"
    }
  ]
}

Environment

OS: macOS:

FastAPI Version 0.63.0:

Python version: 3.9.1

Maybe you tried this before, but if you chain the Depends calls, you only get one error message. I undesrtand what you've described is undesired behavior, but it's not entirely unexpected. You have defined that dependency get_user_model twice, so if it errors, when it solves those it will show both errors.
If you want something with only one error, why not chain the dependencies, like so?
import logging
from typing import Optional

from fastapi import Depends, FastAPI, Request
from pydantic import BaseModel

logger = logging.Logger(__name__)

class UserModel(BaseModel):
    username: str
    name: Optional[str] = None

app = FastAPI()

def get_user_model_data(model: UserModel):
    print("Running function get_user_model_data")
    return model.dict(exclude_unset=True)

def log_new_users(request: Request, data: dict = Depends(get_user_model_data)):
    logger.info(
        f"Got request from {request.client.host} with username {data.get('username')}"
    )
    return data

@app.post("/users")
def create_account(data: dict = Depends(log_new_users)):
    return {
        "status": "OK",
        "data": data,
    }

With that, when I hit the app with: curl -X POST "http://<IP_ADDRESS>:8000/users" -H "Content-Type: application/json" -d '{"username":null, "name":"string"}'
I get only one validation error:
{
    "detail": [
        {
            "loc": [
                "body",
                "username"
            ],
            "msg": "none is not an allowed value",
            "type": "type_error.none.not_allowed"
        }
    ]
}

@falkben yes, of course.
In my case, one depends does not know about the other as these functions are defined in different scopes.
I submitted a pull request to cache the results
#2779


Allow NetworkManager and wireless together

Motivation for this change
When NetworkManager is configured to not manage all interfaces, it's
perfectly fine to have the rest be managed by the standard nixos
wireless scripts.
I use
  networking.networkmanager.unmanaged = [
    "*" "except:type:wwan" "except:type:gsm"
  ];

to control everything using networking.wireless except for the mobile
LTE modem which only works with NetworkManager.
Things done

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

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

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

cc @jtojnar @mic92 @vcunat because you've commmitted to both the network manager service and the modem-manager package.

Rebased on top of master.
Any thoughts anyone? I've been using this successfully for months :)


Serve internal resources using FileResourceServer ensuring path is standardized.

Revert changes to serve internal resources using static file server.
Ensure serving path are standardized.
Add further tests to check serving boundaries.

Codecov Report

Merging #1278 into master will increase coverage by 0.82%.
The diff coverage is 83.96%.

@@            Coverage Diff             @@
##           master    #1278      +/-   ##
==========================================
+ Coverage   88.14%   88.97%   +0.82%
==========================================
  Files          38       39       +1
  Lines        2404     2412       +8
==========================================
+ Hits         2119     2146      +27
+ Misses        285      266      -19

Flag
Coverage Δ

#Kitura
88.97% <83.96%> (+0.82%)
:arrow_up:

Impacted Files
Coverage Δ

...ces/Kitura/staticFileServer/StaticFileServer.swift
94.73% <100%> (+9.55%)
:arrow_up:

Sources/Kitura/staticFileServer/FileServer.swift
91.25% <77.77%> (+8.07%)
:arrow_up:

Sources/Kitura/Router.swift
83.87% <82.35%> (+0.27%)
:arrow_up:

Sources/Kitura/FileResourceServer.swift
85.71% <85.71%> (ø)

Continue to review full report at Codecov.

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


NullReferenceException when trying to access UDP ports?

Running this sample code in my packet handler.
IpV4Datagram ip = packet.Ethernet.IpV4;
UdpDatagram udp = ip.Udp;
Console.WriteLine(udp.SourcePort);

Throws a NullReferenceException when trying to access the var udp

System.NullReferenceException: Object reference not set to an instance of an object

Is there a trigger / reason why this would be null? Or is this unexpected (a bug)? It throws this exception on all packets being captured, so I don't think its an issue with a certain packet.

The full exception text.

System.NullReferenceException: Object reference not set to an instance of an object.
at Coyote.Core.Network.NetworkHandler.OnPacket(Packet packet) in C:\Users\admin\Desktop\Coyote\Coyote.Core\Network\NetworkHandler.cs:line 90
at PcapDotNet.Core.PacketCommunicator.PacketHandler.Handle(Byte* user, pcap_pkthdr* packetHeader, Byte* packetData) in c:\github\pcap.net\pcapdotnet\src\pcapdotnet.core\packetcommunicator.cpp:line 343
at pcap_loop(pcap* , Int32 , IntPtr , Byte* )
at PcapDotNet.Core.PacketCommunicator.ReceivePackets(Int32 count, HandlePacket callback) in c:\github\pcap.net\pcapdotnet\src\pcapdotnet.core\packetcommunicator.cpp:line 181
at Coyote.Core.Network.NetworkHandler.FetchPackets() in C:\Users\admin\Desktop\Coyote\Coyote.Core\Network\NetworkHandler.cs:line 79
at Coyote.Core.Commands.Networking.SniffPacketsCommand.Process(String[] arguments, ICoreContext coreContext) in C:\Users\admin\Desktop\Coyote\Coyote.Core\Commands\Networking\SniffPacketsCommand.cs:line 48
at Coyote.Core.Commands.ConsoleCommandProvider.Process(String content, ICoreContext coreContext) in C:\Users\admin\Desktop\Coyote\Coyote.Core\Commands\ConsoleCommandProvider.cs:line 49

Please use the Pcap.Net Q&A Group to ask questions.


Handle backpressure for Get Operation via RxJava

Closes #559.
Finally, RxJava 1.1.0 contains required operator to keep consistency for Get Operation and handle backpressure at the same time!
@nikitin-da PTAL!

Good news)


gio: Don't imlement Send on GioFuture anymore

This is unnecessary now because GIO-style async functions don't require
Send for their callback anymore, and it also removes the two layers of
checks to enforce at runtime that everything is handled on the same
thread.

This should not be backported as it's breaking backwards compatibility.

Looks good other than the typo in the commit message

Typo fixed, thanks :)


Attempt to fix internal git files being detected by find

This find command seems to work from my bare-bones testing in a personal project. Let's see if it holds up in the context of this action
Closes #66
This PR adds a seperate condition to the find command that, to the best of my understanding, filters out the .git directory and it's contents to the left side of that -o (or) and adding the -print to the right side (the original query) makes it only print the results of that and not the .git directory matched by the left side.
(Not an expert with find - I just looked stuff up and read a bit in the documentation)

Running the test script fails. Strange.
Sending build context to Docker daemon  314.9kB
Step 1/3 : FROM debian:unstable-20211220-slim
unstable-20211220-slim: Pulling from library/debian
bb67f12293f4: Pull complete
Digest: sha256:24699aac07f641f2ec4b2ddd71a659a143997d1a62be7c5d46fa7eb6a1be42aa
Status: Downloaded newer image for debian:unstable-20211220-slim
 ---> 159f224fe340
Step 2/3 : COPY entrypoint.sh /entrypoint.sh
 ---> 9c04335de7fa
Step 3/3 : ENTRYPOINT ["/entrypoint.sh"]
 ---> Running in 5de144b5310e
Removing intermediate container 5de144b5310e
 ---> 435008fcfd96
Successfully built 435008fcfd96
Successfully tagged clang-format-action-test:latest
Installing clang-format-13
debconf: delaying package configuration, since apt-utils is not installed
/entrypoint.sh: line 64: -print: command not found
Installing clang-format-13
debconf: delaying package configuration, since apt-utils is not installed
/entrypoint.sh: line 64: -print: command not found
files that should fail have succeeded!

Oh, it needs to be in the brackets! Will fix that.

Seems to be behaving now when running the test script locally.
Sending build context to Docker daemon    322kB
Step 1/3 : FROM debian:unstable-20211220-slim
 ---> 159f224fe340
Step 2/3 : COPY entrypoint.sh /entrypoint.sh
 ---> da0d23cd155b
Step 3/3 : ENTRYPOINT ["/entrypoint.sh"]
 ---> Running in f5fc4996ab5d
Removing intermediate container f5fc4996ab5d
 ---> b245568d679f
Successfully built b245568d679f
Successfully tagged clang-format-action-test:latest
Installing clang-format-13
debconf: delaying package configuration, since apt-utils is not installed
Installing clang-format-13
debconf: delaying package configuration, since apt-utils is not installed
known_fail/addition.proto:5:16: error: code should be clang-formatted [-Wclang-format-violations]
message add_res { uint64 result = 1; }
               ^
known_fail/addition.proto:5:18: error: code should be clang-formatted [-Wclang-format-violations]
message add_res { uint64 result = 1; }
                 ^
known_fail/addition.proto:5:37: error: code should be clang-formatted [-Wclang-format-violations]
message add_res { uint64 result = 1; }
                                    ^
known_fail/addition.proto:6:16: error: code should be clang-formatted [-Wclang-format-violations]
message add_req {
               ^
known_fail/addition.proto:7:15: error: code should be clang-formatted [-Wclang-format-violations]
        uint64 a = 1; uint64 b = 2;
                     ^
known_fail/addition.proto:10:17: error: code should be clang-formatted [-Wclang-format-violations]
service addition {
                ^

Failed on file: known_fail/addition.proto
known_fail/addition.cpp:6:25: error: code should be clang-formatted [-Wclang-format-violations]
        int integer1, integer2,sum; /* declaration before any executable statements*/
                               ^

Failed on file: known_fail/addition.cpp
known_fail/addition.cxx:6:25: error: code should be clang-formatted [-Wclang-format-violations]
        int integer1, integer2,sum; /* declaration before any executable statements*/
                               ^
Failed on file: known_fail/addition.cxx

known_fail/addition.c++:6:25: error: code should be clang-formatted [-Wclang-format-violations]
        int integer1, integer2,sum; /* declaration before any executable statements*/
                               ^
Failed on file: known_fail/addition.c++

known_fail/addition.cc:6:25: error: code should be clang-formatted [-Wclang-format-violations]
        int integer1, integer2,sum; /* declaration before any executable statements*/
                               ^
Failed on file: known_fail/addition.cc

known_fail/addition.c:6:25: error: code should be clang-formatted [-Wclang-format-violations]
        int integer1, integer2,sum; /* declaration before any executable statements*/
                               ^
Failed on file: known_fail/addition.c

known_fail/addition.ino:6:25: error: code should be clang-formatted [-Wclang-format-violations]
        int integer1, integer2,sum; /* declaration before any executable statements*/
                               ^

Failed on file: known_fail/addition.ino
known_fail/addition.pde:6:25: error: code should be clang-formatted [-Wclang-format-violations]
        int integer1, integer2,sum; /* declaration before any executable statements*/
                               ^
Failed on file: known_fail/addition.pde


[REQ] target for outputting patched sources

What kind of request is this?
New feature
What is your request or suggestion?
similar to debug/sources, dalec should be able to output the source with the patches applies
Are you willing to submit PRs to contribute to this feature request?

[ ] Yes, I am willing to implement it.

duplicate of #359


Embed tutorial videos on static help page https://participedia.net/help-faq-contact

Publish + Edit: https://youtu.be/3HirXGRKTzI
Search + Download: https://youtu.be/h0lC7-kqEw8
Both videos directly above FAQ section (below contact info)

Hang on - I accidentally uploaded the videos to my personal youtube. i need to re-upload to participedia's youtube and i'll update these links!

ok the links are correct now.


Thing

No comment.

Thanks for the thing, could you please squash you commits?

Unfortunately, I had problems merging the commits, but this is the best I can do.

Thanks, fixed at [87c6ab6].(https://github.com/xat/castnow/commit/87c6ab6bbd12fb4e15441b490561871ea1fabf33)


SAA-964: Disallow duplicate pay band and iep level combinations for pay

Codecov Report
Patch coverage: 75.00% and project coverage change: -0.01 :warning:

Comparison is base (416d73b) 84.59% compared to head (fec92cf) 84.59%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #451      +/-   ##
============================================
- Coverage     84.59%   84.59%   -0.01%
- Complexity     2187     2192       +5
============================================
  Files           289      289
  Lines          8201     8205       +4
  Branches        493      495       +2
============================================
+ Hits           6938     6941       +3
  Misses         1007     1007
- Partials        256      257       +1

Impacted Files
Coverage Δ

...ps/hmppsactivitiesmanagementapi/entity/Activity.kt
93.50% <75.00%> (-0.50%)
:arrow_down:

... and 1 file with indirect coverage changes

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


⚠ Bofrost.be has degraded performance

In 01f3cae, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1452 ms

Resolved: Bofrost.be performance has improved in ed8d4bf.


fix: StoryResponsivePreview vertical resize

Description
fix the preview that could not being visually resized vertically.
it's a problem that could not appear because two rules have equals css specificity.
Additional context

What is the purpose of this pull request?

[X] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[ ] If it's a new feature, provide a convincing reason to add it. Ideally, you should open a suggestion issue first and have it approved before working on it.
[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the Commit Convention.
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

It doesn't depends of the story but how the css is construct and ordered
on https://controls.histoire.dev/story/histoire-controls-src-components-checkbox-hstcheckbox-story-vue?variantId=histoire-controls-src-components-checkbox-hstcheckbox-story-vue-0 if you try to resize vertically the story, the window of the story will keep the full height


Question: should we create a "sandbox-mode" activity ?

In #39 I said that it would be nice to have a "sandbox-mode" Activity where users (and ourselves obviously) could try things and see live results.
It could be hidden by default and enabled in the settings. Perhaps this sandbox activity could have a "save as card" feature ? (with better wording though)
What do you think about it ?

I never thought about "device card" in this way, this could be like a console for advanced user?
I like the idea to transform the req/res to a card!
Anyway I wouldn't add it as preference, but as an item in the navigation drawer?!

Yes, I was thinking of adding a "sandbox mode" item in the drawer, but having a preference toggling its visibility. Could be the same as "enable debug mode".

Is related to the fab issue, if we have a separated "sandbox activity", do we really still have the need of the icon DEBUG in ActionMenu and also the hiding toggle for device cards?
I was thinking about your "Sanbox" just as an item menu between "Add environment" and "Settings"


Support docstring templates

Provide auto fill of docstring.
The idea would be that if I type """ right at a docstring position (file, class method), it would generate a proper template for it.
A simple example would be to start typing """ right after def f(a): and it would immediatly expand it to:
def f(a):
"""
:param a
:return
"""

Just FYI for those who are interested, there's a 3rd party extension that provides that: https://marketplace.visualstudio.com/items?itemName=njpwerner.autodocstring


WPT tests for findElement

Prototype: https://github.com/GoogleChromeLabs/chromium-bidi/issues/67
Specification isn't needed anymore, c.f. https://github.com/w3c/webdriver-bidi/issues/150


Dist folder

The dist folder is getting a bit complicated to understand.
Is it crazy to place the plugins in a subfolder called plugins?

You are right. I will think how to structure it.
There is a problem with plugins in Meteor - we have to keep all them in the assets directory

Any news on this?