Document custom Qodot Nodes and custom Qodot Resources

It may be worthwhile documenting the following for completeness(If only to point out for some that they are internal and not relevant to the majority of users). Documenting their fields, methods, signals and inheritance tree.
Resources

QodotFGDClass
QodotFGDBaseClass
QodotFGDPointClass
QodotFGDSolidClass
QodotFGDFile
QodotWorldspawnLayer
QuakeMapFile
QuakePaletteFile
QuakeWadFile

Nodes

QodotPlugin
QodotSpatial
QodotMap
QodotEntity
QodotLight
QodotRotateEntity

An example of this, is that for QodotMap, the signal outputs aren't documented(something I've used for knowing when to generate navmeshes at runtime after generating a map at runtime) and then using the verify_and_build() method available on QodotMap(should make more programmatic use of QodotMap easier to find out about).
As for the resources, the BaseClass, PointClass and SolidClass resources are already documented under the Entities section, but I don't know if we want exhaustive documentation on their fields there. It may be better to present the most useful fields there, and link to the complete reference for the resources.
In my head for this, I have in mind something akin to javadocs or the Godot Docs class Reference https://docs.godotengine.org/en/stable/classes/index.html
BaseClass, PointClass, SolidClass and QodotMap are the particular resources and nodes I imagine being something to prioritise with this ticket as these are the most frequently used in projects using Qodot. The others listed above, I'm aware they exist, but haven't used them or looked much into them(I expect they are mostly internally used).

This issue's been really useful, thanks for that.
I've been working my way through documenting the resources a bit better, but the nodes definitely could use something more concrete.
Should there be a separate class reference from the usage guide? I've been keeping them close together in hopes that everything about FGDs can be found on one page, similar for Entity Definitions, CFGs, etc.


Fix bug in ocgrpc client logging

Fix bug in ocgrpc client logging. Info is nil in the logging branch so info.FullMethodName will cause a crash.

Thanks for the merge @rghetia!


Better error reporting when no reader available

See 85

Wrong ticke it's the one about sql.Date

Should not return reader on object


Added Support for Journal Logs

Resolves #544

@petewall Kind nodes do not have systemd.  I think it might be possible to mount the container logs to /var/log/journal but they would be in docker json format and not journal format.  We would probably have to ingest those logs with fluentd or something like that, covert them and write them back out, it might work but I'm not certain of that and it seems like a lot of extra complexity.
Would it be better to spin up a cluster in GCP or AWS to test as part of CI/CD?  That is obviously a bigger task, but might be worthwhile as the chart and k8s integration can begin to collect different information depending on the provider.


Add smarts to quit if another instance of chron-ocs.sh is running

This will help to schedule the script - say - every hour, so, once the
chronjob completes, next attemt can be made during the next hour itself.
Signed-off-by: Sridhar Venkat<EMAIL_ADDRESS>

With this work, the cronjob can be designed as:
SHELL=/bin/bash
0 * * * * ~/chron-ocs.sh
This will make the script run every hour and quit if it is already running during the previous invocation. When chron-ocs.sh finishes, it can be restarted immediately within an hour.


Write package metadata to package object

It would be advantageous if package metadata such as category could be added to the package object.
This has some difficulties with the current Jamf API, since (as far as I know) it requires multiple requests to achieve, and there is no reliable feedback that the first POST request to upload the package was successful, in order to get the object id with which to PUT the other metadata to the object.

Category is now optionally written. I can incorporate other objects if required.


No rendering man_made=street_cabinet street_cabinet=power

If you're reporting an issue with rendering, please provide:
Screenshot
Link to where this issue occurs on OpenInfraMap
Link to the corresponding entity on OpenStreetMap

and name/ref power=transformer

I also suggest a streetcomplete quest to help to distinguish this tags and giving OIM more objects to show up

It should be said a proposal is currently under RFC and adjustments still required on solutions to be adopted, to deprecate street_cabinet=* as utility mark.
https://wiki.openstreetmap.org/wiki/Proposed_features/Utility_facilities
Any power/telecom/water/whatever should go under common utility=* key please.

Closing in favour of #157.


doc: Update matrix for 22.03

Updates from Alberto's version script.

Some updates after using NGC for more difficult
to discover values.

Until we can replace this matrix with automatically updated information in the Merlin repository, here's the 22.03 update.
https://nvidia-merlin.github.io/NVTabular/review/pr-1450/resources/support_matrix.html
I removed whatever was in the left-most column and added 22.03 on the right.  The version that I removed ought to be available from the version picker.

Looks great!  Thanks @mikemckiernan!


Release to OSSRH staging repository

[x] Uptake new Eclipse versions of dependencies (if any). Use GlassFish 5.1 Release Tracker wiki to find components versions to use.

[x] Release to OSSRH staging repository

[x] Update GlassFish 5.1 Release Tracker with released version number.

Done
https://oss.sonatype.org/content/groups/staging/jakarta/jws/jakarta.jws-api/1.1.1/


Update whats-new.md

The FSLogix whats new link isn't working

@TomHickling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@TomHickling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


torch.cuda.OutOfMemoryError

I'm getting this error when I try to run.

From log, it requests more VRAM , which could not be fixed here - ComfyUI custom node.
In fact, InstantMesh has plan to support low vram env, but not finish at this moment, see

https://github.com/TencentARC/InstantMesh,
Please keek an eye on its process,
I would close this issue


Newbi Question

Ask a question
A clear and concise description of what the question is.
I am making a Django app where my users might have to pay and get credits to send messages. I will require to modify a few codes in the current pinax-messages. Maybe I want to check the credit left field of my user model before a message is sent, maybe I want to modify the delete behavior of the models in the pinax-messages.
My question is how do I extend this app to make my own modifications? Should I download the code and start trimming unnecessary code and add in the required code?  but that doesn't seem the right way to do it.
Any Idea what is the best way to do it? should I fork the repository and then do the modification?
Sorry for this question it might seem very simple I am just starting to learn to program.
Warm regards,
-svb

found a thread


Basic support for splits in hover popovers

This change makes it so that:

Popovers are tracked with a weakmap so the Array hack isn't needed
All leaves are closed when a popover is closed (or the plugin unloaded)
Toggling of pin and minimize states is handled by the popover instead of a leaf
Closing a leaf doesn't close the popover unless it's the last leaf
Closing an active leaf activates the most-recently active leaf in the popover, or in the workspace root if the popover is closing

I've broken it into mutliple commits so you can review it piecemeal instead of in one giant gulp.
Here's what's not done, yet:

The pin icon is a bit wonky with multiple files; it moves to the most-recently opened leaf instead of staying in one spot.  I think it might work better if it were placed in the popover body at an absolute position to just overlay the first leaf's header rather than actually being placed in the leaf's header.  Some experimentation is required, but for now the behavior for a popover with just one leaf is the same as before this PR.
I haven't changed the menu filtering, though in my tests the split operations work fine.  Operations that link or use links between panes don't work properly, although they show the link icon in the panes and do the initial file loading just fine.  Popover leaves are also not seen as a valid target for linking.  These things need some investigation.
Minimizing with horizontal splits still produces ugly results

The intention at this point is this should be mergeable without affecting any single-pane functionality, just making improvements to the usability of multi-panes without making them a fully supported feature yet.

While checking this for any new leaks, I was noticing that Pane Relief is keeping leaf references like this:

How long will PR hold on to that leaf? Just wondering if that's something we need to consider with all of the leaf churn going on.

Pane Relief History objects reference their leaf, and vice-versa, but the only global storage is a WeakMap (domLeaves, as shown towards the bottom of your image).  It should go away when GC runs, if there are no other references.  (The same way as the popovers themselves should go away once GC runs after they're unmounted from the DOM.  If you want, you can grab the weakmaps into the console, where you can inspect their contents and make sure they're not getting larger over time and GC is working properly.


File action rename

added new file action rename:
screen-shot attached

@amitu resolved the conflicts - please merge now!

@gsalunke I was also resolving, so conflicts occurred again, resolved now 👍


add a prohibited emoji to the unicorn function

#413

Thanks for this! I'll test it out and hopefully we can get it live soon.

hey @opaldes this is great.  Did you want to pick this back up or have one of us run with it?  This will be a really nice improvement for the UI!

big thanks to @opaldes for getting this kicked off.  I ran with this in #426 because we didn't hear back for a bit.


Use exit code 2 for failures due to timeouts

Pulled from https://github.com/Shopify/kubernetes-deploy/issues/229 Different exit status for failures and timeouts. This is technically a breaking change, but probably not one that will impact many people.

As far as I can tell every piece of software defines their own exit status numbers:

The timeout command uses exit code 124 for a timeout
Curl uses 28
wget doesn't even bother having a specific timeout exit status

Given the lack of standardization, I'd prefer not to cargo cult something.
Technically this is a breaking change, but not one that makes me thing we need to bump to 1.0 immediately.  (a) we're pre 1.0 so anything goes and (b) 2 is still a failing exit code and I can't image people have exit_status == 1 instead of > 0
Tests coming soon.

@KnVerey This is ready for 👀 again. I like to test the executables, but I think they need to be re factored to make that easy and that should go into a different PR

@klautcomputing Thanks for the review. Adding an error class was a good idea. I've made the changes you suggested with the exception of the one that broke 20 tests. I'd be open to refactor that in a separate PR after this, but think its too much churn for 1 pr.

Changes are in, part of me is worried this has made things worse and not better, but maybe I've just spent too much time staring at this code today.


Setting db provider url at runtime

First, thank you for all of your help!
I am trying to understand the best way to integrate prisma into my project. Right now I have a separate repo with the schema and built clients. This allows different services to import the clients (typescript/golang). One thing I cant figure out is if it is possible to set the provider url at runtime. So I can switch between dev/prod envs. Is this possible and a reasonable approach? Or should I generate a new client from the schema for each package/env?

We plan to build more documentation and tooling around these kind of setups, but we're not quite there yet.
For your use case, I think you can just set everything via environment variables in the prisma schema:
datasource db {
	provider = "postgresql"
	url      = env("DATABASE")
}

generator db {
	provider = env("PROVIDER")
	output = env("OUTPUT")
	package = env("PACKAGE")
}

You can provide both the database connection string for different environments, and you can decide what kind of client gets generated. Does that work for you?

Are these env vars read at generation time or at runtime? I think it is generation time right? So I would need to generate separate clients for each environment?

That is indeed a bit confusing, but they are read at generation time, except for the datasource.url key, which is read at runtime. I think it would work exactly as you want, with a ts and go client and then you can provide the database url at runtime.

I just discussed this internally. In general, we don't support these kind of architecture yet, and we plan to provide official solutions at some point, though that may take quite a while. Feel free to use the env var solution above for now, but when we have an official solution for this it will probably look a lot different, so it'll be a breaking change for you.

Hmm im not sure the env var approach is working. I built with
datasource db {
	provider = "postgresql"
	url      = env("DATABASE")
}

then generate client, import into server app, set env var
export DATABASE_URL=...

but get this error:
{"is_panic":false,"message":"\u001b[1;91merror\u001b[0m: \u001b[1mExpected a String value, but received literal value \"DATABASE_URL\".\u001b[0m\n  \u001b[1;94m-->\u001b[0m  \u001b[4mschema.prisma:3\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\u001b[1;94m 2 | \u001b[0m    provider = \"postgres\"\n\u001b[1;94m 3 | \u001b[0m    url      = env(\u001b[1;91mDATABASE_URL\u001b[0m)\n\u001b[1;94m   | \u001b[0m\n\nValidation Error Count: 1","meta":{"full_error":"\u001b[1;91merror\u001b[0m: \u001b[1mExpected a String value, but received literal value \"DATABASE_URL\".\u001b[0m\n  \u001b[1;94m-->\u001b[0m  \u001b[4mschema.prisma:3\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\u001b[1;94m 2 | \u001b[0m    provider = \"postgres\"\n\u001b[1;94m 3 | \u001b[0m    url      = env(\u001b[1;91mDATABASE_URL\u001b[0m)\n\u001b[1;94m   | \u001b[0m\n\nValidation Error Count: 1"},"error_code":"P1012"}

In your example you set DATABASE_URL, but the env var is called DATABASE. Can you confirm that this happens even when setting the correct env var?

Sorry, that was a mistake in copying the code. The datasource url was DATABASE_URL when I received this error. So it wasn't caused by a mismatch

Are you sure you wrote url = env("DATABASE_URL")? From the error message it looked like you omitted the quotes.

Related to this, I am using this workaround with config like so:
generator client {
  provider = env("PRISMA_PROVIDER")
  output   = env("PRISMA_OUTPUT")
  package  = env("PRISMA_PACKAGE")
}

The output directory seems to have been interpreted correctly, but the generated Go file had package env.

@sejr You're right, it seems for custom attributes this somehow doesn't work. You could just hardcode that one, and if you're using the js generator that value will be ignored, but the go generator will read it.

What if I want to do that
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = env("DATABASE_PROVIDER")
  url      = env("DATABASE_URL")
}

I have a use case for this as well. I'm trying to use testcontainers with prisma, but the port exposed by the container is only known at runtime: https://www.testcontainers.org/features/networking/#exposing-container-ports-to-the-host

Hi I was reading the old comments. I made some tests and I arrived to the next conclusions:

Datasource schema properties provider and url allow environment variables usage
Generator schema properties provider and output allow environment variables usage but not package. For the moment you can solve the package problem hardcoding it.

In this point I'm not sure if the title of the issue is correct (sorry if I'm missing something).
What we need is to allow the usage of environment variables for all properties of generator and datasource, isn't it?

Yeah, thanks for the comment @CerberusVT. I'll close this issue and open new issues for clarity:

#664
#665

Might be relevant - just spent ~30m debugging another unexplainable DATABASE_URL bug
The cause?
DATABASE_URL='mysql://xxxxxxxxxxxxxxxxxxxx:pscale_pw_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx@aws.connect.psdb.cloud/db?sslaccept=strict'
	!=
DATABASE_URL=mysql://xxxxxxxxxxxxxxxxxxxx:pscale_pw_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx@aws.connect.psdb.cloud/db?sslaccept=strict

@emivespa do you mind opening a new issue or discussing for this please, it doesn't seem to be related

You're right, sorry for the noise. Feel free to delete to keep the convo clean, will make my own issue (only if I consider in scope for the repo)


patch: followup dependency cleanup

What type of PR is this?
cleanup

Which issue does this PR fix:
What does this PR do / Why do we need it:
Follow-up on the dependency cleanup, created a fresh lockfile to assure latest usable package versions are used
If an issue # is not available please add repro steps and logs from IPAMD/CNI showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this change require updates to the CNI daemonset config files to work?:

Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Dependency update in this PR currently breaks implementation of the pull image widget.
This is fixed in #340 with the pull image widget redesign.
Only merge this PR after #340 is merged


Add accepts types method

I created this PR as a potential solution for https://github.com/spatie/laravel-event-sourcing/issues/341
This PR adds an acceptsTypes function to the Handlers class that takes an array of strings and filters methods that match the provided types. This should add the flexibility needed in the event-sourcing library to ignore functions with a single untyped or mixed argument.

Looking good, thanks!


Enable Dictionary lookup in OS X

On OS X, the standard Dictionary service does not work (usually set to ctrl + option + D or three-finger-tap). While there is a package that provides a workaround (https://atom.io/packages/dictionary), this should not be necessary, and the system-provided version of this is much less intrusive.

Thanks for the suggestion! This looks like a duplicate of https://github.com/atom/atom/issues/5967 – let's keep the discussion there.

If it is because of -webkit-user-select: none; property in the editor-contents and editor-contents--private selector(As mathiasvr has metioned)? If so, where can I change it by myself?  Or I just need edit the source code and compile it ?


Use AzurePipelinesCredential in spring live test

Description
Use AzurePipelinesCredential in live test
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - spring - tests

API change check
API changes are not detected in this pull request.

/azp run java - spring - tests

/azp run java - spring - tests

/azp run java - spring - tests


Is the rewards of "E1" or "E2" in the step function designed incorrectly?

The paper [1] said "if it steps on one of the two enemy spaces, labeled E1 and E2, with a 10% probability it will be attacked, receiving a penalty and resetting to the home space, losing all it carries." That means if the agent encounters the "E1" or "E2", at the probability of 10%, "self.current_pos" should be self.initial_pos, and the "vec_reward" should be [-1,0,0].
But in function Step(),
elif cell == "E1" or cell == "E2":
if np.random.random() < 0.1:
vec_reward[0] = -1.0
done = True
the "self.current_pos" and the "vec_reward[1]" and the "vec_reward[2]" are not adjusted.
[1] L. Barrett and S. Narayanan, “Learning all optimal policies with multiple criteria,” in Proceedings of the 25th international conference on Machine learning - ICML ’08, Helsinki, Finland, 2008, pp. 41–47. doi: 10.1145/1390156.1390162.

Hi @HONG-ZI ,
vec_reward[1] and vec_reward[2] are already 0 because of their initialization in line 147.
self.current_pos is reset on the reset() method, which should be called next because done=True


Migrate to PyOptInterface + Quadratics + Benchmarks

Motivation :tada:
As discussed internally, this changes brings some important improvements:

[ ] De facto support of more solvers through the PyOptInterface API!
[x] ~2x speed-up since we avoid the slow .lp writing/parsing (based on the facility benchmarks
[x] Lower code area while maintaining functionality (deleted io.py, io_mappers.py, solvers.py, and user_defined.py !)

Closes #57 . Thanks to @metab0t for the powerful PyOptInterface library, and the helpful responses.
Breaking changes ⚠ ⚠

Expression.value is replaced with Expression.evaluate() to highlight that this involves a computation (note that Objective.value remains and queries the solver directly).
Renamed Config.print_float_precision to Config.str_float_precision.
Constraint.relax(...) must be called before the constraint is added to the model.
Model.to_file(...) no longer exists. Use Model.write(...) instead (this directly calls Gurobi's write function). If writing an .lp file, you might optionally want to set Model(..., use_var_names=True, ...) when first creating the model in order to transfer the variable names over.
The signature of model.solve(...) is reduced. For example, solution_file is no longer a valid parameter since .write() can be used instead.

Todo

[ ] Fix quadratic code
[ ] Add proper testing of quadratic example
[ ] Remove old to_str code
[ ] Fix failing tests
[ ] Fix duals!
[ ] Add documentation
[ ] List breaking changes
[ ] Testing of other solvers

I'm going to split this up into separate PRs. See #79 and #77


Add a both_endpoints_in command

By request of @lpetit, add a command to select tracts with both endpoints in the same region.

Added in last commit to the master: 50ebe440a653d6185feb8555ecb600d9e76fc20b .
For instance
cortico_cortical_frontal.left = both_endpoints_in(frontal.left)
will produce the tracts with both endpoints in the left frontal lobe

Added in last commit to the master the command. For instance
cortico_cortical_frontal.left = both_endpoints_in(frontal.left)
will produce the tracts with both endpoints in the left frontal lobe


Import as ES6 "javascript" module not working

When importing the ClipboardJS as ES6 module like this:
import ClipboardJS from "/mjs/vendors/clipboard.js";

I had the following error:
Uncaught TypeError: Cannot set property 'ClipboardJS' of undefined
    at webpackUniversalModuleDefinition (clipboard.js:15)
    at clipboard.js:16

Or when I tried with this:
import * as ClipboardJS from "/mjs/vendors/clipboard.js";

I had the following error:
clipboard.js:15 Uncaught TypeError: Cannot set property 'ClipboardJS' of undefined
    at webpackUniversalModuleDefinition (clipboard.js:15)
    at clipboard.js:16

Expected behaviour
When using javascript modules and try to import an JS module it should import an exported function/class/property. But the ClipboardJS is not been loading.
Exemple, to export something in EcmaScript 6 Module it must be like this:
export default = function foo () {}

OR
export function foo () {}

More info here: https://developer.mozilla.org/pt-BR/docs/Web/JavaScript/Reference/Statements/export

Actual behaviour
Loading errors
Browsers affected
I tested on Chrome and Firefox

I image and try these, but it doesn't work and without any error.
import ClipboardJS from "clipboard"
window.ClipboardJS = ClipboardJS


feat: add validation for empty options while setting two factor auth

Adding validation to the form while setting up the Two Factor Authentication, if the user doesn't select any option, was returning an error.

What do you think of this approach? Defining a default value from the beginning avoiding the null value error?

What do you think of this approach? Defining a default value from the beginning avoiding the null value error?

Excuse me for the late response, I was on a holiday. I will check it asap.

Is it possible for you to fix the PHPStan errors? The solutions seems good to me @lucascnunes

Seems like these errors are still happening @lucascnunes:

Error: Unable to resolve the template type TKey in call to function collect
Error: Unable to resolve the template type TValue in call to function collect
Error: Ternary operator condition is always true.
Error: Process completed with exit code 1.

I'm having difficulties to test it locally,
When I run locally it gives to errors, what am I missing?

Finally! 😅
I've never used PHPStan, have no idea what type of error was that.
If it's all correct, I'm glad I could help a little bit, thanks for the opportunity.

Thanks! I will merge the PR


Add [cassette/video]-tape icons

Alts:

How about rotating the cassette 180º?

How about rotating the cassette 180º?

@jguddas See updated comparison above… Although I think cassette is more recognisable as it is—I’m pretty sure it usually shows up in it’s current orientation… Also helps distinguish it from VHS.

I prefer the one in the second row second column.
Not sure about the videotape icon, it feels a bit cramped to me.

I prefer the one in the second row second column.

Aiming to also be consistent with https://lucide.dev/icon/voicemail here…

Is it still planned to merge this? I personally would need these icons :p

Is it still planned to merge this? I personally would need these icons :p

@Unkn0wnCat That’s up to @ericfennis


Single-site Metropolis-Hastings

I've added a simple Metropolis-Hastings implementation, and I'd like to merge in after feedback/review.
Currently passing 1 test, but I'm open to suggestions on others.

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

@neerajprad Do we need a new file tests/infer/mcmc/conftest.py, or will the new test tests/infer/mcmc/test_mh.py pick up the parent config file tests/infer/conftest.py?

@neerajprad Do we need a new file tests/infer/mcmc/conftest.py, or will the new test tests/infer/mcmc/test_mh.py pick up the parent config file tests/infer/conftest.py?

It should pick up the config from the parent.

i'll try to take a look at the details sometime soon, but i'd also like to see some of the MH tests from webppl pulled in. especially the ones that exercise trans-dimensional / structure change issues, since these are the subtlest to get right in the fw/bw computation.


jackett: 0.20.2325 -> 0.20.2352

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jackett is: "API Support for your favorite torrent trackers"
meta.homepage for jackett is: "https://github.com/Jackett/Jackett/"
meta.changelog for jackett is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.20.2326 with grep in /nix/store/5n360w5chfl777wl3bg8r77dcwglaxwa-jackett-0.20.2326
found 0.20.2326 in filename of file in /nix/store/5n360w5chfl777wl3bg8r77dcwglaxwa-jackett-0.20.2326

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jackett

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5n360w5chfl777wl3bg8r77dcwglaxwa-jackett-0.20.2326
ls -la /nix/store/5n360w5chfl777wl3bg8r77dcwglaxwa-jackett-0.20.2326/bin

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

1 package built:

jackett

Maintainer pings
cc @edwtjo @nyanloutre @purcell for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jackett is: "API Support for your favorite torrent trackers"
meta.homepage for jackett is: "https://github.com/Jackett/Jackett/"
meta.changelog for jackett is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.20.2352 with grep in /nix/store/xd6adcadcxzc06x6bk265vdgiwv4ljs5-jackett-0.20.2352
found 0.20.2352 in filename of file in /nix/store/xd6adcadcxzc06x6bk265vdgiwv4ljs5-jackett-0.20.2352

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jackett

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

jackett

Maintainer pings
cc @edwtjo @nyanloutre @purcell for testing.


logging: use logrus fields

There are many cases in the Docker daemon that could benefit from using fields instead of the printf-family of functions. Fields are one of the major reasons I built Logrus, after being frustrated again and again in production with poor logs that could've greatly benefited from including from more contextual information. Some of the benefits in my mind:

Strong mental model. When I write production code nowadays and write a log statement, I always start with a WithFields and ask myself: If I had to debug this error, what would be the most useful information to include? I didn't find the mental model to be as strong with printf-style functions
Easier to glance. Look below for an example of this. Another example is debugging a value being passed around many places, it's easy to see where it gets changed when the fields are sorted.
More modular.  In the future as we allow people to customize the daemon logging more (e.g. the driver work currently going on), fields can also help provide more flexibility for those drivers.
Log indexers. Fields work a lot better with logging indexers (e.g. Splunk, Logstash, etc.) as the key=value format is extremely common and parsed by default by most indexers. Currently in the Docker source code mostly : is used, which is tougher to parse and most indexers don't do it by default.

There are many hundreds of cases in Docker of printf-style logging that could be replaced with fields, and this PR doesn't do much. It's simply to discuss the issue of whether we should use more field-based logging in Docker (with some example code).
Before

After

Downsides

Performance. With an extreme amount of debugging level logging, allocating a map for a subset of those logging statements can mean a performance hit (mostly from GC). I don't think this will be an issue in Docker. If we ever see it becoming an issue, a new Logrus API can be added (and may be anyway) where WithFields takes arguments in pairs instead of allocating a map.
Coupling with Logrus. This couples Docker's logging more with Logrus. Regardless, I think there's value in creating a culture in the code-base of adding fields and if another logger is chosen in the future it should be possible to script a complete transformation in the code base if the API is different. The hard work is not the API, the work is in decoupling the fields from the messages.

If we agree that this is a direction to move forward, I'll follow this PR up converting more of the existing examples that benefit from this into field-based logging.
@crosbymichael @jfrazelle @LK4D4 @tiborvass

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "logrus-fields" git@github.com:Sirupsen/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

Signed and amended.. arrgh, my alias hadn't propagated. :(

Big +1

On Mar 21, 2015, at 8:00 PM, Simon Eskildsen<EMAIL_ADDRESS>wrote:
Signed and amended.. arrgh, my alias hadn't propagated. :(
—
Reply to this email directly or view it on GitHub https://github.com/docker/docker/pull/11554#issuecomment-84508430.

cooool!

I like it

See also #8948

I'm +1 on this. @LK4D4 @icecrime @crosbymichael let's decide and move forward.

@tiborvass I'm +1 too, so I changing this for code-review
@Sirupsen could you rebase, so we'll merge it as first step to fields logging.

@LK4D4 done

@Sirupsen Thanks!
LGTM

@LK4D4 don't you want him to update more parts like he offered in the description?

@tiborvass Let's start from compile fail :) ping @Sirupsen
No, I think we should merge this as first step. And we can create some issues for contributors with this as example.

Doesn't build :-( Ping @Sirupsen!

:cry:

Got bit by the log => logrus change, should be fixed :)
Agree @LK4D4. This was more of a proposal.

oh sheeeit rebase from hell sorry

I approve!

That test does not seem like my fault :(

oh its not dont worry

LGTM

LGTM


Pact JVM JUnit Provider doesn't fail when States are missing

When tests are run that require a provider state using the JUnit provider, the test proceeds without setting a state if a state annotation is not found.
The test should instead fail with an error message informing the user that there is no annotation containing the given provider state.

The tests will now fail with something similar to the following error:
MissingStateChangeMethod: Did not find a test class method annotated with @State("SomeProviderState")

In my PACT, there are 8 interactions. But at provider I am changing the state of two interactions. Those two tests passes, but remaining 6 failed with following error.
MissingStateChangeMethod: Did not find a test class method annotated with @State
I am using 3.3.3 and this was working with 3.3.1

This is now the default behaviour. You need to define a state change method for all the required states.

It will make the tests with empty @State method.
I think thats not right ! It would be nice if we keep the state change at provider as optional.

The problem is if someone forgets to add a state change, the test may fail in some obscure way and make take a while to diagnose. This change forces you to consider all the state changes that are defined in the pact file.
I agree that it may lead  some empty methods, but I feel in this case it is worth it. This brings it in line with the behaviour from the Ruby version.
If there is a lot of empty state change methods, it might be better to discuss removing those from the consumer tests.

This is good to close on my end... thanks again @uglyog!


fix(mm, ds): fix dupe alloc killing

Fixes RVTEE-585
Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1125
#1124  👈
#1123
#1122
#1119
#1118
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on  Graphite


YARN-6457  use existing conf object as a resource for sslConf object in WebApps for the builder to use in HttpServer2

use the passed conf object as a resource in local sslConf so it can be overridden

https://issues.apache.org/jira/browse/YARN-6457 has been fixed. Closing.


Fixed few more hive dependency issues

Abhishek,
Fixed a few more dependencies with the hive ddl's. I will schedule a functional run. Can you review them in the mean time.
Rahul

Looks good. +1
I scheduled a run with Hive 0.13 and that went thru fine.
On Friday, January 22, 2016, Rahul Challapalli<EMAIL_ADDRESS>wrote:

Abhishek,
Fixed a few more dependencies with the hive ddl's. I will schedule a
functional run. Can you review them in the mean time.
Rahul
You can view, comment on, or merge this pull request online at:
https://github.com/mapr/drill-test-framework/pull/86
Commit Summary

Fixed few more hive dependency issues

File Changes

M framework/resources/Datasources/hive_storage/avro_nullable.q
https://github.com/mapr/drill-test-framework/pull/86/files#diff-0
(37)
M framework/resources/Datasources/hive_storage/hive_storage.ddl
https://github.com/mapr/drill-test-framework/pull/86/files#diff-1
(22)

Patch Links:

https://github.com/mapr/drill-test-framework/pull/86.patch
https://github.com/mapr/drill-test-framework/pull/86.diff

—
Reply to this email directly or view it on GitHub
https://github.com/mapr/drill-test-framework/pull/86.


IGAPP-951: Add filtering to shelter offers

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.
Added minBeds & pets filter
http://localhost:9000/augsburg/de/offers/wohnraum

Tested with chrome. Would be nice if you could add a test for filtering though. Especially as currently there aren't any shelters created in the cms. Or am I missing something 🙈

we gonna create the tiles later. use link above. we won't add test, since this is only temp implementation

Tested with chrome. Would be nice if you could add a test for filtering though. Especially as currently there aren't any shelters created in the cms. Or am I missing something see_no_evil

we gonna create the tiles later. use link above. we won't add test, since this is only temp implementation

Wohnraum offer just not activated in the test cms yet. Should be available shortly though after Sven has activated it. Link above should still work.

Tested with chrome. Would be nice if you could add a test for filtering though. Especially as currently there aren't any shelters created in the cms. Or am I missing something see_no_evil

@Taggotty offer tile should now also be available in test cms.


Display days looks strange for dates in Febuary

countdown(Date.UTC(2018,1,12,20,45)) will produce something like 11 months, 27 days, ... at this particular moment. It will roll over to 1 year when the date rolls over. I believe that in this instance the days from the previous month should be used, not the current one. Very specifically, I would like to see the following unit tests added:
test('February day of month', function() {
	var input = countdown(Date.UTC(2018,1,6,20,45),Date.UTC(2019,1,5,20,45));
	var expected = '11 months and 30 days';
	var actual = input.toString();
	countdown.resetFormat();
	same(actual, expected, '');
});

test('January day of month', function() {
	var input = countdown(Date.UTC(2018,0,6,20,45),Date.UTC(2019,0,5,20,45));
	var expected = '11 months and 30 days';
	var actual = input.toString();
	countdown.resetFormat();
	same(actual, expected, '');
});

test('December day of month', function() {
	var input = countdown(Date.UTC(2018,11,6,20,45),Date.UTC(2019,11,5,20,45));
	var expected = '11 months and 29 days';
	var actual = input.toString();
	countdown.resetFormat();
	same(actual, expected, '');
});

test('February 28 Jan 31 of month', function() {
	var input = countdown(Date.UTC(2018,1,28,20,45),Date.UTC(2019,0,31,20,45));
	var expected = '11 months and 3 days';
	var actual = input.toString();
	countdown.resetFormat();
	same(actual, expected, '');
});

test('February 28  Feb 1 day of month', function() {
	var input = countdown(Date.UTC(2018,1,28,20,45),Date.UTC(2019,1,1,20,45));
	var expected = '11 months and 4 days';
	var actual = input.toString();
	countdown.resetFormat();
	same(actual, expected, '');
});

Currently only the Feb 28 Jan 31 and January test pass, the others fail. This isn't a trivial problem to fix, from what I have been able to do so far, but I will look in to it myself as well.

I've been thinking really hard about this, and there isn't an easy solution. Part of the problem is it is pretty arbitrary. In my opinion, the number of days in a month should be defined as the number of days since the last time the day of the month lined up.  If the start date is Feb 14, 2018, and the end date is Feb 7, 2019, the last time the end date had the same day of the month was Jan 14. That is X months and 24 days away, thus that should be the answer. The logic is a bit less clear if the next month doesn't have that day of the month, as in the end date being March 7, 2019, start date being Jan 31, 2018. February doesn't have 31 days, so this doesn't quite work, but I would could back to the last day of February, so X months and 7 days.


Title: Improving UI of Currency Converter

Which pre-existing extension will you be working on?
Currency converter
Describe your enhancement/improvement
Improving UI
Add any other context or screenshots about the enhancement here.
No response

@whiteHatpro, would like to work under JWOC :)


Setting Text in Collapsible Adds to text instead of replacing

Using the .setText() method of a QCollapsible concatenates the entered text with the current text.
If this is the expected behaviour, the docs should be updated to reflect this.

thanks!  that looks like a vestige of an older design that was updated in #140... fixing in #204


There are precision errors compared with flash_attn_2_cuda.varlen_fwd

There are precision errors compared with flash_attn_2_cuda.varlen_fwd when I use flashinfer.single_prefill_with_kv_cache function to run cohere_plus model,  below is the code I used:
fi_fwd_out = flashinfer.single_prefill_with_kv_cache(q.contiguous(), k.contiguous(), v.contiguous(), causal=True, sm_scale=softmax_scale, allow_fp16_qk_reduction=False)
fa2_fwd_out = flash_attn_2_cuda.varlen_fwd(
q,
k,
v,
out,
cu_seqlens,
cu_seqlens,
max_s,
max_s,
0.0,
softmax_scale,
False,
True,
False,
None,
)
torch.allclose(fi_fwd_out, fa2_fwd_out, rtol=1e-3, atol=1e-3)
It is worth noting that the first half of the layers are same, but second half are different.
can you give an official example code for precision comparison with flash_attn_2_cuda.varlen_fwd? Thanks!

It seems the problem with the accuracy error is that flashinfer.single_prefill_with_kv_cache doesn't support cu_seqlens_q and cu_seqlens_k, If I want to use flashinfer's prefill function， How to call it like flash_attn_2_cuda.varlen_fwd?

@yzh119 thanks very much, I will try it.


Duplicate key value constraint problems

Hi,
One one of our clusters we're fighting with integrity constraint errors:
"level=warn ts=2018-11-13T16:00:00.390643264Z caller=main.go:274 msg="Error sending samples to remote storage" err="pq: duplicate key value violates unique constraint "metrics_labels_metric_name_labels_key"" storage=PostgreSQL num_samples=100
"
This causes all our prometheus-postgresql-adapter pods to fail, is there anything we can do to solve this? We are using the latest master image.

Maybe this PR would fix it? https://github.com/timescale/prometheus-postgresql-adapter/pull/47

@thecodeassassin Please make sure to use the image tagged with :latest or :version. :master tag is discontinued as it's non standard.

We will investigate


Input checkbox broken when initiating with localstorage

Vue.js version
1.0.17
Reproduction Link
http://runjs.cn/code/gl3lbezs
Steps to reproduce

Open link.
Click checkbox to disable
Refresh page

What is Expected?

Shows unchecked checkbox and false label

What is actually happening?

Shows checked checkbox and false label

Very strange behavior.
In the console myV.$data.check outputs false. Running myV.$data.check=myV.$data.check does nothing, but running myV.$data.check=false unchecks the checkbox.

Please use correct localStorage API....

How is the localStorage API wrong?

The type of the localStorage.check is string,not boolean.

Thank you, I fixed it by checking localStorage.check=="true"


Upgrade Embree to the latest official release (3.13.0).

Since Embree v3.13.0 supports AARCH64, switch back to the official repo instead of using Embree-aarch64.
thirdparty/embree/patches/godot-changes.patch should now contain an accurate diff of the changes done to the library.
This should also fix #48822, could be nice to do a test build before merging.

Thanks!

I'll do a dedicated PR and test build of a 3.x cherry pick to confirm that mingw Windows builds are ok.


feat: Added Ollama engine via OpenAI api

Since Ollama is compatible with OpenAI API, I thought we could easily integrate Ollama as a model backend.
Usage
The usage format is get_engine("ollama-full_model_string"), for example.
OLLAMA_MODEL_STRING = "gemma2:9b-instruct-fp16" #Any Ollama model string you have on device
engine = get_engine("ollama-"+OLLAMA_MODEL_STRING)

Changed Files

textgrad/engine/init.py
textgrad/engine/openai.py

Thanks so much for this!
I'll review more thoroughly this evening but a first though: would it make sense to create a new class (Ollama) that inherits from the current ChatGPT implementation?
Maybe having the two separated would make future updates a bit easier (or more difficult, if they don't propagate).

Thanks so much for this!
I'll review more thoroughly this evening but a first though: would it make sense to create a new class (Ollama) that inherits from the current ChatGPT implementation?
Maybe having the two separated would make future updates a bit easier (or more difficult, if they don't propagate).

I think both options are fine. If we're going for a separate file, separate class etc. then we could also just use the Ollama Python Library, where it would have a cleaner API for initiating the client than the OpenAI workaround we have right now.
What we can do, if we like the usage api get_engine("ollama-model-string"), then we can merge this for the time being (since it's not a breaking change as far as I've tested) and then open an issue for a full fledged implementation if that is what we want. Then the usage would stay the same and we'd just change the implementation in textgrad/engine/__init__.py and add a separate 'textgrad/engine/ollama.py'.
How does that sound?

looks good. i left a couple of comments that should be quick to solve.
Would you be so kind to write 1/2 tests to check the objects are instantiated correctly?
I think we can merge this after the edits

Added instantiation test as well. It should be good to go @vinid. Let me know if there's anything more needed

thanks so much for this!!

thanks so much for this!!

Thank you for the fast merge 😁


Upgrade to Rails 4

Test suite is green on Rails 4
Fixed a couple of regressions encountered along the way
Merged upstream FB auth and other changes
Strong params Migration is still outstanding

It would be nice to have end-to-end functional tests for our primary happy user paths.

Holy epic PR

Looks good besides minor comment issues. Also, do you know what we need to do to get travis passing?

Travis will need the config.yml file sorted. Not having Travis is poking me in the eye too. I'll get it working after this PR for sure.

w00t, finally got this deployed


Code that do azimuth and elevation rotation on the Modelnet dataset ?

Hi, will you be kind enough to share the code which do data augmentation of the Modelnet dataset and voxelize the dataset? Thanks .

You can checkout the voxelization code here: https://github.com/charlesq34/3dmodel_feature
We pre-augmented data by rotation (rotate mesh and then voxelize) and saved them to HDF5.


MCMC prediction stability issue - providing NAN values of variance (Issue in Google Colab GPU/HighRam setting)

**This is the issue encountered in Google Colab- under GPU setting T4 and High-RAM
When we run the function- ExactGP.fit()-- it produces NAN values for standard deviation calculation. The error can be repo in all the below modifications
These things I have already tried

Normalize the data in fitting the model
Tried with all the kernels: RBF, Matern and Periodic
Tried with diff prior function: LogNormal, Normal, HalfNormal which are mostly used anyway in GP/BO
Tried with diff noise priors
Also tried with diff sets of training data

With current workaround it seems with reducing the number of total samples in MCMC setting to num_warmup=500, num_samples=500 (Default is num_warmup=1000, num_samples=3000), it is able to provide reasonable outputs.

Encountered the same issue outside of Google Colab without GPU

@nmenon97 - did it happen in the 64-bit precision regime?

@ziatdinovmax yes! I can sometimes get it to work by changing num_warmup and num_samples as suggested above but it isn't a stable solution.

This is due to the peculiar behavior of jax.vmap when approaching a memory limit. There are three ways to deal with this:

Draw multiple random smaller batches of samples (see gpax.acquisition.qEI, .qUCB, etc; you can specify the batch size using the subsample_size argument) and average them.
Assume that the acquisition function is continuous and use gpax.acquisition.optimize_acq to optimize it with num_initial_guesses << total_number_of_points.
Use a device with a larger memory :)


Initial openshift configurations.

Validated and tested on openshift using forked repo.

TODOS:

add a bc for a backup container and hook that up to the db container.
configure sonarqube.

@devinleighsmith, You can update the git references now.


Add support for Settings tab

The Settings tabs (VizFeature, Startup, AppSetup) are almost completely unimplemented. Need to implement or disable GUI elements (remove or make insensitive)

Disable GUI tab

Ready for review


superflore-change-summary.txt doesn't end with a newline

All text files should end with a newline.

#244

Merged.


[4.0.58] select2_ajax filter breaks "Remove filters" button (again)

Bug report
Previous issue - https://github.com/Laravel-Backpack/CRUD/issues/2135
What I did
Same as previous issue.
What I expected to happen
The filters should be removed.
What happened
The filter is removed from the interface, but the list return zero results.
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 7.4.4 (cli) (built: Mar 20 2020 13:47:45) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
with Zend OPcache v7.4.4, Copyright (c), by Zend Technologies
LARAVEL VERSION:
v6.18.3@4e48acfaba87f08320a2764d36c3b6a4a4112ccf
BACKPACK VERSION:
4.0.58@03432680d8ed839bcbd99ce57aed61bcf7a1b695

If you update your app to 4.1 there is already an reset button that will clear the localStorage from browser. In the meanwhile you need to do it manually from your developer tools interface in browser.
Let me know if that fixes the issue for you.
Best

@pxpm is this a bug in 4.0? Or can we close it?

@tabacitu i could not reproduce this in online demo. What version 4.0.** demo is running ?
I am going to close this, if you think this still an issue please re-open.

Ok so it looks like I've stumbled onto this problem myself.
Without a select2_ajax filter everything works fine. I click remove filters and all filters get removed, proper entries are shown:

However, if I add a seelect2_ajax filter and do the same thing (active multiple filters, then click the remove filters button),  it did NOT load all entries like you expected, it would say "No entries":

It looks like what's happening is that the GET parameter of the select2_ajax filter ("user" in this case) keeps being added to the URL, even when it has no value:

Which means that my filter logic would basically try to just show stuff where user_id is NULL:
        CRUD::filter('user')
            ->type('select2_ajax')
            ->label('User')
            ->values(backpack_url('net-promoter-answer/get-user-options'))
            ->whenActive(function ($value) {
                    $this->crud->addClause('where', 'user_id', $value);
            });

Now... this can easily be fixed in userland, by checking if the value is NULL:
        CRUD::filter('user')
            ->type('select2_ajax')
            ->label('User')
            ->values(backpack_url('net-promoter-answer/get-user-options'))
            ->whenActive(function ($value) {
                if ($value != null) {
                    $this->crud->addClause('where', 'user_id', $value);
                }
            });

But it's not intuitive, it's not the way the other filters behave. So select2_ajax should behave this way either, it shouldn't need this check if the $value is null. If the $value is null the filter should not be active, it shouldn't even reach the closure.
So I think we should prevent the select2_ajax filter from putting the GET parameter in the URL, when the value is NULL.

Fixed by #3177 hopefully, so I'm going to close this.


fix(network): negative connecting metric number

What this PR does / why we need it:
Use collection's length directly, this should fix negative connecting number gauge now.
Which issue(s) this PR fixes:

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

/lgtm


Support for c++ .so /.a  files

Here comes a enhancement! It would be great if we can create native c++ for android and ios.
It would be great!
Thx for this amazing work!

@MisterLight why stop with android and iOS why not generate c++ that targets a virtual platform abstraction layer that a developer can hook native c++ methods into for access to hardware, memory etc.


Move git dependencies to [patch.crates-io] section

This allows the crates that depend on basecoin, such as sovereign-ibc, to be able to more easily override the git dependency version by specifying their [patch.crates-io] section

CI fails as it does not have access to the private repos. By the way, merging this as things locally in both basecoin-rs and sovereign-ibc works.


Have option to download ~/.zuliprc from Zulip.

If you're connected to the server, the server knows your host, your email, and your API key, so it can produce this for you:
[api]
<EMAIL_ADDRESS>key=wlueAg7cQXqKpUgIaPP3dmF4vibZXal7
site=http://localhost:9991

This would cut down a little bit of friction for people setting up their bots/integrations.

This is an excellent idea.  I guess we probably could do this in the form of adding a little download button on each bot's card as well as in the "show your own API key" section.


Undefined symbols for architecture arm64

Undefined symbols for architecture arm64:   "_OBJC_CLASS_$_ZipArchive", referenced from:       objc-class-ref in UnityZipFile.o ld: symbol(s) not found for architecture arm64 clang: error: linker command failed with exit code 1 (use -v to see invocation)
It says the same for the other architectures as well. Not able to build for iOS atm.

Same here

To-do's for UniZip for Unity & XCode:
add "PluginsCode>iOS>ZipArchive" files & folder to Unity project "Assets\Plugins"
After that copy from package UnityZip: ">Asset>Plugins" (folders, files) & add to your project : "Assets/Plugins/IOS"
Alterations for UniZip for iOS successful build:
If package is newly imported after above setup:
(1) Open "ZipArchive.mm" file and REMOVE:
-line 270[dc release]
-line 271[gregorian release]
-line 327[comp release]
-line 328[gregorian release]
-line 40[super delloc]
(2) Open "UnityZipFile.mm" file and REMOVE:
-line 32[zip release]
-line 35[list release]
-line 59[zip release]
-Before Build in XCode select "unity framework" from "unity iPhone" and go to "build settings>other>flags" add "-lz" symbol.**


Mistakes in stitched 360 panorama photo

Hello,
i downloaded the file from my samsung gear device and tried your software. Really nice work !
But ... there is a critical mistake in the photo, which is not in the comparion photo from the samsung gear photo director software.
Part of the original "dual lens fisheye" file from the device:

Now the correctly stitched image from the "photo director" software:

And the wrong part from the gear360pano image:

If i could/should/have to change params, pls give me some infos. I already trief the param "-n", but the result was the same.
Of course i would give you the original file from my gear device to analyze (if required). Please pass me an option to send you the file privately, because i would not like to publish my fullsize original photos here in the public channel.
Thanks,
Kim
Edited, because the image from the preview was not stored in the published post.

Any news on this? I have the same issue..

Any news on this? I have the same issue..

unfortunately not - i wished for more support and more attentive moderators in this forum.
I have (i think so) presented the error well documented. getting no reaction at all is sobering

This program is only a shell - the real work is being done either by enblend, or by multiblend.. try specifying multiblend when launching.. see if it stitches any better?


Add results of Stanford's OpenIE, UW OpenIE, and MPI's ClausIE

Stanford's: http://nlp.stanford.edu/software/openie.html
UW's / AllenAI: http://openie.allenai.org/
MPI: https://www.mpi-inf.mpg.de/departments/databases-and-information-systems/software/clausie/

This is already possible in the current tool for comparison.


You have issues

https://github.com/FritsvanLieshout/jwt-validator/blob/d343d317abbb440cbe6811d2701ff8be58efd1d2/src/main/java/io/github/fritsvanlieshout/AuthorizationFilterImpl.java#L22

@f


Create curioss.md

Take a look at the edits I made in the two joined commits.
Thanks for getting this one!


Add health check path

@emmanuel assumin this was a mistake? though prs are welcome :)

@bprashanth yup, this was intended as a PR against our fork, not mainline.
I (and my team) are excited to contribute; we're currently working with Nordstrom's legal team to get a corp CLA approved and signed. Expect some contributions once we've cleared that hurdle 😉.
Pardon the distraction.

@emmanuel great! looking forward to it


[wait for #224] [gradients] Get gradient of each layer

Added function to get the gradient of each layer
Each layer is responsible to fill in the list containing the gradients
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #226. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


TypeScript: Enforce noImplicitAny

Ideally Typescript to block anything unknown.
Besides having simple but effective unit tests for the UI components, the type checking is a solid foundation we must benefit from to make the project more robust.
We can't currently switch tsconfig.json rule noImplicitAny to true because we need to fill the gap and  explicitly define the missing types. Once that's done we could add tsc --outDir /dev/null ./src/**/*.ts to yarn unit:test.

Closing as there is already issue 819.


information_schema queries don't work for case-sensitive databases

The queries .tables and .schema use to get meta information about columns etc both use lower case 'information_schema'  and column names, when in fact these are all in uppercase in SQL Server.
This doesn't affect databases that are configured as case-insensitive (which is the default I believe)
If I get a spare minute I'll submit a pull request but it should just be a matter of changing the case of those table and column names in the queries.

Fixed


Document get_connections and remaining shared RPC methods

A lot of this work has been done external to Chia. It's a matter of integrating this work into docs.chia.net. Here's the repo:
https://github.com/dkackman/chia-api
Specifically for get_connections:
https://dkackman.github.io/chia-api/static/fullnode.html#api-Connections-getConnections

protocols all have the necessary links from #559 , detailed information will not be documented on these per request from product


Alternative regalloc backends

The regalloc2 crate provides a good API (VReg, PReg, Operand, Function, etc) for interacting with a register allocator. It would be nice if other register allocator backends could be made available with the same API to simplify integration on the embedder side.
What I specifically have in mind is a "fast" backend that simply allocates registers in a single pass with the goal of minimizing compilation time. This will have a huge impact on compilation time since my compiler spends 80% of its time in register allocation. In practice I expect to use this for a tiering JIT where hot code can later be re-compiled using the slow regalloc backend that produces better code.

Agreed -- either a linear-scan (in the classical sense, with precomputed live ranges and evicting furthest-future-used values) or a true single-pass allocator (as baseline compilers typically do, evicting all values to stackslots at merge-points and evicting all or per LRU on running out of registers) would be great to have. I don't have the bandwidth to work on this now but I'd be happy to review it if you or someone else wants to work on this.

It's not a high priority for me at the moment, but I'm glad to hear that you're happy to have it as part of the regalloc2 crate.


Feature/Custom Stages

Just checking - is the deleted "flexible-forms"branch dev work somewhere else now?

I’ve merged it with the master.
Its more of a custom stage name branch. More to be added for it to be truly flexible form.
Akshith G

On 03-Feb-2016, at 4:43 AM, Dazza Greenwood<EMAIL_ADDRESS>wrote:
Just checking - is the deleted "flexible-forms"branch dev work somewhere else now?
—
Reply to this email directly or view it on GitHub https://github.com/CollabAcademy/UnWorkshop/pull/19#issuecomment-178879198.

Ok, now I get it!  Thanks.

|   Daniel "Dazza" Greenwood http://dazzagreenwood.com/
|   CIVICS.com http://civics.com/ & Law.MIT.edu http://law.mit.edu/
|<EMAIL_ADDRESS>&<PHONE_NUMBER>
|   1 Broadway, 14th Floor, Cambridge, MA 02142
On Tue, Feb 2, 2016 at 6:17 PM, akshith gunasekaran <
<EMAIL_ADDRESS>wrote:

I’ve merged it with the master.
Its more of a custom stage name branch. More to be added for it to be
truly flexible form.
Akshith G

On 03-Feb-2016, at 4:43 AM, Dazza Greenwood<EMAIL_ADDRESS>wrote:
Just checking - is the deleted "flexible-forms"branch dev work somewhere
else now?
—
Reply to this email directly or view it on GitHub <
https://github.com/CollabAcademy/UnWorkshop/pull/19#issuecomment-178879198
.

—
Reply to this email directly or view it on GitHub
https://github.com/CollabAcademy/UnWorkshop/pull/19#issuecomment-178879937
.

I have also documented it on the README. Almost the same like the old deployment except for one step.
Akshith G

On 03-Feb-2016, at 5:28 AM, Dazza Greenwood<EMAIL_ADDRESS>wrote:
Ok, now I get it! Thanks.

| Daniel "Dazza" Greenwood http://dazzagreenwood.com/
| CIVICS.com http://civics.com/ & Law.MIT.edu http://law.mit.edu/
|<EMAIL_ADDRESS>&<PHONE_NUMBER>
| 1 Broadway, 14th Floor, Cambridge, MA 02142
On Tue, Feb 2, 2016 at 6:17 PM, akshith gunasekaran <
<EMAIL_ADDRESS>wrote:

I’ve merged it with the master.
Its more of a custom stage name branch. More to be added for it to be
truly flexible form.
Akshith G

On 03-Feb-2016, at 4:43 AM, Dazza Greenwood<EMAIL_ADDRESS>wrote:
Just checking - is the deleted "flexible-forms"branch dev work somewhere
else now?
—
Reply to this email directly or view it on GitHub <
https://github.com/CollabAcademy/UnWorkshop/pull/19#issuecomment-178879198
.

—
Reply to this email directly or view it on GitHub
https://github.com/CollabAcademy/UnWorkshop/pull/19#issuecomment-178879937
.

—
Reply to this email directly or view it on GitHub https://github.com/CollabAcademy/UnWorkshop/pull/19#issuecomment-178894698.


Horizontal Scroll

Scrolling using a horizontal scroll bar is time-consuming and inconvenient, so please add a key binding to scroll horizontally using shift + scroll(left/right). I hope this can be added straightforward  :-)

OMG, Just now came to know that I can do horizontal scroll using alt + scroll,
You can close this one. Ha


Wrong parties.conf file is uploaded

Hi,
as follow-up of the unresolved discussion in #29, I opened this issue to track the problem and state it (hopefully) more clearly.
Currently, this prevents us from using protocols like ABY when the parties are located in different AWS regions. I.e. we consider external protocols without a coordinator on AWS.  I did only look at ABY's two-party protocols but the problem should also appear with more parties.
The AWS instances have both, private and public.  The private ip addresses are allocated from a private address range, e.g. from the <IP_ADDRESS>/12 block.  These are the addresses that are bound to the VMs network interfaces, but they are only routed inside the same region.  The public ip addresses are used to connect to the instances from the outside (or from other AWS regions).
So when both parties are in the same region, then the parties.conf file is filled using the private addresses:
https://github.com/cryptobiu/MATRIX/blob/d4b16244a44129606198ba9d53b0b41c866863e4/Deployment/aws_deploy.py#L250-L251
For context of the previous code snipped, the else branch means that only a single region of a single cloud provider is used.  This results (in my example) in the following parties.conf uploaded to the instances:
$ cat ABY/MATRIX/parties.conf
party_0_ip=<IP_ADDRESS>
party_1_ip=<IP_ADDRESS>
party_0_port=8000
party_1_port=8000

In the case with multiple regions, the public IP addresses are used instead:
https://github.com/cryptobiu/MATRIX/blob/d4b16244a44129606198ba9d53b0b41c866863e4/Deployment/aws_deploy.py#L245-L247
Now a parties.conf like the following is uploaded:
$ cat ABY/MATRIX/parties.conf
party_0_ip=<IP_ADDRESS>
party_1_ip=<IP_ADDRESS>
party_0_port=8000
party_1_port=8000

As said above, we can use these to connect to an instance from the other AWS region.  However, trying to listen on these addresses result in errors, since they are not bound to the instances' network interfaces.
To circumvent this issue, special files parties0.conf and parties1.conf are created for the parties 0 and 1, respectively, if they are located in different regions.
https://github.com/cryptobiu/MATRIX/blob/d4b16244a44129606198ba9d53b0b41c866863e4/Deployment/deploy.py#L112-L114
The function create_parties_files_multi_regions creates these files in the following way: For each party, its address is replaced with <IP_ADDRESS> in the file corresponding to its ID. Hence, this results in e.g.
$ cat parties0.conf
party_0_ip=<IP_ADDRESS>
party_1_ip=<IP_ADDRESS>
party_0_port=8000
party_1_port=8000

Note, that we can bind to <IP_ADDRESS> (accepting connections at any IP address).
The problem is, that for protocols like ABY these files (parties0.conf, parties1.conf) are never uploaded to the instances.  Here is the case corresponding to external protocols without coordinator in the run_protocol function in Execution/fabfile.py:
https://github.com/cryptobiu/MATRIX/blob/d4b16244a44129606198ba9d53b0b41c866863e4/Execution/fabfile.py#L133-L139
In contrast to the above, here are the corresponding, working code snippets for the other setups:

external protocol with coordinator

https://github.com/cryptobiu/MATRIX/blob/d4b16244a44129606198ba9d53b0b41c866863e4/Execution/fabfile.py#L124-L128

internal protocol

https://github.com/cryptobiu/MATRIX/blob/d4b16244a44129606198ba9d53b0b41c866863e4/Execution/fabfile.py#L99-L103
As noted before (here and in #29), the missing code existed before but was accidentally removed in the meantime.
So, I would like to ask you to readd these lines again; I have prepared an according pull request based on the current state of branch 1.2: #35.
Please ask me if anything remained unclear or misunderstood.
Best
Lennart
Edit: referenced PR

Thanks for merging, Lior!
There are some other things; I will open issues when I find the time.
Lennart


Implements RFC 8, fixing #606

Make sure these boxes are checked! 📦✅

[x] You have the latest version of rustfmt installed

$ rustup component add rustfmt

[x] You ran cargo fmt on the code base before submitting
[x] You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨
Fixes #606, test included.

I've pulled this branch and tested it. It looks correct. @jpgneves could you pull in the recent changes from master?

I've pulled this branch and tested it. It looks correct. @jpgneves could you pull in the recent changes from master?

@simlay Thanks! I just rebased off of master.

@simlay Not wanting to put undue pressure, is there any chance this can be looked at? Thanks!

@drager bump.

Is there a timeline for when this RFC fix is going to be released? NPM dependencies still seem to be broken in v0.10.2.


HHH-14032 Parse Locales with scripts properly in LocaleTypeDescriptor.fromString

Locales with scripts (e.g. zh-Hant-HK) were not parsed properly by LocaleTypeDescriptor,
causing the script part to get lost.

Sorry for the delay with this. I've gone with a different approach here that does not use a loop or switch statement at all. One caveat is that I decided not to attempt to parse extensions at all, as that is its own little rabbit hole; this PR strictly adds support for parsing scripts. I am not sure how prevalent bcp47 extensions are in practice, but I am assuming that script is more widely used and important. Extensions could be tackled in a follow up, as the way the code is rewritten would allow it to be extended fairly trivially.

Sorry for the delay with this. I've gone with a different approach here that does not use a loop or switch statement at all. One caveat is that I decided not to attempt to parse extensions at all, as that is its own little rabbit hole; this PR strictly adds support for parsing scripts. I am not sure how prevalent bcp47 extensions are in practice, but I am assuming that script is more widely used and important. Extensions could be tackled in a follow up, as the way the code is rewritten would allow it to be extended fairly trivially.

I don't like the approach taken here. I think what we should be doing here is offering a whole new way of storing Locales based on the language tag. Unfortunately, it doesn't look like we can do that in a completely backward compatible way, as I commented on the issue. So we'll need some way for the user to opt in to it. That's unfortunate.
Let's discuss it further on the issue.

i.e. we should really just be using toLanguageTag() and forLanguageTag() instead of parsing locales in our own code.


Invalid Redeclaration Warnings with Swift 4.x & Xcode 9.3

Hi Team,
I am using Xcode 9.3 and added this lib using pod. When I tried to build, found warnings, PFA error screen shot showing warnings.
Could you please resolve this? Is there any Swift 4.x specific branch is available?

Same here looks like this project isn't being maintained anymore.

Thank you, this helps 👍

This is a duplicate issue of https://github.com/Hearst-DD/ObjectMapper/issues/961

FYI this a compile error in Xcode 10 beta 1 (and it's new build system)

I can see a PR pending for the above issue:
#976

Is there any timeline to be getting a fix in for this? I would like to be moving forward using Xcode 10 to prep for iOS 12, however, this is making it so I cannot even compile our main app. Thanks in advance!

News?

#980 fixed this in Xcode 10.
Warnings in Xcode Xcode 9.3 and 9.4 will remain due to the fact that the behavior of IUO is not the same as Xcode 10 and as a result compilation fails in test targets.


Add lxd version to /1.0

Signed-off-by: Chuck Short<EMAIL_ADDRESS>

Can you rebase your branch to have only a single commit and get rid of that weird conflict?

I'll pick this up and send an update pull request.

Closing as my branch includes this.


Allow testing actors with config models

Currently, when tested actor runs and tries to access workflow
config, it fails with WorkflowConfigNotAvailable. Apart from
feeding the actor a mock config model, we need to also make
it aware it should use the config.

Can one of the admins verify this patch?

Strange, it looked like it was working for me during my POC but now it is not. Setting WIP then...

@vinzenz  yep I will confirm once it is in order...

@vinzenz should be hopefully ok now...

https://github.com/oamg/leapp-repository/pull/303 as an usage example...


Repository initial design

This is cleaning up the original DB and Entity code. The changes are as following:

Made POI to Amenity a simple ManyToMany relationship. The original code had a OneToMany which is incorrect. Closes #34.
Eliminated some extra annotations like @Column where possible to reduce clutter.
Made Color an Enum for extra type checking and simplicity. Colors will never change in the DB. Closes #36.
Made the station connection a ternary relationship to greatly simplify the code. Mapped the relationship on the station Entity as a Map since we will often be doing node traversal when doing way finding. Closes #37.
Deleted unneeded Entities. Making POI to Amenity a simple ManyToMany and using a Map for the ternary relationship eliminates the need for the join table classes.
Inserted sample data to create a temporary testing database. We will use this next week to ensure everything is working properly. Closes #28.

Sorry, still learning the reviewing stage lol. I think I could review it on a per commit basis and was just, what I thought, approving the first commit. Still need to look at the other three


Improves IO Parsing

Enhancement:

IO trace events and related one should be printed in on line
IO trace processing should add latency of IO

Fixed in 19.9 milestone


Latitude/longitude enhancement request

I see something similar was requested in #2583 and rejected as being out of scope for iD.  But there are reasons why this would be an actual enhancement to functionality.
Suppose you are given the lat/long of an object.  It's possible.  You contact some company by e-mail, can't figure out their exact location without referring to impermissible sources, so you ask them to give you lat/long of where they are.  Most people can do that one way or another, but asking them to send you a GPS trace in a format you can use is a lot harder for them.
So you have a lat/long.  I can type that into the search box of OSM itself: it takes me to the area and pops up a marker.  There doesn't seem to be an equivalent mechanism in iD.  So the only way I could get it to work as put my finger on the lat/long marker so I could use the leave a note facility to add a note where the lat/long marker was.  Then use a Mark I eyeball to compare the OSM map and the iD map and drop a node in roughly the right position.  Then use the aerial imagery to refine things.  It worked, but it's a real pain.
So some equivalent mechanism in iD would make life a lot easier.  Perhaps also use the empty space at the bottom of the window between the distance scale and the "Edits by" to display lat/long of the pointer.  To prevent it slowing things too much, maybe only provide lat/long when in add an object mode.  Or maybe have a button at the right to toggle the lat/long display.

#3088 and #4197 contains some more reasons why we would prefer to not allow people to enter lat/lng coordinates directly.

So you have a lat/long. I can type that into the search box of OSM itself: it takes me to the area and pops up a marker.
...
So some equivalent mechanism in iD would make life a lot easier.

Yes, you can do this in iD's search field too.  Paste a lat/lng into the search field, and iD will go there, then you can place place your point where it needs to go.

Hmmmm, I tried entering lat/long before and it didn't seem to do anything sensible, in that it might have put me in the right area but didn't show where on the map that point was.
However, the shift-control-L thing, which I hadn't realized was there, does everything I need.  It shows where the pointer is.  I can look at the aerial imagery when the pointer is at the right lat/long and add an object.  Or conclude I've been given silly co-ordinates.
As for the reasons you wouldn't add what people were asking for (the location window probably was what they were asking for but didn't phrase it well), are people really stupid enough to extend a way using lat/long and not check the imagery?  They probably are.


implement continuous integration between this repo and spee.ch server.

Story: As a developer, I want to be able to commit my changes to master and have those changes go live, so that I do not have to spend time manually updating the version on the server.

@alyssaoc yes and we very much want this, it'll help a lot with maintaining updates

We have auto deploy but not integration, @nikooo777 wanted to use autodeploy.
Out of necessity that's all that has been implemented.

for CI we need a set of running tests or a working build flow.
I spent a good chunk of my saturday trying to figure out a clean solution. I have a branch up that almost builds on travis. As soon as you guys figure out the npm run configure part and a way to get npm run build to run without the configs I'll gladly change the auto-deployment to CI.


Add WavIntoSamples

Close #2
I tackled it myself.

Note that I have a work-around for this if you decide not to merge this PR.
This PR is mostly for convenience.

It makes sense to have this. Thank you for writing it, I’ll merge if you fix the typo.

Done

Thanks!


mw-menu-entry directive extend with target attribute

add target attribute for mw-menu-entry directive where we have links as menu items

merged


Remove henryturner

It's my last day today, but just in case I need it please merge after Wed 7th December.

👋

:wave:


Fixing incorrect reference concerning role assignment

Hi @asmalser-msft, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Aaron Smalser). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

#sign-off

@asmalser-msft : Thanks for your contribution to the Azure documentation! The author, @asmalser-msft, has been notified to review your proposed change.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It�s easy to access the private repo version of your articles in GitHub. Just add �pr to the URL, as follows:
Public repo URL: https://github.com/Azure/azure-content/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Azure/azure-content-pr/blob/master/articles/azure-portal-overview.md
We've closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions.


Add a Module::deserialize_file method

This commit adds a new method to the wasmtime::Module type,
deserialize_file. This is intended to be the same as the deserialize
method except for the serialized module is present as an on-disk file.
This enables Wasmtime to internally use mmap to avoid copying bytes
around and generally makes loading a module much faster.
A C API is added in this commit as well for various bindings to use this
accelerated path now as well. Another option perhaps for a Rust-based
API is to have an API taking a File itself to allow for a custom file
descriptor in one way or another, but for now that's left for a possible
future refactoring if we find a use case.
cc #3230

It's worth noting that this will, by default, not play nicely with https://github.com/bytecodealliance/wasmtime/pull/3265. Relocations are the only part of the file which we actually modify today, so some code will be added to tempoarily change the text section to read/write if relocations need to be applied. With https://github.com/bytecodealliance/wasmtime/pull/3254, though, that should be rare. I don't actually know if it's possible for the new backend to generate any other relocations after we remove those for call instructions.

Ok I've added a commit to handle the case that we need relocations now that https://github.com/bytecodealliance/wasmtime/pull/3265 is in main

So I through a little test program together: https://gist.github.com/peterhuene/64a5dc5dd22f28950ec397a5709f0eda
This seems to do what we want: a single mapping where we can keep everything but the text section read-only and have CoW semantics for the text section when applying relocations.

Also, we'll need to have something in CodeMemory for Windows that detects if the mapping is backed by a file and use PAGE_WRITECOPY (that's not a valid protection level for anonymous pages) for the relocations; otherwise we can stick to PAGE_READWRITE.

So I threw a little test program together: https://gist.github.com/peterhuene/64a5dc5dd22f28950ec397a5709f0eda

Thanks for this! I just blindly copy/pasted that into Wasmtime now :)
As you can probably tell I did not indeed run tests at all on Windows beforehand, how naive of me to assume it would be the same! In any case I'm glad you know what you're doing, it would have taken me quite awhile to figure out how to align all those stars...

Oh hey and with your help works on the first try!


add please www.bitcoin.com

add please www.bitcoin.com, i send you $5 in btc cash :)

Hi there, thanks for the suggestion. I personally don't use Bitcoin Cash, and I think it would be confusing to have tickers for two different currencies in the app's current UI. Feel free to fork the repo and make a Bitcoin Cash version though!

no, you did not understand me, I meant to make a course on the site bitcoin.com, I will gladly make a trip to bitcoin or bitcoin cash))

I'm not familiar with bitcoin.com or what a course is on that site, do you have any links with more information? I will look into it, but I can't promise anything because I'm super busy right now.


will you offer a property to set the color of catAxisLine and valAxisLine?

I found the property..it‘s missing in the api doc


ttl link to resources

in the html interface, it would be convenient that when we have a link to a resource in ttl (in parenthesis), for instance on
http://buda1.bdrc.io/query/IIIFPres_itemInfo?R_RES=bdr:I29329
we could have
W29329 (ttl)

with html
W29329 (W29329)

Done as of commit 5a7c048


Fcitx5: Incorrect tray icons

Describe the bug
Fcitx5 is displaying incorrect tray icons.
On the first startup (e.g. after restarting the fcitx5 daemon), it will display a blank icon.
After focusing on an input field, it will display the current input method's icons, but not the one set for my system. I'm not really sure where the icons come from, but they don't even match with the default ones supplied by the repository: https://github.com/fcitx/mozc/tree/fcitx/src/data/images/unix
Steps To Reproduce
With a home-manager configuration:
i18n.inputMethod = {
  enabled = "fcitx5";
  fcitx5.addons = with pkgs; [ fcitx5-mozc ];
};

This will install with pkgs; fcitx5-with-addons.override { addons = [ fcitx5-mozc ] }.
Expected behavior
The correct icons should be displayed.
Screenshots
On the first startup (blank icon):

After focusing on an input field (non-matching icon):

Additional context
I tried putting additional XDG_DATA_DIRS paths in the wrapper:
https://github.com/NixOS/nixpkgs/blob/master/pkgs/tools/inputmethods/fcitx5/with-addons.nix
Like this:
wrapProgram $out/bin/fcitx5 \
  --prefix FCITX_ADDON_DIRS : "$out/lib/fcitx5" \
  --suffix XDG_DATA_DIRS : "${papirus-icon-theme}/share:$out/share" \

But it didn't work.
Notify maintainers
@poscat0x04 @berberman @govanify
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.45, NixOS, 23.11 (Tapir), 23.11.20230812.31a86dc`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.15.1`
 - nixpkgs: `/nix/store/19baybfi9l7bgvpl2zbp6ivngmakc4h3-source`

I'm experiencing somewhat of a similar issue on Manjaro with Plasma 5.27.8, except that instead of a missing icon, I'm getting this whenever I switch to Chinese Pinyin:

At first, I thought it was just the config file referencing the wrong icon (fcitx-pinyin), but when I looked at all the icons on my system with that name, they were all different (showing the correct icon).


Brief may update

status
The past month or so has been a bit slow, and a bit frustrating, and so I want to step back and outline my current thinking & progress.
I've ended up spending a bunch of time just developing a better understanding of how to think about table structure, packing, tracking offsets, etc. I've also spent a bunch of time trying to come up with an API/design that supports both from-scratch compilation as well as allowing reuse of parsed font data.
In brief, there is a fundamental tension between the design of a harfbuzz-style subsetter and the design of a general purpose font compiler. I am comfortable coming up with a design for each of these things independently, but I am finding it challanging to come up with something that does both (although I believe we should be able to have components, such as a repacker, that can be reused in both cases, and this is not insignificant.)
While the subsetter cannot be reused for compilation, general purpose compilation types can be reused for subsetting. We can, if desired, use these types to implement a subsetter or an instancer, in addition to a full compiler.
So I would now like to focus explicitly on the general case of 'types for compilation'. This involves having higher-level representations of the various tables and records, that can be created and mutated and then compiled down to their binary representations. It will be possible to create these types directly, and it will also be possible to convert parsed font types into these higher-level types.
If we find that the performance of this approach is unacceptable, we can then choose to write more specialized implementations. We can also do this piecemeal, specializing only specific tables.
next steps / tl;dr
I am taking the last two weeks of May off. I would like to spend the two weeks leading up to then really focusing on the general case of compiling font tables. I feel like I have been letting myself be too concerned with abstract concerns about performance and flexibility, and that this is letting me expand the scope of the problem too drastically. Once I have a design that basically works, we will be able to determine its limitations, and decide if there are specific areas that need further attention.

So I would now like to focus explicitly on the general case of 'types for compilation'. This involves having higher-level representations of the various tables and records, that can be created and mutated and then compiled down to their binary representations. It will be possible to create these types directly, and it will also be possible to convert parsed font types into these higher-level types.

If underneath your 'types for compilation' you have a serializer layer that takes iterators to compile the binary objects, then the harfbuzz-style subsetter can feed its own iterators to the same serializer.

there is a fundamental tension between the design of a harfbuzz-style subsetter

Can you make that more specific? - I don't disagree, I just think it can be spelled out a little more. IMHO it might be worth starting with examples of specific problems caused by this tension, and then generalizing the concern.
I think, as is perhaps suggested by the comment about repacker, my expected outcome would be that some (most?) things are shared by subsetter and compiler but not all. Hopefully most of the complicated bits :)

I think it might be wise to loop in @garretrieger. He's something of an expert on subsetting and repacking so I'm optimistic if you guys grabbed an hour or two to chat you might come up with some interesting ideas.

For the time being I've got a dummy table packer that just fails on overflow, but I will definitely reach out to garret when it's time to get serious. :)


introduce KYC in the dashboard

Which package/s are you suggesting this feature for?
Dashboard
Is your feature request related to a problem? Please describe
Child of https://github.com/threefoldtech/home/issues/1572
Describe the solution you'd like
Enforcing every user wanting to deploy on the grid to go through the KYC process

in twin details add KYC status
in client twin handler verify KYC
on dashboard login/refresh, check status
on logout, remove status
KYC verification setup should be in an iframe on a separate component that should appear on the button click
Add documentation in manual repo

work completed:

init the kyc componet in UI
add kyc to profile manager store
block the deployments if kyc is not verified
add kyc client to grid client #3531

nothing new waiting for the service domain to continue testing

WIP: integrate kyc in dashboard
add some error handling to the client, modified some method return types, now get token and status are ready will try to test the full idenfy flow tmw

issue update: almost done need to define error types and handle them in dashboard

KYC verified on devnet
Version ffbb750
Steps made to verify :
-Made multiple of new accounts
-Verified that the minimum amounts of tft to verify your account is 100 tft

User can not deploy unless he verifies his account

Steps to verify are clear wether click on verify now in the warning

or to go to Tfchain profile

IDenfy page will open and offer you the choice wether to go and verify from the mobile or user can click on verify from the

user will have the options to verify using a various methods

If the user provided a working info account will be verified

if the user provided an Expired document or blur photos

test section covering all of these

Verified:

QAnet
2.6.0-rc3

only noticed this manual verification and its working fine.


Prefer output format mp4 if possible

Is there any syntax that I'm missing for --merge-output-format that allows you to set it to prefer mp4 over mkv? Currently I'm downloading with this format -f bestvideo[fps=60]+bestaudio/bestvideo+bestaudio/best, and sometimes, depending on the video, the result can be merged into an mp4, but sometimes it can't, so I set --merge-output-format to mkv. However, is it possible to set it to prefer mp4 if possible and fall back to mkv if not?

It always prefers mp4 by default.  If that doesn't work, it merges to mkv instead.
(bionic)ealgase@localhost:~/Downloads/youtube-dl$ youtube-dl -f bestvideo+140 kJQP7kiw5Fk
[youtube] kJQP7kiw5Fk: Downloading webpage
[youtube] kJQP7kiw5Fk: Downloading video info webpage
[download] Destination: Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.f137.mp4
[download] 100% of 122.13MiB in 00:15
[download] Destination: Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.f140.m4a
[download] 100% of 4.27MiB in 00:00
[ffmpeg] Merging formats into "Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.mp4"
Deleting original file Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.f137.mp4 (pass -k to keep)
Deleting original file Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.f140.m4a (pass -k to keep)

(bionic)ealgase@localhost:~/Downloads/youtube-dl$ youtube-dl -f bestvideo+bestaudio kJQP7kiw5Fk
[youtube] kJQP7kiw5Fk: Downloading webpage
[youtube] kJQP7kiw5Fk: Downloading video info webpage
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[download] Destination: Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.f137.mp4
[download] 100% of 122.13MiB in 00:15
[download] Destination: Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.f251.webm
[download] 100% of 4.40MiB in 00:00
[ffmpeg] Merging formats into "Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.mkv"
Deleting original file Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.f137.mp4 (pass -k to keep)
Deleting original file Luis Fonsi - Despacito ft. Daddy Yankee-kJQP7kiw5Fk.f251.webm (pass -k to keep)

if you want to always get an mp4 file, then use a format selector with compatible formats.
https://github.com/ytdl-org/youtube-dl#format-selection-examples

But if I don't specify anything, it'll merge everything into a webm. And if I specify mp4 when that's not possible, I get ERROR:   Stream #1:0 -> #0:1 (copy)

if you want to always get an mp4 file, then use a format selector with compatible formats.
https://github.com/ytdl-org/youtube-dl#format-selection-examples

I'm not talking about the quality from -format, I'm talking about merging it into a mp4 with --merge-output-format. I don't always want mp4, I want the best quality, and I'd prefer it to be merged into an mp4 if possible, and if not, then an mkv.

For example, when using -format you can use the / character to prefer some other format if your first choice isn't available, so I thought that using something like --merge-output-format mp4/mkv would work with the same logic, but it doesn't. Is there something I'm missing?

and @ealgase already answered this, don't touch the --merge-output-format then.

and @ealgase already answered this, don't touch the --merge-output-format option then, this option is used to force a particular container, youtube-dl by default will choose an appropriate container for the selected formats.

Exactly, my question is whether it's possible to force it to use the mp4 container whenever possible (for example when the video format is mp4) with the ability to fall back to mkv (if the format is webm for example)

@kik4444 webm is literally just mkv with a different file extension.

I said don't use --merge-output-format and you will get what you want.


[Access Request] protected var - report 7 "Trial Balance/Previous Year"

Can you change FiscalYearStartDate, FiscalYearEndDate, LastYearStartDate, LastYearEndDate, FiscalYearBalance, FiscalYearNetChange, LastYearBalance and LastYearNetChange to proteced var in report 7 "Trial Balance/Previous Year"?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Resource: named queries

A resource to manage named queries and their description.
More information: once you've built a query in the UI, you can name this query and add a description. This makes it easier to share this query. Additionally, this title is shown on boards.
API: there is no API yet for this
Possible Terraform schema:
resource "honeycombio_query" "example" {
  name        = "My query"
  description = "A description of this query. _Supports Markdown._"

  dataset    = "my-dataset"
  query_json = data.honeycombio_query.my_query.json
}

To discuss:

this might be confusing next to data "honeycombio_query"? Maybe rename the data source to honeycombio_query_spec so it's clear this is only the specification of query

cc. @fitzoh mentioned this in Slack

This becomes possible with the query annotations API: https://docs-doug.honeycomb.io/api/query-annotations/

This is now implemented with #63!