Sync Between Buckets - [ROOST-110]

I'm open to other naming for the sync functions, if others are more ideal.  Currently, I just exposed the sync_remote_to_local  and sync_local_to_remote

@silverjam Makes sense.  I can revert that part and go with a third direction, with up and down

@silverjam Makes sense. I can revert that part and go with a third direction, with up and down

Using sideways would work too 😄

@silverjam Makes sense. I can revert that part and go with a third direction, with up and down

New direction named sideways would work too 😄

Are we sure we want to add a third direction, versus creating a function like sync-across?  Adding a third direction to sync would require all other sync calls to supply a destination bucket

@silverjam Makes sense. I can revert that part and go with a third direction, with up and down

New direction named sideways would work too 😄

Are we sure we want to add a third direction, versus creating a function like sync-across? Adding a third direction to sync would require all other sync calls to supply a destination bucket

We can discuss this more "offline", but maybe the API change we should go for is one sync call that allows the direction of the sync to be inferred from the input parameters.  If we changed the input type of the sync function from simple string to an enum of "Bucket" or "Local" we could eliminate the "direction" enum and then infer the direction based on the input params.  That is, something like:
enum SyncParam {
  Local { path: String },
  Bucket { bucket: String, path: String },
}

Looking good, can we update https://github.com/swift-nav/esthri/blob/master/bin/aws.esthri as well?


boost build failue

Host Environment

OS: [e.g. Windows]
Compiler: revision
win7 vs2017
Package: boost-preprocessor:x64-windows
Vcpkg version: 2020.02.04-nohash

To Reproduce
Steps to reproduce the behavior:
`./vcpkg install boost:x64-windows
Failure logs
Additional packages (*) will be modified to complete this operation.
Starting package 1/139: boost-preprocessor:x64-windows
Building package boost-preprocessor[core]:x64-windows...
-- Downloading https://github.com/boostorg/preprocessor/archive/boost-1.72.0.tar.gz...
-- Extracting source E:/git/vcpkg/downloads/boostorg-preprocessor-boost-1.72.0.tar.gz
-- Using source at E:/git/vcpkg/buildtrees/boost-preprocessor/src/ost-1.72.0-a796360b87
-- Packaging headers
-- Packaging headers done
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.72.0/LICENSE_1_0.txt...
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.72.0/LICENSE_1_0.txt... Failed. Status: 6;"Could
n't resolve host name"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:175 (message):
  Failed to download file.
  If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
  variables to "https://user:password@your-proxy-ip-address:port/".
  Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
installed/x64-windows/share/boost-vcpkg-helpers/boost-modular-headers.cmake:17 (vcpkg_download_distfile)
ports/boost-preprocessor/portfile.cmake:14 (boost_modular_headers)
scripts/ports.cmake:90 (include)
Error: Building package boost-preprocessor:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:

Please try again - github.com has been having sporatic issues the past few days, I myself have experienced this issue and waited a while and tried again for the download to work properly.

Pinging @gaoyunjun

I tried again, and it was the same question;When I was in the browser input: https://raw.githubusercontent.com/boostorg/boost/boost-1.72.0/LICENSE_1_0.txt this address, parse failed, so this should be the reason for my network, rather than VCPKG bug

Thanks for posting this issue. This issue has been fixed. So I am closing this issue for now.


replace installed nginx with separate container

Hey,
as I said in https://github.com/CachetHQ/Docker/issues/43 I'd like to remove the hard coded installation of nginx within the Dockerfile and use the official image via docker-compose instead.
Best regards,
Mirco

Gave it a spin, seems good to me. Will need to update (or remove) the manual process in https://docs.cachethq.io/docs/get-started-with-docker. I am ok with just documenting the docker-compose method of running the continer.


Terragrunt generated block evaluation

Describe the bug
Generate blocks used to create .tf files (tested with providers, backend) from terragrunt.hcl are not evaluated on first terragrunt init command but the files are properly generated.
Steps To Reproduce
create directory structure
---project
|-----terraform
|-----terragrunt
|---------terragrunt.hcl
contents of terragrunt.hcl:
terraform {
  source = "${get_terragrunt_dir()}/../terraform//"
}

generate "providers" {
  path      = "${get_terragrunt_dir()}/../terraform/providers.tf"
  if_exists = "overwrite"
  contents  = <<EOF
terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "~>3.101.0"
    }
  }
}
EOF
}

cd ...../project/terragrunt
terragrunt init
On first init run latest provider is downloaded ignoring version in generated providers.tf
follow up terragrunt init results in error
╷
│ Error: Failed to query available provider packages
│
│ Could not retrieve the list of available versions for provider
│ hashicorp/azurerm: locked provider registry.terraform.io/hashicorp/azurerm
│ 3.102.0 does not match configured version constraint ~> 3.101.0; must use
│ terraform init -upgrade to allow selection of new versions
╵
running terragrunt init -upgrade finally downloads specified provider version as expected
Expected behavior
Generated block create files before terragrunt downloads the tf files to cache and runs init after they are in cache.
(Or am I missing the point of generated blocks in terragrunt ?)
Nice to haves

[ ] Terminal output
[ ] Screenshots

Versions

Terragrunt version: v0.66.3 (v0.55.1)
OpenTofu/Terraform version: v1.9.4 (v1.5.7)
Environment details (Ubuntu 20.04, Windows 10, etc.): Windows 11

Additional context
Add any other context about the problem here.

Hey @RTS651 ,
I think you might be misunderstanding how Terragrunt works.
When you specify a block like that:
terraform {
  source = "${get_terragrunt_dir()}/../terraform//"
}

What you're telling Terragrunt is "Terragrunt, I've got a reusable module at ${get_terragrunt_dir()}/../terraform. Grab that, and use it here. I might ask you to do some other stuff too.
What you're doing with this:
generate "providers" {
  path      = "${get_terragrunt_dir()}/../terraform/providers.tf"
  if_exists = "overwrite"
  contents  = <<EOF
terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "~>3.101.0"
    }
  }
}
EOF
}

Is saying "Terragrunt, I know I said that there is a reusable module at ${get_terragrunt_dir()}/../terraform that I wanted you to fetch and use here, but it needs a bit more, because I don't want to specify a providers.tf file inside of it. Instead, could you just generate a providers.tf file when you fetch it so that I don't have to store that in the module?"
This, for example is very similar to what you wrote:
terraform {
  source = "${get_terragrunt_dir()}/../terraform"
}

generate "providers" {
  path      = "providers.tf"
  if_exists = "overwrite"
  contents  = <<EOF
terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "~>3.101.0"
    }
  }
}
EOF
}

If you run that locally, I imagine it'll work. Notice that the path is set to just providers.tf for the generate block.
That tells Terragrunt to inject a file named providers.tf into the directory where it copies the module. That directory can be found nested under the .terragrunt-cache directory next to the terragrunt.hcl file.
Hopefully that helps! Feel free to re-open this issue if it hasn't.


feat: allow providing options to bluebuild build command

This creates a build_opts input along with some basic validation to ensure that duplicate options are not passed to buildbuild build.

[x] Add a new input for the action called build_opts

Pass that as the BUILD_OPTS environment variable to the image build step
Remove the line BUILD_OPTS="" (or it might need to be made conditional depending on whether the input has been added)

The input defaults to a single space. This allows us to pass it into the environment unconditionally.
Validation step prevents the action running with duplicate arguments.
For remaining arguments, bluebuild build errors should mention arguments explicitly added by the user as being the problem. So they should not be caught completely unawares that --nonexistent-arg is the problem.

Do testing based on your custom branch, at least these two scenarios

[x] Is BUILD_OPTS empty if you pass nothing into the build_opts input? (Build)[
Is it possible to break the build by adding some opts that conflict with what is automatically configured?

Yes. The build_opts and squash values are passed into a validation script that will exit with a non-zero status should the combination of inputs result in duplicate arguments. It also checks for -p or --push since the action provides that option by default.

[x] Maybe we'd want to do an error message that points out that your custom build opts might be the issue in case of a build failure.

Builds fail fast at an initial validation step that explains what's wrong.
Checking for --push/-p even when squash is false

[x] (of course try testing whether the input works for its actual use case too) Successful build using zstd compression

Thanks a lot! This is a fantastic PR! 💙
I'll let this be open for a moment in case other maintaines want to have a say, but will do a full merge + release tomorrow at the latest.


Watch subdirectories in project root instead of watching project root…

Fixes #25629 and Microsoft/vscode#51139
Instead of watching projectRoot, watch the subdirectory in it so as to ensure we arent watching the folders in projectRoot that are excluded and arent part of module resolution failed lookup locations.

Will this fix not make it in to 3.0.2 ?
Just noticed the milestone on the issue had changed from 3.0.1 to 3.1


Can session reusable?

According to the debug log, session will be release at each step, it's possible to reuse/resume/renegotiate dtls connection?

We haven't implemented this. I can also say that as of now we don't actively develop goldy. But you're welcome to submit a pull request to add your favourite feature - we'll review and merge if all OK.


BloggerApp: Integration specs

BloggerApp: Integration specs
In this branch, we have done the following;
✅ used capybara to write integration test for the following views
Login page
User index page
User show page
User post index page
Post show page
✅ fixed linter errors

Code Review Suggestions Implemented
Hello. I have implemented the required changes pointed out  by the previous reviewers.
I had another Micronaut run rails db:migrate on their end and it was successful as shown in the screenshot
All of the 47 tests are now passing.


[TEST] Test issue 2 for 191398

Do not mind this issue, this is a test for https://github.com/Expensify/Expensify/issues/191398

Sorry for the pings @rushatgabhane, I am testing one QA related to linking issues to PRs.

haha, no worries :)

QA has passed, closing.


Bug: Incorrect strings from other languages

We occasionally see things like chart names or filter labels with incorrect translations. Current theories are that some kind of hard coded database id is pointing to an incorrect asset, or the translations are being improperly applied to the correct assets at import time. For instance, these should both be in English:

It happens in both embedded and Superset dashboards, though I'm not sure I've seen it in languages other than English. I believe I have seen it on the non-translated "source" dashboards, however.

We haven't seen this one in a while, I think it may have been due to not deleting the env directory when doing upgrades that deleted Superset assets. That or just clicking on a localized version of a dashboard and not realizing it.


add nix flake

I've added a flake so it can be used with nix develop or as a flake package.

Thanks. I'm sure this is fine, but I don't use Nix so I have no way to maintain or verify this over time. At least for the timebeing, I prefer to keep the repository limited to tools and configurations I can directly support. Appreciate your understanding.


Data extension API

Hi,
The upcoming version of OpenRefine will enable users to create columns based on reconciled columns, using properties from the source database. This relies on an add-on to the reconciliation API, described here: https://github.com/OpenRefine/OpenRefine/wiki/Data-Extension-API
It would be fantastic if reconcile-csv could support this protocol. By the way, if the protocol is hard to comply with, we can still consider making changes to the spec if it helps.
Thanks a lot for your work!

This is duplicate of #3


Rewrite remote URL checking to avoid potential infinite loop.

@cheister I don't expect this would have caused problems on this code base, but changing the algorithm to be consistent with the one in kochiku.

Very stale. Closing.


[FLINK-31780][component=Runtime/Coordination] Allow users to disable 'Ensemble tracking' feature for ZooKeeper Curator framework

What is the purpose of the change
Given PR adds a new configuration to disable EnsembleTracking feature for Apache Curator. It would make possible to actually configure HA for Zookeeper, which is living behind VIP.
Currently HA with ZooKeeper can even lead to the JobManager failure. The scenario of the failure is next:

Flink connects to ZooKeeper via configured URL.
Ensemble tracking gets a new URL of ensemble, which is not obligatory accessible for Flink, because Zookeeper is under VIP.
In case of reconnect Flink fails to Zookeeper, moreover due to "UnresolvedHostException" Flink's jobManager is killed.

Brief change log

Adds a new configuration into HighAvailabilityOptions
Use newly added ZOOKEEPER_ENSEMBLE_TRACKING flag to control Curator ensemble tracking
Default value is enabling ensemble tracking for backward compatibility, see Curator source code

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
The code path is executed in several test scenarios, e.g. ZooKeeperLeaderElectionITCase#testJobExecutionOnClusterWithLeaderChange.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: yes
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs / JavaDocs

CI is still failing.

@flinkbot run azure

@flinkbot run azure

Thanks a lot for the review!
Sorry but I think I cannot merge this PR.


Outdated

Hi @lbrndnr,
I'm trying to build Nuage from source, but it seems like this repository is missing about 2 weeks of changes. Could you push them to GitHub?
Thanks!

Hey :) you're right, I forgot to push the last time. Could you quickly check if it works now? :)

It works, thank you!


Unable to resolve action<EMAIL_ADDRESS>unable to find version v1.0.0

Getting this error:

Getting this error:

Hi @Zaid-maker ,
From now on, I will continue with the v1 tag. I will update v1 too every time I create a new version. For example, v1.0.0 >> v1, v1.0.1 >> v1, v1.2.0 >> v1, etc. The benefit of this is that you will not have to constantly change the version number every time a new version is created; the v1 tag will always point to the latest stable version. So you need to change v1.0.0 to v1 in your yaml file now. Thank you for your understanding.

Thanks <3


TypeError(unsupported operand type(s) for +=: 'dict' and 'dict')

[ ] I have checked the documentation and related resources and couldn't resolve my bug.
Describe the bug
KeyboardInterrupt: Exception raised in Job[1]: TypeError(unsupported operand type(s) for +=: 'dict' and 'dict')
I'm assuming there is a bug in evaluation mertic
Ragas version: 0.1.20
Python version: 3.11.3 | packaged by conda-forge
Code to Reproduce
Share code to reproduce the issue
https://gist.github.com/jinnoh47/f58dae6aabb007bd4b5a356807ec5695
Error trace

KeyboardInterrupt                         Traceback (most recent call last)
Cell In[25], line 54
52 data_point = gpt_llm_dataset[i]
53 try:
---> 54     result = evaluate(Dataset.from_dict({
55         "question": [data_point["question"]],
56         "contexts": [data_point["contexts"]],
57         "answer": [data_point["answer"]],
58         "ground_truth": [data_point["ground_truth"]]
59     }), samoo_llm_metrics)
60     results.append(result)
61 except Exception as e:
File /opt/conda/lib/python3.11/site-packages/ragas/_analytics.py:129, in track_was_completed..wrapper(*args, **kwargs)
126 @wraps(func)
127 def wrapper(*args: P.args, **kwargs: P.kwargs) -> t.Any:
128     track(IsCompleteEvent(event_type=func.name, is_completed=False))
--> 129     result = func(*args, **kwargs)
130     track(IsCompleteEvent(event_type=func.name, is_completed=True))
132     return result
File /opt/conda/lib/python3.11/site-packages/ragas/evaluation.py:295, in evaluate(dataset, metrics, llm, embeddings, callbacks, in_ci, run_config, token_usage_parser, raise_exceptions, column_map, show_progress)
292 scores = []
293 try:
294     # get the results
--> 295     results = executor.results()
296     if results == []:
297         raise ExceptionInRunner()
File /opt/conda/lib/python3.11/site-packages/ragas/executor.py:118, in Executor.results(self)
114         results.append(r)
116     return results
--> 118 results = asyncio.run(_aresults())
119 sorted_results = sorted(results, key=lambda x: x[0])
120 return [r[1] for r in sorted_results]
File /opt/conda/lib/python3.11/site-packages/nest_asyncio.py:35, in _patch_asyncio..run(main, debug)
33 task = asyncio.ensure_future(main)
34 try:
---> 35     return loop.run_until_complete(task)
36 finally:
37     if not task.done():
File /opt/conda/lib/python3.11/site-packages/nest_asyncio.py:84, in _patch_loop..run_until_complete(self, future)
82     f._log_destroy_pending = False
83 while not f.done():
---> 84     self._run_once()
85     if self._stopping:
86         break
File /opt/conda/lib/python3.11/site-packages/nest_asyncio.py:107, in _patch_loop.._run_once(self)
100     heappop(scheduled)
102 timeout = (
103     0 if ready or self._stopping
104     else min(max(
105         scheduled[0]._when - self.time(), 0), 86400) if scheduled
106     else None)
--> 107 event_list = self._selector.select(timeout)
108 self._process_events(event_list)
110 end_time = self.time() + self._clock_resolution
File /opt/conda/lib/python3.11/selectors.py:468, in EpollSelector.select(self, timeout)
466 ready = []
467 try:
--> 468     fd_event_list = self._selector.poll(timeout, max_ev)
469 except InterruptedError:
470     return ready
KeyboardInterrupt:
Exception raised in Job[1]: TypeError(unsupported operand type(s) for +=: 'dict' and 'dict')
Expected behavior
This code is expected to evaluate the dataset using the specified metrics (faithfulness, answer_relevancy, answer_correctness) and handle any errors that occur during the evaluation process. The results should be appended to the results list.
Additional context
Add any other context about the problem here.

@dosu it's using rages evalute function so it isn't possible to use update() function

@dosu the code snippet is shared on this link https://gist.github.com/jinnoh47/f58dae6aabb007bd4b5a356807ec5695

@dosu The evaluate function is used to evaluate a dataset using specified metrics. In this case, the metrics are faithfulness, answer_relevancy, and answer_correctness. The goal is to process each data point in the dataset, handle any errors that occur, and store the results. Also I'm not using any custom metrics or modification.


Borders and markers improvement

In relation to issue #460 and #463.
I have never done any pull requests on GitHub before so I don't know if any more info is required.

i haven't forgotten about this - just no time yet - sorry

aiming for this weekend to merge

And there was much rejoicing! :D

Ha - i believe 2.2.9 is now on jcenter - testing out some demos to make sure there weren't any unexpected side effects

after a quick run through, everything looks good (as expected).  feel free to add samples to the demo, instructions to the readme, or a page to the wiki, but don't feel obligated.
thanks for contributing!


DT user doc updates

@rahiyer please review DT user doc updates
Will start working on RF in parallel.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/385/

Waiting on these bugs to be fixed before I can finish:
https://issues.apache.org/jira/browse/MADLIB-1217
https://issues.apache.org/jira/browse/MADLIB-1218
https://issues.apache.org/jira/browse/MADLIB-1219

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/397/

@rahiyer RF docs ready for review too.

https://issues.apache.org/jira/browse/MADLIB-1217
https://issues.apache.org/jira/browse/MADLIB-1218
https://issues.apache.org/jira/browse/MADLIB-1219
have all been fixed so I made associated changes to the DT and RF user docs.
I think this is final, I mean, I don't have any other commits to make as part of this PR.
So after final review please merge if you think it looks OK.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/405/


contrib/inkscape_tools: use Tinks! art instead of Dino's to test args

Fix #1657, it was actually the test that was broken as the SVG source file is designed for the bash script that preceded inkscape_tools and not inkscape_tools itself (see examples/mnit_dino/tools/). This PR use Tinks! art instead of Dino's. It was already tested by the Tinks! project but here we use it to test the program arguments.

+1


How to convert pp-structure train model to onnx

I was able to convert the PP-structure paddle train model (best_accuracy.pdparams,  best_accuracy.pdopt, best_accuracy.stated) to pytorch model (model.pth). How can I convert the model.pth to onnx model considering that model.pth is training model not inference model?
Should I convert the mode.pth( train) to inference model and then to onnx, or sth?
Thanks

Hi, @iazdan
PP-structure table model is not supported now. You may try layout model or LayoutXLM model.

Do you mean that pp-strucuter train model to pytorch model is not supported, or the pytorch model to onnx?

layout model or LayoutXLM model to pytorch model is supported


Strip '(Read Entire Message)' from forum messages

Adds noise to the data and needs to be removed.

Should I be doing this on the front end or are we going to implement something that handles this before it is ingested in ES.
In other words, should I be posting this issue in Ernest?

Implemented on in the node server.


Migrate Ingest Inspector to AWS

What does this change?
This migrates the Wellcome Ingest Inspector from Glitch into AWS. As part of this migration, the following changes were made:

The application and infrastructure code were moved from their original repository into the storage-service repository
The backend was migrated from Flask to a single Lambda function (placed behind an API Gateway endpoint)
The frontend was migrated from Flask templates to a static Next.js app (hosted from an S3 bucket via CloudFront)
Visual changes were made in the frontend, borrowing visual components from wellcomecollection.org, to make the app more consistent with Wellcome Collection's design system

How to test
Visit the deployed version on https://d1dti1kbjcy26.cloudfront.net and verify that it provides the same functionality as the original version.
How can we measure success?

Have we considered potential risks?

Suggestions to fill out the PR ^

How can we measure success?

The Vercel application has been decommissioned and it's credentials removed. Developers & digital production users are both able to use this new application

Have we considered potential risks?

This application is public facing, but as discussed in the README for this project that risk is mitigated by the lack of secret information in the project. A future improvement would be to put this behind a staff login.

This is great, think this PR good to go.
I think needed to complete (not blocking for merge, but needed to close off this work):

Set up a domain and cert for the app so that it can be reached at https://ingest-inspecter.wellcomecollection.org.
Remove the credentials for the old Glitch app, do some comms in dev and digital production Slack about the switch over.

Some nice to haves:

Find out if we have credentials to close the Glitch account and do so.
Deploy this application via GitHub Actions on merge to main.
Use a Microsoft Entra ID Azure (Azure AD) registered app to authenticate staff users.


Fizzbuzz using list comprehension

3 line code using list comprehension

Thank you for your PR!
Unfortunately a solution similar to yours has already been submitted :/
The goal is to find creative/abstract new solutions to FizzBuzz.
See The top 5 samples here.
Thanks anyway! :)


book.json has errors

Category
Is it a bug

What is the current behavior?
i'm receiving suddenly an error saying

What is the desired behavior?

Additional information
For the GitBook Editor:
- Operating System macos 10.12
- (Web) Browser version chrome 59
- (Desktop) Version of the Editor
For GitBook.com:
- talaviss
- https://www.gitbook.com/book/talaviss/professionalnodejs/edit#/edit/master/book.json?_k=8upo7o

i have the same problem, does any play fix it?

Same here, i just started a new book and I published an early version but now i'm having trouble.


cargo-lambda and cargo --config command line overrides

Thanks for writing cargo-lambda!
This might be documentation issue or a only-affects-rust-beginners-thing or maybe a new feature, but I was thrown off by cargo-lambda and cargo --config i.e. command-line overrides.
Just looking at the cargo docs, one gets the impression that custom cargo subcommands would leverage cargo's own handling of configuration, and thus be able to use config overrides too. But having tried this out and (and finally – read the source for cargo and cargo-lambda), I realize this is not the case.
My use case is that my lambda requires some file paths in the form of environment variables to run correctly. During development, these paths might differ on different developer workstations. So hard-coding them in Cargo.toml won't do. I tried injecting them using something like this:
cargo --config 'package.metadata.lambda.env.WORKING_DIRECTORY = "/foobar"'  lambda watch

But obviously, this won't work.
Perhaps it might make sense to explicitly mention this in the cargo-lambda docs?
If it'd be possible to offer this override functionality, it would be super useful to me. I want to try and offer a PR for it, but in all honesty it seems a bit too complicated for my limited Rust skills.

Thanks!
Working directory was an unfortunate choice of example from me, since I'm more thinking about use case-specific overrides rather than stuff that's useful for any user of cargo-lambda, but perhaps being able to configure the working directory would be very useful for someone.
The cargo lambda --metadata cargo-lambda.toml watch approach would be super-useful to me and my team. I might try and submit a PR if I find the time.


{!module-name}.component.ts unnecessary?

Modifying {!module-name}.component.ts is unnecessary.  There those functions already exist.

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

ID: 4e252813-8f84-461c-2e2a-60fdab347abf
Version Independent ID: d1e280bd-eb22-101c-4255-b7bcb5edf5e5
Content: Add a module to a tool extension
Content Source: WindowsServerDocs/manage/windows-admin-center/extend/guides/add-module.md
Product: windows-server
Technology: manage
GitHub Login: @nwashburn-ms
Microsoft Alias: niwashbu

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Shoot is immediately flagged as Failed, if an error occurs while trying to sync the corresponding Cluster resource to the Seed

How to categorize this issue?

/kind bug
What happened:
At the end of an otherwise successful Shoot reconciliation, after its .status.lastOperation.state has been updated to Succeeded there is an additional sync of the Cluster resource to the seed:  https://github.com/gardener/gardener/blob/2df287b6f671a51f13b9902497c4dcbad556b4f3/pkg/gardenlet/controller/shoot/shoot/reconciler.go#L131-L135
If the sync returns an error, the .status.lastOperation.state is updated to Failed e.g.:
  lastOperation:
    description: 'etcdserver: request timed out'
    lastUpdateTime: '2024-09-24T10:05:31Z'
    progress: 100
    state: Failed
    type: Reconcile

This happens because when checking whether an operation will not be retried here:
https://github.com/gardener/gardener/blob/2df287b6f671a51f13b9902497c4dcbad556b4f3/pkg/gardenlet/controller/shoot/shoot/reconciler.go#L799
true is always returned because the shoot.Status.RetryCycleStartTime was reset to nil after the previous update of the reconcile operation to Successful:
https://github.com/gardener/gardener/blob/2df287b6f671a51f13b9902497c4dcbad556b4f3/pkg/gardenlet/controller/shoot/shoot/reconciler.go#L127-L129
->
https://github.com/gardener/gardener/blob/2df287b6f671a51f13b9902497c4dcbad556b4f3/pkg/gardenlet/controller/shoot/shoot/reconciler.go#L700
Because of this, even if the error from syncing the Cluster resource is retriable, no further automatic reconciliations will be performed.
Additionally, it is not immediately obvious where the error displayed in the Shoot's status comes from as there is no additional contextual info added to it when the sync fails as can be seen in the example above.
What you expected to happen:
Shoot cluster to not become Failed after only 1 error while trying to sync the Cluster resource at the end of the reconciliation operation and a more descriptive error to be displayed.
How to reproduce it (as minimally and precisely as possible):

Modify the shoot reconciliation code to use a function which immediately returns an error here instead of syncing the Cluster resource: https://github.com/gardener/gardener/blob/2df287b6f671a51f13b9902497c4dcbad556b4f3/pkg/gardenlet/controller/shoot/shoot/reconciler.go#L131
Create a shoot cluster or reconcile an existing one
Observe that the .status.lastOperation.state field of the Shoot becomes Failed.

Anything else we need to know?:
Environment:

Gardener version: v1.103

/area ops-productivity
/area robustness


Add stickers to shop and related paypal functionality

I tested this by making a purchase and it worked.

This seems to work well aside from my concerns about the image.  I ordered a set.  :smile:

@SaintPeter I can update the image later - maybe get a good photo from a camper to replace the current one - let's go ahead and get this live sooner rather than later :)

Done and Done.
If you get me mine soon, maybe I can take a nice demo image.

@SaintPeter I'll send yours out today :)


Required abort() in idle state in order to start a scan.

Killed a scan moving energy.  THis is a know problem.  Recovered prompt and pgm_en position in bluesky and could control the mono.
Tried to start a time scan and required an abort() (as expected).  But Bluesky already believed it was in an idle state.
In [67]: mov(pgm_en,932.575-1.4)

   pgm_en

/home/xf23id1/conda_envs/collection/lib/python3.4/site-packages/epics/ca.py:1245: UserWarning: ca.get('XF:23ID1-OP{Mono-Ax:GrtP}Mtr.TDIR') timed out after 1.00 seconds.
  warnings.warn(msg % (name(chid), timeout))
/home/xf23id1/conda_envs/collection/lib/python3.4/site-packages/epics/ca.py:1245: UserWarning: ca.get('XF:23ID1-OP{Mono-Ax:GrtP}Mtr.HLS') timed out after 1.00 seconds.
  warnings.warn(msg % (name(chid), timeout))
/home/xf23id1/conda_envs/collection/lib/python3.4/site-packages/epics/ca.py:1483: UserWarning: ca.get_timevars('XF:23ID1-OP{Mono}Enrgy-SP') timed out after 5.00 seconds.
  warnings.warn(msg % (name(chid), timeout))
    928.759732

    928.759732        vs/collection/lib/python3.4/site-packages/epics/ca.py:1245: UserWarning: ca.get('XF:23ID1-OP{Mono-Ax:MirP}Mtr.TDIR') timed out after 1.00 seconds.
  warnings.warn(msg % (name(chid), timeout))
    927.979957        vs/collection/lib/python3.4/site-packages/epics/ca.py:1245: UserWarning: ca.get('XF:23ID1-OP{Mono-Ax:MirP}Mtr.HLS') timed out after 1.00 seconds.
  warnings.warn(msg % (name(chid), timeout))

In [68]: mov(pgm_en,932.575-1.4)

   pgm_en

    931.223248

In [69]:  RE(ct(num=1800,delay=2))
---------------------------------------------------------------------------
FailedStatus                              Traceback (most recent call last)
<ipython-input-69-5a4cc0867ee5> in <module>()
----> 1 RE(ct(num=1800,delay=2))

/home/xf23id1/conda_envs/collection/lib/python3.4/site-packages/bluesky/run_engine.py in __call__(self, plan, subs, raise_if_interrupted, **metadata_kw)
    576                 exc = self._task.exception()
    577                 if exc is not None:
--> 578                     raise exc
    579             if raise_if_interrupted and self._interrupted:
    580                 raise RunEngineInterrupted("RunEngine was interrupted.")

/home/xf23id1/conda_envs/collection/lib/python3.4/asyncio/tasks.py in _step(***failed resolving arguments***)
    237                 # We use the `send` method directly, because coroutines
    238                 # don't have `__iter__` and `__next__` methods.
--> 239                 result = coro.send(None)
    240             else:
    241                 result = coro.throw(exc)

/home/xf23id1/conda_envs/collection/lib/python3.4/site-packages/bluesky/run_engine.py in _run(self)
    855             self.log.error("Run aborted")
    856             self.log.error("%r", err)
--> 857             raise err
    858         finally:
    859             # call stop() on every movable object we ever set()

/home/xf23id1/conda_envs/collection/lib/python3.4/site-packages/bluesky/run_engine.py in _run(self)
    800                     yield from asyncio.sleep(0.0001, loop=self.loop)
    801                     if self._exception is not None:
--> 802                         raise self._exception
    803                     # Send last response;
    804                     # get new message but don't process it yet.

FailedStatus: MoveStatus(done=True, pos=pgm_en, elapsed=10.7, success=False, settle_time=0.0)

In [70]: abort()
---------------------------------------------------------------------------
TransitionError                           Traceback (most recent call last)
<ipython-input-70-2fe129fe422b> in <module>()
----> 1 abort()

/home/xf23id1/conda_envs/collection/lib/python3.4/site-packages/bluesky/global_state.py in abort()
    156     This function closes over the global RunEngine instance, gs.RE.
    157     """
--> 158     return gs.RE.abort()
    159
    160

/home/xf23id1/conda_envs/collection/lib/python3.4/site-packages/bluesky/run_engine.py in abort(self, reason)
    740         """
    741         if self.state.is_idle:
--> 742             raise TransitionError("RunEngine is already idle.")
    743         print("Aborting....")
    744         self._interrupted = True

TransitionError: RunEngine is already idle.

In [71]:  RE(ct(num=1800,delay=2))
Transient Scan ID: 64668
Persistent Unique Scan ID: '7dda918e-1a76-4fed-a92b-a8b3cf31bf8b'
+-----------+------------+------------+------------+------------+------------+------------+
|   seq_num |       time |   sclr_ch2 | mono_tempa | mono_tempb |  grt1_temp |  grt2_temp |
+-----------+------------+------------+------------+------------+------------+------------+
|         1 | 11:16:41.7 |       5658 |    302.752 |    302.749 |       33.3 |       28.9 |
|         2 | 11:16:45.1 |       5723 |    302.750 |    302.747 |       33.2 |       28.9 |
|         3 | 11:16:48.5 |       5857 |    302.749 |    302.747 |       33.3 |       28.9 |
|         4 | 11:16:51.8 |       5679 |    302.748 |    302.747 |       33.3 |       28.7 |
|         5 | 11:16:55.3 |       5694 |    302.748 |    302.745 |       33.4 |       28.7 |
|         6 | 11:16:58.8 |       5881 |    302.747 |    302.746 |       33.3 |       28.7 |
|         7 | 11:17:02.2 |       5724 |    302.746 |    302.747 |       33.2 |       28.7 |
|         8 | 11:17:05.6 |       5697 |    302.746 |    302.746 |       33.2 |       28.7 |
|         9 | 11:17:09.0 |       5859 |    302.747 |    302.753 |       33.3 |       28.9 |
|        10 | 11:17:12.3 |       5619 |    302.749 |    302.756 |       33.3 |       28.9 |
|        11 | 11:17:15.8 |       5918 |    302.750 |    302.753 |       33.3 |       28.8 |
|        12 | 11:17:19.1 |       4635 |    302.750 |    302.749 |       33.3 |       28.7 |
|        13 | 11:17:22.5 |         76 |    302.750 |    302.748 |       33.3 |       28.8 |
|        14 | 11:17:25.9 |         76 |    302.751 |    302.753 |       33.0 |       28.8 |
|        15 | 11:17:29.3 |         76 |    302.750 |    302.748 |       33.0 |       28.9 |
|        16 | 11:17:32.6 |         76 |    302.750 |    302.743 |       32.7 |       28.8 |
|        17 | 11:17:36.1 |         76 |    302.750 |    302.743 |       32.5 |       28.7 |
|        18 | 11:17:39.5 |         76 |    302.750 |    302.748 |       32.6 |       28.8 |
|        19 | 11:17:42.9 |         76 |    302.750 |    302.750 |       32.5 |       28.9 |
|        20 | 11:17:46.3 |         76 |    302.751 |    302.750 |       32.4 |       28.7 |
|        21 | 11:17:49.7 |         76 |    302.751 |    302.747 |       32.3 |       28.9 |
|        22 | 11:17:53.1 |         77 |    302.752 |    302.749 |       32.3 |       28.8 |
^CA 'deferred pause' has been requested. The RunEngine will pause at the next checkpoint. To pause immediately, hit Ctrl+C again in the next 10 seconds.
Deferred pause acknowledged. Continuing to checkpoint.
^C
Your RunEngine is entering a paused state. These are your options for changing
the state of the RunEngine:

resume()  --> will resume the scan
 abort()  --> will kill the scan with an 'aborted' state to indicate
              the scan was interrupted
  stop()  --> will kill the scan with a 'finished' state to indicate
              the scan stopped normally

Pro Tip: Next time, if you want to abort, tap Ctrl+C three times quickly.

Pausing...
Out[71]: ['7dda918e-1a76-4fed-a92b-a8b3cf31bf8b']

In [72]: abort()

`1```

There is an issue which I should fix with the IOC not setting done when aborting using ^C .. my bad.
However, it seems that bluesky is not so fault tolerant in this regard.

When the RE exits due to a failed move status it automatically aborts, you do not need to explicitly call abort so this is behaving as expected.
The issue is that the MoveStatus is still hanging around.
attn @danielballan did the fix for this not get rolled out or is there still some issues here?

@stuwilkins Could you expand on that?

When the RE exits due to a failed move status it automatically aborts, you do not need to explicitly call abort so this is behaving as expected.

Yes.

The issue is that the MoveStatus is still hanging around.

What makes you say that? I can't see evidence of that in the Traceback.

attn @danielballan did the fix for this not get rolled out or is there still some issues here?

Yes. I think you mean this commit, which did go in before v0.5.2 was tagged.

This is fixed by https://github.com/NSLS-II/bluesky/pull/497


Support for Assistants API v2

Codecov Report
Attention: Patch coverage is 82.73616% with 159 lines in your changes are missing coverage. Please review.

Project coverage is 85.87%. Comparing base (08fee44) to head (dad58a0).
Report is 4 commits behind head on main.

Files
Patch %
Lines

...ela/openai/domain/moderation/ModerationResult.java
0.00%
31 Missing :warning:

...shirestela/openai/domain/assistant/StepDetail.java
39.58%
29 Missing :warning:

...estela/openai/domain/assistant/FileAnnotation.java
31.81%
15 Missing :warning:

...tela/openai/domain/assistant/ToolResourceFull.java
0.00%
11 Missing :warning:

.../openai/domain/assistant/ThreadMessageContent.java
57.89%
8 Missing :warning:

.../io/github/sashirestela/openai/support/Poller.java
38.46%
6 Missing and 2 partials :warning:

...ava/io/github/sashirestela/openai/OpenAIBeta2.java
90.41%
3 Missing and 4 partials :warning:

...o/github/sashirestela/openai/BaseSimpleOpenAI.java
25.00%
6 Missing :warning:

...irestela/openai/domain/assistant/ToolResource.java
62.50%
6 Missing :warning:

...o/github/sashirestela/openai/common/tool/Tool.java
77.77%
4 Missing :warning:

... and 20 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #119      +/-   ##
============================================
- Coverage     90.19%   85.87%   -4.33%
+ Complexity      721      662      -59
============================================
  Files           125      100      -25
  Lines          1153     1430     +277
  Branches         19       28       +9
============================================
+ Hits           1040     1228     +188
- Misses           84      162      +78
- Partials         29       40      +11

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


[DO NOT MERGE] WIP WIP WIP

Using this for now just so I can get CI on my fork's feature branch.

@alext Ignore it, was just poking around.


[Docs] Add websocket port to list of ports in meshery documentation

Current State
Websocket port needs to be added to the meshery architecture network ports table
Desired State

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hi @CyrineG . I've fixed this issue along with #6204 and made PR which was approved by you and @hershd23. Can you/someone with write access approve the changes as other PRs are being committed into the repo which would need me to update the branch again.
Thanks


vultr-cli: 2.3.0 -> 2.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/vultr/vultr-cli/releases.
meta.description for vultr-cli is: "Official command line tool for Vultr services"
meta.homepage for vultr-cli is: "https://github.com/vultr/vultr-cli"
meta.changelog for vultr-cli is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

Warning: no invocation of /nix/store/c6wff36qab54vx8hn6nqw3bwala24i5x-vultr-cli-2.4.0/bin/vultr-cli had a zero exit code or showed the expected version

0 of 1 passed binary check by having a zero exit code.

0 of 1 passed binary check by having the new version present in output.

found 2.4.0 with grep in /nix/store/c6wff36qab54vx8hn6nqw3bwala24i5x-vultr-cli-2.4.0

found 2.4.0 in filename of file in /nix/store/c6wff36qab54vx8hn6nqw3bwala24i5x-vultr-cli-2.4.0

directory tree listing: https://gist.github.com/203a1b1ad9b6e0d7fbcaae1cb435c91d

du listing: https://gist.github.com/1a78f733eb392f7ac65ef239993e8d5d

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
vultr-cli

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

vultr-cli

Maintainer pings
cc @Br1ght0ne @kalbasit for testing.


Rename the navigation folder to navigators.

This will allow it to play nicer with the ignite cli to generate navigators.
Please verify the following:

[x] yarn ci:test jest tests pass with new tests, if relevant
[x] README.md has been updated with your changes, if relevant

Describe your PR
When running npx ignite-cli g navigator Foo, it creates a new app/navigators directory and generates the files in there but the ignite template generates the initial set of navigators in the app/navigation directory. This PR renames the app/navigation to app/navigators so it plays nicer with that generator out of the box.

Note that after running the tests, it left changes to the following files in my repo:

They're all just formatting changes so I left them out of this PR. Let me know if you'd like me to include them.

Good idea. Thanks @mcarcaso ! Very helpful!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add must be verifiable rule

Include this as a project rule and have the inclusion of verifiable=any in links to obs-search from projects consider presence/absence of this rule
This was originally discussed here #819

I think this is all set. There is a verifiable rule, and the links from project show to the obs search set both place_id and verifiable to any (unless otherwise set by the project rules) to override any default values

great - only thing I noticed is that obs search with a verifiable=true param is not properly checking the box in the filters menu (as it properly does when there's no verifiable param)

wow, you have a lot of unread messages!!!
On Fri, Feb 19, 2016 at 4:29 PM, Scott Loarie<EMAIL_ADDRESS>wrote:

great - only thing I noticed is that obs search with a verifiable=true
param is not properly checking the box in the filters menu (as it properly
does when there's no verifiable param)
[image: screen shot 2016-02-19 at 1 26 57 pm]
https://cloud.githubusercontent.com/assets/598026/13189504/afa29ade-d70c-11e5-9f7d-176dd83afc9c.png
—
Reply to this email directly or view it on GitHub
https://github.com/inaturalist/inaturalist/issues/876#issuecomment-186414991
.

--
Charlie Hohn
Montpelier, Vermont

believe it or not they are all 'thank you' type emails that close out back and forth conversations which I get in my inbox, but don't bother to click through to on iNat - it is embarrasing how they accumulate

as i said in a recent ficus thread, i'm not judging :)  But yeah... my iNat
mailbox isn't like that but I have other ones that are.
On Fri, Feb 19, 2016 at 4:42 PM, Scott Loarie<EMAIL_ADDRESS>wrote:

believe it or not they are all 'thank you' type emails that close out back
and forth conversations which I get in my inbox, but don't bother to click
through to on iNat - it is embarrasing how they accumulate
—
Reply to this email directly or view it on GitHub
https://github.com/inaturalist/inaturalist/issues/876#issuecomment-186419532
.

--
Charlie Hohn
Montpelier, Vermont

I fixed that obs search URL to do the right thing for verifiable=true. So I think this is closable


Hi

Summary

Related Issues/PRs

Javascript Strings Lab
lab
Overview
In this lab, we're going to work with strings. Strings in JavaScript are wrapped in single or double quotes, or in back ticks.
Objectives
Manipulate strings in JavaScript
Practice interpolating with template literals
Introduction
Imagine we're planning a birthday party for Bill Nye. There are going to be a lot of people there, so we're going to use JavaScript to help us keep everything straight.
First, we need to practice greeting everyone. (I don't know about you, but I sometimes get nervous and say the dumbest things — but we don't want to embarrass ourselves in front of Bill Nye!)
One might think that we could just type
Hello, everybody!
in our browser's console and be done with it. Give it a try. (If you're on a Mac, that would be Command + Option + J together.)
You should see something like
Uncaught ReferenceError: Hello is not defined(…)
Well, that won't work. (This is why we practice!) In order to greet our guests, we need to tell JavaScript that we're using a string. A string is a collection of characters (letters, numbers, and symbols) wrapped in single or double quotes (or, as we'll see, in back ticks). So to greet everyone, we can write,
'Hello, everybody!'
or
"Hello, everybody!"
Single or double quotation marks can contain a string variable..
What if we want to say hi to a special guest, like Neil deGrasse Tyson? When we wrap strings in single or double quotes, we can join them together using the + operator:
var specialGuest = "Neil deGrasse Tyson"
"Hello, " + specialGuest + "!" // "Hello, Neil deGrasse Tyson!"
This is called concatenation. Notice that the value of the specialGuest variable is also a string!
TOP TIP: Your console might be getting a little full at this point. If at any point you'd like to clear it out and start fresh, you can either click the button in the top left corner of the console — in Chrome, it looks like this:
clear console
Alternatively, you can press ctrl + L or command + K. As long as you don't refresh the page, anything you've declared will stick around for you to reference — you'll just get a nice blank slate on which to code.
When we wrap strings in back ticks, we can use placeholders (${}) and insert variables or evaluated JavaScript directly:
var specialGuest = "Neil deGrasse Tyson";
Hello, ${specialGuest}! High ${3 + 2}! // "Hello, Neil deGrasse Tyson! High 5!"
This is called interpolation.
Lab
You'll find a file called index.js in this directory. Your mission, should you choose to accept it, is to get its tests (in tests/index-test.js) to pass.
You can run the tests using the learn command in your terminal or the Learn IDE. Give that a go now.
waiting
All three tests have failed! This is okay, and it's expected — you haven't written any code yet, after all.
In index.js, you'll see five lines of code:
var greeting = "";
var specialGuest = "Neil deGrasse Tyson"
var greetSpecialGuest = "" + specialGuest + "!";
var topic = "space";
var conversation = ${topic};
Each line has a test associated with it. When the tests fail, they show us what the expected value is — your job is to make that expectation a reality by modifying the code provided.
When you first run learn, you should see something like this:
All tests failing.
Let's walk through that first error together. First, we see the test title:

strings defines greeting:
The title tells us what the test expects our code to do. In this case, "strings" refers to the general problem space in which we're working — we're handling strings.

Continuing on with the test output, we can now make better sense of the next few lines:
AssertionError: '!' == 'Hello, everybody!'

expected - actual

!
+Hello, everybody!
This is a lot to take in, so we'll go through it slowly.

What could AssertionError mean? Well, it probably means that our test asserted (or expected) that something would be true, and that thing wasn't true.
What is that thing? The test expected the empty string, '', to be equal to the string 'Hello, everybody!' — but, of course, these strings are not equal.

expected - actual is a key for reading the statements below it. + expected tells us that the expected output shows up in that yellowish green; - actual tells us what actually happened.

But reading on, we only see +Hello, everybody! — what's going on? Why isn't there any - actual output? Well, there was no actual output — it's just an empty string! That must be the problem!
Next, the title tells us that index.js "defines greeting." Let's look in index.js — sure enough, we see, at the top of the file, var greeting = "";. Seems like a reasonable place to start.
What if, instead of assigning "" to greeting, we assign "Hello, everybody!", like the test expects. Go ahead and change that line in index.js so it reads
var greeting = "Hello, everybody!";
save the file, and rerun your tests. You should see
First test passes!
Nice! You got the first test to pass.
Now use the skills that you learned above to read through the rest of the test output and fix those errors, too! Always remember to save your file before re-running your tests.
NOTE: Because we're dealing with some low-level language features, you might spot some easy ways to "cheat" on this lab, or this lab might seem frustratingly easy. We've given you some starter code to point you in the right direction — try to solve the lab as intended! You can then compare your solution with ours (found in the solution branch of this repository).
When your tests are passing, submit your answer by typing in learn submit in the command line or else create a pull request (use learn submit if "pull request" sounds a bit terrifying).
Good luck!


Force director to use proxy settings for remote download

Previously, the Net::HTTP.start() call in the download helper
for the bosh director was ignoring the http_proxy/https_proxy
environment variables. This remedies that.

@dpb587 or @cppforlife is the pivotal tracker integration for this broken? Shouldn't I see a link to a ticket for this PR?

Story: https://www.pivotaltracker.com/story/show/131738547
This repo doesn't currently use a pr/issue/tracker integration.

Thanks!
Some background for testing - BOSH was behaving with the proxy settings for most everything (talking to the CPI for instance). However when doing bosh upload release or bosh upload stemcell using remote URLs, the content download was not using the proxy (though the upload to the blobstore after content was downloaded is using the proxy).

moved story to backlog.

Merged onto develop via a206491fa1becca018288ff02359af7c7a0297c2.


Support nested subdomains

If localtunnel.me assigns me a subdomain
your url is: https://gqgh.localtunnel.me
it would be great to be able to visit arbitrary subdomains of the gqgh sudbomain, e.g. https://foo.gqgh.localtunnel.me and https://bar.gqgh.localtunnel.me.

Can't do that because of how https certs work unfortunately :( Can't get wild-wildcard certs.


Update support link for Tableau connector

Description
It was requested that the support link for Tableau connector to be updated from a GitHub link to OpenSearch.
Issues Resolved
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #643 (d3ae8aa) into main (d81ef73) will decrease coverage by 31.87%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #643       +/-   ##
=============================================
- Coverage     94.63%   62.76%   -31.88%
=============================================
  Files           279       10      -269
  Lines          7517      658     -6859
  Branches        556      118      -438
=============================================
- Hits           7114      413     -6701
+ Misses          349      192      -157
+ Partials         54       53        -1

Flag
Coverage Δ

query-workbench
62.76% <ø> (ø)

sql-engine
?

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

Impacted Files
Coverage Δ

...search/sql/expression/datetime/IntervalClause.java

.../org/opensearch/sql/sql/antlr/SQLSyntaxParser.java

...nsearch/sql/storage/bindingtuple/BindingTuple.java

...pensearch/sql/planner/physical/RemoveOperator.java

...ch/sql/expression/aggregation/CountAggregator.java

...sql/expression/datetime/DateTimeFormatterUtil.java

...ch/sql/opensearch/client/OpenSearchRestClient.java

...a/org/opensearch/sql/utils/MLCommonsConstants.java

...earch/storage/script/filter/lucene/RangeQuery.java

.../org/opensearch/sql/ppl/antlr/PPLSyntaxParser.java

... and 259 more

Continue to review full report at Codecov.

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


Update TransientFaultHandlingOptions.cs

According to the documentation:

Consider the TransientFaultHandlingOptions record type defined

Hi 👋 @doterik
Thank you so much for posting this issue. We really appreciate it.
The doc is actually incorrect and should be updated, this was a part of a larger issue that was addressed a while back. Initially it was assumed that record types were valid for the options pattern, they are not. Where do you see this in the docs?


Loading the Faster RCNN checkpoint

Greetings
The readme states:
(Feb 01, 2022)
Training codes for MDef-DETR and MDef-DETR minus Language models are released -> training/README.md
Instructions to use class-agnostic object detection behavior of MDef-DETR on different applications are released -> applications/README.md
All the pretrained models (MDef-DETR, Def-DETR, MDETR, DETReg, Faster-RCNN, RetinaNet, ORE, and others), along with the instructions to reproduce the results are released -> this link
Following the link to the google drive, only provides me with the model weight for the Faster-RCNN, but not with instructions on how to load it and which framework to use.
I have tried creating a Faster-RCNN-resnet101 model with pytorch, but when I load the model weight, it states that the layer names does not match.
Any guidance would be much appreciated.
Best regards
Martin

@mmaaz60 Thanks!


Change kpack extraction for fp16 bfp16 group conv fwd

Change gemmkpack extraction logic for fp16/bfp16 group convolution to achieve vectorized reads in Matrix A along gemmK dimension in 3x3 resnext101 filter cases.  Upto 40% gains in resnext101 group configs.

@zjing14 @asroy Please review it. It aims for perf improvement.

@zjing14 @asroy Please review it

Since the fwd and wrw kernels are separated. We can remove the transformation in the wrapper, which reuses fwd for wrw. This is not urgent, can be in a separate PR. You can refer to PR #152.
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/dd16325f0ad5a2fb4d5874a8f4df2062b103b4e5/src/kernels/composable_kernel/src/kernel_wrapper/gridwise_convolution_implicit_gemm_v4r4_gen_xdlops_gnchw_gkcyx_gnkhw_lds_double_buffer.cpp#L59-L86


Embed iframe for border adjustment calculator at ospc.org/bac

https://ospc.shinyapps.io/border-adjustment-calculator/

@MattHJensen what page do you want this on?

My impression was that the easiest would be to just embed an iframe at a new page: ospc.org/bac. Alternatively, we could include it in the gallery -- but that seems harder to me.

I think it fits in the gallery better, but I just have a minor styling concern. It has its own link to taxbrain, and that just looks a little off. Minor concern if it can be fixed quickly.

I'm not seeing the link to TaxBrain -- could you screenshot it for me?

At the top-left.

Let's ignore that for now. It is useful for other places that it is embedded.
If adding it to the gallery is easy even though it is served from shinyapps.io instead of us, then that makes sense to me too.

Ok. My plan is to embed it in the template used for gallery items (http://localhost:8000/gallery/box_plot/) and hard-code the info on the page.

Got it. Sounds good.


[v3] applying display: "hidden" breaks _meta in the subdir

When using nextra and the docs theme of version 3.0.0-alpha.24, setting the display mode of a subdir to "hidden" seems to make the _meta file in that folder not work.
Say I set all pages in the sample dir to be with the article typesetting, as shown like:

After setting the _meta file of its parent dir to:
export default {
  sample: {
    display: "hidden",
  }
}

the page comes like:

Here's the code for a minimal reproducible example: nextra3-meta-not-applied.zip

Same applies to us. This route is hidden but linked to via our customer switcher on top of the sidebar. The sidebar is not rendered at all as the parent meta hides this page.
https://langfuse.com/guides

this was fixed by https://github.com/shuding/nextra/issues/3331 in stable v3


Why do I have to call authenticates_with_sorcery!?

Recently I've started wondering why I need to set the relation between the Sorcery and User class twice. First time, when I set user_class attribute in initializer, and second time, when I call authenticates_with_sorcery! in model.
Although potential issues are rare, rather easy to detect and fix, this just does not feel right to me and I think we should improve the design here.
I think we should do following:

Deprecate and later remove user_class setting from initializer
authenticates_with_sorcery! should set the class it's called in as a user_class variable for config (we could rename it to model_class to avoid confusion or something)
Config class should raise an error if authenticates_with_sorcery! is called twice (for users  who try to have separate user classes)

Deprecate and later remove user_class setting from initializer
I fully agreed with this.
user_class is bad when you have, for example, users and admins - two different models, but both needs authentification.


Cannot fetch some sites

http://feeds.feedburner.com/Baeldung
http://coolshell.cn/feed
these two, but switching to Reeder to fetch locally is OK.

These feeds are working fine for me.

sorry, network is complicated over here. Any plan for setting proxy per subscription?


[Bug/issue] Potenial bug in track_anything when working on multi-video of an object

Hi @gengshan-y,
While preprocessing multiple videos for an object using the track_anything module, I encountered a potential bug in app.py at line 573.
for path in tqdm(input_dir):
...

size.append(original_size)

videos = {}
for id, input, output, osize in zip(uuids, input_dir, output_dir, size):
    videos[id] = {"input": input, "output": output, "size" : osize}

I think size.append(original_size) needs to be inside the for loop to ensure that the size list has the same length as uuids, input_dir, and output_dir when using zip.

Thanks for the report, it should be fixed.


Pass valid variable when dispatch an event

Bug report
Title
Pass array to an event dispatch by right way
Issue Description
The current variables are passed to dispatch function by wrong way, please change to right way
Preconditions
Please provide as detailed information about your environment as possible.
1. framework Version: 1.1.2

Steps to reproduce
It is important to provide a set of clear steps to reproduce this bug.If relevant please include code samples.
1. Dispatch an event, and pass an array into that, i.e. Event::dispatch('checkout.order.orderitem.save.before', $data);
2. Listen above event

Expected result
Receive the $data as an array
Actual result

Just receive the first param of the array

Please change to something like Event::dispatch('checkout.order.orderitem.save.before', [$data]);

Not feasible


HDFStore.append_to_multiple() with a MultiIndex DataFrame leads to duplicate entries

http://stackoverflow.com/questions/38152606/pandas-hdfstore-append-to-multiple-with-pandas-multiindex

these have very little tests (and the API is a bit weird anyhow).


Add CFAI 84

website : https://www.formation-technologique.fr/

@littlebigfrench First stage of verification passed, official domain is formation-technologique.fr as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@littlebigfrench Pull request merged. Thank you!


Sneaking is no longer toggleable when jumping

So basically, when you have toggle sneak on game options, then when jumping when on surface, it's not toggleable as it used to.
https://github.com/Wilyicaro/Legacy-Minecraft/assets/39068731/033695a8-f8ac-4d50-849d-5137aa1c4a7c

Fixed in version 1 6.3b, thanks for reporting.


How can I use LightGBM Ranker if I have many documents for a query and only label below 31 are allowed

I have some samples (~5000) with their features, and I want to rank them in terms of a score. I have already built a regression model that directly predicts the score, but I still want to try the learning to rank methods, so I turned to the LightGBM Ranker.
Since LightGBM Ranker only accepts label value below 31, I have to group the scores into several categories, 1 to 4 for example. After training, the Ranker is able to rank samples and achieves a nice NDCG@20 score, but it is unable to rank items within the same group.
My problem is somewhat like one query v.s. ~5000 documents, seems a bit different from the ordinary IR problems. It would be perfect if the Ranker accepts the full order or actual scores of my samples as labels, but I don't know how to achieve this. Some posts suggest using the label_gain parameter, but I can't find any documentation on how to set it properly.
I am new to the ranking models, please help. Thanks!

firstly, you can set label_gain to adapt the range of your label. If the range of label is [0, k), you should set k values to it.
secondly, you can rank the samples in the same group, you can just sort them by the prediction scores. maybe I mis-understand how you do it, I assume that you put all docs into one query, with label 1~4.

firstly, you can set label_gain to adapt the range of your label. If the range of label is [0, k), you should set k values to it.
secondly, you can rank the samples in the same group, you can just sort them by the prediction scores. maybe I mis-understand how you do it, I assume that you put all docs into one query, with label 1~4.

Yes, I put all docs into one query with label 1 to 4, but docs that have the same label still vary in their actual scores, I am wondering how to deal with this kind of situation.
As you suggested, should I set the label_gain to [0 ... 5000] and give every document a distinct label?
Thanks!

Maybe I didn't explain my problem well. I grouped my docs that have actual scores into 4 groups to train the LightGBM Ranker, but docs in the same group actually have different scores and should be properly ordered according to that score. I couldn't group them into 5000 groups because the Ranker only accepts labels below 31, so I don't know what to do to deal with this.

in ranking problems, we usually focus on the top docs. So a comment solution to label the most "bad" docs to one label, like 0.
If you really need to rank all 5000 docs, maybe it is better to use regression, or the pair-wise ranking.

in ranking problems, we usually focus on the top docs. So a comment solution to label the most "bad" docs to one label, like 0.
If you really need to rank all 5000 docs, maybe it is better to use regression, or the pair-wise ranking.

Thanks for your advice!
I have already built a regression model and want to make a comparison between it and the LTR model, since it is also a kind of ranking problem. Does LightGBM Ranker support pair-wise ranking? I suppose it is list-wise by default, right?

we don't support pair-wise ranking, but you can construct the pair by yourself.

we don't support pair-wise ranking, but you can construct the pair by yourself.

So I can make C(5000,2) groups, which are consist of 2 documents, right?


Allow sorting learning resources with a priority

As per the first half of RHCLOUD-27196.

These changes have to be done on the backend. Once pagination kicks in the data will not match the real values.

See https://github.com/RedHatInsights/quickstarts/pull/227


Gulp plugin is generating :dir() CSS

Hi, Thank you for this great tool!
Gulp plugin is generating :dir() CSS pseudo-class but that doesn't work in chrome, so how can I have it generating [dir=""] selector like your demo on Codepen?

@ahmadalfy  @jonathantneal
Using postcss-dir-pseudo-class is a temporary solution.
It has been more than 5 years and there is no support for :dir in chromium-base browsers! (caniuse)
And I think there is no plan for it!
Isn't better to use [dir]?

@AliMD :dir selector is supported in Firefox and it's a part of selectors level 4.


Target an LTS version of Node

This is a semver-major change that will bring the dependencies up to supported versions and more easily allow migration of tests to supported test frameworks.

This will use lts/*, lts/-1 and lts/-2, but will do so via [14, 16, 18], which allows us to update package.json's engine as we move forward. This also tests latest, so we make sure that we see any coming issues as node is upgraded.

Hi, when will there be a new release with these changes?

As soon as we finish the TS conversion... which is dependent on https://github.com/goto100/xpath/pull/116 landing. We're getting close. FWIW, this release is holding up the v5 release of passport-saml and node-saml, so there is strong motivation to get this along.

@cjbarth so, is possible https://github.com/node-saml/xml-crypto/pull/333 Bump @xmldom/xmldom to 0.8.8 for V3?

Sure, just make a PR against master, we'll land it, and then make the same thing against 3.x.

that's the point, master already has this change, ON THIS  PR
https://github.com/node-saml/xml-crypto/blob/131bcf97259a4023323cc93ebc5aa88290716832/package.json#L38


Clustering fails due to vtkSubdivisionFilter

I have a mesh, that I'm trying to resample with the ACVD algorithm. While the mesh is manifold, the subdivision filter employed by the ACVD algorithm appears to return a non-manifold mesh, that makes the algorithm crash mid-way.
In particular:
import pyvista as pv
from PyACVD import Clustering

mesh = pv.PolyData('shoot_manifold_acvd_crash.ply')
non_manifold_edges = mesh.extract_edges(feature_edges=False, boundary_edges=False,
                                        manifold_edges=False, non_manifold_edges=True)
print(non_manifold_edges.n_cells, non_manifold_edges.n_points)

0 0

The step that fails is the generation of the clusters:
cobj = Clustering.Cluster(mesh)
cobj.GenClusters(mesh.n_points)

Subdividing mesh with 3 subdivision(s)
ERROR:root:Dataset is non-manifold and cannot be subdivided. Edge shared by 4 cells
ERROR:root:Subdivision failed.
ERROR:root:vtkInformation (0x55df794fb890)
Traceback (most recent call last):

  File "<ipython-input-2-be1bf5dcec33>", line 2, in <module>
    cobj.GenClusters(mesh.n_points)

  File "/home/henrik/.local/anaconda2/envs/surface/lib/python3.7/site-packages/PyACVD/Clustering.py", line 184, in GenClusters
    self.PrepareMesh(nclus, subratio, verbose)

  File "/home/henrik/.local/anaconda2/envs/surface/lib/python3.7/site-packages/PyACVD/Clustering.py", line 139, in PrepareMesh
    v = VN.vtk_to_numpy(self.mesh.GetPoints().GetData()).astype(np.float)

AttributeError: 'NoneType' object has no attribute 'GetData'

Digging deeper shows that this is due to the subdivision that is happening inside the mesh preparation step:
mesh.subdivide(3, 'loop')

ERROR:root:Dataset is non-manifold and cannot be subdivided. Edge shared by 4 cells
ERROR:root:Subdivision failed.
ERROR:root:vtkInformation (0x55df796458b0)
PolyData (0x7ff0b08cab28)
  N Cells:	0
  N Points:	0
  X Bounds:	1.000e+00, -1.000e+00
  Y Bounds:	1.000e+00, -1.000e+00
  Z Bounds:	1.000e+00, -1.000e+00
  N Scalars:	0

mesh.subdivide(2, 'loop')

mesh.subdivide(2, 'loop')
ERROR:root:Dataset is non-manifold and cannot be subdivided. Edge shared by 4 cells
ERROR:root:Subdivision failed.
ERROR:root:vtkInformation (0x55df79436670)
PolyData (0x7ff0b08caa68)
  N Cells:	0
  N Points:	0
  X Bounds:	1.000e+00, -1.000e+00
  Y Bounds:	1.000e+00, -1.000e+00
  Z Bounds:	1.000e+00, -1.000e+00
  N Scalars:	0

Notably, the issue doesn't arise for when only a single subdivision is needed, but the output of that is non-manifold.
mesh = mesh.subdivide(1, 'loop')
non_manifold_edges = mesh.extract_edges(feature_edges=False, boundary_edges=False,
                                        manifold_edges=False, non_manifold_edges=True)
print(non_manifold_edges.n_cells, non_manifold_edges.n_points)

18 18

Thanks for pointing this out.  I'm in the process of refactoring this module and I'll have an update soon.

There's some sort of bug with the subdivision filter, and I've found that I've had better luck just creating my own method.  The following code snippet should work for you with the latest version of pyacvd
import pyvista as pv
from pyacvd import Clustering

pv.set_plot_theme('document')

mesh = pv.read('shoot_manifold_acvd_crash.ply')

clus = Clustering(mesh)
clus.subdivide(3)  # 2 also works
clus.cluster(10000)
remesh = clus.create_mesh()
remesh.plot(color='w', show_edges=True, smooth_shading=True)


NewTodoInput component should autofocus when loaded

when the TodoApp is loaded, the 'NewTodoInput' should set focus.

@likui628 Thank you telling me about it, I'll try fix this in a month!

Closed #795


Don't allow to create subprojects with same alias

Added a clean_alias method in the ProjectRelationshipBaseForm which will check if the alias already exists. This will prevent Projects from Creating New subprojects with same alias.
closes #5402

@stsewd Okay. I'll add some tests and update the PR.

@stsewd I have Added some Tests and Updated the PR.

@stsewd Thanks for pointing it out. I couldn’t  find the existing test case as the name is different. I'll move the tests here and update the PR.

@stsewd Moved the tests to test_subprojects.py

@stsewd Should I remove the tests that are already there or this can be merged?

You can try to identify the repeated tests, but it's fine for me.

@stsewd okay cool.

@stsewd Removed Repeated tests. :)

Tested this locally, it's working


Make the arrow position nicer when a dropdown is taller than one line.

I have taken the liberty of opening a PR here, because this looks obviously broken to me (in this, probably rare, edge case), and the fix seemed reasonably obvious too.
Before this change
The taller the dropdown became, the weirder the arrow position was.

After this change

Thanks!


(#671) send offer letter on approve

#671

To generate PDF, we have used mPDF package.
We need to run  composer update command to use the package.

@rathorevaibhav It looks to me like the requirements mentioned in #671 description are complete. Please have a look.

@rathorevaibhav we have completed the development of sent for approval, approve and onboard. Please review the code and provide your feedback.

@rathorevaibhav done with your feedbacks


Error Validating Word 2007 File

Trying to validate a docx file. Changing the File Format Version appears to work for all values except 2007. When I try to use 2007, I get a pop-up error "Unexpected token U in JSON at position 0". Validation passes for all other versions.

Here is the most recent file where I got that error. Thanks for looking at it.
thunder-file_11a69750.docx

Thanks @dstringvc . Can you tell me how this file was created? Was it made with Word or an SDK like the OpenXML SDK} or with something else?

I generated that file using the NPOI library.

Does it open correctly in newer versions of Word?

Based on anecdotal reports from the IT directory, it opens correctly using the Office365 (subscription) desktop version (The only desktop version I have access to is 2007). When opening the document with the Office365 Online version of Word, in "editing" mode, the small table at the top of the document is broken (all data is shown in a single column). However, when I switch to "viewing" mode, the document appears correctly.

Here's something odd. I created a new document from scratch using Word 2007 desktop. It contains nothing fancy, just a few paragraphs with simple text. When I tried to validate it, I got the same error as show in my original post. The file is attached.
bacon_ipsum.docx

Thanks for the file. I am able to reproduce it as well. I'll try to debug this and see what is going on this week.


Move tests and the data they require outside of the installed MASS package

Original report by Joseph Fowler (Bitbucket: joe_fowler, ).

See https://pythonchb.github.io/PythonTopics/where_to_put_tests.html for why larger packages ought to store tests and test data outside of the installed package.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Move all tests out of mass package
See why this has advantages: https://pythonchb.github.io/PythonTopics/where_to_put_tests.html
Fixes #242.


error when applying config

how to apply config? when I apply the config I keep getting errors (i have called the config on the first line), sorry if this question seems stupid but I really don't know how to fix it

Are you using package manager like Packer or Lazy?

solved, I think I caught a typo when I did the config earlier


awsdatacatalog.athenacurcfn_aws_perspective_cost_and_usage.perspective_glue_crawler does not exist

Cost and Usage query does not return anything - when I view recent queries in Athena, I see the error: "awsdatacatalog.athenacurcfn_aws_perspective_cost_and_usage.perspective_glue_crawler does not exist" causing the query to fail.
There were no errors when deploying Perspective, and everything runs as it should except for the cost. How would I go about creating the perspective glue crawler? I have gone through the steps to set up the cost feature as stated in the deployment docs. Also cost_and_usage_data_status table does exist.
Browser (please complete the following information):

Chrome

Hey,
Can you please check in the CloudFormation console that the AthenaGlueCrawlerStack has created the AWSCURCrawler-aws-perspective-cost-and-usage resource?
Cheers

Yes, the AWSCURCrawler-aws-perspective-cost-and-usage has a status of create_complete

So the stack deployed successfully. Can you see the Glue crawler AWSCURCrawler-aws-perspective-cost-and-usage created inside the Glue console?

Yes - AWSCURCrawler-aws-perspective-cost-and-usage exists with a status of ready and table athenacurcfn_aws_perspective_cost_and_usage exists.

Can you run the Crawler manually please and see if it creates the necessary table perspective_glue_crawler

I ran the crawler manually, yet the table perspective_glue_crawler table is still not created.
The logs:

BENCHMARK : Running Start Crawl for Crawler AWSCURCrawler-aws-perspective-cost-and-usage
BENCHMARK : Classification complete, writing results to database athenacurcfn_aws_perspective_cost_and_usage
INFO : Crawler configured with SchemaChangePolicy {
"UpdateBehavior": "UPDATE_IN_DATABASE",
"DeleteBehavior": "DELETE_FROM_DATABASE"
}.
BENCHMARK : Finished writing to Catalog
BENCHMARK : Crawler has finished running and is in state READY

Closing due to inactivity.


Add a possibility to use a magic token instead of real invites

It seems fine to me, although I'd try to name the token itself a little bit more directly: bypassInviteProcedureToken or something similar :)

@czeslavo There we go, I failed to push the changes somehow.


Set in multiple scope

Hi,
We have mySettingKey.all(ScopeFilter(inDependencies(ThisProject))) to read from multiple scopes.
That would be great if we could be able to set in multiple scope. Something like this:
    mySettingKey in ScopeFilter(inDependencies(ThisProject)) := "some text"

(Hopefully I comply with the guidelines for contributing this time, sorry if not.)
Cheers

I am closing this due to inactivity, but if this is still relevant let us know.


get wrong results using the downloaded checkpoint

Describe
I used the 'kosmos2.pt' downloaded from the url for offiline demo,but I get wrong result. Is the model correct?

What error message did you get?

The caption results is wrong and without any logic. The code is correctly running.

@donglixp The caption results is wrong and without any logic. The code is correctly running.

Did you launch the model using the Gradio demo setup script?

yes, I used the Gradio to launch the model. I email the result to your Gmail.

Hello, could you please tell me the versions of torch and torchvison? My code seems to have reported an error due to version issues

yes, I used the Gradio to launch the model. I email the result to your Gmail.

@Tourbillon Thank you for providing the feedback. We are working on reproducing the results.

Hello, could you please tell me the versions of torch and torchvison? My code seems to have reported an error due to version issues
@jiang-hai-qi We used torch 1.13.

alias=whoami | cut -d'.' -f2; docker run -it --rm --runtime=nvidia --ipc=host --privileged -v /home/${alias}:/home/${alias} nvcr.io/nvidia/pytorch:22.10-py3 bash

thank you.

@donglixp @qihai-j Because of some limits, we can not use cuda11.8, so i did not use the docker.
fairseq I used 0.12.0
the apex i used the brach 'f3a960f80244cf9e80558ab30f7f7e8cbf03c0a0'
the torch version is 1.13.1
the cuda version is 11.6

thank you very much

@Tourbillon Hi, I check it but can not reproduce the results you mentiond:

The caption results is wrong and without any logic

The commands are:
alias=`whoami | cut -d'.' -f2`; docker run -it --rm --runtime=nvidia --ipc=host --privileged -v /home/${alias}:/home/${alias} nvcr.io/nvidia/pytorch:22.10-py3 bash

wget -O /tmp/kosmos-2.pt "https://conversationhub.blob.core.windows.net/beit-share-public/kosmos-2/kosmos-2.pt?sv=2021-10-04&st=2023-06-08T11%3A16%3A02Z&se=2033-06-09T11%3A16%3A00Z&sr=c&sp=r&sig=N4pfCVmSeq4L4tS8QbrFVsX6f6q844eft8xSuXdxU48%3D"

git clone https://github.com/microsoft/unilm.git
cd unilm/kosmos-2

bash vl_setup_xl.sh
pip install gradio

# replace the model path in run_gradio.sh with /tmp/kosmos-2.pt
bash run_gradio.sh

After launching the local demo, I can obtain the same results (not enabling the sampling) as the online demo, for the examples in the demo.

More details about default packages or env settings can be found in here

@pengzhiliang @donglixp I sovled the problem by using the fairseq in this project. I used fairseq==0.12.0 fro source code and got the wrong results

@donglixp @pengzhiliang @pengzhiliang @donglixp Thanks for your reply! I sovled the problem by using the fairseq in this project. I used fairseq==0.12.0 fro source code and got the wrong results.

The paper mentioned that you use the spaCy to get noun chunks and referring expression. Will the processing code be released？

@donglixp @pengzhiliang The paper mentioned that you use the spaCy to get noun chunks and referring expression. Will the processing code be released？


Fix: colorB override not working for [Github] Stars

Fix #2170
Hi. I believe this is a Bug fix. So I'm not sure whether I should add a test, but added it anyway.

Warnings

:warning:

This PR modified service code for github but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @Lentil1016!

  Generated by :no_entry_sign: dangerJS

@platan Sorry for bothering. I have handled my careless mistake in this PR, but I hit some wired phenomena when running test on CircleCI. Every time I rerun the test, there is a great chance that multiple timeout failure will be reported in services-pr@node-latest and services-pr. The most recently failed test case is:

Github (open pull requests by label (raw)) [ GET /issues-pr-raw/badges/shields/service-badge.json ]
Github (hit counter) [ GET /search/torvalds/linux/goto.json ]
Github (hit counter for nonexistent repo) [ GET /search/torvalds/not-linux/goto.json ]

And:

There is a chance that some failure goes away at every rerunning.
These failure seems not quite relative to the fix I've done.
These failure case works fine on my machine with curl, not likely to exceed the time limit.
I've doubled the timeout values, but failure remain the same on CircleCI. Then I changed it back.
I checked the build history of #2157. He meet the same phenomena, but not as many times as I did.

I guess this is probably caused by my ignorance or mistake. Luckily it passed the last time.

@Lentil1016 thanks for this PR, for the latest improvements and for thorough description of problems with running tests on CI.
There is a room for improvement for this issue. Currently we have 22 GitHub services (.service.js files in services/github) but only one file with service tests and all of them (near 120) have to pass. It is something we should improve.

Is there a reason to implement this manually in this badge? If we set .colorscheme, wouldn't this be handled elsewhere?

@chris48s Hi. Sorry for I can't understand that clearly, I'm not quite familiar with both JS and this project. It's that mean I should set .colorscheme instead of .colorB there? I've checked how others use '.colorscheme', and still have no clue how to use it there.

@chris48s Could you please explain more precisely how would you like to solve this? BaseService has defaultBadgeData which is used in _makeBadgeData. But github-stars is a legacy service which do not use _makeBadgeData. I think we have to handle this that way.

Thanks. I think I've got the point. It's very kind of you.
Considering the previous commit I do set default value to .colorB, which makes .colorscheme default value trivial. I'll redo my commits.

@chris48s thanks for explanation. I just thought we don't want to change default color value (#4183C4), but is OK to change it to blue and it simplifies code.

@Lentil1016 now it looks good to me :-)
@chris48s could you please look at this small change again?