[question] Record a video error

In the example of recording a video, the action = [env.action_space.sample()],  but now I want to action=[model.predict(obs)] like the following code:
def make_video(env_id,video_folder,video_length,model):
    """

    :param env_id:
    :param video_folder:
    :param video_length:
    :param model:
    :return:
    """
    from stable_baselines.common.vec_env import VecVideoRecorder, DummyVecEnv
    env=DummyVecEnv([lambda:gym.make(env_id)])

    env=VecVideoRecorder(env,video_folder,record_video_trigger=lambda x:x==0,video_length=video_length,
                         name_prefix="random-agent-{}".format(env_id))
    obs = env.reset()
    for _ in range(video_length+1):
        action=[model.predict(obs)]
        obs,_,_,_=env.step(action)
    env.close()

And it will encounter the error, I guess the problem is at obs=env.reset() and action=[model.predict(obs)].
Traceback (most recent call last):
  File "C:\Users\born_\anaconda3\envs\workEnv\lib\site-packages\IPython\core\interactiveshell.py", line 3343, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-2-a075abc16237>", line 1, in <module>
    runfile('C:/Users/born_/PycharmProjects/test_sb/lunarlander.py', wdir='C:/Users/born_/PycharmProjects/test_sb')
  File "C:\Program Files\JetBrains\PyCharm 2020.2\plugins\python\helpers\pydev\_pydev_bundle\pydev_umd.py", line 197, in runfile
    pydev_imports.execfile(filename, global_vars, local_vars)  # execute the script
  File "C:\Program Files\JetBrains\PyCharm 2020.2\plugins\python\helpers\pydev\_pydev_imps\_pydev_execfile.py", line 18, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "C:/Users/born_/PycharmProjects/test_sb/lunarlander.py", line 114, in <module>
    make_video(env_id,video_folder,video_length,model)
  File "C:/Users/born_/PycharmProjects/test_sb/lunarlander.py", line 69, in make_video
    obs,_,_,_=env.step(action)
  File "C:\Users\born_\anaconda3\envs\workEnv\lib\site-packages\stable_baselines\common\vec_env\base_vec_env.py", line 150, in step
    return self.step_wait()
  File "C:\Users\born_\anaconda3\envs\workEnv\lib\site-packages\stable_baselines\common\vec_env\vec_video_recorder.py", line 87, in step_wait
    obs, rews, dones, infos = self.venv.step_wait()
  File "C:\Users\born_\anaconda3\envs\workEnv\lib\site-packages\stable_baselines\common\vec_env\dummy_vec_env.py", line 42, in step_wait
    self.envs[env_idx].step(self.actions[env_idx])
  File "C:\Users\born_\anaconda3\envs\workEnv\lib\site-packages\gym\wrappers\time_limit.py", line 16, in step
    observation, reward, done, info = self.env.step(action)
  File "C:\Users\born_\anaconda3\envs\workEnv\lib\site-packages\gym\envs\box2d\lunar_lander.py", line 250, in step
    assert self.action_space.contains(action), "%r (%s) invalid " % (action, type(action))
AssertionError: (array([0], dtype=int64), None) (<class 'tuple'>) invalid
Process finished with exit code 0

What's the problem and how to fix it?

You do not need to wrap model.predict(obs) into another list with [...]. See the example below that one.

You do not need to wrap model.predict(obs) into another list with [...]. See the example below that one.

Yeah，recording a gif is OK. And the recording a video case, do you mean to remove the []?
I have tried this, it will raise the error (just the last part) :
 File "C:\Users\born_\anaconda3\envs\workEnv\lib\site-packages\gym\envs\box2d\lunar_lander.py", line 250, in step
    assert self.action_space.contains(action), "%r (%s) invalid " % (action, type(action))
AssertionError: array([3], dtype=int64) (<class 'numpy.ndarray'>) invalid

Note what model.predict returns. It does not only return an action but also states. It needs to be action, states = model.predict(obs).

Oh, yeah, thank you!


clock: canonical test data has been improved

The JSON file containing canonical inputs/outputs for the Clock exercise has gotten new data.
There are two situations that the original data didn't account for:

Sometimes people perform computation/mutation in the display method instead of in add. This means that you might have two copies of clock that are identical, and if you add 1440 minutes to one and 2880 minutes to the other, they display the same value but are not equal.
Sometimes people only account for one adjustment in either direction, meaning that if you add 1,000,000 minutes, then the clock would not end up with a valid display time.

If this track has a generator for the Clock exercise, go ahead and regenerate it now. If it doesn't, then please verify the implementation of the test suite against the new data. If any cases are missing, they should be added.
See https://github.com/exercism/x-common/issues/166

PR #218 synchronizes this exercise with canonical test data at https://github.com/exercism/x-common/blob/master/clock.json
I hope that allows me to close the issue.


Incompatible with audioplayers >=3.0.1

I try to install the last version of the package and I get this output:
Because social_media_recorder >=0.3.0 depends on uuid ^4.0.0 and audioplayers >=3.0.1 depends on uuid ^3.0.7, social_media_recorder >=0.3.0 is incompatible with audioplayers >=3.0.1.
So, because we_do_osm_flutter depends on both audioplayers ^5.0.0 and social_media_recorder ^0.3.31, version solving failed.
You can try the following suggestion to make the pubspec resolve:

Consider downgrading your constraint on social_media_recorder: flutter pub add social_media_recorder:^0.1.32
exit code 1

Hello @blablabla123-cell
i uploaded new version
social_media_recorder: ^0.3.32
if you have any issue tell me

Thank you, seems like it works now

@blablabla123-cell you're welcome


Allow Pulsar to use BookieID instead of Bookie Network Addresses

This change allows ZkBookieRackAffinityMapping to deal with BookieId instead of raw BookieSocketAddresses.
Summary of changes:

upgrade to BK 4.12.1
use BookieAddressResolver in ZkBookieRackAffinityMapping
Start BookieServer passing "null" as BookieServiceInfo provider instead of BookieServiceInfo.NO_INFO (this change allows the Bookie bundled with Pulsar to publish local endpoints)

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? not documented
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

I will create a followup issue once the patch is ready to be reviewed and merged

This patch is waiting for BK 4.12.1 release, a VOTE is pending

This patch is waiting for BK 4.12.1 release, a VOTE is pending

@sijie I have removed the configuration entry and created the issue for the docs
https://github.com/apache/pulsar/issues/9174

@sijie I have removed the configuration entry and created the issue for the docs
https://github.com/apache/pulsar/issues/9174

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@sijie @codelipenghui CI passed, can you please merge ?
I will follow up with rebasing the upgrade of ZK, that depends on this patch

@sijie @codelipenghui CI passed, can you please merge ?
I will follow up with rebasing the upgrade of ZK, that depends on this patch

@eolivelli Merged

@eolivelli Merged

cool!
thank you very much @codelipenghui

cool!
thank you very much @codelipenghui


Hi @abhijeetnishal On submitting the shortening button -Green color hover effect should come as for good user experience!! please assign the issue under GSSOC'24

Is your feature request related to a problem? Please describe.
I'm always frustrated when the form submission button does not provide immediate visual feedback during the submission process. This lack of feedback can leave users unsure whether their action was registered, leading to confusion or repeated clicks.
Describe the solution you'd like
I would like the submit button to display a green hover effect specifically when the form is in the submitting state. This will visually indicate to the user that their submission is being processed, providing a clear and immediate feedback that improves the overall user experience.
Describe alternatives you've considered
An alternative solution could be to change the button text or add a loading spinner next to the button text. However, these methods either do not provide a clear visual change (changing text) or might be too intrusive (spinner). A color change is subtle yet effective.
Additional context
Here's an example of the implementation in the button component:

Hey @anmolraj-029, the complete UI will be changed as we have redesigned the UI.
Contribute to other issues, good luck.


older flutter_colorpicker is not supportable with flutter 2.5, so update it.

I am facing this error. Help me out of this
/Flutter_SDK/.pub-cache/hosted/pub.dartlang.org/flutter_colorpicker-0.4.0/lib/src/hsv_picker.dart:731:29: Error: The argument type 'PointerEvent' can't be assigned to the parameter type 'PointerDownEvent'.

'PointerEvent' is from 'package:flutter/src/gestures/events.dart' ('/D:/Flutter_SDK/packages/flutter/lib/src/gestures/events.dart').
'PointerDownEvent' is from 'package:flutter/src/gestures/events.dart' ('/D:/Flutter_SDK/packages/flutter/lib/src/gestures/events.dart').
super.addAllowedPointer(event);
^

I have resolved it. You can download it from here - https://drive.google.com/file/d/1DV1OsriI0su3Is13j65NpOpabRlOhDTZ/view?usp=sharing
Now it will work with flutter 2.5
Enjoy :)


(maint) Update CODEOWNERS with modules team

This commit transfers ownership of certain parts of the codebase from the PDK GitHub team to the larger, more current Modules GitHub team.

 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.

I granted the modules team write permission to the puppet repo, but now it's saying the puppetserver-maintainers team doesn't exist. Maybe drop that part?

Yes sorry, I meant to link to the Slack convo I had with @chelnak about this: https://perforce.slack.com/archives/C049PL6AUCU/p1681886300056429?thread_ts=1681862377.373869&cid=C049PL6AUCU
And I've updated my commit to remove puppetserver-maintainers entirely.

thanks for sorting this out for us 😊

Oh sorry, just saw this is targeting main, did you want 7.x?

Yes, thanks for catching that

Going to close and re-open this to pick up the right set of GitHub checks.


Add show diff flag to update-contract command and fix regression

Closes #962, #1000
Description

Fixes regression after 1.0.0 that broke showing diff on flow project deploy
Adds --show-diff options to flow accounts update-contract

For contributor use:

[x] Targeted PR against master branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the Github PR explorer
[x] Added appropriate labels

Codecov Report

Merging #1001 (c119f76) into master (89c7619) will decrease coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #1001      +/-   ##
==========================================
- Coverage   38.34%   38.33%   -0.01%
==========================================
  Files          37       37
  Lines        1836     1839       +3
==========================================
+ Hits          704      705       +1
- Misses       1050     1051       +1
- Partials       82       83       +1

Flag
Coverage Δ

unittests
38.33% <50.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/accounts/contract-add.go
76.36% <50.00%> (-2.49%)
:arrow_down:

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


VpcEndpointType of AWS::EC2::VPCEndpoint is removed after processing

Any key:value pair that has a value of "Interface" will be stripped and removed from the CFN definition
  VPCEndpoint:
    Type: AWS::EC2::VPCEndpoint
    Properties:
      VpcId:
        Fn::ImportValue: !Sub "${VPCStack}-VPCid"
      ServiceName: !Ref ServiceEndpoint
      VpcEndpointType: Interface
      SubnetIds:
        - Fn::ImportValue: !Sub "${VPCStack}-Subnet-PerimeterInternal1"
        - Fn::ImportValue: !Sub "${VPCStack}-Subnet-PerimeterInternal2"
        - Fn::ImportValue: !Sub "${VPCStack}-Subnet-PerimeterInternal3"
      SecurityGroupIds:
        - Fn::ImportValue: !Sub "${SecurityStack}-SecGroup-${SecurityGroup}"

If you run through generate or upsert the key is removed from properties.
Currently effects 0.3.7 and 0.3.8, it could effect earlier releases.

So this isn't working because the definition for that property is set as (in ./pkg/parsers/resources/EC2-VPCEndpoint.go):
	VPCEndpointType     interface{} `yaml:"VPCEndpointType,omitempty"`

The cloudformation docs define it as (https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-ec2-vpcendpoint.html#cfn-ec2-vpcendpoint-vpcendpointtype):
VpcEndpointType
    The type of endpoint. Valid values are Interface and Gateway.
    Required: No
    Type: String
    Update requires: No interruption

The difference being VPC vs Vpc.
Curiously, it's defined with uppercase in the Cloudformation JSON spec. So I'm inclined to think the actual docs are out of sync with the implementation.

The CloudFormation team has confirmed the spec is incorrect and will work to amend.

Workaround in place. Will resolve this when AWS team fixes and we remove the workaround.

hey is there any workaround i can use? i still get this issue with the latest aws-cdk

Can you try the latest version of kombustion, it was updated recently to pass through everything without doing validation locally. Because, the cfn service is always going to be the source of truth on that, and to better work with macros.


Changing Shadow Color

Materialize team,
Could you point me in the right direction of changing the default shadow color if possible?
It seems as if it should be in the variables under SASS, but is not.
Many websites with non traditional color schemes could benefit from changing shadow color / opacity.
Thanks a ton,
Andrew

https://github.com/Dogfalo/materialize/blob/e5757653366f75ee18881f0bf240d4f97e626eab/sass/components/_global.scss#L57 and please close

You can edit the properties of shadows here


Fix crash in TileMap::update_cell_bitmask

Crash described in #32562 issue when trying to reproduce crash with MeshLibrary set to TileMap

Thanks!


Keep source filename

Right now the plugin renames each file using a hash. I think it might be a better idea to allow users to configure this using options.targetFileName. This could accept a function that gets imageSource to allow users to keep the current functionality.
By default (when !options.targetFileName) it should keep the original file name.

Hi @hacknug,
thanks for the request.
Yes it should be possible to disable the automatic filename generation.
After thinking a bit about your request, I have two examples:
Example 1
If I want to embed an image, which is hosted on imgix, I can define some additional url parameters to transform/optimize/manipulate the image.
This means, the URL is
https://imgix.com/image.jpg?option1=1&option2=2

and not
https://imgix.com/image.jpg

Example 2
The image url doesn't contain a dedicated filename in the url.
Example for this is: http://lorempixel.com/400/200/sports/1

For example 1, it's possible to extract the filename via a simple regexp,
but for example 2, I have no idea how to solve this.
The reason why I have hashed the filename ( based on the Image URL ) is to keep it as simple and make sure, that the image name is unique and doesn't overwrite an existing filename, because a filename only isn't unique for me and if I have different sources with the same filename, I have a conflict.
What do you think?
Thanks!
Regards,
Marcus

@noxify, good thoughts. For example 2, I would suggest:

either to generate a filename based on it´s path and Content-Type (i.e. 400-200-sports-1.jpg) or
stick to hash-based file names if there is no file name present

Cheers,
Daniel

Example 1
If I want to embed an image, which is hosted on imgix, I can define some additional url parameters to transform/optimize/manipulate the image.
This means, the URL is
https://imgix.com/image.jpg?option1=1&option2=2

and not
https://imgix.com/image.jpg

This one I'd probably solve with using URL and getting its pathname property. Should be trivial as you mentioned.

Example 2
The image url doesn't contain a dedicated filename in the url.
Example for this is: http://lorempixel.com/400/200/sports/1

In this case I was going to say I would do what @dsturm suggested but realized images could be inside an /assets directory so I'd say:

do what browsers do and name the file 1.jpg
recreate the source's directory structure and save the file in 400/200/sports/1.jpg

In my case I have a bunch of JSON files with about 1000 image links each, most of which are not unique. What I would like is to be able to download all the files I'm missing and skip whatever's already on my repo. In this project I think I would mind to be able to replicate the sources dir structure.
It would also be cool to be able to specify if I want to force a re-download of everything (false by default I'd say).

Added with https://github.com/noxify/gridsome-plugin-remote-image/releases/tag/v1.1.0
Configuration: https://webstone.info/documentation/gridsome-plugin-remote-image/configuration


Analysis server tech debt: Quickfixes are vulnerable to cast errors

Both explicit cast fix and remove unused parameter had casts where they should have been early returns:
  // wrong
  void fix() {
    Expression e = getTarget(); // implicit downcast
    ...
  }

  // correct, and occurs in tons of visitors.
  void fix() {
    AstNode e = getTarget();
    if (e is! Expression) {
      return;
    }
    ...
  }

It may be a better design to codify this behavior directly:
class FixType<T extends AstNode> {
  bool canFixTarget(AstNode target) => target is T;
  void fix(T target);
}

class FixProcessor {
  ...
    if (fix.canFixTarget(target)) {
      fix.fix(target);
    }
  ...
}

class AddExplicitCast extends FixType<Expression> {
  ...
}

This may not change much code in practice, but I wanted to write it down in case we wanted to explore it.
Note that in the "bad" example, it's not just a problem of it being an implicit downcast. It may seem like that is the intended design, and that the downcast will always succeed. However, that is an incorrect assumption, because of the nature of incorrect code. So the downcast is likely to pass in tests, and still crash in the wild.

Curious about thoughts from @bwilkerson, if this is accurate and/or a good idea or there's a better option.

It's an interesting idea. I see how canFixTarget would take care of invalid casts of the target, but it won't take care of invalid casts of children of the target (or parents, or any other nodes or objects). I think a more comprehensive solution might be better because then there's only one pattern to learn.
It seems to me that this would all be fixed by disallowing implicit downcasts (one alternate solution), and that seems like a better approach.

I still believe that disallowing implicit downcasts in the analyzer packages is the better solution, not just for this problem but for code health in general. (I also didn't mention the fact that some fixes are also assists and that they often work when a variety of node kinds have been selected, so there isn't necessarily a single node type being tested for.)
But as I follow up I'll note that FixProcessor has now been restructured to uses separate objects for each kind of fix (called CorrectionProducers) that have a method similar to the fix method above.
I'd really like to see this issue closed in favor of an issue to disallow implicit downcasts (which would also aid in the move to null safety).


Rename RSSlink to RSSLink

The former will be deprecated and eventually removed from Hugo.
Note: Currently both of them exist in Hugo, which is the reason for the cleanup.

Merged, thanks @bep !


Update init.py

getargspec is dprecated in python 3; signature is used instead

pls don't open addtl PR's for this, rather update #12102


Configure codecov

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

Current dependencies on/for this PR:

master

PR #240   👈

This stack of pull requests is managed by Graphite.


The ability to get random items

Basically an Overpowered ability similar to the gold one: only this one gives you random equipment, you could then divide this into various types of items with one ability for weapons, one for armor, one for any number of items such as support stuff. Basically this is an item version of duplicate gold. Another idea I've got involves some sort of random loot script similar to the ones used by Random Equipment and Worldcrawl where there is a chance to generate random loot on existing containers and corpses but one that actually works because both of those mods I've mentioned no longer work with the latest update and their creators have no intention of updating them, I'm just putting forth ideas whichever one of them you like let me know. Basically, it's to remove some of the monotony of loot in Wrath being the same and allow the Isekai to get excellent gear early on

this doesn't sound like a good idea. after spamming the ability a bunch of times to get the item you want, it just basically becomes another way of getting gold if you sell the items, rendering the dupe gold ability obselete.
Alternatively, I think endgame items could be given as starting equipment based on the background / deity.


Add known dropdown a11y issue

preview page: https://saltdesignsystem-git-3490-dropdown-safari-known-75de3d-fed-team.vercel.app/salt/components/dropdown/accessibility
#3490

Hi @origami-z @joshwooding Confirmed this is no longer an issue on MacOS 15.1.
Can you update the issue details to
OS: MacOS 14
And for Recommendations:
add Update to MacOS 15.1+
Then we can close this and #3490

Hi @origami-z @joshwooding Confirmed this is no longer an issue on MacOS 15.1.
Can you update the issue details to OS: MacOS 14
And for Recommendations: add Update to MacOS 15.1+
Then we can close this and #3490

Updated


Evaluating fitted built-in estimators on new dataset

I would like to evaluate for example PropensityScoreWeightingEstimator on a new dataset, different from the one it was fitted on, basically looking for the equivalent of effect(self, df, ...) of the Econml class.
Is there something like that already or shall I have to create one, say by cloning self along the lines of https://github.com/py-why/dowhy/blob/c14aa0e7e75cadfa904d11940df92ec9c7d650e9/dowhy/causal_estimator.py#L507 ?

Created a PR https://github.com/py-why/dowhy/pull/485


fix mypy typechecks

Still work in progress! Down to last few checks
qreader.py:72: error: Argument "blur_kernel_sizes" to "DecodeQRResult" has incompatible type "tuple[tuple[int, int], ...] | None"; expected "tuple[tuple[int, int]] | None"  [arg-type]
qreader.py:85: error: Incompatible default for argument "reencode_to" (default has type "tuple[str, str]", argument has type "str | tuple[str] | list[str] | None")  [assignment]
qreader.py:122: error: Incompatible types in assignment (expression has type "tuple[str] | list[str]", variable has type "tuple[str, ...]")  [assignment]
qreader.py:225: error: Incompatible return value type (got "tuple[tuple[str | None, ...], tuple[dict[str, ndarray[Any, Any] | float | tuple[float | int, float | int]]]]", expected "tuple[dict[str, ndarray[Any, Any] | float | tuple[float | int, float | int]], str | None] | tuple[str | None, ...]")  [return-value]
qreader.py:231: error: "list" expects 1 type argument, but 2 given  [type-arg]
qreader.py:338: error: Unsupported operand types for - ("int" and "ndarray[Any, dtype[generic]]")  [operator]
qreader.py:359: error: Argument "blur_kernel_sizes" to "__threshold_and_blur_decodings" of "QReader" has incompatible type "tuple[tuple[int, int], tuple[int, int]]"; expected "tuple[tuple[int, int]]"  [arg-type]
Found 7 errors in 1 file (checked 1 source file)

Hahah, this wasn't quite ready yet but it was progress so still good – thx @Eric-Canas


Feature request: Tesseact should be able to preserve the layout of the original document

The original layout may have a more complex nature, such as a single image at the top of the page, and the rest of the page split in two columns of text. But after OCR is done, this original layout is lost: Tesseract converts everything in a single column, block by block.
Also, Tesseract tries to convert everything (even images) into text.
Desired:

Tesseract should keep the original layout intact.
Tesseract should be able to recognize image automatically
(or let the user define an "image" type block manually)

See this feature request for gImageReader for further details.

The thing with images is kind of fuzzy. Some might be embedded tables as images, they might be worth OCRing.
As of the output, I think it depends on the output format. HTML could probably preserve quite a bit of layout, while maintaining semantics. (layout in styles / CSS, content in HTML).

Tesseract is the engine focused on OCR only (e.g. user is responsible for image preprocessing and output postprocessing).
Document (structure) analysis and document reconstruction is a much more complex process (and OCR is only part of that) - we do not have the capacity and aim to do it.


zoomify coolers added as separate process #65

Attempt to resolve #65: zoomify added as 2 separate processes zoom_library_coolers  and zoom_library_group_coolers. Each process start with the highest resolution .cool file per library and library_group correspondingly and zoomifies it.
Notes:

balancing is controlled with the same flag/option used for balancing of single-resolution files
library_group balancing is performed even if there is only 1 .cool per group (should be replaced to copying from corresponding library-level .cool ideally)
extra balancing options are controlled with the same flag/option used for single-res files

Commit is tested locally using the test data. It appears to be doing what one would expect: .multires.coolfiles are indeed multi-res.

nice distillership!


Setup release-please to manage updating verisons of containers and packages

Describe what should be investigated or refactored
This project has a lot of Dockerfiles and zarf.yaml configuration that we want to release in lock-step. Setup release please to help automate a lot of the work away.

NOTE: I am unsure if we can use JUST release-please when specifying the images we are including for our helm charts. We are currently managing this part with Zarf by using the ZARF_PKG_TMPL variables.
Before completly remove the PKG_TMPLs for release-please artifacts, please ensure we have a way to build & test local changes without having to either

Manually change versions in the zarf.yaml / values.yaml temporarily to build the test package
OR
Creating (and publishing locally) a new version of the image/package that are distinctly different from the upstream versions with the same tag.

Duplicate of https://github.com/defenseunicorns/leapfrogai/issues/696


Create Markdown corpus readers

Hello,
I've written these markdown corpus readers to handle markdown documents, and figured I'd ask about including them in NLTK.
There are 4 requirements that would be new:

markdown_it
mdit_plain
mdit_py_plugins
yaml

Hopefully you'll find this useful!

@elespike
#2903 has fixed a broken test case which snuck into this PR. I've merged develop into this PR to resolve this broken test.

@elespike – thanks for your contribution. Expanding NLTK functionality brings a maintenance cost, so there needs to be a clear case for adding code. Can you please make the case, e.g. by pointing us to markdown corpora used in NLP work? Thanks.

@stevenbird While I'm not familiar with and haven't found current NLP work using markdown corpora, the inspiration here is to be able to more easily perform NLP on knowledge bases built with markdown, either with:

traditional text editors
specialized applications such as https://obsidian.md/ or https://www.inkdrop.app/
online/self-hosted platforms from which you can read/export markdown, such as https://hackmd.io/ and https://hedgedoc.org/, https://www.notion.so, https://dillinger.io/, or even entire GitHub Wikis.

These readers allow for easy extraction of quotes, code blocks, links, images, and sections, as well as everything that NLTK can do with plain text. =)

@elespike: thanks for your patience... this does look useful, thanks!


Game crashes when i click create new world or load a world when i have extra.encroaching_emeralds enabled

https://gist.github.com/gitunown/bce59b7db3a389f8881caf1afd3c4bef

Duplicate of #373
This has been fixed but not yet released.


Cant buy whistle

cant buy the whistle. cant find it cant put it in the console nothin, and idk what I'm doing wrong or if its a bug or what
editing to say that I also cant find the option to get the pikmin container. typing it does nothing, like the whistle

Could you send me your mod-list or your r2modman profile code?

yeah sure, will this suffice? unsure if there's a better way you meant or smthn

hello? im unsure how to like, get your attention in this. wondering if there's any idea of what is wrong


Please update packages to latest version

Package not working with my project, causing build errors because of an outdated package
../../../../.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:88:26: Error: Type 'DiagnosticableMixin' not found.
class PictureStream with DiagnosticableMixin {

as well as get it

I have the same error

For now I had to downgrade flutter to 1.16.3.
Navigate to the flutter install directory and do "git checkout 1.16.3" (previous stable flutter release) , then flutter doctor.
Remember to do a flutter clean afterwards. For iOS you might need to delete your Pod files as well because
of the downgrade (similar steps for an upgrade).

Ok thank you.I will try it and let you know

It means that this package does no more works with flutter 1.21.0 ? @WianSnyman

Sorry for this guys, I have been super busy. I will work on updating the packages and squashing a couple of bugs in the coming week.
For the meantime, if downgrading Flutter is too much of a hassle, you can try overriding the dependencies; hopefully, that won't break anything.

It means that this package does no more works with flutter 1.21.0 ? @WianSnyman

For the package flutter_svg 0.17.4  it seems so. Hence the upgrade from 0.17.4 to 0.18.0 is required for the latest flutter version.
@wilburt We appreciate your time and effort, thanks a mil!

Yes @wilburt I thing overriding the flutter_svg is better and it works for me.Thank you for your attention and I am waiting for the package update.Thank you also @WianSnyman

Please, update to to 0.0.9-alpha+2.

Please, update to to 0.0.9-alpha+2.

Thanks @wilburt  but 0.0.9-alpha+2 uses flutter_svg 0.18.1 which seems not to work with Flutter 1.22.2. Perhaps you will have to update package again

@austyking, Please, what version of Flutter are you using? flutter packages get appears to be running as at version 1.22.3.

@wilburt Sorry for the delay in response...I am using Flutter version 1.22.2. This is the message I get while installing rave_flutter:
"Because rave_flutter >=0.0.9-alpha+2 depends on flutter_svg ^0.18.1 and MY_APP_NAME depends on flutter_svg ^0.19.0, rave_flutter >=0.0.9-alpha+2 is forbidden.
So, because MY_APP_NAME depends on rave_flutter ^0.0.9-alpha+2, version solving failed."

@austyking, please use dependency override to force your project to use flutter_svg ^0.19.0.


Remove liquid v1 logic and modernize the rest

Did I remove anything important?

@gabriel-v @mugurrus please have a look. I've tested locally and it works like a charm. You can try the image from docker hub liquidinvestigations/core:simplify, it corresponds to this PR.


[BrainHack22] Changes in Community Documentation

[BrainHack22] Changes in Community Documentation .md files

References #23

Minor markdown changes in the contributor-guide.md
Proposed Changes

Removed backslashes before '
Removed backslashes before "

Change Type

[ ] bugfix
[ ] refactoring
[ ] infrastructure
[X] documentation
[ ] other

Checklist before review

[ ] I added everything I wanted to add to this PR.
[X] [Documentation only] I built the docs locally.
[X] My contribution is harmonious with the rest of the code: I'm not introducing repetitions.
[X] My code respects the adopted style, especially linting conventions.
[X] The title of this PR is explanatory on its own, enough to be understood as part of a changelog.
[ ] I added or indicated the right labels.

[ ] I added information regarding the timeline of completion for this PR.

[ ] Please, comment on my PR while it's a draft and give me feedback on the development!

Should I pull this or not yet?


Update marketing.md ... add email marketing tutorial and also update readme.md

{Write down the list of resources or topics you have added.}
{If you have added a resource, please explain your experience with the resource in context of why you have added it.}

By submitting this pull request I confirm I've read and complied with the below requirements.
Failure to properly do so will just result in the pull request being closed and everyone's time wasted.

[x] I have read the Contribution guidelines and made sure my pull request follows it, specifically the part about promotions and markdown format.
[x] Any resources I am adding in this PR have been used by me and I can personally vouch for their quality.
[x] This pull request has a descriptive title. For example, Add [Resource Name] or Add [Topic Name], not Added new entries or update.
[x] I have double checked that I have only updated the minimum files that are required by this PR.

@PragatiVerma18  Please review

Please make different changes through different PRs, I am accepting it for now, but this is not the way to contribute to a project.


Integrating Deployments

General
Instead of "create-and-run" a job we want to split it into "create-deployment" and "run-job".
This will need two api requests and the storing of deployments in the database.
Tasks

[x] Use Deployment API to save deployments
[x] Use deployment-ids in the run-job request
[x] Upload the Quantum Program directly in the Deployment
[ ] Add some Tests

Old estimation ~ 9


Added support for configuring Sippy with GCS provider in Wrangler

Added support for configuring Sippy with GCS provider in Wrangler. Enabling Sippy now prompts if parameters are not present

Could we get some tests in here?
The wrangler r2 bucket and r2 object commands already have tests.
See https://github.com/cloudflare/workers-sdk/blob/52d37ae94f4bdf9275c33b4680605eb0046cad7a/packages/wrangler/src/tests/r2.test.ts#L52

Also is there a docs update to go along with this new feature?


adapter-static fails with css from node_modules

Describe the bug
    <head>
      <script<EMAIL_ADDRESS>      <link<EMAIL_ADDRESS>rel="stylesheet" />
    </head>

Reproduction
Just add a link to an CSS
Logs
> Using @sveltejs/adapter-static
> 404<EMAIL_ADDRESS>(linked from /)
Error: 404<EMAIL_ADDRESS>(linked from /)
    at file:///Users/darolf/dev/wego-backend/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.146_svelte@3.42.1/node_modules/@sveltejs/kit/dist/chunks/index6.js:84:11

System Info
System:
    OS: macOS 10.15.7
    CPU: (8) x64 Intel(R) Core(TM) i7-6920HQ CPU @ 2.90GHz
    Memory: 294.97 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 16.2.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 7.13.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 92.0.4515.131
    Safari: 14.1.2
  npmPackages:
    @sveltejs/adapter-node: next => 1.0.0-next.39
    @sveltejs/adapter-static: next => 1.0.0-next.16
    @sveltejs/kit: next => 1.0.0-next.146
    svelte: ^3.42.1 => 3.42.1

Severity
serious, but I can work around it
Additional Information
No response

You should either move the assets into static, or import them where they're used


Sample error SESSION_ERROR: The user session was not in the expected state.

Hi,
Some of the users fail to launch the ‘Citrix Application Launcher’ when number of users in the test is set to more than 15.
If I run the test with less than 15 users, then all users can initialize and complete the test without any issue. However, when number of users are increased to 25 then few users(3-4 users) fail to initialize or fail to launch the citrix client.
I am running the test in command mode.
The following did not resolve the issue.
•	Increasing user ramp-up period
•	Increasing delays in-between steps
•	Changed below setting in user properties.
o	bzm.citrix.client_factory.client_property.connect_timeout_ms=600000
o	bzm.citrix.client_factory.client_property.logon_timeout_ms=600000
o	bzm.citrix.client_factory.client_property.activeapp_timeout_ms=600000
o	bzm.citrix.client_factory.client_property.icafile_timeout_ms=600000
o	jmeterengine.threadstop.wait=600000
Error Details
ERROR c.b.j.c.s.CitrixBaseSampler: Sample error SESSION_ERROR
com.blazemeter.jmeter.citrix.sampler.SamplerRunException: The user session was not in the expected state
at com.blazemeter.jmeter.citrix.sampler.CitrixBaseSampler.handleClientAction(CitrixBaseSampler.java:162) ~[citrix-jmeter-0.7.6.jar:?]
at com.blazemeter.jmeter.citrix.sampler.CitrixBaseSampler.sample(CitrixBaseSampler.java:268) [citrix-jmeter-0.7.6.jar:?]
at org.apache.jmeter.threads.JMeterThread.doSampling(JMeterThread.java:651) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.threads.JMeterThread.executeSamplePackage(JMeterThread.java:570) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.threads.JMeterThread.processSampler(JMeterThread.java:501) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.threads.JMeterThread.run(JMeterThread.java:268) [ApacheJMeter_core.jar:5.6.3]
at java.base/java.lang.Thread.run(Thread.java:829) [?:?]
Caused by: com.blazemeter.jmeter.citrix.client.CitrixClientException: The user session was not in the expected state
at com.blazemeter.jmeter.citrix.client.windows.WinCitrixClient.startSession(WinCitrixClient.java:310) ~[citrix-jmeter-0.7.6.jar:?]
at com.blazemeter.jmeter.citrix.client.AbstractCitrixClient.start(AbstractCitrixClient.java:390) ~[citrix-jmeter-0.7.6.jar:?]
at com.blazemeter.jmeter.citrix.sampler.StartApplicationSampler.doClientAction(StartApplicationSampler.java:97) ~[citrix-jmeter-0.7.6.jar:?]
at com.blazemeter.jmeter.citrix.sampler.CitrixBaseSampler.handleClientAction(CitrixBaseSampler.java:160) ~[citrix-jmeter-0.7.6.jar:?]
... 6 more

Hi @Madeppa
The problem is Citrix Workspace, it does not support high concurrency per machine (level of connections and traffic sent per communication channel). And also it's possible from Citrix Server (exist a limit of sessions allowed per machine or per user)
Some versions of Citrix Workspace, when they exceed 15 sessions, begin to have communication problems and begin to leave sessions hanging (either the client loses the connection or the Citrix server disconnects them).
The manual specifies that no more than 10 concurrent sessions per machine are recommended.
https://github.com/Blazemeter/CitrixPlugin/blob/master/MANUAL.md#configure-test-plan-for-load-simulation
In order to perform load simulations of hundreds or thousands of users, it must be scaled horizontally.
We have carried out this type of tests with Citrix's Private Location solution provided by BlazeMeter, which allows the tests to be horizontally scaled.
It requires building hundreds of machines in which around 15 concurrent sessions are generated in each one (although our recommendation if using more recent versions of workspace is not to exceed 10, given that the more modern the version, the more demanding the network is and the more fragile the load tests become)
When load testing is carried out, there are also errors that begin to occur in Citrix services that require adjustment and require administrator participation. It is part of the work of fine tuning the servers, but it requires understanding what is behind these errors on the part of the client when not being able to start a session (you have to go to the Citrix servers and a performance engineering with experience in Citrix and its administration must analyze).
For the session in incorrect state error, you must have access to the view of active sessions on the servers and be able to send them to close if they are in an incorrect state. Someone with administrative powers can do it.
For those who do not have access to these types of tools, you can also first run a test against all users and test in the first cycle which ones give an error and in the second cycle see if there is any account that still has that problem and with that exclude them.
When our Citrix client detects a session in an invalid state, it sends the server to close the session and in many cases it manages to close that session from the client.
That is why the cycle of 2 logins, in the first cycle it is about solving those who gave an error, and in the second cycle it is verified that they could be closed correctly. If in the second cycle that same account has problems again, it is something to report to the administrator (sometimes there are failures in the Windows server, creating corrupt user profiles that prevent using that user until an administrator sends to recreate that user's profile). .
There are many things to consider, but I just wanted to comment on the limitation that we have detected in Citrix Workspace and issues to consider on the part of the Citrix servers and their limit controls configured by the server (these are things to consider, try to adjust or try to find a way to avoid with an alternative path).
I hope this information will be useful.

Hi @3dgiordano,
Thank you for your reply!!!
Since Citrix generally does not allow the same user to simultaneously run the same application. So, we are triggering each user as a separate jmeter session from command line.
We created the test plan with below command for 25 users. However, the main jmeter citrix workflow script is ‘CreateOrderScript.jmx’ in ‘F:\Scripts’ folder.
F:\JMeter\bin\jmeter -n -t "F:\Scripts\CreateOrderScript.jmx" -JvUsers=${citrix_login_1} -l F:\Results\JTL\CreateOrder.jtl -j F:\Results\Logs\CreateOrder.log

Below is the actual script:

However, are launching the user as a separate individual jmeter session in command mode, it works fine for up to 15 users.
After 15 users few of the users open a Citrix application as UI mode like in below screenshot. This user session is opened all the time in test and it won’t close automictically. Ultimately this user fails as a SESSION_ERROR. Somehow the jmeter plugin is not able to recognize this user and its making as fail.

Appreciate your response.

Hi @Madeppa
You have to diagnose the cause of those hung sessions.
One of the possible causes of sessions remaining unclosed is that jmeter, by default, when the test execution finishes, waits very little time for the thread to be released correctly and performs a forced kill. (you can see in the logs the data from jmeter that mention the foced kill)
That is what can cause client sessions not to be closed (that executions on taskbar). If the server does not have time to close the session and notify the client, the client is left hanging.
Try increasing the wait timeout for thread completion, in user.properties or jmeter.properties
jmeterengine.threadstop.wait=90000
In any case, what this should solve is waiting for the sessions to close correctly, which can help you leave the sessions in a better state if they were left hanging on the server due to some other type of disconnection.
Disconnection and sessions remaining in an invalid state when you exceed 15 is something that must be analyzed with a Citrix engineer, because it is something that Citrix makes the decision to not allow more sessions than it can handle.


support "command" input variable type

Input variables have introduced an excellent way to parameterize and configure builtin interactive variables. But they lack support for contributing new variables.
But there is an easy way to combine the new input variables with the existing command based variables: a new input variable type "command" makes it possible to turn any command based variable into an input variable.
Example:
	"inputs":[
		{
			"label": "processPicker",
			"description": "Select process to attach to",
			"type": "command",
			"command": "extension.processes.pickProcess",
                         "args": {
			    "processTypes": [ "node", "iojs", "electron" ]
                         }
		}
	],

/cc @alexr00

a user input of type "command" is now supported.
variables are now prompted for in a single pass: as a result input and command variables are now evaluated in the order they appear in the task or launch config.
cancelling a prompt aborts the task or launch config.
validation of an "input" only occurs for a needed ones.
intellisense now uses the "enumDescriptions" schema feature.


WIP: dynamic args

I'm still very much interested in this feature. what's the state of it? can we help ?

Can you provide some description for such changes and it shoud be used?

Can you provide some description for such changes and it shoud be used?

The goal was to avoid processing some args that will not be used later.


Support scenario s3 only

Support scenario "S3 only"
The changes touch only S3 scenario. The default S1S2 scenario is not affected.
To see the changes in GUI:

in the drop-down list "Select climate scenario", select "OECM (Scope 3) 1.5 degC"
click anywhere on the page, to make selection confirmed by the GUI
wait for the new temperature score to calculate and load

@MichaelTiemannOSC i don't know, how to set reviewers for the PR. Maybe i don't have permissions, to do that. In any case, i ask you to become a reviewer.
Please also try this branch on your machine, i am curious if what you will see is what you expect.
I am open for feedback on both low-level and high-level views

I'll enhance your permissions in the GitHub org!

TODO:

So perhaps a better terminology cleanup would be to describe this as OECM (S3 separate) benchmark.  We don’t need to be so verbose everywhere, but don’t want you to be surprised if/when you talk about scenarios when you mean benchmarks.

Please don't merge yet.  The approval was intended to apply only to a given commit, not the whole PR.


Automatically stop scanning if in area where reports have been created

Currently scanning can be started automatically when the device is moving in an area where no reports have been created recently. Scanning will continue until the device stops moving. The user should be able to toggle whether the scanning should stop automatically if the device moves to an area where reports have been created recently and the scanning was started automatically

This would be definitely quite nice. I mostly stay at fixed places, and NeoStumbler stays on for areas where it has more than enough data, and actually, I noticed that because of my device's GPS accuracy I suppose, it starts to mark more areas as scanned where I'm not near them.
I mean, I could toggle it manually when I'm out and about, but I'll most likely forget it about it.

Would this bring risk that new networks are added or removed in the area and never submitted due to being scanned previously?

Would this bring risk that new networks are added or removed in the area and never submitted due to being scanned previously?

No, there will be a time limit on how old reports will be used to check whether to stop scanning


add attributes to username field

Add attributes class and placeHolder to username field

I think I messed up few styles here. I did not see the sign in page. I should have used the styles that are similar to the one in signin page. Please take a look at it. Thanksl

I think I messed up few styles here. I did not see the sign in page. I should have used the styles that are similar to the one in signin page. Please take a look at it. Thanksl

yea the classes you given may create a visual difference with the other inputs. but its fine we can fix that later if it becomes an issue :+1:


Codegen EntitySelectionTree + MergedSelections Refactor + Perf Improvments

This PR does a number of things to refactor the Codegen engine in preparation for the disabling of merged fragment fields and improve the performance of the Codegen engine.

EntitySelectionTree now computes the merged fields from merged fragments as needed when the merged fields are calculated, rather than when the fragment is initially merged in.

This reduces memory consumption by not storing duplicated selections.
This likely improves execution time as well
Thanks @BobaFetters for the idea to make this algorithmic improvement!

RootFieldEntityStorage renamed to DefinitionEntityStorage and attached to definition.

The new EntitySelectionTree algorithm requires the entity storage at the time of calculation merged fields (since we are no longer merging the fragments and creating new entities when merging the selections in)

IR.Definition refactored from an enum to a protocol.

This improves usage flexibility.

Refactor IR.SelectionSet

Improve initializers and clean up RootFieldBuilder logic.
Move merged selections to new ComputedSelectionSet

The merged selections require a lot of memory since they hold a lot of duplicate instances of the selections from their direct instance. Making them a lazy var on SelectionSet was convenient for use, but it meant they were retained for the entire time the SelectionSet was in memory. Since IR.Operation and IR.NamedFragment retain their rootField these were being retained for the entire execution of codegen.
ComputedSelectionSet, which now computes and stores the merged selections, is created as needed in the SelectionSetTemplate using the new ComputedSelectionSet.Builder. These computed selection sets can be used when needed to render the selection set templates and then be released. This should improve memory consumption greatly.

Changes the generated model format for using referenced fragments (precursor to hoisted types).

Though we haven't implemented intelligent hoisted types yet, there are times when a field accessor for a merged field points to another generated SelectionSet, rather than generating its own child selection set with the same exact shape.
The determination of whether a selection set can use a referenced fragment depends on the merged selections, but we no longer want to compute these and retain them on the IR.SelectionSet, which stays in memory for the entirety of execution.
Since we don't compute the merged selections until the generation of that actual selection set, we don't know if it will be a referenced fragment at the time of the generation of field accessors, selection set initializer parameters, etc.
To work around this, we are generating those references using the name of the selection set as it would be if it were to be rendered as a child of the current selection set. When we compute the merged selections, if we recognize that we should use a reference to another selection set, we then create a typealias pointing to the correct fully qualified name of the referenced fragment instead of generating the selection set.

Create new IRTestWrapper classes in ApolloCodegenInternalTestHelpers.

Because a lot of our tests are using the test subscripts to access and verify the built selections using both the direct and merged selections, they would have required a lot of changes and messy code to build the ComputedSelectionSet for each nested child entity.
These IRTestWrapper instances wrap the existing IR objects. The subscripts for testing are now provided by the IRTestWrapper, which handle the computation of merged fields internally as they are accessed.
This means the changes to the unit test code is minimal, while still maintaining the intentions and functionality of the existing tests.

TODO:

[x] Finish fixing the call sites in unit tests as necessary
[x] Fix type conflict validation (this has to be done in SelectionSetTemplate when the ComputedSelectionSet has been built.)
[x] Improve IRTestWrapper to store the ComputedSelectionSet for reuse during a single test run.
[x] Finish documentation updates
[ ] Run performance tests for metrics

After this PR is completed, the work to create the configuration options for disabling merged fragment fields will be unlocked.

Looking forward to this one

@pm-dev. This is the foundational refactoring needed to enable disabling of fragment field merging. That will finally get us to a point where the codegen will work on your project!


fix field name input issue

When there is already a file field in schama, adding a filename field from the admin panel will be automatically recognized as a duplicate field by the normalizeFieldName function, so filename will become file1name.
https://github.com/pocketbase/pocketbase/blob/1095637bcd9aaf3901bdc075dc430a1003efaa8d/ui/src/components/collections/FieldAccordion.svelte#L91-L100

Could you elaborate more on the issue? Name normalization usually should happen only when the user is typing/paste-ing the field name. There shouldn't be a need for the blur listener.
Currently, if you have an existing field "title", the normalizer will prevent any new fields to be named the same and will try to auto add a suffix counter to the field name, eg. "title1".

Ah, I understand what you mean. I've just reproduced it.
But I think the on:blur is not the best solution for this, because the user will see the correct change (if needed) only on blur and not during typing.

yeah. I think it would be better to use text to inform the user of the existence of duplicate fields, but that might require adding a help block to each schema field.

@clysto I'll close the PR for now. I'll think more on it while working on the admin UI changes required for https://github.com/pocketbase/pocketbase/issues/122.
We always have server-side validations as fallback and maybe your suggestion for showing an error for duplicated field(s), instead of automatically suffixing them, may turn out to be a better approach, but I'll have to experiment with it.

@clysto A slightly better and different duplicated/reserved names handling was added in v0.3.0 release.

wow, the new admin panel is a lot easier to use, thanks for developing it.❤


update screens

needs a new Browse screen

original has the pink theme on second screenshot, yours doesen't.

also, why tf is your statusbar so big??? it's like 1cm tall mlao. orig. screenshots didn't have status bar

Could replace the Latest thing with Feed now I suppose, since latest no longer exists 🙂

I could show an example of feed tomorrow

Ended up making my own
backup file I used to make it

so much yuri in the screens, i bet @OncePunchedMan is pleased


can't pass a File id with a slash

Riiif::Image.file_resolver.find('1/1.5mb-avatar')
ArgumentError: Invalid characters in id `1/1.5mb-avatar`

Fixed by #29


Fix proximity fade in Compatibility renderer

Fixes https://github.com/godotengine/godot/issues/89942.
Supersedes https://github.com/godotengine/godot/pull/89966.
Uses the correct NDC for the Compatibility renderer.
Uses the preprocessor defines for the current renderer introduced in 4.4 in https://github.com/godotengine/godot/pull/98549. We also document a nearly identical reconstruction of world space using NDC in this tutorial, which now uses the preprocessor defines to be renderer-independent.

Looks like this runs into the same problem as https://github.com/godotengine/godot/pull/97646#issuecomment-2529909391 - the preprocessor is not available in BaseMaterial3D. Either that's an engine bug that can be fixed, or this PR is not actually the right approach, and something more like https://github.com/godotengine/godot/pull/89966 is right.

Instead of using the preprocessor, you can do the if/else in C++ and add code to the generated string conditionally.


🌐 Add German translation for docs/de/docs/reference/security/index.md

← reference/openapi/*.md (#10838)
→ reference/encoders.md (#10840)
German translation progress

📝 Docs preview for commit 7ec3c1eb800bdffbfdddc44fd6a2bba3652c5501 at: https://c96e3e9c.fastapitiangolo.pages.dev

📝 Docs preview for commit a2433a5f6ca82a5633a30f568a1427b294e2f06b at: https://e3957b3b.fastapitiangolo.pages.dev

Awesome @nilslindemann! :rocket:
As you are the champion of German translations and you have done so many, and with so much attention to detail, I'm gonna go ahead and merge this. :tada:
If others see things that could be improved later, we can check that in new PRs. :nerd_face:


visual (not sound) feedback on copy with ctrl+c

Description of the new feature / enhancement
show some quick and soft visual feedback, like a little screen flash (like Android on screenshot, but softier) when I press ctrl+c
Scenario when this would be used?
I hit ctrl+c many times just to be sure the copy actually happended. Else sometimes when I paste, I get the previous copied item, becouse the ctrl+c didn't work well, and I didn't had any feedback to know it before pasting.
Supporting information
Ditto clipboard manager have the option for sound feedback, but I don't use sound at work to do not disturbe my fellow workers, so I'd rather a visual feedback.
Also there's a workaround I saw sometime ago to show a default Windows notification, but the Windows notification is too much, I think it should be something that wound take so much space on my screen (the Windows notification disrupt my work somethimes, showing in front of something else I'm using on the screen).

/dup https://github.com/microsoft/PowerToys/issues/981


New package: EchelleInstruments v0.0.1

Registering package: EchelleInstruments
Repository: https://github.com/RvSpectML/EchelleInstruments.jl
Created by: @eford
Version: v0.0.1
Commit: 73eeaf7a8ff02863b4c086ce180e6c93817a4307
Reviewed by: @eford
Reference: https://github.com/RvSpectML/EchelleInstruments.jl/commit/73eeaf7a8ff02863b4c086ce180e6c93817a4307#commitcomment-42769868

fixing and will resubmit


Unable to resolve reference to ui-router $state service

I get an error when using tsng to compile one of my controllers after adding a dependency on the $state service from UI-Router:
Running "tsng:dev" (tsng) task
Warning: Error: Can't resolve dependency for controller MyPage.Features.HomePageController with name ng.ui.IState Use --force to continue.

Here is my controller:
/// <reference path="../../../../bower_components/dt-angular-ui/angular-ui.d.ts" />

module MyPage.Features {
	export interface IHomePageViewModel {
		loading: boolean;
		accounts: Array<Models.IAccount>;
		loadAccount(account: Models.IAccount, $event: BaseJQueryEventObject): void;
	}

	class HomePageController implements IHomePageViewModel {
		public loading: boolean;
		public accounts: Array<Models.IAccount>;

		constructor(private $state: ng.ui.IState, private $stateParams: ng.ui.IStateParams, private accountApiService: Apis.IAccountApiService) {
			this.loading = true;
			accountApiService.getAccounts().then(promiseValue => {
				this.accounts = promiseValue;
				this.loading = false;
			});
		}

		public loadAccount(account: Models.IAccount, $event: BaseJQueryEventObject): void {
			$event.stopPropagation();
			this.$state.go("account.load", {
				accountID: account.accountID
			});
		}
	}
}

Please note that the controller worked fine before I added the $state dependency. The $stateParams service is also part of UI-Router, and that is resolved correctly.
I am very new to using the tsng compiler so any help in resolving this would be greatly appreciated.

I found the issue. The parseConstructor method doesn't account for these auto-property modifiers, so when it builds the reference arrays, it always assume that each dependency has 2 tokens (name and type), when it is perfectly legal in typescript to have 3 tokens.
TSNG should respect this feature of TypeScript and make sure to handle the public and private modifiers on constructor arguments properly.


Screenshots are not added to the appendix in reports

I added a screenshot to a finding in the appendix. But the screenshot and the finding are not shown in reports (PDF and HTML as example). The finding and screenshot could be viewed within PeTeReport.
I've attached screenshots of my test environment, yes with fake data to test.
All Topics within the Appendix in PeTeReport

Screenshot is shown when the appendex is viewed within PeTeReport

In the HTML report the appendix is not present for the Screenshot SQL injection

In the PDFreport the appendix is not present for the Screenshot SQL injection

Update....
In the old version from 11/Nov/2021 the appendix does work...
see screenshots of my first test environment
PDF report with screenshot in appendix

HTM report with screenshot in appendix

t

@mvdvaart I think I found the issue, the appendix linked to a finding with None severity was not showed correctly, that should be fixed in 0.5 release. Take a look and let me know. Now Findings (and their Appendix) with severity None are not added to the report.

@1modm , I will test it somewhere this weekend. Was a bit busy last week

I have tested this in version 0.5 and I have to say sadly it did not work yet..
The screenshots which are added to the appendix related to a finding are not shown in appendix. On the website the appendix is shown..
I've attached some screenshots....

Solved, found the issue in the appendix generation code. Thanks for reporting!


Remove functionality: Edit resources – Link to an existing file

There is a function which isn’t working in either live or staging. However, we do not need this functionality and therefore do not require this to be fixed and this functionality can be removed, please:
Edit resources – Link to an existing file
Raised with Datopian as P3 - SR - 00703

@NHSBSA-Open-Data This functionality has been removed. It's live on https://ckan.nhs.staging.datopian.com. Please take a look and let us know if you have any issues. Thanks


Add hugo docsy website

feat: add hugo docsy website
docs: some changes to existing documentation to make it work and make sense in the website
Description
You can see what the website looks like for this code via the netlify deployment here.  If desired - a more granular breakdown of commit changes on the PR I merged in my fork.
Does this change impact docs?

[x] Yes, PR includes docs updates

Release Note
add hugo docsy website

I guess step 1 is to merge and step 2 would be to setup a netlify deployment based on our repo?

@tallaxes how do you feel about letting the base docs that just copied our existing documentation go in as is minus the doc change?
We need to add docs on

Supported cluster configurations
Self Hosted VS NAP
Comprehensive Nodepool docs
FAQs

ETC
Or do you think we should merge docs with AWS and have them in one place? Does the karpenter website code belong here? Referring to questions still to be answered here

So what are next steps once this merges in?
We buy a domain? Then setup netlify to deploy based on main, as well as preview builds?

@sftim  thank you for your review here btw its appreciated!

You can use the netlify deploy url to start.


ista_conv2d is solving the optimization problem for transposed convolutions

Hi @rfeinman
thanks for providing those practical tools.
However, I might have spotted a mistake in lasso.conv2d.ista.ista_conv2d. Actually, I believe that this function is not returning the solution to the case when $A$ is a convolution. Instead, it is returning the result when $A$ corresponds to a transposed convolution.
The ISTA-step is
$$
x_{k+1} = \mathcal{T}_{\lambda t} \bigl(  x_k - 2tA^T (Ax_k - b)   \bigr) \enspace .
$$
Comparing this to your ISTA-step in  lasso.conv2d.ista.ista_conv2d:
def rss_grad(zk):
   x_hat = F.conv_transpose2d(zk, weight, stride=stride, padding=padding)
   return F.conv2d(x_hat-x, weight, stride=stride, padding=padding)

# ista step function
def step(zk):
    return F.softshrink(zk - lr * rss_grad(zk), alpha * lr)

Hence, your update step calculates
$$
x_{k+1} = \mathcal{T}_{\lambda t} \bigl(  x_k - 2tA (A^Tx_k - b)   \bigr) \enspace ,
$$
which corresponds to the optimization problem applied to transposed convolutions. You can run the following example snippet to test my claim:
# Example Script:
import torch
import torch.nn.functional as F
from lasso.conv2d.ista import ista_conv2d

inp = torch.tensor([[1., 2, 1],
                     [2, 0, 1],
                     [3, 1, 1]]).reshape(1, 1, 3, 3)
weight = torch.tensor([[0.5, 1],
                         [2, 3]]).reshape(1, 1, 2, 2)
output = F.conv_transpose2d(inp, weight)
x0 = torch.randn_like(inp)
inp_ista = ista_conv2d(output, x0, weight, alpha=.0, lr=.01, maxiter=100)
print(inp_ista)

What do you think, @rfeinman ? Am I missing something? Otherwise, I would be happy to contribute by providing a fixed version that is able to handle transposed convolutions, as well as, regular convolutions. Furthermore, note that the Lipschitz-constant estimation procedure can remain the same since $\operatorname{Lip}(A) = \sigma(A) = \sigma(A^T) = \operatorname{Lip}(A^T)$ .

Hi @MikeLasz
Yes, you are correct, I have defined the linear operator $A x$ as pytorch's conv_transpose2d (and therefore $A^T x$ as conv2d). This choice was intentional, but perhaps it is confusing given the function name. In sparse coding, the operator $A$ is often referred to as the decoder; therefore I found it more fitting to associate $A$ with conv_transpose, which is more commonly used in convolutional decoders.
As you note, conv vs. conv_transpose is only a technicality, and switching the configuration is straightforward. For the case of stride=1 it's as simple as passing weight.transpose(0,1).flip([2, 3]) in place of weight (up to a difference in padding ops).


[fix] Naming of slides doesn't match those in slides-pdf/, will break links on course website

Related to #22 in that it also requires changes to the website repository:
Some slides in slides-pdf which are linked from the website as well are named like this
slides-derivfree-1-coorddesc.pdf
slides-derivfree-2-nm.pdf
slides-derivfree-3-sa.pdf
slides-derivfree-4-multistart.pdf

But the corresponding files in slides/07-derivative-free  have different names now for whichever reason:
slides-optim-derivative-free-1-coordinate-descent.tex
slides-optim-derivative-free-2-nelder-mead.tex
slides-optim-derivative-free-3-simulated-annealing.tex
slides-optim-derivative-free-4-multistart-optimization.tex

I don't have any opinion regarding which version is "correct", but they have to match such that the website links to the slides correctly.

To resolve this, the contents of slides-pdf will have to be deleted and recompiled from the updated sources in slides/, and at that point the website will need to be updated with corresponding file path changes etc.


100% CPU use by the Language Server on MacOS after the latest upgrade

Feedback from Twitter: https://twitter.com/HenrikOssipoff/status/1090544805539651584

Love the rapid releases, but I'm experiencing 100% CPU utilisation by the language server on macOS after the latest upgrade :( Downgrade to the previous version seems to have fixed the issue

It's not doing any indexing per se. The old algorithm is iterative and reanalyzes to build type information. This can get dicey in some situations where the number of types a given variable can be keeps growing and does fully complete, hence a rework of the core analysis itself (see https://github.com/Microsoft/python-language-server/issues/479#issuecomment-453196291). We've attempted to curb it by adding limits, making things more strict, but got to the end of the rope.
I don't want to give any hard dates, but you can observe progress here: https://github.com/Microsoft/python-language-server/issues?q=label%3AGA
#546 closes many issues and introduces a new LS based on the new analysis, but it is not yet feature complete and will not be the default quite yet.

@musayev-io - our schedule is public. If you install Zenhub extension to Chrome, you should see our milestones and projected beta availability of the new LS. Ex https://github.com/Microsoft/python-language-server/issues/568#workspaces/python-language-server-5b897936af3c0a2186966c0f/reports

I have the same issue on OSX on a repo I use every day. This makes the Python extension completely unusable (For example, trying to jump to a function definition in the same file doesn't ever complete).

I have the same problem on OSX.
python language server process eats almost % CPU: 95-98  all the time and never complete GO to Definition. no traceback on output python
Code Version 1.31.1 (1.31.1)
OS Version: macOS Mojave (10.14.3)
Python version:  3.6.5 (all tried with env)
Microsoft Python Language Server version <IP_ADDRESS>

Duplicate of #610

0.2.16 is now available on beta channel.
"python.analysis.downloadChannel": "beta"

Mikhail - I added this to my settings.json but it says "Unknown configuration setting" and the latest release I see is 2019.2.5558. Should I be putting this setting elsewhere? Thanks!

It's undocumented, so VS Code will warn about it. You can ignore this warning for testing.

Noted! I assume all the code is already in the 2019.2.5558 release?
Please let me know if there's anything you'd like me to test or a place I can provide feedback. Thank you for delivering as promised! Great work guys 👌🏼

Language server is independent of the extension release and is compatible with that is currently out.

You just need to reload VS Code to get the extension to download the new version. File bugs with analysis here. 😄


Table cell's content is not centered despite |:---:| forced alignment

What's happening?
Here is a table in MyST:
| Column | Left-aligned | Centered | Right-aligned |
|--------|:-------------|:--------:|--------------:|
| 1      | 2            | 3        | 4             |
| A      | B            | C        | D             |

I expect the content of the first and second column to be left-aligned, the third column to be centered and the last one to be right aligned.
Unfortunately all cells ends up left-aligned.
Here is the HTML produced for a row:

A

B

C

D

See how the alignment gets hard-coded in the form of a class attribute? This seems to be fixed if I manually edit each  tag and replace class= by style=.
Reproducer

Go to the auto-generated Sphinx documentation at https://kdeldycke.github.io/meta-package-manager/#supported-package-managers and see how all tickmarks ✓ in the table are left-aligned.
Compare the rendered table above with the original source at https://github.com/kdeldycke/meta-package-manager/blob/develop/readme.md?plain=1#L48 in which all columns containing tickmarks ✓ are constrained to be centered.
Compare how GitHub render that same table, respecting the alignment: https://github.com/kdeldycke/meta-package-manager#supported-package-managers

Expectation
Here I expect the contant of each column to respect the alignment constraint defined in Markdown between the header of the table and its rows.
Code of Conduct

[X] I agree to follow the Code of Conduct.

This sounds like a bug to file against myst-parser.
/cc @choldgraf


Minor improvement to printed warnings

Every time a CN3 is found, mofchecker prints that it found one. Probably a leftover from a debug session. See here.

Every time a new MOF is checked, the waring about OpenBabel is printed. See here. It might be more user-friendly to move that to the __init__ or some place that it'd only be called once. Also, conda install openbabel should be changed to conda install -c openbabel openbabel just for completeness.

Probably a leftover from a debug session.

yep :D

Every time a new MOF is checked, the waring about OpenBabel is printed. See here. It might be more user-friendly to move that to the init or some place that it'd only be called once. Also, conda install openbabel should be changed to conda install -c openbabel openbabel just for completeness.

great suggestion!


Homepage should sorted by the newest

The homepage shows baku hantam cases but seems sorted by the creation date. It's kinda weird to scroll down just to get the latest drama.
It's better to reverse the order. Show the newest first.

I assumed it should the oldest tweet from each topic.

If it's too complicated to determine the date by the tweet's date, then just use the creation date of the topic itself.

Sepertinya sudah ditambahkan fitur sort, ta tutup isu ini


Editor crash when click mesh instance 3d with shader material in 3D view.

Godot version
4.0 Beta 8
System information
macOS v13.0
Issue description
Editor crash when click mesh instance 3d with shader material in 3D view.
Steps to reproduce

Create a mesh instance 3d, choose "Box" shape.
Create a ShaderMaterial with a new empty shader.
Click empty space to remove focus for the mesh instance 3d.
Click mesh instance 3d again.

Minimal reproduction project
N/A

I think this was fixed in the last couple of months. Can you still reproduce in RC3?

Yes, it is fixed.


Issue/7

@rultor merge

@rultor merge

@kerelape OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@kerelape Done! FYI, the full log is here (took me 1min)


SAM pattern?

Hi,
I went through you tutorial and I like how it shows that you don't need much to build an app with modern ideas (view = f(model)). Especially, whatever rendering lib you use, it does not really matter :)
However, I knew meiosis before as a "framework" implementing the SAM pattern (https://foxdonut.gitbooks.io/meiosis-guide/content/sam.html). This part is missing from the tutorial. I'm wondering what's the relation between your previous gitbook and this tutorial. Do you intend to add state + model + next action predicate in that? It would be nice to show how to implement the SAM pattern as an extension to the meiosis pattern.
Just my thoughts, take it or leave it :)
PS: I'm also very eager to see / try to write an example with SAM and Reasonml, as the pattern matching would work really nicely with that, I believe.

Hi @qur2
Thanks for your interest in Meiosis, I appreciate it :-)
As you said, the previous version of Meiosis was a library or small framework aiming to implement SAM. Later, I chipped away at the code and the concepts. I had less and less code until I ended up with practically no code at all! That became the Meiosis pattern.
I believe what makes Meiosis unique and compelling is precisely that it's not a library but a powerful and flexible pattern that you can use and adapt to your needs and requirements.
That being said, the relationship to SAM is not as direct as it was. Meiosis is its own pattern. It can be used to implement SAM but it can also be used without necessarily corresponding to SAM. For example, you can use Meiosis without needing proposals, and also without nap().
On the other hand, you can implement those parts of SAM using Meiosis: this example has a more formal propose/accept structure, and this example has nap(). I haven't put these in the "main" Meiosis Tutorial or Wiki because in most use cases that I've come across, the Meiosis pattern solves the problem with less code. Of course, that does not stop you from implementing SAM with Meiosis, as shown in the examples above.
I hope that answers your questions, let me know if I can be of further help.

Many thanks for the feedback. I've translated a bucklescript example to reason: https://github.com/qur2/reason-tea-sam
I've also enjoyed your basic examples with parceljs (feels so lightweight!)
Next is to investigate the 2 links you've posted and grow both my playgrounds. I'll report with the outcome here (hopefully in the form of some repo).
In the meantime, I have one more question: what sets the meiosis pattern apart from TEA?
Have a nice weekend!

@qur2 good to hear! 👍

In the meantime, I have one more question: what sets the meiosis pattern apart from TEA?

I think one major difference with TEA (and Redux) is that Meiosis doesn't require you to come up with names and constants for all the different types of actions of your application. Furthermore, you don't need switch/case statements, if/else statements, or other branching code that needs to look at the type of action and then decide what to do. Personally I find that this reduces considerably the amount of code and effort required as you build your application.
Another difference is with asynchronous actions, side effects, and so on. In Meiosis, you just perform those normally, then call update(...) when it's time to update the application and (automatically) re-render the view. TEA, I believe, requires you to use Effects or similar specific type and mechanism for side effects. The Meiosis pattern doesn't impose anything on you, and I think this gives you more flexibility.


[/r/toolbox] abandon /w/download in favor of /w/get

It's super confusing that we have both /r/toolbox/w/download and /r/toolbox/w/get.  We need to kill one of them in all links and code.  /w/get has always been more actively maintained and should be the go-to in code and sub links.

yeah, the only reason we have both is to prevent some asshole with malicious intend of inserting his own links.

lol, I did this forever ago.


Use docker/login-action instead of azure/docker-login

You should replace, in the example, azure/docker-login with docker/login-action@v3.
azure/docker-login is (obviously) not maintained any more. You haven't updated it for a long time (current version doesn't support node 20)
docker/login-action syntax is:
        name: Login to ACR
        uses: docker/login-action@v3
        with:
          registry: <registry-name>.azurecr.io
          username: ${{ secrets.AZURE_CLIENT_ID }}
          password: ${{ secrets.AZURE_CLIENT_SECRET }}

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

ID: 1c5d2898-9714-9b98-f065-66a59889f909
Version Independent ID: 6a7e0145-9e57-b2e6-04f6-9aeda314bad7
Content: Deploy container instance by GitHub Actions - Azure Container Instances
Content Source: articles/container-instances/container-instances-github-action.md
Service: container-instances
GitHub Login: @tomvcassidy
Microsoft Alias: tomcassidy

@soderlind
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@tomvcassidy
Can you please check and add your comments on this doc update request as applicable.

OIDC should be used whenever possible, as azure/login does:
    permissions:
      id-token: write

    steps:
      - name: 🗝 Az CLI login
        uses: azure/login@v1
        with:
          client-id: ${{ secrets.AZURE_CLIENT_ID }}
          tenant-id: ${{ secrets.AZURE_TENANT_ID }}
          subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}

Would be nice if the docs reflected a way to safely use the az token with docker. I'm guessing running az acr login and not cleaning up afterwards is a recipe for disaster.

Hi @soderlind ! Thanks a lot for finding this issue. I've made the appropriate changes to the doc, and they've been merged as per this PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/271609
The changes will go live later today.
If you find any other issues with the docs, please let me know.
#please-close


Create static libraries for nrt, helpelib etc. with for target offloading on host

Fixed by PR #32


Clean up sample formatting

Decide on a brace arrangement & apply it consistently w/ Prettier
Related: #27 #28

Closed by #55


/entrypoint.sh: 34: exec: 1000:1000: not found

Had Flatnotes running perfectly in Docker on Synology NAS.
Possibly after this latest update the container stops with above error.
EDIT:
Image used: dullage/Flatnotes:latest
I tried different users in PUID and PGID, from root to admin to normal user account.

Oh whoops, this is related to my merged changes here:
https://github.com/dullage/flatnotes/pull/171/commits/43b2ee00fda688feb1c3f73169c846fecde8a774#diff-6f9d41d046756f0ddc2fcee0626bdb50100d12b88f293734eff742818e03efa2L34-R34
This PR added a new experimental Alpine Linux-based Dockerfile which results in a much smaller Flatnotes container image.
Since the gosu tool used at the entrypoint is not available at Alpine Linux we used the equivalent su-exec instead. To keep the same entrypoint script compatible with both images I added a EXEC_TOOL variable to the Dockerfile telling the command of the tool (either EXEC_TOOL=gosu or EXEC_TOOL=su-exec)
However looks like when updating the Docker container this environment variable doesn't gets defined and is causing the script to fail
A fix for this would be adding the following to the /entrypoint.sh script to keep it working for existing users:
if [ ! "$EXEC_TOOL" ]; then
    EXEC_TOOL=gosu
fi

As a hotfix while the fix is ready you can try manually adding the above to the beginning of this script inside the container, or just reinstalling Flatnotes

Thank you for the clarification and explanation. Unfortunately, I'm unable to find the entrypoint.sh file.
Can you give me pointers as to where the file might be in a Docker environment on Synology NAS?

See comment by Code-Otto

Oh whoops, this is related to my merged changes here: 43b2ee0#diff-6f9d41d046756f0ddc2fcee0626bdb50100d12b88f293734eff742818e03efa2L34-R34
This PR added a new experimental Alpine Linux-based Dockerfile which results in a much smaller Flatnotes container image. Since the gosu tool used at the entrypoint is not available at Alpine Linux we used the equivalent su-exec instead. To keep the same entrypoint script compatible with both images I added a EXEC_TOOL variable to the Dockerfile telling the command of the tool (either EXEC_TOOL=gosu or EXEC_TOOL=su-exec)
However looks like when updating the Docker container this environment variable doesn't gets defined and is causing the script to fail
A fix for this would be adding the following to the /entrypoint.sh script to keep it working for existing users:
if [ ! "$EXEC_TOOL" ]; then
    EXEC_TOOL=gosu
fi

As a hotfix while the fix is ready you can try manually adding the above to the beginning of this script inside the container, or just reinstalling Flatnotes

wil it merged into next release? unfortunately, i met the same issue and i don't no how to edit this code in docker...

I've opened a pull request with a proposed fix
Since I'm not a maintainer I cannot do the changes directly however expect that when @dullage is back an updated image with the fix will be available soon

Fix is now available in v4.0.4. Thanks @Code-Otto.

@banjuer - Were you also running flatnotes on a Synology?

@banjuer - Were you also running flatnotes on a Synology?

yes and now it worked well in v4.0.4, Thank you all !


hdsky 签到失败

1: hdsky 2022-08-05 Reason: Sign_in=> Open api qps request limit reached
2: m-team 2022-08-05 Reason: Sign_in=> response.status_code=503
3: pttime 2022-08-05 Reason: Details=> detail: hr not found.

更新插件


Turn back on Slic macros when Axom built release

If Axom is built not in Debug or RelWithDebInfo, Axom will not add AXOM_DEBUG to the core target and we will lose some of our Slic macros.  This should be guarded against/forced on our side, until Axom works around the following issue:
https://github.com/LLNL/axom/issues/713

@white238 , is this still an issue?

Almost positive this is still needed.  I will investigate.


Support spy.on with nullish prototype

Fixes #117.

Looks like travis CI is no longer working. I'll find some time to port to GitHub Actions and then get this merged, it might be a while though.

I can have a look at writing a GitHub action for it if you want, at least for running the tests on every PR. I don't have any experience with writing an action for publishing to npm, but I'll see what I can do. This should happen for every commit on master right? Is the version increased manually or should this happen in an action as well?

I'd like to move Chai packages to a model I use at work, which is that we have a build step that simply runs npm it (e.g. here: https://github.com/github/relative-time-element/blob/main/.github/workflows/test.yml) and an action which runs whenever we publish a release via the GitHub UI, which publishes the package to the various registries (e.g. here: https://github.com/github/relative-time-element/blob/main/.github/workflows/publish.yml).
If you'd like to try and add those in a PR that'd be a great contribution that would be much appreciated!

@keithamus I created another PR that sets up GitHub actions for CI and publishing (#119). The PR was created off the master branch, so it does not include the changes from this PR yet. I propose to merge #119 first, and then we can merge this one, perhaps after a rebase.

@keithamus Is there anything that prevents releasing this on npm? I'm migrating a large codebase to Vue 3 at the moment and I'm using my own fork in package.json for it now
{
  "dependencies": {
    "chai-spies": "github:sebamarynissen/chai-spies#feature/nullish-prototype",
  }
}

but it obviously would be nicer to be able to just use chai-spies from npm.
Let me know if there's anything I can do to help if needed!

@sebamarynissen v1.1.0 has been released. Thanks!


change logo and order of logos and text

Update footer for partner ... again.

@433eros Can you show me a screenshot of what the footer looks like with this change before merging?


Vinai/UUID

This pr adds UUID support cc: https://dev.mysql.com/doc/refman/8.0/en/miscellaneous-functions.html#function_is-uuid

This adds UUID support


refactor: use helper from controller runtime

Use a more concise way to handle "not found" errors that is made possible by using a controller runtime helper.

@jvanz, @viccuad : I've updated golanglint-ci and then fixed all the warnings via 8d936333779b11f9cb2f1314341325b0f3af80a3. Please take a look at the commit description to see the reasons of the changes I've made

Note: this cannot be merged until the 1.12 release is done

It seems there is something wrong, because tests and linter are failing. =(

@jvanz I'm a bit stuck with other tasks, would you be so kind to take this over once your PR about the owner reference is merged? :pray:

@kubewarden/kubewarden-developers , I've updated this PR rebasing it on top of main and did some fixes to make the CI happy. To make all our CI green I had to update the controller-gen binary used. Otherwise, we get a nil pointer deref. That's why you can see so many line changes, I've commited the result of the generator after updating it. Let me know if you're fine with that. Otherwise, I can update the generator in another PR or commit the updated manifest files in another PR.

@kubewarden/kubewarden-developers I've updated this branch rebasing it on top of main and squashing some commits to reduce the number of them.


WIFI MESH update mesh_data_t size with received bytes when using esp_mesh_recv()  (IDFGH-10850)

Is your feature request related to a problem?
Requiring a checksum byte in every packet to verify payload size. Can be prevented/assisted if received bytes was available.
Describe the solution you'd like.
Can be prevented if mesh_data_t size is buffer size as an input, and received bytes as an output.
Describe alternatives you've considered.
No response
Additional context.
No response

@DarmorGamz There is CRC check in the WiFi MAC layer to ensure data integrity and reliability. We won't add the checksum byte in the mesh_data_t. If you really want it, you can add this check on the application layer.

Thanks for reporting, feel free to reopen.


A11y: Long value of a dropdown menu item is read as 2 menu items by screen reader

Long value of a dropdown menu item is read as 2 menu items by screen reader
Pre-conditions:
there is a long value in the dropdown menu item
Steps to reproduce the behavior:

Launch the screen reader
Navigate by the menu items using  up/down arrow keys
Listen to announcing the long menu item value

Expected behavior:
Value is read as 1 menu item
Desktop:
windows10/chrome 91/nvda

Could not reproduce this behavior while testing in Windows 10 with NVDA. It's properly highlighted then dictated as usual with the program.
This may be a disconnect since I used a virtual machine to replicate the issue. Please verify.

https://user-images.githubusercontent.com/90398760/222519250-dd9c6c0d-eb56-4f67-a59f-8cc48656585d.mp4

After reading the first option the Screen reader render the verbiage ' 1 of 2' after reading the second option it renders '2 of2'  .

The original comment was mentioning how the first of the two options was broken in half. What you captured is expected behavior.


Inference ESC-50 fine-tuned model

Hello, authors.
Thank you for sharing the great work.
I tried to fine-tuned AudioSet pretrained model passt-s-f128-p16-s10-ap.476-swa.pt on ESC-50 dataset by using ex_esc50.py.
I got checkpoints saved in output/esc50/_None/checkpoints/epoch=4-step=2669.ckpt.
I want to load the checkpoint and inference with audio file. I am trying to load the checkpoint model and tried to used passt_hear21 for inference but kinda lost track of the process.
Could you please share how to inference with the saved checkpoints on audio file?

Hi thanks!
you can use passt_hear21 like this:
# Loading the weights
p ="output/esc50/_None/checkpoints/epoch=4-step=2669.ckpt"
ckpt = torch.load(p)
net_statedict = {k[4:]: v.cpu() for k, v in ckpt['state_dict'].items() if k.startswith("net.")} # main weights
net_swa  = {k[len("net_swa."):]: v.cpu() for k, v in ckpt['state_dict'].items() if k.startswith("net_swa.")} # swa weights

# getting the model
from hear21passt.base import load_model, get_scene_embeddings, get_timestamp_embeddings

model = load_model(mode="logits").cuda()
model.net.load_state_dict(net_statedict) # loading the fine-tuned weights

# example
wave_example = torch.ones((3, 32000 * 5))*0.5
logits = model(wave_example)

Thank you so much for the reply.
This is my first time of creating inference script in PyTorch. It was a great help to me.
I am gonna share my inference script here in case someone wants to use.
# References
# 1) https://github.com/YuanGongND/ast/blob/master/egs/audioset/inference.py
# 2) https://github.com/kkoutini/passt_hear21

import csv
import argparse
import numpy as np
import torch
from models.passt import *
import torchaudio
from pytorch_lightning import Trainer as plTrainer
from hear21passt.base import load_model

def load_label(label_csv):
    with open(label_csv, 'r') as f:
        reader = csv.reader(f, delimiter=',')
        lines = list(reader)
    labels = []
    ids = []  # Each label has a unique id such as "/m/068hy"
    for i1 in range(1, len(lines)):
        id = lines[i1][1]
        label = lines[i1][2]
        ids.append(id)
        labels.append(label)
    return labels

if __name__ == '__main__':

    parser = argparse.ArgumentParser(description='Example of parser:'
                                                 'python inference --audio_path ESC-50-master/audio_32k/1-5996-A-6.wav '
                                                 '--model_path checkpoints/epoch=2-step=4799.ckpt')

    parser.add_argument("--model_path", required= True,type=str,
                        help="the trained model you want to test")
    parser.add_argument("--audio_path", required= True,
                        help='the audio you want to predict, sample rate 32k.',
                        type=str)

    args = parser.parse_args()

    label_csv = './esc50/esc_class_labels_indices.csv'       # label and indices for ESC-50 data

    # 1. load audio file
    audio_path = args.audio_path
    waveform, _ = torchaudio.load(audio_path)

    # 2. load checkpoint
    checkpoint_path = args.model_path

    ckpt = torch.load(checkpoint_path)
    net_statedict = {k[4:]: v.cpu() for k, v in ckpt['state_dict'].items() if k.startswith("net.")} # main weights
    net_swa  = {k[len("net_swa."):]: v.cpu() for k, v in ckpt['state_dict'].items() if k.startswith("net_swa.")} # swa weights

    # 3. loading the fine-tuned weights
    passt_model = load_model(mode="logits").cuda()
    passt_model.net.load_state_dict(net_statedict)

    trainer = plTrainer(gpus=1)
    print(f'[*INFO] load checkpoint: {checkpoint_path}')

    passt_model = passt_model.to(torch.device("cuda:0"))
    waveform = waveform.to(torch.device("cuda:0"))

    with torch.no_grad():
        output = passt_model(waveform)
        output = torch.sigmoid(output)
    result_output = output.data.cpu().numpy()[0]

    # 4. map the post-prob to label
    labels = load_label(label_csv)

    sorted_indexes = np.argsort(result_output)[::-1]

    # Print audio tagging top probabilities
    print('[*INFO] predict results:')
    for k in range(5):
        print('{}: {:.4f}'.format(np.array(labels)[sorted_indexes[k]],
                                  result_output[sorted_indexes[k]]))


Registrar.activity() and .context() are deprecated

Use ActivityAware to get the activity.
Use getApplicationContext to get context

https://api.flutter.dev/javadoc/io/flutter/plugin/common/PluginRegistry.Registrar.html#activity()


Automatic tracing of user interactions in the browser (e.g. clicks)

In OpenCensus Web, we developed fully automatic tracing of user interactions such as click events. This worked by monkey-patching the Zone.runTask function (and other Zone functions) to intercept events (code link). Then when a click event occurs, we would start a new tracing zone (equivalent to a scope in OpenTelemetry) and then wait for that zone to become "stable", that is to finish all tasks that it spaws besides setInterval periodic timers.
Here's a (doc about its automatic tracing algorithm).
@obecny what would you think about porting this functionality to OpenTelemetry as a new plugin e.g. UserInteractionPlugin or similar?
It would give people the option of getting started quickly with OpenTelemetry since they would get nice spans for their click events without having to do any manual instrumentation besides enabling the tracer and plugin.

sounds great, pls assign this to me


oneflow->tengine init

Note: The oneflow model file should be generated in oneflow's dev_serving branch

The code is mainly in tools/oneflow/oneflow_serializer.(c|h)pp

The code is mainly in tools/oneflow/oneflow_serializer.(c|h)pp

The code is mainly in tools/oneflow/oneflow_serializer.(c|h)pp

The code is mainly in tools/oneflow/oneflow_serializer.(c|h)pp


Validate multiple alternative schemas

I'm trying to build a validator for a configuration file format that has multiple mutually incompatible versions (similar to docker-compose files for example).
Example of how I had hoped to express this:
# The following line does *not* work
any(include('v1_config'), include('v2_config'))
---
v1_config:
  version: int(min=1, max=1, required=False)
  settings:
    v1_setting_a: str()
    v1_setting_b: str(required=False)
---
v2_config:
  version: int(min=2, max=2)
  settings:
    v2_setting_c: str()
    v2_setting_d: int()

Is there any way to express what I'm trying to do here, or is that simply not supported (yet)?

There's actually a PR out for this: https://github.com/23andMe/Yamale/pull/38
It's just quite large and complicated so I haven't been comfortable merging it. If you try it and it works for you please let me know.

PR #60 will solve this issue

This is available in version 2.0.


  I dug into this a bit..
it seems like when the dv environment issues a stop, the following waveform is created:

Specifically, sda_i is first driven 0 at exactly the same time scl_i transitions.  sda_i is then raised later to create the stop condition.
This works fine when we are using vanilla synchronizers, since both signals get the same delay and the start/stop detection always works out.
When we turn on cdc instrumentation however, they can get delayed differently.  And this can cause the device to trigger a false restart, which causes the tests to fail.
I dug around this a bit, and my interpretation of the spec is that even on the "pre-drive" prior to a stop, we still have to satisfy timing requirements. Ie, we cannot have sda_i go low so close to the clock edge, and instead need to give it some "breathing" room.  I did some quick hacks in i2c_driver for this, and the test began to pass.
Let me know what you guys think.
@jdonjdon
@a-will

@tjaychen Yeh it needs abide tSU,STO, and from the waveform you show, my driver violates it

Sorry I types wrong comment. So I deleted it.

@jdonjdon what did you mean by false restart in nack-stop case?

I dug around this a bit, and my interpretation of the spec is that even on the "pre-drive" prior to a stop, we still have to satisfy timing requirements. Ie, we cannot have sda_i go low so close to the clock edge, and instead need to give it some "breathing" room (data setup requirement). I did some quick hacks in i2c_driver for this, and the test began to pass.

Agreed. It's the responsibility of all chips to ensure scl and sda do not change close to each other, and even the host must continuously sample scl to check for when it actually transitions (requirement for host support of clock stretching). For a stop condition, there is an explicit setup time requirement between the scl rising edge and sda rising edge.

I believe this is fixed by https://github.com/lowRISC/opentitan/pull/17321 need to confirm before closing this issue.

Estimates for M2.5

estimate 16
remaining 2023-03-22 1

Closing as the initial issue reported (CDC failures caused by incorrect behaviour from the I2C UVM driver) is fixed by #17321


Disable threading by default

Set the default number of workers to 1, and users can increase it if they want to run in parallel.

Actually, let's keep it as is and now that we have the debug mode I think it would help a lot on investigating problems.