ci: add typos as spell checker

What type of PR is this? (check all applicable)

[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] ♻ Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[x] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert
[ ] 🌐 Internationalization / Translation

Description
This PR adds crate-io/typos as spell checker in CI process.
Spell errors will cause the CI to fail, with errors reported in the Annotations part of GitHub Actions workflow runs (link to this run):

Its behavior can be controlled with the typos.toml file, with config specifications detailed here.
Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentations?

[ ] 📓 docs (./docs)
[ ] 📕 storybook (./storybook)
[ ] 📜 README.md
[x] 🙅 no documentation needed

Thanks @zyf722
That is nice.
Is there an easy way to run the test locally? e.g. npm package. This can allow us to reproduce problems locally without having to keep pushing to GitHub to see results.
I know there is a vscode extension (which is convenient during authoring) but I'm not sure we can use it to run over the whole code base.
How about cspell? Also has a vscode extension which seems to be very popular. I'm actually using it.
(I know I slipped a few spelling mistakes, but I was not enforcing it with a test! 😅)

Is there an easy way to run the test locally? e.g. npm package. This can allow us to reproduce problems locally without having to keep pushing to GitHub to see results.

typos is written in Rust, and as far as I know, there is no official npm package for typos. Instead, you can either manually download pre-built binaries from the GitHub release or install it via cargo, brew, conda, or pacman.
You might also be interested in their pre-commit hook.

How about cspell? It also has a VSCode extension which seems to be very popular. I'm actually using it.

It's new to me. I will check it out and see if it fits our needs better :)

I was not enforcing it with a test! 😅

Indeed, actually I guess we might be faced with false positives in some cases.
Perhaps it would be better to only throw warning annotations when typos are detected, rather than error annotations and terminate the workflow?

Indeed, actually I guess we might be faced with false positives in some cases.

Actually, cspell does have some false positives in my experience with it.

Perhaps it would be better to only throw warning annotations when typos are detected, rather than error annotations and terminate the workflow?

I agree.

@zyf722
Shall we start moving this forward?
It is currently failing for other translations.
Do you think this can be fixed or should we just ignore this directory?

@hatemhosny
Thank you for reviewing the PR.

Shall we start moving this forward?

I'd love to! As long as you consider it is worth doing so, I'm always ready to help.

It is currently failing for other translations.
Do you think this can be fixed or should we just ignore this directory?

Already fixed. We previously haven't tracked .lokalise.json files in the repository, so the typos config file didn't exclude them. Now there won't be any more issues with translation files.

Thank you @zyf722

Perhaps it would be better to only throw warning annotations when typos are detected, rather than error annotations and terminate the workflow?

Currently, the setup would lead to failure of the build workflow and prevent us from merging if typos were detected. (cannot ignore false positives)
If we change it to warning instead of error, the current UI of GitHub PR would lead us to miss them unless we specifically check the workflow logs. (may miss true positives)
I suggest adding it to a new workflow, which if fails it would be visible in the PR UI, but it does not prevent us from merging, similar to how the workflows for sonarcloud and codacy behave.
What do you think?

I suggest adding it to a new workflow, which if fails it would be visible in the PR UI, but it does not prevent us from merging, similar to how the workflows for sonarcloud and codacy behave.

Indeed, it's a good idea to add a new workflow for this.
I've moved the spell checking step to a separate file CI-typos.yml which has same trigger conditions as CI-build.

Thank you @zyf722
I think this would be very useful.


udemy spesific lecture

i dont know why you dont help me here
i tried with this command to download start from specific lecture but its still start downloading from lecture 1
D:\Downloads\dlmster\uddl.py -o 'D:\contents\New folder' -u<EMAIL_ADDRESS>-p **** --skip-sub -c 6 --lecture-start 053 https://www.udemy.com/growthhacking/
just help me about exact command for this

try the below mentioned command and edit it according to your need
python udemy-dl.py  --lecture-start 035 --lecture-end 036 -u -p https://www.udemy.com/growthhacking/
035 or 35 lecture number give a try to variation, if above not working

@vikasnewports  your command is not working i tried


Multi-channel Image Dataloaders

Fixes # .
Description
This is an example highlighting the use of custom dataloaders for multi channel images. Multi channel images are very commonly used in healthcare, this example shows one such use case in microscopy.
Checks

[x] Data must be downloaded prior to running the notebook
[x] Path to the parent data folder must be provided

thanks, I think it's a good PyTorch example but it's not really using monai. (any suggestions @Nic-Ma @rijobro @ericspod?)

Would it help to use a MONAI model instead of pytorch ResNext? I had used MONAI ViT with the same dataloaders so I can swap out the model if that helps

I think the key contribution here is the dataset class demonstrating how to combine multichannel images. This can be recast using MONAI components (LoadImage) to load files instead of using PIL directly and use our models and other components in the training process which is pretty straight forward. You could use our engine classes as well instead of the regular training loop if you wanted to reduce total code in the notebook.

Thanks Eric, I can replace the non-MONAI components with the MONAI equivalent. Could you please direct me to the engine classes you referred to and any examples for using the same?

Thanks Eric, I can replace the non-MONAI components with the MONAI equivalent. Could you please direct me to the engine classes you referred to and any examples for using the same?

The classes are in the monai.engine module. This tutorial and others in the Tutorials repo and in the Model Zoo help explain their use: https://github.com/Project-MONAI/tutorials/blob/main/acceleration/distributed_training/unet_training_workflows.py

I just saw that this PR is still open. It should be noted that in the meantime I worked with @aasthajh to create a multi-channel microscopy tutorial (already merged, see notebook). This tutorial covers the topics proposed in this notebook PR, but is more MONAI native (e.g. using ConcatItemsd transform and DenseNet169 network). Perhaps it makes sense to close this PR, since it is not really necessary anymore? What do you think @aasthajh?


Translation to Japanese

Hi Folks,
The COVID - ALERT app currently works in 8 different languages. We need this to work in Japanese.  We estimate the time needed for this task would be around 60-90 minutes. Please be very sensitive to the nature of messaging - best would be to slack me "Shreyas" and align on the tone and meaning before publishing it.
thanks

Hey, @Shreyasldl can I work on this issue

This repo has been closed.

Thats sad😕


line protocol + UDP

Hi,
do influxdb support line protocol on udp ?
if yes than do you have example ?

Closing in favor of https://github.com/influxdb/influxdb.com/issues/102 on the actual documentation repo


Bring the plugin back alive to a working state

Previously the plugin has failed to deploy giving each time a "unknown command" with the hostname as error output. This PR solves that issue by using another command syntax which was tested carefully to work with FTP and FTPS servers.
I also moved the username option to be a secret instead, to increase the security and minimize a possible bruteforce attack by just hitting the server with a known username.

Hey!
While it does still work for me what you did looks reasonable so I will use it.
Thanks for the username secret!

Thank you, now it works fine again on Drone CI :)
https://build.codebucket.de/LawnchairLauncher/lawnchair.info/18


Recompiling client protobuf definitions leads to errors

I deleted the client_gorums.pb.go and client.pb.go files and then tried to regenerate them with the protocol buffers compiler using the following command:
protoc -I=/Users/raphael/.go/pkg/mod/github.com/relab/gorums@v0.2.3-0.20210213125733-f04667f97266:. \
                --go_out=paths=source_relative:. \
                --gorums_out=paths=source_relative:. \
                client/client.proto

Unfortunately when executing the server I get the following errors:
generating(async): ExecCommand
# github.com/relab/hotstuff/client
client/client_gorums.pb.go:194:42: msg.metadata.MethodID undefined (type *ordering.Metadata has no field or method MethodID)
client/client_gorums.pb.go:1139:44: req.metadata.MethodID undefined (type *ordering.Metadata has no field or method MethodID)
client/client_gorums.pb.go:1323:2: undefined: ClientClient
client/client_gorums.pb.go:1328:19: undefined: NewClientClient

I've tried compiling with different versions of gorums as well (v0.2.2 and v0.2.1) but same result. It's weird because I didn't have those problems a couple weeks ago when I did the same.
The generated client_gorums.pb.go file also seems to be way larger than the one in the git repo when just cloning (~200 lines vs ~1400 lines).

We are currently using the master branch of gorums, which has seen a bunch of changes since the last release.
@meling we should probably do a new release of gorums now.

Yeah. Will try to do it later tonight.

Have tagged a new pre-release (v0.3.0). Let me know if this helps.

I downloaded the gorums-0.3.0 and put it into a sibling directory of the hotstuff directory. I then recompiled with the following command:
protoc -I=../gorums-0.3.0:. \
  --go_out=paths=source_relative:. \
  --gorums_out=paths=source_relative:. \
  client/client.proto

I did not change anything in client.proto and the errors are still the same. For some reason it also generates a correctable (not async).
generating(correctable): ExecCommand
# github.com/relab/hotstuff/client
client/client_gorums.pb.go:194:42: msg.metadata.MethodID undefined (type *ordering.Metadata has no field or method MethodID)
client/client_gorums.pb.go:1139:44: req.metadata.MethodID undefined (type *ordering.Metadata has no field or method MethodID)
client/client_gorums.pb.go:1333:2: undefined: ClientClient
client/client_gorums.pb.go:1338:19: undefined: NewClientClient

I'm willing to accept that I've configured something wrong somewhere but I can't figure out where. Does anyone of you get the same errors when trying to recompile the protobuf definitions in the manner I did?

I think the problem is a mismatch between the gorums version imported by hotstuff and the gorums version installed in the PATH. This commit in gorums changed the numbers for some of the method options, which explains why it is generating a correctable instead of an async QC. I have updated the hotstuff master branch with gorums v0.3.0. I think that should fix your issue.
@meling this might create problems for others too, any thoughts on how to document or handle this?

Sorry for causing problems here. We haven't had many external users so far, so we have favored cleaning up the APIs without worrying too much about compatibility since we are still in pre-release mode. We will help as much as we can to fix things when they break. That said, I'll think about a way to detect incompatible versions between the plugin and the package; they do something like that in the golang-protobuf library. But I'm not sure if their approach translates easily to our case since we haven't made any v1 releases yet.

No problem at all. Thanks for the help. It works now :)
I guess I'm your reality check; an outsider who uses your library in ways it maybe shouldn't be used.


$databaseName is invalid:

in.env
DB_CONNECTION=mongodb
DB_HOST=<IP_ADDRESS>
DB_DATABASE=test
DB_USERNAME=
DB_PASSWORD=
but
InvalidArgumentException in Database.php line 66:
$databaseName is invalid:
at Connection->__construct(array('driver' => 'mongodb', 'host' => '<IP_ADDRESS>', 'port' => '27017', 'database' => null, 'username' => null, 'password' => null, 'options' => array('database' => 'test'))) in MongodbServiceProvider.php line 27
why 'database' => null?

in config/database.php
'database' => env('test'),
I changed this to
'database' => 'test',
Then it's okay. Why?

I think the right way is:
-- config/database.php
'database' => env('DB_DATABASE', 'test'),
It means that if DB_DATABASE doesn't exist in your env, Laravel should use 'test'


I modified map.html able to make a prototype map that finds the closest school based on distance

I modified map.html to now be able to make a prototype map (in javascript) that finds the closest school based on distance its called mapsFindClosestSchool.html (I removed the districts portion for moment but that can be added in later).  You can see results on https://docs.google.com/document/d/14Uuel-v0EiUFEDfo9Y_ULLMwOhSc-rOg4Kofflrtpyw

Have you looked into Turf? That is probably the easiest way to find the closest school? http://turfjs.org/static/docs/module-turf_nearest.html

Hey, great... can try that next.
On Thu, Jan 29, 2015 at 9:33 PM, Devin Clark<EMAIL_ADDRESS>wrote:

Have you looked into Turf? That is probably the easiest way to find the
closest school? http://turfjs.org/static/docs/module-turf_nearest.html
—
Reply to this email directly or view it on GitHub
https://github.com/codeforokc/school-finder/pull/38#issuecomment-72148970
.


Where is batch norm controller?

Hi~! Could you please point out where is the batch norm controller code in this repo? I didn't find it.
Thanks!

https://github.com/TorchSSL/TorchSSL/blob/main/train_utils.py#L388


Upgrade FAKE version to support VS2017

paket update nuget FAKE group build

@ReedCopsey Can we merge this?  (I'm looking at this TP as a god test case for generative TPs  and noticed this is needed?)


"Negești" is listed in the drop-down menu

Description:
"Negești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Negești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Negești, Argeș (Cotmeana)".


Add build metadata collection

As we are pushing projects to use readthedocs.yml, we are losing the ability to query our database for builds that should be using a certain build container, python version, etc. Having a table that tracks the metadata for each build would be helpful for users as well as for debug and making design decisions on our side.
Some of the metadata we might track:

Build image container
Build image SHA
Python version
Requirements file
I guess most of the settings in readthedocs.yml, though we also want this for historical build metadata that is in the database

Anything that we might want to show to the user on the build output page could be useful to track.
There are a number of ways to address this from the technical side. Concerning storage:

We could make a relation table with fields for each piece of metadata we want to track, though i think we'll want to shift this metadata fairly frequently
We could make and EAV style table for storing this data, though I'm generally 👎 on that pattern
We could store the metadata as a JSON blob

I lean towards JSONField, as we don't have to worry about the schema of what we're storing. Metadata will be mostly used for some display, querying all builds/projects will be more expensive but very infrequent. We might, for instance, want to query to see how many projects are using our latest build image, but that will be rare.

I agree with this idea in general.
I also prefer JSONField because the reason you mentioned and besides because it's easy to make queries and rely completely in the db instead of end writting hacky chunks of Python to query what we want.
Creating a relation table, will involve more work on maintaining it (adding new fields / removing olds / making mistakes on the data type, etc). I prefer to avoid this.
This raised me more questions:

is YAML fully translatable into JSON?
what we are going to do while we support both Admin UI + YAML for settings? We can't just dump the YAML into the db, we will need to dump the mix of them (exactly what we are using today for the build).

Relying on the data from the JSONField to display things to the user under the Build page makes me a little of noise on "relying on data that maybe is not available / chaged the scheme" and won't be easy to notice.

YAML -> dict -> JSON should be fine as we don't have any odd types in there like datetime.
Fun unrelated fact: python's yaml can decode json files. The opposite isn't true though.

Relying on the data from the JSONField to display things to the user under the Build page makes me a little of noise on "relying on data that maybe is not available / chaged the scheme" and won't be easy to notice.

All of this will need to be conditional in the templates, yeah. That is a downside to schemaless here. Writing these templates might be annoying, but we'll have legacy builds that are missing this data to worry about as well.

We are already saving the config used on the db https://github.com/rtfd/readthedocs.org/pull/4749, I guess we can close this.


When using Ropsten testnet for ETH, "Could not create transaction, invalid sender" is returned when sending ETH.

When the IOS BRD wallet code is modified to point to the Ropsten testnet, there is a present
behavior that all send operations of ETH are resulting in the following error "Could not create transaction" "invalid sender". I re-tested this with a local build which previously passed testing using Ropsten, and this version has experienced the same results.
please note I have not tested this using the official BRD IOS Ropsten build, so I would like to confirm that this is not some problem with my local environment, or application.
I first inquired to the BRD support email, and they have asked that I open an issue on this.
I look forward to any further information that may become available regarding this error.

Thanks....
I have built with the testnet scheme, and I have a version of the BRD wallet built that passed testing on Ropsten previously....(I have not rebuilt this)...and this version gets the same error...
I appreciate any further feedback you can provide...

hello team...
I was able to resolve this issue by doing the following....turn off the test net settings using the toggle test net script, then set the logic in the BRD wallet to always use ropsten, and then also set the main net to become 3....when these steps are completed then transactions complete successfully on the ropsten testnet in the deployed BRD wallet.....


Admonitions possible?

Is it possible to anyhow include Admonitions in the markdown?
One Implementation i really enjoy is the one Docusaurus uses:

Hello @CuzImBisonratte, I just realized, we never gave you feedback on this one. This issue has found its way into our ticketing system, so we keep track of it. It should be possible to deploy this - it will be a question of time. Thank you for this great idea! :)
Kind regards,
Claudia H.


add ignore file to exclude logs

fix commit  username

Codecov Report

Merging #38 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #38      +/-   ##
============================================
+ Coverage     63.11%   63.15%   +0.03%
  Complexity      584      584
============================================
  Files           263      263
  Lines         11206    11206
  Branches       1872     1872
============================================
+ Hits           7073     7077       +4
+ Misses         3160     3157       -3
+ Partials        973      972       -1

Impacted Files
Coverage Δ
Complexity Δ

...com/alipay/sofa/rpc/context/RpcRuntimeContext.java
65.62% <0%> (+1.04%)
0% <0%> (ø)
:arrow_down:

...pay/sofa/rpc/transport/ClientTransportFactory.java
65.15% <0%> (+1.51%)
0% <0%> (ø)
:arrow_down:

...lipay/sofa/rpc/codec/antpb/ProtobufSerializer.java
50.72% <0%> (+2.89%)
9% <0%> (ø)
:arrow_down:

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 29ef6c4...aa547d8. Read the comment docs.


refactor(devimint): run DKG using the apis

devimint is supposed to test binaries, but currently DKG is using fedimint as a library, making it harder to run tests with custom fedimintd binaries.
This change drops this "library" usage and makes DKG runs as api calls to the underlining fedimintd process.
One difficulty of implementing this is that we don't know before DKG what will be the peer ids. But we need to start everything with the conventional names and configurations.
So it's tricky to set up different passwords for different peers (we don't know exactly which fedimintd will be which peer). To make things simple, we are now using the same password for all the fedimintds.

One difficulty of implementing this is that we don't know before DKG what will be the peer ids

Because DKG sorts TLS certs to produce peer IDs

@elsirion added a poll_max_retries which matches more closely the desired behavior


renamed cargonauts everywhere to nauts (don't merge yet)

We still need to fix the twitter account

@arminc Are there still things we need to do for this?


Add support for team pr review requests

This PR adds support for team pr review requests. Added a test that covers individual review requests but I'm pretty sure the github-api-test-org will need a team added for me to test team review requests.

Hey @bitwiseman - any chance of getting some PR review here? :)

@farmdawgnation
I know it's a bit to ask but I've just added a mocking framework that will let us run tests in offline mode in CI.
I've resolve the merge conflict, but haven't done a build or further test.
Could you try the instructions in CONTRIBUTING.md for creating new tests and  snapshots of github response data?  It is still a little rough around the edges, but I think it will be a big step forward for testing and stability.  I've invited you to be a collaborator in the test repo that we're using.   I think that's all you'll need from me.  Thanks in advance.
If you don't have time, I'll get back to it later this week.

Could you try the instructions in CONTRIBUTING.md for creating new tests and snapshots of github response data? It is still a little rough around the edges, but I think it will be a big step forward for testing and stability. I've invited you to be a collaborator in the test repo that we're using. I think that's all you'll need from me. Thanks in advance.

I can give it a shot, but I may first attempt to write a test using the existing format for time's sake.

@bitwiseman I've gotten the wiremock data in place for the existing test I wrote and tests now pass locally for me. However, because I was added as an outside collaborator on a single repository it seems I don't have the ability to see what teams exist in github-api-test-org or tag them in reviews so I still can't write my test for the team pr reviews feature.

Could you please add a team that has the kohsuke2 user in it and make it so that I can see it?

Hm my tests are failing in CI, but they work locally. Seeing this in the CI log:
org.kohsuke.github.HttpException: Server returned HTTP response code: -1, message: 'null' for URL: http://localhost:63049/repos/github-api-test-org/github-api/pulls/298
	at org.kohsuke.github.PullRequestTest.testPullRequestReviewRequests(PullRequestTest.java:118)
Caused by: java.net.ConnectException: Connection refused (Connection refused)
	at org.kohsuke.github.PullRequestTest.testPullRequestReviewRequests(PullRequestTest.java:118)

What did I do wrong here?

@farmdawgnation
Sorry for slow response.
I've invited you to be an owner on github-api-test-org.  That will give you rights to do whatever you need to.
Try deleting your wiremock data, rebasing/merging from master, and running again.  I've made a bunch of fixes and improvements to the framework.

@farmdawgnation
FYI, I intend to get a new version published in the next week or two.  I look forward to including this change.

Word, I'll try to get it in :)

Yay! tests pass!


Add MethodResponse utility type

Changes
Adds a new MethodResponse utility type to openapi-fetch to easily reference the return type of a fetch call in your code
How to Review
Understanding the rationale
There are many times in a codebase where you do something like this:
async function getData(...data): Promise<Data> {
   // 1) Process the data in some way
   // 2) Do some other async calls like DB queries or querying other services

   const { data, error } = await client.GET('/data', {});
   if (error != null) { /* handle case */ }
   return data;
}

The return type of this function is based solely on the data endpoint result, so it would be nice to make this relation explicitly clear
Before this PR
The easiest way to do this is the following code, is the following code
async function getData(): Promise<Awaited<ReturnType<typeof client.GET<'/data', {}>>>>

This has a problem though: if client isn't statically known as a global variable. In that case, you cannot get around this by just doing Client<paths>['GET']<'/data', {}> as this is not valid typescript
After this PR
This is the new code which is both shorter, and also works well even if typeof client is not possible and a type variable is used in its place
async function getData(): Promise<MethodResponse<typeof client, 'get', '/data'>>
// this also works
async function getData(): Promise<MethodResponse<MyClient, 'get', '/data'>>

Why not just use schemas?
Another option is to use components['schemas']['GetDataResponse'], which is also valid and just a matter of preference. I think being able to explicitly specify the connection between a method and the response is more flexible than this
For example, you could create a wrapper on the type I added to this library (which I don't think you can easily do before this PR), by doing something like
type GetType<Path extends PathsWithMethod<typeof client, 'get'>> = MethodResponse<
  typeof client,
  'get',
  Path
>;

// you can now use it like this
async function getData(): Promise<GetType<'/data'>>

Additionally, this components['schemas'] option isn't always guaranteed to work. For example, there are cases where the schema is partially inlined into the operations object:
export interface operations {
    GetData: {
        // ...
        responses: {
             200: {
                 content: {
                     "application/json": {
                        dataSummary: components["schemas"]["GetDataResponse"][];
                     }
                 }
             }
        }
    }
}

As you can see in this example, you cannot just use GetDataResponse directly because it's missing the fact it's actually contained inside a dataSummary object
Code
This is just a utility type, so there is no functionality change
Checklist

[x] Unit tests updated
[x] docs/ updated (if necessary)
[x] pnpm run update:examples run (only applicable for openapi-typescript)

Thank you! You can add a patch Changeset if you like (see comment) so you’re credited with this.
Thanks for such a detailed writeup! You made a great argument for adding this utility, and this is a great addition.

Why not use SuccessResponse from openapi-typescript-helpers?
async function getData(): Promise<SuccessResponse<paths['/data']['get']>>

@piousdeer MethodResponse uses SuccessResponse under-the-hood. It basically wraps that utility type to allow you to pass an arbitrary client as the source of the information (instead of having to pass in a specific paths type)


Error display doesn't appear to respect each input's "touched" state

Expected result: Validation errors should only be shown for components which have been interacted with - focused/edited/etc.
Actual result: All form validation errors are shows the first time validation is run, even if this is caused by focusing and blurring a single field without editing it.
Please see repro testcase here: https://codesandbox.io/embed/react-formik-mui-testing-4pn0p

@AubreyF
Should be fixed in version 0.9.5.
Im gonna try it out in your codesandbox as soon as the package is available on NPM

Thanks @mvitzthum!
I just confirmed that the validation system in 0.9.5 is working as expected.


feat: better card styling added

added prop cursorType
added hover transition animation
fixed position of tooltip and selected icons

Nice!


ERROR 500 WHEN TRYING TO EXTRACT DATA

063-01X-8096-01_R1_WM0.pdf
063-01X-0106-01_R1_WM0.pdf
I have been able to extract the data from the table on the right of these type of drawings before but I was trying to do it on a new batch and it kept giving me this error. I was able to find the first file that is cause the issue (attached - 063-01X-8096-01) but I can't see the difference between that corrupt file and the others, thus I can't extract them and there are too many to try one by one. I also attached one drawing that works (063-01X-0106-01)

I've had this happen to me as well. I've currently spent the whole day trying to debug this error:
(NoMethodError) undefined method []' for nil:NilClass`
Is there a way to get a more detailed stacktrace?

@rosenjcb You'll want to run Tabula from source to get the full stacktrace. Once you get it, please share it here; an error like you include above shouldn't be happening at all.

@jeremybmerrill I actually build both tabula and tabula-java from source (I've forked each repo). I get the attached image as my error and nothing in the jetty console besides ***[{"page":1,"extraction_method":"guess","selection_id":"F1536100935342","x1":44.94599395751953,"x2":772.5959939575196,"y1":140.085,"y2":262.84499999999997,"width":727.65,"height":122.76}]!!!!!DEBUG: resetting rack response due exception

Right, if you run it from source with jruby -G -r jbundler -S rackup rather than the Jetty / warbler stuff, you'll get a real stacktrace in the console. (That jruby command is what I use whenever I'm developing on Tabula; the warbler stuff is a far more complicated way of packing the app up for distribution which involves compromises -- like less visibility of error messages.)

@jeremybmerrill It worked! That is a much better stacktrace than what I was getting before.  I was running the server from jar. After reading the instructions again, it seems that running  java -jar build\tabula.jar is for testing the package when production ready. From now on I'll use the rackup command when developing and debugging. Thank you.

Super. If you have recommendations on how to make the README clearer, I'm all ears. LIkewise if the bug you find is rooted in mainline Tabula code (as opposed to y'all's modifications).

Does this mean the error is fixed now?
On Wed, Sep 5, 2018, 1:06 PM Jeremy B. Merrill<EMAIL_ADDRESS>wrote:

Super. If you have recommendations on how to make the README clearer, I'm
all ears. LIkewise if the bug you find is rooted in mainline Tabula code
(as opposed to y'all's modifications).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tabulapdf/tabula/issues/908#issuecomment-418826410,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdX5K2tB20x7L1Fr81g9qU3lkmwkp0cFks5uYBKlgaJpZM4V_ttV
.

@samapera3478 No, it just means I co-opted your thread. You should try compiling the project from source and running the jruby -G -r jbundler -S rackup command for a better stacktrace. Then comeback and let us know what error you see.

I'm sorry I'm not a developer, I have no clue what that means. I just use
it to extract some info from drawings to make our lives a little easier
On Thu, Sep 6, 2018, 11:12 AM rosenjcb<EMAIL_ADDRESS>wrote:

@samapera3478 https://github.com/samapera3478 No, it just means I
co-opted your thread. You should try compiling the project from source and
running the jruby -G -r jbundler -S rackup command for a better
stacktrace. Then comeback and let us know what error you see.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabulapdf/tabula/issues/908#issuecomment-419152890,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdX5K0jzOJxXNw-JQ6MBFUUCaurxBh22ks5uYUl9gaJpZM4V_ttV
.


[Feature Request] Saving to and loading from a cloud storage provider

I use this app on my phone while commuting, and on my desktop PC at home. Currently, transferring a between these two devices involves:

Saving locally from Exet
Uploading to a cloud storage provider (google drive, dropbox, etc.)
Downloading to the other device
Uploading to Exet

This process could be simplified by integrating a cloud storage provider directly into the app.
As an example of how this feature could work (using dropbox):

On each device, Exet authenticates with dropbox (OAuth, or perhaps simple access tokens)
Exet then uses the '/exet' folder in the user's account to store saved puzzles
The list of saved puzzles is displayed and can be loaded directly
An open puzzle can be saved back to dropbox (perhaps using revisions?)

I'm more than happy to help implementing this feature, if you feel it would be useful.

Hi!
First off, I'm amazed that you've been able to use Exet on the phone! I've found it quite clunky, esp, with the keyboard mostly obscuring everything. But perhaps you use some nicer/different/custom set-up. It would be great if you drop a usage note in the Discussions area about that, if there's anything noteworthy.
As to the feature request itself: I think that this is probably not core-crossword-related functionality, and also your specific use case--at least at this time--is not very common. As you might have noticed, my coding philosophy for web apps is to absolutely avoid dependencies on packages/frameworks etc. Of course, I may be a bit over-the-top in that position, but I do think that makes the project more stable over time, more easily reusable/modifiable, less brittle. So, on balance, I think that adding all this authentication and syncing code into Exet is not something I would want.
That said, I think the problem that you're facing is fundamentally a file-sharing problem, not an Exet/crossword problem. Whatever coding you intended to do, can, I am guessing, can also be done in the context of a stand-alone app/web-app that lets me sync some chosen directory (say my Chrome/Firefox/whatever Downloads folder) across devices, using Google Drive or Dropbox or whatever. That would be super-general and useful across many apps. Then, from your phone Exet, you save the current state by downloading a file to the Downloads folder (as is already possible), and that file magically appears in the Downloads folder of the browser on your other computer, and you can load it up in Exet from that browser.
Thanks,
Viresh

I'll close this bug in a day or so, unless there's a follow-up discussion.


SAMNet draft

This is the first version of SAMNet based on the description in the paper.
It is not yet nemo-compliant as there are no ports defined as yet.
The top-level file for SAMNet is ./samnet/model.py

Picking up development on https://github.com/aasseman/visual-reasoning/tree/feat/samnet


Content/data representation

This branch also has @Jamesernator's interactives that were made for the data representation chapter in it. Is that okay?

Note that this has some incomplete work in it (the colours guide).
It is not actually linked into the guide though, so should be okay

@Heidirose Thanks for letting me know, for the sake of sanity we will leave it there. I'm going to do one last comb through and check that everyone works as I've fixed most of the bugs, but some fixes may have broken other content.


How to load PB file saved by Keras model ?

Hi , I had recently trained a model using Keras.
I was able to save the model using model.save(location).
The generated files are assets, variables, keras.metadata.pb , saved_model.pb .
How do I load these model data into tensorflowsharp?
Thanks.

string imgModelPath = System.IO.Path.Combine(assemblyDirectory, "models\img\" + "saved_model.pb");
// Load Image AI
await SimulateSystemTyping("Loading Image AI model please wait...");
await Task.Run(() => ImgAI.StartAI(imgModelPath));
private static TFGraph graph;
public static async Task StartAI(string modelPath)
{
try
{
// Load the pre-trained model
graph = new TFGraph();
graph.Import(File.ReadAllBytes(modelPath));
}
catch (Exception ex)
{
// Handle the exception (log, display a message, etc.)
MessageBox.Show($"Exception: {ex.Message}");
}
}


Can't find the way to resize the output map

Hi everyone. You're package is brilliant! But there is no way to change the output map size. E.G. in Jupyter Notebook it is quite small. How can i do that?

This lib does not output any maps. Are you sure that you use python-gmaps in your Jupyter Notebook? Can you share your code?

I close this issue. Feel free to reopen when you will be able to provide more feedback.


Let ClassDB::add_property use getter/setters with optional params

Fixes https://github.com/godotengine/godot/issues/43821
I referenced https://github.com/godotengine/godot/pull/25018 to determine what changes to make.

I'd like to cherrypick this into 3.2 so I can use it in my project, so if this gets approved I will submit a PR for that.

This unfortunately makes language binding more complicated as languages can no longer expect the setter/getter functions to always have one way of calling. As this is not that much of an useful feature and makes binders more difficult, I am against merging it.

This unfortunately makes language binding more complicated as languages can no longer expect the setter/getter functions to always have the same form. As this is not that much of an useful feature and makes binders more difficult, I am against merging it.

Okay, that makes sense. I originally thought that these changes were a self-contained fix for this limitation. Seeing how this is infrequently used (and has an easy enough workaround https://github.com/godotengine/godot/issues/43821#issuecomment-733850750), I'll close this.

No problem, still thanks for taking the time to do the contribution!


Display icon and size for non image?

flagrow/upload extension version: 0.4.12
flarum version: 0.1.0-beta.6
Upload adapter causing issues:

--
Great plugin as always. I've managed to upload an mp4 successfully and get a URL successfully. For non images, would it be possible to display an icon and size of file?

Love the idea!

Size added, icon is still a todo.


kafka role

[x] Installs cleanly on a fresh build of most recent master branch
[x] Upgrades cleanly from the most recent release
[x] Updates documentation relevant to the changes

Installation documentation available in the role readme. Note that you will need at least 3 workers each with 1 CPU and 4 GB of memory available to Mesos in order to run with the default configuration.
This depends on https://github.com/CiscoCloud/nginx-mantlui/pull/18 being merged and the 0.6.8 tag of ciscocloud/nginx-mantlui being pushed to docker hub.
After successful Kafka installation (it can take a few minutes), verify the following:

that the mantl/kafka scheduler is running and healthy in Marathon
that there are 3 Kafka brokers running in Mesos (broker-1, broker-2, broker-3)

Once the brokers are up, you should be able to produce and consume data. Here is one example of a manual test:
# create a topic
curl -sku admin:password -d "topic=journal" https://control-node-01/kafka/api/topic/add

# on one node, pipe journald logs into the journal kafka topic
sudo journalctl -f | sudo docker run --rm -i ryane/kafkacat -b broker-1.service.consul:9092,broker-2.service.consul:9092,broker-3.service.consul:9092 -t journal -z snappy

# on another node, consume the events from the  journal topic
sudo docker run --rm -it ryane/kafkacat -b broker-1.service.consul:9092,broker-2.service.consul:9092,broker-3.service.consul:9092 -t journal

We can see if the tag has been pushed on the nginx-mantlui docker hub page.

0.6.8 tag of the nginx-mantlui image has been published

We probably don't want to use zookeeper.service.consul, as the zk client does not refresh DNS entries, so we lose benefits of RR dns. The zk hosts should be enumerated.

@stevendborrelli

the zk client does not refresh DNS entries
THAT explains a lot... :)

kafka will start using the list of zk hosts once #1308 is merged


Teams prefixes not visibile on 1.12

Team prefixes are not visible on 1.1.2 clients, 1.11.2 and lower version can see prefixes without problems.
What is the output link of /viaversion dump?
https://gist.github.com/anonymous/18b31e9f65873e3bb6b2827f6f8c2ce3
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
Today tested without proxy
How does this error happen? login? Using an item?
Huh, this is not implemented feature rather than error
Is there an error in the console? Use pastebin.com. Is there a kick message?
No

Could you provide screenshots? What you expect vs what happens?
Is this only on 1.12?

Team prefix '&7' works correctly on 1.1 client, but with '&4&lADMIN &r' prefix looks like not modified.
Server software: PaperSpigot 1.8.8
Client: Minecraft 1.11.2
Result: '&7' and '&4&lADMIN &r' is displayed correctly
Server software: PaperSpigot 1.8.8
Client: Minecraft 1.12
Result: only '&7' is displayed correctly
plugin is using Bukkit.getScoreboardManager().getMainScoreboard()

Oh, It's works like you remove all players from teams and remove teams, then you create teams and add players. Then users with 1.12 client must rejoin to see you own prefix.

Could you post the full code you're using?

Unfortunately not, code is depending on many api classes from core plugin. Core plugin cannot be given to others. I create version for testing with hardcoded settings and I put it in repository on github.

https://github.com/Inuun/TestingTablist
First step: install plugin from 'target' dir
How to reproduce:

join with 1.12 minecraft
see tablist
type /tablist
see tablist (only your nickname is white)
join with 1.11 minecraft
type /tablist
see tablist

I can replicate part of your issue but not all of it, when using the plugin provided (thanks for providing it all much appreciated!) I can only see the tab info on 1.8.8 client, all other versions don't display anything.

hmm, I tested it with ProtocolSupport also

I had ProtocolSupport installed at the time of testing

This server is running CraftBukkit version git-PaperSpigot-"8b18730" (MC: 1.8.8) (Implementing API version 1.8.8-R0.1-SNAPSHOT)
https://image.prntscr.com/image/eNvV3joCTbqq8OhcO1RQFw.png
huh, join <IP_ADDRESS>:77

Tested again. You are right, 1.8.8 client see own name in colors after "reload". Using 1.11.2 and 1.12 you see white nickname.

So this is possibly a scoreboard issue between 1.8.8 and 1.9 then? :)

Probably yes, but is good idea to test it without viaversion on 1.11 spigot. I test it and write about result.

Confirmed, this is issue with ViaVersion. On 1.11 server with TestingTablist after typing /tablist I see my nickname in colors.

As a workaround you can just disable ViaVersion autoteam, /viaversion autoteam. Will hopefully have a proper fix out for this soon :) (But yeah it's to do with our teaming stuff!)

Should be fixed in dev builds while still maintaining functionality. :)


Wager Hut - Coinflip

While waiting until a quest is finished it would be great to do something. E.g. a simple coinflip where you can place a bet with amount x of uruks to head or tail. You could call it "Wager Hut".

Hey moon, nice to see you adding feedback here :)
A separate section called Wager Hut sounds good, but we would call it "Tavern" instead and add more functionalities to it, not only this, but could be a great start, we actually didn't think about this kind of mechanic and section (the Tavern). But we will add it internally in consideration for next sprints :)
Keep in mind that quests are "boring" now and there are not that much stuff to do while doing them, but this will change as we keep adding stuff
See you!


Add Wish Tally excel import

Add import support for Wish Tally 3.0 excel files.
Fixes #145

If you're okay with the changes, I'd appreciate if you could label this PR with "hacktoberfest-accepted".
And consider adding the "hacktoberfest" tag/topic to this repository. Thanks! =)

Example file: Genshin Impact - Wish Tally v3.0.xlsx =)


CI Images: Don't push floating tags from feature branches

Floating tags like "latest" or "v1.14" should only be pushed from main and stable branches.

i guess it does what we want, but it doesn't feel very elegant. requesting review from @nbusseneau to see if he has any suggestions 🚀 🙏 ✨

/test

/test


Card Text Muted Variable

/* Cards */
$card-spacer-y: 0.625rem;
$card-spacer-x: 1.5rem;
$text-muted: $text-secondary;
$card-header-font-size: 1rem;

Is the $text-muted specific to the cards?  Or the entire DS?

I'm not sure if we're still using this variable? I started removing it in some areas where the color could just be applied to the element as far as the cards go to prevent too many classes for a global style. E.g. .ds-card-header.  Thoughts @briansaycocie?

@RachelRVasquez we will definitely want a muted class that we can apply to text elements.  Promotions That Give Back is a good example.  Where we have a special case for one of the card's headers to be a subtle note.  A muted class would eliminate the need for a variation in the cards css.
@briansaycocie

Merged. @jdrf-digitaldesigner @pmmck Please review.

@fuhton I see the change where .text-muted has been added but i'm still seeing this variable grouped with the Card variables, even thought it's not specific to that component.

@fuhton PR: https://github.com/JDRF/design-system/pull/239

@pmmck This has been merged. Please review.

Perfect, thanks.


Terrafrom destroy asking for provider specific info.

Hello ,
Earlier i have raised an issue for terragrunt
https://github.com/gruntwork-io/terratest/issues/503
But now the issue is while destroying ,it is searching for the provider specific information again . which normally reads from commons.tfvar or environment variable file for me during init and apply . It is throwing below error .
TestOci 2020-04-17T09:11:40+05:30 command.go:158: [terragrunt] 2020/04/17 09:11:40 Running command: terraform destroy -lock-timeout=20m -auto-approve -input=false -lock=false
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: Error: No value for required variable
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:   on variables.tf line 3:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:    3: variable "user_ocid" {}
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: The root module input variable "user_ocid" is not set, and has no default
TestOci 2020-04-17T09:11:44+05:30 command.go:158: value. Use a -var or -var-file command line argument to provide a value for
TestOci 2020-04-17T09:11:44+05:30 command.go:158: this variable.
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: Error: No value for required variable
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:   on variables.tf line 4:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:    4: variable "fingerprint" {}
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: The root module input variable "fingerprint" is not set, and has no default
TestOci 2020-04-17T09:11:44+05:30 command.go:158: value. Use a -var or -var-file command line argument to provide a value for
TestOci 2020-04-17T09:11:44+05:30 command.go:158: this variable.
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: Error: No value for required variable
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:   on variables.tf line 5:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:    5: variable "private_key_path" {}
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: The root module input variable "private_key_path" is not set, and has no
TestOci 2020-04-17T09:11:44+05:30 command.go:158: default value. Use a -var or -var-file command line argument to provide a
TestOci 2020-04-17T09:11:44+05:30 command.go:158: value for this variable.
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: Error: No value for required variable
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:   on variables.tf line 6:
TestOci 2020-04-17T09:11:44+05:30 command.go:158:    6: variable "region" {}
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: The root module input variable "region" is not set, and has no default value.
TestOci 2020-04-17T09:11:44+05:30 command.go:158: Use a -var or -var-file command line argument to provide a value for this
TestOci 2020-04-17T09:11:44+05:30 command.go:158: variable.
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: [terragrunt] 2020/04/17 09:11:44 Hit multiple errors:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: exit status 1
TestOci 2020-04-17T09:11:44+05:30 retry.go:80: Returning due to fatal error: FatalError{Underlying: exit status 1}

We can't help debug this much without seeing the code. All I can say for now is that:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: Error: No value for required variable
TestOci 2020-04-17T09:11:44+05:30 command.go:158:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: on variables.tf line 3:
TestOci 2020-04-17T09:11:44+05:30 command.go:158: 3: variable "user_ocid" {}

That error implies you have a user_ocid input variable defined in your Terraform code and you are not passing in a value for it. There are many ways to pass in a value, such as via a Terragrunt inputs = { ... } block, env vars, terraform.tfvars files, etc... So we'd have to see your code to help more.


Remove duplicate code using loop

PR with looping without making array of keys. Created to compare speed with #85 using majian

@bugagashenkj can you please rebase this on master then review PR with @lundibundi


Additional Emogies

Sorry about this, I know it's your second attempt but I've made some changes.
Could you raise a new PR but use git to do it instead of the GitHub UI and add some more cool emojis into the list in the function? That would be great.
The git CLI can take a little getting used-to so here are all the commands you need, please pay attention to the git commit -s instruction:
mkdir ~/go/src/github.com/alexellis/
cd ~/go/src/github.com/alexellis/

git clone https://github.com/alexellis/my-fn/

cd my-fn
git remote add fork https://github.com/sourabhdeshmukh/my-fn

git config user.name "Your Name"
git config user.email "Your GitHub Email"

git checkout -b add_emoticons

# change the files

git add .
git commit -s -m "Your message here" # Do not miss `-s`

git push fork add_emoticons

# Now raise a PR from add_emoticons to my repo in master

Message I received after performing above steps and

git checkout master
git merge  add_emoticons
git push
Permission denied : remote: Permission to alexellis/my-fn.git denied to sourabhdeshmukh


Cache JS inferred class type symbol

Note that many sources merge into a single target, so the source [links] is the one that caches the merged target.
The reason this is a problem is not that many sources merge into a single target, but that both getTypeOfSymbol and getDeclaredTypeOfSymbol
end up calling mergeJSSymbols with the same [source,target] pair. The merge should not happen twice.
Fixes #32997

Uggg breaks chainedPrototypeAssignment.ts
A.prototype = B.prototype = { ... }

I'm not convinced this is sane code, though. Accepting the baselines for now while I think about it.

I thought about it. It's not real code, just a test case I came up with, so I don't feel bad about breaking it.

Adding @andrewbranch and @orta in case they want to think about caching in the compiler. =)

I'm not convinced this is sane code, though. Accepting the baselines for now while I think about it.

I think the new behavior is arguably more sane - references to both A and B now both reflect the same symbol.
Actually, wait.... they're still different constructors, just with separate prototypes, which means... ick. If A and B have differing signatures, this is going to wholly substitute A with B - ick.
@sandersn I think the inferredClassType cache may need to be a map with the appropriate target symbol id, so the same source can be merged into multiple targets.

@weswigham mind taking another look?

Hooray! My insane test case continues to work~~ :alien: :squid: :crab:


issue 16 fixes

There used to be a namespace of  DialInno\Jaal\Core which is correlated to the src/Core folder
The Core folder has been removed and everything inside has been moved one directory level up so most name spaces are prefixed with DialInno\Jaal\ now.
Errors have been moved inside Objects and Exceptions has been moved outside in the root directory(src)
Since there is only one provider class, JaalProvider.php, it  has been moved to the root of the src directory. That being said, the service provider registration in laravel projects in the /config/app.php file
will have to be updated to DialInno\Jaal\JaalProvider::class.
Since all class namespaces were updated, this resulted in unit tests not running properly. Unit test class namespace references have been updated to the new ones. Unit tests are running now and all passing, minus one test that i commented out for now. It was a test that tested the validate method on the doc object, but since that has been reworked as the ValidatesApiRequests trait, it has been commented out for now.

Coverage increased (+0.3%) to 52.964% when pulling<PHONE_NUMBER>d0b6e39e83ea7296187f67cf889edc on surgiie:issue_16_fixes into 8bdab7688887bf629b48306ee99e555297f2ddae on dialinno:dev.


chore: enable stylecheck

Closes https://github.com/celestiaorg/celestia-app/issues/1989

Codecov Report

Merging #2005 (cc3652c) into main (3aa3c09) will not change coverage.
The diff coverage is 35.00%.

@@           Coverage Diff           @@
##             main    #2005   +/-   ##
=======================================
  Coverage   22.08%   22.08%
=======================================
  Files         130      130
  Lines       14479    14479
=======================================
  Hits         3198     3198
  Misses      10981    10981
  Partials      300      300

Impacted Files
Coverage Δ

app/validate_txs.go
0.00% <0.00%> (ø)

test/util/testnode/config.go
0.00% <0.00%> (ø)

test/util/testnode/node_interaction_api.go
0.00% <0.00%> (ø)

x/blob/types/builder.go
39.67% <0.00%> (ø)

x/qgb/client/query.go
0.00% <0.00%> (ø)

x/qgb/types/genesis.go
78.57% <ø> (ø)

test/util/blobfactory/payforblob_factory.go
18.50% <20.00%> (ø)

x/mint/keeper/genesis.go
75.00% <60.00%> (ø)

test/util/blobfactory/test_util.go
72.72% <100.00%> (ø)

x/blob/types/payforblob.go
72.90% <100.00%> (ø)


Copr: Add DecimalDivide to RPN expression framework

What have you changed? (mandatory)
Migrates DivideDecimal to DecimalDivide
What are the type of changes? (mandatory)

New feature (a change which adds functionality)

How has this PR been tested? (mandatory)
cargo test impl_arithmetic

Looks nice.

In the original test case, the following two also checks whether errors are correctly generated when flag SqlMode::ERROR_FOR_DIVISION_BY_ZERO is set:
            (
                ScalarFuncSig::DivideDecimal,
                str2dec("12.3"),
                str2dec("0.0"),
            ),
            (
                ScalarFuncSig::DivideDecimal,
                str2dec("12.3"),
                str2dec("-0.0"),
            ),

/run-all-tests


mock: Parameterization changes for Vulkan SC

Author aqnuep not on autobuild list. Waiting for curator authorization before starting CI build.

Author aqnuep not on autobuild list. Waiting for curator authorization before starting CI build.

CI Vulkan-Tools build # 1009 running.

CI Vulkan-Tools build # 1009 passed.


Export subscriptions from Google account

I want to use ytui in local mode. Is there a recommended way to export subscriptions from YouTube/Google? Like userscript in browser (ideally, fastest) or bash (or any language really) script to parse YouTube takeout. I just requested YouTube takeout so can't play around parsing data until I actually get it.

I don't know of any sorry, you could try ytui show subscribed but you would need to try to auth with youtube first


Custom active/inactive tabStyle for ButtonComponent

Current behaviour
Can style all tabs indiscriminately but no customisability for active and inactive states.
Expected behaviour
Might be cool to provide an api for active/inactive tab styling to style the ButtonComponent inside
Code sample
https://snack.expo.io/HkIWg-YBQ
Has an example TabBar using createBottomTabNavigator with what it might look like with the options configured.
Included is a BottomTabBar in the components folder which has the changes applied (roughly)
Screenshots (if applicable)

Ran a test in an existing project to see what it could look like. Test was on iOS simulator

@ReganL maybe tabBarOptions is what you're after? I've updated the snack.
<EMAIL_ADDRESS>

@benprebble like tabBarOptions but with a bit more control over the whole Button component that exists in the TabBar. Cheers though!


GitHub authentication not working after upgrade

Rancher versions:
rancher/server or rancher/rancher: v2.0.3-rc2
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Bare-metal
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node
Steps to Reproduce:

have a rancher/server:preview running (version 2.0.0)
configure GitHub authentication
have a few users configured, using the cluster
upgrade to version rancher/rancher:v2.0.3-rc2 keeping the same volume
logout of the UI
try to login again

Results:
Attempting to login using GitHub doesn't work and returns that log in the rancher docker container
2018/06/21 23:36:07 [ERROR] Received Error from github bad_verification_code, description from github The code passed is incorrect or expired.
2018/06/21 23:36:07 [INFO] Error generating accessToken from github Received Error from github bad_verification_code, description from github The code passed is incorrect or expired.
2018/06/21 23:36:07 [ERROR] API error response 500 for POST /v3-public/githubProviders/github?action=login. Cause: Received Error from github bad_verification_code, description from github The code passed is incorrect or expired.

Solution:
rolling back to rancher/rancher:v2.0.2 with the same volume fixes the problem

going to close as dupe of https://github.com/rancher/rancher/issues/14143


Extra calls onNavigationStateChange in WebView on Android

Extra calls onNavigationStateChange when update window.location attributes values in WebView on Android RN - 0.17.
For example:

update value of window.location.hash - 2 calls of onNavigationStateChange;
update value of window.location.search - 3 calls of onNavigationStateChange.

On IOS platform all fine 1 call of onNavigationStateChange in this cases.

Can you submit a pull request to fix this please? :smile:
Otherwise it's not likely to be fixed very soon

I shall investigat and try fix. :)

Awesome @barbarosh! Looking forward to it. Feel free to submit a PR with whatever you manage to figure out and we can take it from there. Any code is better than no code! :)

+1

I faced this problem multiple times since I was intensively using WebView for my application. I've yet to deep dive into the WebView Android wrapping implementation but for now anyone that had this problem can try filtering the events using setState inside onNavigationStateChange and handle shouldComponentUpdate or using some other way of setting flags.
You can also use InteractionManager in some scenarios to go around this problem.

@brentvatne @barbarosh sorry to disturb you but has there been any investigation on this?

@barbarosh just for ping.

try to wrap your javascripts in this block
function main() { your scripts here}
if (window.loaded) {
main();
} else {
window.onload = main;
}

I'm also facing this issue plz suggest when you get resolve it

Another one here ,I ended hooking up  debugging and seeing all the callbacks being triggered. Found that onLoadStart and onLoad was only being called once for the url I wanted to act upon. So switched my logic over from onNavigationStateChange to onLoadStart.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/extra-calls-onnavigationstatechange-in-webview-on-android-1

This is still happening on version 0.50 I believe.

Still an issue in 0.51

Please open a new issue, as the original post in this thread was created prior to us settling on the current template.

Still facing this issue in 0.51

still encounter this problem on 0.51.


[Question] Store data in server side

Hello,
I have tried the Discord link but looks like it doesn't work, so I ask here (maybe another people know about it and can help).
I want to continue improving a bit this project because it looks very good and I think that can be even better
To make it a bit better in the MP part, I want to move all the module logic to the server side. This will allow to avoid that if you have 20 clients in the server, to have to calculate the solar status in all of them. Also will allow to a server that doesn't have activated the pause when no clients are connected, to still calculating the energy status and batteries degradation even when no one is connected.
My problem with this is that I have not found a way to save data in server side in a permanent way to allow the module to continue working in a normal way upon restarts. On client side I know that there is an object called modData which allows to store data premanently in the client, but on server that data is only stored while the server is running. Once is restarted, that data is lost.
My question is just if someone knows a way to store data in server side, because I want to store at least the PowerBanks position in the map. Also a way to locate objects in the map will be valid, because it must be done once at server start. This will allow to scan the surrounded objects to detect the consumers and the solar panels attached to that power bank.
I have asked in the official forum, but or no one cares, or there is no way to do it.
Best regards.

Well,
Maybe my problem is already fixed... about three weeks searching for a way to do it and now just before writing the above I have seen this in the latest version changelog:
Fixed GlobalModData.save() not being called on the server.

I'll do some tests tomorrow and I'll close this if works.
Best regards.

I had no luck, so I still searching for it.

Hey Danixu
This is a quick example for saving data on the server
This needs to go in the client folder
`local onKeyPressed = function(key)
local source = getPlayer(); if not source then return end
if key == Keyboard.KEY_SPACE then
    local sourceId = source:getOnlineID();

	if ModData.exists("MyName") == false then
		ModData.create("MyName")
	end

	local MyName = {}
	table.insert(MyName, 1, "Hello")

	ModData.add("MyName", MyName)

	ModData.transmit("MyName")
	print("sending data")

end

end

function DataCheck()
if ModData.request("MyName") ~= false then
print(ModData.request("MyName"))
end

end
Events.OnKeyPressed.Add(onKeyPressed);
Events.EveryTenMinutes.Add(DataCheck)`
and this needs to go in server folder
`function GotData(globalkey, globaltable)
print("getting data")

if globalkey ~= nil then
ModData.add(globalkey, globaltable)

print("Server Key: ", globalkey)

end
--ModData.transmit("MyName")

end
Events.OnReceiveGlobalModData.Add(GotData)`
It's not 100% but it is the general idea I was working with.

local onKeyPressed = function(key)
local source = getPlayer(); if not source then return end
if key == Keyboard.KEY_SPACE then
    local sourceId = source:getOnlineID();

	if ModData.exists("MyName") == false then
		ModData.create("MyName")
	end

	local MyName = {}
	table.insert(MyName, 1, "Hello")

	ModData.add("MyName", MyName)

	ModData.transmit("MyName")
	print("sending data")

end

end

function DataCheck()
if ModData.request("MyName") ~= false then
print(ModData.request("MyName"))
end

end
Events.OnKeyPressed.Add(onKeyPressed);
Events.EveryTenMinutes.Add(DataCheck)

server invite
https://discord.gg/PUjYKbAY

I can't get the format right in the comments, if you are having trouble with the format let me know and I will upload the files

Thanks!!, I'll give a try to see if I can make it work. If works I'll try to move the logic to the server side and if I can to add the connected devices list.

Global object update does that


yAxis should be put on a collection level metadata in STAC

rather than on catalog level for links as is now

will get fixed in 8b6261e6a36da6140473a5c7d821333891d7cb7e


updated workflow

might be worth updating the original template....
https://github.com/decentralized-identity/template-for-profile-workitems/blob/main/.github/workflows/render-specs.yml


Base.@kwdef

I've noticed that there is a macro @kwdef in Base for defining struct with default values. It is not a public API so it's subject to changes, but we can use it to substitute @with_kw imported from Parameters.jl.
In general, it would be more reliable to use intrinsic methods.

Done in #7


All_I_DATA_Runes_powered_by_Apialongrezal_flash speed action must be time couched.

Originally posted by @AlongRezal in https://github.com/gohugoio/hugo/issues/10140
Originally posted by @AlongRezal in https://github.com/mozilla/fx-private-relay/issues/2288

All_I_DATA_Runes_powered_by_Apialongrezal_flash speed action must be time couched


[CI][TEST] update pre-commit hooks and use pre-commit for style tests in CI

Ref issue:

#1408

Changes:

Add .editorconfig

Add pre-commit-hooks:
- repo: https://github.com/pre-commit/pre-commit-hooks
  rev: v4.4.0
  hooks:
    - id: check-symlinks
    - id: destroyed-symlinks
    - id: trailing-whitespace
    - id: end-of-file-fixer
    - id: check-yaml
    - id: check-toml
    - id: check-ast
    - id: check-added-large-files
    - id: check-merge-conflict
    - id: check-executables-have-shebangs
    - id: check-shebang-scripts-are-executable
    - id: detect-private-key
    - id: debug-statements

Add flake8 to pre-commit config and add .flake8 file

Use pre-commit for style tests in CI

Run pre-commit and fix existing violations:

fix trailing spaces
fix end-of-files
fix mod file mode with chmod -x
run autopep8 on existing code
fix flake8 violations

A little curious why so many .cpp and .py files are getting updated. Is that because of the previous version discrepency?

A little curious why so many .cpp and .py files are getting updated. Is that because of the previous version discrepency?

I have double-checked if I reset the changes and run pre-commit with old hook versions again. It produces the same large git differences. Seem that the CI previously did not check the code style properly.

Thanks a lot for the PR :)


[Hi] I add this project to Awesome-CobaltStrike

Welcome to Awesome-CobaltStrike  to give me advices or pr.

Amazing repo mate, thank you for including me :)


Install of features newly-added in dev mode doesn't install auto-features

If I startwith a default, kernel install, and feature:  cdi-2.0 in my server.xml and start dev mode:

mvn clean liberty:dev

then wait for the app to start, then edit src/main/liberty/config/server.xml and activate feature: ejbLite-3.2.
you will see the ejbLite-3.2 feature installed but you will NOT see the cdi+ejb autofeature installed.
You won't see: target/liberty/wlp/lib/features/com.ibm.websphere.appserver.cdi2.0-ejb3.2.mf.
Note:

The issue goes away if you run with -Dliberty.runtime.version=<IP_ADDRESS> (I hit the issue at <IP_ADDRESS> and <IP_ADDRESS>).
The issue doesn't seem to exist if you do a series of mvn liberty:create liberty:install-feature (in between adding the new feature in server.xml).
The issue doesn't seem to exist if you do a mvn liberty:run then a mvn liberty:create liberty:install-feature (in between adding the new feature in server.xml).

That's quite a few scenarios I covered so you should probably double-check that I wrote that up correctly.
See the recreate branch I pushed at:  https://github.com/scottkurz/sample.batch.bonuspayout/tree/dev-mode-auto-feature-issue   (w/ README.md there).
Since the OL runtime level matters it could be a runtime bug rather than an LMP one, but we can start here.
Current version is <IP_ADDRESS>...also known to fail at <IP_ADDRESS>.

@ericglau I think I just noticed this as well with jaxrs-2.1 and cdi-2.0.  Do you think we should open an issue on the OL runtime?

@ericglau I tested the following scenario on Open Liberty <IP_ADDRESS> without dev mode, the auto feature com.ibm.websphere.appserver.cdi2.0-ejb3.2.mf is installed after step (4).

start with just 'cdi-2.0' in server.xml
mvn clean liberty:create liberty:install-feature
add 'ejbLite-3.2' in server.xml
mvn  liberty:install-feature

@ericglau Now I can reproduce the missing auto feature problem with the following scenario.  We will investigate the problem.

start with just 'cdi-2.0' in server.xml
mvn clean liberty:create liberty:install-feature
remove 'cdi-2.0' from server.xml
add 'ejbLite-3.2' in server.xml
mvn liberty:install-feature

So it looks like maybe the https://github.com/OpenLiberty/open-liberty/pull/11065  idea didn't work?
I take it we still think there might be a fix needed in the OL runtime?

Hey sorry for the late response,  https://github.com/OpenLiberty/open-liberty/pull/11203 fixed this issue a while back.

Hey sorry for the late response, OpenLiberty/open-liberty#11203 fixed this issue a while back.

Looks like it's fixed in <IP_ADDRESS> (fixed my recreate sample).    Thanks.


python3Packages.datashader: relax xarray constraint

Motivation for this change
Fix build (https://hydra.nixos.org/build/163299372)
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.datashader
python39Packages.datashader


updated names and workflows to reflect current UI

@overbrimming,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@overbrimming : Thanks for your contribution to the Azure documentation! The author, @v-royhar, has been notified to review your proposed change.

I approve this pull request. #sign-off.

#sign-off


Missing assignment description in assignment Additional Task

The assignment description of assignment Additional Task (DE: Zusatzaufgabe) displays just "Source: #UJRX"
I searched on google for #UJRX but couldn't find anything related to this assignment.

This assignment should be some kind of "riddle" to figure out the right algorithm, but a bit of explaination wouldn't hurt, either

yes, I was able to solve it.
Suggestion: "Find 1 right algorithm for all 4 worlds."

added description


pkg,hack: Update Kubernetes from v1.8.4 to v1.8.5

Starting conformance run.

coreosbot run e2e

coreosbot run e2e

I think conformance test scripts should probably be bumped too

Waiting on latest conformance run - 53.

[383217.190913] golang[6]: [ 4056.174121] golang[6]: Ran 148 of 699 Specs in 3134.111 seconds
[383217.191288] golang[6]: [ 4056.174273] golang[6]: SUCCESS! -- 148 Passed | 0 Failed | 0 Pending | 551 Skipped PASS


chore(UI): history query -> marked fav is too small.

Hi,
The queries in the history panel can be marked favourite by click on the :star:  icon, it can be marked un-fav in the same fashion.
The icon is too small to be clicked, additionally, padding can be added to increase the action area.

Q. On clicking there's a className which gets added/removed. Unfortunately, I am not able to find the corresponding CSS class definition.
https://github.com/graphql/graphiql/blob/1563b6029c5d2e2aeea90ab4ff48742d40538af2/packages/graphiql/src/components/HistoryQuery.tsx#L91

we are about to implement a redesign, so most of this will go away. see the pinned issue!

we are about to implement a redesign, so most of this will go away. there is a pinned issue if you want to discuss the new redesign!

Thanks for the update :+1:


Add a gutter to mark refactoring changes in code diffs

Gutter could help to mark refactoring changes in code diffs. If a user is interested in some details, they could click on the gutter and see refactoring description and corresponding code diff (it could be useful in cases when a piece of code was moved to another file).

[x] Add a gutter to code diffs
[ ] Show gutter only close to refactoring changes

What it looks like:

User opens a diff

User hovers over the gutter

User clicks on the gutter


ClassCastException: java.lang.Boolean cannot be cast to java.lang.Byte

Bug Report
Which version of ShardingSphere did you use?
build from 4.0.0 RC2 dev branch
Which project did you use? Sharding-JDBC or Sharding-Proxy?
sharding-jdbc
Expected behavior
work properly
Actual behavior
Before #2703 merged, ss will throw ClassCastException that can't convert Byte Array value to Byte.
but now, it throws ClassCastException like below:

Reason analyze (If you can)
It will be work well after I add 'Types.BIT' in QueryResultUtil.getValueByColumnType().
Could you help me to explain that?
Steps to reproduce the behavior, such as: SQL to execute, sharding rule configuration, when exception occur etc.
Example codes for reproduce this issue (such as a github link).

Add table structure

@FujiwaraTakumi What's the SQL?
@Saisimon Can you take a look?

@FujiwaraTakumi Can you put up your code completely?

Logic SQL like this:
SELECT
nodeethern0.ID AS ID1_4_,
nodeethern0_.XXX_DATE AS XXX_D2_4_,
nodeethern0_.XXX_DATE AS XXX_D3_4_,
nodeethern0_.DELETED AS DELETED4_4_,
nodeethern0_.GRP AS GRP5_4_,
nodeethern0_.LINKED AS LINKED6_4_,
nodeethern0_.XXX AS XXX7_4_,
nodeethern0_.NAME AS NAME8_4_,
nodeethern0_.SN AS SN9_4_,
nodeethern0_.XXX AS XXX10_4_
FROM
node_ethernet nodeethern0_
WHERE
nodeethern0_.SN =?_

According to MySQL Connector/J 5.1 and MySQL Connector/J 8.0 documentations

java.lang.Boolean if the configuration property tinyInt1isBit is set to true (the default) and the storage size is 1, or java.lang.Integer if not.

Two ways can fix it.

change tinyint(1) to tinyint(2)
datasource url add tinyInt1isBit=false, like jdbc:mysql://dbserver:3306/d0?tinyInt1isBit=false

I see, thanks for your patience.

@Saisimon Can you add my WeChat or call me directly any time when you are free? My phone number is<PHONE_NUMBER>7.

@Saisimon Can you explicitly point out whether it's a problem of ShardingSphere or not. We need more discussion

MySQL driver determines whether to convert the TINYINT(1) type to  byte or to boolean according to the tinyInt1isBit parameter. The default is to convert it to boolean type, but the user wants the character type, the user needs to control the behavior of the mysql driver.

@FujiwaraTakumi Will you meet the same problem if you use raw jdbc?

I use hibernate + druid to connect db directly without ss haven't meet this problem.

After deep discussion with @Saisimon , It make sense that the particular usage should set tinyInt1isBit=false.


"gamepad_api_enabled": "true", MAKE STEAMVR ADD-ONS CRASH

Hi, I ran into a new issue I believe or I might Have done something wrong But I cannot figure it out.
I follow all the guide from Daley Tech on YouTube, and now I reach the point where I am connected my PS move navigator with psmoveservice. Gamepad_API_Enabled is to true but when I start  everything with that set to true now make one of my ps move controller lit on and off and start steam vr under an issue which disable all the Add-Ons do to crash.
If I unblock all Add-Ons but psmove It turn out it work so the issue point to psmove service and I dont know what to do since I cannot find allot about it.
Looking forward for your help .

I finally ended-up resolving the issue myself with that thread "https://github.com/HipsterSloth/PSMoveSteamVRBridge/issues/107"
I had to replace the ":" with "_" in both "psnavi_gamepad_2.json" and "psnavi_gamepad_3.json" files.


Faster Delivery Service

Speeding up Delivery Services both state and worldwide.

It takes many days to deliver goods after arriving to ports of entry.


Create a GetOrganizationById API.

Overview
REPLACE THIS TEXT -Text here that clearly states the purpose of this issue in 2 sentences or less.
Action Items
REPLACE THIS TEXT -If this is the beginning of the task this is most likely something to be researched and documented.
REPLACE THIS TEXT -If the issue has already been researched, and the course of action is clear, this will describe the steps.  However, if the steps can be divided into tasks for more than one person, we recommend dividing it up into separate issues, or assigning it as a pair programming task.
Resources/Instructions
REPLACE THIS TEXT -If there is a website which has documentation that helps with this issue provide the link(s) here.

Working on it, should be done by 3/28.

The API https://test-civictechindexadmin.herokuapp.com/api/organizations/github_id/{github_id} is deployed, testable in Swagger.


WIP: add primitive persistence

More complete version of #32, thx @Kodylow for motivating me to finally build this.

The UI part is just hacked together unfortunately since I forgot a lot about leptos again in the meantime, but I'd rather get this feature out so one can actually use webimint for more than some dev testing.


Make dropdown fields easier to navigate

Describe the solution you'd like
Change the select fields to datalists. This creates the option to also search through the dropdown options.

@diedekerkhof has worked on this and mentioned that the functionality of data lists is different depending on browser. It would still be a great UX feature, but we might hold it off for a bit.
I'll include the future feature label.


[SPARK-26038] Decimal toScalaBigInt/toJavaBigInteger for decimals not fitting in long

What changes were proposed in this pull request?
Fix Decimal toScalaBigInt and toJavaBigInteger used to only work for decimals not fitting long.
How was this patch tested?
Added test to DecimalSuite.

cc @hvanhovell

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4981/
Test PASSed.

Test build #98772 has started for PR 23022 at commit 9c52e09.

Test build #98772 has finished for PR 23022 at commit 9c52e09.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98772/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4990/
Test PASSed.

Test build #98781 has started for PR 23022 at commit 4bdc82b.

Test build #98781 has finished for PR 23022 at commit 4bdc82b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98781/
Test PASSed.

LGTM

ping @hvanhovell

Merging to master. Thank!


Release v10.7.0

Includes

https://github.com/silinternational/ui-components/pull/242
https://github.com/silinternational/ui-components/pull/241
https://github.com/silinternational/ui-components/pull/240
https://github.com/silinternational/ui-components/pull/239
https://github.com/silinternational/ui-components/pull/238
https://github.com/silinternational/ui-components/pull/237
https://github.com/silinternational/ui-components/pull/234

Added

add height prop to StaticChip
add inputID as prop on Checkbox
deepfactor

Removed

sass from peer deps

Fixed

StaticChip: width now expands with content, styles overridable

Updated

dependencies

:tada: This PR is included in version 10.7.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use scope id instead of name for multicast interface on Windows

On Windows, it seems that ::%<name> always falls back to some interface ignoring the name given. To make it work with multiple network interfaces, scope id might be required.
Fixes #359

Thank you, did you found any sources for this behaviour on Google or such? Can you link them? (WOukld like t understand this windows behaviour)

@Apollon77

https://superuser.com/questions/99746/why-is-there-a-percent-sign-in-the-ipv6-address (find "Windows" to highlight relevant content)
https://learn.microsoft.com/en-us/windows/win32/wininet/ip-version-6-support#scope-id

Thank you, will check later today and merge then too I assume :-)

Found some time to read and yes windows is doing stuff differently. ;-))


allow to read config from readers

This change will help us to be independent from pkger.

Replaced by https://github.com/moov-io/base/pull/302


excuse me, how to use db query builder？

HomeController.php
$users = app('db')->table('users')->get();
app('response')->render('home', [
     'users' => $users,
]);

views/home.php
var_dump($users);

the result is :
500 Unknown Reason

Hello @wfsdaj
Sorry for the documentation. I've updated the docs for query builder. The methods are fetchAll() and fetchOne().
$users = app('db')->table('users')->fetchAll();

app('response')->render('home', [
     'users' => $users,
]);

@wfsdaj Also I will soon freeze all the method APIs and will release the first stable version. Meanwhile I will to add more unit tests and improve documentation with possible video tutorials.
I expect this to happen by the end of this month. Please stay notified 👍

Thanks, it has been solved.
In addition, How to trigger the routenotfoundexception error?
All my mistakes are 500.

In development mode, RouteNotFoundException will be thrown for non-existing routes but in production you should see something like this and it takes care of setting appropriate response code:


Add .bazelversion

This repo is already using bazelisk but does not currently pin a Bazel version.
Add .bazelversion to pin Bazel based on the version used in a recent CI run (https://github.com/quantumlib/Stim/actions/runs/9657364117/job/26636504001).
Related to https://github.com/qh-lab/pyle/pull/46833#issuecomment-2204296035
/cc @Strilanc

SGTM


Don't apply locality label unless provided

We also don't need it in match labels
This was causing failures like:
--- FAIL: TestMtlsHealthCheck (1.09s)
    mtls_healthcheck_test.go:78: apps.NewOrFail: kube apply of generated yaml file: exit status 1: service/healthcheck created
        error: error validating "/tmp/mtls-healthcheck-dc5f54fa27684b/_suite_context/env-kube-704200492/istio-kube-accessor-830983486/accessor_775488596": error validating data: [unknown object type "nil" in Deployment.spec.selector.matchLabels.istio-locality, unknown object type "nil" in Deployment.spec.template.metadata.labels.istio-locality]; if you choose to ignore these errors, turn validation off with --validate=false
FAIL

I never actually saw these in CI, so maybe not a real issue, but might as well not set the label if its not provided.

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

Needs approval from an approver in each of these files:

pkg/OWNERS

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

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio-integ-k8s-tests

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.

/lgtm
/approve
the fix for that test failure is incoming...

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

Needs approval from an approver in each of these files:

pkg/OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, liamawhite, nmittler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [nmittler]

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio-unit-tests

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.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio-unit-tests

prow/e2e_pilotv2_auth_sds.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio_auth_sds_e2e

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.

/test istio-unit-tests
/test istio_auth_sds_e2e
From boskos flakes

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test e2e-mixer-no_auth

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.

/test e2e-mixer-no_auth

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio_auth_sds_e2e

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.

/test istio_auth_sds_e2e

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
f05a66b0c4b07d34b6035e01ba7d838e66e73f9f
link
/test istio-integ-k8s-tests

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.


feat(breakpoints): added css variables for breakpoints

Added css variables for breakpoints to have flexibility on styling level.
Example Scenarios

As a developer, I can use radix UI breakpoints for styling components or pages on CSS level
As a developer, I can change CSS variables to change breakpoint to fit on design requests.

Hey @Sly777, did you test this? I don't think using custom properties in @media declarations is valid CSS, is it?

Yes, very sorry @benoitgrelard - I thought first that works properly after build and dev didn't give me any error.


No able to get pages response .

Whenever I make a request for pages using google vision api I always get null .
        $web_detection = $vision->image($imageData, ['WEB_DETECTION']);
        $imageFeatures[] = $web_detection;
        $data = [];
        foreach ($imageFeatures as $images) {

            $results = $vision->annotate($images);
            foreach ($results->web()->pages() as $webImage) {
                $data[] = $webImage->info()["url"];
            }
            return $data;

        }

Please help me with this .

Hi @ceaser1996.
You should consider switching to Google\Cloud\Vision\V1\ImageAnnotatorClient, as that client is more frequently updated. Here's your snippet, rewritten using ImageAnnotatorClient.
use Google\Cloud\Vision\V1\ImageAnnotatorClient;

$client = new ImageAnnotatorClient;
$pages = $client->webDetection($imageData)
    ->getWebDetection()
    ->getPagesWithMatchingImages();

$data = [];
foreach ($pages as $page) {
    $data[] = $page->getUrl();
}

return $data;

@ceaser1996 where do you get null? Is it from $webImage->info()["url"]; or $vision->annotate($images);?
Have you had any success with the snippet @jdpedrie provided?

@ceaser1996 I'm going to close this out now, but please feel free to re-open if you are continuing to have problems.


Fix: Instalments change if a loan is updated

Describe the bug
Since the value for a instalment is given in the save() method of Loan's model, we need to prevent the change in its value if the Loan is updated.
Sugestion: Correct it by conditioning the change instalment field only if its equal Decimal('0')

Included in issue #32


Sublime Text editor for SSH

Is it possible to use the st or stt commands to open a file or directory in Sublime Text while connected via SSH?

How does this relate to oh-my-zsh?

Well, it's a legitimate helpdesk question.
@brianjking: the short answer is no. The longer one is, it can be done, but you need to mess around with "ssh x11 forwarding" because Sublime Text is a window application (i.e. it has a GUI).
Another option, if you want to only edit a specific folder every time, is to mount the remote folder using sshfs and you'll have it on your local filesystem and edit it from there.
But specifically, if you have sublime text installed on the remote system you'll want to go for option 1.

:+1:

Note that the X11 forwarding will only work if the remote machine is a Linux machine. The Mac and Windows Sublime Text GUIs don't use X11.
There are some other options using SFTP or SSH from within Sublime Text, too (including that sshfs mechanism). This lets you edit remote files using a local Sublime Text process (which would generally give you better responsiveness than running the ST GUI over the network). See this StackOverflow question and here.


Setting Default Color

I'm using v2.*... However, I can't seem to set the default color for a line chart. I'm mainly looking to set the color of the x/y table values. figured the below might do it - but it does nothing to the chart at all.
Chart.defaults.global.defaultColor = 'orange',

Help? If someone can provide me with an example to set x/y axis label value colors, that'd be helpful :)

@mbarwick83 you have the correct default color option overall. To change the font colours (using the latest build from the v2.0-dev branch) you can change Chart.defaults.global.defaultFontColor


pnpm support?

since I install via docker, it would be nice to have this registered with npm so I can install it via commands :)

Hi team, just want to check if it's available wih pnpm now?

published to npm as directus-extension-panel-manual-flows. I'm hoping this will make it available in the marketplace


Roblox.RobloxStudio version <IP_ADDRESS>627

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

About the for WinGet part: your users will see that even if they don't use WinGet to install it.
Here I double-clicked on "RobloxStudioLauncherBeta.exe":

Also, could it be possible for you to add a DisplayVersion for the installer to the registry? This should prevent upgrade loops for WinGet.

About the for WinGet part: your users will see that even if they don't use WinGet to install it.
Here I double-clicked on "RobloxStudioLauncherBeta.exe":
Also, could it be possible for you to add a DisplayVersion for the installer to the registry? This should prevent upgrade loops for WinGet.

how to do it?

Just tell me the values to modify

i tried importing the YAML file but it said a warning

Roblox Studio isn't open-sourced, so I won't be able to help you there, but I believe it's using a custom installer. You won't be able to make those changes in the package.
If you contact someone who maintains or has access to the bootstrapper source code, you should be able to tell them to use the bootstrapper version for DisplayVersion.


If a root found pwd, otherwise 'No root directory found' is echoed.

Pull request associated to following issue:
https://github.com/airblade/vim-rooter/issues/32

Thanks for this.
I pulled in the first change which prints the working directory when vim-rooter changes directory (a4a67ed).
I'm not convinced of the need to notify the user when no root directory is found.  Why do you think this would be useful?

You're welcome.
It might not be that necessary, you are right. Once you know it only prints when it can find a git repo root, you also would know that it couldn't find one when nothing is printed. I guess I wanted to be explicit about it.

OK, let's see how it goes and come back to it if need be.

My thoughts, exactly.
Thanks.
On Mar 12, 2015 10:36 AM, "Andy Stewart"<EMAIL_ADDRESS>wrote:

OK, let's see how it goes and come back to it if need be.
—
Reply to this email directly or view it on GitHub
https://github.com/airblade/vim-rooter/pull/33#issuecomment-78441099.


Add requirements file

Could you please add a requirements file with the library requirements to run the notebooks?
Although installing all the imports, it gets challenging to run them as each time a different import error is raised, and I don't know from the beginning which are all the needed requirements.
Thank you! :)

In any case I get errors when trying to work with the data, I have installed the following packages (and their dependencies):
numpy
xarray
dask
pandas
healpy
matplotlib
jupyter
cmocean
cartopy
scipy
intake
intake-xarray
gribscan
eccodes
requests
aiohttp

But when trying to open the dataset:
cat.IFS['IFS_4.4-FESOM_5-cycle3']['2D_monthly_0.25deg'].to_dask()

I get the following error:
FileNotFoundError: [Errno 2] No such file or directory: '/work/bm1235/a270046/cycle3/multio_cycle3/gribscan_monthly_0.25deg/json.dir/atm2d.json/.zmetadata'

My bad, I just saw that although the catalogue is available online, files are only available locally when connecting to the machine (I thought that they were also automatically downloaded when doing the query). I will wait then until I have access.


cudnn lstm test case update

Since it is just a simple update of test case that won't affect any operation, I merge it for testing first