standardise metadata as Metadata{X,Dict{String,Any}}

@ryofurue this PR standardise all RasterStack, Raster and LookupArray metadata to be Metadata{SOURCE,Dict{String,Any}}.
The intention is to allow more strings keys that CF standards allow that are either awkward or impossible with Symbol keys. It's also a breaking change that will require a minor version bump. See also: https://github.com/rafaqz/DimensionalData.jl/issues/406
What is further needed is to define the lists of allowed keys for all metadata subtypes, and allow users to use a plain Dict to define keys, that we can check against allowed keys to warn on mistakes.
@ryoforue if you have any time to collate these allowed key/value combinations from the CF standards, GDAL and Rs GRI/GRI files, or for even some of these types, we could get started on implementing proper metadata checking, conversion, and writing (as outlined in #3 the oldest open issue here!). Seems I haven't found time to do it in the last 3 years :sweat_smile:

collate these allowed key/value combinations from the CF standards, GDAL and Rs GRI/GRI files

Checks to ensure that the metadata conforms to each particular standard would be nice to have, but I was wondering if they would be necessary . . .

I imagine there are pieces of metadata which would cause error when the Julia library (Raster etc) tries to write them into the file. The user needs to be at least notified of such errors.  Even then, the Julia library doesn't have to take any kind actions as long as the presence of the error gets known to the user.  For example, Raster could write a key like "a#b" into a netCDF file, even though the netCDF standard doesn't allow for it.  In such a case, the underlying netCDF library will complain and that would be sufficient.

If the metadata simply breaks "conventions" without causing error, we could let it go.  For example, a netCDF file is usable (and can be useful) even if it doesn't conform to the CF conventions.  The "units" attribute can be useful even if it doesn't conform to the CF conventions. And so on.

So, most of such checks as you describe can wait, I would think.

The reason they are necessary is we can already write a tiff to a netcdf file, and the reverse.
Before we go writing arbitrary metadata between file types we should know the consequences of that.

Before we go writing arbitrary metadata between file types we should know the consequences of that.

I agree.  But, I was concerned about your mention of CF convensions and units.  Violations of these conventions wouldn't have "serious" consequences.  You don't need to implement checks against these. That was my point.
As to serious consequences for netCDF . . .

Names of variables, dimensions, and attributes: In addition to the regular alphanum ASCII characters, several special characters are allowed, such as ".", "_" . . .   (Is there a length limit?)

Data types are precisely defined: Float64, Float32, String (but I don't know whether it's a C-string or not, or what charatersets are allowed), Int32, and so on.

Endianness.  I don't know if it's really possible to cause inconsistency in this area.  But, if there is any chance of inconsistency, one must check against that, I think.  For example, the metadata typically includes the missing value (the _FillValue attribute ), and what if the variable itself and the missing value are stored in different endiannesses?

I can imagine that violation of these would cause serious problems.

Regarding endianness: isn't that taken care of by the netcdf-library through which the reading-writing of the files goes?

Yeah I doubt endianness will be an issue.
Another thing not mentioned here is the fields that NCDatasets.jl writes itself and how it interprets having those fields manually set. That may cause some problems with the current write of metadata as well.

Codecov Report

Merging #304 (d6bba74) into main (5024776) will increase coverage by 0.05%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##             main     #304      +/-   ##
==========================================
+ Coverage   80.02%   80.08%   +0.05%
==========================================
  Files          38       38
  Lines        2889     2897       +8
==========================================
+ Hits         2312     2320       +8
  Misses        577      577

Impacted Files
Coverage Δ

src/sources/smap.jl
0.00% <0.00%> (ø)

src/sources/ncdatasets.jl
87.55% <75.00%> (ø)

src/sources/gdal.jl
91.17% <100.00%> (ø)

src/sources/grd.jl
95.76% <100.00%> (ø)

src/utils.jl
79.72% <100.00%> (+2.45%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ryofurue what I meant by consequences was the ability to write e.g. a scale_factor field that changes the values that NCDatasets.jl will return (as it will automatically adjust the values). Or something similar to that.
It is not clear which of those netcdf attribute fields will actually result in reading different data to what is written, or if that is possible with NCDatasets.jl.


LONGEST INCREASING SUBSEQUENCE

Find the length of the longest subsequence in a given array A of integers such that all elements of the subsequence are sorted in strictly ascending order.
Input Format
The first line contains a single integer n.
Next line contains n space separated numbers denoting the elements of the array.
Constraints
0 < n< 105
0 < Ai < 105
Output Format
Print a single line containing a single integer denoting the length of the longest increasing subsequence.
Sample Input
6
50 3 10 7 40 80
Sample Output
4
Explanation
The longest subsequence in test case is - 3,7,40,80

I would like to work on this as a part of hacktober fest 2020. Can you please assign it to me.

Yes Ofcourse

should it be strictly javascript or can I use c++?

If you want to add in cpp then please go to cpp repo Where there is the same issue raised

i can do this using java script!!
can you pls assign it to me for the hacktober fest 2020???

ok assigning


Allow sorting of peripheral registers by address

Allow the sorting of the peripheral registers and the register bit fields by address in addition to sorting by name. Sorting by name may be confusing if one looks a the MCU reference manual.

We discussed this one and it is a similar issue as #14. If we enabled a sort by address, we would have to represent the tree as a flat list which we don't want to do. In our discussion we did feel that searching by address would make sense as an enhancement and are taking a look at that.

@Spaark84 Sorting by address is now available in the pre-release version of Embedded Tools.

You can switch to the pre-release channel by clicking this icon in the VS Code Marketplace.

Let us know how it works for you! If things look good, this functionality will be included in the next release of the Embedded Tools extension.

@benmcmorran Many thanks for working on that issue.
When enabling sort by address, the peripherals are sorted by address as expected. However, the registers and the fields of the registers remain sorted by name. I would expect them to be sorted by address as well.
Also, I noticed that the sort button disappears when a periheral in the tree is expanded.

@Spaark84 Thanks for the feedback! The sort button shouldn't disappear, is it simply that it moved to the right when a field with a larger name expanded the width of the screen?
As for the sorting, we implemented it with registers and fields accounted for, so there may be an issue with the sorting. Thanks for letting us know and I will look into it.

@gcampbell-msft You are right regarding the button. Maybe it would be better to put the button in the title bar of the peripheral view, e.g.:

@Spaark84 Thanks for checking, we'll get on it and hopefully have a fix in pre-release by tomorrow, or early next week at the latest. 👍


Package request: reindeer

Project description

"Reindeer is a tool which takes Rust Cargo dependencies and generates Buck build rules."
-- https://facebookincubator/reindeer

Related issue: https://github.com/NixOS/nixpkgs/issues/226677
Related PR: https://github.com/NixOS/nixpkgs/pull/232471
Metadata

homepage URL: https://github.com/facebookincubator/reindeer
source URL: https://github.com/facebookincubator/reindeer
license: mit
platforms: linux and darwin (might also work on other platforms)

I should have a PR up for this soon.


Not able to bind custom IP while running Drupal VM with Docker.

I've everything working good when I run following two commands.
docker run -d -p 80:80 -p 443:443 --name=drupalvm --privileged geerlingguy/drupal-vm
docker exec drupalvm install-drupal
Drupal is installed and is available at http://<IP_ADDRESS> and http://localhost. But I was wondering by doing an entry in my hosts file "<IP_ADDRESS>  drupalvm.test" it's not working at all! So, I can't access either http://drupalvm.test OR http://<IP_ADDRESS>
Any ideas?

Hosts file is used to map names to ip address so also that won't per ce bind anything and your docker command does not specify any ip addresses so of course it wont work with <IP_ADDRESS>.
For docker related questions id suggests on dockers help documentation or Stackoverflow especially this particular answer:
https://stackoverflow.com/a/35450145/1566871

How to specify IP in docker command?

@sajalsoni the https://stackoverflow.com/a/35450145/1566871 should answer that

Ok thanks I will check that, thanks for your reply!!

Many thanks @FinBoWa it worked.

could you close the issue also so @geerlingguy doesn't have to :D


Issue when cloning this repo over the extracted Carla tarball

In the instructions mentioned in the README and in the quickstart.sh, we first extract the Carla tarball and then clone this repository over that.
When we do this,
# Download LBC
git init
git remote add origin https://github.com/dianchen96/LearningByCheating.git
git pull origin release-0.9.6

After the last line, git aborts the pull process as there is a conflict in the LICENSE file. (the Carla zip also includes a LICENSE file with the same name)
The current workaround that I use is to add a simple rename command after the tar extraction and before git init like this
mv LICENSE LICENSE_CARLA

After this the git pull seems to work properly.

Thanks for poinint this out! I made the instructions before adding the license file and I forgot to address that. Do you mind creating a PR with your fix above?

Sure, I'll update the readme and quickstart files


Please invite me to the GitHub Community Organization

I want to contribute to open-source projects

@hiiruki  Hey Welcome to openAOD! I have sent an invite to join our organization. We also recommend joining our Discord server : https://discord.gg/wZ83xSJSsN . If you have any questions regarding contributing or anything else feel free to write here or ping me on Discord. Cheers. Have a great day ahead of you!


Support Remix Single Fetch in @shopify/oxygen-remix

What is the location of your example repository?
No response
Which package or tool is having this issue?
Oxygen
What version of that package or tool are you using?
<EMAIL_ADDRESS>What version of Remix are you using?
2.13.1
Steps to Reproduce
Follow the Single Fetch migration docs from Remix and...
Turn on v3_singleFetch in Remix config:
{
  // ...
  v3_singleFetch: true
  // ...
}

Enable the TS behavior for Single Fetch:
declare module "@shopify/remix-oxygen" {
  interface Future {
    v3_singleFetch: true;
  }
}

Update json() usage with data():
import { data, type LoaderFunctionArgs } from '@shopify/remix-oxygen'

export async function loader({}: LoaderFunctionArgs) {
  const headers = new Headers()

  // ...

  return data({ ... }, { headers })
}

// ...

Expected Behavior

@shopify/remix-oxygen exports interface Future from @remix-run/server-runtime so it can be appended to using declare module
@shopify/remix-oxygen exports data() function from @remix-run/server-runtime so it can be used in place of json()

Actual Behavior

@shopify/remix-oxygen does not export interface Future from @remix-run/server-runtime, thus it cannot be appended to using declare module
@shopify/remix-oxygen does not export data() function from @remix-run/server-runtime, thus it cannot be used in place of json()

I forked this repo and made what I believe to be the only needed change to support Single Fetch. See: https://github.com/chinanderm/hydrogen/commit/4cf7a876c92401080bb6b5ea465ccec88db63cc2

You should be able to just import those 2 missing exports from @remix-run/server-runtime. @shopify/remix-oxygen only had to implement createRequestHandler.
We still need some time to turn on single fetch

That's correct - There a couple things we had to do in order to support single fetch.

'defer' is now marked as deprecated in 2.14.0

Can we keep this issue open? It seems that "Support Remix Single Fetch" is not completed yet


Laziness issue between historical metadata and GMM metadata for publishing

When using the publish plugin, it is possible to have the historical metadata and GMM metadata diverge in major ways.

The GMM task generation is fully lazy. It will account for all changes to the component being published up to the point it is run.
The historical metadata generation works by "realizing" the publication, that is extracting information from the component bound to it.

However eager "realization" of the component in the publication can happen if the following methods are called:

for Maven

publication.getArtifacts()

for Ivy

publication.getArtifacts()
publication.getConfigurations()
publication.configurations(Action)

Gradle should detect this and any later changes to the component should be reported as problematic as they will make the two metadata files diverge.

Implemented a reproducer project here.

The workaround is to avoid Maven populate the publication from component before modifying the GMM components (e.g. by skipping variants).


Add Spring Framework

Hi,
I would like to request logos by Spring.
I hope this link would help: https://spring.io/projects
Thank you very much.

Added 67287ea9e3214975c86bc0a61aaf65aeca8e8372

Thank you very much!


Fix localization sample namespace

This address #8219 #8323, @Rick-Anderson seems the original code is copied from the entropy repository which is make the app buggy, to avoid such issues I fixed the namespace and everything now it works as expected

@hishamco 	much appreciated.


TasitAction test suite improvement: cleanup subscriptions that were never assigned (afterEach)

Refs: https://github.com/tasitlabs/TasitSDK/pull/102#discussion_r247694645

@marcelomorgado Is this closed after the PR you mentioned it in?

Yes. Closing.


db tools: Rename "rebuild-" tools and improve digest schemes.

This should have user-facing improvements, and it also cleans up a good chunk of code, either removing code outright or simplifying conditional logic.
There is gonna be a one-time forced rebuilding of the databases to introduce the new digest scheme.

@timabbott If you merge this, please do a little bump on this Zulip thread: https://chat.zulip.org/#narrow/stream/3-backend/topic/rebuilding.20databases/near/860130
As you'll see there, I just try to give folks a bit of a heads up on the one-time changes.

I merged all but the last 3 commits, and posted a few comments.

I have re-pushed.  I think the two new commits address your earlier concerns.
The first commit is brand new to you, and it's all renaming and comments.
Then I dropped the drop-setting commit.
Finally I squashed two intermediate commits that didn't really make the PR easier to read and which were sort of local minima, even though they weren't truly broken.


CI for Node

CI for Node

FAIL test/game.test.js
10
App
11
✕ Contains the compiled JavaScript (5 ms)
12
Game
13
Game
14
✕ Initializes with two players (4 ms)
15
✓ Initializes with an empty board (1 ms)
16
✕ Starts the game with a random player (1 ms)
17
turn
18
✓ Inserts an 'X' into the top center
19
✓ Inserts an 'X' into the top left (5 ms)
20
nextPlayer
21
✕ Sets the current player to be whoever it is not (5 ms)
22
hasWinner
23
✓ Wins if any row is filled
24
✓ Wins if any column is filled
25
✓ Wins if down-left diagonal is filled (1 ms)
26
✓ Wins if up-right diagonal is filled


Is Google source safe to use?

From my understanding of the code, currenlty the Google source does (with the default configuration) an average of the last 4d quotes with an 1h scale.
This behaviour does not match all the other sources which compute a 'last' price (~ real time).
Should we remove Google source to avoid missuse?

Additionnaly Google does not support officially the API... and it has been reported to be unstable.
From pandas-datareader:

As of v0.6.0 Google finance is still functioning for historical price data, although there are frequent reports of failures. Failure is frequently encountered when bulk downloading historical price data.


Add write permission to dokka pipeline

Add write permission to dokka pipeline
Summary
Add write permission to dokka pipeline CI.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add pilot_sds_certificate_errors_total metric for sds

For https://github.com/istio/istio/issues/31779
[X] Policies and Telemetry

@kebe7jun: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
9508e677f413d1ef1f61f6e30b4936e1b136953a
link
/test integ-pilot-multicluster-tests_istio

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. I understand the commands that are listed here.

/retest


[cube] fix DuckDB datasource table format in charts/charts/cube/README.md

Merged. Thanks @WillDaSilva!


UDF hangs with no exception / warning

Description of the bug
When running UDF for parsing, candidate extraction, or labeling, the process will hang for an indefinite amount of time. No exception or warning is generated.
Hangs aren't easily reproducible: UDF will hang on a file for one run, then after a full reset UDF will parse that file without a problem, only to hang on some other file.
To Reproduce
Using very similar code to what's found in the fonduer-tutorials max_storage_temp_tuturial, run the parser UDF on a set of less-trivial PDFs (multiple pages).
Expected behavior
Informative / actionable error reporting so the developer can take further action to diagnose the problem.
Decoupling operations from postgresql and moving towards in-memory operations as mentioned in #137 might address this concern.
Error Logs/Screenshots
N/A, only a frozen progress bar.
Environment (please complete the following information)

OS: MacOS 10.15
PostgreSQL Version: 12.1
Poppler Utils Version: 0.71.0-5
Fonduer Version: 0.8.3

Additional context
I'm using the jupyter/postgres docker containers provided for fonduer-tutorials.

Some updates:

It appears hangs do happen for specific HTML files. Switching parallelism=1 made it clear that specific files trigger this behavior. I can provide example files if necessary.
This behavior occurs when using MacOS, but cannot be reproduced using a Ubuntu VM with the fonduer-tutorials docker image.

Hi @robbieculkin, yes please! Can you include examples of files that cause the issue? Thanks for this additional info.

Sure, here's a ZIP that includes the PDF & HTML (Adobe Acrobat converted).
huang2016.zip
Thanks for your help, @lukehsiao

Hi @robbieculkin,
I test your data with the latest master branch and it works properly (see the figure below).

Can you try the latest master branch and let us know if it's still a problem.
Thanks,
Sen

Hmm, the error remains for MacOS. I've switched to my Ubuntu machine for Fonduer-related development so I can close this. If others have issues, maybe they can reopen?


Add a new declaration (.d.ts)

Hi,
I'd like to use your leaflet plugin, after using npm install tripkit-leaflet did not show any problem with install. But when I use require("tripkit-leaflet") it shows me the following message:
Could not find a declaration file for module 'tripkit-leaflet'. 'c:/Users/Elias/Documents/GitHub/it-project-19-20-h/node_modules/tripkit-leaflet/dist/TripgoRouting.js' implicitly has an 'any' type.
Try npm install @types/tripkit-leaflet if it exists or add a new declaration (.d.ts) file containing declare module 'tripkit-leaflet';ts(7016)
Does this have an impact on my website?
Do you also have a tutorials? Since I'm a junior developer, it would be a huge help.
Sincerly yours,
Elias Samadi

Hi Elias,
I'll try to reproduce the error you mention to help you asap. Anyway to don't block your work, you could download the code and try the integration in the way we explain here


German Language support

I see there's mentioning in doc files of a few German voices, but could not find how to initiate them..
setting language to DE or speaker_id to DE causes errors with files not being found, any suggestion how to get german working?Thanks!
P.S. speed of generation for EN-BR is amazing!
`German
sanfte Männerstimme
sanfte Frauenstimme
unreife Mädchenstimme`

This might help you:
-> https://github.com/myshell-ai/MeloTTS/blob/main/docs/training.md
You have to change the configs accordingly.

@RedBluePrinter > Thanks for providing the links.  For training of a new language, how much voice material is required?


Get Ready to Develop on Your Local Machine

Tutorial URL: https://developers.sap.com/tutorials/cp-cf-sapui5-local-setup.html
Please specify the step you are referring to: Step 4
I think the provided ansers are wrong. According to my console, the supported commands of mbt are (this is the output of mbt --help):
Available Commands:
assemble    Assembles MTA Archive
build       Execute MTA project build
gen         Generation commands
help        Help about any command
init        generates Makefile
validate    MBT validation
Through trial and error, I found out the accepted ansers are "assemble, init". When I also place a checkmark at "build", then my answer is rejected.

Good catch @mjhcorporate
Thanks for your feedback!


[DNM] service/kv: Add end_key support for raw_scan (#3847)

What have you changed? (mandatory)
This PR cherrypicks endkey support for raw scan to release-2.1. This is because pingcap/tidb#10152 needs to be cherry-picked to release-2.1, and it needs reverse querying functionality of region cache, which is provided in pingcap/tidb#8352 , and pingcap/tidb#8352 implements raw reverse scan, and it's implementation made use of the later-merged endkey of raw_scan. For completeness concern, this PR is cherry-picked.
Requires https://github.com/tikv/tikv/pull/5148
What are the type of the changes? (mandatory)

New feature (change which adds functionality)

How has this PR been tested? (mandatory)
By CI
Does this PR affect documentation (docs) or release note? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No
Refer to a related PR or issue link (optional)
Requires: https://github.com/tikv/tikv/pull/5148

This is not needed. Close it.


paper-toggle-button with on-tap eventhandler is not handled backward compatible

[ x] Chrome
[ x] Firefox
[ x] Edge
[ x] Safari 9
[ x] Safari 8
[x ] IE 11

Versions

Polymer: v2.4.0, v2.5.0
webcomponents: v1.1.0

Is this new behavior intended?

Thanks for the issue.  Please see CONTRIBUTING.md for guidelines on filing issues.  Please fill out as many fields in the issue template as possible, and be sure to include a live reproduction so that the team can debug or understand your issue.  You can start from this jsBin template as a starting point and link it in the issue template under "Live Demo".
Feel free to file a new issue following these instructions.  Thank you!


Fixoutdated

Updates to use more recent versions of build tools, the old versions can no longer build canvas-lms. Also fixes for docker image bloat/slowdown caused by using chown -R

Thanks for the PR. I wasn't paying attention during the build but it finished OK. When I run the resulting image I get an error I don't know enough about rails to interpret.

I think I got that as well but that it resolved itself after restarting the container. I'm not sure if there's a better way to fix it.

whoa that actually worked for me too. weird... i mean, working in some capacity is better than not working at all, so I'll probably mrege this. Would you mind adding a big, attention-getting note to the readme that mentions the need for this container restart?

I've added a healthcheck that notices when the landing page is returning a 500 error and restarts the canvas_web service in supervisord


Failing to integrate or create new Cake

Thanks for putting this together!
We're running into issues with both creating a new Cake project and integrating an existing project with Cake. We get this message when integrating or opening a Xcode (Beta 4):

Possibly related. When trying to build, we get this error:
The operation couldn’t be completed. (E.notCake(Path.Path(string: "{ path to the project }")))
Maybe we're doing something wrong? Tried on multiple machines and tinkered with some different build settings. No luck:/

Can you tell me more about the project? Is it a regular project? Or a workspace?

Both. We started trying with our workspace. When that didn't work I make a new regular project and attempted integrating. Then tried creating a project through cake. All with the same results.

Funny “New Cake” doesn't work either.
I just tested both locally and it worked FYI. Must be something about your configuration I didn't expect.

Cake version?

1.0.3 with Xcode 10.2 beta 4
Every time Cake runs I get the NSCocoaErrorDomain.3587 error dialog
Are there any other steps or any environment tools/configurations I might be missing?
Stepping through the Cake code now. Haven't figured out anything yet

Any info here? https://github.com/mxcl/Cake/issues/49
We thought that one fixed, but could be that it's not.

Here’s the main notCake throw (there's one other, but it's the same cause)
https://github.com/mxcl/Cake/blob/4f61b7567d45ab865bf4e5a743c75a1026d4dfdc/Sources/Model/Base/XcodeProject/XcakeProject/Processor/Kitchen/CakeProject.swift#L40
Seems unlikely there'd no Cakefile though. Could be other causes though for that throw.

Which comes from here:
https://github.com/mxcl/Cake/blob/4f61b7567d45ab865bf4e5a743c75a1026d4dfdc/Sources/Model/Base/XcodeProject/XcakeProject/Processor/Cakefile.swift#L34

When first running Cake or creating a new project. This is the first place I hit E.notCake.
https://github.com/mxcl/Cake/blob/43624666bbf41e9ac4bcc4c6d79af3c08b2f5981/Sources/Model/Base/XcodeProject/XcakeProject/Processor/Kitchen/Kitchen.swift#L84-L87
I hit the E.notCake catch and the NSCocoaErrorDomain.3587 dialog kicks off.
paths from diff.added = [
"/Users/Developer/Cake/Cake.xcodeproj"
"/Users/Desktop/Bread Pudding/Bread Pudding.xcodeproj"
]
xcodePath = "/Applications/Xcode-beta.app"
Toolkit
  ▿ cakePath : Path
    - string : "/Users/Developer/Cake/DerivedData/Cake/Build/Products/Debug/Cake.app"
  ▿ xcodePath : Path
    - string : "/Applications/Xcode-beta.app"
  ▿ cakeVersion : 1.0.3-debug
    - major : 1
    - minor : 0
    - patch : 3
    ▿ prereleaseIdentifiers : 1 element
      - 0 : "debug"
    - buildMetadataIdentifiers : 0 elements
  - xcodeProductBuildVersion : "10P107d"

Nothing stands out to me as to why this is catching on line 84.
Later on, when I build the Bread Pudding project (second path above). I get the E.notCake error from:
https://github.com/mxcl/Cake/blob/43624666bbf41e9ac4bcc4c6d79af3c08b2f5981/Sources/Model/Base/XcodeProject/XcakeProject/Processor/Kitchen/Kitchen.swift#L45-L51
because cakes is empty.

Hello, I haven't looked into this well but when i also selected command line tools as beta the errors went away.
sudo xcode-select -s /Applications/Xcode-beta.app

I tried that pretty early on, but got the same results. In my test above you can see the xcodePath variable is pointing to that location.
xcode-select -p prints /Applications/Xcode-beta.app/Contents/Developer

We were able to get pass this problem with these troubleshooting steps. Some are in the guide, but not all.
swift -v swiftc -v should both print version 5.0
xcode-select -p prints /Applications/Xcode-beta.app/Contents/Developer
If not, use sudo xcode-select -s /Applications/Xcode-beta.appor wherever your Xcode 10.2+ is located.
Make sure you clear derived data. The Cake menubar item rm -rf 'Derived Data' closes Xcode and clears it. Or at least it should. You can manually delete it at /Users/{user}/Library/Developer/Xcode/DerivedData if you write at the default location, which leads me to my next point.
I found my derived data was being written to a relative path instead of the Xcode default path. Which I believe was part of my problem. Check this at option Preferences > Locations > Derived Data. I changed this setting back to Default.
At this point I was able to get the Cake to work from Xcode but not from the release version 1.0.3.
I uninstalled Cake and deleted the Cake directory found at /Users/{user}/Library/Developer/Cake
After reinstalling, opening cake, deleting Xcode derived data one more time. Things seemed to work.
Not entirely sure what happened to cause this. I'm underwhelmed with this solution. When all else fails, turn it off and back on again.

Reopening. I think I probably hardcode the derives data path someplace. Not that this should matter.

Same problem here:
I installed Cake 1.0.3, created an iOS "New Cake" project, and got this same error as @kaydenthomson  "The operation couldn’t be completed. (NSCocoaErrorDomain.3587)".
Also when I build the application I also get the same error "error: The operation couldn’t be completed. (E.notCake(Path.Path(string: "/Users/mac/Desktop/Bûche de Noël")))
notice: {
}"
I removed the derived data, no effect
Tried with Xcode 10.2 or 10.2 beta, no effect
Tried to select the other Xcode version with xcode-select, no effect
I switched to Swift 5.0 with SwiftEnv, and created again a new Cake projet, no effect
Any other suggestion ?

I also tried to remove the Cake directory (/Users/{user}/Library/Developer/Cake) and reinstall it, also no effect

Sorry about these problems. I cannot reproduce locally.
The workaround is ensuring the Xcode 10.2 command line tools are installed, perhaps, does that help?

Don't worry and thanks for your answer and for your work @mxcl :)
I don't have access right now but the command line tools should probably be installed. I will have more time to investigate tomorrow


Pre-commit and linting

What

Add pre-commit hooks including golang linting
Update Contributor documentation

Why
By using pre-commit hooks we can easily and automatically catch simple issues. They run automatically on every git commit and we can validate that they ran by running them in the CI pipeline and failing if they report any anomalies.
Pay particular attention to the changes in CONTRIBUTING.md so the new prerequisites are well understood.
This first PR adds the .pre-commit-config file including the following hooks:

check-added-large-files: Fails if any files over 500kb are added. Is great for keeping repos small. If a bigger file needs to be added you can add it by passing the filename to an exceptions list.
check-merge-conflict: Ensures that there are no orphaned Git merge conflict strings in the codebase. These can appear when a git merge is done incorrectly.
detect-aws-credentials: Detects accidentally added AWS credentials before they make it into the Git history. Huzzah!
detect-private-key: Detects private SSH keys and private certificates before they get accidentally added to the git history
end-of-file-fixer: Ensures that every file in the codebase ends with a new line, which is a POSIX standard.
fix-byte-order-marker: Strips BOMs which are not recommended when using UTF-8
trailing-whitespace: Strips trailing whitespace (except for markdown since it has special meaning)
fix-smartquotes: Automatically changes smartquotes to dumb quotes. Robots hate smartquotes.
go-fmt: Automatically runs go fmt on the codebase
golangci-lint: Automatically runs a linter on the golang codebase. This one is commented out for now but it works and can be turned on very easily once we fix the linting errors (there aren't that many of them)

In the documentation I have said that the CI pipeline requires these checks, but that won't be true at the time of merging this PR. That will come later.
Fixes #74
Related to #75

golangci-lint results:

This doesn't add any checks in the CI pipeline right? Someone could try to be sneaky (or accidentally) commit with the --no-verify flag and push non-conforming code.

/test all


Add support for pnpm

Context
This PR adds support for pnpm workspaces. We're switching to pnpm to simplify our integration with Bazel. See the tracking issue for more context:

https://github.com/sourcegraph/sourcegraph/issues/44909

Test plan

Checkout to vb/pnpm-test-3 in the Sourcegraph monorepo.
rm -rf node_modules ./client/*/node_modules && pnpm install
DIR=/path/to/scip-typescript; "$DIR/node_modules/.bin/ts-node" "$DIR/src/main.ts" index --pnpm-workspaces

Looks like the CI / test (18.x) job fails in other PRs not related to the package logic too.

We can add an integration test by adding a pnpm project under snapshots/input and make sure to enable the new --pnpm-workspaces flag here https://github.com/sourcegraph/scip-typescript/blob/4834ff7eef11c0b664da8e54bb2741bcaab91b48/src/main.test.ts#L57

@olafurpg, added https://github.com/sourcegraph/scip-typescript/pull/219/commits/56506add69930f153d7f51787963fc0902aef330 👍

And fixed the issue with the CI / test (18.x) job.


Various usability improvements

Removed debugging log bloat caused by DobbyWorkQueue
Allow underscores in container names
Fixed try/catch misuse in DobbyTool
Improved error handling for DobbyTool start with dodgy paths

LGTM


Remove the JERRY_GET_CURRENT_CONTEXT macro

It was an unused and undocumented macro that enabled diverting the
jerry_port_get_current_context calls, an approach that was not
used anywhere else in the Port API.
JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

It's been a week now and no one spoke up. Can we land this?


Create Role & RoleBinding in the job namespace

Will go ahead and merge, but any feedback appreciated!


Cannot apply rule validation against raw identifiers

Hi, thanks for this beautiful crate.
I found a bug I htink in the way the crate interprets attributes. The following code produces the following error:
#[derive(Validate)]
#[rule(sample_rule(r#type)]
struct MyStruct {
  pub r#type: bool
}

fn sample_rule(r#type: &bool) -> Result<(), serde_valid::validation::Error> {
  Ok(())
}

#[derive(Debug, PartialEq, Clone, Validate, Deserialize)]
   |                              ^^^^^^^^
   |
   = help: message: `"r#type"` is not a valid identifier

I know I can check the attribute r#type directly with #[validate(...)] but my use case implies multiples attributes.

The current workaround is to use serde rename.
use serde_json::json;
use serde_valid::json::FromJsonValue;
use serde_valid::Validate;

#[derive(serde::Deserialize, Validate)]
#[rule(sample_rule(_type))]
struct MyStruct {
    #[serde(rename = "type")]
    pub _type: bool,
}

fn sample_rule(r#type: &bool) -> Result<(), serde_valid::validation::Error> {
    Ok(())
}

#[test]
fn test() -> Result<(), Box<dyn std::error::Error>> {
    let my_struct = MyStruct { _type: true };
    assert!(my_struct.validate().is_ok());

    assert_eq!(
        MyStruct::from_json_value(json!({"type": true}))?._type,
        my_struct._type
    );

    Ok(())
}

The field name "type" seems to be an Issue proposed in many crates. A special workaround is provided.
https://github.com/yassun4dev/serde_valid/pull/15

@Tenshock
I released v0.16.3.

Hello, thanks for the fast update!
I checked the fix and when I run your test I have the following error:
error: expected expression, found keyword `type`
    |
157 |     #[rule(sample_rule(type))]
    |                        ^^^^ expected expression

error: proc-macro derive produced unparsable tokens
    |
153 |     #[derive(Validate)]
    |              ^^^^^^^^

error: could not compile `test_app` (lib test) due to 2 previous errors

my version: rustc 1.71.0 (8ede3aae2 2023-07-12)
Did I miss something?hhhhhhhhhhhhhhhh


downloadProgress block in downloadTaskWithRequest is not called under AFNetworking 3.1.0 When downloading a gzip file

NSURLSessionConfiguration *configuration = [NSURLSessionConfiguration defaultSessionConfiguration];
AFURLSessionManager *manager = [[AFURLSessionManager alloc] initWithSessionConfiguration:configuration];

NSURL *URL = [NSURL URLWithString:@"https://s3.amazonaws.com/awdtest/fullzip.pdf"];
NSURLRequest *request1 = [NSURLRequest requestWithURL:URL];

NSURLSessionDownloadTask *downloadTask1 = [manager downloadTaskWithRequest:request1 progress:^(NSProgress * _Nonnull downloadProgress)
{
    NSLog(@"Progress: %f", downloadProgress.fractionCompleted);
} destination:^NSURL *(NSURL *targetPath, NSURLResponse *response)
{
    NSURL *documentsDirectoryURL = [[NSFileManager defaultManager] URLForDirectory:NSDocumentDirectory inDomain:NSUserDomainMask appropriateForURL:nil create:NO error:nil];
    return [documentsDirectoryURL URLByAppendingPathComponent:[response suggestedFilename]];
} completionHandler:^(NSURLResponse *response, NSURL *filePath, NSError *error)
{
    NSLog(@"File downloaded to: %@", filePath);
}];
[downloadTask1 resume];

Response headers:
HTTP/1.1 200 OK
x-amz-id-2: CW06YcgIycOHZQy8bCJrT3aNfhatM9pty1mOjgYHumjCxRmNAQ+jhJHRQwl7mDIaQeTHI0fyrnU=
x-amz-request-id: 105E010DECC91897
Date: Sat, 14 May 2016 09:26:38 GMT
Content-Encoding: gzip
Last-Modified: Sat, 14 May 2016 09:21:29 GMT
ETag: "88bbe0b318bf11dd56a31176d3384e78"
Accept-Ranges: bytes
Content-Type: application/pdf
Content-Length: 1243325
Server: AmazonS3

it works if I use this file which is not gzipped: https://s3.amazonaws.com/awdtest/full.pdf
Response headers:
HTTP/1.1 200 OK
x-amz-id-2: gFOVfhheaMdeJOBb+7H8oaXjLLeOqlQl616XnYx6C2Gj7PBVLKZ9kMIN2fJOrGBcSgQ/7nbQOc0=
x-amz-request-id: 26669D9B576E300A
Date: Sat, 14 May 2016 09:54:58 GMT
Last-Modified: Fri, 16 May 2014 05:42:35 GMT
ETag: "6f16d7e09023ce8b50fd67abba7825c4"
Accept-Ranges: bytes
Content-Type: application/pdf
Content-Length: 1411131
Server: AmazonS3

Any update on this?

i am also facing this issue.

i am also facing similar issue with .txt and .mp3 files.

now，me too

If you want to get an approximate progress you can add the following code to get the totalUnitCount from the Content-Length in the response header
AFURLSessionManager:

(void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(NSDictionary<NSString *,id> *)change context:(void *)context
    .....

    else if ([keyPath isEqualToString:NSStringFromSelector(@selector(countOfBytesExpectedToReceive))]) {

      self.downloadProgress.totalUnitCount = [change[NSKeyValueChangeNewKey] longLongValue];

      if (self.downloadProgress.totalUnitCount == NSURLSessionTransferSizeUnknown) {

          NSURLResponse * response = [((NSURLSessionDownloadTask *)object) response];

          if ([response isKindOfClass:[NSHTTPURLResponse class]]) {

              NSHTTPURLResponse *httpResponse = (NSHTTPURLResponse*)response;

              if ([httpResponse respondsToSelector:@selector(allHeaderFields)]) {

                  NSDictionary *dictionary = [httpResponse allHeaderFields];

                  if ([dictionary valueForKey:@"Content-Length"] != [NSNull null]) {

                      self.downloadProgress.totalUnitCount = [[dictionary valueForKey:@"Content-Length"] longLongValue];

                  }

              }

          }

      }

  }

  .....

me too~

i have solved it, because the file is too small,you  can download a large file,then the block will run.

When you download some binary file like zip, build the request like this:
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:url]; [request addValue:@"" forHTTPHeaderField:@"Accept-Encoding"];
Setting blank for "Accept-Encoding" tells the the server that the client is not ready to accept the response in any specific format. So server simply bypasses "gzip" ping the response stream and returns the raw bytes. NSURLSession delegate expected length will not return NSURLSessionTransferSizeUnknown any more.


fix(region,host): slvm disk create use guest host lvm disk create from backup

What this PR does / why we need it:

fix(host): lvm disk create from backup
fix(region): slvm disk create use guest host

Does this PR need to be backport to the previous release branch?:
release/3.9

/lgtm


FileType detection while uploading is based upon first dot?

Hi,
We tried to migrate from the ruby CLI to the java one, and stubbled upon a potential falsy filetype detection
Scenario:

Setting up crowdin (2.0-13) ...
File 'messages.po'-\| - OK
File 'users-auth-create-reset.text.txt'/- - ERROR
code : 7
message : Invalid file extension. Allowed are: *.txt

More details : https://circleci.com/gh/CoderDojo/cp-zen-platform/1025
Crowdin.yaml : https://github.com/CoderDojo/cp-translations/blob/master/crowdin.yaml
cc @danielBrierton

The configuration itself seems to be ok:
  -
    source: '/email-templates/auth-create-reset-en_US/users-auth-create-reset.text.ejs'
    dest: '/users-auth-create-reset.text.txt'
    translation: '/email-templates/auth-create-reset-%locale_with_underscore%/%file_name%.ejs'
    type: 'txt'

Checking right away

@Wardormeur @danielBrierton
Please add the following parameter on the top of your configuration:
preserve_hierarchy: true

Hello @Wardormeur, @DanielBrierton!
This problem will be fixed in Crowdin CLI 2.0.15  (#41)
Thanks for report!


Can I rename the file when click "save and exit"?

Hi, it's a nice plugin for obsidian to draw some flow chart.
I found the saved files name are Diagram {num}.svg and Diagram {num}.svg.xml, and I want to rename them when clicking "save and exit" rather then renaming both of them in file explorer. Maybe a configuration option in setting tab.

Hi, thanks.
I understand the request, and will look into it as a possible feature for a next version.


[Bug] [Authz] Failed to build and test Authz plugin with Ranger below 1.0 with failures of data masking

Code of Conduct

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

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
✅  OK to build and test with Ranger 1.0+(1.1.0/1.2.0/2.1.0/2.3.0)
❌  Failed to build and test Authz plugin with either 0.6.0 or 0.7.0.
build/mvn clean package -pl :kyuubi-spark-authz_2.12 -Pspark-3.3 -Dranger.version=0.6.0
build/mvn clean package -pl :kyuubi-spark-authz_2.12 -Pspark-3.3 -Dranger.version=0.7.0
Total number of tests run: 455
Suites: completed 18, aborted 0
Tests: succeeded 307, failed 148, canceled 37, ignored 0, pending 0
*** 148 TESTS FAILED ***
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------

Affects Version(s)
1.7.0/master
Kyuubi Server Log Output
No response
Kyuubi Engine Log Output
- auth: databases *** FAILED ***
  Expected exception org.apache.kyuubi.plugin.spark.authz.AccessControlException to be thrown, but no exception was thrown (RangerSparkExtensionSuite.scala:155)
- auth: tables *** FAILED ***
  Expected exception org.apache.kyuubi.plugin.spark.authz.AccessControlException to be thrown, but no exception was thrown (RangerSparkExtensionSuite.scala:177)
- auth: functions *** FAILED ***
  Expected exception org.apache.kyuubi.plugin.spark.authz.AccessControlException to be thrown, but no exception was thrown (RangerSparkExtensionSuite.scala:215)
- row level filter *** FAILED ***
  SELECT value FROM default.src Array([1], [2], [3]) did not equal List([1]) (RangerSparkExtensionSuite.scala:248)
- [KYUUBI #3581]: row level filter on permanent view *** FAILED ***
  SELECT value FROM default.perm_view Array([1], [2], [3]) did not equal List([1]) (RangerSparkExtensionSuite.scala:291)
- show tables *** FAILED ***
  0 did not equal 2 (RangerSparkExtensionSuite.scala:310)
- show databases *** FAILED ***
  Array() had length 0 instead of expected length 2 (RangerSparkExtensionSuite.scala:321)
- show functions *** FAILED ***
  Array() had length 0 instead of expected length 1 (RangerSparkExtensionSuite.scala:340)
- show columns *** FAILED ***
  0 did not equal 2 (RangerSparkExtensionSuite.scala:367)
- show table extended *** FAILED ***
  0 did not equal 5 (RangerSparkExtensionSuite.scala:397)

- [KYUUBI #3426] Drop temp view should be skipped permission check *** FAILED ***
  Array() had length 0 instead of expected length 2 (RangerSparkExtensionSuite.scala:453)
- [KYUUBI #3428] AlterViewAsCommand should be skipped permission check *** FAILED ***
  java.lang.reflect.UndeclaredThrowableException:
  at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1894)
  at org.apache.kyuubi.plugin.spark.authz.SparkSessionProvider.doAs(SparkSessionProvider.scala:76)
  at org.apache.kyuubi.plugin.spark.authz.SparkSessionProvider.doAs$(SparkSessionProvider.scala:74)
  at org.apache.kyuubi.plugin.spark.authz.ranger.RangerSparkExtensionSuite.doAs(RangerSparkExtensionSuite.scala:37)
  at org.apache.kyuubi.plugin.spark.authz.ranger.RangerSparkExtensionSuite.$anonfun$new$138(RangerSparkExtensionSuite.scala:466)
  at org.scalatest.OutcomeOf.outcomeOf(OutcomeOf.scala:85)
  at org.scalatest.OutcomeOf.outcomeOf$(OutcomeOf.scala:83)
  at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
  at org.scalatest.Transformer.apply(Transformer.scala:22)
  at org.scalatest.Transformer.apply(Transformer.scala:20)
  ...
  Cause: org.apache.spark.sql.catalyst.analysis.TempTableAlreadyExistsException: Temporary view 'temp_view' already exists
  at org.apache.spark.sql.catalyst.catalog.SessionCatalog.createTempView(SessionCatalog.scala:602)
  at org.apache.spark.sql.execution.command.CreateViewCommand.run(views.scala:124)
  at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult$lzycompute(commands.scala:75)
  at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult(commands.scala:73)
  at org.apache.spark.sql.execution.command.ExecutedCommandExec.executeCollect(commands.scala:84)
  at org.apache.spark.sql.execution.QueryExecution$$anonfun$eagerlyExecuteCommands$1.$anonfun$applyOrElse$1(QueryExecution.scala:98)
  at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$6(SQLExecution.scala:109)
  at org.apache.spark.sql.execution.SQLExecution$.withSQLConfPropagated(SQLExecution.scala:169)
  at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$1(SQLExecution.scala:95)
  at org.apache.spark.sql.SparkSession.withActive(SparkSession.scala:779)
  ...
- [KYUUBI #3343] pass temporary view creation *** FAILED ***
  scala.util.Try.apply[org.apache.spark.sql.DataFrame](RangerSparkExtensionSuite.this.sql.apply(scala.StringContext.apply("CREATE TEMPORARY VIEW ", " AS select * from values(1)").s(tempView))).isSuccess was false (RangerSparkExtensionSuite.scala:495)
== Physical Plan ==
CommandResult <empty>
   +- CreateNamespace org.apache.spark.sql.execution.datasources.v2.jdbc.JDBCTableCatalog@33770455, [ns2], true

- [KYUUBI #3424] CREATE DATABASE *** FAILED ***
  Expected exception org.apache.kyuubi.plugin.spark.authz.AccessControlException to be thrown, but no exception was thrown (V2JdbcTableCatalogRangerSparkExtensionSuite.scala:84)
- [KYUUBI #3424] DROP DATABASE *** FAILED ***
  Expected exception org.apache.kyuubi.plugin.spark.authz.AccessControlException to be thrown, but java.lang.reflect.UndeclaredThrowableException was thrown (V2JdbcTableCatalogRangerSparkExtensionSuite.scala:94)

### Kyuubi Server Configurations

_No response_

### Kyuubi Engine Configurations

_No response_

### Additional context

_No response_

### Are you willing to submit PR?

- [ ] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to fix.
- [X] No. I cannot submit a PR at this time.

cc @yaooqinn


Oncoplot

Hi,
Thank you so much for an amazing tool. I am trying to make an oncoplot using the following code:
oncoplot(maf = laml, top = 10, clinicalFeatures = "Diagnose", removeNonMutated = TRUE, sortByAnnotation = TRUE, annotationDat = "clinical.data")
However, I get this error
Error in split.default(anno, as.factor(as.character(anno[, 1]))) :
first argument must be a vector
What does this mean? I have no problems in making a simple oncoplot without clinical data, such as this: oncoplot(maf = laml, top = 10) ..
Thank you in advance!

Hello,
First, Thank you for using the tool, I am glad that you find it useful.
Did you include clinical data (sample annotations) while reading and generating  laml object ?
Why is clinical.data is in quotes ? Is it a file name ?
Also could you post your sessionInfo ?


[FEATURE]  langchain4j-spring support qianfan model and service

Rapid integration qianfan model implements agent development through langchain4j's spring boot starter

There is SB started for Qianfan: https://github.com/langchain4j/langchain4j-spring/tree/main/langchain4j-qianfan-spring-boot-starter


Modifying filter on input from the set of icons selected in select.

Just by removing the "fixed" icon set in the input, and replacing it with the selected icon set in select.

Thanks @patrickmonteiro :heart:


bookmarks don't work

Description
Suddenly my favorites have been deleted and now I can't add new favorites, it just doesn't work.
What happens is that the bookmark is added to the windows system not to Files.
Files adds the bookmark to windows, not itself.

Bookmarks don't work
https://user-images.githubusercontent.com/19652075/226177663-97767f61-ef4a-4705-8f50-eed1cb72ccf0.mp4
Steps To Reproduce
Add a bookmark
Check to see which Files has been added to the bookmark to windows.
Requirements

eg. Fix crash that occurs when...
eg. Correct action happens when clicking...

Files Version
<IP_ADDRESS>
Windows Version
10 64
Log file
...

Can you please upload (Do not copy and paste content) the debug.log file directly to GitHub (email attachments don't work) . You can get it in Files settings > About > Open log folder.

debug.log
Ok this is

Can you see if this is an issue in the preview version? https://files.community/download

Same as version <IP_ADDRESS>, favorites don't work.
debug.log

I've seen this issue happen when there are drives pinned to favorites in File Explorer.


multiple contracts for generateOpenApi

Hello,
I am evaluating ts-rest as a backend for my next project and I could not figure out how can I add multiple contracts to openAPI swagger UI. Am I missing something but first param of generateOpenApi does not except an array of contracts.
	// messages contract
	createExpressEndpoints(messagesContracts, messagesRouter, app)
	// posts contract
	createExpressEndpoints(postsContracts, postsRouter, app, { jsonQuery: true })

	const openApiSchema = generateOpenApi([messagesContracts, postsContracts], {
		info: { title: 'API Docs', version: '0.1' }
	})

how can I combine contracts for generateOpenApi ?

Another problem I see, if I describe a post contracts  as Typescript instead of zod, I can't get Swagger UI to display Request body params

@kurdin I think this is all intended behaviour!
As you've found thats the correct way to combine routers, you can also spread them together if you don't want to introduce another layer to the object.
Given that c.type<> doesn't exist at runtime it's impossible for us to generate OpenAPI docs from the types sadly - so that's why we require Zod here

@oliverbutler sorry for reopening this issue.
First of all, thank you for the ts-rest project. I think you found a very elegant way to describe contracts and routes with validation, and I believe this project deserves more visibility in the similar field.

Given that c.type<> doesn't exist at runtime it's impossible for us to generate OpenAPI docs from the types sadly - so that's why we require Zod here

Ok, so does this mean that if I use Typescript in contracts for validation instead of Zod, it wont work as well on runtime ? If so, what is the point to use typescript like this in the example you provided.
body: c.type<{
  title: string
  description: string | null
  content: string | null
  published: boolean
  tags: string[]
}>()

Thank you again.


3D not working

3D is now working.
transform.Rotate had to have a "Space.World" flag added to it. This fixed everything.

Using Quaternions to do the rotation was an even better solution all round.


Release Google.Cloud.PubSub.V1 version 3.14.0

Changes in this release:
Bug fixes

Fix possible race condition when restarting the stream (commit 4410bb1)

New features

Add service_account_email for export subscriptions (commit f73410e)
Make StreamingPull fail after 100 consecutive failures (commit d9d658b)
Improve Pub/Sub streaming pull retries (commit 8537ed4)

The last two of these features can cause user-visible changes. In particular:

Persistently-failing streaming pulls will eventually cause a SubscriberClient to fail
Authentication-based failures (e.g. using an expired service account) will now cause a SubscriberClient to fail
SubscriberClient will retry immediately after a streaming pull stream is terminated server-side, if the pull was active for 45 seconds or more before being terminated. This should improve responsiveness in some situations.

All of these changes should be "net positive", but if they cause any issues, please file a bug.

Created release for Google.Cloud.PubSub.V1-3.14.0

This didn't failed: https://www.nuget.org/packages/Google.Cloud.PubSub.V1/3.14.0


2.0: Remove exposed references to Endpoint and Protocol in favour of EndpointName

Describe the Change
On the following functions:

Add-PodeRoute
Add-PodeStaticRoute
Get-PodeRoute
Get-PodeStaticRoute
Remove-PodeRoute
Remove-PodeStaticRoute

There are Endpoint, Protocol and EndpointName parameters. The EndpointName is the successor to the former two, which lets you directly reference endpoint(s) from Add-PodeEndpoint to bind Routes against.
I also noticed that Move-PodeResponseUrl could have a new Endpoint parameter as well.
Note: with the removal of Endpoint, I'm quite tempted to rename EndpointName to just Endpoint (This will then fall inline with the new -Auth parameter I have ideas for).
With the rename, there will still be internal references for Endpoint and Protocol - it could be worth renaming Endpoint in these to Address to avoid confusion.

Furthermore, Name on Add-PodeEndpoint will remain optional.

Decided to leave -EndpointName as is, and not rename it to -Endpoint - it would have gotten confusing internally.


[Feature] Alternative execution model for Playwright event loop

This is somewhere between a feature request and a bug. My production system recently threw a StackOverflowException.
Upon further inspection I've determined that the following is happening:

I register BrowserContext.onRequestFailed() and onRequestFinished() callbacks in order to gather page telemetry for the requests being made. In that callback, I invoke Request.response() to gather some additional details (response content length as well as some encoding-related response headers).
I Page.navigate() to a URL.
The page at that URL makes a considerable number of out bound requests (>100).
I invoke Page.waitForFunction() to wait for a particular condition to be met to indicate that my process has completed.
Playwright invokes my callback for the first request. The callback invokes Request.response(), which puts me back in the Playwright event loop (ChannelOwner.runUntil()).
Playwright invokes my callback for the second request. The callback invokes Request.response(), which puts me back in the Playwright event loop.
…and on and on and on.
Execution finally reaches a stack depth that the JVM is unhappy with and a StackOverflowException is thrown.

The issue here is that the stack can't begin to unwind until the most recent Request.response() is able to return, even if every prior Request.response() is available.
It may be worth considering a more reactive model using Future/CompletableFuture and a dedicated event processing thread in Playwright to help avoid generating stacks of such depth.
For what it's worth, this would also make it much clearer to callers what data they're fetching that can be returned locally and what requires a full round trip back to the Playwright/browser process. That's often unclear, potentially leading to people making calls that are far more expensive and less reliable than they realize.

It may be worth considering a more reactive model using Future/CompletableFuture and a dedicated event processing thread in Playwright to help avoid generating stacks of such depth.

We considered that and decided to proceed with the current approach as all the alternative seemed way less ergonomical. It is true that with the current message dispatch model you can end up with stack overflow when you perform API calls that may lead to interaction with the driver. In all of the cases we've seen so far there were workarounds. In general we recommend using waitFor* methods instead of on* listeners as they are not susceptible to this problem. E.g. prefer waitForRequest to onRequest.

For what it's worth, this would also make it much clearer to callers what data they're fetching that can be returned locally and what requires a full round trip back to the Playwright/browser process. That's often unclear, potentially leading to people making calls that are far more expensive and less reliable than they realize.

I'd say each of the action methods in Playwright API can may require interaction with the driver, so it'd be save to assume that any such method always talks to the driver. Most of the time (except for the cases with recursive event dispatch) you should not worry about it.

In general we recommend using waitFor* methods instead of on* listeners as they are not susceptible to this problem. E.g. prefer waitForRequest to onRequest.

This very naively assumes that:

There's only one thing the application is ever going to want to wait for
The application knows when to wait for all of the things it wants to wait for

In our application, we add onRequestFailed/Finished callbacks so we can instrument outbound calls. We don't actually know if any are going to be made, let alone how many will be made. We're loading a URL provided to us by an outsider. We also have onConsole callbacks, onPageError, onPageCrash, and so on. That's just for what we want to instrument so we can go back and see what the page was doing in the event we need to debug some issues later on.
That then frees us up to use the newly added waitForCondition method for the page to reach a "ready" state, which is when we begin taking screenshots and generating PDFs. We utilize waitForCondition because we manage our own timeout external to Playwright. That timeout incorporates signals from bindings we've created in the page so that the page can signal "progress" towards readiness.
In short, what you suggest sounds nice and simple. But our use case does not fit into that pattern. As I mentioned before, if it's helpful I'm more than happy to jump on a video call to explain and demonstrate what we're doing. It's not web testing, as you may have gathered. So maybe this isn't necessarily something Playwright wants to invest time in helping out with, which I would understand. But if it's valuable to more folks than just us, I'd be more than happy to get into greater detail.

In short, what you suggest sounds nice and simple. But our use case does not fit into that pattern. As I mentioned before, if it's helpful I'm more than happy to jump on a video call to explain and demonstrate what we're doing. It's not web testing, as you may have gathered. So maybe this isn't necessarily something Playwright wants to invest time in helping out with, which I would understand. But if it's valuable to more folks than just us, I'd be more than happy to get into greater detail.

This all makes sense, thanks for providing more details! I understand that in scraping and other use cases this may be more of an issue as the web apps behavior is less predictable, whereas in testing scenarios one usually knows what requests the app may/is expected to send. Our primary focus has been web testing so far.

I am having similar issues when running tests.
Basically I have a test that has some setup it has to do, it kicks off some actions on the page, and then wants to continue only after a request has been made to a specific url.
What I have ended up with is setting up an onRequest listener to record when a request is made to that url, and that listener delivers to a clojure promise (a countdown latch, a read from it blocks until it has been deliver to, similar to some uses of CompletableFuture). Then I expect to be able to wait on the promise until the callback fires, but that is not the case, I have to loop alternating between calling waitForTimeout (to process events) and checking the promise (to see if the callback fired).
This took me a bit to figure out because I assumed if I registered callbacks they would just get called when events come in. I can see how this might make things somewhat more deterministic for tests, which is great, but the ux on a platform with good native multithreading like the jvm is weird.

unless I am completely off base advice like "prefer waitForRequest to onRequest" actually introduces race conditions:
if you consider something like:

use the api to click a button in the browser
the browser makes a request for page P

if you do 1, and then wait for 2, the browser may have done 2 before you called the waitForRequest method.
will a callback approach you can register the callback before doing 1 so that it will be in place whenever 2 happens.

if you do 1, and then wait for 2, the browser may have done 2 before you called the waitForRequest method.

This is why you click inside the callback passed to waitForRequest, see its usage examples in the docs.

ah, sorry, my mistake, I was looking at some wait* method in Frame.java and assumed waitForRequest was the same.
looking at Frame.java it looks like the example usage for waitForUrl is the above race condition.
https://playwright.dev/java/docs/api/class-frame#frame-wait-for-url


no response to http requests on pico w /CP8 Beta

tried circuitpython 8 beta 1/2/3 but the HTTPServer is not usable on pico w.
minimal examples like all three from this very lib: https://github.com/adafruit/Adafruit_CircuitPython_HTTPServer/tree/main/examples
are not repsonding to simple browser or curl requests

ERR_CONNECTION_REFUSED (like about every 50. request does in fact get a response)

then i read this, and he is absolutly right: https://github.com/adafruit/Adafruit_CircuitPython_HTTPServer/issues/14#issuecomment-1283293120
replacing the .mpy to source .py and changing L:334 to
self._sock.setblocking(True)
will make the HTTPServer respond every time
https://github.com/adafruit/Adafruit_CircuitPython_HTTPServer/blob/ba2da4352290b880592ae5ab5db83ee50c045244/adafruit_httpserver.py#L334
not sure if this is related to this issue, https://github.com/adafruit/Adafruit_CircuitPython_HTTPServer/issues/9 wich dies belong to this MR https://github.com/adafruit/Adafruit_CircuitPython_HTTPServer/pull/10/commits/bcabd1d924b9845ce4e9fc03788f1d7ebafd53a7 wich originally does change this very line to the opposite that makes the lib work for me and @justbuchanan.
sadly this is not a real solution, since it is now waiting/blocking to a incoming request and no other magic can happen while waiting :-(

When I originally (briefly) tested Pico W with http_server it worked, must have gotten lucky. But more recently, I'm seeing the same behavior with a simple asyncio http_server ...successful connections are rare. I wonder if there is less buffering in the internals on the Pico W. (On the bright side, the asyncio http_server occasionally hardfaults on esp32-s2, but not on Pico W.)

Should be fixed now with #19

Should be fixed now with #19

wonderful. thank you.
i testet the change locally and it is now working as expected.


Allow David Goss to release Cucumber-JS

For context, see https://cucumberbdd.slack.com/archives/C612KCP1P/p1616759048003700

We've been using docker for a while to make cucumber-js releases:

https://github.com/cucumber/cucumber-js/blob/master/CONTRIBUTING.md#release-process
https://github.com/cucumber/cucumber-js/blob/master/Makefile

So is this just to get him onto keybase then?

Yes, @davidjgoss just DM your keybase username to @mattwynne once you've set it up and he should be able to grant you access. (I can also grant you access, but I want to make sure @mattwynne can do it too).

Thanks @mattwynne @aslakhellesoy, I can pull the secrets and run the docker container successfully so happy to close this now.
Once we get https://github.com/cucumber/cucumber-js/pull/1637 merged I'll try doing a minor release.


sample oozie test job

Added a tools directory where we can use small smoke-test tools.
Added a sample oozie test case that tests an oozie shell action and pushes the resulting job id to hdfs for verification.
I envision more simple tools being added as time goes on.

I really want to merge this, because it would be useful to me, but I don't understand whether Lev's comment should be addressed, first.

@pu239ppy - the overrides provide a way to override default values, allowing to test if the defaults are set properly or not, as I've seen it cause issues.

I think oozie coordinator jobs are more appropriate for recurrent actions. And since this just a test job, shouldn't we have oozie workflow  job ? In that case, we avoid the onus of updating start and end time each time we run the job.

I have replaced the job with an even simpler shell action that is just a workflow and has no overrides.  I have also added a setup script to make running it easier.

Can you please squash the commits?

I will, once we all agree that the code is good and ready to merge.

Commits have been squashed.

Instead of using /tmp which will collide if multiple users try to run this, how about using ${wf:user()} in the paths?


[sub-mac] fix elapsed time in GetCslWindowEdges()

Do all the calculations on uint32_t to prevent
bad wraparound check due to mixing uint32_t with uint64_t.
Fixes #7054

@marius-preda


Add Peblar

Hey EVCC team,
Here is a charger implementation for the Peblar chargers (https://peblar.com/products/ev-charging). Developer info on the modbus API is located at developer.peblar.com.
I've been using this implementation for a while and tested it for the 1p3p switchover to work. This switchover won't work on all variants due to relay configurations (and 1 phase connected systems ofc).
I've checked the doc build to run fine and cleaned out some lint errors. Can you review this code?

Hmm, something bad happens to the currents, I'm getting a lot of these:
"Garage: charger logic error: current mismatch (got 15.1A measured, expected 12.2A)"
Need to find out if it is related to the recent changes.

There is a 1A difference between set current and max phase current observed:
		// use measured phase currents as fallback if charger does not provide max current or does not currently relay from vehicle (TWC3)
		if !isCg || errors.Is(err, api.ErrNotAvailable) {
			// validate if current too high by more than 1A (https://github.com/evcc-io/evcc/issues/14731)
			if current := lp.GetMaxPhaseCurrent(); current > lp.chargeCurrent+1.0 {
				if shouldBeConsistent {
					lp.log.WARN.Printf("charger logic error: current mismatch (got %.3gA measured, expected %.3gA)", current, lp.chargeCurrent)
				}
				lp.chargeCurrent = current
				lp.bus.Publish(evChargeCurrent, lp.chargeCurrent)
			}
		}

Hmm, something bad happens to the currents, I'm getting a lot of these:
"Garage: charger logic error: current mismatch (got 15.1A measured, expected 12.2A)"
Need to find out if it is related to the recent changes.

I've seen this happen on 2 different vehicles, it is not vehicle related..

I assume the new current is set the end of the loop (pv charge current: x = y + z)
The vehicle has plenty of time to set this new current (in my case 20s)
The loadpoint charge currents used in lp.GetMaxPhaseCurrent are gathered in the start of the next loop.

Still need to dig into this further where this mismatch is coming from. Right now I can only say it happens on "changing" conditions such as cloud-sun-cloud.

I've seen this happen on 2 different vehicles, it is not vehicle related..

Mhhm. And the version before the brushing did not show this?

I've seen this happen on 2 different vehicles, it is not vehicle related..

Mhhm. And the version before the brushing did not show this?
I don't remember seeing this on my initial fork of about 2 months ago but the check seems added afterwards. I'll place some low level debug to see if this comes from the charger API.

Did you find out whats wrong?

Did you find out whats wrong?

Not really.. I'm trying to see what happens in the "charger logic error". For that, I added logging to the setCurrent & getPhaseValues functions. Here is what I get during a charger logic error:

In loop "n-1" the current is set to 15631mA via modbus to the charger. That current is applied in at most 5 seconds by the EV. (This EV always draws 0.8A less than the advertised limit).
In loop "n" the current is read back and is according to (my) expectation (14.9A). However the expectation from the charger logic error is another value, namely, a value mentioned as "max charge current" mentioned in loop n-1, being 13.4A.
At the end of loop "n" this "max charge current" of 13427mA is eventually set towards the charger.
My conclusion would be that the expectation of the charger logic error is actually 1 iteration too soon. What do you think?

Didn‘t check the code yet, but why does the actual setCurrent only happen with one cycle delay? It is using the wrong value imho.

Hi @andig, I've been testing this branch for the past weeks and have not seen more warnings regarding the current sanity check.
Is there anything you would like to see changed? Otherwise this could be merged.


exclamation mark shortcut doesn't work

Issue Type: Feature Request
I don't really know what happened, I don't remember changing any files that could affect it, but the exclamation mark shortcut doesn't work, even though other emmet things work
VS Code version: Code 1.69.0 (92d25e35d9bf1a6b16f7d0758f25d48ace11e5b9, 2022-07-07T05:28:36.503Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.69.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.69.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

it didn't help :(

Duplicate of https://github.com/microsoft/vscode/issues/154375


Properly retrieve folder names from html pages of all mirrors

When https://download.qt.io is not available, aqt list-* command falls back to a random mirror.
Different mirrors provide different html layouts.
Mirrors with <table> similar enough to be properly handled by the current script:
https://download.qt.io/
https://ftp.jaist.ac.jp/pub/qtproject/
https://qt-mirror.dannhauer.de/
https://master.qt.io/
Mirrors with <table> which isn't properly handled by the current script:
https://mirrors.aliyun.com/qt/
https://mirrors.sjtug.sjtu.edu.cn/qt/
https://mirrors.dotsrc.org/qtproject/
Mirrors with a list of <a> elements inside <pre>, no <table>:
https://qtproject.mirror.liquidtelecom.com/
https://mirrors.ustc.edu.cn/qtproject/
https://ftp.yz.yamagata-u.ac.jp/pub/qtproject/
https://ftp.fau.de/qtproject/
https://mirror.netcologne.de/qtproject/
https://www.nic.funet.fi/pub/mirrors/download.qt-project.org/
https://mirrors.ukfast.co.uk/sites/qt.io/
https://qt.mirror.constant.com/
Mirrors with a list of <a> elements inside <pre>, with <table> which doesn't contain folders:
https://ftp2.nluug.nl/languages/qt/
https://ftp1.nluug.nl/languages/qt/
If the mirror has similar enough html layout, the program works and prints the requested info.
If there is no <table> element,
v2.1.0 fails with the error described in #519 ;
the master branch prints
Failed to retrieve the expected HTML page at https://download.qt.io/online/qtsdkrepository/linux_x64/desktop/

This is an unrelated bug. The program should be printing the fallback url, not baseurl.
If there is a <table> element, the program prints
No versions available for this request.

or
No tools available for this request.

Solution.
Get folder names from value of href attribute of <a> elements.
Folder links point to a page one level deeper relative to the current page,
i.e. the value should look like ./foo or foo,
NOT https://foo, ../foo, .., /foo, ?foo.
How to test.
git clone https://github.com/mite-user/aqtinstall.git
echo '__version__ = "handle-html-test"' > aqtinstall/aqt/version.py

Open aqtinstall/aqt/settings.ini,
set baseurl: to http://localhost,
remove all mirrors expect the one to be tested from fallbacks:.
cd aqtinstall
python3 -m aqt list-qt linux desktop

This commit might fix #519 .

Updated the original commit

Using urllib.parse.urlparse.
Handling ./ at the beginning and / at the end with posixpath.normpath.
Checking for one and two dots only. Apparently three and more dots point one level deeper.

Second commit
Added test html files for test_list_versions_tools() in tests/test_list.py.
Notes
Absence of <table> element no longer triggers the error added in #518 . There should be some other way to check if html contents are unexpected. test_list_bad_html() now always fails.
list-tool doesn't sort the resulting list. So the order depends on what is provided by html. Looks like all or almost all mirrors provide sorted links by default. They are unsorted at https://download.qt.io .

Changed link_to_folder().
Removed try: except (AttributeError, IndexError):.
With the new code, exceptions are unexpected and shouldn't be ignored.
Changed link.get("href") to link.get("href", default="").
Without default="", it will return None instead of empty str if there is no href in <a>.

Absence of <table> element no longer triggers the error added in #518 . There should be some other way to check if html contents are unexpected. test_list_bad_html() now always fails.

For the record, I’d like to say that I support removing the test_list_bad_html() test that now fails. I don't think that it really adds all that much.
I wrote that test under the assumption that #517 was caused by being in a coffee shop, or by some http error. Given the evidence that @mite-user presents here, l think that it's much more likely that a different html structure in a mirror site was the cause. The requests library should be able to detect serious HTTP errors that would result in malformed html, and I don't think you get a 200 status code when a router asks you to sign in.

@miurahr Any further thoughts on this? I still think we should remove test_list_bad_html() and merge this. I don't think there's any obvious way to merge this without removing that test, and I don't think the test is useful.

@ddalcino

I still think we should remove test_list_bad_html() and merge this.

I think it is OK to remove and merge this.

Should I add a commit to this pull request to remove test_list_bad_html()?

Should I add a commit to this pull request to remove test_list_bad_html()?

IMHO, yes, you should add a commit that removes the failing test. I don’t think we need a separate PR for this. If we did, I think we would have heard from @miurahr by now.
This PR effectively replaces #518, which is where test_list_bad_html() comes from. Speaking as the author of that test, I think the right way to replace it is to just remove it.


fix: filter section is overflowing for small screen in dashboard sect…

…ion #1564

Description
This PR resolved the error of overflowing for a small screen.
Related issue(s)

Resolve #1564

@Tusharjain123 do you plan to continue with this one?

// @akshatnema

@Tusharjain123 Kindly resolve the conflicts inside this PR.

@Tusharjain123 Kindly resolve the conflicts inside this PR.

@akshatnema please take a look, I have resolved the conflicts

@Tusharjain123 Add screenshots for the changes you made in UI in description of this PR.

/rtm


Distinguish between overt pronoun / gender identification by a Twitter user and inferred gender in results

https://github.com/ajdavis/twitter-gender-distribution/blob/221175cdd42189d091e3373d4b30437bcd753ef4/analyze.py#L107
                g = detector.get_gender(name, country)
                if g != 'andy':
                    # Not androgynous.
                    break

Hi there. The above section (and others that return values from the lookup table rather than taking a user's expressed gender/pronoun in to consideration) should probably be categorised differently in the results table - perhaps as 'probably male based on name' rather than being included in the 'male' column. Otherwise you are saying that the name to gender mapping you have used is absolutely accurate for all people who share a given name in the list.

I am certainly not claiming absolute accuracy. =) I think it would be useful in the results page to display, in small text, "12% declared pronouns" to indicate what percent of each category of accounts have declared pronouns. The genders of the rest are guessed. Would you like to make a PR?

I'm very rusty in Python, but I could attempt a PR. Might take a while 😄
I think fundamentally we might disagree on whether it's appropriate to be grouping results at all, though. I don't believe any automated tool can state with any confidence that an individual's gender is any particular value unless that individual has explicitly stated it. So the counts based on pronoun are resonable, but to included inferred gender based on name is inappropriate in my opinion.
Any PR I make would alter the output table to make this distinction clear. Would that be OK?

Inferring gender based on name is the main function of this tool.
I'm curious what percentage of the people you follow on Twitter, or who follow you, declare their pronouns; it's my experience that very few Twitter users declare their pronouns in their profiles. Since the purpose of the tool is to help men like me follow more women and nonbinary people, I need a way to estimate the gender proportion of the maximum number of my Twitter contacts. I prefer an estimate of a large number of my contacts instead of a definitive count of a small number. Do you agree with that goal?
Given my goal, and how few people on Twitter announce their pronouns, I think the display should emphasize the estimate, rather than emphasize the number of people who announce their pronouns.

Here's what my results look like now:


Add long run yields

Create a new dataset that has all data, including long-run series, on crop yields.

After some discussions on this issue and on this slack thread, we have decided to use the long-term crop yields data by default in all our charts and the explorer.
So I have made two changes:

I created a new version of the attainable_yields dataset, that ingests long_term_crop_yields instead of faostat_qcl.
I created a new version of the crop-yields explorers step, that ingests the latest attainable_yields dataset.
I'll merge this PR, check the new explorer and charts, and possibly come back to this on a different PR.


Adding ncvis as ncvis-climate to conda-forge

xref https://github.com/conda-forge/staged-recipes/pull/23033
Currently, it seems it builds correctly, but running into linking errors that have been discussed in the repo here.

Is the name ncvis-climate okay? There is already ncvis in conda-forge
Please have a look at the conda-forge recipe, and let me know if you can help fix the linking issues. I also am more than happy to add whomever is interested to the list of maintainers

cc: @rajeeja

@mahf708 - this would be wonderful!  I just installed on perlmutter and got the following error that suggests some fonts might need to be added to the package:
$ ncvis e5.oper.an.vinteg.162_089_vilwn.ll025sc.2020030100_2020033123.nc
./src/common/filename.cpp(2027): assert ""Assert failure"" failed in IsValidDirComponent(): invalid directory component in wxFileName
NcVis 2022.08.28 Paul A. Ullrich
ERROR: Font "/global/common/software/m1517/perlmutter/climatetools/bin/Ubuntu-Regular.ttf" not found

conda reports that it's not available in that channel

sorry, should have said that I tried that too - doesn't seem to be visible yet to me on conda-forge

oh, it's using the package cache .. hang on, let me figure out how to override that

not a problem at all.  Unfortunately just ran out of time for the day - need to head out.  I'll pick this back up on Monday (maybe tomorrow)

@mahf708 Q about how the conda forge version gets update?  Does someone have to do something or does it pull the latest (tag? main?) from here?


SSL verifier fix

I have added the method to specify the certificate file. ie (PEM). I
have added a working example to the test folder along with the
certificate. You can verify it and also you can download the ssl
certifcate from firefox for any website that offers ssl and when saving
save it as .pem file instead of .crt and choose the option to download
with chain.

@yousafsyed I just released v2.0.0 can you please review and refactor your changes accordingly?
also, some more unit-tests (with phpunit style) would be great!
thank you!

hey @yousafsyed, any progress?
thanks :)

How to set pem files in unirest request.

@kamaljeetrathi please open a ticket on the unirest-java project


Products page sort by functionality

https://trello.com/c/i2VywVhj/1246-products-page-html-design-revision-and-refresh-sort-by-functionality
Description

Add Sort by in line with search results statement on the Products page
Re-use and generalise the sort implementation from case search page
Products sort options:

Newly added
Name
Relevance (only show Relevance when a search term is submitted)

Checklist:

[ ] Have you documented your changes in the pull request description?
[ ] Does the change present any security considerations?
[ ] Is any gem functionality overloaded? Eg: Devise controller methods being overloaded.
[ ] Has acceptance criteria been tested by a peer?
[ ] Has the CHANGELOG been updated? (If change is worth telling users about.)

General testing (author)

[ ] Test without JavaScript
[ ] Test on small screen

Accessibility testing (author)

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable CSS - does content make sense and appear in a logical order?

Codecov Report

Merging #1691 (f509ec9) into master (7c56bb8) will decrease coverage by 78.53%.
The diff coverage is 34.37%.

@@             Coverage Diff             @@
##           master    #1691       +/-   ##
===========================================
- Coverage   89.83%   11.29%   -78.54%
===========================================
  Files         402      403        +1
  Lines        8639    12222     +3583
  Branches     1243      344      -899
===========================================
- Hits         7761     1381     -6380
- Misses        700    10800    +10100
+ Partials      178       41      -137

Impacted Files
Coverage Δ

app/controllers/products_controller.rb
0.00% <0.00%> (-82.61%)
:arrow_down:

app/helpers/investigations_helper.rb
16.15% <ø> (-81.69%)
:arrow_down:

app/helpers/products_helper.rb
49.20% <0.00%> (-42.73%)
:arrow_down:

app/helpers/search_params.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/helpers/sort_by_helper.rb
41.66% <41.66%> (ø)

app/models/product.rb
84.61% <66.66%> (-15.39%)
:arrow_down:

app/models/contact.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/country.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/accident.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/incident.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 332 more

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 7c56bb8...f509ec9. Read the comment docs.


Claim display in actionbar(popup)

Please make sure your issue complies with these guidelines:

[x] Idea must have not been suggested previously

[x] Be detailed on what you want & make sure the idea is plausible

Description

To be very honest, doing /f map every time kind of sucks. So when a player walks into a claim(counts for the players in that faction too), it shows a message(keeps showing until you get out of the claim) in the actionbar(pop up) saying something like "Area claimed by: {faction}"
Extra Information
No extra information

Faction of the territory you're walking into is already displayed as a title, unless you are still within that faction's territory.


⚠ Bofrost.fr has degraded performance

In ff528c4, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1467 ms

Resolved: Bofrost.fr performance has improved in 59798e0 after 6 minutes.


name suggestion index was not updated in 2.19.6

Hi,
It looks like I need to wait for another few months to get something new from index.
Is it OK to directly pull the data from dist when you release any newer version in the future? The official "release" of index serves little value IMO because the list is being frequently updated.

@cicku I'm working on this in #8305 and it's just about ready
You can use the NSI v5 data on this branch:  https://ideditor.org/nsi-v5/
But please be careful with using the validator to upgrade tags, and post any issues that you find on #8305


Creating a coverage script based on kcov

Hm, what is this for?

I was trying to determine what code was covered by existing tests. kcov does the job rather nicely, and produces output like

I thought it was useful and might be worth sharing. Feel free to decline the PR, though.

Hm, I’m not sure simple coverage statistics like that are particularly informative.
I’m pretty sure though we are testing most interesting paths; sometimes writing a test takes an unreasonable amount of time, or is just not feasible (e.g. for path handling logic or the executables in src/ops).

As a new contributor, it was very useful to me to be able to see an objective view on how those decisions had been made in the past. It helps inform where I should spend time adding tests for new code.
On other projects, I've more than once found that the indifferent reporting of a coverage tool helped identify code paths that I was sure were tested but had been overlooked.
As I said, it was an interesting exercise (and the ease of setting up coverage was gratifying), but if it's not useful to the project, please do decline the PR.


Add edge_subgraph graph method

This commit adds a new method, edge_subgraph(), to the PyGraph and
PyDiGraph classes for generating a edge induced subgraph from a graph
object.
Related to #389

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

85 of 85 (100.0%)  changed or added relevant lines in 2 files are covered.
9 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.004%) to 95.472%

Files with Coverage Reduction
New Missed Lines
%

src/astar.rs
9
91.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
5819

Relevant Lines:
6095

💛  - Coveralls


Set hour format

The widget is taking the time format of the device, but I need it to be equal on every device. It should have 24 hour format on every device. Is that somehow possible?

set alwaysUse24HourFormat to true, check if it works


[FR]: Add privacy manifest to Installations&Messaging

Description
It appears that Installations & Messaging was missed when privacy manifests were added to the various modules in this repository.
I was expecting to see at least one entry for NSPrivacyAccessedAPICategoryUserDefaults.
FirebaseInstallations:
https://github.com/search?q=repo%3Afirebase%2Ffirebase-ios-sdk++userDefaults+path%3AInstallation&type=code
FirebaseMessaging:
https://github.com/search?q=repo%3Afirebase%2Ffirebase-ios-sdk++userDefaults+path%3A%2F^FirebaseMessaging\%2F%2F&type=code
https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api?language=objc
API Proposal
No response
Firebase Product(s)
Installations, Messaging

Hi @tetsu888, thanks for opening. Could you please re-open an issue using the bug template? That template will include other information that will be helpful in triaging this.
In Firebase 10.22.0, privacy manifests were added for both of these products:

https://github.com/firebase/firebase-ios-sdk/tree/10.22.0/FirebaseMessaging/Sources/Resources
https://github.com/firebase/firebase-ios-sdk/tree/10.22.0/FirebaseInstallations/Source/Library/Resources


Fix parseEvery for year false detection

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/metautil/commit/1a0dead32f5566e45010f355a9354eeb07d8401c


How to run ts-tests

The README.md in ts-tests is outdated

Refer to
https://github.com/paritytech/frontier/blob/master/.github/workflows/rust.yml#L29


Compute min/max values for columns to support file-level and chunk-level pruning.

Problem Statement
Using min/max values to skip / pruning chunks is a common practice in columar storages. Let's add it as well to support coarse grained filtering.
Open questions:

Should we use different bit width to support different data types (i.e., 8 bits for int8/uint8, 64 bits for int64/uint64/double, or just one implementation for all.
How to support string values efficiently. Do we only want to support min/max for dictionary string values or string values in general?

Desired behavior
Compute min/max values and store them in a fashion that does not hurt either full scan or point queries. Ideally, such indices are only loaded when necessary, while also taking advantage of optimal I/O size on S3/GCS, and vectorizations.

use the datafusion stats abstraction for this or?

Oh this is about the on disk format , where to store them and how many IOs
to read them
On Mon, Feb 13, 2023 at 9:40 PM Chang She @.***> wrote:

use the datafusion stats abstraction for this or?
—
Reply to this email directly, view it on GitHub
https://github.com/eto-ai/lance/issues/11#issuecomment-1429151350, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAEFSM73HA24YFIV5TRLJLWXMLFHANCNFSM53AFX2CA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Lei Xu
Eto Labs https://eto.ai | Calendly https://calendly.com/leixu/30min

This is subsumed by more specific issues and design docs from @wjones127 so closing as duplicate


Cassowary Algorithm, closes #90

Uses a cassowary algorithm based on kiwi.js to perform oil/uranium matrix calculations.

Let's say your only output product is Rocket Fuel.

Ok, and I use Basic Oil Processing for it: https://factoriolab.github.io/list#eJwdiEEKACAIwH7TOU2yBv4p6v9QxmCMrZDWaUjZIaqA96dyAv1YDjdkTKgfMriJpA0H
By the way, here I was able to choose Assembling Machine 1, which does not support recipes with liquid.
Now I see that it takes 10 more Light Oil. I add the required quantity with a second product: https://factoriolab.github.io/list#eJwdikEKwCAQxH7jqYedVbQN7J+K/h90JTCEIX+odip6ZMKQlRlyB0Y/U1bgl5bHaOj9OGFCChv7ww7K
And now I need 20 Light Oil already. But since I can not choose another process for the production of Light Oil, since it is rigidly selected in the settings, the whole calculation becomes useless.

@hhrhhr I think that should probably be best entered as a separate issue to be figured out and solved outside the scope of this PR.
If you add the same settings on staging you have an additional recipe listed and water required...
https://factoriolab.github.io/staging/list#eJw9jEsOgCAQxW7jykkg7l7CYYCMMJFfHLy/srHbNh3u7vHiSefDBQfsXiTlSV0KDKzZ1GHhVbmGIi1R9TFLY7JA8CpxtTS+Dat+Hj8vpwkesw==
I think there is a bug there.

@hhrhhr if you specify to use basic oil processing then there's no way to produce light oil (since that produces no light oil) so it lists the light oil by itself since it cannot be produced.
The new algorithm should basically do the same thing but for some reason it thinks the cracking recipes are ok even when using basic oil processing, so I need to investigate that.
The fact you can pick assembler 1 for rocket fuel is a separate bug and can be opened as a new issue. I'm on vacation this week otherwise I'd create it myself.


How about enabling the Wiki?

As you mentioned, there is not much documentation about the code.
Therefore a Wiki should be a good choice to collect all the insights on working with the code.
Would you like to enable it for public editing?

Good idea


Add logs feature to recents feature

This commit resolves issue
https://github.com/AY2122S2-CS2103T-T17-3/tp/issues/52.
Now, users would be able to view all contacted information for a
specific client.

Codecov Report

Merging #84 (422ec85) into master (01340cc) will decrease coverage by 0.09%.
The diff coverage is 47.61%.

@@             Coverage Diff              @@
##             master      #84      +/-   ##
============================================
- Coverage     69.45%   69.35%   -0.10%
- Complexity      591      597       +6
============================================
  Files           103      105       +2
  Lines          1964     1994      +30
  Branches        199      202       +3
============================================
+ Hits           1364     1383      +19
- Misses          547      556       +9
- Partials         53       55       +2

Impacted Files
Coverage Δ

...ss/logic/commands/ListContactRemindersCommand.java
0.00% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
82.50% <0.00%> (-2.12%)
:arrow_down:

...dress/logic/commands/ListContactedInfoCommand.java
23.07% <23.07%> (ø)

...s/logic/parser/ListContactedInfoCommandParser.java
100.00% <100.00%> (ø)

...a/seedu/address/logic/commands/HashtagCommand.java
100.00% <0.00%> (ø)

...edu/address/logic/parser/HashtagCommandParser.java
100.00% <0.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 01340cc...422ec85. Read the comment docs.


SncRedisBundle SwiftMailer integration needs SwiftMailer to be installed

Since https://github.com/snc/SncRedisBundle/commit/a26798998d7eeac85cfc241be812c7055df3ecdf
I get
SncRedisBundle SwiftMailer integration needs SwiftMailer to be installed
if I try to use dev master with
 "symfony/assetic-bundle": "^2.8",
    "symfony/dotenv": "^4.0",
    "symfony/monolog-bundle": "^3.2",
    "symfony/swiftmailer-bundle": "^3.2",
    "symfony/symfony": "3.4.*",

Oops, thanks and fixed!


build: add delve, and add make-directive for building the executable …

…and running delve.

Looks like you already have a demo on how to use it here.
Should we introduce this in the doc as well?

Looks like you already have a demo on how to use it here.
Should we introduce this in the doc as well?

Sorry, yes, these PR's are linked. This PR won't be merged until/if we decide to merge the one in helloworld.


Add global formatting for the webapp.

Adds a global format command that runs prettier's --write on all specified file types.
Uses the following options within the config (prettierrc.json):

"semi": false: prevents addition of semicolons on save
"trailingComma": "es5": https://prettier.io/docs/en/options.html#trailing-commas
"singleQuote": false: uses double quotes for imports/elsewhere.
"tabWidth": 2: @norswap did you want this as 4?
"useTabs": true: https://prettier.io/docs/en/options.html#tabs
"printWidth": 100: increase the number of characters that prettier will try to wrap around, prevents the dependency arrays of useEffect and useCallback hooks from splitting into a column unless absolutely needed (https://prettier.io/docs/en/options.html#print-width).

Also added eslint-config-prettier to be used as a plugin as the recommended mediator between linting and formatting.

there's another interesting plugin we could use: https://www.npmjs.com/package/eslint-plugin-simple-import-sort
this helps sort imports in different files according to your specs, here's an example one:
"simple-import-sort/imports": [
      "error",
      {
        "groups": [
          // Packages. `react` related packages come first.
          [
            "^react",
            "^next",
            "@metamask|@walletconnect|clsx|@reduxjs|async-mutex|flowbite-react|redux-persist|pubnub|redux",
            "^(ethers|google|lodash|bn.js|styled-components|child_process|cluster|console|constants|crypto|dgram|dns|domain|events|fs|http|https|module|net|os|path|punycode|querystring|readline|repl|stream|string_decoder|sys|timers|tls|tty|url|util|vm|zlib|freelist|v8|process|async_hooks|http2|perf_hooks)"
          ],
          // Internal packages.
          ["^@?\\w"],
          // Parent imports. Put `..` last.
          ["^\\.\\.(?!/?$)", "^\\.\\./?$"],
          // Other relative imports. Put same-folder imports and `.` last.
          ["^\\./(?=.*/)(?!/?$)", "^\\.(?!/?$)", "^\\./?$"],
          // Style imports.
          ["^.+\\.s?css$"],
          // Side effect imports.
          ["^\\u0000"]
        ]
      }
      ```

As well as: https://github.com/tailwindlabs/prettier-plugin-tailwindcss

"tabWidth": 2: @norswap did you want this as 4?

Yes let's go for 4!

"useTabs": true: https://prettier.io/docs/en/options.html#tabs

This is always religious, but I do prefer spaces myself. I guess with prettier it matters less, but it ensures proper display on Github for instance.
Any counter here? Like would you strongly prefer a 2 or 8 spaces local display?

"tabWidth": 2: @norswap did you want this as 4?

Yes let's go for 4!

"useTabs": true: https://prettier.io/docs/en/options.html#tabs

This is always religious, but I do prefer spaces myself. I guess with prettier it matters less, but it ensures proper display on Github for instance.
Any counter here? Like would you strongly prefer a 2 or 8 spaces local display?

No hard preference here, 4 is good!