unify integration specs

I've noticed that there are multiple places where integration specs are placed:

/examples
/spec/integration

I think it would be a good idea to unify these specs and group them together. Currently the example specs also differ in writing style from the spec/integration specs. I would also suggest to pursue one writing style for integration specs. I also think these specs are a good place to "document" the different features of virtus, which are not all visible in the README. I like projects which explanatory integration specs to demonstrate the features.
What do you think? Is there a reason for these specs to coexist? What style of writing do you prefere?

What I like about the 'examples' style of writing is the setup. It looks like production code to show-off a specific feature. I think this is a great resource to discover the library. What I don't like about them is the spec-doc output the examples generate. It looks a lot like production code and does not specify the intent of virtus.
@solnic Please let me know when you've decided what direction to take with the integration specs. I would then rewrite them to fit.
@emmanuel @dkubb are there guidelines for these kind of specs in aequitas or veritas?

What I like about the 'examples' style of writing is the setup. It looks like production code to show-off a specific feature. I think this is a great resource to discover the library. What I don't like about them is the spec-doc output the examples generate. It looks a lot like production code and does not specify the intent of virtus.
@solnic Please let me know when you've decided what direction to take with the integration specs. I would then rewrite them to fit.
@emmanuel @dkubb are there guidelines for these kind of specs in aequitas or veritas?

@senny ah yeah, I like that...one suggestion though, I write specify when the description doesn't refer to 'it'

@solnic I updated the gist to use specify. One thing I'm not sure about is how the top level describes should be organized. If we want to have an integration spec for every feature, then a well organized specdoc would be like a 'table-of-contents' for everything. Given that the two describes in my example would not make a lot of sense on the top level... what do you think?

One other approach we could use is something like https://github.com/avdi/spartan
I've often wondered if we could write integration tests like this so that they communicate intent, and all the rspec overhead is gone, all it is are working code examples.
I don't know how well this would work for virtus, but I figured I'd throw it in in case anyone was interested in running with it.

@dkubb interesting approach... This would follow the writing style we currently have in examples/ where we have tests from an application perspective. What we would loose is the spec-doc-output and I think we would gain clarity for people just glancing at the examples.
@solnic let me know what you prefere and I'll open a pull-request with a few rewritten examples to discuss the approach taken.

@dkubb @senny I prefer integration specs written with rspec


Configure more Ubuntu build targets for PPA

Current, we use a PPA to build and publish .deb packages for Ubuntu Jammy. While this is the latest LTS version of Ubuntu, it would be nice to automatically build packages for more Ubuntu versions (namely the latest non-LTS version supported). While this can be accomplished locally, ideally this would be rolled into the automation currently configured to push sources to Launchpad for build.

Hi @dvdmuckle  I was going through web regarding PPA and stuff, I got the definition understanding, can you help me understand more about this ? this seems completely new thing to me would like to try this out

@PHAGUN-JAIN for sure! So, right now, we only publish packages for the latest version of Ubuntu LTS, which right now is 24.04, codenamed Noble Numbat (or just noble for short). However, there are other versions of Ubuntu out there, such as Ubuntu 24.10, codenamed Oracular Oriole. It would be really cool if we had some kind of script or automation that can push builds for multiple version of Ubuntu at the same time. Crucially, I don't think you can specify multiple version of Ubuntu in the deb package changelog file, which the PPA system uses to determine what version of Ubuntu to build for.


Too long password are not accepted

📑 I have found these related issues/pull requests
No related issues
🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
No response
👟 Reproduction steps
Set a long password
👀 Expected behavior
The password is setted and i can login with the new one
😓 Actual Behavior
The password is setted but i can't login with the new one and i need to reset it trought cli with a shorter one
🐻 Uptime-Kuma Version
1.23.10
💻 Operating System and Arch
Ubuntu 22.04.3 LTS x86_64
🌐 Browser
Brave 1.62.162
🖥 Deployment Environment

Runtime: Docker version 25.0.1, build 29cf629
Database: sqlite/embedded
Filesystem used to store the database on: ext4
number of monitors: 6

📝 Relevant log output
No response

How long is it?

I tried setting a password, which is
>>> "1A"+"a"*1000
'1Aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa'

=> cannot reproduce on https://demo.kuma.pet/

I don’t think anyone would go that far for a password

Max I would or recommend to go is 512

I think this issue is some form of user-errror => closing as resolved
We can reopen @ColdbloodDK  goes further into details how to reproduce this issue


[schema] Add ByteBuf schema and fix ByteBuffer schema

Motivation
ByteBuffer is a variant of bytes. so it should be SchemaType.BYTES, not SchemaType.BYTEBUFFER
Changes

Fix bytebuffer schema type
Add bytebuf schema

This should be part of 2.2 change, since we should not even store SchemaType.BYTEBUFFER.


Break up DatadogBuildListener.java into mutliple files

The main source at DatadogBuildListener.java is approaching 800 lines. Let's break this into multiple files in a way that makes sense.

Happened with #38


Minor Update

PR Checklist
Please check all that apply to this PR using "x":

[ x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number:
What is the current behavior?
What is the new behavior?
Other information

there is no need for this yet, thanks anyway :v:


UEFI DXE driver status: Not loaded (0x000000000000000A)

I get this when I open the tool
UEFI DXE driver status: Not loaded (0x000000000000000A)
I even tried manually building the 2 files
NvStrapsReBar.exe and NvStrapsReBar.ffs
still the same thing
CSM Disabled
Above 4G Decoding Enabled
C.A.M. (Clever Access Memory) Enabled
system
intel i5 9400f
asrock b 365m Phantom Gaming 4
gtx 1660 zotac dual fan

You need to update the UEFI image on your motherboard. You should follow the instructions on the project page, and make sure you go through these steps:

download the current UEFI image from the motherboard manufacturer
open the image in UEFITool (or similar) and add NvStrapsReBar.ffs to the image in the right location.
flash the motherboard with the new image.

The process is not easy, so make sure you read and properly follow the instructions.

A similar situation, but different hardware.
UEFI DXE driver status: Not loaded (0x000000000000000A)
i5-8400
asus prime b360m-a (3202 bios This firmware has off support for Resize BAR)
gtx 1650
Resize BAR - ON
CSM Disabled
Above 4G Decoding Enabled
C.A.M. (Clever Access Memory) Enabled
I tried to flash the modified firmware (with ffs) according to the instructions, but when I tried to flash via EZ flash 3 I got an error - Selected file is not a proper BIOS

Try this:
https://github.com/terminatorul/NvStrapsReBar/discussions/61

Try this: #61

I upgraded to a modified BIOS, but the problem remained.

I tried everything from different guides and faced "UEFI DXE driver status: Not loaded (0x000000000000000A)".
Unfortunately, it's not working for me.


Ask for an example code of wasm usage.

Hi
I had build the wasm version, but cannot figure out how to run the djxl or decode_oneshot, decode_progressive in javascript.
Please provide some example code to run djxl or decode_oneshot, decode_progressive in javascript.
Thanks in advance.

currently wasm support is limited, to the usage of djxl and cjxl
in short, you need to call the module as if it were a program.
I would however recommend making your own wrapper of now, or waiting a while as wasm support is still being worked on.


Simple "try out" template

I think the current templates and onboarding documentation work. For people unfamiliar with FSC-PS and GitHub, I propose a more simplified template and onboarding.
The template would just be for D365FSC extensions. It would have one build action and just the minimum of .json files to support that. Documentation for the template would explain

how to set it up
how to connect to it from a development environment
how to add a simple extension
how to run the build for that extension

Future work could add similar templates for commerce extensions. It could also add documentation that show how to add the other stuff (CI, CD, repo update, ...).
Aside from a more simplified onboarding, I also think this would remove concerns around allowing a third party access to the repo (the repo update) or having to upload an existing extension to a public url (for the import).

Good idea. Will create a super simple guide.

I gave this a shot with https://github.com/FH-Inway/fscps.fscsimpletpl
Tested it with https://github.com/FH-Inway/fscsimpletpltest01


Adds CanAccessNetwork to checkout extensions config

Resolves an internal Shopify checkout issue (will link that issue to this PR directly)
The change being shipped is paired with https://github.com/Shopify/shopify-cli/pull/2222. Please see that PR for more context and testing instructions.
Kudos to @dmhenry for pairing with me on this change :)

Looks straightforward enough.


service-worker.js requested from webroot / even though DRAWIO_BASE_URL contains path

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
If DrawIO is hosted in a subdirectory of the webroot, the "editor" tries lo load service-worker.js from / rather than from /<path> and fails with status 404.
To Reproduce
Steps to reproduce the behavior:

Configure DRAWIO_BASE_URL as https://server/drawio/
Run application on https://<IP_ADDRESS>:8080/
Setup ReverseProxy (e.g. via Apacher Webserver), that forwards https://server/drawio/ to https://<IP_ADDRESS>:8080/
Use https://server/drawio/ in an IFRAME embedding scenario

Expected behavior
DarwIO application can be loaded properly in the same way as id the IFAME would embed the diagrams.net endpoint
Screenshots
If applicable, add screenshots to help explain your problem.

draw.io version 17.1.3

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version 99

Smartphone (please complete the following information):
N/A
Additional context
Issue occurs when embedding a self hosted DrawIO application using an IFRAME

We are also running into this issue. When trying to run Draw.io as a subpath to a gitlab installation in kubernetes, some of the static assets are being sent to webroot instead of https://gitlab.domain.com/-/draw/

Thanks for the report. This will be fixed in the next release.


Document standard release management best practices

Document best practices for release management:

naming convention for version tags: [app_name]/v#.#.# for semantic versioned applications like APIs, and [app_name]/v#.# if semantic versioning is not desired (e.g. for user-facing apps where the concept of "breaking changes" isn't really meaningful
operational considerations like how to check what version of an app is currently deployed in a particular environment - might need to add a script or make target to do this (e.g. maybe read the image_tag variable from the service layer using terraform)

Duplicate of #208?

Good point thanks I'll move the notes here to #208


When using extend "html" can appear anywhere in the selector

In the documentation SASS specifies it avoids creating useless selectors (e.g. #main#footer) when using extend. But I've been coming across things like
div html {
}
Should some rules for placing of special tags such as html and body be added to the @extend feature?

Sass avoids building in HTML-specific knowledge, in part because CSS can in theory be used with other sorts of XML documents. I don't think we can be confident enough that these sorts of selectors are useless to get rid of them.

In that case, maybe this should be a feature request: for SASS to be configurable to treat html as a special selector if the user wants it (and maybe this should be the default setting).
Putting classes on the html tag is so common these days, and those classes are often to apply generic styles to be inherited by other elements, so being able to use @extend on the html selector without risking lots of useless selectors being added to the stylesheet would be of great benefit I think.


Add media-range matching support

Support media ranges in response like image/*, */*
https://tools.ietf.org/html/rfc7231#section-5.3.2

Thank you @sunspikes

@lezhnev74 @scaytrase thank you! It would be awesome if we could get a new release.

@sunspikes https://github.com/thephpleague/openapi-psr7-validator/releases/tag/0.3 here it is

thank you so much!!


Event Status Feedback Not Returning

Actions for Start, Stop, Reset working as expected for Elemental Live event 1. When adding Feedback for the same event, the background color never changes based on status.
Noting the following in Debug for Module -
debug: TypeError: Cannot read properties of undefined (reading 'replace')
I've checked in Postman for event status - http://{hostname}/api/live_events/1/status and it's returning data.

Also noting the System variables are shown as undefined.

Looks like it should be fixed in module version 1.1.0 via https://github.com/bitfocus/companion-module-aws-elementallive/pull/13


Update mixin installation step to handle non-default mixins and specific mixin versions

It is possible we may want to use mixins other than ones from the default Porter mixin feed, or specific versions of mixins other than latest.
We currently look at the list of mixins from the quickstart Porter manifest, and install the the latest version based on name. We also currently have a hardcoded workaround in the action to handle the helm3 mixin.
Porter does not currently allow you to specify the explicit feed URL/version for the mixin in the manifest, so proposal is to have a mapping file for each quickstart.
e.g.
mixins:
- helm:
    feedUrl: https://mchorfa.github.com/porter-helm3/atom.xml
    version: ~v0.1.3

Then the action can use this mapping file to install the correct set of mixins when building the bundle. We can probably assume that if the mixin is not included in the mapping file, then we try to install the latest version from the default feed.

This is the issue in Porter for handling this from the manifest as part of build
https://github.com/deislabs/porter/issues/975


Consider Incorporating Wiki Page: "Future Areas of Development"

https://github.com/ucsb-wsg/ucsb-wsg.github.io/wiki/Future-Areas-of-Development

I'll create an issue for each of these topics

Here's the content of the wiki page (which I will now delete):
--
Ready to Begin Implementation

Best Practice Example favicons (#58)
How to Use Headers (H1 - H6) (#68)
How to Use Copyrighted Images
SEO Standards
Media query recommendations
Form design
ARIA
Embedding HTML5 video

Need Discussion

JavaScript Standards
UX (Usability and User Experience) Standards
Marketing Standards
PHP Standards
XML Standards
Cloud Hosting/Storage Recommendations
Google Analytics Standards and Resources
Source Control Standards and Resources
Razor Syntax Standards
HTML5 (and HTML5 markup skeleton)


[material-ui][docs][TableRow] Escape markup so docgen tools don't parse it as HTML

[x] I have followed (at least) the PR section of the contributing guide.

This is a minor change to the TableRow TS prop description that allows docgen parsers to use the description without rendering the markup as HTML. See screenies for example:
Before

After

@millerized, I don't see this error in the API docs for the TableRow component when I run the docs server locally. Can you show on which page the error is shown?

Hi @ZeeshanTamboli! Thanks for your help. This error occurs in my storybook documentation that uses react-ts-docgen to create the props tables. This change is just a minor improvement for folks that are consuming the MUI types in their own systems.


Pagination of Graph query is not provided

Graph API for GetAllEvents query is provided with pagination support, but this functionality is not demonstrated in the example project.

Hello @kerimovscreations
Any specific use-case you are trying to see in the example project? The pagination query is part of the generated schema and you can use it by providing the pagination token.
Thanks,
Rohan

Thanks for the response.
I would like to see, how pagination will be applied to table view after list elements' count exceeds the query limit.

@kerimovscreations
I have added this as a feature request in our backlog and have submitted to the team for prioritization. I do not have a specific timeline that I can share at this point. We will let you know, either on this thread, or through a forum announcement when the sample is updated.

@cbommas
Thanks for a response. Is your team open for contributions? I would like to pull request my solution for this feature.

@kerimovscreations
Absolutely. Please go ahead and submit a PR and we will work through it with you.

Closing this issue, since we will track it further in a PR.


net/freeradius: Add Radius-Proxy

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I'm convinced that mine is new.
[x] When the request is meant for an existing plugin, I've added its name to the title.
Is your feature request related to a problem? Please describe.
I try to set up WiFi networks for Windows- and Mobileclients on a remote site (s2s-vpn) with Freeradius as local RADIUS-Server.
The users are stored in Microsoft ActiveDirectory on main site.
I have tried to use ldap as authentication-backend in freeradius and it works if the password is a cleartext password. But practically the clients will not send cleartext passwords (PEAP/MSCHAPv2).
In this scenario you will get something like this:
mschap: ERROR: FAILED: No NT/LM-Password.  Cannot perform authentication
Afaik it is only possible to do MSCHAPv2 against AD if the radius-server(firewall) is a domainmember and does ntlm_auth.
Describe the solution you'd like
For me it would be enough to have the possibility to configure freeradius as radius-proxy for proxying radius-requests to Microsoft-NPS.
Describe alternatives you've considered
Alternative would be to not proxy and use SMB. (https://wiki.freeradius.org/guide/FreeRADIUS-Active-Directory-Integration-HOWTO).
I know about the pros and cons of using SAMBA on a firewall, but that would give other plugins the possibility to use it as well..
Additional context

Why don't you just use the IP of your AD Radius within your APs?
Do you already have experience with radius proxy?

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.


⚠ Bofrost.com has degraded performance

In 321b666, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 967 ms

Resolved: Bofrost.com performance has improved in 414510a.


Code Review : To Do list: list structure

Description
In this project, a tasks list structure was generated using javascript bundledthrough webpack.
General requirements

[x] There are no linter errors.
[x] Followed the Github flow.
[x] The documentation is professional.

HTML/CSS/JS requirements

[x] We followed the list of best practices for HTML & CSS.
[x] We followed the list of best practices for JavaScript.

Project requirements

Use webpack to bundle Javascript
Create an index.html file and write your HTML markup here. Create an empty To Do List placeholder( or  element). The index.html file must be set as a template using the HTML Webpack Plugin.
Create an index.js file and set an array of some simple to do tasks (array of objects).
Create task object that contains three keys:
1. description [string].
2. completed [bool].
3. index: [number].
Write a function to iterate over the tasks array and populate an HTML list item element for each task.
On page load render the dynamically created list of tasks in the dedicated placeholder. The list should appear in order of the index values for each task.
Create a style.css and set rules for the To Do List. CSS must be loaded by Webpack Style/CSS Loader.

Thank you for taking the time to review this pull request.

Approved ✔
Hi @amiraabouhadid,
Congratulations on completing this project! 👏🏼 🎉
You have correctly implemented all of the requirements📃 correctly and without any errors⚠ or bugs🪲.
Cheers and good luck in your next milestones! 🍀
Feel free to leave any questions or comments in the PR thread(don't forget to tag me 😄) if something is not 100% clear.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


[SPARK-13942][CORE][DOCS] Remove Shark-related docs for 2.x

What changes were proposed in this pull request?
Shark was merged into Spark SQL since July 2014. The followings seem to be the only legacy. For Spark 2.x, we had better clean up those docs.
Migration Guide
- ## Migration Guide for Shark Users
- ...
- ### Scheduling
- ...
- ### Reducer number
- ...
- ### Caching

How was this patch tested?
Pass the Jenkins test.

Test build #53342 has started for PR 11770 at commit f91c548.

Removing Shark docs part looks OK. The slightly controversial bit is making SparkEnv private. People might depend on that. @rxin

Probably OK to make it private since everything else is  - let's see if people complain. But we should discuss it at the dev list.
For now, can you just remove the SparkEnv change from this pr and let's do the doc change first?

Sure! No problem.
Thank you for discuss about this.
Thank you, @andrewor14 and @rxin .

Test build #53361 has started for PR 11770 at commit e6707f1.

BTW, if you're going to consider changing SparkEnv then I'd remove the deprecated methods from back when it used to be a thread-local.

Now I cleaned up the description of this PR and JIRA consistently.

Test build #53370 has started for PR 11770 at commit 74e51b1.

LGTM

Thank you, @rxin !

I'm merging this in master. Thanks.

Test build #53342 has finished for PR 11770 at commit f91c548.

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

Merged build finished. Test PASSed.

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

Test build #53370 has finished for PR 11770 at commit 74e51b1.

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

Merged build finished. Test PASSed.

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

Test build #53361 has finished for PR 11770 at commit e6707f1.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkEnv (

Merged build finished. Test PASSed.

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


Revise OwnTracks region handling, support iBeacons.

This PR re-works OwnTracks entry / leave event handling.
The previous approach needed to be enabled with use_events which then disabled normal location tracking.
This takes another approach allowing both Location events and Region events to be used together. It does this by storing a state to indicate a device is in a region - and using this to temporarily disable Location events until you exit the region.
There is also revised logic on exiting a region to attempt to update the location based on gps location, and only setting state to Away if this didn't already change the current location. This works well with iBeacons inside a larger Home area.
There is a small change to allow regions of size '0' not to be picked as priority by location code. This means you can have 'beacon only' zones, which still have a defined location.
The odd thing about the code is using a special state. Open to re-work this another way! I have in mind this could be extended to allow more stanes to support more iBeacon functions (eg are you close to your keys). Revised to use a dictionary (thanks @balloob ).
There is a lot of change here - so would be grateful for some feedback - and happy to re-work.

@balloob Good call on the local dictionary.  Of course it's handles it OK if you exit big first!.

This component is getting quite complex now, I would like to see some tests to verify correct behavior.

Agreee on the complexity and some tests. Will add in the next few days.

It looks good. You can merge it when you're happy with it :dolphin:

Mess up this branch - will re-open a new PR.

You can do things like revert and interactive rebase to fix messed up
branches. After that you force push to this branch and it would have been
all good
On Fri, Jan 29, 2016, 01:35 Greg Dowling<EMAIL_ADDRESS>wrote:

Closed #976 https://github.com/balloob/home-assistant/pull/976.
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/pull/976#event-531042340.


Publish new version to npm

Just wondering when the next version will be published to npm? Since I'll be using this with Travis I'd prefer not to have to pull master. Using NavigationExperimental so I need the newer version.

u can specify a commit number instead of master

@sibelius I did that but it failed since the folder build generated from npm run build was not present. I had to clone it and run npm link which then built the build-folder.

I though prepublish script https://github.com/lelandrichardson/react-native-mock/blob/master/package.json#L7 would handle the generation of build
we need to fix this to enable install from github directly

I have been thinking of some kind of react-native@next  that would auto deploy from travis. But yes, I shall push out a new release later today

@RealOrangeOne what about install directly from github?

that would work, but currently the app will require a build step unless youre using something like babel-node

Why not just add the build dir to the repo?

because commiting the built source is a nightmare for PR review, and generally a bad idea.

I would like to disaggree ^^ but thats only my opinion.
An automated build step and push to npm after the ci is green seems to be the solution then ;)

Version 0.2.6 has been released. Will be working on an 'unstable' release schedule soon! Closing this issue for now!


Exception occurred in Gauge executable

Describe the bug
Exception occurred in Gauge executable
To Reproduce
Steps (or project) to reproduce the behavior:

Initialize a gauge project via IDEA
See error after Gradle sync started

Actual behavior
Logs
#### Gauge process exited with code ```1```
Gauge daemon initialized and listening on port: 58345
* IDE version: 2021.2.3
* API version: IU-212.5457.46
* Plugin version: 212.4746.52
* Gauge version: 1.4.2

Expected behavior
No errors while project initialized

Closing this as seems old. Please raise or comment if there is still an issue with latest Gauge.


Handle 201 responses

Fixes #91

I had to cheat a bit with the screenshot, because that's what it's supposed to look like. I think we have a bug in how the next/previous links are rendered and it's probably just in the halloween theme.
But the screenshot is the intent

@pschwyter see screenshot --^


How to add element template to custom bower bpmn js project ?

we are trying to add element template into below custom bower project  using code from here https://forum.bpmn.io/t/element-templates-usage-in-property-panel/809/3 we could not achieve element template with this documentation. If we can get some detail help on this issue it would be great ?
index.js
`var BpmnViewer = require('bpmn-js/lib/Modeler');
var camundaModdleDescriptor = require('camunda-bpmn-moddle/resources/camunda');
var additionalModules = [
  require('bpmn-js-properties-panel'),

  // 1. require camunda provider instead of the bpmn one (!)
  require('bpmn-js-properties-panel/lib/provider/camunda')
];
BpmnViewer.prototype._modules = BpmnViewer.prototype._modules.concat(additionalModules);
// 2. include camunda moddle descriptor
BpmnViewer.prototype._moddleExtensions = { camunda: camundaModdleDescriptor }
module.exports = BpmnViewer;`

You cannot use element templates with the BpmnViewer (it is a diagram viewer). Use the BpmnModeler instead.
If you got more questions, please open a respective topic in the bpmn.io forum.

so what changes i have to make to incorporate BpmnModeler ?

Just saw you were actually using the Modeler in the example code you provided.
How do you bootstrap the modeler in your application?

we have modeler/ properties panel as a bower component which we attach to index.html file in an angular project.Here how we initiate modeler from angular controller.
modeler.controller.js

var BpmnViewer = window.BpmnJS;

    var bpmnModeler = new BpmnViewer({
      container: '#canvas',
      propertiesPanel:{
        parent: '#js-properties-panel'
      }
    });

var bpmnJS = new BpmnJS({
  container: $el,
  additionalModules: [
    propertiesPanelModule,
    propertiesProviderModule
  ],
  elementTemplates: elementTemplates
  ...
});

You need to provide the template descriptors as an [] during modeler instantiation.
We will add a more detailed example show-casing this via bpmn-io/bpmn-js-example#32.

So this is piece of code should be added to bower-viewer-custom.js ?

leave a comment

i follow this process
this.modeler = new Modeler({
container: '#canvas',
width: '100%',
height: '600px',
propertiesPanel: {
parent: '#properties'
},
additionalModules: [
propertiesPanelModule,
propertiesProviderModule,
ReplaceMenuProvider,
customTranslateModule
],
elementTemplates:botdetails.botdetails,
moddleExtensions: {
camunda: camunda.camunda
}
});
and i am able to set element templates but after set element templates when we click on task and based on that task need filter accordingly so element templates will display
anybody tell me how can i acheive this thing


XHR Failed

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:- VS Code Version: Version: 1.57.1 (user setup)
Commit: 507ce72a4466fbb27b715c3722558bb15afa9f48
Date: 2021-06-17T13:28:07.755Z
Electron: 12.0.7
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19043
OS Version: Windows 10 Home version 21H1

Steps to Reproduce:
I cannot dowload and open any extension from marketplace. I'm not under any proxy and my dns are <IP_ADDRESS> and <IP_ADDRESS>. VsCode gives me XHR Failed

It seems you have network issues, not sure if it is VS Code related or not.

I do not think is related to my network because i use vs code in the same computer under ubuntu and it works perfectly.

Same issue here

Can you follow the steps below and attach the netlog.json file,

launch vscode with code --log-net-log=<some-absolute-path>/netlog.json
Perform the steps that create the xhr timeout issue
Quit the application

Also can you clarify if you are running vscode as administrator ?

I have to converto from .json to .txt because github does't support .json.
Yes I'm running code as administrator
netlog.txt

Thanks for log, can you try running with code --no-sandbox or not run as administrator and see if the issue happens.

Yes, the issue still remains

I also hit this issue. My VM doesnt have internet access, and so it will never be able to do this XMLHttpRequest. Earlier version of ms-python worked and didnt need internet access, such as ms-python.python-2021.3.658691958

I was able to get past this error with the following versions. Its very difficult to figure out manually which versions are compatible without spending 2 hours Googline and guessing and trying. This workflow should be easier :)

Install
jupyter-2021.8.1195043623.vsix - https://www.vsixhub.com/vsix/67262/
Install ms-python v2021.4.765268190

This is for vscode-1.59.1

I am trying to offline install a plugin in VS Code. During installation, the error "XHR failed" occurs. Internet access is limited, so there is no need to talk about network settings.
What have you tried?
Copied the .vsix file to C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\bin, then on the command line: code --install-extension ms-python.python-2021.8.1159798656.vsix
The output was the following:
C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\bin>code --install-extension ms-python.python-2021.8.1159798656.vsix Installing extensions... (node:10564) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead. (Use Code --trace-deprecation ...to show where the warning was created) getaddrinfo ENOTFOUND marketplace.visualstudio.com Failed Installing Extensions: file:///c%3A/Users/User_tst/AppData/Local/Programs/Microsoft%20VS%20Code/bin/ms-python.python-2021.8.1159798656.vsix (node:10564) UnhandledPromiseRejectionWarning: Canceled: Canceled     at D (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cli.js:5:1157)     at C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cli.js:9:60398     at U.fire (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cli.js:8:1712)     at F.cancel (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cli.js:8:5928)     at R.cancel (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cli.js:8:6469)     at Object.cancel (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cli.js:9:60531)     at ie.cancel (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cliProcessMain.js:15:25392)     at C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cliProcessMain.js:15:16154     at Array.forEach (<anonymous>)     at x.installExtension (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cliProcessMain.js:15:16132)     at processTicksAndRejections (internal/process/task_queues.js:93:5)     at async b.installVSIX (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cliProcessMain.js:10:260)     at async C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cliProcessMain.js:9:10268     at async Promise.all (index 0)     at async b.installExtensions (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cliProcessMain.js:9:10222)     at async C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cliProcessMain.js:17:35192     at async Object.M [as main] (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cliProcessMain.js:17:38625)     at async N (C:\Users\User_tst\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\cli.js:12:13842) (node:10564) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag--unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
Can you please tell me if there are any other installation methods?

Does the issue persist with latest stable 1.73 ?

I don't have the access to that machine anymore, sorry.

Thanks for confirming.


CORE-5774 Remove cli package plugin

Removes cli package plugin as it is being ported to corda-runtime-os repo with #1666.

We are ready to merge now


missing System.Drawing.Color assembly

Hey,
I am using the latest from DotNetGraph from Nuget and compiling into a dotnet v 4.6.2 project.
Getting this error:
[Error  : Unity Log] TypeLoadException: Parent class failed to load, due to: Could not load type of field 'DotNetGraph.Attributes.DotColorAttribute:<Color>k__BackingField' (0) due to: Could not resolve type with token 0100002e (from typeref, class/assembly System.Drawing.Color, netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51) assembly:netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 type:System.Drawing.Color member:(null) signature:<none>
...
Stack trace:
DotNetGraph.Compiler.DotCompilerWorker.CompileNode (DotNetGraph.Node.DotNode node, System.Int32 indentationLevel) (at <aa62545a0724477d9ad05e69e648d004>:0)
DotNetGraph.Compiler.DotCompilerWorker.CompileGraph () (at <aa62545a0724477d9ad05e69e648d004>:0)
DotNetGraph.Compiler.DotCompilerWorker.Compile () (at <aa62545a0724477d9ad05e69e648d004>:0)
DotNetGraph.Compiler.DotCompiler.Compile (System.IO.TextWriter writer, System.Boolean indented, System.Boolean formatStrings) (at <aa62545a0724477d9ad05e69e648d004>:0)
DotNetGraph.Compiler.DotCompiler.Compile (System.Boolean indented, System.Boolean formatStrings) (at <aa62545a0724477d9ad05e69e648d004>:0)
DotNetGraph.Extensions.DotGraphExtensions.Compile (DotNetGraph.DotGraph graph, System.Boolean indented, System.Boolean formatStrings) (at <aa62545a0724477d9ad05e69e648d004>:0)

Any clue what I can do about this?

Hum, not sure it could be because of the framework version, there is no reason .NET 4.6.2 doesn't include the Color type. Could you try to create a console application with the same framework version please? (but outside unity) so we could determine if the problem is coming from Unity.

I have just tested on a Windows machine and it works fine outside Unity, so the problem is coming from Unity.
Have you tried this already? https://stackoverflow.com/a/70217835

Thanks for the answers. What I did now is to copy the System.Drawing.dll into my project and let it, together with Syste.Drawing.Common.dll get repacked to the target .dll - but this still does not change the issue I am facing...
 The text file would not require things like a color, would it?
Can I somehow work around this need for the extra libraries to just assemble the text file?

 It requires the Color type because I wanted the users to be able to set the various color properties (node/edge color/background...) using a .NET type (converted to hex format in the compiled dot file). But since it can cause problems in some environments I might create my own color type based on the System.Drawing's one.

for what I am trying to do (drawing graphs of in-game objects) colorless would also be fine. Any ideas on when you might have time to do a change that does not require System.Drawing?

I've just pushed a beta version with my own color type DotColor, it is available here: https://www.nuget.org/packages/DotNetGraph/3.0.0-beta3
This is version 3.0 so it will require some changes on your side, creating graph/node/edge is a bit different and using fluent extension methods. You can check the readme file on master for some examples, it shouldn't be difficult!

Very nice :)
Are you using a special StringWriter lib that supports async? I get this error copying the code from README:
Type 'System.IO.StringWriter' used in a 'await using' declaration must be implicitly convertible to 'System.IAsyncDisposable' or implement a suitable 'DisposeAsync' method. Did you mean to use 'using' instead of 'await using'?

btw I am not sure if you really need to use the async writer, do you?
calling like this did also work (at least for a small example
var writer = new StringWriter();
directedGraph.CompileAsync(new CompilationContext(writer, new CompilationOptions()));
var result = writer.GetStringBuilder().ToString();

You don't need the await using on the string writer, this is a new feature of .net that is not in .net framework.
But you still need to await the CompileAsync method.

@FelixReuthlinger Is it all good? Can I close the issue?

yes, sorry, been hiking the last couple of days ;) works, thx a lot :D


[Feature Request] option to disable auto-promote with disco_deploy.py test

Currently disco_deploy.py test does provision and test new untested AMI in CI and automatically promote it to Staging if all tests pass.
But in Burst we ran into an issue where some db schema changes have been made manually in CI for development but not yet available in Staging. Because tests work fine in CI, new AMIs are tested and auto-promoted. This causes inconsistency between database and code, therefore raises error in Staging.
There are two ways to work around this problem.

Add an option to only provision and test AMIs with disco_deploy.py test

Add integration tests on Staging as soon as possible, and mark any failed AMIs as failed and therefore no new instances are provisioned in Staging.

As an immediate solution, it looks that former option may be easier to implement.
Let me know your thoughts. Any suggests are welcomed.

Why not just remove the affected hostclasses from the staging and production pipelines until you are able to put integration tests in place?


Is CLI still working?

I try to run the backup CLI but no matter what parameters I supply, I just get an "all done" message right away:
macair$ bin/dynamo-backup-to-s3 -b BBB
Finished backing up DynamoDB
Has anybody tried this CLI recently? Shouldn't it at least give verbose data instead of providing a false message (obviously it hasn't finished backup. Moreover. It hasn't even started doing so)
I'm glad to do a PR but I wanted to be sure I'm not missing something silly first

@dusterio I see the same problem. It looks like the AWS environment variables are being ignored by the CLI. So, I simply added --aws-key ${AWS_ACCESS_KEY_ID} --aws-secret ${AWS_SECRET_ACCESS_KEY} --aws-region ${AWS_DEFAULT_REGION} to the command line and it worked fine.

Regardless of what I tried, it did the same response:
$ dynamo-backup-to-s3 -b test-data --aws-key ${AWS_ACCESS_KEY_ID} --aws-secret ${AWS_SECRET_ACCESS_KEY} --aws-region ${AWS_DEFAULT_REGION}
Finished backing up DynamoDB

@didopop3 I have the same issue with you. Have you already got the resolve method?

@fgcui1204 with all parameters(--aws-key --aws-secret --aws-region) entered, it will work fine.

Supply all the params and make sure that the IAM user you're using has permissions to DynamoDB and S3 - if it does not then dynamo-backup-to-s3 will also simply complete without errors but without doing anything.


Very Confused with iOS installation process

Ok, so I started my project using Expo so I detatched to ExpoKit. That part is fine, but I am taking a look at the step one installation for iOS and I am so confused. It says to right click libraries in Xcode but I dont see that in my project. Libraries in my project so I am concerned I am importing these files to the wrong place.
Step 4 says I should end up with a certain kind of file structure but my file structure looks nothing like that either. The project I have looks similar to the file structure in step 3, but where am I to find the Xcode file that I am supposed to have open for step one?
In my my_project/ios there is a my_project.xcworkspace that I am told to open in order to run my iOS simulations on the ExpoKit documentation page (see step 3 in following link): https://docs.expo.io/versions/v16.0.0/guides/expokit.html
That is where I tried to do the import of the libraries but I just dont see any Library folder there when the .xcworkspace file is open i just see a my_project folder as well as a Pods folder

This is what my file structure looks like when I follow step one and I'm pretty sure that is wrong
And then lastly, for that script in step 6, where do I copy paste that?
help please
EDIT:
I watched a youtube video and it had step by step instructions. I did not run react-native-link. After that the Libraries folder showed up. Here is the video I watched to help install:
https://www.youtube.com/watch?v=1Bm6qoYp8qQ
I still don't understand step 6 though? Where would I paste that script? And would I need to tweak that script at all?

it's nice to see there is an awesome video.

I still don't understand step 6 though? Where would I paste that script? And would I need to tweak that script at all?

alright, I have ios document updated.
now we have step1 ~ step4 and two appendix
are you referring to appendix B ? please check new document again to see if it solves your problem
https://github.com/oney/react-native-webrtc/blob/master/Documentation/iOSInstallation.md

Thank you! yes the update clears the rest up. I was referring to appendix B. I have never had to run a script for this before so I just wasn't sure.

hooray!

@zxcpoiu Thanks for doing that! What do you think about putting that script in the "tools/" directory? That way users don't need to copy and paste...

yes sure, will do it soon!

done!

Thanks!


Lengthen the cache time for the beta domain

User story
As a developer, I want to prepare stage and production to cache the cache-able endpoints within the beta domain (primarily the document API and the SSR document pages, but also perhaps the landing page and others) for a lengthy period of time (1 hour rather than 5 minutes).
Acceptance criteria

[x] Verify that all cache-able beta endpoints on stage and prod are cached for 1 hour rather than 5 minutes

Tasks

[x] Increase shared_cache_control on all beta views from 5 min to 1 hour
[x] Ensure contributors are populated when generating SSR pages for documents (since we're going to be caching the results for a lengthy period of time)

I don't know where else to express this but...
We have a boto3 call that updates S3 with the document JSON data. It would greatly benefit us if that boto3 call is immediately followed by a boto3 call to CloudFront too. ...on the JSON API endpoint.
Now, when we have that, we could easily deduce what the "web endpoint" would be too. E.g.
cloudfront = boto3.resource('cloudfront')
cloudfront.invalidate({"endpoint": "api/v1/docs/en-US/Web/CSS"})
cloudfront.invalidate({"endpoint": "docs/en-US/Web/CSS"})

I wonder if CloudFront invalidation, although it's not immediate and it's weirdly unpredictable, at least we should work on this before we work on the TTL increase.

Confirmed 3600 second (1 hour) caching for the following beta endpoints:

document endpoints via the cache-control header (e.g., https://beta.developer.mozilla.org/en-US/docs/Web)
home page via the cache-control header (https://beta.developer.mozilla.org/en-US/)
the document API endpoints (e.g., https://beta.developer.mozilla.org/api/v1/doc/en-US/Web)(via the TTL setting on the api/v1/doc/* behavior on each of the stage and prod CDN's)
https://beta.developer.mozilla.org/contribute.json via the cache-control header
https://beta.developer.mozilla.org/robots.txt via the cache-control header
https://beta.developer.mozilla.org/favicon.ico via the cache-control header
https://beta.developer.mozilla.org/sitemap.xml via the cache-control header
https://beta.developer.mozilla.org/sitemaps/* via the cache-control header
https://beta.developer.mozilla.org/humans.txt via the cache-control header

Confirmed 1 year caching for the following beta endpoints:

https://beta.developer.mozilla.org/*/files/* via the cache-control header (these are simple redirects to the attachments domain mdn.mozillademos.org)
https://beta.developer.mozilla.org/*/@api/deki/files/* via the cache-control header (these are simple redirects to the attachments domain mdn.mozillademos.org)

The following endpoints are never cached:

https://beta.developer.mozilla.org/api/v1/whoami
https://beta.developer.mozilla.org/healthz
https://beta.developer.mozilla.org/readiness
https://beta.developer.mozilla.org/_kuma_status.json
https://beta.developer.mozilla.org/media/revision.txt
https://beta.developer.mozilla.org/media/kumascript-revision.txt

See https://github.com/mozilla/kuma/pull/5428


Add FixedShingleFilter

Relates to #1204.
Shingle token filter is useful for some advanced applications (e.g., fast phrase search, or implementing word n-gram language models).
Recently I was trying to implement a shingle token filter for my own project. This is more like Lucene's FixedShingleFilter that emits only fixed-sized shingles rather than ShingleFilter; it's easier to implement, and I think fixed-sized shingles are often sufficient in many practical use-cases.
This is my prototype and works well for my application - making shingles is possible by maintaining a buffer that remembers the past tokens (keeps only tokens that are needed to generate the next shingles). However, to me, there is one problem - emitting the correct token position. The current implementation is not compatible with StopWordFilter (or filters that remove some tokens from the stream). token() and token_mut() methods return a reference rather than a new Token object, so the original stream's position information is not available after overwriting it. I tried to align that problem but have failed so far. But I could be missing something... your help or feedback is welcome.

Here is what I'd love to have for shingles to improve phrase query speed incredibly at a very low cost on indexing.
A shinglefilter that takes a trigger dictionary...
At indexing:
Emit all tokens all of the times
If a given token is encounterred, then also emit the 2-gram starting at the trigger token.
If "and" is a trigger then:
"Of mice and men" would emit at indexing

"of"{pos=0, length=1}
"mice"{pos=1, length=1}
"and"{pos=2, length=1}
"and+men"{pos=2, length=2} # where + would actually be one these exotic unicode separator codepoint.
"men"{pos=3, length=1}

And at query time:

"of"{pos=0, length=1}
"mice"{pos=1, length=1}
"and+men"{pos=2, length=2}

The point here is that the shingle trick is mostly potent for some specific token with a very high doc frequency.
For other words it is not as beneficial to emit shingles, and the cost at indexing is quite high.

@mocobeta For this to be useful, we need to have some asymetry between the indexing tokenizer and the search tokenizer isn't it? We need to land #1073 or some variant of it?

At indexing:
Emit all tokens all of the times
If a given token is encounterred, then also emit the 2-gram starting at the trigger token.

I didn't think of that, but this is an interesting idea to me.
Besides indexing performance, the obvious downside of shingles is their vocabulary size... Generating shingles only when some criteria (dictionary- or rule-based?) are met significantly reduces the index size.
Here are some stats of my tantivy index with bi-gram and tri-gram tokens made by this patch. (data: 47 million web pages)
Total indexed docs: 47187784
Total term freq in unigram field:<PHONE_NUMBER> (unique terms: 29185)
Total term freq in bigram field:<PHONE_NUMBER> (unique terms: 96691414)
Total term freq in trigram field:<PHONE_NUMBER> (unique terms: 913295070)

The whole index size is 100GB (unigram + bi-gram + tri-gram) though, indexing was done within an hour with 36 threads - impressively fast :)

I think the shingle filter alone works well without #1073, but the combination could be useful in some situations?

@mocobeta ah yes, you are right. We don't need any asymetry between search and indexing here.
It is only helpful with the extension I suggested.
LGTM.
I just added a few minor comments. Please ignore all of them but the unit tests.

Total indexed docs: 47187784
Total term freq in unigram field:<PHONE_NUMBER> (unique terms: 29185)
Total term freq in bigram field:<PHONE_NUMBER> (unique terms: 96691414)
Total term freq in trigram field:<PHONE_NUMBER> (unique terms: 913295070)
The whole index size is 100GB (unigram + bi-gram + tri-gram) though, indexing was done within an hour with 36 threads - impressively fast :)

Awesome!
So my theory (never tested it) is that the "shingle on trigger" trick gives most of the benefit of shingles, but cuts down the
number of bigrams drastically.
One mostly wants shingles for super high docfreq terms. In english with 100 static stop words, that would lead to at most 2.9M bigrams (and actually way less than that).

Codecov Report

Merging #1205 (61d3b3e) into main (8802d12) will increase coverage by 0.00%.
The diff coverage is 89.10%.

:exclamation: Current head 61d3b3e differs from pull request most recent head 3989238. Consider uploading reports for the commit 3989238 to get more accurate results

@@           Coverage Diff            @@
##             main    #1205    +/-   ##
========================================
  Coverage   94.03%   94.04%
========================================
  Files         205      206     +1
  Lines       34504    34670   +166
========================================
+ Hits        32447    32605   +158
- Misses       2057     2065     +8

Impacted Files
Coverage Δ

src/tokenizer/mod.rs
93.16% <ø> (ø)

src/tokenizer/shingle_filter.rs
89.10% <89.10%> (ø)

src/fastfield/reader.rs
94.06% <0.00%> (-0.85%)
:arrow_down:

src/store/mod.rs
99.18% <0.00%> (+0.01%)
:arrow_up:

src/indexer/segment_updater.rs
94.97% <0.00%> (+1.03%)
:arrow_up:

src/store/compressors.rs
87.23% <0.00%> (+6.74%)
:arrow_up:

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 8802d12...3989238. Read the comment docs.


android: unset pluginChannel in onDetachedFromEngine()

Closes: https://github.com/UnifiedPush/flutter-connector/issues/104

Sounds good :+1: Once s1m is back/free he can merge this, I don't have merge permissions here, unfortunately.

Gentle ping :)

Thanks !


Params are missing in Request when fallbackToNetwork is enabled

Version: 7.3.6
Issue
When using fetch-mock with the fallbackToNetwork option set to true the fallback request is missing the body of the original Request object.
Example
https://codesandbox.io/s/fetch-mockbugrequest-vzbcg

Thanks for the thorough recreation info. Will be a while til I have time to look at fixing, but I have found the source of the bug if you fancy submitting a PR.
Essentially, it revolves around the fact that fetch-mock responses can be functions that return a Response. fetch is also a function that returns a Response, so fallbackToNetwork piggybacks on the function response behaviour, simply setting the response function to be fetch. This becomes problematic when executed because fetch-mock normalises any Request into a (url, options) pair, but this step doesn't extract body from the Request because that would lead to all sorts of complication elsewhere. So when this line runs https://github.com/wheresrhys/fetch-mock/blob/master/src/lib/fetch-handler.js#L18, the assumption is that response is a hand crafted function that expects a normalised Request, but in fact it's just fetch
Will get a fix out soon.
It's nice to get a bug report that's a genuine bug and not a compatibility issue for a change 😄

fixed in 7.3.7

Thank you for the quick fix 💯


fix(docs): Provide more motivation for the cross-validation example

Fix #631

Closing in favor of https://github.com/probabl-ai/skore/pull/633


CI timeouts seem too aggressive

I see tests sometimes turn red because the timeout of 7min has been reached:
https://dev.azure.com/monacotools/Monaco/_build/results?buildId=116592&view=logs&j=3792f238-f35e-5f82-0dbc-272432d9a0fb&t=0d7e5bc9-922f-51dd-b06a-a90d0b9feeeb&l=15858

I have looked at the pipeline analytics and I see that the browser unit tests for the Mac take about 260 seconds at 80% percentile. In the past when I have looked at this, a timeout indicated an issue with a test, as in those particular cases I tried to bump up the timeout to 10 minutes for example and the task still timed out.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 4f78261, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 8704 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 59e657f.


Logging changes

As requested..

Add support for prefix-grouped loggers

First off, these need to be able to share logfiles
After that, need to look into full inheritance

Solution: revert logbook

A lot of the things Logbook provides are nice, but until it was used it was hard to see how/if it'd suit things, and well, I'm not so sure now. And I can't remember why I decided to use it.
We do still need a logging system; if we're going to plan a new one, let's actually plan it out this time.

So, the plan is to use standard library logging; it provides everything we need, unlike logbook.
What requirements do we have? Children/inheritance is a native concept in logging, so that's not an issue. Log rotating is also supported by using a TimedRotatingFileHandler handler, which the example/default config can be set up with. There are also handlers for SMTP, HTTP, syslog, and some other stuff. Is there anything else we'd want to offer? Personally, all I need is TimedRotatingFileHandler and inheritance.

Define "inheritance" - One thing I like about logbook is the stack of handlers model it does. Would it be similar to that?

I can configure the plugins logger to log to plugins.log or something, then plugins.urls and plugins.urltools would also go in there. I could also override plugins.web to save to web.log as well as or instead of plugins.log. It's a tree system, where each node inherits from its parent.
That's a basic example, but logbook doesn't have that concept built in. They have LoggingGroup, but that's a one-level thing (well, you may be able to nest them within each other) and it's defined in code; the logger name has no influence over it, which is silly when loggers are named in an organised tree already.

Ah, yeah, I see, that does make sense.
Can we integrate this with metrics as well? I mean, builtin logging has an exception logging call, right?

We could make a MetricsHandler that submits to metrics, yeah.


Location of network data on disk?

I ran into some of the bugs already mentioned here with my network configuration (unable to delete, ghost controllers, etc) and would like to completely reset my network configuration, ideally making it so the files on disk/info in the world save are exactly how they were before I started messing with the mod. Is the network data stored in some TileEntity or something in the world save data or is it in a separate file somewhere else?

It's stored in the world nbt data. Which you can find in world/data. the file is called "sonar.flux.networks.configurations"


Menu Nav overlay issue on iPhone 12 Pro Max

On iPhone 12 Pro max, latest version of Helium iOS app, the system white bar overlaps the navigation labels on the 4 key navigation elements:

This is fixed in 2.7.0


Simulation magic command pages do not specify parameter formatting

On the documentation pages related to the %simulate, %simulate_sparse, %simulate_noise, %debug, and %trace, the expected format of parameters is not documented. For example, it is unclear how an array parameter should be specified.
It would be helpful to add a link to allowed parameters and their expected format/how they are parsed, along with further examples for common composite types such as arrays or tuples.

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

ID: 712d1d38-3e7e-8cc0-605c-b70bee55e4c5
Version Independent ID: 64f641a0-ad83-dcd9-dcf7-0989fa7871fd
Content: %simulate (magic command) - Q# reference - Microsoft Quantum
Content Source: api/iqsharp-magic/simulate.md
Product: qsharp
GitHub Login: @anjbur
Microsoft Alias: anburton

Thanks for creating this issue @adrianleh. I am transferring the issue to the iqsharp repository, in which the magic commands are implemented.


[vault-secrets-webhook] When using configMapMutation the variables of vault-secrets-webhook prevails over pod annotations

Hi,
When deploying the webhook with HELM, enabling configMapMutation the variables:
VAULT_ROLE: vault-secrets-webhook
VAULT_ADDR: "http://vault.vault.svc.cluster.local:8200"
VAULT_PATH: kubernetes

must be defined in vault-secrets-webhook It can't not be possible to resolve the secrets of configmaps by the annotations defined in the pod which is using that configmap?
vault.security.banzaicloud.io/vault-addr: "http://vault.vault.svc.cluster.local:8200"
vault.security.banzaicloud.io/vault-role: Service1
vault.security.banzaicloud.io/vault-path: kubernetes

And if these variables are defined in the vault-secrets-webhook pod, when resolving the secrets of the rest pods, is possible to use the annotations of the pod which uses that secrets. For the moment always prevails the variables of vault-secrets-webhook.
/kind bug

Hey @ichasco-heytrade,
It's been a while since you posted this issue, sorry for the long delay.
It shouldn't be the case at the moment, since when the Webhook processes a resource, first it tries to parse the config from annotations, if a specific annotation is not present, then it uses the defaults set as env-vars.
Also give our new enhanced webhook a try: https://github.com/bank-vaults/secrets-webhook
I'm going to close this now, if you happened to encounter anymore issues, please open another ticket under the secret-webhook repo.


Html character encoding

Hi guys,
awesome tool you are building here!
we are having problem with the following:
<mjml>
  <mj-body>
    <mj-container>
      <mj-section >
        <mj-column>
          <mj-text>
            Grüezi
          </mj-text>
        </mj-column>
      </mj-section>
    </mj-container>
  </mj-body>
</mjml>

results in the ü being converted back to Grüezi
From my perspective this was unexpected and undesired.
Best,
Michael

Hi @schurpf and thanks for opening this issue. As it is a duplicate of #343, I'll close this one so we keep everything on the same issue.


Advertise KubeCon/CloudNativeCon in homepage banner

As per Natasha's request.

Lgtm
Sent by mobile; please excuse my brevity.
On Tue, Apr 10, 2018, 18:56 Julius Volz<EMAIL_ADDRESS>wrote:

As per Natasha's request.
You can view, comment on, or merge this pull request online at:
https://github.com/prometheus/docs/pull/1000
Commit Summary

Advertise KubeCon/CloudNativeCon in homepage banner

File Changes

M content/index.html
https://github.com/prometheus/docs/pull/1000/files#diff-0 (2)

Patch Links:

https://github.com/prometheus/docs/pull/1000.patch
https://github.com/prometheus/docs/pull/1000.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/1000, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuEIy9PnKEUyuZxxbeeEPajp9a6ggUYks5tnOQegaJpZM4TOo1h
.


allow disabling controllers via HiveConfig

Introduce spec.disabledControllers as a list of strings that - when provided - will be used to set the --disabled-controllers param when the Hive operator is creating/updating the Hive controllers Deployment.
Also, move the log-level param to also live under spec.containers[].args (instead of spec.containers[].command).

/test verify

/test verify

Need to run make update-bindata.

/test verify

Need to run make update-bindata.

Sorry. This is the deepcopy not the bindata.

/test unit

/test e2e

/test e2e


'az upgrade' not upgrading to the latest

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug

az upgrade command doesn't upgrade to the latest. First time upgrading it upgraded to 2.27 and then to 2.28 when running the command for the second time. I think this can be misleading to customers especially when the features they want to use are only included in the latest version.
First upgrade

Second upgrade

To Reproduce

Expected behavior

Environment summary

Additional context

@jiasli for awareness

@jiasli Thank you so much for quick feedback!

Sorry I didn't get your question. Could you elaborate?

I am facing this error while upgrade az cli


Add S3 tests to ensure dot segments are preserved

Summary
This PR adds s3 specific tests to ensures the following:

Leading dot segments in the key value are preserved in the URL path.

Example: ../key.txt

Embedded dot segments in the key value are preserved in the URL path.

Example: foo/../key.txt

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.06%. Comparing base (e0f04ab) to head (672cc58).
Report is 48 commits behind head on develop.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3244      +/-   ##
===========================================
- Coverage    93.15%   93.06%   -0.10%
===========================================
  Files           66       66
  Lines        14287    14287
===========================================
- Hits         13309    13296      -13
- Misses         978      991      +13

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


Trevdev challenge2

Added my name to Contributions.md and added a tardis to the space scene. Looks like my saving updated formatting as well.

I would request the indentation of the challenge2.css to be reverted to what it was originally and that the solar-system remain a div. Otherwise Terri can merge these changes.

We'll keep the formatting for now 😄


Refactor: Use db_select in dosomething_reportback_get_gallery_item_ids

Currently using db_query.

This function was been removed from the codebase.


QR-Code instead of Download wg0.conf

For mobile devices like android or iOs, it's easier and faster using qr-code to scan and get config instead of downloading config.
Maybe its possible to decide "okay this is a mobile phone so please show qr-code"

This feature already exists

Oh nice!! I did not see that.


fix: cef dropping

CEF events with a "label" field but without the value field caused a drop

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


Add "(approximate)"  to TS partial mode hovers / IntelliSense

Follow up on https://github.com/microsoft/TypeScript/pull/40061
/cc @DanielRosenwasser

Here's what this now looks like:

For verification:

Open the VS Code workspace
Open a TS file
While the status bar loading indicator is showing, hover over a variable

Confirm (loading...) is shown in the hover

Now wait for TS to finish loading.
Hover again

Confirm that there is no longer any (loading...) indicator

Also test hover in the serverless cases. This should never display (loading...) since we will never load the full server

Looks good! I'm a fan and agree with the serverless mode behavior.

Verified but I would have preferred to see some pretty codicon, like the loading codicon


Get the component name in jatos.js

Would it be possible to have the componentName in the jatos object?
When I import a component to a different jatos installation, the component numbers change. It would be nice to have something invariant for the analysis scripts.
If it's not easy to have this (say because of potential special characters), it's of course very simple to include it with a script.

It should be possible. :)
On 20 Mar 2015 14:13, "elisafilevich"<EMAIL_ADDRESS>wrote:

Would it be possible to have the componentName in the jatos object?
When I import a component to a different jatos installation, the component
numbers change. It would be nice to have something invariant for the
analysis scripts.
If it's not easy to have this (say because of potential special
characters), it's of course very simple to include it with a script.
—
Reply to this email directly or view it on GitHub
https://github.com/JATOS/JATOS/issues/68.

Just checked. The component name and also the study name are already available. The first is in the jatos.componentProperties and the second in the jatos.studyProperties. So by using jatos.componentProperties.title or jatos.studyProperties.title you can get them.


Upgrading Istio using Helm

In the instructions of Upgrading Istio I think that users who installed Istio 0.8.0 using Helm should simply be instructed to execute:
helm upgrade istio install/kubernetes/helm/istio --namespace istio-system

to upgrade.
Also upgrade commands seems to be not updated (no more istio.yaml in releases, proxy image incorrect, are those the right values to be used?).

/assign rbaballeromx

@sdake: GitHub didn't allow me to assign the following users: rbaballeromx.
Note that only istio members and repo collaborators can be assigned.

In response to this:

/assign rbaballeromx

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rcaballeromx - see https://github.com/istio/istio.github.io/issues/1752#issuecomment-404365076 - but feel free to work on it and resolve it.
Cheers
-steve

/assign rcaballeromx

@rcaballeromx: GitHub didn't allow me to assign the following users: rcaballeromx.
Note that only istio members and repo collaborators can be assigned.

In response to this:

/assign rcaballeromx

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ymesika validated upgrade here: https://github.com/istio/istio/pull/7062

Related to #1798

/assign rcaballeromx

@rcaballeromx Do you have a PR for fixing this?

Fixed by #1883


external changes in the web interface

Redesigned web interface similar to ChimeraOS website.

I do appreciate the effort, I am just not sure if I want to go in this direction for the web app.


changing password ?

i've created :
version: "3"
services:
wg:
image: ngoduykhanh/wireguard-ui:latest
container_name: wgui
environment:
- WGUI_USERNAME=alpha
- WGUI_PASSWORD=testthis-unusual-password
ports:
- 5000:5000
logging:
driver: json-file
options:
max-size: 50m
volumes:
- ./db:/app/db
- /etc/wireguard:/etc/wireguard
but after a docker-compose up the username and password that are accepter are admin/admin ?

The docker hub image is updated now.


[Feature] build manager image in build-images.sh

Feature Description (功能描述)
Now, we have manager container introduced by https://github.com/apache/incubator-hugegraph-computer/pull/259. Thus, we have to modify image building script.
BTW, may I ask, currently, how to trigger new image building and uploading in HugeGraphComputer? It seems like publish_images.yml in .workflow is deprecated?

cc @coderzc

Feature Description (功能描述)
Now, we have manager container introduced by #259. Thus, we have to modify image building script.
BTW, may I ask, currently, how to trigger new image building and uploading in HugeGraphComputer? It seems like publish_images.yml in .workflow is deprecated?

due to lack some "GITHUB_TOKEN" permission here, so transfer to
https://github.com/hugegraph/actions/blob/master/.github/workflows/publish_computer_image.yml
so as other similar CIs

Hi @imbajin , get it, thanks. May I ask why we comment the cron schedule in publish_computer_image.yml? If we comment these, we should let user know how to build the latest images by themselves, maybe have to maintain the latest script(with new image manager) in this repo, either, and declare in our doc.
https://github.com/hugegraph/actions/blob/4eb9db3d644408df8f47558ecf151b510a1ffd24/.github/workflows/publish_computer_image.yml#L3-L5

Hi @imbajin , get it, thanks.
However, may I ask why we comment the cron schedule in publish_computer_image.yml? If we comment these, we should let user know how to build the latest images by themselves, maybe have to maintain the latest script(with new image manager) in this repo, either, and declare in our doc<EMAIL_ADDRESS>
Good catch & thanks for the reminder (Context: computer has not been updated for a period of time before, and it is unreasonable to keep updating the image on a regular crontab. We should use a hook mechanism that updates when there is a new commit)
@Radeity @aroundabout could u follow the PR (https://github.com/hugegraph/actions/pull/2) to update computer image? (when free)


Wrong result for JAX-RS Response with Generic Collection of enum

Hi
Following Case:
// Enum Class
public enum Car {
Ferrari , Ford, Fiat

}
......................................
@Path("/service")
@RequestScoped
public class CarService   {
@GET
@Path("/cars/{id:[0-9][0-9]*}")
@Produces(MediaType.APPLICATION_JSON)
public Response getCars(@PathParam("id") long id) {
List<Car> mycars =  new ArrayList<Car>(){};
return  Response.ok(mycars).build();
}
}
.............................................
Wrong Result :
..............................................
"responses": {
"200": {
"description": "OK",
"headers": {
},
"schema": {
"type": "array",
"enum": [
"Ferrari",
"Fiat",
"Ford"
]
}
}
}
...................................
Expected Result:
.....................................
"responses": {
"200": {
"description": "OK",
"headers": {
},
"schema": {
"type": "array",
"items": {
"type", "string",
"enum": [
"Ferrari",
"Fiat",
"Ford"
]
}
}
}
}

Generic List is created correctly.
But if Generic List  holds "enum" , result is wrong.
If Rest Services Returns just "enum" Type, not a List,  result is correct.

Using javax.ws.rs.core.GenericEntity<List<enum>> for JAX-RS Service Response  also produce wrong Result, just empty reference... in swagger yaml/json

jars-analyzer version 0.16, running with -b swagger

Ok, that's interesting... Yes, I think your assumption with EnumTypeRepresentation goes into the right direction. I don't know if I can find the time to look into it within the next days, but please feel free to verify it using some tests in the mean time :) Have you seen the https://github.com/sdaschner/jaxrs-analyzer-st project? That helps in verifying local builds.

@Override
public void visit(final TypeRepresentation.EnumTypeRepresentation representation) {
    builder.add("type", "string");
    if (!representation.getEnumValues().isEmpty()) {
        final JsonArrayBuilder array = representation.getEnumValues().stream().sorted().collect(Json::createArrayBuilder, JsonArrayBuilder::add, JsonArrayBuilder::add);
        if (inCollection) {
            builder.add("items", Json.createObjectBuilder().add("enum", array).build());
        } else {
            builder.add("enum", array);
        }
}

}

basically last if else fix it from code above

Would be nice if you could release new version :-)

sorry... below correct if else :-)

if (inCollection) {
builder.add("items", Json.createObjectBuilder().add("type", "string").add("enum", array).build());
} else {
builder.add("enum", array);
}

https://github.com/sdaschner/jaxrs-analyzer/pull/165
Successfully tested with quite complex swagger contract :-)

Hallo  Sebastian
Hier noch pull request https://github.com/sdaschner/jaxrs-analyzer/pull/165 https://github.com/sdaschner/jaxrs-analyzer/pull/165 mit dem erfolgreich getesteten fix :-)
Gruss Radek

Am 08.06.2018 um 08:53 schrieb Sebastian Daschner<EMAIL_ADDRESS>Ok, that's interesting... Yes, I think your assumption with EnumTypeRepresentation goes into the right direction. I don't know if I can find the time to look into it within the next days, but please feel free to verify it using some tests in the mean time :) Have you seen the https://github.com/sdaschner/jaxrs-analyzer-st https://github.com/sdaschner/jaxrs-analyzer-st project? That helps in verifying local builds.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/sdaschner/jaxrs-analyzer/issues/164#issuecomment-395666663, or mute the thread https://github.com/notifications/unsubscribe-auth/AA-YSr6n2272ak8ah0egNcHQLRH8mlJcks5t6h-GgaJpZM4UfJbh.


red button for copy private key

we had once done this hacky thing of swapping the confirm and cancel buttons for certain risky actions, but now we have more technology for coloring things red (e.g. for reset wallet). could we switch over the copy private key dialog too?

reexamining issues I opened. this issue remains

(for the Alice test account)


Align OpenAPI 3 example with Moonwalk example

Removed 201 response from GET
Fixed typo in description

@darrelmiller Please review & merge if you agree

LOL. Thanks for the fix.  Based on my writing you would never guess that English is my first language.  :-)


Auto-import adding functionality often adds unnecessary prefixed imports

First - the auto import adding feature generally seems to be working well and is definitely useful, so thanks! I have noticed a number of specific cases where it could be improved. A large percentage of the time I end up modifying the import that it adds to do it in a different way.
I also have a more general comment that the import prefixes that are provided are not something that any user would use on their own - they are simply prefixN always. There are some very simple heuristics that could be used here to greatly improve the prefix (package name?).
Some of the specific cases I have come across are listed below, I will follow up here with more as I remember/come across them, but these impacted me just now:
I reference something that is already imported but it adds a new prefixed import
This doesn't always happen but it does sometimes, I don't know why but it is really annoying. Screenshot here:

I already have an import to the suggested library in my file. What is really weird here is for the first 2 characters I type it actually has the correct auto-suggest that won't use a new import, it is only after the 3rd character that it starts suggesting the auto-import.
I am already importing the library but with specific show <identifier> statements.
Current Behavior: A new import is added with a prefix
Desired Behavior: Add an additional identifier to the existing show.

re: import prefixes https://dart-review.googlesource.com/c/sdk/+/116746

Updating show combinators - not now.
Maybe in the future.


Testing deployment

This pr is only for testing.

This pr enables testing openshift deployment, the following actions are required:

Create at least 2 vms and place your public key in authorized hosts.
In rails c MiqAeDatastore.reset_to_defaults
rake evm:start
Using a rest-client

Post request to:
data: {           "deployment_method" : "unmanaged",           "ssh_user" : "Your user name",           "deployment_type" : "origin",           "ssh_private_key" : "Your private key",           "deployment_master_id_or_ip" : "firstIP",           "masters_id_or_ip"  : [],           "nodes_id_or_ip"    : ["SecondIP","firstIP"],           //"nfs_id_or_ip" : ["<IP_ADDRESS>"],           //"labels" : {"SecondIP" : {'region': 'primary', 'zone': 'default'}},           "auth_name" : "my_openid_connect",           "auth_kind" : "AllowAllPasswordIdentityProvider" }

@miq-bot  wip

@alongoldboim this definitely needs rebasing (something gone wrong as you have 250 commits)


bugfix: mixed timestamps returned from pi webapi

have not had time to sort out the pre-commit stuff

Thanks! I'll sort out the pre-commit, no problem :)
If you are interested, I think setting up pre-commit on your computer should be pretty much straight forward:

"pip install pre-commit" for installation in this project only
"pre-commit install"
"pre-commit run --all-files" to do a full run, or just git commit and it will run automatically.


Remove explicit -enable-large-loadable-types flag.

LoadableByAddress was re-enabled by default on tensorflow branch in https://github.com/apple/swift/pull/27923.
IRGen tests no longer require an explicit -enable-large-loadable-types flag.

@swift-ci Please test tensorflow


RFE: ability to restrict sources used during a compose

If a user has multiple sources defined when doing composes, osbuild will try to do depsolving using all of the sources regardless of the distro specified in the blueprint or the type of compose. This can lead to undesirable results if multiple sources contain multiple versions of RPMs.
The current method to avoid using undesired sources is to remove them from osbuild, complete the compose you want to do, and then re-add the removed sources at a later time.
For instances of osbuild that support multiple distro versions and multiple composes requiring additional sources, it could be useful to be able to restrict which sources are being used during the compose of an artifact. Perhaps at the blueprint level?

There are some plans floating around for dropping the internal management of these things (sources and blueprints) from osbuild-composer entirely and instead having the user pass them on the command line when starting a compose (something like composer-cli compose start --blueprint ./bp.toml --source ./myrepo.toml --source ./another.toml qcow2).
If we end up doing that, it solves this problem too.


HADOOP-18548. Added source and dest file systems for HAR tool

Description of PR
HADOOP-18548
Hadoop archive tool doesn't acquire delegation tokens for the source and destination file systems therefore failing.
How was this patch tested?
All unit tests executed and tested on a local cluster by executing a simple archival.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 53s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
46m 15s

trunk passed

+1 :green_heart:
compile
0m 27s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
compile
0m 23s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
checkstyle
0m 26s

trunk passed

+1 :green_heart:
mvnsite
0m 28s

trunk passed

+1 :green_heart:
javadoc
0m 29s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
0m 22s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
spotbugs
0m 47s

trunk passed

+1 :green_heart:
shadedclient
26m  5s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 25s

the patch passed

+1 :green_heart:
compile
0m 20s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javac
0m 20s

the patch passed

+1 :green_heart:
compile
0m 17s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
javac
0m 17s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 12s

the patch passed

+1 :green_heart:
mvnsite
0m 19s

the patch passed

+1 :green_heart:
javadoc
0m 16s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javadoc
0m 15s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
spotbugs
0m 40s

the patch passed

+1 :green_heart:
shadedclient
26m  4s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m 41s

hadoop-archives in the patch passed.

+1 :green_heart:
asflicense
0m 33s

The patch does not generate ASF License warnings.

109m 13s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5355/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5355

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 57953d0bf2da 4.15.0-200-generic #211-Ubuntu SMP Thu Nov 24 18:16:04 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / dd2c3ed412fae80f64bb8ae7748a833ecb5cb1ba

Default Java
Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5355/1/testReport/

Max. process+thread count
542 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-archives U: hadoop-tools/hadoop-archives

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5355/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@jojochuang tested it on a real cluster. The change works. PTAL
hadoop archive -archiveName foo.har -p ofs://ozone1/vol1/bucket1/key1/ ofs://ozone1/vol2/bucket2/dst/ 23/03/13 15:47:26 INFO security.TokenCache: Got dt for ofs://ozone1; Kind: OzoneToken, Service: ... 23/03/13 15:48:08 INFO mapreduce.Job: Job job_1678722051853_0001 completed successfully ...

Archival tool doesn't have unit tests right now. The change was tested in a real cluster with the necessary changes made.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 51s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
41m 56s

trunk passed

+1 :green_heart:
compile
0m 25s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 22s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
0m 24s

trunk passed

+1 :green_heart:
mvnsite
0m 27s

trunk passed

+1 :green_heart:
javadoc
0m 28s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 22s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
0m 50s

trunk passed

+1 :green_heart:
shadedclient
23m  2s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 19s

the patch passed

+1 :green_heart:
compile
0m 19s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 19s

the patch passed

+1 :green_heart:
compile
0m 16s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
0m 16s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 12s

the patch passed

+1 :green_heart:
mvnsite
0m 19s

the patch passed

+1 :green_heart:
javadoc
0m 16s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 14s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
0m 40s

the patch passed

+1 :green_heart:
shadedclient
23m  3s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m  6s

hadoop-archives in the patch passed.

+1 :green_heart:
asflicense
0m 33s

The patch does not generate ASF License warnings.

98m 22s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5355/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5355

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux e4617f2dcfe0 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 41d25e2763e44ba06b8aa766b3312c2c77d4c729

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5355/2/testReport/

Max. process+thread count
530 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-archives U: hadoop-tools/hadoop-archives

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5355/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


ImportError: cannot import name 'string_int_label_map_pb2'

Traceback (most recent call last):   File "VehicleMoniter.py", line 40, in <module>     from utils import label_map_util   File "/Users/ferdinakusumah/Development/API/machine-kearning/license-plate-recognition/vehicle_detection/utils/label_map_util.py", line 22, in <module>     from object_detection.protos import string_int_label_map_pb2 ImportError: cannot import name 'string_int_label_map_pb2'
Is any wrong in my configuration?
Im sorry my english not good enough

Did you install the object detection properly if not check it and try to reinstall it.

Hi, I'm also receiving the same error. Could you provide an info on how to install the object detection models


Tranche label font changes when you click on it

When you click on the Tranche label, the font changes when it loads in the information. (staging)
https://user-images.githubusercontent.com/62548908/144089065-09e8538d-9a84-446e-b5e7-6fd4a5520453.mov

@pgriscti - Has this changed pushed to Staging yet ? I can't see any difference in the font there yet.

This should be on staging now @cpuja84

The bug fix works on Staging / Stagnet 1


Component refactoring

Creates new classes Transform, Mesh and Material, and refactors related traits.
This should make it easier to add new game objects down the line.
1deca7c fixes #15
3bfc94a sort of fixes #7 by making it irrelevant (multiple objects can share the same meshes/materials now).

Should probably wait for ae0ef7f before pushing this.

Should be ready for review now.