Complex alert monitoring configuration using graph

Right now end analyst user is not able to configure alert monitoring themselves.
Whenever analyst require new type of complex alert, they are not able to configure using graph. For that he/she require to write complex query and as analyst, they don't have much knowledge for writing this type of complex query, need special developer for it.
Require to support some kind of complex feature using graph configuration
Please add some more analytics feature with graph option, so end user is configure some more complex monitor themselves and not require complex query writer developer for it.

@jaykansagara Can you add more examples of how this feature will look like?


SAMZA-2280: LocalApplicationRunner should use final, planned set of configs

LocalApplicationRunner should use final planned set of configs when creating stream processors. Prior to this change, LAR was relying on unplanned configs from the ApplicationDescriptor provided at construction time - this set of configs lacked system generated values coming from, for example, input and output descriptors defined in Application.describe()

Discussed offline. Though there is effectively no change in control flow, this PR brings the standalone case slightly closer to parity with YARN in that the ApplicationDescriptor will be constructed using the planned set of configs.
I've additionally (in the latest commit) removed some redundancy between config parameters and application descriptor, to make it more clear to the reader what configs are being used throughout.

Good point. In YARN we already do this, since the app descriptor is instantiated once during planning and again on container start up. So, at least there would be parity between standalone and YARN control flows. There are certainly cases where weird things could happen dependent on the idempotency of the user's code. Whether this breaks anything I think would be a question of what users are already doing in their describe methods.

Closing this PR as we decided to take a different approach in the short term. Some open questions need to be resolved before we bring this change into open source. Please refer to SAMZA-2312


Add Grid Tutorial

Description
Addition of a tutorial that exemplifies how to calculate quantities, specifically the molecular orbitals, on a grid different than the DFT grid. Useful for people who want to visualize quantities without needing a cube file.
The tutorial ended up covering some material already in the current 4a. My guess is that this tutorial could replace the current 4a.
I spoke with @dgasmith, and he mentioned the possibility of moving the subject into its own folder. This could be a great idea If additional grid tutorials (@hokru 👀) are added. I hope you can help take a look as well, @hokru.
Hopefully, there are not too many Psi4 malpractices in it 😅. Thanks!
Status

[x] Click when ready for review-and-merge

I think you accidentally removed the old tutorial and replaced with it with your new one? Having the be 4a/4b I think makes sense for now.

Reading through your comment I think they should approach different topics? 4a was a "compute the DFT energy" while 4b is more "cast arbitrary orbital-like matrices to a grid". It has been awhile since I have seen this folder so go easy on me.

Reading through your comment I think they should approach different topics? 4a was a "compute the DFT energy" while 4b is more "cast arbitrary orbital-like matrices to a grid". It has been awhile since I have seen this folder so go easy on me.

No problem!
4a_DFT_Grids plots how the DFT grid looks, whereas the 4b does talk about "cast arbitrary orbital-like matrices to a grid" but is specifically oriented to computing the exchange-correlation energy.
The tutorial does show what was already in 4a_DFT_Grids.  It also focuses on generating the blocks of points from scratch, and it uses them to visualize the orbitals. I guess that it can be seen as the combination of 4a and 4b.
I initially had this new tutorial as 4f until I noticed the similarities with 4a.

I think it's ready.
Do we need another reviewer to approve it?

Hello Daniel 🐶! Just wondering if there's anything else we want to add/fix before merging?

Let's do it! This looks amazing- thanks!


Yes, at shell alias/function level.

@Haroenv are you willing to code it? 😃  What shell do you commonly use?

I use Fish usually, and yes, I'll look at how this would work internally

Awesome! I use Fish too! Please feel free to ask for help, although I'm sure you'll nail it :o)

So I've checked a bit, and alias like:
alias fuck-it='export THEFUCK_REQUIRE_CONFIRMATION=False; fuck; export THEFUCK_REQUIRE_CONFIRMATION=True'

should work.

Thanks, and sorry for not following up on the PR

After changes in #620 we can have -y/--yes:
➜ git br
git: 'br' is not a git command. See 'git --help'.


Cannot build/install t-route modules using pip >= 23.1

When attempting to build with pip 23.1, the follow error occurs
Current behavior
run compliler.sh and get the following error:
running build_ext
Processing /Users/nelsfrazier/workspace/ngen/utilities/hydrofabric_test/t-route/src/troute-network
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [17 lines of output]
      Traceback (most recent call last):
        File "/Users/nelsfrazier/workspace/ngen/utilities/hydrofabric_test/venv/lib/python3.9/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/Users/nelsfrazier/workspace/ngen/utilities/hydrofabric_test/venv/lib/python3.9/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "/Users/nelsfrazier/workspace/ngen/utilities/hydrofabric_test/venv/lib/python3.9/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
        File "/private/var/folders/dk/f2t2z5mx6y161tmr6yyf3vlc0000gp/T/pip-build-env-73sgqv7z/overlay/lib/python3.9/site-packages/setuptools/build_meta.py", line 341, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
        File "/private/var/folders/dk/f2t2z5mx6y161tmr6yyf3vlc0000gp/T/pip-build-env-73sgqv7z/overlay/lib/python3.9/site-packages/setuptools/build_meta.py", line 323, in _get_build_requires
          self.run_setup()
        File "/private/var/folders/dk/f2t2z5mx6y161tmr6yyf3vlc0000gp/T/pip-build-env-73sgqv7z/overlay/lib/python3.9/site-packages/setuptools/build_meta.py", line 487, in run_setup
          super(_BuildMetaLegacyBackend,
        File "/private/var/folders/dk/f2t2z5mx6y161tmr6yyf3vlc0000gp/T/pip-build-env-73sgqv7z/overlay/lib/python3.9/site-packages/setuptools/build_meta.py", line 338, in run_setup
          exec(code, locals())
        File "<string>", line 4, in <module>
      ModuleNotFoundError: No module named 'numpy'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

Expected behavior
Successful build/install
Steps to replicate behavior (include URLs)

pip install -U pip==23.1
./compiler.sh

Screenshots

Can downgrade pip using
pip install -U pip==23.0

As a temporary workaround.


Changes in vitetest.config.ts are not detected (in some cases)

Describe the bug
I have run some tests via the extension and everything is fine. Then I changed the vitest.config.ts file.
Re-running the tests did not take the changes into account.
Debugging the tests take these changes into account.
That is, tests behave differently depending on whether they are run or debugged.
In my case I have a vitest.workspace.ts file, but when trying to reproduce the bug, I was able to reproduce the same behaviour w/o a vitest.workspace. However, when I tried to create an example (see github repo) the error only occurred when setting up a vitest.workspace.
Reproduction
Install:
git clone https://github.com/jpilgrim/vitest_bug_demo.git
cd vitest_bug_demo
npm i
code .

This little project (created via npx create vite@latest demo -- --template react-ts) has a vitest.workspace.ts file (with only one workspace).

Testing view, run all tests, ok
Run src/demo.test.ts ok
Debug src/demo.test.ts ok
Open vitest.config.ts, uncomment line 8 (setupFiles: ['setup2'])
Run src/demo.test.ts ok <-- this is the bug!
Debug src/demo.test.ts
Error: Failed to load url ...vitest_bug_demo/setup2

Output
Run output:

[18:20:33] Initiating deferred test run
[INFO 18:20:33] Running 1 file(s): src/demo.test.ts
[18:20:33] The runner is starting because tests src/demo.test.ts were started due to a file change
[18:20:33] No task result for "demo.test.ts", ignoring
[18:20:33] Enqueuing "Test 1" because it was just collected
[18:20:33] No errors found for "demo.test.ts"
[18:20:33] Marking "Test 1" as passed
[18:20:33] Ending test run src/demo.test.ts
[18:20:34] [VSCODE] Ignoring file: node_modules/.vite/vitest/results.json

Debug output:
[INFO 18:21:02] [DEBUG] Starting debugging session /.../.nvm/versions/node/v20.17.0/bin/node
[INFO 18:21:02] [DEBUG] Debugging started
[18:21:03] Initiating deferred test run
[INFO 18:21:03] Running 1 file(s) with name pattern: ^\s?Test 1$
[18:21:03] The runner is starting because tests src/demo.test.ts were started due to a file change
[18:21:03] Enqueuing "Test 1"
[18:21:03] [VSCODE] File deleted: vitest.config.ts.timestamp-1731259263303-68a2728011195.mjs
[Error 18:21:04] Error in /.../vitest_bug_demo/src/demo.test.ts Error: Failed to load url /.../vitest_bug_demo/setup2 (resolved id: /.../vitest_bug_demo/setup2). Does the file exist?
    at loadAndTransform (file:///.../vitest_bug_demo/node_modules/vite/dist/node/chunks/dep-BWSbWtLw.js:51920:17)
[18:21:04] Marking "demo.test.ts" as failed with 1 errors
[18:21:04] Marking "demo.test.ts" as failed with 1 errors
[18:21:04] Ending test run src/demo.test.ts
[18:21:04] [API] Vitest WebSocket connection closed, cannot call RPC anymore.
[18:21:04] Disposing test runner
[18:21:04] Ending test run <none>
[18:21:04] [VSCODE] Ignoring file: node_modules/.vite/vitest/results.json

### Version

1.6.6

### Validations

- [X] Check that you are using the latest version of the extension
- [X] Check that there isn't [already an issue](https://github.com/vitest-dev/vscode/issues) that reports the same bug to avoid creating a duplicate.
- [X] Check that this is a concrete bug. For Q&A open a [GitHub Discussion](https://github.com/vitest-dev/vscode/discussions) or join our [Discord Chat Server](https://chat.vitest.dev).
- [X] The provided reproduction is a [minimal reproducible example](https://stackoverflow.com/help/minimal-reproducible-example) of the bug.

One additional remark why this bug is really annoying: Why do you debug a test? Well, because it does not work. Fixing a test by changing the config is not that uncommon, in particular when you set up a project (and when you are maybe new to vitest). In this case, the bug is absolutely confusing. In my case, I spent hours trying different things before I noticed that it is just a bug of the extension.

I can reproduce it now, thank you for opening an issue.


H265: Support HEVC over HLS.(#465)

compile by ./configure --h265=on, run by ./objs/srs -c conf/hls.conf
Support HEVC/H.265 in HLS.

It works! 👍


[Feature]: Enable linting to check arguments mentioned in docstrings

What problem or use case are you trying to solve?
In some cases, arguments mentioned in docstrings have become stale, and no longer match the actual function signature. It would be better if we checked docstrings through linting so this doesn't happen.
Here is one example of an out-of-date docstring: https://github.com/OpenDevin/OpenDevin/blob/b4cdebec06715c98827256ab01caf4fc513f670f/opendevin/llm/llm.py#L44
(at least llm_top_p is not documented)
Do you have thoughts on the technical implementation?
This can probably be enabled by adding the following to pyproject.toml
[tool.ruff.lint]
select = ["D"]

[tool.ruff.lint.pydocstyle]
convention = "google"

We might want to then disable the conventions that require docstrings for every function (at least for now) because that would be too much work to add docstrings to every missing function.

Would be exciting if anyone can use OpenDevin to fix this

@neubig @li-boxuan : I am keen to try my luck on this . Any tips from where to start ?

I tried to give some direction in the issue description above, but if you have any additional questions I'm happy to help

@neubig: Cool, I am fancying the idea of using  OpenDevin to fix OpenDevin and see how it goes.

@neubig: Cool, I am fancying the idea of using OpenDevin to fix OpenDevin and see how it goes.

Hello @nlauchande,
have you had the time yet to take a look into this enhancement?

Hello, I would love to start contributing to OpenDevin. Since this is labeled as a "small effort", can I take a look at it if no one else is?

Hello, I would love to start contributing to OpenDevin. Since this is labeled as a "small effort", can I take a look at it if no one else is?

Hello,
since we haven't heard from the previous poster, we'd be happy for your contribution! 😃

Please don't let ruff make the dev's life harder with docstring issues. 😬
Not that upon trying to commit the whole codebase will be checked and comes back with 200 warnings.
IF could ruff fix smaller docstring issues by itself, with simple re-formatting them, lets try that first.
You might also look into the "dev_config/python" folder, which is important when building from source and committing (pre-commit)

There are about 383 docstring-related issues, with 208 of them as safe fixes and 116 as unsafe fixes.
I have been trying to check ruff's documentation to isolate unsafe fixes to have a look at them but haven't had any luck. Any ideas there? I could save the safe fixes and later look at the unsafe fixes.
Additionally, do we want to consider including docstring-related issues as part of pre-commit checks sometime in the future?

We could start out by sending a PR with all the safe fixes. For other docstring related issues we might want to decide whether it's with the effort to fix them (maybe there are better uses of our time now?)

sure I can create the PR

pyproject-fmt seem to fail continuously fail when I try to commit the above changes to pyproject.toml
It tries to add a new line after swebench = { git = "https://github.com/OpenDevin/SWE-bench.git" } each time I try to commit.
I tried revising the version of pyproject-fmt from 1.7.0 to 2.0.4 and that seems to work

pyproject-fmt seem to fail continuously fail when I try to commit the above changes to pyproject.toml It tries to add a new line after swebench = { git = "https://github.com/OpenDevin/SWE-bench.git" } each time I try to commit.
I tried revising the version of pyproject-fmt from 1.7.0 to 2.0.4 and that seems to work

It does love to wrap long lines, that's for sure. 😃 I just let it do it and commit that change.
@iFurySt could you look into the version bump of that package, if there'd be breaking changes if we went with v2.0.4?

@tobitege or @neubig can this issue be closed with the PR being merged now?


spicedb: add feature overview

We should probably add a whole doc talking about migrations (which would cover migrate and head commands), but completion comes out of the box and is installed for you automatically so it's unimportant.


Tags aren't pulling into NewDot with our NetSuite integration.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.37-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @zsgreenwald
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1726591465899479
Action Performed:

Go to a workspace where NetSuite is connected
Go to Import, and select any field as a Tag
Go back to LHN and see if Tags appears in LHN
If not, go to More Features and enable Tags

Expected Result:
Tags are enabled after integration
Actual Result:
Tags aren't pulled in after Netsuite integration
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/user-attachments/assets/27f399bd-088b-41ba-bb0a-0c585557e7a5
https://github.com/user-attachments/assets/7020006b-a7e8-4d2a-bb96-86a0ce09ac3f

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @twisterdotcom

Actually, this will be fixed with https://github.com/Expensify/Web-Expensify/pull/43574 hopefully

@m-natarajan can you retest this? I think this issue has been solved.

This is resolved.


Add: Itinerary page

I thought there would be more to do in building the itinerary page, but I think a lot of it was already done.
This PR adds:

"Itinerary" to the sidebar for riders
remove ability to link to the campaign directly, since that's unauthorized for riders.
on rider logging in, redirect them to the /itinerary route.

If I understand correctly, this maybe closes #136?

One small nit: let's switch to the :for syntax as we work on the templates

Ready for re-review!


fixes Call to a member function getDefaultValueDefinition() on null

cleaner solution here
https://github.com/ApiGen/ApiGen/pull/665


Air Pressure not shown in Homekit

I have been running the latest version for a while. Yesterday I noticed that Air Pressure was no longer being reported in Homekit. (Shows up fine in Eve). What used to say "Air Pressure: 1013hPa now only says "Occupancy sensor". Temp and humidity are shown correctly. When I look at line 277 in index.js it shows where the characteristic is updated with the 'new' name, but that appears to not be happening.
From my config.json:
{
"platform": "WeatherPlus",
"name": "WeatherPlus",
"service": "openweathermap",
"key": "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx",
"displayName": "Outside Temp",
"units": "us",
"compatibility": "home",
"hidden": [
"WindDirection",
"WindSpeed",
"RainBool",
"SnowBool"
],
"location": 802,
"locationCity": "Brick, US",
"locationGeo": [
40.0707,
-74.1281
]
}
Let me know what other info you may need. Besides this issue everything else is working fine.
Thanks!

Did u update anything? iOS on your phone? Do u have a homekit server device like an iPad oder Apple TV?
Another user has the same issue since forever. So one of the devices must prevent the name change.

Same as #97 still no idea :/ I will leave the other more generic issue open.


find-users command

Please provide details on the following items. Failure to do so may result in deletion of your feature request.
What's the user value of this feature request?
To be able to find if users exist in their foundry
Who is the functionality for?
This functionality is for operators of foundries
How often will this functionality be used by the user?
How often will they use this feature?
Who else is affected by the change?
This would not be a breaking change as it would represent an addition to the core CLI
Is your feature request related to a problem? Please describe.
In our organisation we receive staff member leavers tickets for all members of staff who may or may not have accounts within our foundries, This is a standard procedure to ensure that all accounts across a wide range of business systems and functions are closed in a timely manner when a member of staff is due to leave. Currently we have to use the UAA client to check if the staff member has an account and then handle the ticket appropriately.
Describe the solution you'd like
a command to sit within the user command context that would enable checking if a user had an account or not
Describe alternatives you've considered
Our current work around is to log in to UAA using the UAA client and to perform checks directly on UAA. This is a slightly clunky process and would benefit from being part of the core CLI.

Hi @LeePorte thank you for creating this feature request. I'm curious as to whether this plugin could also help https://github.com/govau/cf-report-users.

Hi @abbyachau, the plugin isn't quite what i'm looking for. I could script around this. Ideally I'm looking for something along the lines of cf find-user <username> and a return of a confirmation of an account with that username existing or not.
A next step would include fuzzy logic on the search to handle partial usernames and then return a list of matching usernames.


Can't register a new version of a model

Hi there,
I'm currently trying to implement this stack at my workplace and facing an issue that I'd like to understand if I'm doing something wrong or if it's a configuration error.
Since we only have 2 workspaces, one for prod and the other for QA and development, with a shared Unity catalog, my idea was to configure the Unity catalog with the name "ml-ops." Then, in the schema, use the model's name, in this case, "prometheus," and within the registry of each schema for each model, name the models as follows: prod-prometheus-model, staging-prometheus-model, and dev-prometheus-model.
For this, I made the following modifications in the code of the following files:
ml-artifacts-asset.yml
resources:
  registered_models:
    model:
      name: ${bundle.target}-${var.model_name}
      catalog_name: ml-ops
      schema_name: prometheus-model
      <<: *grants
      depends_on:
        - resources.jobs.model_training_job.id
        - resources.jobs.batch_inference_job.id

databricks.yml
bundle:
  name: ${bundle.target}-${var.model_name}

variables:
  experiment_name:
    description: Experiment name for model training.
    default: /Users/${workspace.current_user.userName}/${bundle.target}-prometheus-experiment
  model_name:
    description: Model name for model training.
    default: prometheus-model

model-workflow-asset.yml
resources:
  jobs:
    model_training_job:
      name: ${bundle.target}-${var.model_name}-model-training-job
      job_clusters:
        - job_cluster_key: model_training_job_cluster
          <<: *new_cluster
      tasks:
        - task_key: Train
          job_cluster_key: model_training_job_cluster
          notebook_task:
            notebook_path: ../training/notebooks/Train.py
            base_parameters:
              env: ${bundle.target}
              # TODO: Update training_data_path
              training_data_path: /databricks-datasets/nyctaxi-with-zipcodes/subsampled
              experiment_name: ${var.experiment_name}
              model_name: ml-ops.${var.model_name}.${bundle.target}-${var.model_name}
              # git source information of the current ML asset deployment. It will be persisted as part of the workflow run
              git_source_info: url:${bundle.git.origin_url}; branch:${bundle.git.branch}; commit:${bundle.git.commit}

However, after the first deployment, when the CI/CD pipeline runs:
databricks bundle deploy -t staging

I get the following error:
Updating deployment state...
Error: terraform apply: exit status 1

Error: cannot create registered model: Function or Model 'ml-ops.prometheus-model.staging-prometheus-model' already exists

  with databricks_registered_model.model,
  on bundle.tf.json line 188, in resource.databricks_registered_model.model:
 188:       }

Besides that, everything runs perfectly, and I can serve the model without a trouble. Also, I'm using the demo model. I haven't implemented our own yet.
I'm not sure if I'm doing something wrong. Any guidance would be appreciated.

If it's worth something this is the requirements.txt I'm using:
mlflow==2.7.1
numpy>=1.23.0
pandas>=1.4.3
scikit-learn>=1.1.1
matplotlib>=3.5.2
Jinja2==3.0.3
pyspark~=3.3.0
pytz~=2022.2.1

@Eiley2 thanks for opening this issue! A few things:

The error says that the model already exists. Would you mind confirming if it does, and if so, delete the model if safe? This error prevents accidental overwrite of models that already exist when using bundle deploy.
Above you said the schema name is "prometheus" but in the code I see the schema name is "prometheus-model". Is that intentional?
I wouldn't rename the bundle: name: in databricks.yml to something variable dependent since that is the name of the entire bundle as a whole. By default, we use the project name for the bundle name and would recommend doing that as well to prevent unintended behavior down the line.

Thanks for your answer @arpitjasa-db !

Yeah it does. When I delete it and deploy it for first time and run databricks bundle deploy -t dev/staging/prod I'll work perfectly, but when the github actions will deploy it again to refresh the code, it throws the error saying that it exists already.

Yeah, sorry, in the picture you can see it's correct the schema.
Gotcha, will rename it to prometheus.

I just deleted the prod version. And ran databricks bundle deploy -t prod and it went okay. If I run it again with some chance, I'll say it's already deployed. Is this not the right way to deplyo a change? Sorry if I'm not getting it right.

Oh you're not using the CI/CD workflows for deployment? Either way this should be working. What is supposed to happen is when you do databricks bundle deploy it will deploy the resources and mark them as having come from this bundle using a state, so subsequent deploys will check that state and only deploy necessary resources, overwriting as necessary.
What seems to be happening is after deploying, the CLI is not recognizing that this resource was created from this bundle but thinks it was created elsewhere, which is why it fails with that error I mentioned above for safety.
Are you running the command from the same directory each time? If so, would you mind opening the .bundle/ directory and listing out all the contents in the subdirectories?

Your suggestion got me thinking, so I went ahead and deleted the .databricks folder with everything within it, and it did the trick!
Looks like when I was deploying from the CI/CD workflows and I tried debugging, somehow I combined my credentials in my terminal and the service account credentials when I tried to deploy. After deleting the folder the bundle was created from scratch and the CLI recognized it came from this bundle. Thanks for your help!
Now I have another issue, whenever I'm running the train task it gives me this error

RestException: INVALID_PARAMETER_VALUE: Invalid name 'ml-ops.prometheus-model.staging-prometheus-model'. A workspace registered model name must be a non-empty UTF-8 string and cannot contain forward slashes(/), periods(.), or colons(:). If trying to access models in the Unity Catalog, make sure to install version 2.4.1 or above of the MLflow Python client, then configure the MLflow client to access models in Unity Catalog via mlflow.set_registry_uri('databricks-uc') before accessing models.```

I solved it doing what it says in the error

The line that's commented is how I solved the error. Is this a intended feature or a bug in the cookie cutter generation in the project? If so, I could make a PR with the solution in the base project.
![image](https://github.com/databricks/mlops-stacks/assets/84003742/885279d8-8f24-4ee0-a6ef-ab941508f47e)

Awesome glad to hear it!

I had the same issue. Deleting the .databricks folder didn't work for me. Strangely, running
databricks bundle deploy --var="model_name=original_name-model-new"
with a new model_name variable worked.


Shouldn't TimeSpec::from_duration preserve monotonicity?

When creating a TimeSpec from a std::time::Duration with more than libc::time_t seconds, the resulting value might be negative and will no longer preserve monotonicity with respect to the input duration. This happens to the integer casting in TimeSpec::from_duration.
See this example on Rust Playground.
I learned about this behavior through errors for negative timeouts from ppoll on Linux at runtime (https://github.com/serialport/serialport-rs/issues/207). I did not expect this for the "porcellain" type Duration and the docs did not hint at it. And likely other users of the "porcellain" as well. When manually converting to the "plumbing" typetime_t I wold have looked deeper into the nitty gritty integer casts and cared for this case by saturating.
So shouldn't TimeSpec::from_duration preserve monotonicity? Isn't an inaccurate but still large TimeSpec less surprising?

The problem is that a Unix Timespec can't represent times greater than i64::MAX seconds. Saturating the conversion would be bad, because then subtracting two TimeSpecs that were produced by TimeSpec::from_duration wouldn't be guaranteed to give the correct answer.

This is a fair point against saturating.

And perhaps a TryFrom<Duration> impl, too.

What about transitioning towards this impl over some time? I could prepare PRs for adding TryFrom<Duration> and once this is released the successor for deprecating From<Duration> so that other young players could get safely around this.

What about transitioning towards this impl over some time? I could prepare PRs for adding TryFrom<Duration> and once this is released the successor for deprecating From<Duration> so that other young players could get safely around this.

Yes, that would be good.

I started #2522 for asking if this in going in the right direction @asomers.
I also had a first look into std::time::Instant - but as far as I can tell this type is completely opaque and I don't see how to access its underlying time data. Am I missing the obvious here? Could you give me a hint?


Trusted publishing

Adds trusted publishing for pypi.
Test pypi keeps using the token.

Tried to publish the same version which gives error, expected result. It is done to avoid publishing again.


Update Composer dependencies (2021-11-12-00-07)

Visual regression test passed!
View the visual regression test report


Humidity is way to high

Hi,
i am currently using this in a stm32wb project.
So the sensor is hooked up correctly and i am able to communicate with it.
But for some reason the Humidity the sensor reports is 30% higher than what it should be.
I am using the following code:
int16_t sht4x_measure_high_precision(int32_t* temperature, int32_t* humidity) {
    int16_t local_error = NO_ERROR;
    uint16_t temperature_ticks;
    uint16_t humidity_ticks;

    local_error =
        sht4x_measure_high_precision_ticks(&temperature_ticks, &humidity_ticks);
    if (local_error != NO_ERROR) {
        return local_error;
    }
    *temperature = (float)(temperature_ticks) / 65535.0f * 175.0f - 45.0f;                   /* convert raw temperature */
    *humidity = (((float)(humidity_ticks) / 65535.0f) * 125.0f - 6.0f);

    /*
    *temperature = convert_ticks_to_celsius(temperature_ticks);
    *humidity = convert_ticks_to_percent_rh(humidity_ticks);
    */

    return local_error;
}

The temperature is around 23 degrees (which matches).
But the humidity gets read out with 86% (58% should be)
Regardless how often i read it is always that high.
As you can see i already used the formula coming from the data-sheet instead of this milli rh stuff.
In the end both are returning the same rh.
This is how i call the lib:
  sht4x_init(SHT40_I2C_ADDR_44);

  sht4x_soft_reset();
  sensirion_hal_sleep_us(10000);
  uint32_t serial_number = 0;
  error = sht4x_serial_number(&serial_number);
  if (error != NO_ERROR) {
    printf("error executing serial_number(): %i\n", error);
    //return error;
  }
  printf("serial_number: %u\n", serial_number);

  int32_t temperature_milli_degC = 0;
  int32_t humidity_milli_RH = 0;
  uint16_t repetition = 0;
  for (repetition = 0; repetition < 500; repetition++) {
    sensirion_hal_sleep_us(20000);
    error = sht4x_measure_high_precision(&temperature_milli_degC,
                                           &humidity_milli_RH);
    if (error != NO_ERROR) {
      printf("error executing measure_lowest_precision_ticks(): %i\n",
             error);
      continue;
    }
    printf("Temperature milli °C: %li ", temperature_milli_degC);
    printf("Humidity milli percent RH: %i\n", humidity_milli_RH);
  }

Maybe i am doing something wrong or its a falsy sensor.. But i dunno.

Hmm... maybe i need this conversion stuff for absolute humidity?
https://github.com/Sensirion/embedded-sht/blob/master/utils/sensirion_humidity_conversion.c
I am a bit confused with this. Coming from an SHTC3.
Applying the conversion function on the value i got with the example above it boils down to be a good match, but i dont get why.
So value from sensor (wich should be RH?) is: ~80% -> Wrong
Taking this value now and put it into the "absolute Hum" function -> 46% -> Right.

Tested another sensor. But same readings

Hi,
since you tested with two different sensors I don't think there is anything wrong with the sensors itself.
The embedded driver does not include floating point operations, since a lot of embedded targets don't support floating point. If you prefer output as floating point you can also read the temperature and humidity as ticks and convert them to floating point using the following functions (taken from the Rasperry pi driver: https://github.com/Sensirion/raspberry-pi-i2c-sht4x/blob/master/sht4x_i2c.c#L53C1-L66C2
float sht4x_signal_temperature(uint16_t temperature_ticks) {
    float temperature = 0.0;
    temperature = (float)(temperature_ticks);
    temperature = ((temperature * 175.0) / 65535.0) - 45.0;
    return temperature;
}

float sht4x_signal_humidity(uint16_t humidity_ticks) {
    float humidity = 0.0;
    humidity = (float)(humidity_ticks);
    humidity = ((humidity * 125.0) / 65535.0) - 6.0;
    return humidity;
}

Since I don't know your environment and your reference I can't really verify if your relative humidity is wrong or not. E.g. during rain, resp. in humid conditions a relative humidity of ~80% is possible.
Another possibility is, that there is a numerical issue with your platform. If you can log the raw ticks you can convert them to relative humidity and temperature using your computer (e.g. calculator). If you get the same values, this would rule out any numerical issue.
Converting the relative humidity to absolute humidity is not what you want to do. They are not the same. Absolute humidity is in g/m³ and expresses the actual amount of water vapor per unit volume (e.g. m³). The relative humidity is in percent and is the percentage of the amount of moisture the air can hold.
Best regards,
Pascal

@psachs thank you for the heads up!
I will try those things you mentioned and report back here.
Calculating the humidity manually means that i should use the formula mentioned in the data-sheet or the formula you provided above?
To my environment: Its just normal room environment, sunny day quite warm and dry. So the rel hum. is around 45%.

I now used the formula from above. Same results.
Then i calculated the hum using:
(47109 * 125) / 65535 - 6
So the raw sensor output was: uint16_t ticks = 47109
Also doing it manually i get the same result. The humidity is around 30% higher than it really is.
fyi @psachs

Hi @Exeu
The formula and conversion is correct. In that case the ~83% relative humidity is what the sensor is measuring on your platform.
I am not sure how this is possible in a rather dry environment. I assume your humidity signal is changing when you e.g. lightly blow on it and is not constant at 47109 ticks.
I never saw any similar issue before where the humidity signal is off by that much.
Best regards,
Pascal

@psachs you won't believe me but: I now put the same code on my second board featuring the same sensor.
And this one is spitting out now the values i expect.
So it may is really a faulty sensor then. I don't know why this sensor was reading the same values as the other one in the beginning -> But now it seems to work (maybe i had there the wrong calculation)..
With this sensor i get now around 44% hum instead of 80 :)
I am quite happy currently (since this took me two days of debugging now)! Thank you for the help and nice support, highly appreciated :)

Hi @Exeu
Great to hear that you get things working. If you think the sensor has problems there is one more thing you can try:
Use the heater for a couple minutes (e.g. 5minutes) as follows:
int32_t temperature = 0;
int32_t humidity = 0;
for (int i = 0; i < 300; i++) {
    if (temperature < 100000) {
        // only active the heater if the temperature is below 100°C to ensure the sensor does not overheat.
        sht4x_activate_highest_heater_power_short(*temperature, *humidity);
    }
    sensirion_hal_sleep_us(100000); // sleep for 100ms
}

If the SHT41 sensor is contaminated (e.g. due to storage or shipping) the heater could fix the issue.
Best regards,
Pascal

I will try that. Thank you for the hint :)


Update git-add.md

Fix inaccurate description of git add -u

Overview
TL;DR

Fix inaccurate description of git add -u

reference: https://git-scm.com/docs/git-add#Documentation/git-add.txt--u
Questions
NA
Next Steps
NA
Review
NA

@rwnfoo, would you have bandwidth to review and merge this PR?


Fix broken headings in Markdown files

GitHub changed the way Markdown headings are parsed, so this change fixes it.
See bryant1410/readmesfix for more information.
Tackles bryant1410/readmesfix#1

?

It's targeting readmes in the embedded dependencies.

Oh, I think I understand now. But why do you embed dependencies and save them in the repo? Should I filter something out?

They are embedded for faster CI times. You should filter out the Pods folder

Ok. Sorry and thanks!


Get Monday in a month

I want to get all monday for a specific month i am doing with the following code
const dayInMonths = [];
let filteredSelectedDay = moment(monthDate)
.startOf('month')
.day("Month");
But its giving me incorrect result any thing i am doing wrong please help.

I agree with Ash - Please post on Stack Overflow (you will get more help there)
Good luck :)


Fix string in popover-term

Motivation
In https://pulsar.incubator.apache.org/docs/latest/clients/go/#Readers ,
span tag is displayed as it is.

Modifications
I removed | in _data/popovers.yaml.
Result
span tag is working properly.

retest this please

retest this please


Add metrics port to waypoint

https://github.com/istio/istio/pull/53351 but for waypoint

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
cc01ad283e57ab57c22aa586208f33a5b45ee484
link
true
/test unit-tests-arm64

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
cc01ad283e57ab57c22aa586208f33a5b45ee484
link
true
/test unit-tests-arm64

unit-tests_istio
cc01ad283e57ab57c22aa586208f33a5b45ee484
link
true
/test 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-sigs/prow repository. I understand the commands that are listed here.


Completed Unit Tests

This patch completes the existing unit tests that were previously
not implemented. This patch also adds the "make unittest" command
that allows the uer to test the unit tests from any directory.
Signed-off-by: Rian Quinn<EMAIL_ADDRESS>

Should open a ticket to remove the staticlib stuff if and when the HippoMocks cross library symbol issue is resolved.


Exclude resources

New tag to exclude resources from tagged scenario.
Usage:
@exclude_aws_s3_bucket.mybucket or @ignore_resource_type.resource_name
Addresses #384

The above doesn't seem to work on Scenario Outline
When it assesses the child module inside the child module it fails. When it assess the resource, it passes the tests.
Terraform version: Terraform v0.13.3
Terraform-compliance vesrion terraform-compliance v1.3.9 (reproducible on 1.3.6 too)
module.example.module.aws_lambda_function.aws_lambda_function.this will be created

resource "aws_lambda_function" "this" {

arn                            = (known after apply)

description                    = "Example lambda function"

filename                       = ".terraform/modules/example.aws_lambda_function/function/.terraform/archive_files/lambda_placeholder_content.zip"

function_name                  = "nonprod-uk-pc2-pipelines-example"

handler                        = "main.handler"

id                             = (known after apply)

invoke_arn                     = (known after apply)

last_modified                  = (known after apply)

memory_size                    = 128

package_type                   = "Zip"

publish                        = false

qualified_arn                  = (known after apply)

reserved_concurrent_executions = -1

role                           = (known after apply)

runtime                        = "python3.7"

signing_job_arn                = (known after apply)

signing_profile_version_arn    = (known after apply)

source_code_hash               = "blah"

source_code_size               = (known after apply)

tags                           = {

"Account"     = "nonprod"
"Component"   = "aws-lambda"
"Environment" = "pc2-pipelines"
"Handle"      = ""
"Location"    = "uk"
"Name"        = "nonprod-uk-pc2-pipelines-example"
"Service"     = "example"
"Squad"       = "blah"
"Tribe"       = "blah"
}

timeout                        = 3

version                        = (known after apply)

tracing_config {

mode = "Active"
}
}
Scenario:

Scenario Outline: Ensure that specific tags are defined
Given I have resource that supports tags defined
When it has tags
Then it must contain <tag_keys>
And its value must match the "<tag_value>" regex
Examples:
| tag_keys          | tag_value                                                                                |
| Name              | .+                                                                                |
| Environment       | ^(playground|sandbox|dev|staging|preprod|prod|pc2-pipelines|pc2-service-pipelines)$      |
| Component         | .+                                                                                |
| Service           | .+                                                                                |
| Location          | ^(uk|global1|sg|us|ca)$
Failure:

Scenario Outline: Ensure that specific tags are defined
Given I have resource that supports tags defined
When it has tags
Then it must contain <tag_keys>
And its value must match the "<tag_value>" regex
Examples:
    | tag_keys    | tag_value                                                                           |
    | Name        | .+                                                                                |
            Failure: module.example.module.aws_lambda_function.aws_lambda_function.this (aws_lambda_function) does not have Environment property.
    | Environment | ^(playground|sandbox|dev|staging|preprod|prod|pc2-pipelines|pc2-service-pipelines)$ |
      Failure:
    | Component   | .+                                                                                |

    | Service     | .+                                                                                |
    | Location    | ^(uk|global1|sg|us|ca)$     |

Even by adding the exclude tag, it still fails for this resource. So far the only workaround I found is to skip the whole scenario but that isn't ideal

The above doesn't seem to work on Scenario Outline
When it assesses the child module inside the child module it fails. When it assess the resource, it passes the tests.
Terraform version: Terraform v0.13.3
Terraform-compliance vesrion terraform-compliance v1.3.9 (reproducible on 1.3.6 too)
module.example.module.aws_lambda_function.aws_lambda_function.this will be created

resource "aws_lambda_function" "this" {

arn                            = (known after apply)

description                    = "Example lambda function"

filename                       = ".terraform/modules/example.aws_lambda_function/function/.terraform/archive_files/lambda_placeholder_content.zip"

function_name                  = "nonprod-uk-pc2-pipelines-example"

handler                        = "main.handler"

id                             = (known after apply)

invoke_arn                     = (known after apply)

last_modified                  = (known after apply)

memory_size                    = 128

package_type                   = "Zip"

publish                        = false

qualified_arn                  = (known after apply)

reserved_concurrent_executions = -1

role                           = (known after apply)

runtime                        = "python3.7"

signing_job_arn                = (known after apply)

signing_profile_version_arn    = (known after apply)

source_code_hash               = "blah"

source_code_size               = (known after apply)

tags                           = {

"Account"     = "nonprod"
"Component"   = "aws-lambda"
"Environment" = "pc2-pipelines"
"Handle"      = ""
"Location"    = "uk"
"Name"        = "nonprod-uk-pc2-pipelines-example"
"Service"     = "example"
"Squad"       = "blah"
"Tribe"       = "blah"
}

timeout                        = 3

version                        = (known after apply)

tracing_config {

mode = "Active"
}
}
Scenario:

Scenario Outline: Ensure that specific tags are defined
Given I have resource that supports tags defined
When it has tags
Then it must contain <tag_keys>
And its value must match the "<tag_value>" regex
Examples:
| tag_keys          | tag_value                                                                                |
| Name              | .+                                                                                |
| Environment       | ^(playground|sandbox|dev|staging|preprod|prod|pc2-pipelines|pc2-service-pipelines)$      |
| Component         | .+                                                                                |
| Service           | .+                                                                                |
| Location          | ^(uk|global1|sg|us|ca)$
Failure:

Scenario Outline: Ensure that specific tags are defined
Given I have resource that supports tags defined
When it has tags
Then it must contain <tag_keys>
And its value must match the "<tag_value>" regex
Examples:
    | tag_keys    | tag_value                                                                           |
    | Name        | .+                                                                                |
            Failure: module.example.module.aws_lambda_function.aws_lambda_function.this (aws_lambda_function) does not have Environment property.
    | Environment | ^(playground|sandbox|dev|staging|preprod|prod|pc2-pipelines|pc2-service-pipelines)$ |
      Failure:
    | Component   | .+                                                                                |

    | Service     | .+                                                                                |
    | Location    | ^(uk|global1|sg|us|ca)$     |

Even by adding the exclude tag, it still fails for this resource. So far the only workaround I found is to skip the whole scenario but that isn't ideal

Hi @hpapadopoulos
Thanks for reporting this issue. I suspect the problem you are facing could be more about using modules and less about using scenario outlines. I'm making some changes on how this tag works.
Does #448 fix your problem? If not, could you convert this report to an issue so that it doesn't get lost.
If you happened to open an issue, could you also share an anonymized plan. It makes reproducing your problem easier :)

Hi @hpapadopoulos
Thanks for reporting this issue. I suspect the problem you are facing could be more about using modules and less about using scenario outlines. I'm making some changes on how this tag works.
Does #448 fix your problem? If not, could you convert this report to an issue so that it doesn't get lost.
If you happened to open an issue, could you also share an anonymized plan. It makes reproducing your problem easier :)

Hi @Kudbettin , apologies for the huge delay in coming back to you. Yup, looks like that PR works ok.
Thank you for looking into that


Example added.

Quantum Harmonic Oscillator, usage of QuDyamic solvers.

@acroy is this ready for the merge ?

@acroy the I have added the printing of trace.

@acroy I have changed the links (the link of QuTiP was a tweaked version of original implementation which involved comparing of results). I have replaced it though. I guess this is ready !

Should be fine.

Merging this !


How to scale binned predictor value back to real raw value scala.

I have a question about scaling back to the raw data scale in the paper. I've read both the paper and the reference paper on quantization of time series, but I'm still unsure. I understand that we transformed the data and then binned it, the the LLM will predict a bin index. Then my question is, how do we scale it back to the raw data scale? Do we use the mid-point value of the bin? I'm concerned that this approach may introduce a resolution issue in the results.
Thanks!

Hi @iphyer, the de-tokenization precisely uses the center point of the bin, see eq (1) in the paper. Then this is scaled back by the same factor used on the input size. The whole thing happens here in the code. There are of course some limitations to this approach that one should be aware of (see Fig. 16 in the paper and associated paragraph) but this does not seem to be a problem for many datasets.

Thank you for the timely answer, really appreciate!


issue with acceptance_fraction sampler property when using sample(...,store=False)

[re emcee version 3.0.3.dev40+g408f422]
I use sample(..., store=False). Most of the important information (to me) is stored and recoverable from the State which EnsembleSampler.sample yields (ensemble.py, line 362). Unfortunately, "accepted" which is computed in ensemble.py line 347 is lost (not recorded, not recoverable), except via the Backend but only if one uses sample(..., store=True).
This is not an issue per se since many EnsembleSampler properties don't make sense when using the sampler via sample(..., store=False) anyways. However, since I want to do my own storing (store=False), in order to track the acceptance_fraction, I have written a (bad, minimal) work-around for myself for now by modifying ensemble.py line 347 to "yield state, accepted".
But I am wondering if maybe you agree this is perhaps a property that should be recoverable after (a) step(s)? If so, it would maybe make sense to add "accepted" as a State property? Or maybe there's a more emcee-minded way to make this info available when running sample(..., store=False)?
Or maybe you don't think that's appropriate at all, in which case please feel completely free to ignore this (non)issue and close it. I just thought I would raise it as a possible issue and see what you think.

@cbeauc: Sorry about the slow response. I totally agree that it would make sense for accepted to be available on State. I think the reason for the current design is historical: State was once a namedtuple and the accept property wasn't known when instantiated. I'd be totally in favor of adding an accepted property to the current State object. Thanks!

@cbeauc: Sorry about the slow response. I totally agree that it would make sense for accepted to be available on State. I think the reason for the current design is historical: State was once a namedtuple and the accept property wasn't known when instantiated. I'd be totally in favor of adding an accepted property to the current State object. Thanks!


add: dn-middleware-rollup

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fuzzy search

Regex and https://github.com/bripkens/fuzzy.js

save output in session store so you can return to search results


Is the plugin working ? (Block VPN ?)

Working ! But the system of command when the ip is anonymous doesn't work where can i replace that by $player->close() in the code ?

Make sure you use the proper format when placing setting commands in the config.yml
command:
- kick %p You seem to be using a VPN or Proxy

And yes you may replace the for loop https://github.com/HiddenMotives/VPNGuard/blob/master/src/basprohop/libraries/Async.php#L75 with $player->close() if you really wish to do so.

Working thanks 👍


train error!

when I use the commend "python tools/train.py configs/ReDet/ReDet_re50_refpn_1x_dota15.py" train DOTA dataset I meet the follow error,how I can do to solve it. Thank you!

Done (t=2.77s)
creating index...
index created!
2021-03-22 11:11:13,786 - INFO - Start running, host: why@why, work_dir: /home/why/DL/ReDet-master/work_dirs/ReDet_re50_refpn_1x_dota15
2021-03-22 11:11:13,787 - INFO - workflow: [('train', 1)], max: 12 epochs
Traceback (most recent call last):
File "tools/train.py", line 95, in
main()
File "tools/train.py", line 91, in main
logger=logger)
File "/home/why/DL/ReDet-master/mmdet/apis/train.py", line 61, in train_detector
_non_dist_train(model, dataset, cfg, validate=validate)
File "/home/why/DL/ReDet-master/mmdet/apis/train.py", line 197, in _non_dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/mmcv/runner/runner.py", line 358, in run
epoch_runner(data_loaders[i], **kwargs)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/mmcv/runner/runner.py", line 264, in train
self.model, data_batch, train_mode=True, **kwargs)
File "/home/why/DL/ReDet-master/mmdet/apis/train.py", line 39, in batch_processor
losses = model(**data)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 150, in forward
return self.module(*inputs[0], **kwargs[0])
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/home/why/DL/ReDet-master/mmdet/models/detectors/base_new.py", line 95, in forward
return self.forward_train(img, img_meta, **kwargs)
File "/home/why/DL/ReDet-master/mmdet/models/detectors/ReDet.py", line 143, in forward_train
*rpn_loss_inputs, gt_bboxes_ignore=gt_bboxes_ignore)
File "/home/why/DL/ReDet-master/mmdet/models/anchor_heads/rpn_head.py", line 51, in loss
gt_bboxes_ignore=gt_bboxes_ignore)
File "/home/why/DL/ReDet-master/mmdet/models/anchor_heads/anchor_head.py", line 177, in loss
sampling=self.sampling)
File "/home/why/DL/ReDet-master/mmdet/core/anchor/anchor_target.py", line 63, in anchor_target
unmap_outputs=unmap_outputs)
File "/home/why/DL/ReDet-master/mmdet/core/utils/misc.py", line 24, in multi_apply
return tuple(map(list, zip(*map_results)))
File "/home/why/DL/ReDet-master/mmdet/core/anchor/anchor_target.py", line 108, in anchor_target_single
cfg.allowed_border)
File "/home/why/DL/ReDet-master/mmdet/core/anchor/anchor_target.py", line 173, in anchor_inside_flags
(flat_anchors[:, 2] < img_w + allowed_border) &
RuntimeError: Expected object of scalar type Byte but got scalar type Bool for argument #2 'other' in call to _th_and

We have tested on Pytorch1.1/1.3. Try lower version Pytorch or see here

Thanks for your replay! I have update my Pytorch==1.1,CUDA=10.0, the up problem have solved,but I meet a new problem
inside_flags4 = torch.tensor(valid_flags, dtype=torch.uint8)
THCudaCheck FAIL file=src/riroi_align_kernel.cu line=389 error=7 : too many resources requested for launch
Traceback (most recent call last):
File "tools/train.py", line 95, in
main()
File "tools/train.py", line 91, in main
logger=logger)
File "/home/why/DL/ReDet-master/mmdet/apis/train.py", line 61, in train_detector
_non_dist_train(model, dataset, cfg, validate=validate)
File "/home/why/DL/ReDet-master/mmdet/apis/train.py", line 197, in _non_dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/mmcv/runner/runner.py", line 358, in run
epoch_runner(data_loaders[i], **kwargs)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/mmcv/runner/runner.py", line 271, in train
self.call_hook('after_train_iter')
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/mmcv/runner/runner.py", line 229, in call_hook
getattr(hook, fn_name)(self)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/mmcv/runner/hooks/optimizer.py", line 17, in after_train_iter
runner.outputs['loss'].backward()
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/tensor.py", line 107, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/autograd/init.py", line 93, in backward
allow_unreachable=True)  # allow_unreachable flag
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/autograd/function.py", line 77, in apply
return self._forward_cls.backward(self, *args)
File "/home/why/DL/ReDet-master/mmdet/ops/riroi_align/functions/riroi_align.py", line 58, in backward
grad_input)
RuntimeError: cuda runtime error (7) : too many resources requested for launch at src/riroi_align_kernel.cu:389
How I can do to solve it.
Thank you!

@why228430
How do you solve the first problem?
How I can do to solve it.
Thank you!

@why228430 Have you re-compiled the ops after your re-installation of Pytorch? Please del all *.so and re-compile the ops.

@why228430 Have you re-compiled the ops after your re-installation of Pytorch? Please del all *.so and re-compile the ops.

del all *.so and re-compile the ops.
but the problem still exists
How I can do
Thank you!

@why228430 Have you re-compiled the ops after your re-installation of Pytorch? Please del all *.so and re-compile the ops.

del all *.so and re-compile the ops.
but the problem still exists
How I can do
Thank you!

@1127244933 What is your GPU? I have tested on V100 and Titan Xp with Pytorch=1.3.1, cuda=10.1.
Besides, is the inference (demo_large_image.py) ok?

Thank you so much for answering my question in the night!
I did as you said, but the problem still exists !
My GPU is 2080ti with Pytorch=1.1.0, cuda=10.0.
When I try inference demo_large_image.py, I meet the "No module '_polyiou'"!

For the polyiou, you need:
sudo apt-get install swig
cd DOTA_devkit
swig -c++ -python polyiou.i
python setup.py build_ext --inplace

For the riroi_align, I suggest to change THREADS_PER_BLOCK from 1024 to 512. Whether it's ok or not, please infer me to fix the bug.
https://github.com/csuhan/ReDet/blob/7430068dbffa008bf5db158d225e87de4f2bbfa5/mmdet/ops/riroi_align/src/riroi_align_kernel.cu#L11

First I update the demo!
then I del all *.so and re-compile the ops.
finally I  use recommend "python setup.py develop"
the "polyiou" has solved !
but the " RuntimeError: cuda runtime error (7) : too many resources requested for launch at src/riroi_align_kernel.cu:389" problem still exists
Thank you!

I find from the issue that someone has run  code .but I also meet the " RuntimeError: cuda runtime error (7) : too many resources requested for launch at src/riroi_align_kernel.cu:389" problem. Is it problem with my environment?

@why228430 try lower like 256.
I met same problem on 2080ti, 768 is okay on my platform

@why228430 @wangchengtza  I met same problem on Tesla T4. And I have tried even 64 for THREADS_PER_BLOCK . Problem still exists.
My env:
GPU: Tesla T4
Pytorch: 1.3.1 with cudatoolkit=10.0
How I can do...
Thank you!

After change THREADS_PER_BLOCK, you should re-compile "bash compile.sh"!

After change THREADS_PER_BLOCK, you should re-compile "bash compile.sh"!

Yes, I did re-compile "bash compile.sh". But problem still exists. Did my environment cause the error? I use docker to impl this project.
docker: pytorch/pytorch:1.3-cuda10.1-cudnn7-devel
I have installed ReDet following official instructions.

I don't use docker! I am not sure if your environment problem!
My environment is 2080ti with Pytorch=1.1.0, cuda=10.0.
In my platform the project is okay!

@why228430 Thanks a lot. I'll try new env. What is the training speed in your platform (2080Ti)?

{"mode": "train", "epoch": 1, "iter": 100, "lr": 0.00465, "time": 1.45851, "data_time": 0.21686, "memory": 4806, "loss_rpn_cls": 0.29471, "loss_rpn_bbox": 0.11749, "s0.rbbox_loss_cls": 0.48488, "s0.rbbox_acc": 88.70508, "s0.rbbox_loss_bbox": 0.71225, "s1.rbbox_loss_cls": 0.32175, "s1.rbbox_acc": 92.81233, "s1.rbbox_loss_bbox": 0.22031, "loss": 2.15139}
{"mode": "train", "epoch": 1, "iter": 150, "lr": 0.00532, "time": 1.49202, "data_time": 0.11837, "memory": 4806, "loss_rpn_cls": 0.20144, "loss_rpn_bbox": 0.08253, "s0.rbbox_loss_cls": 0.47648, "s0.rbbox_acc": 88.93945, "s0.rbbox_loss_bbox": 0.51546, "s1.rbbox_loss_cls": 0.35854, "s1.rbbox_acc": 91.87267, "s1.rbbox_loss_bbox": 0.39948, "loss": 2.03393}
{"mode": "train", "epoch": 1, "iter": 200, "lr": 0.00599, "time": 1.77337, "data_time": 0.17864, "memory": 4806, "loss_rpn_cls": 0.18083, "loss_rpn_bbox": 0.09426, "s0.rbbox_loss_cls": 0.47266, "s0.rbbox_acc": 88.01562, "s0.rbbox_loss_bbox": 0.47843, "s1.rbbox_loss_cls": 0.3567, "s1.rbbox_acc": 90.61376, "s1.rbbox_loss_bbox": 0.42733, "loss": 2.01021}
{"mode": "train", "epoch": 1, "iter": 250, "lr": 0.00665, "time": 1.44145, "data_time": 0.19534, "memory": 4806, "loss_rpn_cls": 0.15557, "loss_rpn_bbox": 0.09309, "s0.rbbox_loss_cls": 0.49047, "s0.rbbox_acc": 86.23047, "s0.rbbox_loss_bbox": 0.52675, "s1.rbbox_loss_cls": 0.40528, "s1.rbbox_acc": 88.74186, "s1.rbbox_loss_bbox": 0.5826, "loss": 2.25377}
Here are some logs in my training process ！

As I am not familiar with CUDA programming, the current code is not so efficient and robust.
I will continue to optimize the code, and pulling requests are welcomed!

We have tested on Pytorch1.1/1.3. Try lower version Pytorch or see here

Using pytorch 1.3.1, met the same problem ...

Hello, I meet the problem that 'PyFloTorch_CHECKExact' was not declared in this scope in src/soft_nms_cpu.cpp:765:36

To solve the " RuntimeError: cuda runtime error (7) : too many resources requested for launch at src/riroi_align_kernel.cu:389" problem. I suggest to reduce the value of THREADS_PER_BLOCK
https://github.com/csuhan/ReDet/blob/7430068dbffa008bf5db158d225e87de4f2bbfa5/mmdet/ops/riroi_align/src/riroi_align_kernel.cu#L11

what should the value of THREADS_PER_BLOCK be？

when I use the commend "python tools/train.py configs/ReDet/ReDet_re50_refpn_1x_dota15.py" train DOTA dataset I meet the follow error,how I can do to solve it. Thank you!

Done (t=2.77s)
creating index...
index created!
2021-03-22 11:11:13,786 - INFO - Start running, host: why@why, work_dir: /home/why/DL/ReDet-master/work_dirs/ReDet_re50_refpn_1x_dota15
2021-03-22 11:11:13,787 - INFO - workflow: [('train', 1)], max: 12 epochs
Traceback (most recent call last):
File "tools/train.py", line 95, in
main()
File "tools/train.py", line 91, in main
logger=logger)
File "/home/why/DL/ReDet-master/mmdet/apis/train.py", line 61, in train_detector
_non_dist_train(model, dataset, cfg, validate=validate)
File "/home/why/DL/ReDet-master/mmdet/apis/train.py", line 197, in _non_dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/mmcv/runner/runner.py", line 358, in run
epoch_runner(data_loaders[i], **kwargs)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/mmcv/runner/runner.py", line 264, in train
self.model, data_batch, train_mode=True, **kwargs)
File "/home/why/DL/ReDet-master/mmdet/apis/train.py", line 39, in batch_processor
losses = model(**data)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 150, in forward
return self.module(*inputs[0], **kwargs[0])
File "/home/why/anaconda3/envs/redet/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/home/why/DL/ReDet-master/mmdet/models/detectors/base_new.py", line 95, in forward
return self.forward_train(img, img_meta, **kwargs)
File "/home/why/DL/ReDet-master/mmdet/models/detectors/ReDet.py", line 143, in forward_train
*rpn_loss_inputs, gt_bboxes_ignore=gt_bboxes_ignore)
File "/home/why/DL/ReDet-master/mmdet/models/anchor_heads/rpn_head.py", line 51, in loss
gt_bboxes_ignore=gt_bboxes_ignore)
File "/home/why/DL/ReDet-master/mmdet/models/anchor_heads/anchor_head.py", line 177, in loss
sampling=self.sampling)
File "/home/why/DL/ReDet-master/mmdet/core/anchor/anchor_target.py", line 63, in anchor_target
unmap_outputs=unmap_outputs)
File "/home/why/DL/ReDet-master/mmdet/core/utils/misc.py", line 24, in multi_apply
return tuple(map(list, zip(*map_results)))
File "/home/why/DL/ReDet-master/mmdet/core/anchor/anchor_target.py", line 108, in anchor_target_single
cfg.allowed_border)
File "/home/why/DL/ReDet-master/mmdet/core/anchor/anchor_target.py", line 173, in anchor_inside_flags
(flat_anchors[:, 2] < img_w + allowed_border) &
RuntimeError: Expected object of scalar type Byte but got scalar type Bool for argument #2 'other' in call to _th_and

改成这样好像就可以了

@why228430 你的第一个问题，我是这样解决的，你可以试一下
把出错地方的代码改成这样

RuntimeError: cuda runtime error (701) : too many resources requested for launch at src/riroi_align_kernel.cu:389
这个错误可以修改ReDet/mmdet/ops/riroi_align/src/riroi_align_kernel.cu里的 #define THREADS_PER_BLOCK 1024，改成256
然后删掉所有的*.so文件，然后执行下面的命令
bash compile.sh
python setup.py develop
swig -c++ -python polyiou.i
python setup.py build_ext --inplace
然后再重新训练模型


test-branch/execute-job/comment-from-wrong-user-on-pr/76eaf362-fd05-4482-bb8f-09f9b95bf2be

PR for testing 9f04a7a3eade15402ee8abb0a6e72241cfe8d615

/canonical/self-hosted-runners/run-workflows 9f04a7a3eade15402ee8abb0a6e72241cfe8d615


[stable/chart/drone] could not configure bitbucket server as a source control.

Describe the bug
To configure bitbucket server as source control, I dont see the required values defined in the values.yaml file and there is no entry in template for bitbucket server.
Version of Helm and Kubernetes:
Helm version: 2.14.2 K8's: 1.15
Which chart:
drone
What happened:
pods are not getting created since some values definition is missing
What you expected to happen:
It should create the drone server pod up and running
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:

I think this is more generic - I am unable to setup gitlab as well nor haven't found any article or mention someone successfully doing that. I can do it with plain docker, but not with helm on k8s. The documentation and comments are quite confusing, I wasn't able to do it by myself.
I was able to make it work for github though. So I don't think I am that incompetent. But for github, there is at least example.
Could I maybe kindly ask @raunyoliveira-hotmart as the one of the last editors of values.yaml?

@hnykda sorry for the late reply, I'm going to try both: bitbucket and gitlab and let you guys know if it works.

I encountered this issue too, the difference is I'm trying to set bitbucketCloud as the source control.
Describe the bug
A clear and concise description of what the bug is.
Version of Helm and Kubernetes:
Helm: v3.0.0
K8s: 1.16.2-do.0
Which chart:
stable/drone
What happened:
OAuth doesn't work.
What you expected to happen:
Bitbucket OAuth Confirm access to your account.
How to reproduce it (as minimally and precisely as possible):
Deploy stable/drone helm chart with following value.yml
sourceControl:
  provider: bitbucketCloud
  github:
    clientID: xxxx
    clientSecretKey: clientSecret
    clientSecretValue: xxxx
  bitbucketCloud:
    clientID: xxxxx
    clientSecretKey: clientSecret
    clientSecretValue: xxxx
  server:
    host: xxxx
    protocol: http

Anything else we need to know:
If I change the source control to github, it works.
I have deployed another Drone Server using docker instead of helm chart, BitBucket OAuth works there.


Does CpuWaitResetCollector collect only one episode in each batch?

It says in CpuWaitResetCollector that

It waits to reset any environments with completed episodes until after
the end of collecting the batch, i.e. the done environment is bypassed
in remaining timesteps, and zeros are recorded into the batch buffer.

Does it mean that there is always only one episode in each batch? I.e., if batch_B=1, batch_T=500, episode length=1, there will be 1 data and 499 zeros in a batch

Yes, that is correct.

Thanks


Changed to import PropTypes from 'prop-types';

I added import PropTypes from 'prop-types'; Oh, don't forget to do a npm install.

Thanks! But I think you need to merge into devel, not in master?

@SachaG I am a bit confused. For whom is your comment?

Oh wait did I merge this? Hmm I think that was a mistake on my part… Hopefully it doesn't matter too much, but I meant that you made the PR to master instead of devel.

@SachaG my pull request was to devel not Master. That was another reason why I was confused. Also, none of my changes is merged. Wanna do this again?

Yeah I think you better do another PR based on the latest devel. Not sure what happened here…


Remove not needed code

Hey @kant2002 ,
why is this no longer needed?

Hey @kant2002 ,
why is this no longer needed?

Oh I get it, the bug is fixed, so this is no longer needed.


Please clarify the License Policy

Hi DataBricks,
I am not sure which license shall be applied to the Dolly-v2-12b so that I cannot give proper credits to your guys when I distribute your model.
In your Model View(https://github.com/databrickslabs/dolly#model-overview), it states that, dolly-v2 -12b applies CC-BY-SA, but do not specify the specifical version of CC-BY-SA. As you know, there is four versions of CC-BY-SA (See https://opendefinition.org/licenses/cc-by-sa/). What makes me more confused is that the LICENSE shows Apache 2.0 License text.
So just clarify the license applied to the Dolly-v2-12b model and many thanks to you

Duplicate of https://huggingface.co/databricks/dolly-v2-12b/discussions/86
No, it says that the Dolly 15K dataset used for fine-tuning is licensed as CC-BY-SA 3.0, and that is elaborated at https://huggingface.co/datasets/databricks/databricks-dolly-15k
The model card says the model itself (weights) are MIT licensed.


Installation Problem under Big Sur

I have noticed the following problems installing organize under Big Sur

"pip3 install -U organize-tool[textract]" is not working

pip3 install -U organize-tool[textract]
zsh: no matches found: organize-tool[textract]

"pip3 install -U organize-tool" is running but results in a couple of warnings.

WARNING: Value for scheme.headers does not match. Please report this to <https://github.com/pypa/pip/issues/9617>
distutils: /Library/Frameworks/Python.framework/Versions/3.9/include/python3.9/UNKNOWN
sysconfig: /Library/Frameworks/Python.framework/Versions/3.9/include/python3.9
WARNING: Additional context:
user = False
home = None
root = None
prefix = None
Requirement already satisfied: organize-tool in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (1.10.1)
Requirement already satisfied: exifread<3.0,>=2.1 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (2.3.2)
Requirement already satisfied: docopt<0.7.0,>=0.6.2 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (0.6.2)
Requirement already satisfied: colorama<0.5.0,>=0.4.4 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (0.4.4)
Requirement already satisfied: pendulum<3.0.0,>=2.0.5 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (2.1.2)
Requirement already satisfied: macos-tags<2.0.0,>=1.5.1 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (1.5.1)
Requirement already satisfied: appdirs<2.0.0,>=1.4.4 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (1.4.4)
Requirement already satisfied: simplematch<2.0,>=1.3 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (1.3)
Requirement already satisfied: Send2Trash<2.0.0,>=1.5.0 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (1.5.0)
Requirement already satisfied: PyYAML<6.0.0,>=5.4.1 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from organize-tool) (5.4.1)
Requirement already satisfied: mdfind-wrapper<0.2.0,>=0.1.3 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from macos-tags<2.0.0,>=1.5.1->organize-tool) (0.1.4)
Requirement already satisfied: xattr<0.10.0,>=0.9.7 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from macos-tags<2.0.0,>=1.5.1->organize-tool) (0.9.7)
Requirement already satisfied: pytzdata>=2020.1 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from pendulum<3.0.0,>=2.0.5->organize-tool) (2020.1)
Requirement already satisfied: python-dateutil<3.0,>=2.6 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from pendulum<3.0.0,>=2.0.5->organize-tool) (2.8.1)
Requirement already satisfied: six>=1.5 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from python-dateutil<3.0,>=2.6->pendulum<3.0.0,>=2.0.5->organize-tool) (1.15.0)
Requirement already satisfied: cffi>=1.0.0 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from xattr<0.10.0,>=0.9.7->macos-tags<2.0.0,>=1.5.1->organize-tool) (1.14.5)
Requirement already satisfied: pycparser in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages (from cffi>=1.0.0->xattr<0.10.0,>=0.9.7->macos-tags<2.0.0,>=1.5.1->organize-tool) (2.20)
WARNING: Value for scheme.headers does not match. Please report this to <https://github.com/pypa/pip/issues/9617>
distutils: /Library/Frameworks/Python.framework/Versions/3.9/include/python3.9/UNKNOWN
sysconfig: /Library/Frameworks/Python.framework/Versions/3.9/include/python3.9
WARNING: Additional context:
user = False
home = None
root = None
prefix = None

"filecontent" Filter is not working. Probably related to missing "textract" module

    - (FileContent) ERROR! textract is not installed. Install with pip install organize-tool[textract]
    - (FileContent) ERROR! textract is not installed. Install with pip install organize-tool[textract]
    - (FileContent) ERROR! textract is not installed. Install with pip install organize-tool[textract]

I also noted pretty different behaviour running the installation under Mojave.

a. no warnings
b. the "textract" module is working although I have not used the "pip3 install -U organize-tool[textract]" Command
c. I cannot start Organizer by e.g. "organizer some command" but always have to write "python3 -m  organize some command"

I found a workaround.
It seems that the installation scripts are not compatible with the new default shell which is "zsh shell".
When switching to back to "bash shell" the installation was working and also textract installed fine.
But as in Mojave I have to use "python3 - m " prefix to start organize.

The workaround is only partly working.
In python filter scripts using "import textract" I am getting an error
    - (Python) ERROR! The command xxxx failed because the executable
    `pdf2txt.py` is not installed on your system. Please make
    sure the appropriate dependencies are installed before using
    textract:

        http://textract.readthedocs.org/en/latest/installation.html

the "filecontent" is not working independent of the shell I am using.

Can you try this command:
pip3 install -U "organize-tool[textract]"

zsh seems to go into grep mode when it sees []

And please try
pip3 install pdfminer

pip3 install -U "organize-tool[textract]" under zsh shows the same log file as in bash but it resolves the issue running Big Sur (x86) Architecture.
Will repeat the test with an M1 Mac. Update following

Great, thanks for the heads up!


Add a basic set of file I/O primitives.

This is mostly for review purposes, to see whether you think I'm on the right track.
This adds completely untested support for basic stdio-like file access primitives; open, close, read, write, seek, tell, size, truncate. You can have up to eight files open, with user-side numbering. File offsets are 32 bits (but of course you can only read or write 64kB at a time).
I don't know how much spare memory the Pico has; each file structure contains a 512-byte buffer, so the eight files here use slightly more than 4kB.
Missing features: directory access, creating directories, deletion.
What do you think?
See also: https://github.com/davidgiven/cpm65/issues/55.

Pretty much what I had in mind.  If you are still okay with this what I suggest we do is to merge the pull request and assuming no issues (can't see why !) not document it immediately, just in case you need to tweak something.

Well, this is the easy bit. The next bit I need is directory listing, which will be much less fun.
BTW: I noticed that DSPGetString() always copies to a fixed size 32-byte buffer, which is a bit small for paths. Is there a reason why it does a copy rather than just returning a pointer to the string in cpuMemory? It should be nul-terminated there, and therefore safe to use.
Also, is ff compiled with long filename support?

(Also, happy to merge this. The biggest thing which might need tweaking are the return error numbers, which are currently just 0 or 1, but that can happen later.)

static const char* const modes[] = {
"rb",	// 0: FIOMODE_READ
"r+b", 	// 1: FIOMODE_WRITE
"r+b",	// 2: FIOMODE_RDWR
"w+b",	// 3: FIOMODE_CREATE
};
if (mode >= 3)
return 1;
Not sure this is correct. mode can be 3.

Yikes. Yes, that's wrong. Fixed. I've also adjusted the FIOMODE constant names a bit for clarity.

Or, rather obviously (!) I could change the filing system functions in BASIC to use ASCIIZ and convert them there. This would not be particularly difficult and might actually be the best solution.

I've added an important missing feature, which is the ability to close all files (by trying to close file handle -1).

Would it be easier to convert the current system to ASCIIZ ?

I think that ASCIIZ is the more boring option, as it's the one which most C and assembler programmers will be used to (I just kind of assumed that myself, even after I'd looked at the conversion function). So I think switching does make sense. But it would be really disruptive to users. neobasic itself isn't an issue, but there's a third-party Pascal --- is there anyone else?
From my side it makes no difference, as from machine code the difference between constructing a Pascal string vs a C string is a couple of instructions.

No, there's no one else. I'll ask the Pascal guy.


Unexpected argument. Enter '-?' for help.

Hello,
I tried to mount/create automatically a database (it works 3 months ago).
I executed this command:
docker run -d -p 1433:1433 -v C:/Users/florent/Desktop/DEMO08/DB/:C:/Temp/ --env attach_dbs="[{'dbName':'Orchard','dbFiles':['C:\\Temp\\Orchard.mdf','C:\\Temp\\Orchard_log.ldf']}]" -e sa_password=‘Welcome123!’ -e ACCEPT_EULA=Y --name sqlserver microsoft/mssql-server-windows:latest
In logs of the container I've this:

VERBOSE: Starting SQL Server
VERBOSE: Changing SA login credentials
VERBOSE: Attaching 1 database(s)
VERBOSE: Invoke-Sqlcmd -Query sp_detach_db "Orchard";CREATE DATABASE "Orchard"
ON (FILENAME = N'C:\Temp\Orchard.mdf'),(FILENAME = N'C:\Temp\Orchard_log.ldf')
FOR ATTACH ;
Sqlcmd: 'Orchard";CREATE DATABASE "Orchard" ON (FILENAME = N'C:\Temp\Orchard.mdf'),(FILENAME = N'C:\Temp\Orchard_log.ldf
') FOR ATTACH ;"': Unexpected argument. Enter '-?' for help.
VERBOSE: Started SQL Server.

Any idea?
Thanks.
Florent

I have the same error... Please fix!

#173 seems like it's the same (see for workaround)

Just ran into this issue; running the generated CREATE DATABASE statement directly with sqlcmd throws the same error. For some reason, removing the double quotes from around the database name makes it run -- I though double quotes were valid?

Hi,
I tried your solution but doesn't work:

At line:1 char:103

... top/DEMO08/DB/:C:/Temp/ --env attach_dbs=[{'dbName':'Orchard','dbFile ...

                                                   ~~~~~~~~~~

Unexpected token ':'Orchard'' in expression or statement.
At line:1 char:113

... op/DEMO08/DB/:C:/Temp/ --env attach_dbs=[{'dbName':'Orchard','dbFiles ...

                                                            ~

Missing argument in parameter list.
+ CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : UnexpectedToken

Florent

Faced the same problem with the latest -express and -developer images.
The only 2017 image that worked for me was microsoft/mssql-server-windows, but looks like it was dropped from DockerHub recently.
Our build infrastructure literally crashed because of this problem :(

@Flodu31 I mean the double quotes that get wrapped around the dbName in start.ps1 (https://github.com/Microsoft/mssql-docker/blob/bcea633a5c1557b9961e805220d89c03c76f89a6/windows/mssql-server-windows-developer/start.ps1#L72)
This wrapping wasn't always happening, looking at the history of that file.. but some kind of quoting is required for names with spaces or other weird characters. Maybe changing the double quotes to square braces would make sqlcmd happy.

[Update] Sorry for the delayed response on this issue. There was a bug in the ps script that we believe is now fixed. The latest developer and express images should now work with the attach_dbs environment variable.

microsoft/mssql-server-windows-express (https://hub.docker.com/r/microsoft/mssql-server-windows-express/)
microsoft/mssql-server-windows-developer (https://hub.docker.com/r/microsoft/mssql-server-windows-developer/)

Could you please verify that this problem is fixed on your end before we close this issue?
Thanks,
Perry

Thanks @perrysk-msft! It works for me now, but I see the following in the logs when attaching:
Msg 15010, Level 16, State 1, Server 1FD90C3F14BA, Procedure sp_detach_db, Line 40 The database 'sign' does not exist. Supply a valid database name. To see available databases, use sys.databases.
I think the statement at https://github.com/Microsoft/mssql-docker/blob/cd10b8f345eb07feedf1a31759e556f41fff2185/windows/mssql-server-windows-developer/start.ps1#L53 should have IF EXISTS before attempting to detach the database, not IF NOT EXISTS.

@joshuashort - Good catch!
I'll rebuild and re-publish both the Express and Developer images by the end of today. I'll update this thread when this has been completed.

[update]
The updated images for Developer and Express with the "IF EXISTS" fix are now published on Docker Hub.
Thanks again @joshuashort!

Thank you so much! I've tested our dev setup with the new version, and it's working beautifully. It'd be nice to have a hook to run some custom SQL or PowerShell, but this seems to fix the issue.
Great appreciated!

Hello,
Ok I've the same problem with the default image (who is not available anymore on Docker Hub).
But it works with the express version.
Thanks for the update.
Florent


Getting result after stream completed

Is your feature request related to a problem? Please describe.
So, now when you're using streaming feature after its over there's no information provided as a result of method invoke. Here's the example
@client = OpenAI::Client.new(access_token: 'myToken')
@response = @client.chat(
      parameters: {
        model: 'gpt-3.5-turbo-16k', # Required.
        messages: [*messages],
      stream: proc do |chunk, _bytesize|
        doSomething(chunk)
      end
})

puts @response
=> []

Its not returning errors either, but there's open PR to fix that. But it also doesnt have any information about tokens user or anything else returned by openAi api.
Describe the solution you'd like
I'd want @response object to behave same way for stream/non stream types of request
Describe alternatives you've considered
Additional context

Ok, I did a bit of debugging and seems like it's the Faraday gem issue. Nothing comes in response. Closing as not relevant to this gem

Thanks for following up!


parse errors improvements

removed 'Parse Error' prefix
included token name into 'unexpected token' error

Also part of https://github.com/sorbet/sorbet/pull/1670

Motivation

Test plan

See included automated tests.

@elliottt I've rebased my branch, fixed your comments and updated  https://github.com/sorbet/sorbet/commit/5b610553607e42fb31586b7fc06680fe54d66599 to have a new error message

Thank you!

Looks like the build got stuck. @elliottt could you restart it please?

I'm not sure of a better way to restart it, but I changed the title to include [wip] and then changed it back.


add the option to use ticksArray as array of objects, with legend and value properties

same as we do with steps array, but to customize the ticks without limiting the selection steps.
I know this can be already done with a variety of methods, but AFAIK only with functions. In my case I store these labels in a mongo db, so storing logic is not ideal. Putting those labels in a table of objects without limiting the selection as stepsArray solves both issues, storing and displaying.

Codecov Report

Merging #662 into master will decrease coverage by 0.2%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master     #662      +/-   ##
==========================================
- Coverage     100%   99.79%   -0.21%
==========================================
  Files           1        1
  Lines         970      974       +4
==========================================
+ Hits          970      972       +2
- Misses          0        2       +2

Impacted Files
Coverage Δ

src/rzslider.js
99.79% <60%> (-0.21%)
: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 64c3f86...8850a54. Read the comment docs.

Hey @vdiez, thanks for this PR!
Could you create a JSFiddle showing how to use this option? For this you can fork the example one from the ReadMe and point the script (angular-slider.js) to your repo instead of this one. You'll need to run the build script on your PR to generate the different needed files.
Also, please provide a unit test for this option so we keep the coverage at the same level as before.
Thanks :)

Hi @ValentinH,
fiddle here: http://jsfiddle.net/3jgzv4wc/1/
Cheers

Released under 6.7.0 🎉


Runtime Analysis is determined complete on multiple checks.

Fixes: #1403
feat(projectStateService.ts): enhance hasInvestigatedFindings check
test(instructions.test.ts): add test for runtime analysis status transitions

Apologies for not letting you know earlier, but we don't want to add system tests because they seem to be inherently flaky and fail intermittently. Can you make the system tests into integration tests or unit tests? Thanks!

Added integration tests, have used repeatUntil otherwise it used to fail. Let me know if that's alright.

Thanks for converting the system test into an integration test. Using repeatUntil is fine.
I just tested this out, and if I generate maps, but don't complete the Explore AppMaps step, then I click on the Runtime Analysis step it shows a confusing message that setup is complete for the project:

Can you make it so that the Runtime analysis step is only marked as completed if all of the other instruction steps are completed? Thanks!

It looks like your changes caused one of the system tests to fail. Can you fix that? Thanks!

Fixed.


defmt-test: Allow use in test module

Hey, I'd like to propose a feature request:
Support use statements in the #[defmt_test::tests] macro.
It is pretty common to import the output scope via use super::*;.
But this is currently not possible:
 testsuite/tests/test.rs | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/testsuite/tests/test.rs b/testsuite/tests/test.rs
index d1002ce..7a8d75b 100644
--- a/testsuite/tests/test.rs
+++ b/testsuite/tests/test.rs
@@ -7,6 +7,8 @@ use {{crate_name}} as _; // memory layout + panic handler
 // feature)
 #[defmt_test::tests]
 mod tests {
+    use super::*;
+
     #[test]
     fn assert_true() {
         assert!(true)

$ cargo test
   ...
   Compiling use-test v0.1.0 (/tmp/use-test)
   Compiling testsuite v0.1.0 (/tmp/use-test/testsuite)
error: only function items are allowed
  --> testsuite/tests/test.rs:10:5
   |
10 |     use super::*;
   |     ^^^

error: aborting due to previous error

error: could not compile `testsuite`

As we can see, this case is currently just not taken into consideration:
https://github.com/knurling-rs/defmt/blob/2657a8db30a67c7f225611e2c14cb2759df876da/firmware/defmt-test/macros/src/lib.rs#L133-L138
The current workaround I use, it to declare the use statement in the function scope, e.g.
 #[defmt_test::tests]
 mod tests {
-    use super::*;
-
     #[test]
     fn assert_true() {
+        use super::*;
+
         assert!(true)

If you point me into the right directions I could try to implement this
feature myself. But I've no experience writing procedural macros, so any help
is appreciated :)

This would be a great addition!
I think in terms of implementations this should work:

collect use items ("imports") in a vector in this for loop (the imports correspond to the Item::Use variant): https://github.com/knurling-rs/defmt/blob/2657a8db30a67c7f225611e2c14cb2759df876da/firmware/defmt-test/macros/src/lib.rs#L37

include those imports in the output right before this line: https://github.com/knurling-rs/defmt/blob/2657a8db30a67c7f225611e2c14cb2759df876da/firmware/defmt-test/macros/src/lib.rs#L197 -- the quote! syntax for lists (e.g. vectors) is #(#list)*; you'll want to interpolate the whole vector you collected in (1)

check the changes using cargo-expand. Starting from the app-template, use the [patch] section in the root Cargo.toml to use your local version

[patch.crates-io]
# NOTE change the `file` path and the `branch` here to match your local setup
defmt = { git = "file:///home/japaric/rust/defmt", branch = "allow-imports" }
defmt-rtt = { git = "file:///home/japaric/rust/defmt", branch = "allow-imports" }
# (..)

With those changes, run cargo expand -p testsuite --test test to see the expansion of the testsuite/tests/test.rs file. Modify test.rs file and rerun cargo-expand to see if imports are being included in the output.
NOTE: all changes in you your local defmt repository must be committed or app-template won't pick them up! Also, you need to run cargo update from app-template to make it pick up new commits in your local defmt repository


fix: Create more debug logs to aid in troubleshooting

Just added some logs to maybe get a better sense of what's happening under the hood.

Tested and working 👍


suggestion: consider making the delay configurable

might set a minimum delay to prevent the router from getting hammered.
from here: https://www.reddit.com/r/2007scape/comments/1dsisty/comment/lb6p4vm/?utm_source=share&utm_medium=web3x&utm_name=web3xcss&utm_term=1&utm_content=share_button

Seems this should be prioritized...

closed by #7


[lstm] Update LSTMLayer so as to support Keras  'return_state' functionality

Hi,
I am trying to modify the LSTMLayer in order to support the functionality of the return_state=True of Keras LSTM layer. Thus, I need to return 3 tensors as the output of the layer.
In the LSTMLayer in the finalize function, I have set the number of output dimension as three (3).
const TensorDim output_dim(batch_size, 1, return_sequences ? max_timestep : 1,
                             bidirectional ? 2 * unit : unit);
const TensorDim output_hidden_state_dim(batch_size, 1, 1,
                             bidirectional ? 2 * unit : unit);
const TensorDim output_cell_state_dim(batch_size, 1, 1,
                             bidirectional ? 2 * unit : unit);
context.setOutputDimensions({output_dim, output_hidden_state_dim, output_cell_state_dim});

In the forwarding function, I get the three output tensors as follows:
static constexpr size_t RETURN_STATE_OUT_IDX = 0;
static constexpr size_t RETURN_STATE_HIDDEN_IDX = 1;
static constexpr size_t RETURN_STATE_CELL_IDX = 2;

Tensor &output = context.getOutput(RETURN_STATE_OUT_IDX);
Tensor &output_hidden = context.getOutput(RETURN_STATE_HIDDEN_IDX);
Tensor &output_cell = context.getOutput(RETURN_STATE_CELL_IDX);

And then I fill in the tensors accordingly.
My problem is that, while in the LSTMLayer::forwarding when we print out the dimension of the output we get the expected size (3) , when we call the inference of the graph having as its last layer the LSTMLayer, then the number of the returned output tensors is 1.
For example:
I define the following graph:
  std::shared_ptr<ml::train::Layer> input_layer = ml::train::layer::Input({"input_shape=1:1:1:320", "trainable=false", "name=input0"});
  std::shared_ptr<ml::train::Layer> lstm_layer  = ml::train::layer::LSTM({
                    "name=conformer_prediction_lstm_0",
                    "unit= 320", "activation=tanh", "return_sequences=true",
                    "recurrent_activation=sigmoid", "integrate_bias=true",
                    "input_layers=input0", "trainable=false"
  });

predict_net = ml::train::createModel(ml::train::ModelType::NEURAL_NET);

predict_net_NN.addLayer(input_layer);
predict_net_NN.addLayer(lstm_layer);

predict_net_NN.compile();
predict_net_NN.initialize();

When I try to call the inference:
nntrainer::Tensor predicted_tensor(nntrainer::TensorDim(1,1,1,320));
predicted_tensor.setZero();
auto out = predict_net_NN.inference({MAKE_SHARED_TENSOR(predicted_tensor)});

the size of out is of dimension=1
Is this the expected dimension for the output? I was thinking that it should be 3.
Can you please advise me?
Thank you in advance

Hi alexzami.
Sorry for late reply.
That's because the key string of hidden state activation property for lstm is "hidden_state_activation" not "activation".
So if you modify lstm layer as
  std::shared_ptr<ml::train::Layer> lstm_layer  = ml::train::layer::LSTM({
                    "name=conformer_prediction_lstm_0",
                    "unit= 320", "hidden_state_activation=tanh", "return_sequences=true",
                    "recurrent_activation=sigmoid", "integrate_bias=true",
                    "input_layers=input0", "trainable=false"
  });

it will return output size of 3.
FYI activation property means that lstm layer needs following activation layer so that's why the output is 1.
If you have any further problem please leave it in issues :)

Thank you for the quick reply. that solves the issue :)

Can I close this issue if there is no other problems?

One minor comment
I tried to pass a second input to the LSTM layer, in order to initialize the cell state and to implement the "initial_state" function of Keras
in this case, I noticed that the output size is 4 instead of 3
not a big problem for me. I post this as an observation


A wonderful way to get Started in Visual Studio/C++

Thank you for this tutorial - I am a mentor for final year System Engineering Students tasked with building a device based on Arduino Nano and a pressure transducer.  I specifically asked for the code for the GUI to be written using Visual Studio, and C++. Therefore I thought I should learn Visual Studio and C++ myself so as to be a better mentor.
I've programmed in other languages using other IDEs (editors really) many years ago, so I found the learning curve steep. I trolled YouTube for tutorials... wonderful for learning other things in the past but not for this! So I thought that I would go to get instruction right from the "horses mouth".
Your tutorial founded on the console calculator got me going really quickly - and it was fun!!!
I am hoping that one day I will be able to confidently code Windows/multiplatform GUIs that control read data from and processing data from the Arduino Nano in various ways.
Thanks again  * * * * *

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

ID: f51e5807-3f78-ff63-ef41-4629b7ae84d5
Version Independent ID: c1dbef6c-f895-a6d7-9d83-e2ac8f864327
Content: Create a console calculator in C++
Content Source: docs/get-started/tutorial-console-cpp.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Thank you, @Not-the-mama
Super glad it helped. You made my day.


[WIP] support for double and int columns

Druid currently stores numbers as longs or floats. When you index something with a doubleSum aggregator, results can have small precision errors because numbers are stored as floats in the column, and aggregated as doubles on the query side. This PR aims to address this issue. Storing numbers as doubles will double the column size, and should only be used if that level of accuracy is required.
Requires https://github.com/druid-io/druid-api/pull/72
TODO:

more tests
docs

Closing this as it conflicts with work done in https://github.com/druid-io/druid/pull/3294


Fill Description

-Reserved
{{ Fill Reserved Description }}

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

ID: ff34ec3c-ad28-7266-223d-9c60cb0486e6
Version Independent ID: 0be56c12-346c-bc92-b2ad-36b48f8d2589
Content: Start-SPOUnifiedGroupMove (SharePointOnlinePowerShell)
Content Source: sharepoint/sharepoint-ps/sharepoint-online/Start-SPOUnifiedGroupMove.md
Service: sharepoint-powershell
GitHub Login: @trent-green
Microsoft Alias: trgreen

Thanks for reporting @hwkwicre


Swift 4 migration

This repository is being archived. See https://github.com/jatoben/CommandLine/issues/111.