Versioned indexing when alias is not set to latest version

Did something like this:
# make my-corpus-1 index and set alias
yarn django index my-corpus -p
yarn django alias my-corpus

# make my-corpus-2 index, but something goes wrong so I do not transfer the alias
yarn django index my-corpus -p

# try to make my-corpus-3 index
yarn django index my-corpus -p

The last operation will fail because the indexing function tries to make my-corpus-2 a second time.

From the documentation: you need to add the -r flag to roll over the index version number. This is not done by default, so you can still check your new version before switching the alias. So I think this is a feature, not a bug.

My original example makes three indexing commands, can you be more specific in which command you would have used -r?
I think you mean that should have done it on the second indexing command. Perhaps my original description wasn't so clear.
When I said "something goes wrong so I do not transver the alias", I don't mean that I forgot to do that, I mean that something went wrong with indexing and the resulting index was incorrect or incomplete, so I did not transfer the alias on purpose.
Indeed, this is a good example of why -r absolutely should not be the default behaviour.
The issue I describe here isn't that the index wasn't set to the latest version, but that when the index is not set to the latest version, you can't make new indices.
For the sake of clarity, here is another scenario where you could run into this:

Create a my-corpus-1 index and assign the  my-corpus alias, start using that version in production.
Later, update the data by making a my-corpus-2 index and transfer the alias, but don´t yet delete my-corpus-1.
Notice that the my-corpus-2 index contains a mistake and roll back the alias to my-corpus-1, but don't yet delete my-corpus-2.
Try to create a new my-corpus-3 index.


feat(transform): implement syntax for component transform

TODOS:

[x] Implement resolution for let ... transform
[x] Support new <ident> syntax/resolution/grammar to allow referencing the component from a transformation to be used in an instantiation.
[ ] Update Docs
[ ] Tests/Example

Update: WAVE has recently undergone a major refactor to introduce an AST (among other improvements). So we no longer need to commit the wave crate here.

Closing this for now since I don't have much bandwidth to drive this to completion.


Running with config file for first time.

Description
When you run the server, I've only tested this with docker, It only produces this issue on bungee, if you copy over a config file to the /plugins/LuckPerms and then start the server, luckperms will get this error:
`java.lang.RuntimeException: Unable to create libs directory
at me.lucko.luckperms.common.dependencies.DependencyManager.setupCacheDirectory(DependencyManager.java:204) ~[?:?]

at me.lucko.luckperms.common.dependencies.DependencyManager.<init>(DependencyManager.java:75) ~[?:?]

at me.lucko.luckperms.common.plugin.AbstractLuckPermsPlugin.load(AbstractLuckPermsPlugin.java:117) ~[?:?]

at me.lucko.luckperms.bungee.LPBungeeBootstrap.onLoad(LPBungeeBootstrap.java:144) ~[?:?]

at me.lucko.luckperms.bungee.loader.BungeeLoaderPlugin.onLoad(BungeeLoaderPlugin.java:46) ~[?:?]

at net.md_5.bungee.api.plugin.PluginManager.enablePlugin(PluginManager.java:398) ~[waterfall.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:8d64a1a:485]

at net.md_5.bungee.api.plugin.PluginManager.loadPlugins(PluginManager.java:300) ~[waterfall.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:8d64a1a:485]

at net.md_5.bungee.BungeeCord.start(BungeeCord.java:278) ~[waterfall.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:8d64a1a:485]

at net.md_5.bungee.BungeeCordLauncher.main(BungeeCordLauncher.java:67) ~[waterfall.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:8d64a1a:485]

at net.md_5.bungee.Bootstrap.main(Bootstrap.java:15) ~[waterfall.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:8d64a1a:485]`

Reproduction Steps
Create all of the files needed for a waterfall/bungee intance, add Luckperms to the /plugins and in /plugins/LuckPerms place your config file, then start the server for the first time.
Expected Behaviour
Should startup normally using the config
Server Details
waterfall.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:8d64a1a:485
LuckPerms Version
LuckPerms-Bungee-5.4.21
Logs and Configs
https://fdow.nl/ak9iA
Extra Details
No response

/server/plugins/LuckPerms/libs: Not a directory

Does the file for that path exist? If so delete it and restart the proxy

The whole point is that it needs to be able to be ran only once

In an ideal world yes, but things can go wrong which require to run things more than once.
What Java version and Operating System does the system run? Can you try creating the LuckPerms/libs directory manually?

The whole point is that it needs to be able to be ran only once

In an ideal world yes, but things can go wrong which require to run things more than once.
What Java version and Operating System does the system run? Can you try creating the LuckPerms/libs directory manually?

I made a hotfix which starts it without the config and then puts the config in and starts it but it really should work anyway. Manually creating the libs dir does nothing to help.

My guess is that when you create the plugins/LuckPerms/ directory and copy the config, your script is not setting the correct file permissions / owner.
The server directory should be "owned" by the same user as the server process. You should be able to rectify this by running chown after the directory/config files are created.

okay


Acceptance tests: upgrades in a multi-node environment

Split of https://github.com/rancher-sandbox/os2/issues/28
Upgrades in a multi-node environment. After machines are bootstrapped, the test suite should try to upgrade the nodes with the ManagedOs resource and also manually from the cli

Is updated part done in https://github.com/rancher/elemental/pull/133? WIth osImage but not ManagedOS, I will check this.
Manual part still needed.


Incorrect iso8601 timestamp string parsing

Incorrect parsing while using ``metricq.Timestamp.from_iso8601`
It adds an hour to the provided string.
    >>> metricq.__version__
    '3.1.1'
    >>> print(metricq.Timestamp.from_iso8601("2023-02-16T02:47:03.0Z"))
    [1676515623000000000] 2023-02-16 03:47:03+01:00

I believe it is working as intended. ISO8601 strings are time zone aware. Your input string is in Z, aka UTC or +00:00. The output is in +01:00, presumably your local time zone, hence the difference in the hours.

Yes, this looks correct and as per the documentation.
__str__ just  gives you "a human-readable date-time string in the local timezone". (See also datetime.astimezone.
You can use Timestamp.datetime to get an aware datetime object to do any kind of custom formatting.
Do you think there is any addition to the documentation needed to clarify this?

Thank everyone. I am satisfied with the explanation. My bad for not paying close attention to the documentation. I seems we can close this issue.


Impressão UTF-8

I need it #63
I'm using an MTP-3 thermal printer and when printing characters like 'ç' 'é', 'ã', the print is replaced by Chinese/Japanese characters, I'm not sure the language.

facing same issue any update?


remove todos

https://github.com/fibo/dflow/search?utf8=✓&q=todo

N/A dflow was reimplemented with Deno


add example combining POSITION and the array expansion operator

I initially made the mistake of doing something like this:
FOR row IN coll
   FOR entry in doc.array
      FILTER entry.field_name != @field_name
   LIMIT @limit
   RETURN DISTINCT doc

I now realize that this is not how it works (the last two liens should be indented one more and hence this will in most cases not yield @limit many entries.
The intention behind this PR is to add a concrete example pushing folks in the right direction to do something like this
FOR row IN coll
    FILTER POSITION(doc.array[*].field_name , @field_name) == false
    LIMIT @limit
    RETURN doc

instead.

Sorry for the duplication, that was me sucking at git 🙈 .


Cannot set workspace to a root path or a shared directory

When setting workspace to a root path or shared folder it says 'Cannot use root directory x. Please select another path' and continues to work normally (workspace list not updated).

Closing as it doesn't seem to bother anyone, feel free to reopen if it does.


Pass ssa instead of stmt to visit_custom!()

We need to pass the SSA index, rather than the statement itself so that visit_custom!() can inspect the type of the statement under visitation.
This is more consistent with the transform!() API as well.

This is a good change.
There are several tests which hack around this lack.
In the forward_diff_no_inf tests
They should be updated to not work around it.

Hmm this did not fix those tests.  No matter, I will follow up on that later.


App Crash on LoggingService class while initiating the Service (Exception trace attached)

--------- beginning of crash
2020-04-18 01:22:08.927 12195-12195/edu.uw.covidsafe E/AndroidRuntime: FATAL EXCEPTION: main
Process: edu.uw.covidsafe, PID: 12195
android.app.RemoteServiceException: Context.startForegroundService() did not then call Service.startForeground(): ServiceRecord{17f93e5 u0 edu.uw.covidsafe/.LoggingService}
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1918)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loop(Looper.java:216)
at android.app.ActivityThread.main(ActivityThread.java:7263)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:494)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:975)

https://stackoverflow.com/questions/44425584/context-startforegroundservice-did-not-then-call-service-startforeground
Please log and see time diff between the two method calls to see if that's an issue.

On my own device I can get this exception if I try to use startForegroundService with the PullService.
I don't get this with the LoggingService.
But with the PullService I notice that the time between startForegroundService() and startForeground() are less than 5 seconds.
And still this exception occurs.
So requires more investigation...

@justinklchan , i tried this but didn't work . investigating it further

@justinklchan : By mistake it got closed , opening it again

@justinklchan : found the root cause for the issues. It seems we were logging the state of service incorrectly. Linked PR will resolve the issue completely. That part can be cleaned and refactored a bit. But that PR will resolve the issue for now


Actual inital budget or a channel mis-matches with the output

Project Robyn
Describe issue
I have recently switched to the new version of Robyn and the graph 'Initial vs Optimized Budget Allocator' shows incorrect original budget for channels.
Channel -1
Actual Original Spend: 5.95%
Initial Budget Allocated( Graph Output): 24.6%
Channel-2
Actual Original Spend: 0.72%
Initial Budget Allocated( Graph Output): 4.0%
Channel-3
Actual Original Spend: 58.49%
Initial Budget Allocated( Graph Output): 49.6%
Channel-4
Actual Original Spend: 22.61%
Initial Budget Allocated( Graph Output): 19.2%
Channel-5
Actual Original Spend: 3.10%
Initial Budget Allocated( Graph Output): 2.6%
Provide dummy data & model configuration
Issues are often related to custom input data that is difficult to debug without. If necessary, please modify your data to mask real values and share a dataset that is able to reproduce the issue. Please also share your model configuration.
Environment & Robyn version
R version (R --version)
Please make sure you're using the latest Robyn version

we exclude 0 spend days from this calculation, so if you have a lot of 0 spend days for the channels that are lower than the budget allocator that would be why.

besides what Kyle has explained, please also check this thread for more context

Please reopen if this issue reoccurs.


Suggestion: Put episode number (or link) in a readme where the examples are located.

This way we can find the episodes corresponding to the examples found here.

I think I'll close this - I'm trying to be consistent at putting in episode #'s or links.


(Trivial) Fix badly formatted URL, typo

I was typing up some additional instructions and included these fixes in that commit.


[Feature request] Support for yr.no API

Adding weather data from Yr would be a great feature, because of the great accuracy of their forecasts, worldwide reach and the fact that their data is free. There is already a Homebridge plugin for Yr, but it uses deprecated XML format forecasts.
Information on their current APIs can be found at https://developer.yr.no
I do understand a bit of JS (used to develop a bit of actionscript nearly 20 years ago, but turned to UX instead), but not nearly enough to actually complete this myself. I do however love to learn new stuff, and would be happy to contribute to making this feature a reality.
A quick intro to YR from their website.
Yr is the joint online weather service from the Norwegian Meteorological Institute (met.no) and the Norwegian Broadcasting Corporation (NRK). These two public institutions have worked together since 1923 transmitting weather forecasts to Norway and abroad.
Yr is unique in Europe because of our very detailed weather forecasts and our free data policy. Information about the free weather data service.
Yr offers weather forecasts in English (in additon to Norwegian Nynorsk and Norwegian Bokmål) for around 1 million places in Norway and 12 million places worldwide.

That looks to be a completely different API from OpenWeatherMap, with different attributes. and different forecast periods.  It will be easier (and less work) to create a new plugin for this, than to adapt Homebridge WS to work with both APIs.

Thanks for having a look at it. Guess that means I will have to try to figure out how to write a plugin, then.


out-of-bounds vector access in constructor

The Stan code data {int N;} parameters {real y;} model {y ~ normal(0,1);} generates an hpp file with the following lines in the constructor:
    try {

      pos__ = 1;
      context__.validate_dims("data initialization","N","int",
          context__.to_vec());

      current_statement__ = 3;
      N = context__.vals_i("N")[(1 - 1)];
    } ...

If the var_context (context__) is empty, the line context__.vals_i("N")[(1 - 1)]; will segfault because operator[] does not do any checking. One needs to use std::vector::at() to raise an exception, so context__.vals_i("N").at((1 - 1)).
I believe this is what is causing a segfault in pystan 3, see https://github.com/stan-dev/pystan-next/issues/137

@ariddell PR open: https://github.com/stan-dev/stan/pull/2949
If you want this urgently, please review or find someone to review.


Remove default import of gold-password-input-strength-meter.html

Could you remove this import from gold-password-input.html.  I would like to add this import on demand in custom element when I apply strength-meter attribute.
<link rel="import" href="./gold-password-input-strength-meter.html">
bower_components/zxcvbn/dist/zxcvbn.js have 801KB :-(

done, I will do a new release in a few minutes! (implementation is a bit different)


WIP⚠CMS-288: Update table style in accordion

Jira Ticket:
CMS-288
Description:

Update table style in accordion

Replaced by https://github.com/bcgov/bcparks.ca/pull/1439


Unhandled Promise rejection: Failed to load

[Hi, i am getting unhandled and live server not responding due to mdc.select.min.css find in @material>select. following issue started out of blue just by restarting the computer. i can see in file structure of @material that mdc.select.min.css should be in select>dist folder. but still getting the error. i have spend 4 hours with no luck.
Failed to load resource: the server responded with a status of 404 (Not Found)
<EMAIL_ADDRESS>Failed to load resource: the server responded with a status of 404 (Not Found)
polyfills.bundle.js:3057 Unhandled Promise rejection: Failed to load<EMAIL_ADDRESS>; Zone:  ; Task: Promise.then ; Value: Failed to load<EMAIL_ADDRESS>undefined
api.onUnhandledError @ polyfills.bundle.js:3057
polyfills.bundle.js:3059 Error: Uncaught (in promise): Failed to load<EMAIL_ADDRESS>at resolvePromise (polyfills.bundle.js:3184)
at resolvePromise (polyfills.bundle.js:3155)
at polyfills.bundle.js:3232
at ZoneDelegate.webpackJsonp.../../../../zone.js/dist/zone.js.ZoneDelegate.invokeTask (polyfills.bundle.js:2838)
at Zone.webpackJsonp.../../../../zone.js/dist/zone.js.Zone.runTask (polyfills.bundle.js:2605)
at drainMicroTaskQueue (polyfills.bundle.js:2998)
at XMLHttpRequest.ZoneTask.invoke (polyfills.bundle.js:2904)

api.onUnhandledError @ polyfills.bundle.js:3059
](url)

You're confusing this project with https://github.com/material-components/material-components-web

@jelbourn. i am using angular material 2

@mtr986 I'm also getting this issue. Any luck?


Fix link-mode=clone on linux

Do not attempt to reflink directories on linux
Refactor clone_recursive

Summary
On linux, reflink does not work on a directory. Currently, we first attempt to reflink directory, and only if it fails with AlreadyExists we attempt to reflink recursively.
This has the effect that, on linux, uv pip install --link-mode=clone would always fall back to copy.
We resolve this by only attempting to reflink directories on macos. In the process, we refactored clone_recursive in an attempt to make it easier to reason about its logic.
Test Plan
I tested that after this change, uv pip install --link-mode=clone numpy would behave as expected in the following cases:

linux, btrfs filesystem, venv on the same filesystem as cache (correctly reflinked)
linux, btrfs filesystem, venv on a different filesystem than cache (fallback to copy)

I have not tested it on macos or windows, as I currently don't have access to any macos or windows machines, unfortunately.

I pushed a couple of extra commits to make the linter happy, and close a gap for a race condition I missed initially, and caused a CI failure on macos.
This is ready for review.

Oh smart, thank you. Will review. I’ll admit I’ve never actually tested this on btrfs.


Modifications to AdminServices configuration API

Additions to Admin Services configuration properties to support IntegrationViewService and SolutionViewService configuration properties.

Fixed by #3655


Getting error while trying to build the project

Hello, I am pretty new to Svelte world and landed here following your awesome post on how to create a MD blog using sveltekit. Thank you for all the effort, I played around with the project in development mode and the level of time and effort put into this amazing work shows well.
When I try to build the project (after cloning it from github), it errors out with the following message. Any idea what I am doing wrong ?
Error message gives no clue as to what's missing.
Steps executed:

Clone the repo

git clone https://github.com/josh-collinsworth/sveltekit-blog-starter.git

CD into sveltekit-blog-starter and install the dependencies.

cd sveltekit-blog-starter
npm install

Run the project. Everything works fine am able to open every single post and page.

npm run dev

Build the project:

npm run build

Throws the below error.

. . .
.svelte-kit/output/server/chunks/README.js                                                     11.28 KiB
  301 /blog/page -> /blog
  301 /blog/category/page/* -> /blog
  301 /blog/page/* -> /blog
node:fs:585
  handleErrorFromBinding(ctx);
  ^

Error: ENOENT: no such file or directory, open 'C:\Dev\nodejs\svelte-static-blog\sveltekit-blog-starter\.svelte-kit/output/prerendered/pages/api/posts/category/*'
    at Object.openSync (node:fs:585:3)
    at writeFileSync (node:fs:2155:35)
    at save (file:///C:/Dev/nodejs/svelte-static-blog/sveltekit-blog-starter/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:317:4)
    at visit (file:///C:/Dev/nodejs/svelte-static-blog/sveltekit-blog-starter/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:196:3) {
  errno: -4058,
  syscall: 'open',
  code: 'ENOENT',
  path: 'C:\\Dev\\nodejs\\svelte-static-blog\\sveltekit-blog-starter\\.svelte-kit/output/prerendered/pages/api/posts/category/*'
}
[vite-plugin-svelte-kit] Prerendering failed with code 1
error during build:
Error: Prerendering failed with code 1
    at ChildProcess.<anonymous> (file:///C:/Dev/nodejs/svelte-static-blog/sveltekit-blog-starter/node_modules/@sveltejs/kit/src/exports/vite/index.js:443:15)
    at ChildProcess.emit (node:events:520:28)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:291:12)

If it helps, this is the content of my package.json file:
{
  "name": "sveltekit-blog-starter",
  "version": "1.4.2",
  "scripts": {
    "dev": "vite dev",
    "build": "vite build",
    "preview": "vite preview",
    "lint": "prettier --ignore-path .gitignore --check --plugin-search-dir=. . && eslint --ignore-path .gitignore .",
    "format": "prettier --ignore-path .gitignore --write --plugin-search-dir=. ."
  },
  "devDependencies": {
    "@sveltejs/adapter-static": "^1.0.0-next.42",
    "@sveltejs/kit": "next",
    "eslint": "^7.32.0",
    "eslint-config-prettier": "^8.3.0",
    "eslint-plugin-svelte3": "^3.2.1",
    "mdsvex": "^0.9.8",
    "prettier": "^2.4.1",
    "prettier-plugin-svelte": "^2.4.0",
    "rehype-autolink-headings": "^6.1.0",
    "rehype-slug": "^5.0.0",
    "sass": "^1.43.4",
    "svelte": "^3.42.6",
    "vite": "^3.1.0"
  },
  "type": "module",
  "dependencies": {
    "just-throttle": "^4.0.1",
    "svelte-preprocess": "^4.9.8"
  }
}

Thanks for the detailed report @kshwetabh! Hmmmm odd, I went through the same steps and didn't run into that issue. Everything ran and built fine for me.
However, I did run into a separate issue when attempting to update the site. So I suspect perhaps the issue has to do with the next version of SvelteKit. I'll keep this open for the time being, since it seems like a patch will likely be forthcoming quickly. If this isn't resolved for you soon, let me know.
The only other thing I can think is that it's an issue with either the installed version of Node.

Circling back to this: I just updated some of the packages. It may be worth re-trying and seeing if you get a better result now.

Thanks for your time @josh-collinsworth. Still no luck. Giving the same error.  I am running Node v16.18.1.
I noticed that during the build process the output category directory is empty possibly causing the error/build failure.

@kshwetabh Ok, I'm not sure why it's causing an error on your end and not mine (not even if I change my Node version to match, weirdly), but I do think I've figured out what the root issue might be.
I've realized the /api/posts/category/* routes aren't prerendering properly because they're never actually used anywhere on the site; when SvelteKit's prerender crawler doesn't see a link to something, it doesn't know what to generate, and so we just end up with the * folder.
Since the category routes in the API are never used anywhere anyway, I just pushed an update that removes those (both from src/routes, and from the config's kit.prerender.entries list).
Try a fresh install now, and let me know if you get a better/different result.

Closing this as complete. Feel free to respond if issue persists.


add pt_BR.json translate

Hi! this is my first ever pull request, and I am happy to contribute with your project, because it is my main way to listen to podcasts.
I noticed that you did not have a translation from Brazil (Brazilian Portuguese), so I made one, since its my native language.

Hi willdowglas, thanks a lot for your contribution!


[FLINK-25129][docs]project configuation changes in docs

What is the purpose of the change
A unified configuration section for both DataStream users and Table API users.
Brief change log

configuration pages split up into multiple pages and put into its own section under Application Development

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
This change is already covered by existing tests, such as (please describe tests).
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

There are still some unresolved comments, once they're solved we're ready to merge


histogram: add UI tests for rendering histogram

This PR adds tests for DataTable.tsx
It uses mount instead of shallow so we can capture the <SeriesName /> and <HistogramString /> components within the datatable.
A little hacky but I had to create the histogramData object as I can't access these props through DataTableProps without TypeScript complaining.
Addresses #11423

:+1: Thanks! Btw. the enzyme-based tests will all go away soon with the new Mantine-based UI, since enzyme doesn't work with React 18. We'll have to switch to a different way of testing things, like React Testing Library.


move path to be contained within user space

Codecov Report

Merging #334 into experimental will decrease coverage by 70.39%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           experimental     #334      +/-   ##
================================================
- Coverage         84.67%   14.28%   -70.4%
================================================
  Files                23       22       -1
  Lines              1005      987      -18
================================================
- Hits                851      141     -710
- Misses              154      846     +692

Impacted Files
Coverage Δ

vent/core/rq-worker/file_watch.py
14.28% <ø> (ø)
:arrow_up:

vent/core/rq-worker/settings.py
0% <0%> (-100%)
:arrow_down:

vent/core/rq-dashboard/rq_dash_settings.py
0% <0%> (-100%)
:arrow_down:

tests/test_api_plugins.py
3.03% <0%> (-96.97%)
:arrow_down:

tests/test_api_actions.py
4.25% <0%> (-95.75%)
:arrow_down:

vent/api/plugins.py
2.72% <0%> (-89.8%)
:arrow_down:

vent/core/file-drop/test_file_drop.py
26.31% <0%> (-73.69%)
:arrow_down:

vent/api/actions.py
3.12% <0%> (-70.63%)
:arrow_down:

...ent/core/rmq-es-connector/test_rmq_es_connector.py
30.55% <0%> (-69.45%)
:arrow_down:

tests/test_api_templates.py
36.36% <0%> (-63.64%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


Adding new Raycast2d example project

Adding the raycast2D example project.

Here's a diff file to make your code fit our guidelines, along with some more code feedback. To apply it, download the file below, unzip it, and from the command line, use git apply path/to/file.diff. You can then commit the changes.
raycast-2d-code-update.zip

Signals should be past tense and describe an action or event that happened
Code order: signals, dependencies (onready var ... = $...), then public, and finally private properties
I removed superfluous parameters: don't add variables to store the properties of a directly accessible node, e.g. colliding = ray.is_colliding(). This adds extra code maintenance at no benefit. It's less explicit, and every time you forget to manage the variable you'll get bugs.
Methods should have type hints even if they don't return anything func ...() -> void:

Finally, could you move your folder to 2019 and name it like the other demos? Use git mv -r Raycast2D 2019/08-12-raycast-2d to do so.

Moved files to the proper place, and I'll go back through and re-name the signals before I make the video, but does everything seem good to start recording?

Hey there,
I had a bit of free time today so I made a rough outline of the fps sample project.  I broke it down into four parts

Setup and X/Z Movement and Jumping.  I initially wanted to cover Jumping on its own but after getting the code in it seems too small for its own video, so if you have another suggestion of where I could split this into two videos that's fine.

Camera Movement.  I thought it was important to show camera movement with both keyboard and mouse and a gamepad.  I haven't seen too much online for working with the gamepad,

Firing with a raycast.  I made a super simple decal for a bullet hole, so that's temporary, but I had some issues with the rotation upon its creation.  There must be something with the 3D sprite that I don't understand

Juicing.  I know you have a video on the channel about juicing games, and I haven't looked at it yet, but I got as far as adding a camera kick, and a very basic particle system.

Could you re-create two PRs? I didn't pay attention but you put the FPS project in the Raycast 2D PR - you should separate pull requests, one per task or project.
I merged but reverted the change. Also, you included some .ds_store files and these that shouldn't end up in the repository. Could you either add them to .gitignore or ensure you don't add them?


Fixed #27768 -- Allowed migration optimisation of CreateModel order.

1) Allowed migration optimisation to view later operations.
The optimisation of certain migration operation sequences can only be made if the reduce() method can tell which operations occur after operation.
2) Allowed migration optimisation of CreateModel order.
An AddField operation that adds a ForeignKey between two models can only be folded into its CreateModel if the ForeignKey target model was created earlier in the list of operations. As such, reordering the CreateModel operations allows for greater optimisation.

Hey @edmorley, thanks for patch!
I've thought about this issue myself in the past few days and I've come to the conclusion that it isn't required to modify the signature of reduce() (by introducing after to solve this issue), let me explain myself.
While not used much at the moment the reduce() method can also return True to let the optimizer know that this operation can be optimized across which is exactly what we want to achieve here. What I suggest we do is make CreateModel.reduce() fallback to or not in_between_references where in_between_references = any(operation.references_model(self.name_lower, app_label) for operation in in_between) if it's super() call is returns False. Most of the existing CreateModel.reduce()optimizations will need to account for the cross-operation optimizations by returning in_between + [whatever_they_were_returning] but it should work.
Let's take the following operations set for example (on_delete, app_label and such excluded for readability)
operations = [
    CreateModel('Foo'),
    CreateModel('Bar'),
    AddField('Foo', 'bar', ForeignKey('app.Bar')),
    AddField('Bar', 'foo', ForeignKey('app.Foo')),
]

So, going through the optimizer algorithm the steps would be

CreateModel('Foo').reduce(CreateModel('Bar'), []) which actually returns False but now would return True from not any(operation.references_model(self.name_lower, app_label) for operation in []) telling the optimizer to optimize across.
CreateModel('Foo').reduce(AddField('Foo', 'bar', ForeignKey('app.Bar')), [CreateModel('Bar')]) would return in_between + [CreateModel('Foo', [('bar', ForeignKey('app.Bar'),)])] as in_between_references is False which would make the set of remaining operations [CreateModel('Bar'), CreateModel('Foo', [('bar', ForeignKey('app.Bar'),)]),  AddField('Bar', 'foo', ForeignKey('app.Foo'))].
I think you get the picture from here but the following steps optimizations would be prevented as in_between_references would be True.

FWIW here's what I came up with so far by allowing reduce() to return the full list of operations it replaces (in_between included) https://github.com/django/django/compare/master...charettes:ticket-27768
Down to 4 failures for migration tests which I suspect are either false positives or related to the previous mishandling of app_label=None.

Hello @edmorley! I just submitted #7999 which incorporate your tests but use a different approach that doesn't require changing reduce()'s signature. I gave you attribution for the tests in the last commit but if you judge this isn't enough and that you'd prefer to be attributed --author just let me know.
Thanks for your work on this and sorry for the double work here! Hopefully you've learned a couple of things about the optimizer innards and you'll be able to review my PR.

Sorry for the delayed reply - has been a busy two weeks!
No problem at all - agree that approach sounds sensible - just happy to see this fixed one way or another :-)


0.7.1: Unlimited Results

The Results type now allows unlimited results by setting pageSize to 0.

Warnings

:warning:

You should add at least 1 more reviewer to the PR

:warning:

The README has not been updated. Please update the README.

Messages

:book:

Awesome! Thanks for the well-formatted PR!

  Generated by :no_entry_sign: dangerJS


fix: lex abstract as identifier in js

abstract is a keyword in typescript but a valid identifier in javascript. Corrected the lexer.

Thank you for the PR.
I think you have some misunderstandings about this.

abstract and friends are contextual keywords in TypeScript, abstract = 1 is actually valid.
You never need to check if it's TypeScript in the lexer.
I normally cross reference babel and tsc for this kind of things

Sorry I have to close this PR since this is incorrect.

Thank you for your informative reply. Sorry for the wrong PR. I was playing around exactly with abstract = 1 in typescript and found that oxc can't parse it so I assumed it's not valid. I did some digging and found the following code path is responsible:
https://github.com/Boshen/oxc/blob/2cce71a7ebf3f610a21d0b09b4e621c62a2661e6/crates/oxc_parser/src/js/statement.rs#L89-L91
Would you like me to post another patch? Maybe changing it to if self.at(Kind::Abstract) && self.peek_token().kind == Kind::Class?

That would be great! Let's try and see if it passes some of the ts errors by running cargo coverage

If you want to make improvements to the code, you can try and port the isStartOfDeclaration part of parseStatement from typescript's parser.


https//rxinfer.ml Certificate

Firefox & Chrome cannot verify the certificate for the https://rxinfer.ml website and show a warning as below:

More info from mozilla

Seems like I just have to update the certificates - works on one PC (with correct & latest certificates issues 04.11.2022) but not on the other (older certificates issued ~2018)

I have the same issue with the Home link in the header of the documentation, can you please check @abpolym on your side?

here is teh email I got from my web hosting provider (versio.nl):
Hi Bert,

Thank you for your message.

The A record for rxinfer.ml. pointing to our redirect IP <IP_ADDRESS> was missing. The record is now created. Please wait 4-8 hours for the changes to propagate.
Please note a redirect only works with http://rxinfer.ml and not with httpS:// in our DNS panel.
Feel free to get in touch if you have any further questions.
Met vriendelijke groet,
Fabien
Customer Support Engineer

Got an answer from teh support desk:
Hi Bert,
If a browser forces the https:// connection visitors can see a warning instead of the redirect. It can only be solved if your link the domain to a hosting package and provide an SSL certificate for the domain. Then you can create the redirect through the redirect option on the hosting in DirectAdmin.
Met vriendelijke groet,
Fabien
Customer Support Engineer

@bvdmitri @abpolym @albertpod not sure how to do that SSL stuff. Perhaps you know how to proceed?

@bertdv @albertpod I think redirect should not be a problem if we setup a custom domain for github pages. Lets discuss on Monday.

I have the same issue with the Home link in the header of the documentation, can you please check @abpolym on your side?

Yeah it still is an issue on my side. I thought it was only an issue of older certificates for one of my local setups - but the with newer certificates and a fresh browser profile (no cookies remembering that I "accepted the risks") it shows me the initial warning prompt.
I have to look up how to proceed but mozilla gives some good troubleshooting

Can we close this one?

Works as expected now


Add link to EveryMac serial lookup

Was quick and easy to add so I figured why not, gives a good second opinion, straightforward url, clean site that doesn’t do any silly business. Haven’t tested Serial with earlier device serials yet but I know EveryMac do a good job of that. They don’t do estimated shipping iOS version though (that actually is valuable to know for some devices like iPhone 3GS).

Looks good!


Referred image link in set-up-your-wallet.md is not existing

## Describe the bug
The referred image link is not existing, can't be visualized
Shall change the path to the one below:
"../static/img/metamaskwallet.png"
## System settings
Not relevant.
## More context

Already create pull request with problem fixed, review is pending


X initialization fails with permissions error while trying to enter-chroot

Please paste the output of the following command here: sudo edit-chroot -all

name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20161129162558~master:5f2f11f9
release: xenial
architecture: amd64
Please describe your issue:
Startup command:
/bin/bash -c 'exec sh -e /usr/local/bin/enter-chroot -t gnome exec xinit /usr/bin/mate-session'
Above command line, which has started up the desktop without problems 100 times before has suddenly stopped working correctly and falls into an infinite loop during X initialization with the error:
Error: "dconf-CRITICAL: unable to create file '/home/cvs/.cache/dconf/user': Permission denied. dconf will not work properly".
If known, describe the steps to reproduce the issue:
NB:
home is ~cvs
~/.cache/dconf/ is owner cvs.cvs, mode 600
~/.cache/dconf/user is owner root.root, mode 600, size 2 bytes
~/.cache/dconf/user contents, piped to od -c, are:
0000000 \0 \0
0000002

@cspoleta,
I haven't run into that error before but if your permissions get hosed you can usually fix things by doing the following:
sudo enter-chroot
sudo chown -R 1000:1000 "$HOME"

It might help in your situation.
Hope this helps,
-DennisL


Add support for multiline strings

I am shepherding a change in Protobufs to allow multi-line string values:
https://github.com/protocolbuffers/protobuf/issues/1297
https://github.com/protostuff/protostuff-compiler/issues/60
This ticket serves as a marker for the corresponding effort required in this project.  I have forked this project and gotten a working version on my fork.  I'll submit a PR once things start solidifying in the protobuf project.

Looking good...

The plugin is aware of two kinds of strings, ones that are for string values and ones that are for string names, e.g. import paths:

When a string value is used somewhere a string name is required, the proper error occurs:

This issue is closed as plugin is not supported anymore.
Please switch to https://github.com/jvolkman/intellij-protobuf-editor.


Print the state of the Kademlia k-buckets in the system_networkState RPC query

Once https://github.com/libp2p/rust-libp2p/issues/1003 is done, we should improve the state() method in substrate-network-libp2p/src/service_task.rs to also include the content of the Kademlia buckets.

This has never been a necessity, and we now report the number of entries in the k-buckets to Prometheus, allowing us to detect any potential issue here. Closing.


Support custom domain names used as OneFuzz endpoints in deployment

Summary of the Pull Request
This PR adds support for custom domain names used as OneFuzz endpoints through an optional argument to deploy.py for --custom_domain. Using this argument will include the custom domain passed in the Allowed token audiences setting of the Azure Function App authentication settings.
Fixes #2695

Codecov Report

Merging #2720 (607174e) into main (07a0a92) will decrease coverage by 3.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2720      +/-   ##
==========================================
- Coverage   28.74%   25.40%   -3.34%
==========================================
  Files         320      123     -197
  Lines       38144    12884   -25260
==========================================
- Hits        10963     3273    -7690
+ Misses      27181     9611   -17570

Impacted Files
Coverage Δ

src/agent/win-util/src/aedebug.rs

src/agent/onefuzz/src/machine_id.rs

src/agent/onefuzz-agent/src/worker.rs

src/agent/coverage-legacy/src/block/linux.rs

src/agent/onefuzz/src/env.rs

src/agent/stacktrace-parser/src/lib.rs

src/agent/onefuzz-agent/src/work.rs

src/agent/debugger/src/dbghelp.rs

src/agent/onefuzz/src/lib.rs

src/agent/coverage-legacy/src/code.rs

... and 187 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


feature request: Casbin.NET integration

what do you think about it? any plans? hints, tips?

Hi @kayamax. To be fair, I don't know a lot about Casbin.NET - I have just done a quick read up about it.
In summary Beef is largely agnostic, by design, regarding authentication and authorization. It looks as if Casbin is focused on authorization, so I will respond to that directly.
There is a concept of either Role or Permission in Beef that can be used to authorize; however, Beef has no specific underlying implementation - this is up to the developer to implement. Within the ExecutionContext there are virtual methods IsInRole and IsAuthorized that can be overridden to implement.
Differences:

Role is expected to be a string, e.g. 'Manager' that would be checked against a list of known roles for the authenticated user.
Permission is meant for more 'Entity.Action' style checking; where the Entity is the object being checked, and the action being what is being performed e.g. Read, Create, Update, Delete, etc. So a Permission could be Person.Create for example. The value, structure, format of this is entirely up to the developer; as is the underlying code to validate the value.

There are attributes/properties within the Entity.xml on the Entity and Operation objects to set the value; then the appropriate ExecutionContext method will be invoked.
Looking at the Casbin it is based around {subject, object, action}. This seems like a logical fit for the Beef Permission, where subject is the user, object is the Entity, and action is the Action. Plus it also looks like Casbin has the concept of Role as well.
As you can see you should be able integrate Casbin into Beef using the generic capabilities that current exist. If, this does not suit, what do you think extra is needed to support?
Cheers...

thx for detailed reply chullybun, I wonder if there is any feature to reverse generate from existing mysql database? how does best practices look like??

@kayamax, Beef as a runtime is agnostic to the data source and leverages the likes of Entity Framework which is compatible with MySQL, PostgreSQL, etc. The primary component of Beef that is mostly SQL Server specific is the Database management and related code-gen; so for MySQL you could exclude its usage and leverage the primary components of Beef. If there was specific demand, then we could engage in extending if this was beneficial. Cheers...

Sorry I think, I was not specific enough with my question/request, what is the best practices for "database first" variant? is there any tool generating draft datasource(dsl.xml) from existing db-schema? BR

Hi. There is no official tool for generating "database first" per se. I believe there should be a formal level of separation between the entity that is being exposed and the underlying data repository. This allows the data to naturally evolve between the two as necessary. Plus, data within a database is simply a row, with one or more columns - there is no real structure or nesting to the data which can be achieved, that is more natural within an API entity. Having said that I have seen folks do a one-off generation of the Entity.xml based on the database to help accelerate - however, there is nothing out-of-the-box to do that.


Update to newest jsone

The referenced jsone#1.4.0 is not compatible to OTP 21. Could you update the version to 1.4.6?

updated to jsone 1.4.7


MDT -  Service/Team not populated

http://prntscr.com/fhc79t - comes as an empty value from middleware;

thanks @dkryvosheya
If you are clear that this/other issues are related to the Ripple QEWD middleware and not a PulseTile UI issue, can you please raise them as issues there with @robtweed
https://github.com/RippleOSI/Ripple-Qewd/issues
and update this issue here accordingly with a link to that issue
Once he has accepted them, closed them there and retested in PulseTile, you can close them off here.
Many thanks in advance
Tony

Please update this issue here accordingly with a link to the QEWD related  issue before closing it

This one is not particular QEWD issue, but data source (Marand or Ethercis) - what should I do in this case, @tony-shannon ?

Best task as a QEWD issue for these ones.. we can pass onto DB team if
confirmed.
Thanks
T
On 14 June 2017 at 08:43, dkryvosheya<EMAIL_ADDRESS>wrote:

This one is not particular QEWD issue, but data source (Marand or
Ethercis) - what should I do in this case, @tony-shannon
https://github.com/tony-shannon ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PulseTile/PulseTile/issues/18#issuecomment-308348223,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5X3I_n-qDq-EG28dqvAkY88sG_RQRhks5sD483gaJpZM4Nzw7y
.

Duplicated in QEWD repository: https://github.com/RippleOSI/Ripple-Qewd/issues/37
QEWD - MDT - Service / team data is not populated;


[minor] convert notify_webook to annotation

https://github.com/COVID19Tracking/covid-publishing-api/pull/149#issuecomment-694508063

Related to this, I'm realizing that we only call notify_webhook when we publish a new batch, but not on edits (data or state metadata). This made sense when publishing a new batch was always a separate operation, but now it's not. To avoid delays with edits going live, we should call it after edits too.

I was tired of the public API build not updating after edits and making the diffs confusing, so I did this in #162.


Add issues support

Add support for converting tags into Issue markers in Allure reports based on predefined prefix.

@vania-pooh please, review

@eduard-sukharev looks good for me. If you're actively using this adapter - you're the primary tester. :) So if it works - please go ahead and merge.


The case of the haunted stream request

Hi,
I'm not sure if this is a bug in requests or a misconfiguration in the server I'm interacting with, but I have the following very puzzling result:
>>> from requests import get
>>> response = get('http://edok01.tib.uni-hannover.de/edoks/e01dh15/815761511.pdf', stream=True)
>>> next(response.iter_content(4))
'%PD'

which returns 3 characters instead of the 4 that I requested, but also:
>>> from requests import get
>>> response = get('http://edok01.tib.uni-hannover.de/edoks/e01dh15/815761511.pdf', stream=True)
>>> iter = response.iter_content(1)
>>> next(iter)
'%'
>>> next(iter)
'P'
>>> next(iter)
'D'
>>> next(iter)
'F'

which instead returns all the 4 characters. Note also that this happens with both unicode_decode=True or unicode_decode=False.
How is this possible? This is happening with requests==2.14.2. The description of the problem in https://github.com/kennethreitz/requests/issues/3980#issue-223287665 looks relevant, but its fix in #3984 looks less relevant. In fact, I still see the problem on a local checkout of master.

iter_content transparently decompresses gzipped data received from the server. This means that when we receive bytes from the network we don't know how many bytes that will actually resolve into. For that reason, we don't guarantee that you'll get literally that number of bytes: we just promise that you won't get the empty string. That's why iter_content(1) appears to "work".
This is the expected behaviour, I'm afraid.

Ah, thanks a lot. At one point I had the feeling it had something to do with compression, but I didn't investigate in that direction.


http proxy is not working using node 0.12.7

Hi,
Unfortunately, updating the nodejs version, the librato backend is not working anymore.
More precisely, we are getting the following excpetion
_http_client.js:73
    throw new TypeError('Request path contains unescaped characters.');
          ^
TypeError: Request path contains unescaped characters.
    at new ClientRequest (_http_client.js:73:11)
    at TunnelingAgent.exports.request (http.js:49:10)
    at TunnelingAgent.createSocket (/opt/statsd/node_modules/tunnel/lib/tunnel.js:116:25)
    at TunnelingAgent.createSecureSocket [as createSocket] (/opt/emarsys/statsd/node_modules/tunnel/lib/tunnel.js:188:41)
    at TunnelingAgent.addRequest (/opt/statsd/node_modules/tunnel/lib/tunnel.js:80:8)
    at new ClientRequest (_http_client.js:154:16)
    at Object.exports.request (http.js:49:10)
    at Object.exports.request (https.js:136:15)
    at post_payload (/opt/statsd/node_modules/statsd-librato-backend/lib/librato.js:64:18)
    at post_metrics (/opt/statsd/node_modules/statsd-librato-backend/lib/librato.js:149:3)

As you can see is a problem in the tunnel library, I was wondering If it would make sense to use another library.

@basetta Are you aware of any alternative tunneling libraries that work well and have Node.js 12 compatibility? I'm not bound to anyone library, but I don't do a lot of Node.js work nor use the tunneling support so I'd likely need some assistance on porting.

@mheffner yes I did a small pull-request https://github.com/librato/statsd-librato-backend/pull/54.
Please have a look. Keep in mind that I am not a node developer at all :D


Config override

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

297 of 616 (48.21%)  changed or added relevant lines in 39 files are covered.
34 unchanged lines in 7 files lost coverage.
Overall coverage increased (+0.7%) to 63.404%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/crowdsec-cli/decisions.go
2
3
66.67%

cmd/crowdsec-cli/explain.go
0
1
0.0%

cmd/crowdsec/pour.go
0
1
0.0%

pkg/acquisition/modules/journalctl/journalctl.go
1
2
50.0%

pkg/types/utils.go
0
1
0.0%

cmd/crowdsec-cli/hubtest.go
0
2
0.0%

pkg/cwhub/download.go
2
4
50.0%

pkg/exprhelpers/exprlib.go
0
2
0.0%

pkg/apiserver/middlewares/v1/api_key.go
1
4
25.0%

cmd/crowdsec-cli/hub.go
3
7
42.86%

Files with Coverage Reduction
New Missed Lines
%

pkg/acquisition/modules/docker/docker.go
2
69.73%

pkg/cwhub/download.go
3
65.87%

pkg/cwhub/loader.go
3
73.33%

pkg/cwhub/install.go
4
60.98%

cmd/crowdsec-cli/scenarios.go
5
54.14%

cmd/crowdsec-cli/utils.go
7
21.17%

pkg/types/utils.go
10
37.37%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
9789

Relevant Lines:
15439

💛  - Coveralls


#582: Unknown Serviceaccount error

Seems like there was attempts to fix it but change didn't make it to master

@dooman87 can you please sign the cla?

@njuettner signed it couple hours ago. Sorry, had problems with signing and had to raise a ticket.

/lgtm
/approve


output container hidden tabs with table not formatted correctly for h…

I have fixed problem but found very strange issue.
Some time ago the code for a table redraw worked fine
_.defer(function(){ scope.table.draw(false);});
It doesn't work for unhided cell now.

Open 'Output Containers and Layout Managers' tutorial
Hide output cell for Stacked example
Evaluate cell
Unhide output cell

I found only one way to fix it
$timeout(function () {
_.defer(function(){ scope.table.draw(false);});
tableChanged = false;
}, 0);
I ready for any new ideas

even if the output is open the whole time i don't see the body of the table:


Could not find module @bigtest/interactor

Hey, I saw this tweet suggesting that people try out the new beta, so I did that with travis-web. All seems well except in my attempt to use the new helper in a component integration test.
It may be that I did something wrong though!
Version of x-select:
4.0.0-beta.1
Ember Version / Ember CLI Version:
Ember Version: 3.5.1
Ember CLI Version: 3.5.0
Expected Behavior
I’d expect to be able to import the helper as described in the documentation.
Actual Behavior
The import fails, as you can see here:
     Error: Could not find module `@bigtest/interactor` imported from `travis/tests/helpers/x-select`

Ember Twiddle / Example repo / Failing test:
I tried to reproduce in this Twiddle but the error is different, so I don’t know if it’s a true reproduction. This travis-web PR produces the above-mentioned error.

Hey @backspace!
Thanks for reporting, I had a feeling something like this would happen 😱 We're using ember-auto-import to import the interactor package from npm. https://github.com/thefrontside/emberx-select/blob/master/package.json#L29-L30
The setup looks straight forward: https://github.com/ef4/ember-auto-import#usage-from-addons I might have missed the included step. Let me take a look and I'll report back

Oh! This actually might be something we have to do 🙁 Have you tried installing as a dev dep in your project? yarn add -D @bigtest/interactor (or npm i -D @bigtest/interactor)
I think I've seen this issue where even though we specify it as a dependency of emberx-select, ember-auto-import can't seem to resolve it.

It doesn’t seem to have helped! The same failure happened. 😢

Dang, I've reproduced it locally, so I'll dork around and see I can figure out what's going on here.

So I was able to get it working and work in the travis app! Not so great news, I had to setup ember-auto-import there. I think this is because we're using the test-support folder, which gets merged with your ember apps test folder.
I think we're gonna have to ditch the test support folder to get this to ship correctly.


Profiles: Shareable and version controlled

When learning about profiles and their relationship with workspaces, I was excited to think that this meant that we could have everything we needed in a repository to set up a fully working environment but this doesn't seem to be so simple ( unless I'm missing something).
When working as a team it would be good if everyone is working within an environment that is identical.
This would mean that someone could open a workspace and have all the settings, extensions, launch configurations etc ready.
It seems that profiles are associated with workspaces but only for the individual computer user.
Whereas the other configurable parts are stored in json that can be committed to version control,  easily shared and automatically applied to a project ( workspace config, launch configs, linting configs etc) it seems that profile data is stored locally on the machine ( e.g. /Users/xxxx/Library/Application Support/Code/User/profiles/xxxx/settings.json ).
I have looked for how to store this in our repository, alongside our workspace configs, but can not seem to find this option.
I understand that I can export the configuration, in which case another developer could then manually import it, but this doesn't seem to be a sensible way of providing an isolated development environment configuration. This could get particularly frustrating when onboarding developers where we have multiple workspaces each with it's own profile configuration.
My use case is around a mono-repo config where we have groups of functionality ( FE, BE, APIs, libraries, etc ). We have different workspaces depending on which groups of projects we need access to. Workspaces works fine for this. We are able to set up our launch configs and most other things based on a per project or per workspace basis. We are missing the ability to have the extensions installed and enabled while switching between the workspaces, which I thought would be done through profiles.
Therefore, request for feature would be:

have profile configuration saved alongside workspace json ( or embedded inside workspace json ) and be a part of the project filesystem so it can be shared and version controlled
if a workspace has an associated profile present, then automatically apply that profile when loading the workspace.

Of course if I have missed anything and this is already possible then I would appreciate a pointer in the right direction but until seems that it can only be handled by installing external extensions.
Thanks
Phil

have profile configuration saved alongside workspace json ( or embedded inside workspace json ) and be a part of the project filesystem so it can be shared and version controlled

You can already do this by exporting the profile and saving that in your source control repository.

if a workspace has an associated profile present, then automatically apply that profile when loading the workspace.

This is missing right now and I will convert this feature request to provide this functionality.

CC @isidorn

A related profile feature would be the ability to save profiles to a distributed location. This would be immeasurably beneficial for enterprise organizations, especially those that may or may not be able to leverage cloud providers for various reasons. That being said, I understand this might be in conflict with VSCode and might be something more suited to the SLA for VSCodium. Either way, subscribed for updates 🙂


ColoredConsoleTarget - Allow using Condition-logic for word highlighting rules

Before tuning:

After tuning (Faster handling of newlines and reduced number of console color changes):

Cool!
Would be nice if we could cover these lines:

Would be nice if we could cover these lines:

Yes I'm missing unit-tests. Just playing around with comparing Microsoft AddConsole and NLog ColoredConsole. Noticed that Microsoft AddConsole is not cheap.

@304NotModified Was just in a battle with code-smells, and needed some retries before it worked. Also the Azure-Build-Server sometimes like to timeout, so one needs an extra run.

Well it's stuck on azure then,as the timeout is 60min and it should be < 20min.

Sigh. Now my refactoring revealed that we never really tested other control-characters than newline. New unit-test is needed.

Sigh. Now my refactoring revealed that we never really tested other control-characters than newline. New unit-test is needed.

So convert this PR to draft? Or is this fixed in the force push?

So convert this PR to draft? Or is this fixed in the force push?

Fixed as code-coverage is back at 100%

ow indeed! NICE!

Thanks! merged!

Updated wiki: https://github.com/NLog/NLog/wiki/ColoredConsole-target


move to roles

Summary
Copying anything from the about window was crashing and the only way to make it work is to use the role instead of a custom function. I tried to use commands while focusing on the renderer but seems I can't copy anything from the tabs or zoom in there (which is the expected and desirable) so I moved some commands to use roles, and removed any click function that was already with a role property (it gets ignored)
Issue link
MM-20227

Does role mean events in this case or? I was confused because I thought roles will be referred to as user roles :D

role means predefined action/label, so 'copy' will behave as the OS dictates.

@devinbinnie added a fix to the focus issue, should be working now for you


RFC 5987 and 6266 are not fully supported

Uploading a file with a unicode filename according to RFC 5987 does not work.
The following test fails:
import cStringIO
from werkzeug import formparser

utf8_file_input = """
--ABoundary
Content-Disposition: form-data; name="a_file"; filename*=UTF-8''"%c2%a3%20and%20%e2%82%ac%20rates"
Content-Type: application/x-object

XXX content of file XXX
--ABoundary--
"""

env = {
    'wsgi.input': cStringIO.StringIO(utf8_file_input),
    'CONTENT_LENGTH': len(utf8_file_input),
    'CONTENT_TYPE': "multipart/form-data; boundary=ABoundary",
    "REQUEST_METHOD": 'POST',
}
stream, form, files = formparser.parse_form_data(env)
assert "a_file" in files
assert u"£ and € rates" == files["a_file"].filename

Is there a web browser that actually sends filenames like this? Because last time I was checking nobody did. I went over this a few years ago when I wanted to implement this.

Looks like this is indeed supported by browsers now. Might be worth implementing.

Exactly, which is why I implemented it. :smile: I also have a small unit test for this case. I'll push it tomorrow.

Okay tests pass in all environments. You could merge the pull request if you don't have any further remarks.


Update machine-learning-create-experiment.md

Hi @hning86, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Add resizable button and scroll richtext

Title
Add resizable button and scroll Richtext and textarea fields
Type of Change

[x] New feature
[ ] Bug fix
[ ] Documentation update
[ ] Example update
[ ] Refactoring
[ ] Hotfix
[ ] Security patch
[ ] UI/UX improvement

Issue
#348
Screenshots

Impact
No Impact

Thanks for this!
Looks good, except for one little thing. Even though the resizing feature show up, it doesn't seem to work when I try to expand the size of the text box.


Replace np.float() deprecated in newer numpy version with the built-i…

…n float()

@stephjuneau the np.float() fix seems to be working fine. Can you please review and approve this PR?


st2stream traceback's with TokenExpiredError: Token has expired.

This morning one of our hubot's has been disconnected from StackStorm due to a traceback and associated restart of st2stream.
2016-06-15 06:44:27,235<PHONE_NUMBER>22224 ERROR hooks [-] Token has expired.
Traceback (most recent call last):
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/pecan/core.py", line 630, in __call__
    controller, args, kwargs = self.find_controller(state)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/pecan/core.py", line 786, in find_controller
    controller, args, kw = super(Pecan, self).find_controller(_state)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/pecan/core.py", line 516, in find_controller
    self.handle_hooks(self.determine_hooks(controller), 'before', state)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/pecan/core.py", line 793, in handle_hooks
    return super(Pecan, self).handle_hooks(hooks, *args, **kw)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/pecan/core.py", line 320, in handle_hooks
    result = getattr(hook, hook_type)(*args)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2common/hooks.py", line 115, in before
    user_db = self._validate_creds_and_get_user(request=state.request)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2common/hooks.py", line 203, in _validate_creds_and_get_user
    token_in_query_params=token_in_query_params)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2common/util/auth.py", line 65, in validate_token
    raise exceptions.TokenExpiredError('Token has expired.')
TokenExpiredError: Token has expired.

This seems to be the most common cause of hubot not sending relies to commands back to Slack.

About the same time hubot logged the following in it's log:
Wed Jun 15 2016 06:44:26 GMT+0100 (BST)] ERROR Stream error: Event { type: 'error' }
[Wed Jun 15 2016 06:44:27 GMT+0100 (BST)] ERROR Stream error: Event { type: 'error', status: 401 }

Ideally hubot should also reconnect (or exit and restart) when getting this error.

This happened again this morning with our other StackStorm installation.

@jjm Enabled support for api key for hubot-stackstorm. This way you can use an API key instead of using username/password.
https://github.com/StackStorm/hubot-stackstorm/pull/115. Also, new package has been published to npm https://www.npmjs.com/package/hubot-stackstorm.
st2chatops for v1.5 should have the new package but if you want it earlier you can run npm update hubot-stackstorm in /opt/stackstorm/chatops.
root@st2test:/opt/stackstorm/chatops# npm list | grep hubot-stackstorm
├─┬<EMAIL_ADDRESS>root@st2test:/opt/stackstorm/chatops# npm update hubot-stackstorm
<EMAIL_ADDRESS>node_modules/hubot-stackstorm
└──<EMAIL_ADDRESS>root@st2test:/opt/stackstorm/chatops# npm list | grep hubot-stackstorm
├─┬<EMAIL_ADDRESS>


Fix some grammatical errors and trailing whitespace

This fixes a couple of grammatical errors and removes some trailing whitespace.

@drguildo : Thanks for your contribution! The author, @CelesteDG, has been notified to review your proposed change.

@drguildo - Thank you for your help!
@CelesteDG - verified typo fixes and merged.


strange hydration mismatch

Describe the bug
in console there appears an error like this, hot reload stops working and any interactivity on the page doesn't work either:
Error: Hydration Mismatch. Unable to find DOM nodes for hydration key: 0-0-0-0-0-0-0-0
    at getNextElement (chunk-2LCSCBTA.js:740:13)
    at get items (+Page.tsx:14:25)
    at Object.fn (+Page.tsx:22:60)
    at runComputation (chunk-4LHGDXYW.js:809:22)
    at updateComputation (chunk-4LHGDXYW.js:788:3)
    at createMemo (chunk-4LHGDXYW.js:273:5)
    at TextTable2 (+Page.tsx:22:25)
    at @solid-refresh:25:42
    at untrack (chunk-4LHGDXYW.js:513:12)
    at HMRComp.createMemo.name [as fn] (@solid-refresh:25:28)

Your Example Website or App
https://stackblitz.com/edit/dj8qvo?file=pages%2Findex%2F%2BPage.tsx
Steps to Reproduce the Bug or Issue
see attached stackblitz repro
Expected behavior
as a user, i expected everything to work fine
Screenshots or Videos
No response
Platform

Browser: Chrome
Version: solid-js 1.8.16
vike-solid: 0.4.2

Additional context
the issue can be worked around if i remove the createMemo call, so i initially thought this is the same bug as #1485.
but looking closer it might not be, since it can also be fixed by making the value field in the prop accept a function instead of a JSX.Element directly:
...

-  {item.name}: {item.value}
+  {item.name}: {item.value()}

...

as well as by removing the internal  tag in the value itself:
 value: (
   <>
-    foo bar
+    foo
   </>
 ),

which would probably mean that it's somehow related to the fact that jsx is constructed in the parent component. but then it's unclear how does removing createMemo fix this...

solid start example of same error

@teidesu , so the JSX should be returned in a function like this:
interface TextTableProps {
  items: {
    name: string;
    value: () => JSX.Element;
  }[];
}

and then the value can be set as:
value: () => (
              <>
                foo
div

              </>
            ),

and used in a template as:

  {item.name}: {item.value()}

It's that you are recreating the items every time you access the prop. We evaluate expressions lazily. In so you are creating DOM elements that haven't been inserted in the DOM and when the client tries to find them they aren't there. So the error is correct and by design.
We recommend using children helper(https://docs.solidjs.com/reference/component-apis/children) if you wish to read children outside of the JSX and use them multiple places.
  const propItems = children(() => props.items).toArray();
  const maxNameLength = createMemo(() =>
    Math.max(...propItems.map((item) => item.name.length))
  );

  const items = () =>
    propItems.map((item) => (

        {item.name}: {item.value}

    ));

This will not have any issue. Alternatively you can have the JSX in the passed in template be created lazily itself by wrapping it in a function as suggested above.


Fixes path #/HI problem

Fixes the encoding mistake leading to path ending in #/HI. Also provides decoding backstop just in case that crops up again.
RE #535

I'll add the mm/dd/yyyy fix to the date filters on top of this PR because it needs the #/HI fix. Coming soon.

@ostermanj I'm approving this PR now. If you push the date filters fix, can you open a new PR?


Deprecated things in app

Android studio keeps complaining about background and onBackground being deprecated. One should use surface/onSurface instead. However, surface/onSurface are already used.
This might lead to the app looking strange, since there is no colour distinction between the background and the containers on it.

This is also already deployed in the beta preview. (Please, i want a good looking Mensa App again 😉 )


verify that --add-yum-repo works with scratch

It certainly doesn't, the plugin is missing in simple_inner.json. The build json rendering code will probably have to be changed too.


Feature/make snapshot

Add a flag to pip-compile to handle absence of requirements file (it generates .in and .txt)
Add a snapshot command to capture the situation at a certain point inside an env and maybe let external tool to handle a system to tracking requirements changes.
Also, adding --no-use-wheel as flag.
ps: I'm thinking to add some test, some hints? :)

Rather than coding up an entire command for this, couldn't you just run pip freeze > requirements.in? This tool is about as complex as I can/want to maintain for now. I'd like to keep pip-tools as simple and focused as possible, as pip contains a lot of inherent complexity.

@nvie I see and i agree, but i tought to avoid keep excluded requirements even in that phase using pip-tools functions, anyway thanks for review.


Add support for GO SDK access to the App Insights via Service Principal

Currently, we access AppInsights with a token,  which limits us to only working with a single application insights instance.  It also requires additional configuration of secrets.  It would be better to use the existing Service Principal that is configured and add appropriate permissions.  I tried this according to the docs in AppInisghts but I am getting 401.  This may just require documenting the correct configuration for the SP.
Additional information
I am using the GoMetrics client to access the App Insights API. I set up the Azure Authentication for a service principal as defined in the docs but I receive the following 403 error message:
bad request: insights.MetricsClient#Get: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code='InsufficientAccessError' Message='The provided credentials have insufficient access to perform the requested operation' InnerError={'code':'InvalidTokenError','message':'Could not validate the request. Challenge failed: SignatureVerificationFailed'}

At first glance in the code everything seems to be correct with the authorization code  gets the token from the AADEndpoint and that seems to align with the docs on auth endpoints.

closed by #26


[backport 1.11] DCOS-20978: Fix CLI install link

back port #2716

Alfred test this please :)

run system tests


Max retries

PR for https://github.com/junhoyeo/threads-api/issues/129

Thanks @junhoyeo for such kind words. Amazing work with threads-api!


Add "spellcasting ability" to the Class Features snippet

The small bit of formatting for "spellcasting ability" seems to trip people up. We can include it into the Class Features snippet:

## Class Features
As a fletcher, you gain the following class features
#### Hit Points

**Hit Dice:**                    :: 1d10 per fletcher level
**Hit Points at 1st Level:**     :: 10 + your Constitution modifier
**Hit Points at Higher Levels:** :: 1d10 (or 6) + your Constitution modifier per fletcher level after 1st

#### Proficiencies

**Armor:**   :: Medium armor, Shields, Heavy armor
**Weapons:** :: Martial weapons, Squeegee
**Tools:**   :: Artian's tools, Thieve's tools

**Saving Throws:** :: Intelligence, Dexerity
**Skills:**        :: Choose two from Perception, Deception, Nature, Acrobatics

#### Spellcasting Ability

{{text-align:center
**Spell save DC** = 8 + your proficiency bonus + your Charisma modifier
:
**Spell attack modifier** = your proficiency bonus + your Charisma modifier
}}

Okay, I'm pretty new at this - but this seemed like a nice place to get my feet wet contributing.
I've made this change to both classfeature.gen.js on the snippets and snippetsLegacy folders.   They seem to work on my end; I added some randomization similar to the other elements.
If this looks OK, I can submit a pull request from my fork.
return dedent`
		## Class Features
		As a ${classname}, you gain the following class features
		#### Hit Points

		**Hit Dice:**                    :: 1d${hitDie} per ${classname} level
		**Hit Points at 1st Level:**     :: ${hitDie} + your Constitution modifier
		**Hit Points at Higher Levels:** :: 1d${hitDie} (or ${hitDie/2 + 1}) + your Constitution modifier per ${classname} level after 1st

		#### Proficiencies

		**Armor:**   :: ${_.sampleSize(['Light armor', 'Medium armor', 'Heavy armor', 'Shields'], _.random(0, 3)).join(', ') || 'None'}
		**Weapons:** :: ${_.sampleSize(['Squeegee', 'Rubber Chicken', 'Simple weapons', 'Martial weapons'], _.random(0, 2)).join(', ') || 'None'}
		**Tools:**   :: ${_.sampleSize(['Artian\'s tools', 'one musical instrument', 'Thieve\'s tools'], _.random(0, 2)).join(', ') || 'None'}

		**Saving Throws:** :: ${_.sampleSize(abilityList, 2).join(', ')}
		**Skills:**        :: Choose two from ${_.sampleSize(skillList, _.random(4, 6)).join(', ')}

		#### Spellcasting Ability

		{{text-align:center
		**Spell save DC** = ${_.sample([6, 8, 10])} + your proficiency bonus + your ${spellSkill} modifier
		:
		**Spell attack modifier** = your proficiency bonus + your ${spellSkill} modifier
		}}

		#### Equipment
		You start with the following equipment, in addition to the equipment granted by your background:
		- *(a)* a martial weapon and a shield or *(b)* two martial weapons
		- *(a)* five javelins or *(b)* any simple melee weapon
		- ${_.sample(['10 lint fluffs', '1 button', 'a cherished lost sock'])}

		`;

Yeah go ahead and submit a PR. We appreciate the contribution!

This has been implemented and deployed. No reports of errors. Closing.


Delete Banner.java

 Quality gate: Failed
1 Issues

 1 Code issues

Details

 Quality gate: Failed
1 Issues

 1 Code issues

Details

 Quality gate: Failed
1 Issues

 1 Code issues

Details


granite: 0.5 -> 5.0

Motivation for this change
Can't believe I forgot this :)
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build granite transporter regextester nasc

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: granite, transporter, regextester, nasc
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: granite, transporter, regextester, nasc
Partial log (click to expand)
-- Compiling GSettings schemas
glibPreFixupPhase
post-installation fixup
Wrapping program /nix/store/34zfq6l8m0s7w08cmb3yv0q2a5xy706w-regextester-0.1.7/bin/com.github.artemanufrij.regextester
shrinking RPATHs of ELF executables and libraries in /nix/store/34zfq6l8m0s7w08cmb3yv0q2a5xy706w-regextester-0.1.7
shrinking /nix/store/34zfq6l8m0s7w08cmb3yv0q2a5xy706w-regextester-0.1.7/bin/.com.github.artemanufrij.regextester-wrapped
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/34zfq6l8m0s7w08cmb3yv0q2a5xy706w-regextester-0.1.7/bin
patching script interpreter paths in /nix/store/34zfq6l8m0s7w08cmb3yv0q2a5xy706w-regextester-0.1.7
checking for references to /build in /nix/store/34zfq6l8m0s7w08cmb3yv0q2a5xy706w-regextester-0.1.7...

Success on aarch64-linux (full log)
Attempted: granite, transporter, regextester, nasc
Partial log (click to expand)
shrinking /nix/store/8i36ac900sm39nqz4rr30v62h6v0b3zq-nasc-0.4.7/lib/libqalculatenasc.so
shrinking /nix/store/8i36ac900sm39nqz4rr30v62h6v0b3zq-nasc-0.4.7/bin/.nasc-wrapped
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/8i36ac900sm39nqz4rr30v62h6v0b3zq-nasc-0.4.7/lib  /nix/store/8i36ac900sm39nqz4rr30v62h6v0b3zq-nasc-0.4.7/bin
patching script interpreter paths in /nix/store/8i36ac900sm39nqz4rr30v62h6v0b3zq-nasc-0.4.7
checking for references to /build in /nix/store/8i36ac900sm39nqz4rr30v62h6v0b3zq-nasc-0.4.7...
/nix/store/akz2q37scrvs1n5fmf9n0lx91n8b728f-granite-5.0
/nix/store/ckk22g1q0v9gy7hqq8p6iafa9s039974-Transporter-1.3.3
/nix/store/pfhm4qaakgh5fakg5gf3gbn3d9wff2is-regextester-0.1.7
/nix/store/8i36ac900sm39nqz4rr30v62h6v0b3zq-nasc-0.4.7


add namespace in kubectl commands

Example file dex.yml is expecting namespace dex but all example
commands aren't using this namespace so dex won't be able to find
secrets.

Thanks @tomkukral


Update pom.xml for JDK8 builds, gitignore modified, Jenkinsfile added

Small update to gitignore to avoid listing unnecessary .xml and .iml files generated by some development environments
Jenkins version has been bumped to 1.642 to allow for JDK8 packaging without errors during unit tests. Java level 7 is enforced to maintain backwards compatibility
Jelly index updated for JDK8 compatibility
Added Jenkinsfile for automated tests

Looks good to me.


Stuck in a loop while scanning folders

As far as I am concerned the course "1087353" breaks the downloader. It scans trough the  exercise session groups over and over again.

Please run your script in verbose mode by setting m.setParam('verbose', True) and post the output. Unfortunately, the 'Time Series Analysis' ILIAS group is private, so I see no way to reproduce the error from here.

I recorded a video of the output since it gets stuck in a loop while scanning the groups.
https://youtu.be/SK5wE9jWGdA

Thanks! Atm, I'm quite busy and won't be able to have closer look at it within the next 2-3 weeks. However, in the meantime, I'll ask the lecturer for group access.

No Problem!
Is it possible to just download a specific folder?

Is it possible to just download a specific folder?

Sure, since each folder has a unique ref_id, you can just pass any folder'siliasid to m.addCourse(), i.e. use
m = IliasDownloaderUniMA()
m.setParam('download_path', '/path/where/you/want/your/files/')
m.login('your_uni_id', 'your_password')
m.addCourse(iliasid)
m.downloadAllFiles()

to only download the files inside the folder with the corresponding iliasid.

Oh, I didn't notice that, Thanks!
By adding the course folders manually I got it working now.

By adding the course folders manually I got it working now.

Nice! By the way, I have emailed the lecturer and he will not grant me access to the ilias group. So I see no way to reproduce the error.


Default value for tag does not work

My configuration contains this:
        # Does not work
        tag: 'myRepoOne'
        refs:
          - 'refs/heads/feature_(.*)'

But that does not use the tag on creating jobs. I had to put a tag on the ref config:
        # Works
        tag: 'myRepoOne'
        refs:
          - 'refs/heads/feature_(.*)'
              'tag': 'myRepoOne'

I've had the same issue. The problem for me was that the xpath expression doesn't match anything so there is this trace as a result:
Traceback (most recent call last):
  File "/usr/local/bin/jenkins-makejobs-svn", line 11, in <module>
    sys.exit(_main())
  File "/usr/local/lib/python3.4/site-packages/jenkins_autojobs/svn.py", line 135, in _main
    main.main(argv[1:], config=config, create_job=create_job, list_branches=list_branches)
  File "/usr/local/lib/python3.4/site-packages/jenkins_autojobs/main.py", line 160, in main
    job_name = create_job(branch, tmpl, config, branch_config)
  File "/usr/local/lib/python3.4/site-packages/jenkins_autojobs/svn.py", line 127, in create_job
    tag_method=config['tag-method']
  File "/usr/local/lib/python3.4/site-packages/jenkins_autojobs/job.py", line 92, in create
    self.tag_config(tag, tag_method)
  File "/usr/local/lib/python3.4/site-packages/jenkins_autojobs/job.py", line 72, in tag_config
    desc_el = self.xml.xpath('/project/description')[0]
IndexError: list index out of range

I don't know much about jenkins config.xml format, but with version 1.625.3 I had:
<maven2-moduleset<EMAIL_ADDRESS>  <actions/>
  <description>Hello</description>
  ...

Hello @arienkock,
I believe you've encountered a new issue with jenkins-autojobs. Apparently, it's possible to have a config.xml without a //project/description element. Commit 0619831e4809ba7807b6fea575febdcefacc8b95 should fix this.
As for the original issue, I still can't seem to figure out how it can't work. Running with -d would really help shed some light on the problem. As mentioned, the test_tag_element and test_tag_description tests in tests/test_git.py check exactly for this behavior.
Groetjes,
Georgi


Refactor events extensions

Small refactor to purge unpaid tickets after 24h.
Enforce the use of admin key to create, update and delete events.
Hotfix email validation method

Can we merge and make a new release? @dni ?


Inline custom widgets

Hi!
I'm experiencing some issues with creating custom widgets that are inline with the text, like this
Lorem ipsum dolor sit amet, <Navigation item="more"/> consectetur adipiscing elit.

I'm expecting this to show a selector that writers can use to select a navigation item when using rich text, but none is shown and the preview text is also not shown.
This works fine when they are on their own, like this
Lorem ipsum dolor sit amet,

<Navigation item="more"/>

consectetur adipiscing elit.

This shows a selector that the writers can use to select a navigation item when using rich text
So is it possible to even get it to show custom widgets when they are inline, like bold or italic? Or am I trying to do something that is not possible to do with custom widgets currently?
Image showing the issue:

CMS.registerEditorComponent({
	id: "navigation",
	label: "Navigation",
	fields: [
		{
			name: "item",
			label: "Item",
			widget: "select",
			options: Object.keys(itemMap).map(key => {
				return { label: itemMap[key].text, value: key }
			}),
			required: true
		},
	],
	pattern: /<Navigation item="([a-zA-Z\-_]+)"\/>/,
	fromBlock: function(match) {
		return {
			item: match[1] || 'library',
		};
	},
	toBlock: function(obj) {
		return `<Navigation item="${obj.item || 'library'}"/>`;
	},
	toPreview: function(obj) {
		return `<p>${obj.item || 'library'}<p/>`
	},
});

This code is the code for the custom widget. It gives a warning on all items that are inline for example Sent invalid data to remark. Plugin: navigation. Value: <Navigation item="more"/>. Data: {"item":"more"}. I have tried looking for a solution but I have not found one.

Same issue here. I'm trying to integrate custom Hugo shortcodes with the editor, but it currently does not seem possible which makes Netlify less ideal if non tech-savvy writers are involved.

Anybody had any luck with getting inline custom editor widgets to work?
I'm also looking into using the rel or refrel Hugo shortcode into the Richtext editor to allow users to add links to other parts of the site using the shortcode (as opposed to copy & pasting a URL). Any links created with this widget should be displayed inline (not block) in the Richtext view (as well as the end result, obviously). The widget would allow to select a page to link to, specify the title, and other properties like target).
From my understanding, that would require a way to specify to netlify that the custom widget (let's call it the Hugo Links widget) should be rendered as an inline element in the richtext editor (using a text slate object as opposed to a block).
Then, using a customised version of the existing Object widget we could display the HugoLinks widget inline with the text, click on it to expand/edit the properties, and click again to toggle back to inline.
I just started to look into netlify a couple of days ago so this may not be right approach, so I would greatly appreciate the input of more experienced members. The ability to not easily handle links is kind of a deal breaker, especially for non-technical users.

Any update on this? It's weird that you can match on inline patterns but not create an inline component.


Unhandled NPE in VMSnapShotManagerImpl.orchestrateCreateVMSnapshot produces snapshots that cannot be deleted

ISSUE TYPE

Bug Report

COMPONENT NAME

Server

CLOUDSTACK VERSION

at least since 4.11, still present in master

CONFIGURATION

N/A
OS / ENVIRONMENT

VMware ESX 6.5
CentOS 7
SUMMARY

Under certain conditions (races or similar), CloudStack may bails out in the middle of a snapshot creation process. Due to incorrect handling of this condition, a stray snapshot entry may be left in the database, even though no snapshot exists.
When trying to delete this bogus entry, another error is thrown (because the snapshot doesn't actually exist) and the entry stays in state "Expunged" indefinitely.
I have traced the source of the problem to the return null in https://github.com/apache/cloudstack/blob/master/server/src/main/java/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java#L516-L519
This causes a NullPointerException in https://github.com/apache/cloudstack/blob/master/server/src/main/java/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java#L1197-L1199 when getId() is called.
In my opinion, orchestrateCreateVMSnapshot should never return null, but rethrow the exception instead. This would cause the problem to be handled properly later, and an error message returned to the client.
It's unclear, however, under which circumstances takeVMSnapshot would throw an exception exactly, and if cleanup may be needed. In my opinion, such cleanup should happen right in takeVMSnapshort, not be forgotten later.
STEPS TO REPRODUCE

N/A, because we don't know the exact reason why the snapshot creation process may fail.
The error printed in the log is as follows:
2019-07-24 09:58:04,419 ERROR [cloud.api.ApiAsyncJobDispatcher] (API-Job-Executor-64:ctx-8c081a18 job-2196835) Unexpected exception while executing org.apache.cloudstack.api.command.user.vmsnapshot.CreateVMSnapshotCmd
java.lang.RuntimeException: Unexpected exception
       at com.cloud.vm.snapshot.VMSnapshotManagerImpl.createVMSnapshot(VMSnapshotManagerImpl.java:482)
       at sun.reflect.GeneratedMethodAccessor817.invoke(Unknown Source)
       at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
       at java.lang.reflect.Method.invoke(Method.java:498)
       at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:338)
       at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:197)
       at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163)
       at org.apache.cloudstack.network.contrail.management.EventUtils$EventInterceptor.invoke(EventUtils.java:107)
       at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:174)
       at com.cloud.event.ActionEventInterceptor.invoke(ActionEventInterceptor.java:51)
       at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:174)
       at org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:92)
       at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:185)
       at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:212)
       at com.sun.proxy.$Proxy198.createVMSnapshot(Unknown Source)
       at org.apache.cloudstack.api.command.user.vmsnapshot.CreateVMSnapshotCmd.execute(CreateVMSnapshotCmd.java:113)
       at com.cloud.api.ApiDispatcher.dispatch(ApiDispatcher.java:150)
       at com.cloud.api.ApiAsyncJobDispatcher.runJob(ApiAsyncJobDispatcher.java:108)
       at org.apache.cloudstack.framework.jobs.impl.AsyncJobManagerImpl$5.runInContext(AsyncJobManagerImpl.java:581)
       at org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(ManagedContextRunnable.java:49)
       at org.apache.cloudstack.managed.context.impl.DefaultManagedContext$1.call(DefaultManagedContext.java:56)
       at org.apache.cloudstack.managed.context.impl.DefaultManagedContext.callWithContext(DefaultManagedContext.java:103)
       at org.apache.cloudstack.managed.context.impl.DefaultManagedContext.runWithContext(DefaultManagedContext.java:53)
       at org.apache.cloudstack.managed.context.ManagedContextRunnable.run(ManagedContextRunnable.java:46)
       at org.apache.cloudstack.framework.jobs.impl.AsyncJobManagerImpl$5.run(AsyncJobManagerImpl.java:529)
       at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
       at java.util.concurrent.FutureTask.run(FutureTask.java:266)
       at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
       at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
       at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.NullPointerException
       at com.cloud.vm.snapshot.VMSnapshotManagerImpl.orchestrateCreateVMSnapshot(VMSnapshotManagerImpl.java:1199)
       at sun.reflect.GeneratedMethodAccessor818.invoke(Unknown Source)
       at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
       ... 26 more
2019-07-24 09:58:04,420 DEBUG [jobs.impl.AsyncJobManagerImpl] (API-Job-Executor-64:ctx-8c081a18 job-2196835) Complete async job-2196835, jobStatus: FAILED, resultCode: 530, result: org.apache.cloudstack.api.response.ExceptionResponse/null/{"uuidList":[],"errorcode":530,"errortext":"Unexpected exception"}

EXPECTED RESULTS

If such an error occurs, it should be handled properly and no stray snapshot entry should stay in the database.
ACTUAL RESULTS

A snapshot entry is created that cannot be deleted.

@onitake are you writing a PR for this?

I can replace the return null with a throw, but I don't know if this has unexpected side effects.

@onitake I think it is best to remove the stray snapshot entry from the database and then throw an exception that explains the user, wrapping the original exception. If you/we cannot test we can not completely fix so this would be a good first step to get to the bottom. The exception is already there and no further handling will happen in that thread. at least this bit of clean-up will be done. details or snapshot-store links may have to be cleaned as well. You'll need to check the data model for that.


Update dependencies

Bump everything except spectre, see #45.
#46 turned out to be not a dependency issue.

Coverage remained the same at 85.128% when pulling 0a7716efe8512ecc40c261e9dde218b54aff2c8a on update-deps into 54c3bae2a122cfb24911fdebd701c43505058f14 on master.


Verify that the interpupilary distance is actually being applied properly

Resolved the issue here: https://github.com/borismus/webvr-boilerplate/commit/390458aa26d7397a53b34c699763522ecc7fdaec


Add queue_id to the MNIST OPs

All of these need Q_ID:
Linear
Softmax
NLLLoss
SGD
Argmax
ElemwiseBinary/ELEMWISE_BINARY_EQ
Typecast
Sum

API change to add Q ID

@tt-asaigal, can we move forward with https://github.com/tenstorrent/tt-metal/pull/7847 to move forward with Q ID?

@tt-asaigal, can we move forward with #7847 to move forward with Q ID?

I'll make a proper PR with run_operation infra to support CQ id propagation. We can then migrate all these ops.

Thanks for https://github.com/tenstorrent/tt-metal/pull/9039

Moreh already completed the list.


Not able to select time on mobile

There seems to be a problem with the "threshold" that is treated as a "touch and move", if i try to touchclick a time in the timepicker, it will not register as a click, but rather a touch-n-move, one has to be extremely careful to be able to select a time.
Is this something that can be fixed easily?
Willing to pay for a fix.

Hello, had the same problem, after adding:
".on('touchstart', 'div', function () { currentlyScrollingTimeDiv = false; })"
before the
".on('touchmove', 'div', function () { currentlyScrollingTimeDiv = true; })"
on line 2416 fixed the problem for me.

Did not fix the issue

Same problem.


Wrong strides in MobilenetV2

The strides used for MobilenetV2 seems to be different from that in the MobileNetV2 descriped in the paper : https://arxiv.org/pdf/1801.04381.pdf
Expected strides (as in the paper).........: [2, 1, 2, 2, 2, 1, 2, 1, 1]
Strides used in this repo...........................: [2, 1, 2, 2, 1, 2, 2, 1, 1]
Because of this, the MAdds (Multiply Adds) of the MobileNetV2 is higher compared to that of the paper.

we followed the table in the first version of the paper. (there are typos.)
they fixed the typos in the second version.

It will be good to highlight this difference in the Readme of this repository. It may help other's like me who had a hard time spotting the exact difference with that described in the paper.
Thanks for this work.


Properly serialize default value to state value string

Failure to do so results in default value of {} being represented in a field as [object object].

:+1:
(although, without tests, did it really happen? :P)

lol you are persistent aren't you =)
yeah, you're right, it's time to stop slacking on tests


don't set padding and margin if cell spacing is 0

:tada: This PR is included in version 2.0.0-beta.7 :tada:
The release is available on:

v2.0.0-beta.7
GitHub release

Your semantic-release bot :package::rocket:

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

v2.0.0
GitHub release

Your semantic-release bot :package::rocket:


Add simple favicon

Had a go at making you a basic favicon, only low res at the moment. Maybe you prefer it to the react icon for the time being :)

This is fantastic @wardy thanks!


AcessoryView - support both Right and Left.

AcessoryView - support both Right and Left.

AcessoryView - support both Right and Left.


Docs: reduce subnavbar z-index

The super-high (1080) sticky z-index is unnecessary and interferes with modal dialogs and the planned off-canvas side modal https://github.com/twbs/bootstrap/pull/29017
Example of the current behaviour, using the off-canvas preview as an example:

actually, it seems that the z-index issue has since been addressed (it's now 1020 rather than 1080 as it appears in #29017), so this is now moot.

actually, it seems that the z-index issue has since been addressed (it's now 1020 rather than 1080 as it appears in #29017), so this is now moot.

ah indeed, here's the update https://github.com/twbs/bootstrap/commit/f95004f6548f580d95837e63e15122b9d3fd0dc2

ah indeed, here's the update https://github.com/twbs/bootstrap/commit/f95004f6548f580d95837e63e15122b9d3fd0dc2


Catch HTTP 4xx and 5xx errors

This patch attepts to fix problem, when http request to kraken api gets 504 gateway timeout response. This happens from time to time, but before this patch, any HTTP 4xx/5xx error is reported as "JSON decode error" which is fairly inaccurate and hard to distinguish from actual decode errors.

I am getting this JSON decode Error a lot lately. Is the fix to replace with your code or is the "KrakenAPIClient.php" file update and I can copy and paste that?


[[FIX]] Immediately-invoked arrow functions' parameter doesn't need parentheses.

Fixes #2351

Thanks!

Committed - https://github.com/jshint/jshint/commit/d261071e7b523549499077bb13e1affb7e25158e


Add equals/hashcode to DefaultAggregatedHttpMessage.

Will make it easier to, for example, verify in mocks. Since this is aggregated, not streamed, it doesn't have the semantic issues that HttpRequest does.

@trustin PTAL

Current coverage is 63.48% (diff: 0.00%)

Merging #218 into master will decrease coverage by 0.05%

@@             master       #218   diff @@
==========================================
  Files           274        274
  Lines         10523      10537    +14
  Methods           0          0
  Messages          0          0
  Branches       1520       1526     +6
==========================================
+ Hits           6686       6689     +3
- Misses         3099       3112    +13
+ Partials        738        736     -2

Powered by Codecov. Last update 21688e0...544b8a0

Thanks!


I build lib for Mac M2 , but crashed when unity load plugin

Describe the bug
follow the steps , I build lib for Mac M2 pro , but unity crashed when unity load plugin.
To Reproduce
crashed when start unity ( 2021.3.6f1)
Logs
Editor.log
upm.log
Development Environment

Version of the AWS GameKit  1.0.1
Unity 2021.3.6f1
XCode 15.3

Additional context
gamekit pre build not working on Mac M2 pro unity

I think the plugin for unity is not test on Apple Silicon.
Even i build dylib on Mac again, It is still error on unity editor load.