Update notebooks submodule

What does this PR do?
Updates the notebooks submodule and docs config to explicitly filter in the notebook directories that we want.
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
Fixes #1
cc @borda

Do we need this in the bug-fix release? Set the appropriate milestone.


WhatsApp icon not working on TileBrowser

Hi @maebeale! We (my open source school club) are interested in this issue!
Are you able to assign it to me? We can do a co-authored commit.  Is there any additional context needed before starting?

@leahriffell Fantastic! I don't think there's any more to share, other than that the icon needs to work both on the Vue side (/contributions) AND on the Rails side (/contact_methods).

Ok thank you @maebeale! We just submitted our first PR :)


Incorrect Information in the Documentation

First of all, I want to thank you for your hard work making this awesome plugin and providing detailed documentation.
I just spotted a mistake in the information about Grid Component > Row Flow. I think you mistakenly copied the info from the Column Flow to the Row Flow.

Another thing, the last sample for Fragments (see below) doesn't work for me. It only shows a blank slide.
<grid drag="60 55" drop="5 10" bg="red" frag="1">
60 x 55
</grid>

<grid drag="25 55" drop="-5 10" style=bg="green" frag="2">
25 x 55
</grid>

<grid drag="90 20" drop="5 -10" bg="gray" frag="3">
90  x 20
</grid>

Thx for the feedback, i've corrected the docs on row flow and i've created a better sample for fragments. It will be online with the next release of Advanced Slides.
Until then here the fixed sample for fragments:

### Fragments:
+ Right
+ Bottom
</grid>

<grid drag="25 55" drop="-5 10" bg="green" frag="1">
Right Grid
</grid>

<grid drag="90 20" drop="5 -10" bg="gray" frag="2">
Bottom Grid
</grid>

fixed with 1.0.1

Thanks!


Added htmlToPrism convertor

This allows you to convert html to standard markup for prism.
eg:

{{ htmlToPrism('
hello
') }}

Any quotes will need to be escaped i.e:

{{ htmlToPrism('
Dave\'s house
') }}

You could potentially use block assignment in nunjucks with that @htmlandbacon? E.g:
{% set chunkOfHTML %}
<fieldset class="inline">
  <legend>
    Do you agree to the Carer\'s Allowance unit contacting anyone
    mentioned in this form?
  </legend>

    <input id="radio-inline-1" type="radio" name="radio-inline-group" value="Yes">
    Yes

    <input id="radio-inline-2" type="radio" name="radio-inline-group" value="No">
    No

</fieldset>
{% endset %}

{{ htmlToPrism(chunkOfHTML) }}

Might make it easier to edit perhaps?

I think, that would be next steps. Maybe even a little macro to wrap the whole thing.
Didn't have time to do a full refactor and check the elements didn't die. Will do it after I have sorted the issues with some of the html.


How to deal with inheritance?

In CSS (Cascading Style Sheets), inheritance is one of the core concepts that reduces the amount of code needed to style a website considerably. The current approach to defining typography tokens does not make use of inheritance because Figma does not offer this concept (in a way font styles resemble the concept, but font styles are not inheritable).
How can we profit from the efficiency of inheritance of CSS while not optimising the token system for the web (design tokens should be generic design decisions and not targeted at a specific platform).
Issue with the current implementation
The current implementation defines a composite font token for every component. This token contains values for font-family, font-weight, line-height, font-size and letter-spacing:
{
  "legend-l": {
    "value": {
      "fontFamily": "{post.sem.font-family.default}",
      "fontWeight": "{post.core.font-weight.700}",
      "fontSize": "{post.core.font-size.32}",
      "lineHeight": "{post.core.line-height.120}",
      "letterSpacing": "{post.core.letter-spacing.default}"
    },
    "type": "typography"
  }
}

Those values are all inheritable. When set at document root level, they only have to be set where they derive from the default. Defining all those values over and over again, creates a lot of unnecessary repetition.
Proposed solution
Since inheritance is a web related feature, it should only be taken into account for the file exports targeted at the web platform in the tokens package (the .scss files). With the custom transformer written to generate these files, it could be possible to de-dupe font definitions that don't deviate from the standard.
The challenge with this approach will be to track down all references through the semantic layer to figure out if the component needs a definition for letter-spacing because there might be a change based on one of the semantic layer definitions.
Alternative solution (manual)
Another solution could be to manually analyse each component and define the properties that could change and therefore need to be set in the components style definition (font-weight for titles as an example because that deviates from the standard font weight).
The challenge with this approach would be maintainability over time and the big possibility that token changes get introduced later that don't get propagated to the dev team which would lead to incorrect styling.
Have any other ideas?

Resolution: Can't change the current way since inheritance is not a feature in all implementations


Adapt service comparison to only check fields we're interested in

What does this PR do?
Updates the comparison options for services to use a custom function, in order to avoid the continuing issues we face with new defaults in Services on updated versions of k8s.
What issues does this PR fix or reference?
Fixes https://github.com/devfile/devworkspace-operator/issues/627
Fixes https://github.com/eclipse/che/issues/20577
Is it tested? How?
Start a DevWorkspace on k8s v1.22 (e.g. minikube v1.23.2)
PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)

[ ] v8-devworkspace-operator-e2e: DevWorkspace e2e test
[ ] v8-che-happy-path: Happy path for verification integration with Che

/test v8-devworkspace-operator-e2e, v8-che-happy-path


Updating console.log statement to use this.ui.writeLine instead

This will make the output obey the --silent flag in ember-cli commands.

Oh that's great, didn't know about this.


Multiple canvas renders not binding popups

How to reproduce

Leaflet version I'm using: 1.2.0
Browser (with version) I'm using: Chrome 61, Firefox 55
OS/Platform (with version) I'm using: Windows 10

Steps

Add a vector layer to the map using a specified canvas renderer and bind a popup to it
Add another vector layer to the map using a different canvas renderer and bind a popup to it

What behaviour I'm expecting and which behaviour I'm seeing
I expect that the popup bound on the layer from step 1 to open when clicked, but it does not. The popup bound to layer 2 does work, however.
The cursor pointer also does not change to indicate the step 1 layer is clickable.
Minimal example reproducing the issue
https://codepen.io/jaymmartin/pen/yoQJzw

Hi!
Thanks for reporting this. The issue is that a canvas renderer acts as a layer covering the whole map container. Although the canvas is transparent (except for what's drawn on it), it will in fact swallow pointer events. So, putting two canvases on top of each other, only the topmost will receive the pointer events.
This is not something specific to Leaflet, but rather a fact of how the browser's DOM works.
BTW, I have a kludge workaround for this that you can have a look at: https://gist.github.com/perliedman/84ce01954a1a43252d1b917ec925b3dd

I see now.
Multiple canvas renderers are useful, as redraw performance is much better if you can segment the layers on different canvases. For my case, I am using mouseover/mouseout, so that workaround will not work.
What about a solution where a canvas object forwards an event to the Map object, which then forwards it to the next canvas object? L.Map would need to track all of its canvas renderers and order in the DOM if it doesn't already. I'll see if I can get something out of this and update here.
// L.Canvas
onAdd: function () {
	Renderer.prototype.onAdd.call(this);

	// register with map as canvas
	this._map.registerCanvasRenderer(this);

	// Redraw vectors since canvas is cleared upon removal,
	// in case of removing the renderer itself from the map.
	this._draw();
}

_handleMouseHover: function (e, point) {
	var layer, candidateHoveredLayer;

	for (var order = this._drawFirst; order; order = order.next) {
		layer = order.layer;
		if (layer.options.interactive && layer._containsPoint(point)) {
			candidateHoveredLayer = layer;
		}
	}

	if (candidateHoveredLayer !== this._hoveredLayer) {
		this._handleMouseOut(e);

		if (candidateHoveredLayer) {
			DomUtil.addClass(this._container, 'leaflet-interactive'); // change cursor
			this._fireEvent([candidateHoveredLayer], e, 'mouseover');
			this._hoveredLayer = candidateHoveredLayer;
		}
	}

	// if no layer was found, forward it to the next canvas renderer if it exists
	if (!candidateHoveredLayer) {
		this._map.forwardCanvasEvent(this, e, point);
	}
	if (this._hoveredLayer) {
		this._fireEvent([this._hoveredLayer], e);
	}
},

// L.Map
registerCanvasRenderer: function(layer) {
	this._canvasRenderers.push(layer);
}

forwardCanvasEvent: function(layer, e, point) {
	for(var i = 1; i < this._canvasRenderers.length; i++) {
		if (this._canvasRenderers[i-1] == layer) {
			// would really need to figure out which event happened here
			// maybe pass it off to a method in L.Canvas, or have this
			// method be told what to call
			this.canvasRenderers[i]._handleMouseHover(e, point);
		}
	}
}

I can see this solution messing with separation of concerns. I wouldn't want L.Map to be so involved in the intricacies of L.Canvas. So maybe generalizing the pattern would be helpful. Something like event forwarding/notification of sibling layers? I don't know the Leaflet codebase well enough to know how this plays into the event framework.


brain1_image.nrrd    brain1_label.nrrd

brain1_image.nrrd， brain1_label.nrrd,  this is two type files in  test data, what do they represent and what is the difference between them, how can I get them from medical imaging

You can visualize the data in brain ik image and label using e.g. slicer.
Image is the image, I.e. the stack of MRI slices converted from DICOM into format PyRadiomics can use. Label or mask is an image which identifies the region of interst in the image by setting the corresponding pixels to some value, e.g. "1", with "0" being the background.
If this is very new to you, my advise would be to start with PyRadiomics using the SlicerRadiomics extension in 3D Slicer, as you can directly visualize the image and label/mask that is used to extract features. Once you get a better feel for how it works, you can use the standalone version to easily extract features for a batch of cases.

@JoostJM thank you for your replies.


[viostor] Backport vioscsi SendSRB fixes and improvements

Backports fixes and improvements from vioscsi PRs #1150 and #1162
virtqueue struct vq was also removed in favour of adaptExt->vq[QueueNumber], which results in a minor performance increase.

This will likely fix issues #756 and #907.

@kostyanf14
I'm taking a punt that you're the person to ask the following. Please let me know if I should ask someone else.
Any idea why the checks are failing here and for #1175 also?
Looks like mostly file not found errors or what look to be environment-based unexpected reboots...
Is there a temporary problem with the test environment...?
Would appreciate your insight, especially if there are problems with my PRs...!
Best regards,
Ben

@benyamin-codez
We have an upstream bug for vioscsi. Viostor is unexpected behavior, we need to investigate the reason.

@kostyanf14
From a quick look it appears to be the same issue, i.e. an unexpected 0x80070002 on a file copy...

@YanVugenfirer
If I'm not mistaken, this one should be good to go too, per https://github.com/virtio-win/kvm-guest-drivers-windows/pull/1181#issuecomment-2466981842.
I note I have presumed (imho, fairly) that the resend logic wrapping the spinlocks in RhelDoFlush() is still valid.
Best regards,
Ben
cc: @vrozenfe, @kostyanf14

Thanks!

Can anyone advise if these fixes to viostor have made it into the binary downloads available from https://fedorapeople.org/groups/virt/virtio-win/direct-downloads/ yet?
I can't see any mention in the changelogs there as to what's included.

@robleady
I don't believe they are in v266. See my comment here.
Per Vadim's comment above that post, the next stable public build will be released in mid-end of January 2025.


Fix Tests

Although this is not a priority, several of the tests need to be updated to be compatible with the now-updated dependencies. This will also fix the Build:Failing message on the git page.

In case this becomes a nightmare which sucks time and energy: Ping me and we postpone to a later milestone.


Black screen when running a project

Godot version:
Commit d0aaf4a
OS/device including version:
OS: Arch Linux x86_64
WM: bspwm
GPU: Nvidia GTX 1070
Vulkan backend
Issue description:
When I start the game with F5, I get a black screen. Editor 3D view works fine.
I am running with --single-window -e --path ~/Documents/gd4project.
Steps to reproduce:
Run the project from editor.
Minimal reproduction project:
Does not really depend on the project. I created a scene with a MeshInstance and a camera pointing at it.

There is no default environment anymore, so you need to add a light to your scene to be able to see anything. (If you're placing an OmniLight or SpotLight, make sure it's not placed inside the mesh.)


Alter categorisation of transactions based on statement info

Credits and debits are now recorded as such, and transactions which
are not payments to a prisoner are recorded as "administrative".
Requires: https://github.com/ministryofjustice/money-to-prisoners-api/pull/110

by the way I'n open to suggestions for a better word than 'administrative'


Adds an option to automatically retry on a 429 response

Signed-off-by: Christopher Grote<EMAIL_ADDRESS>

Thanks for your contribution!


docs: revise paper to include Statement of need

According the the requirements of JOSS paper , we should include a section of Statement of need. I have just restructured the manuscript a bit to include this.
Besides, shall we also add a Acknowledgements section? Requirements of JOSS paper  also mentions this.

Looks good! I think we can submit the paper now, can't we?

Yes! I will do that.


Add proper parser for url() function

Description
Add proper parser for url() function and improve results. Also add new test.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@moazin Could you take a look at this PR please?


Evolve all, but keep at least 1

Expected Behavior
Add new config options for evolve_all that keep at least X pokemon of this type.
Actual Behavior
Evolve all
Steps to Reproduce
{
        "type": "EvolveAll",
        "config": {
          "evolve_all": "Rattata,Pidgey,Slowpoke,Caterpie,Metapod,Weedle,Kakuna,Pidgeo$
          "evolve_cp_min": 10,
          "evolve_speed": 20,
          "use_lucky_egg": false
        }
}

Other Information
OS: Linux
Git Commit: 494bc055543c3cb674cca781ec545311d3c9bad1
Python Version: Python 2.7.9

Configurate your "release" tot his:
    "release": {       "any": {"keep_best_cp": 1, "keep_best_iv": 1, "release_below_cp": 999, "release_below_iv": 0.9, "logic": "or"} }
and make sure the evolve task in your tasks array is BEFORE your transfer task.

I already have "any": {"keep_best_cp": 2} and the evolve task is before the transfert.
But I think you missunderstood, when evolving all, I want to evolve total - 1 and then keep 1 not evolved. It is not about releasing pokemon...

good idea
evolve NOT ALL (Rattata,Pidgey...) but keep one 'safe' and then evolve all other.
which one? can be the starred/preferred one ? or 1st (2nd?) by iv / cp?

I think we can apply the same options than release keep_best_cp, keep_best_iv, ...

scenario : have
Vaporeon 14/14/14
Eevee (nicknamed Rainer ) 15/15/15
Evee 14/14/14
is not better keep eevee 14/14/14 and evolve 15/15/15 ?

Hum... so we need to have new options like evolve_best_cp, ...?

Where do you paste this ?
Config.json or config_release ?
"release": { "any": {"keep_best_cp": 1, "keep_best_iv": 1, "release_below_cp": 999, "release_below_iv": 0.9, "logic": "or"} }


Search image data from /api/conversation/custom when using langchain

Required by https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator/issues/323
Description
When calling the /api/conversation/custom endpoint and using the langchain orchestrator, generate embeddings of the question using computer vision and query the image embeddings using the question embeddings. We should continue to create question embeddings using text-embeddings-ada-002 in addition to be used to search the image description.
Tasks

[ ] Generate question embeddings using computer vision
[ ] Query the search index using the generated embeddings
[ ] Pass any returned images to gpt-4-vision along with the chat

This will be covered by #750, as the changes are in the question answer tool


Funnel all intents to same alexa-app intent handler? Please help.

I just searched through the alexa-app readme.  I did a full search for the word "intent" and read the text surrounding every one of the 85 occurrences of the word "intent".  I also searched for the word "unhandled" (0 occurrences).  Nowhere could I find out how to either:

Catch an unhandled intent name
Funnel multiple intents to the same intent handler.

I have several hundred highly specified intents, all handling different semantic and contextual situations.  I need to route them all to the same alexa-app intent handler.  I can't figure out how to do this.  Can someone tell me how I can do either #1 or #2 above?
If not, can someone point me to the source Javascript file in the alexa-app package that looks for the incoming matched intent name and then calls the corresponding intent handler defined during the app.intent() declaration?
I'd hate to after hack the package code just to change the intent handler code to always call the same intent handler, but that's what I'll have to do if there isn't some facility in the package to do this.

For 2, just create a single stand-alone function that handles an intent, with request and response as parameters. Then define all your intents and pass this function as the handler. That way you'll still have lots of different intents, but just one handler function.
But why do many intents? Since a single intent can have hundreds or thousands of different utterances attached to it, why not combine all those intents into just a few, with lots of different phrases?
For 1, there is no way to trap a named intent that does not have a handler function. Because, this shouldn't even be possible. Since the schema should be generated from your skill code, and the code defines the intents, it would be impossible to have an intent that does not have a handler. If you are doing these two things separately, then that could happen, in which case you will get an exception thrown that there was no handler defined.
If you're looking for the more general handler, where Alexa didn't recognize the text spoken and didn't match it to an intent, and you want to handle that case, then that's not current a feature of Alexa at all. But it will be coming.

@matt-kruse Hi Matt.  I'm a veteran NLP developer so I already have a large body of code that handles tons of user input variations.  I just need to get the input coming from Alexa to the code.
I'm OK now.  I've looked at the index.js file for alexa-app and found the code that throws an error if a named intent handler could not be found:
      if ("IntentRequest" === requestType) {
        var intent = request_json.request.intent.name;
        if (typeof self.intents[intent] !== "undefined" && typeof self.intents[intent].handler === "function") {
          if (self.intents[intent].isDelegatedDialog() && !request.getDialog().isCompleted()) {
            return Promise.resolve(request.getDialog().handleDialogDelegation(request, response));
          } else {
            return Promise.resolve(self.intents[intent].handler(request, response));
          }
        } else {
          throw "NO_INTENT_FOUND";
        }
      }

I'm just going to add code that instead of immediately throwing an error if a named intent is not found, looks for an intent handler named "unhandled" first and calls that handler if it exists.  If not, then it throws the handler.

Closing this issue now since I have my solution.  I added this to the alexa-app index.js file.  I just tested it and it works:
      if ("IntentRequest" === requestType) {
        var intent = request_json.request.intent.name;
        if (typeof self.intents[intent] !== "undefined" && typeof self.intents[intent].handler === "function") {
          if (self.intents[intent].isDelegatedDialog() && !request.getDialog().isCompleted()) {
            return Promise.resolve(request.getDialog().handleDialogDelegation(request, response));
          } else {
            return Promise.resolve(self.intents[intent].handler(request, response));
          }
        } else {
          // ROS: Add support for calling a intent handler named "unhandled" if it exists.
          var unhandledIntentHandlerName = "unhandled";

          if (typeof self.intents[unhandledIntentHandlerName] != 'undefined' && self.intents["unhandled"] != null)
            // Call the unhandled intent handler.
            return Promise.resolve(self.intents[unhandledIntentHandlerName].handler(request, response));
          else
            // Throw an error.  No suitable intent handler found.
            throw "NO_INTENT_FOUND";
        }
      }

That's one way to fix it, but you won't be able to upgrade alexa-app since you've modified its internals.
I guess I still don't understand your problem well enough, because this really should be unnecessary. This doesn't feel like the right solution for any situation.

@matt-kruse I appreciate your help and I wouldn't expect you to modify alexa-app for my situation since it is extremely different from anyone else that will use your package.  Thanks again for your help.

I would quickly submit a patch to accomplish this, but to be honest this project has become almost too cumbersome to contribute to. Quick, simple enhancements get buried in tooling, tests, type definitions, etc. I'd love it if this could move faster with less friction so we could adapt to the rapidly-changing Alexa environment. There are lots of new features just on the horizon that will need to be supported with code changes!

I would like to see PR's merged that do not have corresponding tests, or that maybe cause the automated checks to fail, if the code is known to be good. Tests can be added later, etc. Because in the end, all the build and automation doesn't matter, it's whether index.js runs as intended.
Ideally, it would be great to have everything 100% green and good with each commit. But for people like me, I just can't afford that much overhead, I've got lots of other things to move forward. I understand the value, not saying it's bad, just saying it prevents me from contributing fixes that I know would help others.

What you call overhead just moves the burden on others to debug your code when it doesn't work or when your code breaks someone else's code accidentally. I don't have time for that, so personally, I am not interested in participating or using a library that doesn't ensure that existing features weren't broken or new features added actually work at the time when they are written.
You as the originator of this project can take it any direction, but I encourage you to look at the hard work that was put in here by 37 contributors including some very big things by @ajcrites @kobim or @lazerwalker since your last major commit in 2016 and how smoothly things have gone by ensuring a level of quality that burdens the developers to write tests with their new features.

I understand, and I'm fine with it staying that way. I like it being that robust. For me, personally, I can't keep up with that level of complexity in the process. I don't understand all the layers that are in place now. And even though I have advance access to new Alexa API's and can code them into alexa-app to have ready the day they are released to the public, I find that I can't do so because of the burden of tests, etc. My last simple change to add the .on() handler took a lot of time and patience for just a few lines of code. I wish I had time to invest in understanding all of it, but I just don't.
So I'm fine with it staying how it is, but it will depend on others to implement new API features, and will significantly lag behind the coming API's from Amazon. For example, I'm currently coding a layer of support for the Gadgets API, but I'm doing it in a module that I will release separately, because I won't be able to meet all the requirements for contributing to this code base.
I guess the real question is, how can we be accommodating to developers who want to contribute good code to the project, but lack the knowledge/experience required to write tests, declare types, etc? Or are contributions limited to only those who understand the tooling in place? The latter is not my preference, but I understand it, and I will continue to do what I can in that mode.

Thanks. I think there're a few things I can suggest.

Make pull requests that don't have tests and have working code and just add that you don't have time to finish the PR and hopefully someone can jump in and do it.
Try to do just a tiny bit of extra work. For example, I personally find that working with tests is much faster than without, I get more done in less time by writing a failing test first.
Find time to consider learning the new tooling, like TypeScript, it will change a lot of how you think. Using things like VSCode with TS is just amazing. Keep an open mind.

@matt-kruse Will you be making your changes to a fork of the project instead?  That Gadgets API sounds interesting.


⚠ Bofrost.be has degraded performance

In 985a353, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1343 ms

Resolved: Bofrost.be performance has improved in 4b16836 after 10 minutes.


0xarthurxyz/escrow docs update

Updates Escrow docs

Adds detailed Mermaid diagrams that visualise

the two payment flows (private key- and phone number-based)
the revocation flow

I added support for Mermaid diagrams in the Celo docs with this separate PR: https://github.com/celo-org/docs/pull/331

Adds links to Valora implementation, ContractKit wrappers, and smart contract source code
Adds smart contract details such as emitted events, main mapping and attributes of escrowedPayment struct

👉 You can find the built Escrow page with diagrams and changes here 👈

In the efforts of getting this merged, I think it'd be sufficient to:

delete copied functions
delete unfinished/incorrect snippets

Once merged we can add new working snippets.
@0xarthurxyz

@arthurgousset any plans on reworking on this?

@arthurgousset any plans on reworking on this?

Not personally, but someone else might use this PR for reference. I'm closing it on that basis.


Support dynamic configuration of model databases

Feature
Feature description
dbt has historically only supported a single database as the destination for models. This comes from a Redshift-centric worldview, in which reading/writing data across databases is not supported. With Snowflake, BigQuery, and Presto, cross-database reads/writes are indeed supported, and frequently very useful.
dbt should support the dynamic configuration of model databases, as it current does with schemas and aliases. Though different adapters use different terminology for databases, dbt should only accept the config value database.
Example usage:
models/something.sql
{{ config(database='dev') }}

select * from somedb.someschema.sometable

Assuming a profile config:
database: analytics
schema: dbt_drew

dbt will build the relation:
dev.dbt_drew.something

Example dbt_project.yml usage
models:
  my_project:
    postgres:
      database: postgres
      ....

Implementation Details
On Snowflake and BigQuery, I think dbt can get away with just writing:
create table database.schema.identifier as (
   ...
)

On databases like Presto, dbt may instead need to issue a use {catalog} query (or similar) in order to create the relation in the appropriate catalog. Further investigation is required to determine the exact syntax that is required.
In the general case, dbt shouldn't try too hard to restrict users from performing invalid operations. While there is generally little use for custom databases on Redshift, dbt should allow the user to use custom databases and then return an error message from the database accordingly. We can definitely document this, but support for cross-db queries is pretty varied, and the database will always do a better job of generating error messages than we will.
Who will this benefit?
Snowflake, BigQuery, Presto users

A couple questions as I start to dive into this:

On BigQuery, we already support "projects" in some ways, right? Is that their version of "database"? If so, should bigquery keep calling it "projects" throughout? This is mostly going to be a thing about flipping the "include policy" to true.

Maybe we should define some consistent names for internal-to-dbt use and a formal map of aliases or whatever that we maintain so users can give consistent behavior. Maybe adapter plugins could define their own aliases for the "canonical names" and dbt could automatically handle it?

regarding this bit:

While there is generally little use for custom databases on Redshift, dbt should allow the user to use custom databases and then return an error message from the database accordingly.

Does this mean we should issue the database.schema.identifier syntax on Redshift if it's specified (and I guess flip the include_policy flag at that point), or that we should open a new connection to the database with the new database field? I would expect dbt to try to connect to the new database and issue the command there, since Redshift does have a concept with that name. Or I would expect an error.

@beckjake you're totally right - BigQuery's "projects" map to dbt's general notion of "databases". I like the idea of aliasing these configs in the adapters. On BQ for instance, schemas are actually called "datasets". This leads to a nonzero amount of confusion, and it would be good to let people specify a dataset config instead of a schema config on BigQuery. Historically, my fear is that the code would become a mess if we tried to do this, but if you can divine a good mechanism for doing this, then i'm into it!
I think aliasing profile configs like database/schema is super appropriate, but I definitely do not want to implement our own nomenclature for table-level configs. There is some overlap between:
Redshift: sort/dist keys
Snowflake: clustering keys
BigQuery: clustering keys / table partitioning
It sounds to me like each database will require its own database/connection logic. On Snowflake, you can connect to any database you want, then execute queries against a different database. Here, I connected to dbt_test:
create schema dbt_test_2.debug;
create table dbt_test_2.debug.new_table as (select 1 as id);
select * from dbt_test_2.debug.new_table;

^ That definitely doesn't work on Redshift, and I'm unsure how well it's supported on other databases (eg. Presto). I imagine it will be much simpler to connect to a single database, then execute queries against a different database using the database.schema.identifier syntax when possible. If dbt did make a new connection for each distinct database, I imagine our connection pooling code would become a lot more complicated :)
I think to start, dbt should just connect to the database specified in the profile, then use database.schema.identifier everywhere. This will break for cross-db queries on Redshift, and that's a-ok with me. I think it will work with Snowflake/BQ/(+Presto?), which is where this feature is really useful.
I'm still clarifying my own thoughts here, so happy to follow up if any of this doesn't make sense!

In Presto, to my knowledge, you do not connect to a specific catalog at all. You can use {catalog}.{schema} during a connection to switch the "active" catalog/schema, but the syntax you outlined above on snowflake will work fine. I do not understand at all how Presto chooses the default catalog/schema, but I can tell you that it seems to be 'hive' even if you don't have a connector with that name configured!
When you say this:
I think to start, dbt should just connect to the database specified in the profile, then use database.schema.identifier everywhere. This will break for cross-db queries on Redshift, and that's a-ok with me. I think it will work with Snowflake/BQ/(+Presto?), which is where this feature is really useful.

How do you see redshift/postgres' dbname interacting with the database config? Does database just override it? Same goes for the bigquery project thing. One thing I feel this issue is missing is how profile configs should interact with configurations.

yeah, great question. Let me carve out some time to look through all of the existing configs in dbt and build out the mapping. I imagine that each adapter can supply accepted aliases for profile configs, but it would be an error to supply a value for one of these configurations with > 1 alias. Will follow up here shortly!

I think it will work with Snowflake/BQ/(+Presto?),

Postgres

Maybe we should define some consistent names for internal-to-dbt use and a formal map of aliases or whatever that we maintain so users can give consistent behavior.

Love this idea. Though I would be in favor of soft deprecating dbname for now and eventually getting rid of it. project and catalog are less clear, because those are meaningful in the context of those adapters.

Ok, thanks for the clarifications! I think since in the adapters as plugins world we basically make the plugin responsible for deciding how to parse its configs, this will come out pretty clean on the profile front - the plugin already has control of that since it provides its own credentials class+json schema.

Apologies for reopening a closed issue, but I'm not sure that this works if you specify a database in your profiles.yml file - if I set up my dbt_project.yml file like so:
models:
dwh:
folder:
+database: DIFFERENT_DB_FROM_PROFILES_YML_FILE
dbt run gives an error:
"Schema '<PROFILES_DB>.SCHEMA' does not exist or not authorized."
Interestingly, this seems to actually succeed - the correct table (DIFFERENT_DB.SCHEMA.MODEL_TABLE) does get updated in Snowflake when I run this - but DBT is telling me it fails and triggers all the associated alerts I have set up.
Entirely possible this is user error on my part - does anyone know how this is supposed to work? Or, any workarounds so a single DBT project can write to multiple databases?


add most popular alternative medicine practitioners

i.e. all have above 100 usages. Osteopathy has 1300+ down to... homeopathy has 100+ or something.
I used the https://fontawesome.com/icons/spa?s=solid&f=classic icon (I think this is supposed to be some leaves?) for all of alternative medicine (except ayurveda, which got the hinduism icon) because it should not have the same icon as hospital, IMO.

https://fontawesome.com/icons/leaf-heart?s=regular&f=classic would also be a reasonable icon.

Not all alternative medicine practices are related to "natural medicine" / herbs though, so displaying something that can clearly be identified as a leaf could be misleading.
https://fontawesome.com/icons/spa?s=solid&f=classic is more ambivalent. TBH I don't exactly know what it ought to resemble, leaves was just a guess. Maybe it is a lotus flower. In any case, it seems to be a popular logo for alternative medicine.
For Traditional Chinese Medicine (and acupuncture, because it is actually part of it), another logo that may work well is the logo commonly used for the range of religious practices known as Chinese folk religion since the former is heavily related if not part of these religious practices:

禄 lù or 子 zi symbol

I understand it’s pretty hard to find something that fits everything. Looking at logos online there’s usually leaves, with the implication that the medicine is “natural” rather than pharmaceutical. I liked the leaf+heart because it adds something to relate it back to health, but I I’m happy with your original choice as well.

https://fontawesome.com/icons/hand-holding-medical?s=solid&f=classic could have also potentially worked as an icon here, but I don't mind the proposed one.

Chinese folk religion […] 禄 lù or 子 zi symbol […] would need to be added to temaki?

yes, it would be best if this icon could be added to temaki


Add check for incompatible numerical data type in standardisation

Signed-off-by: Beat Buesser<EMAIL_ADDRESS>Description
Add check for incompatible numerical data type in input standardisation of classifier
Fixes #224
Type of change
Please check all relevant options.

[ ] Bug fix (non-breaking)
[x] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@minhitbk That's a good observation. Attacks that apply gradients leading to negative values in x would produce unexpected results.  Should we apply the check everytime even for preprocessing=None e.g. by moving the changes of this PR up by one line to before if self.preprocessing is not None:

@beat-buesser I agree, that check should be moved to the beginning of the function to cover both cases of with/without preprocessing.


Edit Sticker Text on Double Tab

How can I edit sticker text on Double Tab of Sticker view it will help to edit that sticker text can you please help me how should I integrate it in android

Currently，you only can single tap to change the text.

Still the text is not changing showing same text

Maybe you need to call explicitly stickerView.invalidate()
2016-12-15 12:50 GMT+08:00 supriya456<EMAIL_ADDRESS>
Still the text is not changing showing same text
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wuapnjie/StickerView/issues/11#issuecomment-267238290,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKY_wimDOh_AjV1GYE42g85UtZJTZ1yFks5rIMcdgaJpZM4LMnKJ
.

Yes solved issue thanks for help

& one more thing I have to add Edit Sticker Text on Double Tab so what I have to do can you please help me for this one?

Ok. I will add double tap callback soon

Great!
Please let me know after adding it......

I add a double tap callback.But you double tap also will trigger two click callback, so you need to choose which click you really need. You can check demo to see log in logcat with double tap and click and other events

Yes Sure thanks!

I have added more than one text. i want to change color of the text, but problem is that color is changed of the last text. how can i get active sticker to change the color of active sticker?
One more thing i want to add one more sticker image which is used to flip the text vertically.how is this possible?

You can hold all TextStickers’ reference.Then you can change all text colors. And I don’t understand the another thing.

Dear,
         No, I just want to click on one sticker and change that color using color picker dialog.
Second thing is that i want a button sticker on the fourth corner of sticker which is used to flip the text vertically.
I attached an image, please check this image.
On Monday, February 6, 2017 5:30 PM, FlyingSnowBean<EMAIL_ADDRESS>wrote:

You can hold all TextStickers’ reference.Then you can change all text colors. And I don’t understand the another thing.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

No, I just want to click on one sticker and change that color using color picker dialog.
Second thing is that i want a button sticker on the fourth corner of sticker which is used to flip the text vertically.
I attached an image, please check this image.

You can listening the click event on StickerView.
You can easily check the source code to add flip vertically function. Currently this library does not contains this function.

I added the button on StickerView and create  a method for vertical flip.here is code
if (currentMode == ActionMode.FLIP_VERTICAL && handlingSticker != null) {
handlingSticker.getMatrix()
.preScale(1, -1, handlingSticker.getCenterPoint().x,
handlingSticker.getCenterPoint().y);
                handlingSticker.setFlipped(!handlingSticker.isFlipped());
                if (onStickerOperationListener != null) {
                    onStickerOperationListener.onStickerFlippedVertical(handlingSticker);
                }
                invalidate();
            }

//But the problem is that when i click the button the whole sticker orientation is changed.

You can check this latest update. commit
In this update, you can easily config your sticker icon and its event and its position.You can check it in latest demo or readme.
And this library contains four origin event: DeleteIconEvent    FlipHorizontallyEvent    FlipVerticallyEvent  and ZoomIconEvent.  You can see them in source code.

thank you sir its really helpful.
But Please, change the gradle version to (gradle-2.14.1-all.zip).
I am stucked in the beta version of android studio. I am not able to change the gradle version (gradle-2.14.1-all.zip) to (gradle-3.3-all.zip) for two days. I am not able to change the build tool from (classpath 'com.android.tools.build:gradle:2.2.3') to (classpath 'com.android.tools.build:gradle:2.3.0-beta3').
please convert it into lower version that is (gradle-2.14.1-all.zip)  and build tools to (classpath 'com.android.tools.build:gradle:2.2.3').

You can edit this config by yourself.

How can i get current current touching TextSticker to change the text color?
I have added more than one text using dialog with input field edittext. i want to change different color for different text.
//here is my peace of code for adding text using custom dialog on button click:
LayoutInflater layoutInflaterAndroid = LayoutInflater.from(StickerViewActivity.this);
View mView = layoutInflaterAndroid.inflate(R.layout.cust_dialog, null);
AlertDialog.Builder alertDialogBuilderUserInput = new AlertDialog.Builder(StickerViewActivity.this);
alertDialogBuilderUserInput.setView(mView);
            final EditText userInputDialogEditText = (EditText) mView.findViewById(R.id.userInputDialog);
            alertDialogBuilderUserInput
                    .setCancelable(false)
                    .setPositiveButton("Add Text", new DialogInterface.OnClickListener() {
                        public void onClick(DialogInterface dialogBox, int id) {
                            sticker = new TextSticker(getApplicationContext());
                            sticker.setDrawable(
                                    ContextCompat.getDrawable(getApplicationContext(),R.drawable.transparent_background));
                            string = userInputDialogEditText.getText().toString();
                            sticker.setText(string + "");
                            sticker.setMaxTextSize(16);
                            sticker.resizeText();
                            stickerView.addSticker(sticker);
                            stickerView.invalidate();
                        }
                    }).setNegativeButton("Cancel",
                    new DialogInterface.OnClickListener() {
                        public void onClick(DialogInterface dialogBox, int id) {
                            dialogBox.cancel();
                        }
                    });

            AlertDialog alertDialogAndroid = alertDialogBuilderUserInput.create();
            alertDialogAndroid.show();

//And this is how i am changing color of text
private void ColorPickerDialog() {
int initialColor = Color.RED;
ColorPickerDialog colorPickerDialog = new ColorPickerDialog(
StickerViewActivity.this, initialColor,
new ColorPickerDialog.OnColorSelectedListener() {
@Override
public void onColorSelected(int color) {
stickerView.getCurrentSticker();
sticker.setTextColor(color);
stickerView.invalidate();
}
});
colorPickerDialog.show();
}
PROBLEM: problem is that whet i add text and change that color it is changed.And i add another text and change that color, it is changed. But when i touch or click the first text and want to change the color,
color of first text does not change. Color of second text is changed, first text color is not changed On which i click or touch.

You can set a listener to listen all events.Just like click,drag,zoom and
more.

I do not understand. how can i set listener?

stickerView.setOnStickerOperationListener
2017年2月9日 下午8:57，"ejaz1626"<EMAIL_ADDRESS>I do not understand. how can i set listener?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wuapnjie/StickerView/issues/11#issuecomment-278633078,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKY_wqvOnU2NyfdLC3USz_2lrudis5sZks5raw0sgaJpZM4LMnKJ
.

I have set listener.
The color of all the text is changed when i click.
I want to set different color for different text.

please help me
i have a problem to set different text color for different text.
please!

Sorry. In my test.  I can change the text  color separately.

I also can change the text color separately.
But at first time i can change the text color and i add another text and change that color.
When i click the first text, the text color is not changed, on the same time the color of second text is changed which i added.

You can check the latest demo now.

Thank you so much for your effort.
i can apply this before. it is not my solution.
thanks again i shall solve my problem by myself.
Thanks for giving your valuable time.
And sorry for too much disturbance!

The String of Current Textsticker cannot be change :
@Override public void onStickerClicked(Sticker sticker) {
//stickerView.removeAllSticker();
if (sticker instanceof TextSticker) {
((TextSticker) sticker).setTextColor(Color.RED);
((TextSticker) sticker).setText("Test");
stickerView.replace(sticker);
stickerView.invalidate();
}
Log.d(TAG, "onStickerClicked");
}
text of current Textsticker still not Changed, but color and the other changed.
can you help me please?
Sorry for my english language

@kyky04 you can try call sticker.resizeText() after change the text of stickerview.

Is there any code to add shadow to text.
If any way is possible pls let me know

@kyky04  you can change the text via replacing sticker with new one.

Hy can any body help me in sorting out that i'm trying to add new text sticker but its text size on zooming get blurred i need this in good quality.
@wuapnjie @supriya456 @ejaz1626 @kyky04 @awind

@chnouman Try changing the background size of the text
StickerView/tree/master/sticker/src/main/res/drawable/sticker_transparent_background.xml

i think bigger size is better to get good result

its not working @kyky04 dear.

@kyky04 i'm having difficulty in getting text of textSticker i want to change the font color of textSticker when i do this it replaces text with default text of textSticker. Kindly help me please.

@kyky04  please i need your help i want text shadow.

@chnouman check your email

how to change text of textsticker?

TextSticker dump = (TextSticker) stickerViewx.getCurrentSticker();
recentText= input.toString();
final TextSticker sticker = new TextSticker(EditingActivity.this);
sticker.setText(input+"");
sticker.setMaxTextSize(500);
sticker.setTextColor(dump.getTextColor());
sticker.setTypeface(dump.getTypeface());
sticker.setTextAlign(Layout.Alignment.ALIGN_CENTER);
sticker.resizeText();
stickerView.replace(sticker);

TextSticker dump = (TextSticker) stickerViewx.getCurrentSticker();
recentText= input.toString();
final TextSticker sticker = new TextSticker(EditingActivity.this);
sticker.setText(input+"");
sticker.setMaxTextSize(500);
sticker.setTextColor(dump.getTextColor());
sticker.setTypeface(dump.getTypeface());
sticker.setTextAlign(Layout.Alignment.ALIGN_CENTER);
sticker.resizeText();
stickerView.replace(sticker);
where is from input?

i want to change text on double tab textsticker .. is there any solution of it ?

TextSticker dump = (TextSticker) stickerViewx.getCurrentSticker();
recentText= input.toString();
final TextSticker sticker = new TextSticker(EditingActivity.this);
sticker.setText(input+"");
sticker.setMaxTextSize(500);
sticker.setTextColor(dump.getTextColor());
sticker.setTypeface(dump.getTypeface());
sticker.setTextAlign(Layout.Alignment.ALIGN_CENTER);
sticker.resizeText();
stickerView.replace(sticker);

what is input here


Feature request: download, events

I'll probably end up doing this myself, I just wanted to write it down as a reminder

[x] download
[ ] events
[ ] transcode jobs
[ ] optimize media
[ ] share/unshare
[ ] delete
[ ] add/delete users

Juts curious, whats the difference between sync (which someone added years ago) and download?

Sync doesn't seem to download the files, only listing the synced items and telling the server some thing is synced. I could be wrong tho.

Added in https://github.com/mjs7231/python-plexapi/pull/100/commits/0c987d6a73e3d4da243603e5d208212824fbc366

Be careful with delete. If the server doesn't validate that delete is enabled, maybe we should get the setting and validate it on our end.
Not sure what events are, but it sounds interesting.

I haven't really looked into how to delete a item. I was hoping I could send a http delete request and pms would send http 405 if it wasn't enabled. And we could just handle it from there. I'll start on this today.

You should probably work on the 3.0.0 branch (or a fork if you wish), or we're never have it conflict free. :D

I forked of 3.0.0 and targeted that branch for the pr.

I realized I basically recreated the same type of bug which is a list of enhancements. Merging the two now.


Fix invalid cloning commands

Fixed invalid commands for cloning the repository. The commands mentioned in the readme have invalid user.

Actually, 'username' indicates that anyone who want to contribute should put their GitHub username instead of 'username'.
After forking the repository.

Ah okay, I thought these commands were for cloning the main repo, my bad.

Ah, no problem.


I want to know why initialize dim，token_dim and channel_dim  using 512,256,2048?

One reason could be Computer/GPU algorithms in general are optimized for values in powers of 2 because that way we can utilize the memory access in the most efficient manner.


Different syntax for fitting the models. What's the difference?

Hello.
I would like to use MixedModels.jl to fit regression models with random effects.
Looking at the official docs and some examples on the Internet I've found different syntaxes.
For example if I want to fit a Logit model (my outcome is binary) I can do it...:
fit(GeneralizedLinearMixedModel, @formula(y ~ 1 + x + (1 | id)), df, Bernoulli())
fit!(glmm(@formula(y ~ 1 + x + (1 | id)), df, Binomial(), LogitLink()))

What's the difference between both syntaxes?

@skanskan This should work in the current release.


[ZEPPELIN-2956] [font-end] Downloaded CSV/TSV data will get unexpected division when the column value contains both delimiter and quotation mark.

What is this PR for?
Fix for ZEPPELIN-2956. This PR is to deal with the case when column value contains both delimiter and quotation mark. So the downloaded data can be well recogized and seperated by Excel.
What type of PR is it?
Bug Fix
Todos
None
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2956
How should this be tested?
Manual test.
After applying this PR, you can test the patch as following steps:
1、suppose one of the column value contain both delimiter and quotation mark:
=>  hello  beijing  i  lo"v,"e  you
2、download the data as csv/tsv file
take download as csv format file =>  hello,beijing,"lo""v,""e",you
3、open the data file through Excel to verify the bug fix.
Screenshots (if appropriate)
before patch:

after patch:

Questions:

Does the licenses files need update?
no
Is there breaking changes for older versions?
no
Does this needs documentation?
no

could you setup CI tests
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration

@felixcheung  I have setup the CI tests, but Jenkins server seems to have no space on device.
FATAL: Unable to produce a script file
java.io.IOException: No space left on device
at java.io.FileOutputStream.writeBytes(Native Method)
Caused: java.io.IOException: Failed to create a temp file on /home/jenkins/jenkins-slave/workspace/zeppelin-pull-request
at hudson.FilePath.createTextTempFile(FilePath.java:1386)
at hudson.tasks.CommandInterpreter.createScriptFile(CommandInterpreter.java:162)
at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:94)

hmm, that's odd, I don't think I've seen that before.... perhaps.. try again?

I tryed again, but CI test failed job 6.4, and I could not figure out what the error means.
https://travis-ci.org/BruceXu1991/zeppelin/jobs/281177302

@BruceXu1991 Right, https://github.com/apache/zeppelin/pull/2603 fix CI job 6.4 error.
You can try merge master and see if CI becomes green!

After merging to master, CI tests still have some exception, but the exception seems have no relation with this patch.

job 7.2 https://travis-ci.org/BruceXu1991/zeppelin/jobs/281329390
job 7.3 https://travis-ci.org/BruceXu1991/zeppelin/jobs/281329391

Failed: No element found using locator: By(xpath, //div[@class="new-paragraph" and @ng-click="insertNew('below');"])

15:11:48,992 ERROR org.apache.spark.scheduler.LiveListenerBus:91 - Listener  threw an exception
java.lang.NullPointerException
	at org.apache.zeppelin.spark.Utils.getNoteId(Utils.java:156)
	at org.apache.zeppelin.spark.SparkInterpreter$1.onJobStart(SparkInterpreter.java:189)
	at org.apache.spark.scheduler.SparkListenerBus$class.doPostEvent(SparkListenerBus.scala:37)
	at org.apache.spark.scheduler.LiveListenerBus.doPostEvent(LiveListenerBus.scala:36)

Hi, https://github.com/apache/zeppelin/pull/2323 added a new table library and it supports downloads. (for 0.8.0-SNAPSHOT)

_ http://ui-grid.info/docs/#/tutorial/206_exporting_data
Could we remove the custom csv / tsv generation for 0.8.0-SNAPSHOT and above versions?
What do you think of?

@1ambda  If the table library in #2323 could deal with the problem of downloading column value both contain delimiter and quotation mark, It may be better to use library.

I wonder if it's related to https://issues.apache.org/jira/browse/ZEPPELIN-1803 ?

@Tagar  Yes, ZEPPELIN-2956 and ZEPPELIN-1803 point to the same issue. RFC-4180 explains the issue.

ok, I will do it.

@felixcheung The API(http://ui-grid.info/docs/#/tutorial/206_exporting_data) only support export data as csv format but no tsv format supporting. So if using this API, tsv format related PR should be submited to the API lib first. Maybe we could change to #2601 (comment) after the API support tsv format.

@BruceXu1991 I think that's fair. @1ambda what do you think?

@1ambda   hi, is there any conclusion?

@mebelousov   OK, I will close this PR.

I believe one could remove old button "Download Data as CSV".

filed https://issues.apache.org/jira/browse/ZEPPELIN-3511 to remove old export csv/tsv
This idea was also briefly discussed on PR-2971


IllegalArgumentException: Too many seed candidates for specified population size.

Hey,
I'm working on simple example using multiple islands.
I have code as follows:
List<EvolutionEngine<FloatGenotype>> islands = new ArrayList<>();

Random rng = new MersenneTwisterRNG();

for(int i = 0; i < this.islands; i++) {
    CandidateFactory<FloatGenotype> factory = new FloatGenotypeFactory();

    List<EvolutionaryOperator<FloatGenotype>> operators = new LinkedList<>();
    operators.add(new AverageFloatCrossover());
    operators.add(new UniformFloatMutation());

    EvolutionPipeline<FloatGenotype> pipeline = new EvolutionPipeline<>(operators);

    FitnessEvaluator<FloatGenotype> evaluator = new FloatRastriginEvaluation();

    SelectionStrategy<Object> strategy = new TournamentSelection(selectionPressureControl
    .getNumberGenerator());

    EvolutionEngine<FloatGenotype> engine = new GenerationalEvolutionEngine<>(factory, pipeline,
    evaluator, strategy, rng);

    islands.add(engine);
    }

    IslandEvolution<FloatGenotype> islandEvolution = new IslandEvolution<>(islands, new RingMigration(),
    true, rng);
    islandEvolution.addEvolutionObserver(monitor);

    return islandEvolution.evolve(populationSize, // Population size per island.
    20, // Elitism for each island.
    50, // Epoch length (no. generations).
    15, // Migrations from each island at each epoch.
    terminationConditions;

Everything works fine when I'm setting populationSize on 120 or above. But if i try to use lower value i get following error on the end of first epoch, around 50th generation.
Exception in thread "main" java.lang.IllegalStateException: java.util.concurrent.ExecutionException: java.lang.IllegalArgumentException: Too many seed candidates for specified population size.
    at org.uncommons.watchmaker.framework.islands.IslandEvolution.evolve(IslandEvolution.java:248)
    at pl.agh.edu.evolution.experiment.Experiment.main(Experiment.java:104)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:483)
    at com.intellij.rt.execution.application.AppMain.main(AppMain.java:140)
Caused by: java.util.concurrent.ExecutionException: java.lang.IllegalArgumentException: Too many seed candidates for specified population size.
    at java.util.concurrent.FutureTask.report(FutureTask.java:122)
    at java.util.concurrent.FutureTask.get(FutureTask.java:192)
    at org.uncommons.watchmaker.framework.islands.IslandEvolution.evolve(IslandEvolution.java:220)
    ... 6 more
Caused by: java.lang.IllegalArgumentException: Too many seed candidates for specified population size.
    at org.uncommons.watchmaker.framework.factories.AbstractCandidateFactory.generateInitialPopulation(AbstractCandidateFactory.java:66)
    at org.uncommons.watchmaker.framework.AbstractEvolutionEngine.evolvePopulation(AbstractEvolutionEngine.java:135)
    at org.uncommons.watchmaker.framework.islands.Epoch.call(Epoch.java:51)
    at org.uncommons.watchmaker.framework.islands.Epoch.call(Epoch.java:27)
    at java.util.concurrent.FutureTask.run(FutureTask.java:266)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    at java.lang.Thread.run(Thread.java:745)

I'm not setting seed candidates count or seeds candidates themselves so I guess error is not on my side. To be sure if it not dependent on elitism and migrations number I have lowered them to 5 and 3 respectively (as in user manual) but it has changed nothing. Still needed at least 120 objects in population to work.

Hi,
Can you share FloatGenotypeFactory code?

Sure. It's just class wrapping list of Doubles. AbstractGenotype contains nothing at all.
    public class FloatGenotype extends AbstractGenotype {

        private List<Double> genes;

        public FloatGenotype(final List<Double> genes) {
            super();
            this.genes = genes;
        }

        public FloatGenotype(final FloatGenotype genotype) {
            super();
            this.genes = genotype.getGenes();
        }

        public List<Double> getGenes() {
            return genes;
        }

        public void setGenes(final List<Double> genes) {
            this.genes = genes;
        }

        @Override
        public String toString() {
            String result = "[";
            for(Double f : getGenes()) {
                result += f + " ";
            }
            return result + "] ";
        }
    }

Hi,
It's FloatGenotype I was interested in FloatGenotypeFactory

My mistake.
public class FloatGenotypeFactory extends AbstractGenotypeFactory<FloatGenotype> {

    private static final Double UPPERBOUND = 1.0d;

    private static final Double LOWERBOUND = 0.0d;

    private static final Integer DIM = 3;

    private Integer dim;

    public FloatGenotypeFactory() {
        super(LOWERBOUND, UPPERBOUND);
        this.dim = DIM;
    }

    public FloatGenotypeFactory(final Double lowerbound, final Double upperbound, final Integer dim) {
        super(lowerbound, upperbound);
        this.dim = dim;
    }

    @Override
    public FloatGenotype generateRandomCandidate(final Random rng) {
        List<Double> genes = new ArrayList<>();
        for(int i=0; i<dim; i++) {
            genes.add(uniform(rng));
        }
        return new FloatGenotype(genes);
    }
}

public abstract class AbstractGenotypeFactory<T extends AbstractGenotype> extends AbstractCandidateFactory<T> {

    private Double lowerbound;

    private Double upperbound;

    public AbstractGenotypeFactory(final Double lowerbound, final Double upperbound) {
        this.lowerbound = lowerbound;
        this.upperbound = upperbound;
    }

    protected Double uniform(final Random rng) {
        return rng.nextDouble() * (this.upperbound - this.lowerbound) + this.lowerbound ;
    }
}

Hi,
I had similar problem but not same. In my case sometimes population size was less then elite count (even in configuration was not). I resolved it in my forked version, seems like it was top of iceberg, and framework for some reasons can keep population on fixed size. If you can provide me with your whole source code, I'll try to resolve the problem in my fork(as this repo is not changed for years)


Fix meshes location such that specifying the meshes prefix is not required

Refer to https://github.com/dic-iit/matlab-whole-body-simulator/pull/20#pullrequestreview-622637634 :

I may be wrong, but meshFilePrefix should not be necessary anymore since iDynTree 3, see https://github.com/robotology/idyntree/releases/tag/v3.0.0 and robotology/idyntree#817 .
... I have to set the parameter to some value, since it's a required input parameter of +iDynTreeWrappers/prepareVisualization

If the meshFilePrefix is an empty string, the default behaviour of iDynTree should be used. This was not working when the mesh file was a .dae. Instead, it works fine if the mesh type and extnsion is .stl. Fixed in #23 .


ddev: Failure with groupadd and single/double quotes on webImage

I've got a Windows Home 64bit and use the Docker Toolbox. I'm able to use the t3master TYPO3 ddev git, so generally ddev and docker seem to work.
Versions: Docker Client 19.03.1, Docker Engine 19.03.5. DDev:
DDEV-Local version      v1.13.1
commit                  v1.13.1
db                      drud/ddev-dbserver-mariadb-10.2:v1.13.0
dba                     phpmyadmin/phpmyadmin:5
ddev-ssh-agent          drud/ddev-ssh-agent:v1.13.0
docker                  19.03.5
docker type             Docker Toolbox
docker-compose          1.25.0-rc4
os                      windows
router                  drud/ddev-router:v1.13.0
web                     drud/ddev-webserver:v1.13.1

However, trying to run a different project fails. This is the (to me) simple ddev config:
APIVersion: v1.13.1
name: t3dummy
type: typo3
docroot: htdocs
php_version: "7.2"
webserver_type: apache-fpm
router_http_port: "80"
router_https_port: "443"
xdebug_enabled: true
additional_hostnames: []
additional_fqdns: []
mariadb_version: "10.1"
nfs_mount_enabled: false
provider: default
hooks:
  post-start:
  - exec: composer install
  - exec: ansible-playbook -i build/config/hosts build/build.yml --limit local
omit_containers: [dba]
use_dns_when_possible: true
timezone: Europe/Berlin

After running ddv start, I get this:
Service 'web' failed to build: The command '/bin/sh -c (groupadd --gid $gid "$username" || groupadd "$username" || true) && (useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' --uid $uid "$username" || useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' "$userna' returned a non-zero code: 2

To me this looks like there's a problem because the --comment option uses single quotes, where it should use double quotes. I am not familiar enough with docker/ddev to recognize where this is coming from.
I seem to not be able to manually edit the .ddev/.webimageBuild/Dockerfile to change those chars, as the file gets re-generated everytime dynamically. So my feeling is that ddev somehow injects this wrong internally?
(In a similar issue I found a note to the mariadb container issue; I tried using both 10.1 and 10.2 as mariadb_version, without change)

I copied your config and used it on WIn10 Home with ddev v1.13.1 and didn't have any problems.
You would never edit anything in .webimagebuild...  Is that related to what you're doing? You're right that it gets overwritten. Custom Dockerfile work is done in .ddev/web-build/Dockerfile, but don't do anything like that until you have this sorted out. Do you have any other customizations?
Try to see if there's anything else you might be doing
I did get a failure on the post-start hook doing ansible-playbook stuff. How are you doing to install ansible-playbook in the web container? I assume you haven't gotten that far yet.
Could you do a plain vanilla TYPO3 install and see what happens? Do it in another directory of course.  Just use https://ddev.readthedocs.io/en/latest/users/cli-usage/#typo3-quickstart
I suppose it's possible that you're out of allocated space in your Docker Toolbox installation, please make sure you have a big enough disk allocation, https://ddev.readthedocs.io/en/latest/users/docker_installation/#windows-installation-docker-toolbox
Also, Docker Desktop for Windows (edge) now works with WSL2 on Windows 10 Home (still requires Windows Insider builds, until the next release of Win10). It's wonderful. It's a whole different approach, but it's way faster and way more likely to succeed in general.
That said... I had no trouble.

Just a note, you said:

looks like there's a problem because the --comment option uses single quotes, where it should use double quotes

Note that quoting isn't a function of commands in Linux, it's a function of the shell, and in this situation there's interpolation of the environment variable $username, which is allowed by double quotes and not by single quotes.  That doesn't answer what's happening to you, but it might help you understand the context.

That's really odd. Maybe it is related to my initial problems - I had to start "ddev start" about 20 times until the packages were finally downloaded. It stalled all the time in one or another package, until it finally ran through (even though I had no network issues - but could also not see why the download was stalling; I was about to write a ticket when it finally magically went through). Is there a way to "validate" the downloaded packages or purge them to retry? Sadly I am not really docker-savy enough to know how.
I didn't edit those files before the problem was already there, so that can't be the initial trouble. The project I am trying to run is not from me, and there is a customized Dockerfile:
`
ARG BASE_IMAGE
FROM $BASE_IMAGE
RUN apt-get update
RUN apt-get install --yes --no-install-recommends apt-utils
php-ldap und php-yaml
für alle PHP Versionen
RUN for v in $PHP_VERSIONS; do apt-get -qq install --no-install-recommends --no-install-suggests -y $v-ldap $v-yaml; done
frontend tool 'frctl'
RUN npm i -g @frctl/fractal grunt
Tool für die Shell (ddev ssh)
RUN apt-get install -y mc
ansible für legacy build scripts
RUN apt install --yes ansible
wichtige hostnames
RUN echo -e "example.com ecdsa-sha2-nistp256 AAAAE2VjZHNhLXNoYTItbmlzdHAyNTYAAAAIbmlzdHAyNTYAAABBBGm4yXRX9081tbg4s0v3GaIyH0uCZhC1uBV6b63fDUDKG9FjZJ3ZRCgrL6hOjoM0Ef0wpi7T59PyEYlHlBqxZo4=" >> /home/.ssh/known_hosts
`
These run statements are all shown to be executed without errors, only after the last line the groupadd-command gets appended.
And yeah - I think I haven't gotten to the ansible part yet, but the playbock comes from a 'build' directory and would be executed after that (post-start hook).
I enlarged the docker-machine and now (since it probably purged all downloads) I get the same download problem where it stalls:
v1.13.1: Pulling from drud/ddev-webserver
6729c2f8f6ae: Pulling fs layer
9129eece9b41: Pulling fs layer
cf6182c65a22: Pulling fs layer
423b95031acb: Pulling fs layer
515e936a6c88: Pulling fs layer
ef7169731276: Pulling fs layer
2648b59f8244: Pulling fs layer
eed5c3f26701: Pulling fs layer
2648b59f8244: Downloading  5.947kB/497.4kB
30dad26f3f57: Waiting
7c82b5c9656d: Waiting
0f1acbb6475c: Waiting
6b908999bfec: Waiting

(About the quoting - I am really not sure what is happening there. Could you tell me how I could get the actuall command output returned, instead of just 'error-code2', is there a way for me to "log in" to the image and try to perform the command manually? Sorry for only grasping bits and pieces yet, I had hoped this was just a quoting problem of a code-template and not possibly a misconfiguration or corrupted image)
((Also looking forward a lot to the WSL2 integration!))

It's not stalled... it just takes some time. You might want to do it on faster internet. You can also docker pull drud/ddev-webserver:v1.13.1, and when that's done you won't have to wait for ddev start any more.

Well, it was running now unchanged for an hour after I wrote the last comment, and nothing changed. I have a 1GBit downstream (and was streaming video to download too without errors, so it can't be my network connection itself; only the bridging of VirtualBox [recent version] could be a culprit; using NDIS6, not NDIS5?).
Can I execute this command within the VM somehow (instead of within the docker shell) with some kind of debugging, so I could maybe try to wget the URL that is failing?

Your package install should be something like RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y -o Dpkg::Options::="--force-confnew" --no-install-recommends --no-install-suggests php7.2-yaml to be really successful. You can so all versions if you want, or just the PHP version you want.

Just do the docker pull. And when it's done ddev won't have to do it any more. If it's timing out, it's your internet connection or docker hub or something.

If I use docker pull within the VirtualBox VM I just get:
Using default tag: latest
Error response from daemon: pull access denied for drud/ddev-webserver-v1.13.1, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

so I assume it's not meant to be run there inside the VM?
I just performed a "wget" with a 2GB file within the VM and that downloaded without problems. I do think I can rule out my personal network, so maybe if it's docker hub, that could be. Do you know a way to wget a file from docker hub manually so I could verify that? Otherwise I'm sorry, please close the issue if it seems to stem from me not getting docker to run properly inside this poor Windows machine :(

Per https://github.com/drud/ddev/issues/2109#issuecomment-597692543 you want to do docker pull drud/ddev-webserver:v1.13.1 and you do that in the Docker Quickstart Terminal or git-bash.

YeahI did that, this one is stalling - so I thought I could see direct output inside the VM where this would be executed, because in the Docker Quickstart Terminal (run as root) the timeout seems to happen. Too bad I can't see the HTTP URLs it is trying to fetch so I could run network tests on the host machine.

Please don't ever run anything as root. ddev and docker are not designed to have nything to do with root.
If you're stalling, it's your internet (or perhaps your host computer). I just did docker pull drud/ddev-webserver:v1.13.1 on two different computers, including Win 10 Home, with no trouble. It is big. So if you have slow internet you will in fact have to wait.

So, I took the time to install docker and ddev on my Linux machine.
There, the download worked at least flawlessly (same network though, so the connectivity issue must be specific to the windows install - I installed the docker Toolbox without the old NDIS module and had trouble using the bridged connection until I googled around and disabled/enabled it, which did get the t3master to work but obviously creates other trouble.)
However, I actually get a similar error like on my Windows machine:
Step 19/19 : RUN (groupadd --gid $gid "$username" || groupadd "$username" || true) && (useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' --uid $uid "$username" || useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' "$username"  ---> Running in 836e7c03b563 /bin/sh: 1: Syntax error: end of file unexpected (expecting ")") ', stderr='Creating volume "t3dummy-010-mariadb" with default driver Building db Building web Service 'web' failed to build: The command '/bin/sh -c (groupadd --gid $gid "$username" || groupadd "$username" || true) && (useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' --uid $uid "$username" || useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' "$username"' returned a non-zero code: 2'
So in fact, there really is a ")" missing in the useradd statement. Actually, that ")" is also missing in the report from my windows machine.
Can you tell me from where this could come from? I mean ansible isn't getting run at this stage to interfer, right - is this from ddev or from a docker image, docker-composer or where?

It comes from a failure to execute the instructions in .ddev/.webimageBuild/Dockerfile. If you look at that file, you should see what's being executed.
When trying to debug something like this, the first step is to try to simplify. So you'd normally remove anything custom, then start adding the custom stuff in again.
I'd suggest that you at least rm -rf .ddev/webimageBuild and ddev start again. And remove your .ddev/web-build/Dockerfile.
But as I suggested above,

Could you do a plain vanilla TYPO3 install and see what happens? Do it in another directory of course. Just use https://ddev.readthedocs.io/en/latest/users/cli-usage/#typo3-quickstart

I think you'll find it works fine. Then you can take a look and find out what you've done with customizations on this one.
I built and started just fine with the .ddev/web-build/Dockerfile you shared in https://github.com/drud/ddev/issues/2109#issuecomment-597664443

Sorry, I misread you asking that. I did install vanilla, and it did work with that.
 You wouldn't believe it (well maybe you will). A specific comment in that file:
# für alle PHP Versionen

seems to contain a bad byte sequence (ISO? UTF-8? Not sure, didn't investigate). Once I removed that ü Umlaut, the build process went through.
It seems since the ")" was the LAST character of the file, the extra character messed up the string length of the file or so, thus not transferring the whole Dockerfile to the RUN command. When you copy/pasted it from here, that character might have been fixed due to recoding by Github or something like that.
So, my lesson learned: Only use ASCII charset in a Dockerfile.
Terribly sorry how much time I took away from you with this. But so many thanks for bearing with me and getting into this so deeply, you rock. This was a great learning curve for me and I am one step closer to finally using ddev productively :-D

Yay!
The actual rule is "Only use ASCII charset when coding".  Only pain can result otherwise. Data, different. But code, files, directories, don't use spaces, don't use UTF-8.  :)
So glad it's working!


Version 1.8

Signed-off-by: Yannick Schaus<EMAIL_ADDRESS>

@kaikreuzer:
openHAB-pebble-1.8.zip
Amazingly the cloudpebble build server still works :)
You should be able to publish this one to the store and rename the repository afterwards.

Amazingly the cloudpebble build server still works :)

Not surprising, see https://dev.fitbit.com/blog/2018-01-24-pebble-support/ - but this is only until June 30:
"Pebble devices and mobile apps will continue to work after June 30, 2018, but the following Pebble services will end: [...] CloudPebble development tool"

Ah, missed that, good to know!

Thanks, I have just published the release!
Could you maybe add the 1.7 and 1.8 releases to https://github.com/openhab/openhab.pebble/releases as well?


added python script.

This is a small python scripts using Turtle.

Fixes # <#394>

Domain of Contribution 📊

[ ✔] Python Projects

Checklist ✅

[ x] I follow Contributing Guidelines & Code of conduct of this project.

Screenshots / Gif (Optional) 📸

@ron2111 please check .

[Update] This is a beginner-friendly practice repository!! The merged PR is not counted towards Hacktoberfest 2021


Clean up byrefs formatting

Bad header for C#
Bad code sample

Is our CI down after the rename from VSTS to Azure Devops?

Closing and reopening to begin new build.

@cartermp, the PR validates now, if you'd like to merge.


Add wildcard selection to despawning items

I'm not certain if it's a thing, but being able to just define something like '@tags.trinkets:' with '*=forever', to make everything with a trinket tag have this despawn property applied would be very nice.
The reason this is a question and not feature request is because... it might already be a thing and I'm just not aware. Maybe I missed something in the config or it's not documented.

I have actually never used this feature.
but ye, the tags thing appears to exist and it can be used for this (assuming trinkets are a tagged item).

changed the issue to a feature request. probably won't get around to implementing it anytime soon. so don't wait for it


AssertionError... Again...

I really hope this issue gets resolved! What a pity that I do not understand how to solve this problem...
Traceback (most recent call last):
File "/opt/homebrew/bin/aspeak", line 8, in
sys.exit(main())
File "/opt/homebrew/lib/python3.9/site-packages/aspeak/cli/main.py", line 138, in main
speech = create_speech(args.locale, args.voice, audio_format)
File "/opt/homebrew/lib/python3.9/site-packages/aspeak/api/api.py", line 138, in init
super().init(locale, voice, audio_format, output)
File "/opt/homebrew/lib/python3.9/site-packages/aspeak/api/api.py", line 43, in init
self._config()
File "/opt/homebrew/lib/python3.9/site-packages/aspeak/api/api.py", line 51, in _config
assert token is not None
AssertionError

Closed as a duplicate of #46


Error with cargo lambda build on Windows

Hello,
I have tried following the readme on how to get up an running with this tool but I have hit this problem on two separate windows 10 computers. When I get to the lambda build command I get the below error.
PS C:\temp\test_fn> cargo lambda build --release
Error:
  × The prompt configuration is invalid: Available options can not be empty

I tried to run this with various other switches but I cannot get it to build. Other commands like cargo lambda watch were working correctly.
I tried to search to see if others had the issue but could not find any, apologies if it is already logged.
Thank you,
Adam

I am getting it on all terminals I can test, PowerShell 7, PowerShell 5 and cmd. Its strange, I have no idea what is passing the error to me. Not sure if I am missing some kind of prerequisite? I generated the project using cargo lambda new test_fn

Perfect, that was the issue. Once I installed Python3, I got a prompt:
PS C:\temp\test> cargo lambda build --release
? Zig is not installed in your system.

I had to do one more step which was add zig.exe to path.
And then it successfully built. I did not have python scoop or chocolatey installed when I got the bugged error message.
Is it not possible to cross compile by changing the rust linker? When I followed the AWS article: https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/building-custom-runtimes.html I was able to compile from windows using:
[target.x86_64-unknown-linux-musl]
linker = "rust-lld"

Nevertheless, thank you for your help!
Thanks,
Adam


Update mini-dashboard version

Checked with @mdubus, we would like to update mini-dashboard version to the latest one to get the last bug fixes.
⚠ @mdubus, you have to open a PR changing the mini-dashboard version pointing to release-v0.29.0, not to main

Closed by #2755, will be available in v0.29.0rc3 toda


private package support: pass 'authorization' header through on index/tar gets

Fixes #60
I recently started using npm private packages instead of sinopia, and I was sad that I had to give up offline npm install. Found this package, but realized that it didn't support private packages!
This is a very simple change, since npm uses a bearer token. We pipe that through to downstream requests to the npm registry. Because npm_lazy already proxies other requests, you can do an npm login like normal. :0)
Thanks for this package, by the way. Offline installs are important!

awesome! thanks for looking into this. I'll merge and publish a new version..

published as v1.11.0


phase 1: documentation

Need to formally document:

Test results, ensure results are the same and then the time it took and attempt to make sure the environment was similar.  Multiple test will compensate for a lack of a completely stable environment.  HOLD:  Waiting for system admin to open port 7077.
DONE. Processes for standing up a IPython notebook server.
DONE. How to initiate a SparkContext from Python, what works and what doesn't.  Try to document why.
DONE. The data set used, its structure and some meta data.

Have gotten spark to work on re-built clean system with my documentation.  I fixed, updated and added the documentation where there were holes in it.


Use SOX play command for more beautiful voice

Custom commands currently sound very metallic and not human. This pull request uses the SOX play command to play the generated .wav files. No filters are needed, the voice sounds much better.

Codecov Report

Merging #61 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #61   +/-   ##
======================================
  Coverage    79.2%   79.2%
======================================
  Files           5       5
  Lines         202     202
======================================
  Hits          160     160
  Misses         42      42

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 4ca5dbb...5043eb7. Read the comment docs.

@TimoCrafter, thanks for the PR!
I don't think we need to use sox, as the problem is more likely to be the filter than aplay. The filter was developed for a pre-production speaker (to avoid the exact problem it's causing now!) so maybe we need to change or remove it.
Could you compare these two alternatives on your speaker and let me know which sounds better?

Change FILTER_B = np.array([1.75161639, ... to FILTER_B = 0.63 * np.array([1.75161639, ....
Change player.play_bytes(eq_bytes, sample_rate=SAMPLE_RATE) to player.play_bytes(raw_bytes, sample_rate=SAMPLE_RATE).

2 should sound the same as your change with sox - let me know if it doesn't!

Using 0.63 * ... makes the voice sound more beautiful, but it still sounds better with SOX.
Without using a filter, it doesn't sound better.

Without using a filter, it doesn't sound better.

You mean that it sounds the same as originally? That's very surprising - can you double-check this?

It doesn't sound the same as originally, but it also sounds terrible. You can't say if better or worse, but different. Using SOX is definitely the most beautiful variant.

Sorry for taking so long to follow up on this - at the time I didn't have the right hardware to reproduce this.
For reference: I can confirm that it sounds much better with SoX. pico2wave is generating clipped audio files, which lead to horrible artifacts when they go through ALSA's rate conversion. SoX's rate conversion handles them much better. Additionally, there's no need for the equalization with production speakers.
I wish we could avoid pico2wave's clipping in the first place, but it's much easier to just use SoX.
@TimoCrafter, thanks for PR and testing the alternatives for me!

FYI: one can also select a better resampler for alsa if that is the problem (e.g. in /etc/asound.conf)
defaults.pcm.rate_converter "speexrate_medium"

For reference: I just tested ensonic's suggestion. All the speex converters have the same problem, although defaults.pcm.rate_converter "samplerate" does resolve the issue. For now, though, I'm happy to stick with SoX, as people will get it by pulling and don't have to reinstall asound.conf.
If we find any reason to drop the SoX dependency we can revisit.


double init ?

ynab_client init() is called in the main function, and also here anytime the webhook is called,
https://github.com/scottrobertson/fintech-to-ynab/blob/master/python/routes.py#L39
Either we init and for every webhook call (then the main call is not needed), or we init only once when the app is created, in main (then the call for each route is not needed)  ?

It should just be calling sync on at the start of each webhook.


infinite scroll in timline

when scrolling to bottom of timeline auto fetch next content

Consider the following lib:
https://infinite-scroll.com/options.html


Creating TCPSocket fails with getaddrinfo: Non-recoverable failure in name resolution (Socket::Error)

May be related to #2568?
Using the same code in that issue:
require "socket"
p TCPSocket.new("google.com", 80)
p TCPSocket.new("google.com", 80)

The resulting output is (for Crystal - it works with ruby code, so It's probably something related to libevent):
getaddrinfo: Non-recoverable failure in name resolution (Socket::Error)
[4316279] *CallStack::unwind:Array(Pointer(Void)) +87
[4316170] *CallStack#initialize<CallStack>:Array(Pointer(Void)) +10
[4316122] *CallStack::new:CallStack +42
[4368809] *Socket::Error +41
[4368745] *Socket::Error +105
[4363191] *TCPSocket#initialize<TCPSocket, String, Int32, Nil, Nil>:Nil +1655
[4361512] *TCPSocket::new<String, Int32, Nil, Nil>:TCPSocket +184
[4361323] *TCPSocket::new<String, Int32>:TCPSocket +27
[4293247] ???
[4301177] main +41
[140049360811841] __libc_start_main +241
[4291273] _start +41
[0] ???

I'm on ArchLinux, running Crystal version 0.17.3, and libevent version 2.0.22-2.

We should maybe revert the commit that introduces this "feature" (anti-feature, I'd say, because it's broken). Problem is, dns_timeout would be ignored for now, and of course things will be a bit slower. I'll check with @waj when he comes back, I prefer to have something that works but is slower than something fast but broken. Of course the best thing to do is to fix this (don't know how, maybe not using libevent), but that will take more time.

If we don't use libevent dns, then we're up to blocking Crystal altogether whenever we make a request... not just the current fiber, but everything, which can take from some hundred milliseconds to 5 seconds if unlucky :(
I looked around and there aren't any correct solution. Even getaddrinfo_a, (only available by glibc) uses signals instead of a couple of functions (start, poll).

I guess then we'd need to do what @waj said here

So, is this something that's only broken on Linux or is it an issue elsewhere as well? I'll have to take a closer look at the IPSocket code, but I'm not sure what's causing this, since the example libnotify code for a DNS lookup works in C.

Should we close all these tickets and create a more general meta ticket about implementing a proper resolver?

@jhass Please! And thank you for cleaning up some issues :-)

Closing in favor of #2660


Use tumbleweed artifacts for efi

Looks like only the tumbleweed shim is properly signed to allow
secureboot
Signed-off-by: Itxaka<EMAIL_ADDRESS>

This makes EFI work back again and secureboot as well.


Battery and Damange meters in Camera Drones

The battery and damage meters are too far on the edge of the hud when controlling the camera drones, making it nearly impossible to read.

+1


Improves the debugger icon

Hi, some fixes about the icon and presence of an empty button in debug bar when there is no query. I need to check if this don't break the debugger version 1.

Ok all is great with Symfony 2.7 / PHP 5.6 (still supported).

thanks create work! I have add request-method and the raw request-data to the table.


[important from KSITM]  Data Collection

We are going to launch a massive field data collection on camps. however, the volunteers going there may not have data in proper format (it may be in paper as well) can we have a provision to upload scanned images (of that paper) on to our system and have it available in backend (web) somewhere . I will have operations team in backend going through each such paper reports and convert them onto our format and enter

I am working on it

is this done? can we add emergency contact number/name and relation to the file

I need more information on this as we will be using Ushahidi for the data import / export etc. Also there is someone working on data export for 2 days.


[GC] Removed unreferenced event Q for summarizer and log unreferenced events only if node was not revived first

Reviewer guidance
This is targetting a test branch (test/gc-2.3). Since this has legacy alpha API breaking changes which are not allowed to be merged until v2.3, this is a workaround to get the changes reviewed and merged in a test branch. Later when breaking changes are allowed, these will get merged to main.
Description
In summarizer clients, unreferenced events (inactive, tombstone ready and sweep ready) are added to a queue and are logged after the next GC completes. This was done to reduce false positives in 2 scenarios:

A node is used after it is revived. Since the revive notifcation to GC was delayed until the object reviving was loaded, GC could not build correct timeline.
GC may get multiple reference notifications on object loads. GC was notified of references whenever an object with handles was loaded.

With the reference op work, both of these problems don't exist. So, we can remove the unreferenced event queue for summarizer clients. Also, we can check that an object is not revived before we log an event for it which will reduce false positives for non-summarizer clients too.
This PR makes 3 main changes:

It removes the unreferenced event queue for summarizers and logs unreferenced events right away.
It checks that GC did not see a reference for an unreferenced object before it was used. If it did, it does not log because the usage isn't unexpected.
Added packagePath and fromPackagePath properties to addedGCOutboundRoute. Since this information is already present with the runtime when it calls GC, it doesn't need to ask for it.

AB#8938

@microsoft/fluid-cr-api Note that this is against a test branch and not main.

As discussed with @markfields offline and per this comment https://github.com/microsoft/FluidFramework/pull/21961#discussion_r1689055623, closing this PR for now. This can regress behaviors and can be more costly than it's worth. We can revisit this later and remove the whole thing once GC is stable. We will only need tombstone and deleted logs then.


Support theme setting

Support theme setting? https://mermaid.js.org/config/theming.html

@LogicDancing hey, thanks for the suggestion!
This plugin does not control how Mermaid is rendered in any way. I suggest you post this on the official forums!


Docs with material-from-mkdocs

This PR setup the documentation using Material for MkDocs (easier to use and setup than Sphinx) and automatic building via GitHub Actions.
The API documentation will be added in a subsequent PR using Sphinx  which has more robust automatic documentation from docstrings, particularly for numpy-style.

This PR setup the documentation using Material for MkDocs (easier to use and setup than Sphinx) and automatic building via GitHub Actions.
The API documentation will be added in a subsequent PR using Sphinx which has more robust automatic documentation from docstrings, particularly for numpy-style.

I had once considered to migrate Mesa's documentation to MkDocs, but bailed out because of the auto documentation from docstrings. It should be possible to generate the Sphinx docs without RTD, by using GH Action, shouldn't it?


chore: Adjusting the page layout

Current State
Description of the current state/issue:The current layout looks too small, need to adjust the layout to a larger
Link to the page:https://apisix.apache.org/zh/showcase/
Screenshots (if relevant):

Desired State
Description of the change you are proposing:
Make the layout bigger
Would you like to fix this issue?
Yes! Count me in

Sry. Can you please tell me why we need this improvement? And what are your goals for improvement? I think this is something that needs to be discussed, rather than just starting to do it.

对不起。你能告诉我为什么我们需要这种改进吗？您的改进目标是什么？我认为这是需要讨论的事情，而不是刚开始做。

I made a change just now, do you see it like this?

I made a change just now, do you see it like this?

@juxianzheng I understand. But I think we need to make the logo bigger more than the layout. The larger layout does not make the layout look better.

@juxianzheng
As I recall there are three sections that need to be changed here

[ ] change the logo to a high-definition
[ ] the logo should be unified, for example, Tik-Tok Travel - the black background does not match with others
[ ] better adjust the page layout? Now it looks too small.

@guoqqqi Hi, for the first and second tasks, we already have the corresponding issues, it's #580.

@juxianzheng Pls create a PR for an issue if possible.


server: replace NextID with moov's base.ID() shared function

Fixes: https://github.com/moov-io/ach/issues/555

Codecov Report

Merging #571 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #571      +/-   ##
==========================================
- Coverage   90.46%   90.45%   -0.02%
==========================================
  Files          64       63       -1
  Lines        5086     5080       -6
==========================================
- Hits         4601     4595       -6
  Misses        351      351
  Partials      134      134


[CSL-346] Add genesis block creation logic to modern code

@gromak
:/

@gromak
I am decreasing estimation, because main part will be done in CSL-355.


#901: User Setting Not Working: Amount of Videos on Dashboard

fix initMedia method in dash-ctrl: currentUser.data always undefined, user`s data was in currentUser
fix amountOfMediaEntries parameter saving: add 'include' into bindData() in UserController

thanks for the contrib!


WAF: Add notice about Managed Rulesets running at most once per request

Addresses https://jira.cfops.it/browse/PCX-1912.

@rboulton-cf Updated notice: rewrote first paragraph and added a new one at the end.


help: limit to 75 chars print

This PR addresses issue https://github.com/odeke-em/drive/issues/247.
cc @indragiek

Hello @indragie, please get the latest from master.
2c2,4
< 	share - share files with specific emails giving the specified users specifies roles and permissions
---
> 	share - share files with specific emails giving the specified users
> specifies roles and permissions
>
3a6
>
5c8,9
< 	Specify the emails to share with as well as the message to send them on notification
---
> 	Specify the emails to share with as well as the message to send them on
> 	notification


Amend Russian translation

Task/Issue URL: https://app.asana.com/0/1125189844152671/1171372280578300
Tech Design URL:
CC:
Description:
Amends one onboarding string in Russian.

Internal references:
Software Engineering Expectations
Technical Design Template

is there anything that needs translate n2 Spanish

is there anything that needs translate n2 Spanish

Hey @DarklyMind1, gracias por preguntar!
You can have a look at https://github.com/duckduckgo/Android/blob/develop/TRANSLATIONS.md, we don't usually require new translations but are opened to improve the existing ones.

chido está bien gracias d todos modos para la otra entonces