"Form" component documentation page is broken

Just realized that this page doesn't appear in the component sidebar and also has an incorrect title and description: https://thumbprint.design/components/form/scss/
It likely broke when we made the move to MDX a while back.
Regardless, this seems like a good candidate for removal as part of The Great Alignment work (#558):
https://github.com/thumbtack/thumbprint/blob/master/packages/thumbprint-scss/scss/form.scss
@tomgenoni — Thoughts? We'd probably need to move the placeholder styles to some other file.

tp-input-wrap is used in 7 files (4 are an internal tool)
tp-form-fields is used in 10 files.
tp-form-field__item is used in 12 files.

The latter 2 classes can likely be find and replaces w/ Atomic. We may want to replace the input wraps with a similar React component or Atomic depending on the instance.

Yeah, moving it out seems like the right call. Let me know if you want any help with this.


Authz has scope

Motivation
In some cases filters/interceptors need to know if a request has a specific scope, regardless of the request access level. At the moment there is no API for checking that.
Approach
the new utility function in the authz package provides simple API for checking if a request user has a specific scope, abstracting the actual scope retrieval from the token.
Pull Request status

[x] Initial implementation
[ ] Refactoring
[x] Unit tests
[ ] Integration tests

 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.

Coverage increased (+0.07%) to 88.203% when pulling e902c2f35f0750ffcf2e53b693eeff4896d79ac7 on authz-has-scope into bd66c4f65c769dfd86d976b73b9d661d8cdea161 on master.


Idea: show GBIF "enrichments"

Feature or enhancement
Data published through GBIF go through a series of checks and "enrichments", such as aligning scientificNames to the complete name as published in the nomenclature databases, flagging issues like a country coordinate mismatch, and detecting related records such as herbarium duplicates which may carry more data (e.g. images) or have a different identification/determination.
I gather TaxonWorks now makes it possible to publish in GBIF. It would be a fairly simple integration on specimen pages, to be able to present the user with the observations GBIF makes on the published records. This could be pulled using the /occurrence/N, /occurrence/N/verbatim and the /occurrence/N/experimental/related APIs. If the GBIF ID (N) is not known the datasetKey and local occurrenceID can be used e.g. like this. GBIF could provide a different endpoint that provides it all (JSON or HTML) if useful.
If it is of interest we would be happy to discuss how to do this, and how it should look, and offer a PR with implementation.
CC @mjy
Location
Specimen pages
Screenshot, napkin sketch of interface, or conceptual description
No response
Your role
Aggregator of content, looking for ways to give back

@timrobertson100 Thanks.  Jim Beach's example of this yesterday got me thinking about it as well.
I think we're actually going to spin this off as a software agnostic module to do this, as a (more than) proof of concept.  The idea will be Google maps widget style, anyone can add it to their page after including the JS, into their pipeline, via CDN etc.
This will further explore our thought of a DwC "vector" (data + headers) being a key UI driving data structure.  I imagine that a minimal initialization will be possible with a single configuration option, among many other ways:
<div id='my_comparsion' class='vue_dwc_enrichments'
  data-source-dwc="https://api.taxonworks.org/api/v1/collection_objects/123?extend[]=dwc_fields"
  data-option-1
  data-option-n
>

@jlpereira is on holiday, we'll start poking at this for our updated CollectionObject page when he gets back.

Thanks, @mjy
One idea could be GBIF offers this kind of JS widget that provides a view of the record that was published, and the enrichments we apply. You may prefer to make it a TaxonWorks-specific thing though, which brings in more than just GBIF, like the Specify example Jim showed. It might make sense for TW to do one, and still for GBIF to offer this as standard for other publishers.
Either way, let us know if we can help

Of course we'd love it if GBIF built widgets like this that would work JS pipelines. We've spun off a handful of key internal libraries ourselves (see below for a couple examples), and it's in our mission to try and do this where possible, so I don't mind working on a very basic version of this one.

https://github.com/SpeciesFileGroup/sled
https://github.com/SpeciesFileGroup/svg_radial_menu
https://github.com/SpeciesFileGroup/taxonworks_autocomplete
https://github.com/SpeciesFileGroup/waxy

So maybe we'll try it and GBIF could refine, add code, fork it, use it for inspiration etc. as they see fit.  Or if you beat us too it and provide a npm module we'll just use that.

@timrobertson100 Is there an API endpoint like /v1/occurence/:occurrenceId?  We want to hit a restful resource using a unique id to get a single record.

Is there an API endpoint like /v1/occurence/:occurrenceId?

Because occurrenceID is unique, you need the datasetKey as well, so .../occurrence/<datasetKey>/<occurrenceID> such as this example.
You can find datasetKey's using the registry API. If you don't have those, then you'd need to search using .../occurrence/search?occurrenceID=123 but you'd need to disambiguate results when there are more than one.

@timrobertson100 Is it possible to get the original data from GBIF, not just the interpreted?

Yes. We have 3 versions of the record:

The raw view that the publisher provided as picked up on our crawling stream; .../occurrence/gbifID/fragment. This may return JSON text for DwC-A or XML or possible other text data. It's intended mainly for diagnostics.
The verbatim view which captures the raw data (1) reformated into Darwin Core without interpretation beyond what is needed to represent in DwC; .../occurrence/gbifID/verbatim
The interpreted view which you will be most familiar using; .../occurrence/gbifID or the method above

The API docs are also here.
Please say if you need more. Thanks.

Yes. Sorry about that

@timrobertson100 https://github.com/SpeciesFileGroup/gbifference.
We have that widget going live in 0.30.0 this week.  More docs coming there.  Closing this for issue tracking there.

Thanks for letting me know - and congrats. Please ping us if you need anything changed.


Doctrine decimal column type mapped incorrectly as float

When applying the rule TypedPropertyFromColumnTypeRector
See https://getrector.org/demo/1ec73834-ba14-69bc-8cfe-29f5df412c73
Doctrine decimal type should be mapped to string to preserve precision as per:
https://www.doctrine-project.org/projects/doctrine-dbal/en/latest/reference/types.html#mapping-matrix
https://www.doctrine-project.org/projects/doctrine-dbal/en/latest/reference/types.html#decimal
In this case we can also use numeric-string var annotation to specify that it's a numeric string and not just any other string.

Hi, thanks for reporting and links.
The fix might require to change this type to StringType:  https://github.com/rectorphp/rector-doctrine/blob/4ce012666dd939e2956fbed329e5eef79179856f/src/NodeManipulator/ColumnPropertyTypeResolver.php#L56


GodotStart Opens a New Godot Editor

I am running macOS catalina with neovim and iTerm 2.  I have installed vim-godot and when I run ":GodotRun /path/to/scene.tscn" in vim, Godot will not only run the scene, but also open a whole new Godot editor.  This frustrating because I have to close out of the new editor anytime I want to test my game, or else I end up with a lot of instances of godot running.  I believe the error may be at lines 66-67 of autoload/godot.vim.

I don't have macOS so can't check it.
Could you try to (in terminal):

navigate to the folder where your scene is.
run godot scenename.tscn

I just tested it and I got the following error:

However, If I just run "godot" from the directory, it runs the executable without creating it.
I'm going to search around and see how to rectify this error to see if there are any other problems on my end.

Update: If I run "godot scenes/myscene.tscn" from where my whole project is located, it will open the correct scene without creating a new editor window.

What about open godot scenes/myscene.tscn?

When I run open godot scenes/myscene.tscn

When I run open godot scenes/myscene.tscn:

I mean do it from the command line, open is the OSX version of xdg-open of linux and start version of windows.

Oh, my bad.  I ran it while my Godot editor was closed and when I type open scenes/Level.tscn nothing happens.  However, if I have the Godot editor already open, then it will display the scene correctly.

Oh, my bad. I ran it while my Godot editor was closed and when I type open scenes/Level.tscn nothing happens. However, if I have the Godot editor already open, then it will display the scene correctly.

Interesting.  Do you mean the scene is run or is opened in an editor?
If you are able to run a scene with this, could you replace:
https://github.com/habamax/vim-godot/blob/7697cc88a9ae3e5da923a1739f9753ec4c47617b/autoload/godot.vim#L67
with
call system('open ' . shellescape(a:scene_name))

Run godot, restart vim, and try :GodotRun scenes/Level.tscn

I meant that the scene is run without opening an editor.  I will try this now.

Update: so I've tested this and when I type :GodotRun Scenes/Level.tscn, nothing will happen strangely enough.  From what I see, the problem is the "open" call.  On the terminal, you should just type Godot /path/to/scene.tsn without the word "open", everything seems to work as expected.

@rcoconnor Did you follow the Command line tutorial in the Godot docs? It suggests adding the executable to the PATH environment variable; however, as I found in the related issue, opening directly from the executable (e.g. /Applications/Godot.app/Contents/MacOS/Godot) inside the app opens an extra instance. If you use let g:godot_executable=/Applications/Godot.app (adjust as necessary pertaining to where you have it installed, of course), it should work as expected.
@habamax I imagine adding the executable to PATH is fairly common, but for the sake of accessibility, I think you could fix it one of two ways:

Keep the code as is and add documentation stating that Mac users should use the .app file for g:godot_executable
Update the code to account for either case

@rcoconnor Did you follow the Command line tutorial in the Godot docs? It suggests adding the executable to the PATH environment variable; however, as I found in the related issue, opening directly from the executable (e.g. /Applications/Godot.app/Contents/MacOS/Godot) inside the app opens an extra instance. If you use let g:godot_executable=/Applications/Godot.app (adjust as necessary pertaining to where you have it installed, of course), it should work as expected.
@habamax I imagine adding the executable to PATH is fairly common, but for the sake of accessibility, I think you could fix it one of two ways:

Keep the code as is and add documentation stating that Mac users should use the .app file for g:godot_executable
Update the code to account for either case

@miltsghostrehab Does it mean that current implementation works if g:godot_executable is set to Godot.app?

@miltsghostrehab Does it mean that current implementation works if g:godot_executable is set to Godot.app?

Yeah, I'm pretty sure it does :)

Yeah, I'm pretty sure it does :)


Adds for migration of connection strings to appsettings.json

[x] You've read the Contributor Guide and Code of Conduct.
[x] You've included unit or integration tests for your change, where applicable.
[x] You've included inline docs for your change, where applicable.
[x] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

PR Title
Adds for migration of connection strings to appsettings.json
PR Description
Added IUpdater implementation to connection strings.
Based on the existing implementation of AppSettingsConfigUpdater.
Includes changes to the integration tests.
Addresses #64

 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.:x: johanbenschop sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

/azp run


Fix image-actions configuration

See https://github.com/calibreapp/image-actions/blob/08437f2fb2e1548570f2e24cf8c902191c2f5432/README.md#installation for "expected" config
Should fix failure of this check in #3596

Looks good to me, but we will only know once merged...

Fortunately, it runs the new workflow on the PR, so it did catch an issue and failed on my first commit, but I fixed that one issue and it's green now.  So unless there's something else going on with #3596 we should be good.

Ah, awesome


[BUG]: Multiplayer race results

Description
When the multiplayer race ends, the race results are never displayed, it hangs forever.

(optional) What browsers are you seeing the problem on?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I'm working on this


getting started

I am currently trying to get tegola up and running using the Getting-Started tutorial. Can anyone help me and tell me what I am doing wrong?
docker run --name postgis_postgres -e POSTGRES_PASSWORD=password -e POSTGRES_USER=postgres -p 5432:5432  postgis/postgis
export PGPASSWORD=password
export PGUSER=postgres
export PGHOST=localhost
psql -c 'CREATE DATABASE bonn;'
psql -c "CREATE USER \"user\";"
psql bonn < bonn_osm.dump
psql -c "CREATE USER tegola;"
psql -d bonn -c "GRANT SELECT ON ALL TABLES IN SCHEMA public TO tegola;"
psql -c "ALTER USER tegola WITH PASSWORD 'password';"
// Copy config.toml to current directory and change password to 'password'
./tegola serve --config=config.toml

At localhost:8080 i get this:

The v0.15.0 image does also not contain the viewer @ARolek - wasn’t sure if that was intentional or not.

@iwpnd thats not intentional. I'm wondering if the CI ran into an issue building the UI for embedding during the release.

@iwpnd I think I found the issue, it lies with out build system, I am working on a fix.

@wroge @iwpnd  can you take a look at these artifacts  and make sure it works as expected. Thanks.

It works very well, thanks for the quick fix :)


Update script to upstream version 0.1.1, remove dead code, add german translation

First of all thanks for creating this!
I've noticed that upstream has released a version 0.1.1, so I've forked the project and updated the script to the current upstream version.
Also, I've removed all dead code, added error handling for the main application and added a German translation.

Thank you for your contribution!


Button styles

References #2483
Part two of link styles update (part 1: #2505)
Rationalises our button styles.
We now have primary, secondary, tertiary and control buttons, all of which can be either button or a (link) HTML elements.

The documentation for all of these buttons is in markdown, so should be editable through GitHub by anyone on the team (initial notes are copy/pasted from the Zeplin file).
TODO:

[ ] The 'pewter' style for disabled buttons looks very similar to the green to my broken eyes. I've raised this with @Heesoomoon and @GarethO for consideration (probably another PR).

Not part of this PR: slider control buttons. These are currently built in JS, which makes them tricky to update using the new button components. I think that leaving this change until we've moved over to NextJS makes more sense. If not, at least a different PR, because I'm fairly sure it would get gnarly.
Our buttons touch a lot of files – sorry for the large-ish PR.

My question I suppose still sits on whether we have 1 component at the API level (We only ever expose <ControlButton>) and then deal with the complexity within that.
e.g. if someone sends over an href we render a a, else make it a button.

if someone sends over an href we render a a, else make it a button.

I see what you're saying. I could have a look at implementing that.

I could have a look at implementing that.

@davidpmccormick do you think we should?


CFF support in handlr (conversion between citation file formats) in R

Resolve https://github.com/ropensci/handlr/issues/16 by implementing CFF support in handlr.

[ ] Clone handlr
[ ] Requires CFF 1.1.0
[ ] Implement CFF support in handlr

Possibly relevant https://github.com/pik-piam/citation


403 Client Error: Forbidden for url: https://bgp.tools/as/12389

Is there any workaround for this exception?
python3 asnban.py --ufw --asn 12389
Traceback (most recent call last):
  File "/tmp/asnban/asnban.py", line 66, in <module>
    prefixes = bgptools.query_prefixes(args.asn)
  File "/tmp/asnban/providers/bgptools.py", line 13, in query_prefixes
    r.raise_for_status()
  File "/usr/lib/python3.10/site-packages/requests/models.py", line 960, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://bgp.tools/as/12389

Works fine from browser though.

I pushed d057deefea1f9e8a7e240413917c30275e073bc2 which should hopefully fix this error. :)


DS502-1703-MXNET-Week1-Homework-Deadline(OCT 12)

Please submit the link to your repository here for homework.
I will review it and leave comments here. '
Please complete week1 part 1 homework, week1 part is not required for this week :)

https://github.com/tomcattigerkkk/DS502-AI-Engineer/tree/master/DS502-1703/Homework/week1

https://github.com/florasdream/ds502hw1.git

https://github.com/zyxstudycs/DS502-week1-part1.
Yixing Zheng

https://github.com/amigao0502/AI.git

https://github.com/zhengyjo/DS502-AI-Engineer/tree/master/DS502-1703

https://github.com/selinatang95/DS502/tree/master/MXNET_course

https://github.com/CharlesLiuyx/CharlesLiuyx.github.io

Xueying Zhang

Boxuan Guan


Guides for beta release

We are going to create multiple placeholders with the contents to be covered:

basics

what is CARTO VL?
what can I do with CARTO VL
how authentication works

viz

what is a viz object ?

supported properties

styling points
styling polygons
styling lines
colors
style by value

cc @inigomedina @iagolast

I'm going to close this one in favour of https://github.com/CartoDB/carto-vl/issues/362


Pagination, right and left arrows get darker and darker when you click them until you can't see the icons

I've seen this on the demo page as well as my own local datatable. When you click on the pagination buttons for left or right. The buttons get darker until the point where you cant see the icons. Any have a thought/potential fix for this?
I'm thinking short term to track down the css and disable/simplify the click,hover effects.
***** Looks like this issue can be closed. Problem is caused by materialize being imported multiple times on a page. This is a materialize css issue. I found a few cases of multiple imports I had throughout my app. ******

Indeed I had just checked, this does not occur on my local build. It's most likely a materialize css issue.


Library doesn't work with multi-module project

Hi! Thanks for all your work! Library doesn't work with multi-module project because it generates the same classes with same packages for each module :(

Hi @graisVictory ,
It would be great if you can specify in which way the library has been used in your case. I've double tested the library in multi module projects and all seems to be working as expected. There might be some other issue we're not aware of that is causing problems in your setup.
We would appreciate a bit more information about the issue you're experiencing so we can act accordingly :)
Kind regards

Hi @stjepanbanek, I have encounter issues with multi-modules project as well.
There are two issues.

When there are multi modules depending on the library. It will have error in mergeLibDex

Caused by: com.android.tools.r8.internal.f: Type com.infinum.jsonapix.JsonXExtensions$format$1 is defined multiple times: .....

Seems to me, that's caused by there is a JsonXExtension.kt for each module. And they all contains property format.

ClassA and ClassB are in different modules and instance of ClassB is property of ClassA.

In :moduleA
@Serializable
@JsonApiX(type = "classA")
data class ClassA(
   @HasOne(type="classB")
    val classB: ClassB
)

In :moduleB
@Serializable
@JsonApiX(type = "classB")
data class ClassB(
    val foo: String
)

When compiling :moduleA, an error will be thrown.
JsonXExtensions.kt: (): Unresolved reference. None of the following candidates is applicable because of receiver type mismatch:
OtherClass.toResourceObject()
AnotherClass.toResourceObject()

Please let me know if I have done anything incorrect. Thanks!

Hi @howardtamhk!
Can you share your dependency setup from the project and module level build.gradle files?
I suspect you might be right about JsonXExtensions being generated twice.
I will test myself when I get the chance. If I can reproduce the issue, I will add it to the planned fixed for the next release.
Thanks for using the lib and the feedback! And for the patience :)

Thanks @stjepanbanek!
I've created a quick sample https://github.com/howardtamhk/JsonApiXTest. Let me know if you cannot reproduce the issues.


flatMap; flattenIterable to include unflattenable elements instead of throwing

right now flatMap throws an error if any items of the mapped result are unflattenable (not Iterable or Object). This behavior is weird; it is more reasonable to not throw and instead just include the unflattenable elements.

resolved by 13d5458


Update files-troubleshoot-smb-connectivity.md

I had recently worked on a customer request where they received the respective error when they were using the right syntax and mount script and had no network connectivity or port issues.
After certain troubleshooting we could find that only issue was service "Workstation" was in stopped state and as soon as we started it, issue was resolved and cx was able to mount the file share.
Ref doc, Also discussed/ confirmed this with windows team: https://tiktoknews.tistory.com/entry/How-to-fix-Windows-error-code-0x800704b3
Pull request guidance
Thank you for submitting your contribution to our support content! Our team works closely with subject matter experts in CSS and PMs in the product group to review all content requests to ensure technical accuracy and the best customer experience. This process can sometimes take one or more days, so we greatly appreciate your patience.
We also need your help in order to process your request as soon as possible:

We won't act on your pull request (PR) until you type "#sign-off" in a new comment in your pull request (PR) to indicate that your changes are complete.

After you sign off in your PR, the article will be tech reviewed by the PM or SME if it has more than minor changes. Once the article is approved, it will undergo a final editing pass before being merged.

https://github.com/MicrosoftDocs/SupportArticles-docs-pr/pull/6332 is merged, so closed this public PR.


fix: handle values before prefix

It should now better handle values before the prefix, and cursor should not jump around if you input values proceeding another value.
Issue #78

Codecov Report

Merging #80 into master will decrease coverage by 2.19%.
The diff coverage is 66.66%.

@@             Coverage Diff             @@
##            master      #80      +/-   ##
===========================================
- Coverage   100.00%   97.80%   -2.20%
===========================================
  Files           12       12
  Lines          174      182       +8
  Branches        63       66       +3
===========================================
+ Hits           174      178       +4
- Misses           0        4       +4

Impacted Files
Coverage Δ

src/components/CurrencyInput.tsx
94.52% <60.00%> (-5.48%)
:arrow_down:

src/components/utils/cleanValue.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


morgen: 2.5.9 -> 2.5.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/morgen/versions.
meta.description for morgen is: "All-in-one Calendars, Tasks and Scheduler"
meta.homepage for morgen is: "https://morgen.so/download"
meta.changelog for morgen is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.5.10 with grep in /nix/store/kksd6h3i1g7hbng9lc0alcyj6qipghzx-morgen-2.5.10
found 2.5.10 in filename of file in /nix/store/kksd6h3i1g7hbng9lc0alcyj6qipghzx-morgen-2.5.10

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
morgen

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kksd6h3i1g7hbng9lc0alcyj6qipghzx-morgen-2.5.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A morgen https://github.com/r-ryantm/nixpkgs/archive/349ac490d1be39ba1541aa1527eaf907cfc7299d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kksd6h3i1g7hbng9lc0alcyj6qipghzx-morgen-2.5.10
ls -la /nix/store/kksd6h3i1g7hbng9lc0alcyj6qipghzx-morgen-2.5.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

morgen

Maintainer pings
cc @WolfangAukang for testing.

Result of nixpkgs-review pr 188200 run on x86_64-linux 1

1 package built:

morgen


Any interest in a typescript and npm PR?

Forked octo about a week ago. Added npm package management, converted the code to typescript, and used webpack to build. Getting close to decent state I think. Doing manual testing and resolving issues.
Any interest in those changes?
In progress here:
https://github.com/zZeck/Octo/tree/typescript
Never tried to contribute to a project before, so hopefully I'm asking this in the right place...
Really like octo. Helped me sort out if my own chip8 interpreter had problems or if I was just seeing ROM quirks.

Hi, @zZeck! Glad to hear you're enjoying Octo!
I'm not currently interested in moving mainline Octo to TypeScript. However, if you do find any Octo bugs while tinkering with your fork (and I'm sure there are still plenty lurking in there), I'd be happy to accept pull requests. Small individual patches that are easy for me to review are greatly appreciated.
Also, If you aren't already aware, some friends and I host an annual October-long game jam centered around Chip8- If you're interested in participating, I can ping you in late September when we're getting everything ready.

Fast response! I was thinking the change might be a bit dramatic, but I figured I'd ask.
Definitely will make PRs for any issues I find. Plan to add automated tests after I get the basics done. Might turn up some stuff.
The octo-ber jam videos were very interesting. Watched all those. Read through most of the something awful thread as well.  Useful information.  Surprisingly difficult to find information about chip8 quirks.
No idea what I'll be doing around September. If I get something together by then, I'll definitely reach out if it's interpreter related, or submit the program it if it's a chip8 game/demo.
Thanks for all the work you've put into this project!


[Tachyon-449] Simplify constructor parameters of StorageTier

Since all information can be gotten from TachyonConf

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3196/
Test PASSed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3197/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3198/
Test PASSed.

@calvinjia  I have modified the code according to your comments, Thanks!

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3209/
Test PASSed.

+1 LGTM

LGTM


Black background when draw Rect

Thank you for create a usefull library but I have 2 issues, please help me.

could you please provide a simple console app (just the Program.cs file) which reproduces the problem?

I am so sorry because Issue 1. It is because I am call wrong function to draw.
About Issue 1. The function DrawText still is my issue. In temp, I use DrawTextWithBackground to draw text.

A problem with Win 7 is: Transparent color become black screen. Because Win 7 disable transparent key. Use this .reg file to fix it.
Open notepad, copy all, save to abc.reg file then run it.
Windows Registry Editor Version 5.00
[HKEY_CURRENT_USER\Software\Microsoft\Windows\DWM]
"ColorizationOpaqueBlend"=dword:00000000

thank you! i did not know about this registry key.
The general fix was to enable the "Windows Aero" feature


[DPP-20] Produce list of Ebay urls

In order to complete DPP-21 it is necessary to have a list of url's to fetch, produce the list following this format:

BangGood URL;Ebay URL

urlbgeb.txt


Rest endpoint for timetable creation not working

This error is logged in the rest logs
wake_robin_rest_server listening on <IP_ADDRESS>:9010
gtfs_export_new arguments: agency_id_array: [
  "1532"
] name: "annie-test-export"
result.rows[0]:  {
  fetch_url: 'https://data.trilliumtransit.com/gtfs/lubbock-tx-us/lubbock-tx-us.zip',
  email<EMAIL_ADDRESS>}
createResponse:  { error: 'Typethis.stream.clearLine is not a function' }
gtfs_export_new arguments: agency_id_array: [
  "493"
] name: "victorville-ca-us"
result.rows[0]:  {
  fetch_url: 'https://data.trilliumtransit.com/gtfs/merrimackvalley-ma-us/merrimackvalley-ma-us--ttable.zip',
  email<EMAIL_ADDRESS>}
createResponse:  { error: 'Typethis.stream.clearLine is not a function' }
[wakerobin@gtfs-h ~]$

And it's seen in the gtfstohtml logs
/home/gtfstohtml/.pm2/logs/gtfs-to-html-service-out.log last 15 lines:
1|gtfs-to- | Importing - timetable_stop_order.txt - 400 lines imported
1|gtfs-to- | Importing - timetable_stop_order.txt - 600 lines imported
1|gtfs-to- | Importing - timetable_stop_order.txt - 700 lines imported
1|gtfs-to- | Completed GTFS import
1|gtfs-to- | Completed GTFS import for 1 file
1|gtfs-to- | [  Error: this.stream.clearLine is not a function
1|gtfs-to- | ]
1|gtfs-to- | event - build page: /
1|gtfs-to- | wait  - compiling...
1|gtfs-to- | event - compiled successfully
1|gtfs-to- | event - build page: /next/dist/pages/_error
1|gtfs-to- | wait  - compiling...
1|gtfs-to- | event - compiled successfully
1|gtfs-to- | wait  - compiling...
1|gtfs-to- | event - compiled successfully

Testing on https://data.trilliumtransit.com/gtfs/merrimackvalley-ma-us/merrimackvalley-ma-us--ttable.zip

@evantrillium @ekeaton FYI! This is a gtfs-to-html-service issue not an email issue, looking at it now

petaluma's ttable url works on all g2h-service servers and local
merrimack's ttable url works on local, but no g2h-service servers

@evantrillium Ok this is working for me now! Successfully got an email for merrimack's timetable feed.
I think I'll open an issue to send an email when problems with timetable generation occurs, rather than just leaving the requester hanging and wondering if it's still running.


remove the duplicated and combine them

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

Needs approval from an approver in each of these files:

content/OWNERS

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

Hi @LillyWu. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@LillyWu Thank you for the contribution!  This recently popped up as a problem in a prior review.  Thanks for taking the time to repair the problem.
@andraxylia you recently mentioned someone had merged some work here that duplicated information.  I've made some guesses about what you were after, but could you provide a review please so the intent is clear?
Cheers
-steve

@lillywu just noticed https://github.com/istio/istio.github.io/pull/1948 - Seems like @andra may be further along in achieving a desirable outcome.  Would you be kind enough to consolidate your work on top of that PR once it merges?
Cheers
-steve

Close this PR as it's duplicated


REL: Logo suggestion

Starting with the base image, some prompting with ChatGPT-4 and also remove.bg and an upscaler, I have gotten to:

I think it should work for the first release..

Included in #11.


No command line help no --version, no --help

I run "atom --help" and get no help.
Could be a windows / shimgen issue.
I tried multiple shells including powershell, msys bash, cmd.
When I run it directly it works:
"C:\ProgramData\chocolatey\lib\Atom.0.138.0\tools\Atom\atom.exe" --help
[4888:1020/152630:INFO:CONSOLE(0)] Atom Editor v0.138.0

Usage: atom [options] [path ...]

One or more paths to files or folders to open may be specified.

File paths will open in the current window.

Folder paths will open in an existing window if that folder has already been
opened or a new window if it hasn't.

Options:
  -d, --dev             Run in development mode.                                             [boolean]
  -f, --foreground      Keep the browser process in the foreground.                          [boolean]
  -h, --help            Print this usage message.                                            [boolean]
  -l, --log-file        Log all output to file.                                              [string]
  -n, --new-window      Open a new window.                                                   [boolean]
  -s, --spec-directory  Set the spec directory (default: Atom's spec directory).             [string]
  --safe                Do not load packages from ~/.atom/packages or ~/.atom/dev/packages.  [boolean]
  -t, --test            Run the specified specs and exit with error code on failures.        [boolean]
  -v, --version         Print the version.                                                   [boolean]
  -w, --wait            Wait for window to be closed before returning.                       [boolean]

$ "C:\ProgramData\chocolatey\lib\Atom.0.138.0\tools\Atom\atom.exe" --version
[4440:1020/153010:INFO:CONSOLE(0)] 0.138.0

This is now fixed if you upgrade to the latest version, 0.174 or later.


Add simple Interface

And may be you deal a standart circle on interface like a prontoface aka repetier host for moving cnc. and options for moving cnc via keypad.

I've created a pull request to add this functionality as it bothered me for quite some time now.

fixed on PR https://github.com/pknoe3lh/cncgcodecontroller/commit/6018c81f5da4e045df3ca587e1cd8abfd9aa948d i think we can close this item

added in 5.0

good upgrade.
I'm trying.
thank you.
----- 原始邮件 -----
发件人：pknoe3lh<EMAIL_ADDRESS>收件人：pknoe3lh/cncgcodecontroller<EMAIL_ADDRESS>主题：Re: [cncgcodecontroller] Add simple Interface (#14)
日期：2015年04月04日 03点50分
Closed #14.
—
Reply to this email directly or view it on GitHub.


Problem in using umap.plot

Hi,
I'm trying to use umap.plot following the instruction from this umap-plotting but I am unsuccesful. I am getting this error:
import umap.plot
ImportError                               Traceback (most recent call last)
 in ()
----> 1 import umap.plot
ImportError: No module named 'umap.plot'
I have installed all dependencies mentioned in the link above but I'm still getting the same error.
Any help will be useful, thanks.

The umap.plot package is only available as part of the 0.4dev branch right now. That means the only way to get it is to check out that branch from github and build and install it yourself. If you are installing with pip on conda it will not be available at this time. Hopefully soon 0.4 will actually be released.

Thank you for your prompt reply. Installing the 0.4dev manually solve the issue.

I have umap-learn 0.4.0rc3 installed and I still get the same error.
I  cloned the dev repo
installed the requirements
and installed the package
I still get ModuleNotFoundError: No module named 'umap.plot'

There must be some installation issues; old versions can potentially cause conflicts if they are kicking around somewhere. If you are using conda try creating a new clean environment and installing into that perhaps.

Hice una pequeña guía, https://mbojao.wordpress.com/2020/04/03/instalar-umap-en-environment-conda-y-windows-10/.
PD: esta en español.


Fix: set remap-path-prefix, closes #6538

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[ ] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information

I wonder if this should be opt-out? I could imagine that it interferes with things like sentry?


"error": "invalid character 'm' looking for beginning of value"

What is the issue?

OS
Windows
GPU
Intel
CPU
Intel
Ollama version
ollama version is 0.1.44

curl http://localhost:11434/api/generate -d '{"model": "qwen2:0.5b","params":"who are you?"}'

Hi there make sure to send the body as JSON in postman


[Event Change Request] page 7011 "Purchase Price List Lines" - OnAfterUpdateColumnVisibility

Could you please include the sender in the event OnAfterUpdateColumnVisibility as requested already (https://github.com/microsoft/ALAppExtensions/issues/21778)?
//+Change
//-[IntegrationEvent(false, false)]
[IntegrationEvent(true, false)]
//-Change
local procedure OnAfterUpdateColumnVisibility(PriceListHeader: Record "Price List Header"; var SourceTypeVisible: Boolean; var JobSourceTypeVisible: Boolean)
begin
end;

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


Fixed #25782 – update docstring on CacheMiddleware and cache_page.

Trac ticket number
ticket-25782
Branch description
Mention a warning against a drawback of using CacheMiddleware and cache_page, if cache keys are different this might be storing the same values twice and results in doubling the cache size.
Note
Sticking to the initial ticket and only "discouraging this",
next step is to clarify do we need to handle this?
raise an exc inside cache_page if CacheMiddleware is already used in the project?
or fire a warning if this was happening?
Checklist

[X] This PR targets the main branch.
[X] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[X] I have checked the "Has patch" ticket flag in the Trac system.
[ ] I have added or updated relevant tests.
[ ] I have added or updated relevant docs, including release notes if applicable.
[ ] I have attached screenshots in both light and dark modes for any UI changes.

Thank you for this PR @wassef911 🎉
I think I would expect to update the documentation of our docs rather than this docstring, most likely adding a warning in here: https://docs.djangoproject.com/en/5.0/topics/cache/#django.views.decorators.cache.cache_page?
Having something like
.. warning:: Avoiding setting ``cache`` when  ``UpdateCacheMiddleware`` is
    installed

    If ``cache`` is set to anything other than the
    :setting:`CACHE_MIDDLEWARE_ALIAS` setting and
    :class:`~django.middleware.cache.UpdateCacheMiddleware` is installed, the
    response will be saved to both caches.

But probably worded a bit nicer
Have you had a look at what it would take to resolve the issue? This might be preferable than documenting it 🤔

thanks for the pointer @sarahboyce,
I'm still a bit lost on how things work TBH. (my first pr)
I'm starting to look into this and wrote a test case under cache.tests.CacheMiddlewareTest. But that issue seems fixed? and I just rushed in without trying I guess?
    def test_view_decorator_and_middleware_not_duplicating_values(self):
        default_view = cache_page(3, cache="other")(hello_world_view)
        middleware = CacheMiddleware(default_view, cache_alias=None)
        request = self.factory.get("/view/")

        response = default_view(request, "1")
        result = middleware.process_request(request)

        # Process the response through the middleware
        processed_response = middleware.process_response(request, response)

        self.assertEqual(response.content, b"Hello World 1")
        self.assertEqual(middleware.cache_alias, "default")
        self.assertIsNone(result)
        self.assertEqual(processed_response.content, b"Hello World 1")

any idea on what todo next? close the ticket or bring it up on the forum?

thanks for the pointer @sarahboyce, I'm still a bit lost on how things work TBH. (my first pr)

No problem 👍

I'm starting to look into this and wrote a test case under cache.tests.CacheMiddlewareTest. But that issue seems fixed? and I just rushed in without trying I guess?

I can see that the response is cached twice so I think it's not fixed
@override_settings(
    CACHES={
        "default": {"BACKEND": "django.core.cache.backends.locmem.LocMemCache"},
        "other": {
            "BACKEND": "django.core.cache.backends.locmem.LocMemCache",
            "LOCATION": "other",
        },
    },
)
class DuplicateCacheKey(TestCase):
    def test_cache_page_conflict(self):
        @cache_page(3000, cache="other")
        def test_view(request):
            return HttpResponse(b"Hello, world")

        request = RequestFactory().get("/view/")
        cache_key = "views.decorators.cache.cache_page..GET.5b30bdb36b35025daf3570b18c3bb916.d41d8cd98f00b204e9800998ecf8427e.en"

        self.assertEqual(caches["default"].has_key(cache_key), False)
        self.assertEqual(caches["other"].has_key(cache_key), False)

        # Process the request through the middleware.
        UpdateCacheMiddleware(test_view)(request)
        # This has now been saved in both caches.
        self.assertEqual(caches["default"].has_key(cache_key), True)
        self.assertEqual(caches["other"].has_key(cache_key), True)

to address this,
added a cache control key called "x-successive-cache" on the response. (no strong opinions about the name. it seems unlikely it will clash with anything.)
and the overall approach is for the cache_page to set that using @cache_control internally, then the middleware will short-circuit if it exists... as it does for "private".
the approach was suggested in the ticket description and sounds sane.
quick side reference:
MDN Web Docs - Cache-Control.

now, with that issue in mind, let's loop back to square 0 ? 😂
idea is:

if we skip ".set" call in the middleware like I did above, we solve duplication but we lose accounting for varys headers set by other successive middlewares in the "cache_key".

if we skip ".set" call in the decorators, we also solve duplication but lose granular control allowed by the decorators in projects having any subclass of "UpdateCacheMiddleware".

so to keep both and still solve initial problem, we should catch when these "two ways of doing things" are conjunction and let the decorator only mark the response for caching, not actually doing it, while forwarding whatever kwargs it has to the middleware to override it's end behaviour.

to maintain the same implementation we'd ideally create a new middleware that does nothing, only marking by a temporary header carrying that behaviour. ("Cache-Marker" : {**things})
for testing we'll have a middleware actually installed instead of just processing then keep the same assertions... ?

another implementation for the same idea might go like:
rewrite the cache decorator to either instantiate a middleware and use or append the key to headers...
but it will alot typing to get async/sync views to both work.
so I'd say we stick to use of "decorator_from_middleware_with_args" ??

@sarahboyce, update is that I backed out from pushing what's mentioned above.
I guess ticket-15855 will have to be worked on in a separate PR.
The implementation I had in my draft wasn't adequate.

for now, I updated release notes for the approved change.
and, Yes I confirm, UpdateCacheMiddleware still needs to be at the top for it to handle undecorated views etc...

Confirming whether it is still necessary to have UpdateCacheMiddleware at the top of the middlewares list after applying that change (and if not updating the docs)

Its not necessary to have UpdateCacheMiddleware on top, it's only a workaround suggested by #15855, but let's question it?
as without FetchFromCacheMiddleware, the UpdateCacheMiddleware will be just setting the value to some cache with the correct headers but not actually serving them as the value by this decorator will be the one served... so the workaround just causes this problem and fixes nothing.
with this PR, will avoid that unnecessary extra write in the middleware, and continue having “Vary headers“ missing.
on the other hand I think the "vary header" problem is smaller to have compared to doubling cache size so we should proceed with this until someone gathers the courage to deprecate the decorator?


Twitch Vod Delete detection seems to not work properly?

As the title says, I have streams that are marked as VOD is missing on Twitch when in fact that's not true:

Link: https://www.twitch.tv/videos/1293859054
Running the endpoint api/v0/cron/check_deleted_vods seems to not change anything/unmark the delete vod from the stream.

is there any odd logging when you run the cron?

The only thing I see after calling that endpoint:
2022-02-20 20:28:24.353 | cron | <INFO> Cronjob deleted check start
2022-02-20 20:28:24.378 | vodclass | <ERROR> No chapters on outerheaven_2022-02-10T22_07_51Z_45480164221!
2022-02-20 20:28:24.603 | vodclass | <ERROR> Trying to check vod valid while not finalized on sodapoppin_2022-02-20T19_19_21Z_45598808509
2022-02-20 20:28:24.604 | cron | <INFO> Cronjob deleted check end

oh right, there's a check in the cron thing that disables checking if it's removed. ?force=1 in the cron url to force it, not sure if i should disable the check considering vods rarely get un-deleted

Ok it ran and it actually fixed it..
I think what happened is I again had the issue #41 (I think) and after that I think the vod checks failed and marked everything as VOD deleted.


Other pointerdown events cause a sticky timeline dot when fast-forwarding

Current Behavior:
If you integrate a pointerdown event on the player's website, e.g. execute alert('test') or open new window/popup etc., and fast-forwarding in the player on the timeline, the third party pointerdown event is executed as expected. If you then close it/return to the player, the cursor is stuck on the timelines dot.
(This may also affect other events, I have not tested any further.)
Expected Behavior:
release the sticky dot and stop the timeslider thumbnails. Click should just play the new desired position.
Steps To Reproduce:
Integrate this on the html page with the player:
    <script>     document.addEventListener('pointerdown', function() {       alert('pointerdown');       console.log('pointerdown');     });     </script>
reload & click on the progress bar to fast forward somewhere.
Environment:
Chrome 119
Windows@11
vidstack/examples.git 1.5.6@next (Web Components -> Default Theme -> Video Player)
Anything Else?
https://github.com/vidstack/player/assets/77133851/496dbd0d-239b-4649-ad52-f5434c1abce5

I don't think there's much we can do here since the pointerup event is interrupted by the alert. It's a little old firing an alert when clicking the page, don't think it's good UX but not sure on full context.
From MDN:

The alert dialog should be used for messages which do not require any response on the part of the user, other than the acknowledgement of the message. Dialog boxes are modal windows - they prevent the user from accessing the rest of the program's interface until the dialog box is closed. For this reason, you should not overuse any function that creates a dialog box (or modal window).

I'll close this out as there's not much we can do here. Ideally this alert should be shown in a more user-friendly way rather than clicking anywhere on the document.


Failed Vote TX

SUMMARY

When I tried to submit vote tx, it gave an error:
level=error module=grpc/client caller=grpc.go:242 ts=2022-03-25T03:06:47.13475968Z msg="request failed" method=/oasis-core.Consensus/SubmitTx req_seq=1 rsp=null err="registry: no such node: governance: failed to query entity node: registry: no such node"
ts=2022-03-25T03:06:47.134838648Z level=error module=cmd/consensus caller=consensus.go:124 msg="failed to submit transaction" err="registry: no such node: governance: failed to query entity node: registry: no such node"

**Laser Lemon** from Slack said it was because I have one unregistered node listed on my entity.  After removed the unregistered node from my entity, my vote tx finally went through.
ISSUE TYPE

Bug Report

COMPONENT NAME

https://github.com/oasisprotocol/oasis-core/blob/32eeab67e9f096c144b174201789edf73cf4dcf4/go/consensus/tendermint/apps/governance/transactions.go#L232
OASIS NODE VERSION

Software version: 21.3.10
Consensus:
  Consensus protocol version: 4.0.0
Runtime:
  Host protocol version:      4.0.0
  Committee protocol version: 3.0.0
Go toolchain version: 1.17.7

STEPS TO REPRODUCE

Make a vote tx as guided in this link https://docs.oasis.dev/general/run-a-node/upgrade-log/#2022-04-11-900-utc---damask-upgrade
When doing the last step (submit the generated transaction), it throw an error.

ACTUAL RESULTS

level=error module=grpc/client caller=grpc.go:242 ts=2022-03-25T03:06:47.13475968Z msg="request failed" method=/oasis-core.Consensus/SubmitTx req_seq=1 rsp=null err="registry: no such node: governance: failed to query entity node: registry: no such node"
ts=2022-03-25T03:06:47.134838648Z level=error module=cmd/consensus caller=consensus.go:124 msg="failed to submit transaction" err="registry: no such node: governance: failed to query entity node: registry: no such node"

EXPECTED RESULTS

Transaction went through.

Would it be enough to change the order in entity.json file?

I think the best approach would be to re-register your entity with all non-working nodes being removed.
We're also working on documenting this. See https://github.com/oasisprotocol/docs/issues/56.


Log report docs

Add more information for the user on how to enable logging and where the logs get saved.

This is replaced with #255

This is replaced with #255


Fixed UI gitch and clean up the code

Description
Fixed UI gitch and clean up the code
Fixed #221
Type of change

[x] Fixed alignment in iPad and other modes.
[x] Clean up the code(Removed unnecessary warnings)

Please check options that are relevant to your PR.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have pulled the latest changes in the repository
[x] I have squashed my commits
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Screenshot

@vaishali614 will you please review my PR.

Clean up code part looks good, but the cards still are not at the center for many view-ports

Will improve it.

@Manvityagi Done! Please check

@Manvityagi Done! Please check

It has improved a lot as compared to the last PR,
but there are still some problems with some screen sizes,
See the gif attached

@Manvityagi  Please check, I have updated all viewports and also removed unnecessary media queries.


jsdoc unexpected token leads to JSDoc not finding files

js-doc is invoked as part of the sphinx-js tooling. Sphinx complains about jsdoc not finding "js" files in a directory that contains valid js files.
It seems that this is a jsdoc configuration issue (or perhaps a babel configuration issue), since in the following I see an invalid token and an indication that this is the destructuring syntax "..." that is the problem.
(docs) [dsargrad@sdbie-sargrad docs]$ make html
Running Sphinx v4.5.0
/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/mkdirp/lib/opts-arg.js:7
    opts = { mode: 0o777, fs, ...opts }
                              ^^^

SyntaxError: Unexpected token ...
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:549:28)
    at Object.Module._extensions..js (module.js:586:10)
    at Module.load (module.js:494:32)
    at Object.load (/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/requizzle/lib/loader.js:105:18)
    at Requizzle.requizzle (/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/requizzle/lib/requizzle.js:87:31)
    at infectProxy (/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/requizzle/lib/loader.js:79:31)
    at Module.targetModule.require.filepath [as require] (/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/requizzle/lib/loader.js:97:44)
    at require (internal/module.js:20:19)

Sphinx error:
jsdoc found no JS files in the directories ['/home/dsargrad/dev/ves']. Make sure js_source_path is set correctly in conf.py. It is also possible (though unlikely) that jsdoc emitted invalid JSON.
make: *** [html] Error 2

How can I determine if this is a jsdoc configuration issue, or a sphinx issue?
I am using jsdoc version 3.6.0.

I'm also using sphinx (4.5.0) and sphinx-js (3.1.2)

I've verified that even with a small program I see this as a jsdoc "issue":
(docs) [dsargrad@sdbie-sargrad test]$ jsdoc main.js
/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/mkdirp/lib/opts-arg.js:7
    opts = { mode: 0o777, fs, ...opts }
                              ^^^

SyntaxError: Unexpected token ...
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:549:28)
    at Object.Module._extensions..js (module.js:586:10)
    at Module.load (module.js:494:32)
    at Object.load (/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/requizzle/lib/loader.js:105:18)
    at Requizzle.requizzle (/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/requizzle/lib/requizzle.js:87:31)
    at infectProxy (/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/requizzle/lib/loader.js:79:31)
    at Module.targetModule.require.filepath [as require] (/home/dsargrad/.conda/envs/docs/lib/node_modules/jsdoc/node_modules/requizzle/lib/loader.js:97:44)
    at require (internal/module.js:20:19)

(docs) [dsargrad@sdbie-sargrad test]$ cat main.js
/**
 *  A person generator
 *
 *  @param {string} name The persons full name
 *  @param {number} age The persons age
 *  @param {boolean} isDeveloper The person is a developer
 */
function createPerson(name, age, isDeveloper) {
	return {
		name: name,
		age: age,
		isDeveloper: isDeveloper
	}
}

Which version of node are you using?

Yeah, I think this is caused by an outdated version of Node.js.


python310Packages.extractcode: disable failing test

Description of changes
Fix build (https://hydra.nixos.org/build/186222562)

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 185457 run on x86_64-linux 1

4 packages built:

python310Packages.extractcode
python310Packages.scancode-toolkit
python39Packages.extractcode
python39Packages.scancode-toolkit


x/tools/cmd/goimports: can't extract issues from gofmt diff output

After upgrading from macOS 12.6 to 13.0, I cannot run goimports linter. The error message I get is:
ERRO Running error: 1 error occurred:
	* can't run linter goimports: goimports: can't extract issues from gofmt diff output "... (\n": can't parse patch: parsing time "2022-10-25 14:43:33" as "2006-01-02 15:04:05 -0700": cannot parse "" as "-0700

Go version and Environment
$ go version && go env
go version go1.19.2 darwin/arm64
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/username/Library/Caches/go-build"
GOENV="/Users/username/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/victor/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/victor/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/homebrew/Cellar/go/1.19.2/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/homebrew/Cellar/go/1.19.2/libexec/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.19.2"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/username/Projects/projectname/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/x5/1q2gh29n4r7_njfgc8hkspgm0000gn/T/go-build2375447467=/tmp/go-build -gno-record-gcc-switches -fno-common"

how are you running goimports?

how are you running goimports?

via golangci-lint run --disable-all -E goimports.

please try running it without golangci-lint

please file an issue with golangci-lint


Hello, 1337

When the original author asked who had the ability to update the R3nzskin project and handed it over to you, why did you act like a cockroach, unable to even update an address feature code, but needing someone else to submit it! After the x64 update, you have to stop R3skinx86. Have you done anything?

Did you fix it?

When the original author asked who had the ability to update the R3nzskin project and handed it over to you, why did you act like a cockroach, unable to even update an address feature code, but needing someone else to submit it! After the x64 update, you have to stop R3skinx86. Have you done anything?

I have contributed to the project way before i've gotten a push access.


Implement airline memberships

A user needs to be able to join a VA. A user can join multiple VAs. We need to implement this in the DB and have a possibility of switching the active airline in the frontend. For the API, the airline is chosen by the parameters of the request. A first start of the implementation was made in PR #25.

This is now implemented barebones in PR #27


Unnecessary keepalive frames

I think, If data is read from a connection, then there is no point in sending a keepalive frame. It is necessary to send the keepalive frame after some timeout from the last read operation. Because success read means that connection is alive, and keepalive frame in this case unnecessary.

can you elaborate?

Currently this code https://github.com/xtaci/smux/blob/master/session.go#L383 send keepalive frame (cmdNOP) every 10 seconds.
During intensive data transfer (cmdPSH), keepalive frames(cmdNOP) are not needed. Because the fact of reading data (cmdPSH) already means that the connection is live.
So, I suggest start sending keepalive frames (cmdNOP) after the last successful reading of data.
The simplest solution is replace https://github.com/xtaci/smux/blob/master/session.go#L384 timer.Ticket to time.Timer, and after successful reading cdmPSH reset time.Timer ;)

can you PR to this issue

for receiving only connection, it's necessary to send cmdNOP perodically.  only 8 bytes in 10 seconds.

Maybe you are right and this is overkill :)


Provide HTTP endpoint for generating Events

It would be nice to create K8S Events out of the POD while executed.
Example:
Post to controller HTTP endpoint /event/
curl -vX POST https://server/event/ -d "{ 'event': 'Antivirus scan Started' }
cheers BigG01

Released with v1.0.0

Great work! :-) and fast as lightning!


Add > 50 FPS Support for camera view

Add 50+ FPS Support for camera view to support people with higher hz than 60

Support.


RNMT-3471 Proper fix for WebView shift in iOS13

Description

As mentioned in https://github.com/OutSystems/cordova-plugin-camera/pull/8 the workaround presented had the benefit of unblocking the customer while upstream worked on a proper fix (rooted on this plugin) for us. Well upstream has indeed just committed a proper fix so this pull-request is threefold:

Apply the proper fix (we can now optionally revert the commit in https://github.com/OutSystems/cordova-plugin-camera/pull/8)
Fix the Upload Widget which did not benefit from said workaround.
Bring us up to speed with upstream (mostly changes in documentation and CI stuff)

Context

Fixes https://outsystemsrd.atlassian.net/browse/RNMT-3483
Upstream pull-request https://github.com/apache/cordova-plugin-statusbar/pull/157

Type of changes

[ ] Feature (change which adds functionality)

[ ] BREAKING CHANGE (existing functionality will not work as expected due to new feature)

[x] Fix (change which fixes an issue)

[ ] BREAKING CHANGE (existing functionality will not work as expected due to fix)

[ ] Performance (change which improves performance)

[ ] BREAKING CHANGE (existing functionality will not work as expected due to performance improvement)

[ ] Refactor (non-breaking change that is neither feature, fix nor performance)
[ ] Style (non-breaking change that only affects formatting and/or white-space)

Components affected

[ ] Android
[x] iOS
[ ] JavaScript

Tests

Manual tests.
Screenshots (if appropriate)

Checklist

[x] Pull request title follows the format RNMT-XXXX <title>
[ ] Tests have been created
[x] Code follows code style of this project
[ ] CHANGELOG.md file is correctly updated
[ ] Changes require an update to the documentation

[ ] Documentation has been updated accordingly

So, basically, from the 30 files, i'v only paid attention to:
src/ios/CDVStatusBar.m
All others were .md, or CI files.

EZPZ 🍋 SQZY


Investigate getaddrinfo ENOTFOUND error

Why?

When the server has no connection.
Watching for changes to graphql...
{ FetchError: request to https://api.cloud.prisma.sh failed, reason: getaddrinfo ENOTFOUND api.cloud.prisma.sh api.cloud.prisma.sh:443
    at ClientRequest.<anonymous> (/Users/steida/dev/este/node_modules/node-fetch/index.js:133:11)
    at ClientRequest.emit (events.js:182:13)
    at TLSSocket.socketErrorListener (_http_client.js:382:9)
    at TLSSocket.emit (events.js:182:13)
    at emitErrorNT (internal/streams/destroy.js:82:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:50:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)
  name: 'FetchError',
  message:
   'request to https://api.cloud.prisma.sh failed, reason: getaddrinfo ENOTFOUND api.cloud.prisma.sh api.cloud.prisma.sh:443',
  type: 'system',
  errno: 'ENOTFOUND',
  code: 'ENOTFOUND' }

Whole Este is going to be rewritten so no relevant anymore.


The read alternative to mapfile is not equivalent

Indeed, IFS=$'\n' read -d "" -ra file_data < "file" will discard empty lines!

Added an example which doesn't eat lines and made a note of the original snippet eating lines. :)


Insert Title Here

Describe the issue in detail here. Attach any relevant screenshots or files. The section below was automatically generated. Remove any information that you do not wish to include in the report.

Debug Data

Property
Value

Name
"Wolfram/Chatbook"

Version
"0.0.30"

EvaluationEnvironment
"Session"

FrontEndVersion
"Desktop: 13.3 for Microsoft Windows (64-bit) (June 8, 2023)"

KernelVersion
"13.3.0 for Microsoft Windows (64-bit) (June 8, 2023)"

SystemID
"Windows-x86-64"

Notebooks
True

DynamicEvaluation
True

SynchronousEvaluation
False

TaskEvaluation
False

Settings
Inherited

Stack Data
catchTop
throwFailure
trimURL

Failure Data
<|
  "Evaluation" :> Wolfram`Chatbook`PersonaInstaller`Private`scrapeResourceFromShingle["https://www.wolframcloud.com/obj/tamas.simon/DeployedResources/Prompt/AILCETest/"]
  "Failure" -> Failure["ConfirmationFailed", <|"Information" :> "StatusCode", "MessageTemplate" -> "Assertion `` failed.", "MessageParameters" -> {HoldForm[Wolfram`Chatbook`PersonaInstaller`Private`resp$58943["StatusCode"] === 200]}, "ConfirmationType" -> "ConfirmAssert", "HeldTest" -> Hold[Wolfram`Chatbook`PersonaInstaller`Private`resp$58943["StatusCode"] === 200], "Test" :> False|>]
  "Arguments" -> {}
|>

Duplicate of #184


How can I access the common file?

How can I access this file on the server? The server has Python 2.7 and 3.5 but I couldn't find it
https://github.com/ytdl-org/youtube-dl/blob/master/youtube_dl/extractor/common.py

Hey ?


Mail subject does not work when using queue:work

Laravel Version: 8
PHP Version: 8
Database Driver & Version: mysql

Description:
I just experienced a problem where for no reason the subject of my emails was not being sent. Googling i found this resource:
https://stackoverflow.com/questions/61836901/laravel-mailable-subject-not-working-if-i-queue-the-mailable
Where it was stated that the problem could be solved by using queue:listen instead of queue:work. Is there an specific reason for why mail subject shouldn't work when using queue:work especially since it is known to have better performance than queue:listen?

You'll need to share your mailable class code and the Mail facade call.
The below example shows the expected email subject line is correctly set for both commands queue:work and queue:listen.

App\Mail\Foo.php
namespace App\Mail;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Mail\Mailable;
use Illuminate\Queue\SerializesModels;

class Foo extends Mailable
{
    use Queueable, SerializesModels;

    public function build()
    {
        return $this->subject('foo')->view('mail.foo');
    }
}

resources/views/mail/foo.blade.php
bar

.env
MAIL_MAILER=log

Laravel Tinker
Mail::to('baz@example.com')->queue(new App\Mail\Foo);

command line
Run php artisan queue:work, then repeat the above step and run php artisan queue:listen.
storage/logs/laravel.log
This email header/body is logged using both commands.
From: Laravel<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: foo
MIME-Version: 1.0
Date: Fri, 25 Feb 2022 01:00:18 +0000
Message-ID<EMAIL_ADDRESS>Content-Type: text/html; charset=utf-8
Content-Transfer-Encoding: quoted-printable

bar

Keep in mind if you're using Mail::fake() and Mail::assertSent()/assertQueued() while testing, the mailable instance build() method must manually be called to make assertions on the subject line.
use App\Mail\Foo;

Mail::fake();

Mail::to('baz@example.com')->queue(new Foo);

// this fails
Mail::assertQueued(Foo::class, function ($mail) {
    // build() hasn't been called so the subject isn't filled yet
    return $mail->subject === 'foo';
});

// this passes
Mail::assertQueued(Foo::class, function ($mail) {
    return $mail->build()->subject === 'foo';
});

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Thanks @derekmd


Can non-angular applications be mocked using protractor?

Hi ,
I am using browser.ignoreSynchronization=true for my non angular application. So, protractor loose track of the http calls in backend. So, while trying any of the mock libraries available for protractor it is not working.
Is there a way to mock non angular applications with protractor?

By what I've read online browser.ignoreSynchronization is deprecated. The recommended way is using browser.waitForAngularEnabled(false)

@Fuun347
Yup even if you write so, protractor still loose track the http asynchronous calls. In this circumstances, what is the way to setup mocks for my protractor tests?

Not too sure myself as I've never had to work with mocks, but this issue here might have an answer for you https://github.com/angular/protractor/issues/125

Try following in your spec.js file
Try setting false for non-angular app page
browser.waitForAngularEnabled(false);
Try setting true for angular app page
browser.waitForAngularEnabled(true);


Micro transpose op ported and tested for TFLM

Fixes #45695
Fixes #43472
Addition of TRANSPOSE operation and its relevant test file to TF Lite for Microcontrollers. This operation has been successfully tested in the following ways:

Running transpose_test with Bazel
Building TFLM for a target nRF52840 DK (Cortex-M) and verifying that the target error Didn't find op for builtin opcode 'TRANSPOSE' version '2' disappeared

More details abut point 1.
The command used is:
bazel test //tensorflow/lite/micro/kernels:transpose_test --verbose_failures

It returns:
.....
Removed for brevity
.....
INFO: Analyzed target //tensorflow/lite/micro/kernels:transpose_test (0 packages loaded, 0 targets configured).
INFO: Found 1 test target...
Target //tensorflow/lite/micro/kernels:transpose_test up-to-date:
  bazel-bin/tensorflow/lite/micro/kernels/transpose_test
INFO: Elapsed time: 0.156s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
//tensorflow/lite/micro/kernels:transpose_test                  (cached) PASSED in 0.0s

Executed 0 out of 1 test: 1 test passes.
INFO: Build completed successfully, 1 total action

More details abut point 2.
I always build locally my version of TFLM with the command:
make -f tensorflow/lite/micro/tools/make/Makefile \
    TARGET=cortex_m_generic \
    TARGET_ARCH=cortex-m4+fp \
    TARGET_TOOLCHAIN_ROOT=/opt/gcc-arm-none-eabi-9-2020-q2-update/bin/ \
    OPTIMIZED_KERNEL_DIR=cmsis_nn microlite

Before applying the fixes of this PR, the error on the target nRF52840 DK was:
[ERR] ./model/debug_log.cc:12: Didn't find op for builtin opcode 'TRANSPOSE' version '2'. An older version of this builtin might be supported. Are you using an old TFLite binary with a newer model?
[ERR] ./model/debug_log.cc:12:
[ERR] ./model/debug_log.cc:12: Failed to get registration from op code TRANSPOSE
[ERR] ./model/debug_log.cc:12:
[ERR] ./model/debug_log.cc:12: Failed starting model allocation.
[ERR] ./model/debug_log.cc:12:
[ERR] ./model/debug_log.cc:12: AllocateTensors() failed

Now, after applying the fixes of this PR, the error disappears and the code can process further at runtime.

@googlebot I fixed it.

Hi @victorromeo, I think I succesfully signed the CLA, but the bot keeps complaining. What about your side? Did you sign with the same email used for the commit?
Thanks!

@dmpiergiacomo I have determined that the email the commits were done under was not a verifiable email address and as a result, I am unable to sign the CLA, without modifying the email addresses first.
I've tried a git filter-branch style replace on a separate branch to swap the email addresses, but this appears to have modified more than I had anticipated. Am hoping to get my historical commits assigned to<EMAIL_ADDRESS>if you know how to do this?

@victorromeo, thank you for the explanation, I understand the problem.
I think the easiest solution would be the manual intervention of a Googler. I can see here that they should have the power to override the cla: yes issue label. @gbaned is this correct? Could you help us?
Another solution could be a git rebase through which I could amend your commits switching to the email address<EMAIL_ADDRESS>I could do this from the same branch of this PR. I would however prefer the first solution, if possible for a Googler.

Alternative 3) Please let me know if you'd like me to re-commit my changes on a clean branch, using the correct email account.

I would however prefer the first solution, if possible for a Googler.

Agreed, as using a Googler will maintain the code base in its current state, however either solution is fine by me.

@dmpiergiacomo Can you please make sure to use same GitHub username and email-id associated with it.

Hi @gbaned apologies for the delay, I had some hard deadlines to respect.

@dmpiergiacomo Can you please make sure to use same GitHub username and email-id associated with it.
I believe I am using the same GitHub username and email-id associated with it. If not, could you please clarify?

From my understanding the issue is that the email used by @victorromeo to push his changes can no longer be accessed to sign Google CLA. We would therefore kindly ask you to override the CLA flag with your admin power. Would this be possible?
Thank you.

Thank you @gbaned for forcing the CLA flag to yes.
It looks like the community CI build now fails, probably something has changed in the meanwhile. Does it make sense to debug it, or better to wait for @advaitjain review first?
Thank you.

Sorry for the delay here. I'm going to close the current PR since https://github.com/tensorflow/tensorflow/pull/48192 is doing the same and is soon going to be merged.

Hi @advaitjain, I believe you might have provided a wrong link in your last message. If it's the case, could you provide the correct one? Thanks.


Change wheel cache path

Change the path of the wheel cache.

I've made it so that the PackageDb now has the WheelCache including the other caches that it has.


Keep getting Failed to connect to AE

I am trying to export my Figma frame (Windows 10) to After Effects (17.5.0 using Creative Cloud latest update)
Using AEUX_0.7.5a
But keep getting: Failed to connect to AE
After Effects is running all normal.
What can I try?

Check that the Ae extension is installed and Ae is open. The panel doesn't have to be open in Ae to work, but double check that it will open.

Try a system restart. Sometimes the connection system gets a little tired and restarting can clear out the cobwebs.

If that doesn't work, there might be a problem with reading the art (and there isn't a check for it). What happens if you try to send a simple rectangle? If that works then the apps are talking to one another and there is a problem with layers. Share a copy of the problematic layer(s) with<EMAIL_ADDRESS>

Any luck?

I have the same problem on my Mac using Figma. When I try to export the frame, it shows a "Failed to connect to Ae". I have After Effects CC2018 installed, and when I open the AEUX plugin, I only see an empty white panel. I also tried the manual installation, but still nothing. Any idea on what's wrong?

@robertabelf AEUX supports CC2019+. Try updating to get the Ae panel working and that should allow for a working connection between apps.


DenoGrant isn't being exported in mod.ts

This doesn't work
import DenoGrant, { OAuth2Client, Providers } from 'https://raw.githubusercontent.com/w3cj/deno_grant/main/mod.ts'

Updated. Thanks!


Claim list empty

Hey! I tried this plugin in MC 1.13.2 with GriefPrevention 16.11.6 but it doesn't seem to work.
Do you require any specific version?
Image: https://i.imgur.com/QR69r1y.png

No idea what version you're using or anything.

Sorry!
ClaimClassifier: The version published in bukkit (1.6 in plugin.yml)
Config: https://pastebin.com/c8f9P0rR
GriefPrevention 16.11.6 - Default config
Server version: Paper version git-Paper-641 (MC: 1.13.2) (Implementing API version 1.13.2-R0.1-SNAPSHOT)

try disabling clickable claimslist?

Yes, with ClickableClaimslist false the claim list work, but this is the function that interests me.

Ok thank you for testing. I'll put some debugging in.


Fix for Cmd-B shortcut conflict on macOS

Fix for https://github.com/godotengine/godot/issues/56961 / https://github.com/godotengine/godot/issues/35429 where the shortcuts on macOS have a conflict with Cmd-B: "Build Project" and "Go to Next Bookmark".  Moves "Go to Next Bookmark" to Shift+Option+Cmd-B. Not great, but the B key is fairly overloaded.
Rationale for changing it is that Cmd-B is one of the most important hotkeys and having it not work out of the box, sometimes, is frustrating.
Discussed briefly here: https://github.com/godotengine/godot/issues/35429#issuecomment-1831917271

Did something go wrong with your commits? It doesn't look like any files are changed 🙂

Oops! Fixed that.

Please squash the commits, see Pull request workflow for details.

Please squash your commits into one, you introduced a merge commit when pushing, see here, remember to push with force

Thanks, and congrats on your first merged Godot contribution!


[BUG] CI failures (cannot access directory)

Describe the bug
There have been a few CI failures recently with the following log output:
[2024-01-16T19:27:38.583Z] ls: cannot access ./rapids_4_spark_profile/runtime.properties/*.csv: Not a directory
[2024-01-16T19:27:38.836Z] Number of '*.csv' in './rapids_4_spark_profile/runtime.properties' less than 4
[2024-01-16T19:27:38.836Z] + '[' 0 -lt 4 ']'
[2024-01-16T19:27:38.836Z] + echo 'Number of '\''*.csv'\'' in '\''./rapids_4_spark_profile/runtime.properties'\'' less than 4'
[2024-01-16T19:27:38.836Z] + exit 1

Steps/Code to reproduce bug
Expected behavior
Environment details (please complete the following information)
Additional context

CI/CD is fixed by @parthosa


Refactor nix rust packages in pkgs/ to re-use logic around filtering source

Currently there's loads of copy-pasta in each of the rust packages under pkgs/ - we should refactor to simplify.

copy-pasta 🍝


Check if backend support is needed for Params support in queries.

@bharath31 @satbir121 @ravikp7 - for any particular API in the uneval data tree, we have config.path that contains the parameters we want to pass to it in this format - api/people/{{this.params.id}}
We can use this to extract the params and show in the UI


Use managed resource name as external name

Description of your changes
This PR introduces Initializer hook to generic managed reconciler. Existing Establishers are refactored to be Initalizers.
Initialize is called before any ExternalClient calls to make sure everything is ready for queries.
The first use is calculating the external name for the managed resource. The default initializer that all managed reconciler will use is now ManagedNameAsExternalName, which means, if not already given, crossplane.io/external-name annotation will be populated by the name of the managed resource.
Additionally;

During Bind operation, we propagate back the external name to the claim's annotation.
During managed resource creation we propagate down the external name annotation from claim to the managed resource.

Fixes https://github.com/crossplaneio/crossplane/issues/624
Implements https://github.com/crossplaneio/crossplane/blob/master/design/one-pager-managed-resource-api-design.md#external-resource-name
Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[x] Updated any relevant documentation, examples, or release notes.
[x] Updated the RBAC permissions in clusterrole.yaml to include any new types.

I am thinking of adding ExternalNameGetter interface to Managed interface so that we can have the following in all managed resources:
func (m *ManagedType) GetExternalName() string {
    if m.Annotations == nil {
        return ""
    }
    return m.Annotations[v1alpha1.ExternalNameAnnotationKey]
}

Instead of using cr.Annotations[v1alpha1.ExternalNameAnnotationKey] everywhere, we can just call that function.

I am thinking of adding ExternalNameGetter interface to Managed interface.

The Managed interface is already pretty large - I'd prefer not to let it grow too much more unless it really needs to. If we really need such functionality I think it would be better to add it as a function in crossplane-runtime/pkg/meta. e.g.:
type Annotated interface {
    GetAnnotations() map[string]string
}

func ExternalName(a Annotated) string {
    # No need to check for nil - a zero value map is safe for reads, and will return its zero value, i.e. ""
    return a.GetAnnotations()[v1alpha1.ExternalNameAnnotationKey]
}

@negz I think this is ready for merge. Just opened https://github.com/crossplaneio/crossplane-runtime/issues/51 to track the finalizer UX thing we discussed.


Various small documentation related changes

Remove tables in README, because go-md2man cannot convert them properly
Few edits in top-level README
Using usage label and command
Remove Fedora Dockerfile from 6-toolchain since it is the same one as in 7-toolchain

The last commit also removes the unnecessary release label.


Fixed the statility issue of the autotuning process

(1) Add an environment variable TF_AUTOTUNE_EPSILON (default=0.03) to enforce a fussy time comparison to reduce the chances of the autotuning process flipping back and forth between close algorithms.
(2) Add a max global autotuning iteration count to completely avoid the case that the autotuner cannot stabilize after many iterations if the two algorithms are very close to each other.
To problem (2), for example:
Suppose the winner sequence is 0 1 0 1 0 1 0 1..., (meaning algorithm 0 and 1 are very close to each other, and they appear as the winner alternatively).
According to the current auto-tuning algorithm and if we have set the TF_AUTOTUNE_THRESHOLD = 2, meaning the target score=2, and max (local) count = 5x2x2=20.
The auto tuner will do this:
0 wins: create node 0 (1, 1) # the numbers in () mean the current score and local_iteration_count
1 wins: demote node 0 (0, 2) and delete node 0 # because the winner changes
0 wins: create node 0 (1, 1)
1 wins: demote node 0 (0, 2)
....
This means the auto-tuner will never end, since the score and the local iteration count can never reach 2 and 20.
FYI @nluehr

Any updates on this PR?

@jlebar Can you review this PR for us?

@jlebar Can you review this PR for us?

I work on XLA, and this is a TensorFlow change, so I can't.  @gunan is also not the right reviewer, that may be why you haven't gotten a response.  cc @tatianashp @chsigg.
(That said, as I understand it this PR says that if algorithm N is 2.9% slower than algorithm N+1, TF will always choose algorithm N.  If we were trying to make this change in XLA I'd have a lot of questions as far as whether this is the best and simplest way to accomplish the goal...)

Kaixi, do you think just adding the max global iterations would be sufficient to prevent endless toggling? Then we wouldn't risk the issue that Justin is describing.

Hi @chsigg , thanks for your comments.
Yes. I have removed other parts and only keep the max global iteration now.


Update restore-deleted-onedrive.md

Fix #2240

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@kaarins PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Consequent NOTE: signs have different sizes

The following
NOTE: Предыдущая версия плагина не поставляется вместе с данным Hotfix'ом.

NOTE: Номер мажорной версии предыдущего релиза не отличается от версии текущего релиза.

gives:

Note that signs sizes are different.

HTML export is fine though

This behavior is as expected. It is due to the fact that the first
admonition only has one line and thus not enough vertical space to fit the
icon at the preferred size. If you want all icons the same size, you need
to customize the theme to make the preferred size smaller or the padding on
the admonition content greater. See the theming guide for details.
We could consider default settings that would allow the preferred size to
fit in the case of a single line.

I see, thx for clarification.
I can't find 'preffered' size in a documentation though.
I tried to set up:
admonition:
  icon:
    note:
      size: 32

and it affects only multirow note:

You found the right property. 24 is the default, so you have to go smaller
like 18.

So it should be somehow aligned with the basefont size?

BTW, I have just tried:
admonition:
  column_rule_style: dotted
  column_rule_width: 2
  icon:
    note:
      size: 32

column_rule_stuff have no effect

And the last question, am I correct that I have to set up sizes for every icon type?

The column rule change is not yet in a release. It will be in alpha.14.


Make test selection more similar to Foundry

Addresses #104
Matches test selection behavior more closely to foundry. Instead of selecting tests with --test where the user is required to use regex to select exactly one test with each supplied --test flag, with this change you now use --match-test which will run any number of tests that match the given regex.
The differences still from foundry are:

Non-test functions are considered
Foundry only allows one --match-test flag. We already support multiple so I don't see a reason to remove this ability.

We should probably update help for --match-test to reflect these changes, i.e., that if a function is overloaded, one can provide a full signature, but otherwise all tests matching the regex or string prefix will be executed:
https://github.com/runtimeverification/kontrol/blob/b4f2513fed8b01e1dc8d00f605acdfcf8a40e698/src/kontrol/__main__.py#L505.

I realised that another diff from Foundry's interface is that we request the user to prove ContractName. in the prefix, but I don't know if we want to address it now.

@anvacaru @RaoulSchaffranek @lucasmt @PetarMax can I ask for your feedback on this change before we merge it? Basically, it's renaming --test to --match-test (to align with Foundry's options) and execute all tests that match the provided string prefix, not a unique one.
I.e., in RollTest with three functions test_roll_set(), test_roll_set(uint256 x), test_roll_setup(), all three will be executed by
kontrol prove --match-test RollTest.test_roll_set

But only test_roll_set(uint256 x) will be executed by
kontrol prove --match-test 'RollTest.test_roll_set(uint256)'

That's what forge test --match-test RollTest.test_roll_set does too ( I just checked), but please let me and @nwatson22  know if you have any usability concerns.

Could you please update help for --match-test to reflect these changes, i.e., that if a function is overloaded, one can provide a full signature, but otherwise all tests matching the regex or string prefix will be executed?
https://github.com/runtimeverification/kontrol/blob/b4f2513fed8b01e1dc8d00f605acdfcf8a40e698/src/kontrol/__main__.py#L505

@palinatolmach I updated it to say: "Specify contract function(s) to test using a regular expression. This will match functions based on their full signature,  e.g., 'ERC20Test.testTransfer(address,uint256)'. This option can be used multiple times to add more functions to test." I hope this gets across that they can use the full signature if necessary or omit it if it's not overloaded. It also removes the explicit request to include the contract name, as this is not actually required. (You  can omit the contract name and it will match methods of the specified name in all the available contracts).