FR: Data Attribute as Callback Functions?

Hi there,
I'm currently working a bit with AJAX and was wondering if it's possible to attach callback functions as data attributes. I tried it with data-option-callback='callBackFunction', but it seems that this.$options only allow primitives, such as strings or numbers, or am I seeing this wrong? The callback is important for me at this point because I animate the inserted content after successful loading with GSAP, and therefore need to inject JS. But there are surely also some other use cases where such callback functions would be interesting. Is this idea a good extension to the framework?
Here's an example using options, which currently doesn't seem to be functioning properly. Additionally, it's worth noting that even if it did work, there would likely be issues when multiple AjaxLoad components are present on the same page. In such a scenario, implementing some form of context awareness would probably be necessary

Loading...

An error occurred.

// AjaxLoad.js
import { Base } from '@studiometa/js-toolkit';
import type { BaseProps } from '@studiometa/js-toolkit';

export interface AjaxLoadProps extends BaseProps {
  $refs: {
    loading: HTMLElement;
    error: HTMLElement;
  };
  $options: {
    src: string;
    terminateOnLoad: boolean;
    callback?: (content: string, element: HTMLElement) => void;
  };
}

/**
 * AjaxLoad class.
 */
export class AjaxLoad extends Base<AjaxLoadProps> {
  /**
   * Config.
   */
  static config = {
    name: 'AjaxLoad',
    refs: ['loading', 'error'],
    options: {
      src: String,
      terminateOnLoad: Boolean,
      callback: Function,
    },
  };

  /**
   * Load the content on mount using async/await.
   */
  async mounted() {
    if (!this.$options.src) {
      this.$log('The `src` option is missing.');
      return;
    }

    try {
      const response = await fetch(this.$options.src, {
        headers: { 'X-Requested-With': 'XMLHttpRequest' },
        method: 'GET',
      });

      if (!response.ok) {
        throw new Error('Network response was not ok');
      }

      const content = await response.text();
      this.handleContentLoaded(content);
    } catch (error) {
      this.handleError(error);
    } finally {
      this.finalizeLoading();
    }
  }

  /**
   * Handle loaded content and execute callback if provided.
   */
  handleContentLoaded(content: string) {
    this.$refs.loading.style.display = 'none';
    this.$el.innerHTML = content;

    if (typeof this.$options.callback === 'function') {
      this.$options.callback(content, this.$el);
    }
  }

  /**
   * Handle error.
   */
  handleError(error: any) {
    this.$log(error);
    this.$refs.error.style.display = 'block';
  }

  /**
   * Finalize loading process.
   */
  finalizeLoading() {
    if (this.$options.terminateOnLoad) {
      this.$terminate();
    }
  }
}

Hey @iuscare, thanks for the feedback and suggestion!
Allowing functions as options would need some extra care to parse and evaluate the function, as this could introduce security issues. I am not sure we will have time to implement it in the near future, as this is a feature we do not need for our own development. But I will keep it in mind!
For your use case, I would suggest defining the callbacks that you need in an object with keys that could be used as reference in the data-option-callback attribute.

  ...

import { Base } from '@studiometa/js-toolkit';
import { isFunction } from '@studiometa/js-toolkit/utils';
import gsap from 'gsap';

const callbacks = {
  myCustomCallback(content, el) {
    gsap.to(el, 1, { opacity: 1 });
  },
};

export default class AjaxLoad extends Base {

  // ...

  handleContentLoaded(content: string) {
    this.$refs.loading.style.display = 'none';
    this.$el.innerHTML = content;

    const callback = callbacks[this.$options.callback]

    if (isFunction(callback)) {
      callback(content, this.$el);
    }
  }
}

Another option would be to extend the AjaxLoad class as many times as custom callbacks you need:
import AjaxLoad from './AjaxLoad.js';

export class AjaxLoadFadeIn extends AjaxLoad {
  handleContentLoaded(content: string) {
    super.handleContentLoaded(content);

    gsap.to(this.$el, 1, { opacity: 1 });
  }
}

export class AjaxLoadSlideIn extends AjaxLoad {
  handleContentLoaded(content: string) {
    super.handleContentLoaded(content);

    gsap.from(this.$el, 1, { y: 100 });
  }
}

A third option would be to use options to configure the gsap parameters you want:
import { Base } from '@studiometa/js-toolkit';
import { isFunction } from '@studiometa/js-toolkit/utils';
import gsap from 'gsap';

export default class AjaxLoad extends Base {

  static config = {
    name: 'AjaxLoad',
    options: {
      duration: { type: Number, default: 1 },
      from: { type: Object, default: () => ({ opacity: 0 }) },
      to: { type: Object, default: () => ({ opacity: 1 }) },
    },
  };

  handleContentLoaded(content: string) {
    this.$refs.loading.style.display = 'none';
    this.$el.innerHTML = content;

    gsap.fromTo(
      this.$el,
      this.$options.duration,
      this.$options.from,
      this.$options.to
    );
  }
}

We used a similar approach to create our ScrollAnimation component, it works quite well.
Or you could implement a component similar to our Frame component, which is itself based on our Transition component.

Hey @iuscare,
Just to let you know that we have added an Action component to our https://github.com/studiometa/ui package that might interest you. It lets you declare methods that should be called on other component right from your HTML.
I'll close this issue for now, but feel free to keep the discussion going.

@iuscare I have opened a PR on our UI package, adding some components that might be of interest for you: https://github.com/studiometa/ui/pull/256.


DBZ-4012 Promote Outbox SMT to GA

https://issues.redhat.com/browse/DBZ-4012
This PR is based on PR #2877 and therefore once the parent PR is merged, this can follow afterward.

The most recent SQL Server test failure is related to DBZ-4082, which I've fixed in a separate PR #2934

@Naros, the original PR for the MongoDB outbox router is merged. Could you rebase this one here? Thx!

Pushed a commit to fix the build issue which showed up for Vitess.

And applied.


replace boot.js for boot.dart

 Issue by sigmundch
Originally opened as dart-lang/sdk#18007

this is blocked by a couple issues:
 - issue dart-lang/sdk#18001: we need to be able to use spawnDomUri with package imports
 - issue dart-lang/sdk#18006: we want to include boot.dart in polymer.html in the same way we include boot.js

 Comment by sigmundch

Closing this until we set our new plan with the new script semantics.

Added NotPlanned label.

 Comment by sigmundch

Seems that some our plans might be changing here. I deleted the portions of boot.dart that were equivalent to boot.js in the change that reintroduces initPOlymer.
In case it is useful in the future, here is what the code looked like when I deleted it:
    main() {
      var scripts = discoverScripts(document, window.location.href);
      var sb = new StringBuffer()..write('library bootstrap;\n\n');
      int count = 0;
      for (var s in scripts) {
        sb.writeln("import '${s.importUrl}' as prefix_$count;");
        count++;
      }
      sb.writeln("import 'package:polymer/src/mirror_loader.dart';");
      sb.write('\nmain() => startPolymerInDevelopment([\n');
      for (var s in scripts) {
        sb.writeln("  '${s.importUrl}',");
      }
      sb.write(']);\n');
      var isolateUri = _asDataUri(sb.toString());
      spawnDomUri(Uri.parse(isolateUri), [], '');
    }

 Comment by jmesserly

Removed Area-Polymer label.
Added Pkg-Polymer, Area-Pkg labels.


Python returning no results for queries with %20 or whitespace.

Your service returns no results for multi keyword searches using spaces or %20 for python.
Illustration: About a month ago I ran this query through curl:
curl cht.sh/python/exponential%20backoff
#  Exponential backoff: time.sleep with random.randint(0, 1000)
# .... etc etc.
#  [NPE] [so/q/27438273] [cc by-sa 3.0]

But now your service returns the error: "Unknown cheat sheet"
curl cht.sh/python/exponential%20backoff
#  Unknown cheat sheet.
#  Please try to reformulate your query

$ curl "cht.sh/python/exponential backoff"
#  Unknown cheat sheet.
#  Please try to reformulate your query

Looks like I have to use plus only, instead of spaces, like this to get a response:
$ curl cht.sh/python/exponential+backoff
#  Exponential backoff: time.sleep with random.randint(0, 1000)
# .... etc etc.
#  [NPE] [so/q/27438273] [cc by-sa 3.0]

Same occurs with browser, clicky:  https://cheat.sh/python/exponential backoff
More examples of what used to return a helpful response from last month, but is not working now:
curl "cht.sh/python/exists dictionary"
curl "cht.sh/python/collapse+interior+spaces"
curl "cht.sh/python/octal example"
curl "cht.sh/python/deep copy"
curl "cht.sh/python/flush stdout print"
curl "cht.sh/python/lambda example"

clicky:
https://cheat.sh/python/exists dictionary
https://cheat.sh/python/lambda example
https://cheat.sh/python/deep copy
All of these used to return a stackoverflow entry exists%20dictionary -> so/q/1602934 but now it returns "Unknown cheat sheet."
It's been like this for the last 48 hours or more.  The mapping between %20 literal space, and + seems to be messed up somewhere in transit.

Yes, I can acknowledge it. I will investigate

Looks fixed.  You should put up a paypal donation button so I can throw some peanuts at you for the occasional swift kick of this amazing curl service.

@sentientmachine Eric say you for the potential donation :) But to say truth, your feedback and ideas is that what I appreciate even more! I am not actively working on the project at the moment, but I hope it will be better someday

If the having of impact on the other side of the planet is where you derive motivation to keep going on this project, here is a gif to show you exactly how your curl service fits into my vim muscle memory command sequence:

If I understood it right, for each cheat.sh query, a new tab in cheat.sh is automatically opened,
and you copy the answer from it. Ok.
My questions:

can you handle things like comments (on/off)?
is the language for the query (python in your case) automatically detected from your buffer configuration?

@dbeniamine what do you think, maybe you should add a new destination for the answer for the vim plugin? a tab? additionally to a buffer? I think it can be pretty convenient sometimes

Yes vim calls curl cht.sh and the stdout goes to a new ~/.vim/cht/python/root+mean+squared+error.py, makes a new tab like this execute "tabedit " . the_filename and inflicts a filetype like execute "set ft=" . inflicted_filetype.
So now the results of cht.sh can be navigated with normal navigation keys as I browse the stackoverflow answer.  I can highlight and copy lines to vim global register.
Comments and cht parameters  Yes.  If I want to pass extra parameters like: "remove comments" or "give me the n'th answer" vim just hacks the query and looks for the parameters, pluck them off and put them back on in the curl cht.sh statement correctly.  See https://imgur.com/3zt4ZvQ
Another way the editor "just knows" that files under the cht directory are python files (even when they're missing a .py extension or a shebang at the top) is this code:

   elseif whole_path =~ '.*/snippets/python/.*' || whole_path =~ '.*/cht/python/.*' || whole_path =~ '.*/usr/bin.*'
         call PythonBothSetFiletypeSetup()
     elseif whole_path =~ '.*/snippets/garpython36/.*' || whole_path =~ '.*/cht/garpython36/.*'
         call PythonBothSetFiletypeSetup()

These lines are in a function named: GuessFiletype() and run on every file that is ever opened in vim.  The vimscript8 peeks at the current directory and if it sees the file is in our cht history or a snippet history, it says aha, I know this is "python on windows vista/8.1" and not python 3.8, 3.6, 3.4, or 2.7 on gentoo.  It's a haaxxxx.  But that's how I like it.
I'm sure dbeniamine's plugin is perfect and great for normies, however it whisks me away to a new place, he can't know about how my muscle memory is set when I'm mid stream hacking away on a thing.  Muscle memory is encoded in the neuronal registers in my fingers and arms evolved to win slapping contests 800 million years ago.  Bram talks about how to encode programming skill in your registers in your arms rather than making the 8 foot network call journey to your head.

@chubin : I am not sure to see what openning in a new tab will change from the buffer.
cheat.sh-vim already open in a vim buffer setting the filetype to the language if explicitly set in the request. From what I understand of @sentientmachine comment the main difference is that he have a hack to guess the programming language of the answer if not set in the query.
Am I missing something else ?

@dbeniamine David, the difference is minimal; just an additional option to make it possible (with some additional key combination) to open answer in a new tab instead of the buffer (just for tab-lovers). Nothing more

Why not, I am oppening an issue for this feature


flash2-attention support thread safety

Does flash2-attention support thread safety? Can it be invoked through multi-thread x multi-stream?

I'm not sure, I've never tried. It uses Pytorch default CUDA stream so it should work with a Pytorch CUDA stream context manager


Please add 50 character limit for Channel names to documentation.

Please add 50 character limit for Channel names to documentation.

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

ID: 905e2c09-ef3e-07cc-0809-6ff171157c47
Version Independent ID: c0d33162-183d-fff8-32e9-a999f391ea51
Content: Limits and specifications for Microsoft Teams - Microsoft Teams
Content Source: Teams/limits-specifications-teams.md
Service: msteams
GitHub Login: @MicrosoftHeidi
Microsoft Alias: heidip

Hi @scanum, Could you please inform me about the reason for closing this issue? The referenced documentation have not been updated since 11/13/2021. Hope to hear from you, thanks!


python37Packages.azure-mgmt-resource: 15.0.0 -> 16.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.azure-mgmt-resource is: "Microsoft Azure SDK for Python"
meta.homepage for python37Packages.azure-mgmt-resource is: "https://github.com/Azure/azure-sdk-for-python"
meta.changelog for python37Packages.azure-mgmt-resource is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 16.0.0 with grep in /nix/store/s14wrgg534zy687f91cw38v38xlzs0x1-python3.7-azure-mgmt-resource-16.0.0

directory tree listing: https://gist.github.com/df2beadd81a8a13e08b2cb1feab53299

du listing: https://gist.github.com/411aecc3d3cf9e18c4abee62e7020127

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

17 total rebuild path(s)

5 package rebuild(s)

4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
5 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
appdaemon
azure-cli
python37Packages.azure-mgmt-resource
python38Packages.azure-mgmt-resource
python39Packages.azure-mgmt-resource

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s14wrgg534zy687f91cw38v38xlzs0x1-python3.7-azure-mgmt-resource-16.0.0 \
  --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 python37Packages.azure-mgmt-resource https://github.com/r-ryantm/nixpkgs/archive/3b4a3b786c60da7c6196560d77722f9d2cfa96ad.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/s14wrgg534zy687f91cw38v38xlzs0x1-python3.7-azure-mgmt-resource-16.0.0
ls -la /nix/store/s14wrgg534zy687f91cw38v38xlzs0x1-python3.7-azure-mgmt-resource-16.0.0/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

4 packages built:

appdaemon
azure-cli
python37Packages.azure-mgmt-resource
python38Packages.azure-mgmt-resource

Maintainer pings
cc @olcai @mwilsoncoding @jonringer for testing.

Result of nixpkgs-review pr 114851 at 3b4a3b78 run on aarch64-linux 1

4 packages built:

appdaemon
azure-cli
python37Packages.azure-mgmt-resource
python38Packages.azure-mgmt-resource


Investigate how we get auto diff back

For training this will be most useful, but one complication is that we can't seem to hook into TF's autograd subsystem as we're not subclassing tf.Tensors. What options do we have? Do they pose any challenges towards MPC? What happens when TF switches to eager mode by default?

We now know how to do this.  Could use a healthy prototype, but here's what we'd need to use.
Replacing a single node:

tf.RegisterGradient
Graph.gradient_override_map

Subgraph:

tf.custom_gradient is a decorator for doing something similar to tf.RegisterGradient and tf.Graph.gradient_override_map more concisely and for a whole subgraph instead of a single node

Here's an example from Pond:
@tf.custom_grad
def _mul_masked_masked(prot, x, y):
    assert isinstance(x, PondMaskedTensor), type(x)
    assert isinstance(y, PondMaskedTensor), type(y)

    a, a0, a1, alpha_on_0, alpha_on_1 = x.unwrapped
    b, b0, b1, beta_on_0, beta_on_1 = y.unwrapped

    with tf.name_scope("mul"):

        with tf.device(prot.crypto_producer.device_name):
            ab = a * b
            ab0, ab1 = prot._share_one_seed(ab)

        with tf.device(prot.server_0.device_name):
            alpha = alpha_on_0
            beta = beta_on_0
            z0 = ab0 + (a0 * beta) + (alpha * b0) + (alpha * beta)

        with tf.device(prot.server_1.device_name):
            alpha = alpha_on_1
            beta = beta_on_1
            z1 = ab1 + (a1 * beta) + (alpha * b1)

        z = PondPrivateTensor(prot, z0, z1, x.is_scaled or y.is_scaled)
        z = prot.truncate(z) if x.is_scaled and y.is_scaled else z

        def grad(dz):
                 return dz * y, dz * x

        return z, grad

The one thing I'm uncertain of here is that I'm not sure we can do this while still passing in prot as the first arg.  According to the docs, tf.custom_gradient generally expects its input function to have a signature f(*x), where x is a tuple of tensors.  I'm not sure if that's a strict requirement, but if it is we'll probably want to loop that into the larger discussion of API improvements.  FWIW I think this could go well with previous thoughts around removing the Protocol abstraction, or at least allowing high-level tfe operations to be able to delegate their internals to whichever protocol they want/need to.

I suggest we close this up for now, wdyf @jvmancuso? Don't think we have found the perfect solution yet but maybe we've minimised the issue with #470 et al.?

Agreed, we can work on having autograd for higher level layers and think about what to do on the general case later on


Quality, security and legal reporting/notifications

Related to #31
Acceptance criteria:

[ ] As a visitor, I want to see all security/legal/quality in a public GitHub Project
[ ] As a FINOS Project Team member, I want to be able to get notified about security/legal/quality issues

Tasks:

[ ] Collect public security/legal/quality issues across all FINOS hosted projects
[ ] Add collected issues into a GitHub Project (using API)
[ ] Document and socialize the "FINOS Projects reporting Board"
[ ] Document and socialize issue notifications (see https://help.github.com/en/github/receiving-notifications-about-activity-on-github/about-notifications)

Dependencies:
This story depends on the following ones:

[ ] Quality : #28
[x] Security: #81
[ ] Legal: #30

Implementation
As we try to enforce GitOps as paradigm to bring continuous quality, security and legal compliance on across all our repositories, also reporting and notifications should align.
For example, in order to report on security, it is already possible to get the raw data from https://api.github.com/search/issues?q=org:finos label:"security vulnerability"
Using JQ, it is possible to export the data in any format.

As soon as https://github.com/finos/metadata-tool/pull/60 is merged, the (FINOS internal) Metadata Tool nightly run will also report all repositories having issues labeled with security vulnerability and quality checks, allowing to have reporting abilities across security and quality aspects of our hosted code.

Given the introduction of LFX (specifically, Insights and Vulnerability Detection) in our infrastructure, we are going to rely on those collaboration tools.
Closing issue and marking it with LFX label.


How can I access the depth values of each pixel in the output?

The video is successfully processed and saved in the target folder. I can see the depth video that is fine but I am not able to see the output values of each pixel. Where can I find it? Being able to access to those depth values is key and can do something with it. Or I don't know where I need to look or I overlooked. Can anyone help me understand where can I find those?

Hi, you can re-write the run_video.py script by saving the per-frame depth maps as numpy arrays.


improve japanese translation

The purpose of this change
Improvement for Japanese translation.
I know that the translation files are auto-generated, so this is suggestion for translation against obvious mistranslations.

Thank you! I will share these suggestions with our translators.  When they are accepted a new PR will be generated with the updated translations and we will close this one.  Since these files are generated, we don't want to change them manually without updating the source.

This was added to PR #1560

This was added to PR #1560


Fix count counter in find_text

Proposed change

Fix count counter in find_text
Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #339

Codecov Report

Merging #340 (bde7ce3) into 0.3.0 (19e683c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##            0.3.0     #340   +/-   ##
=======================================
  Coverage   97.44%   97.44%
=======================================
  Files           3        3
  Lines         744      744
=======================================
  Hits          725      725
  Misses         19       19

Impacted Files
Coverage Δ

custom_components/mail_and_packages/helpers.py
96.75% <100.00%> (ø)

Codecov Report

Merging #340 (bde7ce3) into 0.3.0 (19e683c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##            0.3.0     #340   +/-   ##
=======================================
  Coverage   97.44%   97.44%
=======================================
  Files           3        3
  Lines         744      744
=======================================
  Hits          725      725
  Misses         19       19

Impacted Files
Coverage Δ

custom_components/mail_and_packages/helpers.py
96.75% <100.00%> (ø)


Add RQL syntax support for lt, le & gt, ge

https://github.com/agco-adm/harvest/commit/c00cb53d32511da3dede0e026b9900ef7533b69b?diff=unified#diff-0

The RQL spec recommends gte, lte rather than ge, le
Is this an overlook ?


client user: rework object model

Currently, because of content addressing, a short object (e.g. a Note that says "Hi") will be shared among all messages sent by anyone who writes the same message. This makes it hard to delete messages, and to attribute actions to the Object of a particular message.
By adding the attributedTo field the same content can be stored multiple times for multiple actors. If many messages relate to the same content, this will be tracked per-actor.
This brings up an issue: do objects need to be signed? Anyone can attribute an object to anyone else. Maybe this is OK?

This turned into a bigger question: why are objects signed? Is it to know where some data originates, or to know how it got to you?
Trying to answer where is not a tractable problem. Some data has no real origin (e.g. the sequence of prime numbers). Some data will originate outside of Chatter Net (e.g. images from wikicommons).
Answering how / why it got to you is more interesting. In the current web it gets to you as the result of a request to a server you trust. This doesn't mean it originates from the server. This is the purpose of a message: it is an object with an actor and a proof asserting that the actor carried out some action with some data.
Thus attributedTo should be optional. Some data cannot be attributed. Data is already stored only if it has an associated message. This is what is needed, a signed activity relating to data which indicates how the data gets to an actor.
To clarify this relationship:

the server should explicitly store the mapping of message to object, and it should store objects only if they have an associated activity
a Create activity should produce an object with attributedTo being the same as the actor of the Create activity

when an object is received, if its attributedTo differs from its Create actor, the Create message is deleted
likewise a message should be rejected if its object is has a different attributedTo

validating an object requires validating the proof it it exists
deleting a Create activity can be done only by its actor, and will delete its object

This task ends up being:

server should explicitly map message to body
rename object to body to clarify that this isn't a generic ActivityStream object
use interfaces in server to clarify interface versus storage


added 'add to dictionary' and top spellcheck options to context-menu

resolves #1745
Seems like most of the work was done and just commented out! Went ahead and added a "add to dictionary" button too.
Let me know if you have any feedback.

Looks good!


Customize file name

for example to include a timestamp instead of the web address.

Hi @evadknarf this is on my roadmap, I’ll let you know when it goes live.


Changed defualt for enable_create_defaults

Fix #15
Disables enable_create_defaults since it impacts default resources.

Should we even keep the enable_create_defaults feature?
If we expose the default_network_acl_id and default_route_table_id, users can manage the resources that would be created by this feature themselves.

Should we even keep the enable_create_defaults feature?
If we expose the default_network_acl_id and default_route_table_id, users can manage the resources that would be created by this feature themselves.

Yes I agree, therefor I would say as first step we remove the behaviour by default. In this case we keep the module compatible. We have projects using this feature right now.


Get a list of unique metadata fields

Let's say I have a metadata field named "Field", I wish to fetch a document each for every value of Field.
If Fields were "A", "B", and "C" I would have 3 documents.
Effectively I wish to get a list of unique values for a metadata field without manually iterating over them.
Related SO Thread
client = chromadb.PersistentClient(path=CHROMA_DIR)

collection = client.get_collection("project_store_all")
unique_keys = collection.get(where_document={"$distinct" : "title"}, where={"title": "state_of_the_union"}) #not working as expected
print(unique_keys)

I tried the following query but it seems I cannot pass a distinct to where_document

@blacksmithop, Chroma does not have (yet) any aggregation functions like in regular relational or NoSQL DBs. Therefore, you will have to iterate over the collection to generate the unique values.
Here's a sample code to do that:
import chromadb

client = chromadb.PersistentClient(path="uniq_metadata")

col = client.get_or_create_collection("metadata")

col.upsert(ids=["0","1","2","4"], documents=["doc 1","dc 2","doc 3","doc without metadata"],metadatas=[{"name":"name1","value":"value1"},{"name":"name2","value":"value2"},{"name":"name3","value":"value3"},None])

unique_values={m['name'] for m in col.get(where={"name":{"$ne":""}},include=['metadatas'])['metadatas'] if 'name' in m}
print(unique_values)

You may have to adjust the '{"name":{"$ne":""}}' expression to meet your need (the one above only assumes that you are interested in the metadata field name that doesn't have values).
Additionally, if your collection is large you may want to paginate results with limit and offset in the get().

I see, thank you!


refreshToken support

Below is a local extension of the application-route-mixin to support refreshTokens
import Ember from 'ember';
import ApplicationRouteMixin from 'ember-simple-auth-auth0/mixins/application-route-mixin';

const { $,
        Mixin,
        RSVP,
        get,
        getProperties,
        run,
        computed,
        set,
        testing } = Ember;

export default Mixin.create(ApplicationRouteMixin, {

  domain:       computed.readOnly('auth0.domain'),

  sessionData:  computed.readOnly('session.data.authenticated'),

  idToken:      computed.readOnly('sessionData.idToken'),
  refreshToken: computed.readOnly('sessionData.refreshToken'),
  userID:       computed.readOnly('sessionData.profile.user_id'),

  sessionInvalidated() {
    this._clearJobs();
    this._invalidateRefreshToken();
    this._logout();
  },

  _clearJobs() {
    run.cancel(get(this, '_expireJob'));
    run.cancel(get(this, '_refreshJob'));
  },

  _invalidateRefreshToken() {
    let { refreshToken,
          domain,
          userID } = getProperties(this, 'refreshToken', 'domain', 'userID');

    if (refreshToken) {
      let url = `https://${domain}/api/users/${userID}/refresh_tokens/${refreshToken}`;

      return this._makeAuth0Request(url, "DELETE");
    }
  },

  _logout() {
    get(this, 'auth0').navigateToLogoutURL();
  },

  _makeAuth0Request(url, method) {
    let headers = { 'Authorization': `Bearer ${get(this, 'idToken')}` };

    return $.ajax(url, { type: method, headers: headers });
  },

  _setupFutureEvents() {
    if (testing) {
      return;
    }

    this._clearJobs();
    this._scheduleExpire();

    if (get(this, 'refreshToken')) {
      this._scheduleRefresh();
    }
  },

  _scheduleRefresh() {
    run.cancel(get(this, '_refreshJob'));

    let remaining = get(this, '_jwtRemainingTimeInSeconds');
    let earlyRefresh = 30;
    let refreshInSecond = (remaining < (earlyRefresh * 2)) ? remaining / 2 : remaining - earlyRefresh;
    let refreshInMilli = refreshInSecond * 1000;

    if (!isNaN(refreshInMilli) && refreshInMilli >= 50) {
      let job = run.later(this, this._refreshAccessToken, refreshInMilli);
      set(this, '_refreshJob', job);
    }
  },

  _refreshAuth0Token() {
    return new RSVP.Promise((resolve, reject) => {
      get(this, 'auth0')
        .getAuth0Instance()
        .refreshToken(get(this, 'refreshToken'), (error) => {
          if (error) {
            reject(error);
          } else {
            resolve(this._setupFutureEvents());
          }
        });
    });
  },

  _refreshAccessToken() {
    this._refreshAuth0Token()
      .then((data) => this.trigger('sessionDataUpdated', data));
  }

});

This does look like a valid solution. However, I specifically left the refresh token flow out of this addon after talking with an engineer at Auth0 and he said it is a pretty big security concern if you are allowing the user to continuously log back in with a refresh token.

Interesting point; we migrated from our custom JWT implementation to Auth0.
Previously we used short lived tokens only, which we used for API authentication as well as to periodically request a new token from the API, while a user was active on the application.
The session was automatically terminated either due to the token expiring or by explicitly killing the session after defined amount of inactivity.
I agree with you that using refresh tokens which are long lived and need to be deleted explicitly may not be the best translation of our previous approach to Auth0. Do you know of any option to use short lived tokens only, but allow the session to stay active while the user is actively using the application?

Since i'm struggling a bit with how to handle expired sessions/token, what is the recommended way to handle when the users token expired? (detected by a request to the backend-api for example)

Don't know what the recommendation is, but we came back to using the idToken to refresh itself. So there is no long lived refreshToken in play. As long as the user stays active the idToken is refreshed periodically. If the token expires due to inactivity the user is logged out.
import Ember from 'ember';
import ApplicationRouteMixin from 'ember-simple-auth-auth0/mixins/application-route-mixin';

const { Mixin,
        RSVP,
        get,
        getOwner,
        getWithDefault,
        run,
        computed,
        set,
        testing } = Ember;

export default Mixin.create(ApplicationRouteMixin, {

  domain:       computed.readOnly('auth0.domain'),

  sessionData:  computed.alias('session.data.authenticated'),

  idToken:      computed.readOnly('sessionData.idToken'),
  userID:       computed.readOnly('sessionData.profile.user_id'),

  sessionInvalidated() {
    this._clearJobs();
    this._logout();
  },

  _clearJobs() {
    run.cancel(get(this, '_expireJob'));
    run.cancel(get(this, '_refreshJob'));
  },

  _logout() {
    get(this, 'auth0').navigateToLogoutURL();
  },

  _setupFutureEvents(data) {
    if (testing) {
      return;
    }

    if (data) {
      let authenticator = getOwner(this).lookup('authenticator:jwt');
      let tokenData = authenticator.getTokenData(data.id_token);

      set(this, 'sessionData.idToken', data.id_token);
      set(this, 'sessionData.idTokenPayload', tokenData);
      set(this, 'sessionData.expiresIn', data.expires_in);
    }

    this._clearJobs();
    this._scheduleExpire();
    this._scheduleRefresh();

    return data;
  },

  _scheduleRefresh() {
    run.cancel(get(this, '_refreshJob'));

    let remaining = get(this, '_jwtRemainingTimeInSeconds');
    let earlyRefresh = 30;
    let refreshInSecond = (remaining < (earlyRefresh * 2)) ? remaining / 2 : remaining - earlyRefresh;
    let refreshInMilli = refreshInSecond * 1000;

    if (!isNaN(refreshInMilli) && refreshInMilli >= 50) {
      let job = run.later(this, this._refreshAccessToken, refreshInMilli);
      set(this, '_refreshJob', job);
    }
  },

  _refreshAuth0Token() {
    return new RSVP.Promise((resolve, reject) => {
      get(this, 'auth0')
        .getAuth0Instance()
        .renewIdToken(get(this, 'idToken'), (error, data) => {
          if (error) {
            reject(error);
          } else {
            resolve(this._setupFutureEvents(data));
          }
        });
    });
  },

  _refreshAccessToken() {
    this._refreshAuth0Token()
      .then((data) => {
        let session = get(this, 'session.session');
        let authenticatorName = get(session, 'authenticator');
        let authenticator = session._lookupAuthenticator(authenticatorName);
        let sessionData = get(this, 'sessionData');

        set(sessionData, 'idToken', data.id_token);

        authenticator.trigger('sessionDataUpdated', sessionData);
      });
  },

  _expiresAt: computed('sessionData.idTokenPayload.exp', {
    get() {
      if (!get(this, 'session.isAuthenticated')) {
        return 0;
      }

      const exp = getWithDefault(this, 'session.data.authenticated.idTokenPayload.exp', 0);

      return exp;
    }
  })

});

This need ought to be covered by #120/#137, so I'll go ahead and close this one.


Initial entity relationship support

e.g. company to employee relationship
Modifications on UI: New Entity->Add Field
Add 'Relationship' as last option to dropdown where primary data types appear.
Selecting Relationship should add/show 3rd column as Related Entity. This dropdown will contain a list of existing entities from the current workspace.
@ranjan-rk question: Don't we need related entity's column name in relation as in case of RDBMS?

Assigned to @rka12

It is partially done. Close this task and create a new task based on what is pending.

@rka12 Basic entity relationship to be made working for now. New issue will be opened for what is pending from this.


Speed up point iteration process

There are two times in the builder script where each point in the geometry is iterated over. I want to see if these can be combined into one loop or if they are necessary

I think both of these can be eliminated. Here's some sample code.
In the first loop, the iteration is intended to mainly add a constant z dimension to the coordinate points in the polygons. That can be done as a set operation rather than a loop using a lambda function. Here's an example of how all of the points in polygon can be updated with a z value with one line of code on an example polygon named poly:
poly = box(*Point(1.23, 9.87).buffer(1).bounds)
polyz = shapely.ops.transform(lambda x, y: (x, y, 0.1), poly)

The second loop seems to be mainly to combine all of the Polygon records from the GeoDataFrame into a single multipolygon. This can also be done in a single line of code with a list comprehension. Also, you don't need to keep track of the bounds, because you can get that directly from the Polygon and MultiPolygon objects in a single call.
# Combine all of the individual Polygon rows into a MultiPolygon and extract the bbox
miltipolygon_z_4978 = MultiPolygon([gdf_4978.geometry[row_id] for row_id in range(gdf_4978.shape[0])])
box_degrees_4978 = miltipolygon_z_4978.bounds # note this does not have the z dimension, which might need to be added

So I reworked the point iteration process but it didn't speed up the execution time at all and we still need two loops. Once for the original projection to add the z values and a second time to calculate the geometry triangles as the first step of the gltf conversion.
I implemented Matt's suggestion for the first loop:

polyz = shapely.ops.transform(lambda x, y: (x, y, 0.1), poly)

https://github.com/PermafrostDiscoveryGateway/viz-3dtiles/blob/d35dad0b34798880e55a014b665d13570f7c2662/build-3d-tile.py#L55-L63
But this actually slows things down because in the main branch, you can see that we are only adding the z values to the convex_hull ring of the polygon. So I added back the simplify() process on the develop branch.
https://github.com/PermafrostDiscoveryGateway/viz-3dtiles/blob/d35dad0b34798880e55a014b665d13570f7c2662/build-3d-tile.py#L52

The second loop seems to be mainly to combine all of the Polygon records from the GeoDataFrame into a single multipolygon. This can also be done in a single line of code with a list comprehension. Also, you don't need to keep track of the bounds, because you can get that directly from the Polygon and MultiPolygon objects in a single call.

The second loop does a lot more than that. It creates a Multipolygon for each Polygon and calculates the geometry triangles for that polygon. The entire dataset cannot be lumped into one Multipolygon of the gltf will be created as if those many polygons are all part of the same model/feature. Cesium will also read it as one model/feature. This was one of the primary issues I fixed on the main branch last week.
So all in all, I was not able to compact the code into one loop like I wanted to, and I didn't find any ways to speed up the run time.


Getting FrankaModelInterface in franka_state_controller.cpp

I'm trying to publish jacobian from the franka_state_controller node. To do that, I need to get a FrankaModelInterface from the robot_hardware object passed into the init() function defined in franka_state_controller.cpp. I tried to add this line
auto* model_interface = robot_hardware->get<franka_hw::FrankaModelInterface>();

However, this function call always returns nullptr. Any idea what might be the cause?
Thanks

Hi !
This post dates back a bit, but I was wondering : Did you find a way to publish the jacobian matrix ?
Thanks for you answer !

I think you might need to add franka_hw::FrankaModelInterface as template parameter to he MultiInterfaceController when declaring the FrankaStateController class.

Like @rtkg suggested, that will the most likely fix of your problem. Feel free to reopen if the issue persists


Dont expand to full sidebar on hover when using fixed and sidebar-mini

I have taken a look at #336 and #840, however, neither were acceptable solutions at this time...
I am attempting to have a fixed layout with a sidebar-mini and sidebar-collapse
On hover, I want the menu to look like

Not like the following

Any ideas on how I can use a fixed layout with sidebar mini and have the menus open like the first screenshot in this post?

I also found
Because AdminLTE uses SlimScroll for the fixed layout, it is not possible to mix sidebar-mini and the fixed layout. However, there is a simple solution that AdminLTE automatically applies in case the two configurations were found active, which is expand on hover. The gif bellow shows the behavior of the sidebar when a user hovers over it and the option expand on hover is enabled. Of course, if you like this effect and prefer to use it with the normal layout, you can always enable it from the javascript options.

and I have to say, thats a great solution, however, i really could use the other solution...

found some more #660 and #420

ok, so from what I understand, the issue is, slimScroll will not display anything over the content area that it scrolls... i saw someone else say, challenge accepted, with this issue going back so many years, i am going to take a crack at it, there is a way to display over that content area, the fixed layout does that... so here we go

this works as most want with adminlte.js, app.js, and demo.js removed, so there is a solution here, we just gotta find it =]

I think i got it!
body.lte-fixed {
    header.main-header {
        position: fixed;
        width: 100%;
    }
    div.content-wrapper {
        margin-top: 50px;
    }
}

Quick fix
i guess this solve the problem just add this maybe
  PushMenu.prototype.UnExpandOnHover = function () {     $(Selector.mainSidebar).off();   };
and call the function to the app.js or demo.js to disable the hover


Set iOS badge via background handler doesn't work

The iOS > Badge > Integration docs lay out the following snippet
// Your app's background handler for incoming remote messages
firebase.messaging().setBackgroundMessageHandler(async (
  remoteMessage: FirebaseMessagingTypes.RemoteMessage
) => {
   await notifee.displayNotification(...)
   // Increment the count by 1
   await notifee.incrementBadgeCount();
})

This doesn't work for me, and I have read a few reports in not-so-related issues of this same problem.
Does this actually work for some of you ? If so, which versions are your running ?

setBackgroundMessageHandler was the culprit
notifee.incrementBadgeCount() works fine for me
setBackgroundMessageHandler sovled with
https://github.com/invertase/react-native-firebase/issues/5656

@icecapp Hi! How did you solve this issue? notifee. incrementBadgeCount is not working for me in background/quit mode


Hyper convection lathes and industrial ore processor

About the PR

Partly #21055 and partly #21462
Replaces the main useful lathe upgrades with 3 new machines. Autolathes and Protolathes now have corresponding hyper-convection varieties. These have a 30% material reduction (between T3 and T4 balancing currently) at the cost of a slightly slower print time and producing heat in the atmosphere while printing (AL upstream???)
The ore processor has had its upgrades replaced with the industrial ore processor, which has a 25% material reduction and a 50% speed increase. This one doesn't have any downsides in order to try and foster that sweet sweet salv/sci communication.
The new lathes are available in the Industrial Engineering tech (repriced to 10k) and the industrial ore processor is in the salvage tech.
Why / Balance

Lathe upgrades are some of the most blatantly dominant upgrades in the game. It's not a stretch to say that materials and protolathes/autolathes are science's sole method of actually utilizing the fruits of their research. Higher tiers of machine parts are utilized almost exclusively for their interactions with the lathe, since they are used so commonly and the cost reduction is so effective over time.
By balancing these boosts with technically manageable downsides (heat can be funneled out or nullified by spacing) the upgrades can remain reasonably balanced while additionally being more discoverable and decoupled from the experimental discipline.
The ore reclaimer upgrade has been made pretty trivially available to the benefit of salvage, who can use it to aid the entire station in terms of material supply.
Media

sprites courtesy of devi (ubaserb)

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Added the Hyper convection protolathe and autolathe to Industrial Engineering tech.


readAddress(..) failed: Connection reset by peer

My config file is shown in the above picture, the log file is shown in the following figure.

the error is : [ERROR] epollEventLoopGroup-3-8 com.amazonaws.ml.mms.http.HttpRequestHandler -
io.netty.channel.unix.Errors$NativeIoException: readAddress(..) failed: Connection reset by peer
I used multi-model-server 1.1.4 version.
The service logic is to use mms to create an algorithm service and call the service to identify the target.
This problem is sporadic，I want to know how to solve this problem， thank you.

@maaquib


Add option to reverse filter on projects page

What does this PR do and why?
Add option to reverse filter on projects page. Lets you see earliest projects on various versions (or projects with no volume if you care to see them).
Acceptance checklist

[x] I have evaluated the Approval Guidelines for this PR.
[x] I have tested this PR in all supported browsers.
[x] I have tested this PR in dark mode and light mode (if applicable).

@aeolianeth I considered this, but the bar would become crowded — especially on mobile. I think fine to leave as is for now (especially considering Sepana updates will likely change this entire section). Unless @strath-m strongly disagrees!

@filipviz great point, feeling good about it as is. Let's merge


Logo

Should we create a Hex Logo for the package and include it in the README?

@santoshenrique2021 what do you think about something like this?

#11 #12


Helm status improvements and fixes

First of all, the helm status command is super useful :), but using it, I noticed a couple of issues:

Alignment: It would be great if we could align the columns in the context of every resource.
Namespace value not shown: Reproduced in GKE and Minikube with the Helm's latest version in master.
Resources order: the order of the resources can change in consequent calls to helm status. Basically in some cases you can for example get services before deployments and vice-versa. This can be annoying if you use this command to find for example the public IP of the services,

Output example.
helm status giggly-hound
Last Deployed: Thu Sep  1 14:09:40 2016
Namespace:
Status: DEPLOYED

Resources:
==> v1/Secret
NAME                TYPE      DATA      AGE
giggly-hound-odoo   Opaque    2         4m
giggly-hound-postgresql   Opaque    1         4m

==> v1/Service
NAME                CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
giggly-hound-odoo   <IP_ADDRESS>   <pending>     80/TCP    4m
giggly-hound-postgresql   <IP_ADDRESS>   <none>    5432/TCP   4m

==> extensions/Deployment
NAME                      DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
giggly-hound-postgresql   1         1         1            1           4m
giggly-hound-odoo   1         1         1         0         4m

Yeah, the alignment is a bit wonky. It uses the kubectl to print things out, and I was hoping it would actually like you know, align them :) I'll take a look and see what can be done. Could manually spin through each resource and find the longest and set the tabs that way, but was hoping for something a bit cleaner.
Interesting on the namespace, I thought it just pulls it from the Release.Namespace, but perhaps that's not the case. I'll take a looksie...
I could sort the keys for the resource types so they would at least be alphabetical. But I'm curious why does the order of entries matter for finding the public ip for a service? Wouldn't you just find the entry for:

v1/Service and then go from there.

But, even then, I think we should support more structure in this output for easier parsing, perhaps something like -o json that would actually give the json representation of the resources. But if there's value in making sure the order is preserved, that's a much quicker fix and I can do that.

Yeah, so the Namespace is indeed pulled from the Release stashed in the persistence:
https://github.com/kubernetes/helm/blob/master/cmd/tiller/release_server.go#L194
and it's used to query the k8s for the resources, so interesting that it's not being preserved through the client.

Is the server also running the code from the master? I added the namespace feature after that, wonder if the container hasn't been updated to it yet. When I run tiller locally and connect to it, namespace is filled in.

Is the server also running the code from the master? I added the namespace feature after that, wonder if the container hasn't been updated to it yet.

@vaikas-google, you are right. My client was updated but my server was not. I apologize for that.
Definitely having this https://github.com/kubernetes/helm/issues/761 will help.

Alignment issue is now captured in #1593, which means I think we can now close this one.

Sorting is covered in #3127.


Update README.md

Added syntax highlighting for the README.

Added syntax highlighting for the README.

Thanks for the feedback!


MongoDB 4.4 check initialization error.

Output of the info page
Check Initialization Errors
mongo (2.5.0)
Instance 0
could not invoke 'mongo' python check constructor. New constructor API returned:
Traceback (most recent call last):
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\site-packages\datadog_checks\mongo\mongo.py", line 70, in __init__
      self._config = MongoConfig(self.instance, self.log)
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\site-packages\datadog_checks\mongo\config.py", line 49, in __init__
      self.clean_server_name = self._get_clean_server_name()
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\site-packages\datadog_checks\mongo\config.py", line 81, in _get_clean_server_name
      server = build_connection_string(
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\site-packages\datadog_checks\mongo\utils.py", line 21, in build_connection_string
      netloc = '{}:{}@{}'.format(quote_plus(username), quote_plus(password), host)
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\urllib\parse.py", line 867, in quote_plus
      string = quote(string, safe + space, encoding, errors)
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\urllib\parse.py", line 851, in quote
      return quote_from_bytes(string, safe)
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\urllib\parse.py", line 876, in quote_from_bytes
      raise TypeError("quote_from_bytes() expected bytes")
TypeError: quote_from_bytes() expected bytes
Deprecated constructor API returned:
__init__() got an unexpected keyword argument 'agentConfig'
Loading Errors
mongo
Core Check Loader: Check mongo not found in Catalog
JMX Check Loader: check is not a jmx check, or unable to determine if it's so
Python Check Loader: could not configure check instance for python check mongo: could not invoke 'mongo' python check constructor. New constructor API returned:
Traceback (most recent call last):
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\site-packages\datadog_checks\mongo\mongo.py", line 70, in __init__
      self._config = MongoConfig(self.instance, self.log)
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\site-packages\datadog_checks\mongo\config.py", line 49, in __init__
      self.clean_server_name = self._get_clean_server_name()
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\site-packages\datadog_checks\mongo\config.py", line 81, in _get_clean_server_name
      server = build_connection_string(
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\site-packages\datadog_checks\mongo\utils.py", line 21, in build_connection_string
      netloc = '{}:{}@{}'.format(quote_plus(username), quote_plus(password), host)
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\urllib\parse.py", line 867, in quote_plus
      string = quote(string, safe + space, encoding, errors)
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\urllib\parse.py", line 851, in quote
      return quote_from_bytes(string, safe)
   File "C:\Program Files\Datadog\Datadog Agent\embedded3\lib\urllib\parse.py", line 876, in quote_from_bytes
      raise TypeError("quote_from_bytes() expected bytes")
TypeError: quote_from_bytes() expected bytes

config:
## All options defined here are available to all instances.
#
init_config:

    ## @param service - string - optional
    ## Attach the tag `service:<SERVICE>` to every metric, event, and service check emitted by this integration.
    ##
    ## Additionally, this sets the default `service` for every log source.
    #
    # service: <SERVICE>

## Every instance is scheduled independent of the others.
#
instances:

    ## @param hosts - list of strings - required
    ## The host (and optional port number) where the mongod instance (or mongos instances for
    ## a sharded cluster) is running. You can specify a hostname, IP address, or UNIX domain
    ## socket. Specify as many hosts as appropriate for your deployment topology:
    ##
    ##   - For a standalone, specify the hostname of the standalone mongod instance.
    ##   - For a replica set, specify the hostname of a mongod instance as listed in
    ##     the replica set configuration. You need one check instance per mongod instance
    ##     in order to collect replication metrics and events.
    ##   - For a sharded cluster, you need one check instance for each mongod instance in
    ##     each shard (including the configsvr shard) as well as one additional check instance
    ##     that connects to one (or more) mongos node.
    ##
    ## If the port number is not specified, the default port 27017 is used.
    ##
    ## Even if the host is an arbiter node, you still need to provide authentication credentials
    ## as the check will create an additional connection to the primary in this specific case.
    ##
    ## Do not specify more than one host here unless you are connecting to a mongos instance
    ## and you want to balance requests performed by the agent to multiple mongos instances.
    #
  - hosts:
      - localhost

    ## @param username - string - optional
    ## The username to use for authentication.
    #
    username: datadog

    ## @param password - string - optional
    ## The password to use for authentication.
    #
    password: 123456

Additional environment details (Operating System, Cloud provider, etc):

OS: Win 10 Pro
MongoDB Version: 4.4

Steps to reproduce the issue:

Init mongodb datadog user
Modify mongo.d/conf.yaml
Restart datadog

My bad, password should be string.


Download button is missing from image pages

What

Download button is missing from image pages, although it is still available on the catalogue page.
See https://wellcome.slack.com/archives/C8X9YKM5X/p1685442176097839
Where

https://wellcomecollection.org/works/hc2bkbes/images?id=jayhbytr
Also
https://wellcomecollection.org/works/fr7j2q34/items

fixed in #9871


Pass-through used profile for command recommendations

I'd like to have the currently used profile passed through in command recommendations.  For example, the container vuln scan command output currently looks like this:
lacework vuln container scan index.docker.io alannix/my-awesome-container latest -p my-awesome-profile
A new vulnerability scan has been requested. (request_id: 151e6d4c-3948-4430-80ed-b07f0c8d548e)

To track the progress of the scan, use the command:
  $ lacework vulnerability container scan-status 151e6d4c-3948-4430-80ed-b07f0c8d548e

I think we should add the profile if possible to make it more copy/paste - like this:
lacework vuln container scan index.docker.io alannix/my-awesome-container latest -p my-awesome-profile
A new vulnerability scan has been requested. (request_id: 151e6d4c-3948-4430-80ed-b07f0c8d548e)

To track the progress of the scan, use the command:
  $ lacework vulnerability container scan-status 151e6d4c-3948-4430-80ed-b07f0c8d548e -p my-awesome-profile

JIRA: https://lacework.atlassian.net/browse/ALLY-634


Controls messed up after setting up a gamepad

When I have any gamepad configured, if I open the game without the gamepad, the controls get crazy, and I get unable to do anything, the only solutions i've found are: plug the controller in; delete settings completely
the gamepad I try to setup is a wired Xbox 360 controller

That's something to do with analog input being registered since it default position to 0 (hold up and left) instead of 128 (centered). Flick your sticks horizontal and vertical should fix the controller issue.


IDELAYCTRL ignored by VPR

The current LiteX design instantiates the IDELAYCTRL with inputs only. This causes VPR to ignore the relative nets.
By driving the output signal (RDY) to logic, VPR fails to route due to some issues with the rr_graph, ending in an unroutable situation:
# Routing
  RR Graph Nodes: 1288326
  RR Graph Edges: 7938537
Confirming router algorithm: TIMING_DRIVEN.
---- ------ ------- ---- ------- ------- ------- ----------------- --------------- -------- ---------- ---------- ---------- ---------- --------
Iter   Time    pres  BBs    Heap  Re-Rtd  Re-Rtd Overused RR Nodes      Wirelength      CPD       sTNS       sWNS       hTNS       hWNS Est Succ
      (sec)     fac Updt    push    Nets   Conns                                       (ns)       (ns)       (ns)       (ns)       (ns)     Iter
---- ------ ------- ---- ------- ------- ------- ----------------- --------------- -------- ---------- ---------- ---------- ---------- --------
Warning 186: No routing path for connection to sink_rr 245847, retrying with full device bounding box
Warning 187: No routing path for connection to sink_rr 551334, retrying with full device bounding box
Cannot route from BLK-TL-HCLK_IOI3.HCLK_IOI_IDELAYCTRL_RDY[0] (RR node: 501707 type: SOURCE location: (2,133) class: 3 capacity: 1) to BLK-TL-SLICEL.A2[0] (RR node: 551334 type: SINK location: (12,153) class: 1 capacity: 1) -- no possible path
Failed to route connection from 'serdes_test.idelayctrl' to '$auto$alumacc.cc:485:replace_alu$51.slice[0].carry4_1st_full' for net 'serdes_test.RDY' (#76)
Routing failed.
# Routing took 0.36 seconds (max_rss 2351.8 MiB, delta_rss +0.0 MiB)
Circuit is unroutable with a channel width factor of 500.
VPR failed to implement circuit

the rr_graph_walk tool did not find a possible route between the SOURCE and SINK nodes

I took a look at Vivado, and the path from IDELAYCTRL.RDY is relatively short.  It must be a graph import bug or something.

@acomodi Have you create an upstream VTR issue on this problem yet?

Not yet, I can open it right now.

Not yet, I can open it right now.

To be clear, the issue is VPR not pruning BELs with inputs and no outputs.

@acomodi : I just remembered:
  --sweep_dangling_blocks {on, off}
                    Controls whether dangling blocks are removed from the netlist (Default: on)

Try turning off sweep_dangling_blocks.  This will leave IDELAYCTRL in place, even in the absence of the RDY signal working

Right, I think I had already tried this, but I need to double-check.

  VCC          : 2
  SYN-VCC      : 2
  GND          : 1
  SYN-GND      : 1
  IOB33S       : 5
  BLK-TL-IOPAD_S : 5
  BLK-TL-HCLK_IOI3 : 1
  REG_FDSE_or_FDRE : 24
  BLK-TL-IOPAD : 10
  BLUT         : 1
  C5LUT        : 3
  B5LUT        : 1
  CEUSEDMUX    : 8
  BLK-TL-SLICEL : 6
  BLK-TL-PLLE2_ADV : 1
  SRUSEDMUX    : 8
  SLICE_FF     : 8
  CARRY4_VPR   : 6
  IDELAYE2     : 1
  CE_VCC       : 32
  IOB33_MODES  : 20
  FF_FDSE_or_FDRE : 8
  IDELAYCTRL   : 1
  SLICEL0      : 6
  SLICEM_MODES : 2
  OSERDESE2    : 1
  IOBUF_VPR    : 1
  C_DRAM       : 2
  COMMON_LUT_AND_F78MUX : 1
  FDRE         : 32
  BLK-TL-CLK_BUFG_BOT_R : 1
  PLLE2_ADV    : 1
  outpad       : 10
  inpad        : 11
  ILOGICE3     : 1
  OLOGICE3     : 1
  SR_GND       : 32
  SLICEM       : 2
  A5LUT        : 3
  B_DRAM       : 1
  BLK-TL-SLICEM : 2
  ALUT         : 3
  IOB33M       : 5
  BUFGCTRL1    : 1
  CLUT         : 2
  BUFGCTRL10   : 1
  BLK-TL-IOPAD_M : 5
  COMMON_SLICE : 8
  A_DRAM       : 2
  ISERDESE2    : 1
  lut          : 7
  BUFGCTRL0    : 1
  BUFGCTRL_VPR : 4
  IOB33        : 10
  BUFGCTRL11   : 1

        EMPTY: # blocks: 0, average # input + clock pins used: 0, average # output pins used: 0
        BLK-TL-SLICEL: # blocks: 6, average # input + clock pins used: 13.3333, average # output pins used: 8.5
        BLK-TL-SLICEM: # blocks: 2, average # input + clock pins used: 10, average # output pins used: 6.5
        BLK-TL-BRAM_L: # blocks: 0, average # input + clock pins used: 0, average # output pins used: 0
        BLK-TL-IOPAD: # blocks: 10, average # input + clock pins used: 4.1, average # output pins used: 1.3
        BLK-TL-IOPAD_M: # blocks: 5, average # input + clock pins used: 0.6, average # output pins used: 0.4
        BLK-TL-IOPAD_S: # blocks: 5, average # input + clock pins used: 0.4, average # output pins used: 0.6
        BLK-TL-CLK_BUFG_BOT_R: # blocks: 1, average # input + clock pins used: 32, average # output pins used: 4
        BLK-TL-CLK_BUFG_TOP_R: # blocks: 0, average # input + clock pins used: 0, average # output pins used: 0
        BLK-TL-PLLE2_ADV: # blocks: 1, average # input + clock pins used: 32, average # output pins used: 3
        BLK-TL-HCLK_IOI3: # blocks: 1, average # input + clock pins used: 1, average # output pins used: 0
        SYN-VCC: # blocks: 2, average # input + clock pins used: 0, average # output pins used: 0.5
        SYN-GND: # blocks: 1, average # input + clock pins used: 0, average # output pins used: 1
Absorbed logical nets 73 out of 164 nets, 91 nets not absorbed.
FPGA sized to 150 x 162 (xc7a50t-bottom-test)
Device Utilization: 0.00 (target 1.00)
        Block Utilization: 0.00 Type: BLK-TL-SLICEL
        Block Utilization: 0.00 Type: BLK-TL-SLICEM
        Block Utilization: 0.10 Type: BLK-TL-IOPAD
        Block Utilization: 0.10 Type: BLK-TL-IOPAD_M
        Block Utilization: 0.10 Type: BLK-TL-IOPAD_S
        Block Utilization: 1.00 Type: BLK-TL-CLK_BUFG_BOT_R
        Block Utilization: 0.00 Type: BLK-TL-CLK_BUFG_TOP_R
        Block Utilization: 0.50 Type: BLK-TL-PLLE2_ADV
        Block Utilization: 0.50 Type: BLK-TL-HCLK_IOI3
        Block Utilization: 2.00 Type: SYN-VCC
        Block Utilization: 1.00 Type: SYN-GND

# Packing took 0.51 seconds (max_rss 39.7 MiB, delta_rss +0.0 MiB)
Error 1:
Type: Other
File: /tmp/really-really-really-really-really-really-really-really-really-really-really-really-really-long-path/conda/conda-bld/vtr_1575656400987/work/vpr/src/pack/pack.cpp
Line: 180
Message: Failed to find device which satisifies resource requirements required: BLK-TL-SLICEL: 6, BLK-TL-SLICEM: 2, BLK-TL-IOPAD: 10, BLK-TL-IOPAD_M: 5, BLK-TL-IOPAD_S: 5, BLK-TL-CLK_BUFG_BOT_R: 1, BLK-TL-CLK_BUFG_TOP_R: 0, BLK-TL-PLLE2_ADV: 1, BLK-TL-HCLK_IOI3: 1, SYN-VCC: 2, SYN-GND: 1 (available BLK-TL-SLICEL: 2754, BLK-TL-SLICEM: 867, BLK-TL-IOPAD: 102, BLK-TL-IOPAD_M: 48, BLK-TL-IOPAD_S: 48, BLK-TL-CLK_BUFG_BOT_R: 1, BLK-TL-CLK_BUFG_TOP_R: 1, BLK-TL-PLLE2_ADV: 2, BLK-TL-HCLK_IOI3: 2, SYN-VCC: 1, SYN-GND: 1)
The entire flow of VPR took 0.79 seconds (max_rss 39.7 MiB)

@litghost  IDELAYCTRL is there, but VPR failed on VCC blocks. It seems there are too many of them packed, therefore VPR cannot find a suitable device

Maybe we can change the sweep logic to only sweep if there are 0 inputs and 0 outputs?  That second VCC is likely totally disconnected?

I'll look into that

@litghost I believe is easily doable actually, the sweeping mechanism checks only if the block has no outputs. I'll change, that, make a wip branch if it works, and integrate it.

@litghost I believe is easily doable actually, the sweeping mechanism checks only if the block has no outputs. I'll change, that, make a wip branch if it works, and integrate it.

Before you do all that, make sure it fixes our immediate issue.

Sure

I've identified a bug in the routing import that will prevent both IDELAYCTRL RST and RDY from being connected to the graph. I'm looking into a fix.

I've identified a bug in the routing import that will prevent both IDELAYCTRL RST and RDY from being connected to the graph. I'm looking into a fix.

I've isolated the issue to be an error in the aliveness check.

The IDELAYCTRL can now be correctly placed and routed, closing this issue.
Fixing PRs:

https://github.com/verilog-to-routing/vtr-verilog-to-routing/pull/1484
https://github.com/SymbiFlow/symbiflow-arch-defs/pull/1698


Created form validation

In this PR I created form validation for invalid email addresses that are in capital letters

@Kamba56 Hello, in my dashboard, its showing review pending


[win] Fix path separators for Windows.

The Windows CI added another broken test with the changes in d37fd2f85dae7f375ad2b84af052038689834bcc. This should fix it.
It should happen sooner or later in the Azure CI.

@swift-ci please smoke test


missing ligatures in this PDF

in page 1, search for "nal labeling". There's a "fi" before, but PDF.js does not report it in its text layer.
It is displayed well though.
icvgip14_table.pdf

Adobe Reader has the same issue, Mac OSX Preview has not.

Chrome PDF viewer (Pdfium) works.

On Jan 20, 2016, at 8:19 PM, Yury Delendik<EMAIL_ADDRESS>wrote:
Adobe Reader has the same issue, Mac OSX Preview has not.
—
Reply to this email directly or view it on GitHub.

Closing as fixed by PR #7550.


zig tracking issue

This issue lists what's important to hermetic_cc_toolchain from upstream zig.
Universal headers
Users of hermetic_cc_toolchain tend to run on older versions of glibc (e.g. 2.24 is still known to be used by some as of writing). The bigger the discrepancy, the higher the risk of each glibc upgrade.
Universal headers would mitigate the glibc upgrade risks. We are eaglerly waiting.
Pre-filling zig cache folder
Bazel is in principle a hermetic toolchain: it needs to know all inputs and outputs of each target (target in this context is a Makefile-target). In the ideal world, when building a binary, Bazel creates an empty directory, puts in all inputs and outputs, chroots there, and invokes the build command. The only allowed external ABI is linux kernel.
However, zig's cache folder goes against that: for Bazel, Zig cache folder is an opaque directory that we have to pass through. We must share the Zig cache, otherwise each linking action takes an extra ~1 minute (mostly for glibc and libcxx). The current hermetic_cc_toolchain instructions ask the user to pass-through /tmp from their chroot to the host:
build --sandbox_add_mount_pair=/tmp

ZIG_GLOBAL_CACHE_DIR is configured to be /tmp/zig-cache. With this flag, different Zig invocations can share the cache. This approach has the following disadvantages:

if /tmp is shared across different Unix users, it will spew "permission denied" errors.
does not work well with remote execution.
every user of hermetic_cc_toolchain is non-hermetic by design. If you look at the name of the project, it's more than ironic.
folks from Bazel community, please chime in more in the comments.

Note that we only care for glibc (including crt*), libcxx and similar. We do not care about caching of the intermediate object files that the user builds: Bazel is aware of them and caches it on it's own.
Suggestion
Create a zig sub-command that pre-compiles all the necessary "system" stubs for a given -target and -mcpu into a directory, tarball or whatever. The following zig cc commands should accept the path to the artifact as an environment variable. That way, Bazel knows all the requisites it needs to build a specific target.
Prior art: that's what rules_go does for Go. In Go case it's simpler, because there is ineed a single .a file for each GOOS (linux, windows, etc) and GOARCH (arm64, amd64, etc) combination.
Possible complications
System stubs rely on more than -target and -mcpu: optimization flags, debug info and much more are all taken into account when compiling, say, crt*.o. We need to find the balance between the number of the flags we want to support versus building the most optimized intermediate artifact that zig dynamically or statically links.

closing in favor of #153


updating title to note reference

What does this PR do?
updates a topic title because there are duplicate titles
Motivation
Help users understand that the topic is a reference topic.
Additional Notes
Change was reviewed by @paulcacheux

/merge

/merge


nuked nodes could cause intervals to be misaligned for peers attempting to pullsync with the nuked peer

⚠ Support requests in an issue-format will be closed immediately. For support, go to Swarm's Discord.
Context

Summary
When a node is nuked, it's binIds are also nuked, and after the node reboots from a nuke, the binIDs start from zero.
Peers that restart syncing with the nuked nodes will miss the certain intervals of chunks because the locally stored sync intervals will be ahead of the nuked binIDs
Expected behavior
Syncing peers should reset their own intervals when they detect that the peers has been nuked.
Actual behavior

Steps to reproduce

Possible solution
The nodes can store a timestamp of when they first starting running.
When ever there is a change in that timestamp when peers exchange cursors, the peer can reset the sync intervals of that peer.

the nuke, with the new the localstore, should not remove the binIDs, but remove everything else.

ALSO: consider removing the nuke command all together!

Removing the db nuke would probably just result in folks preserving the keys and deleting all other files.  At least with nuke, the node is somewhat in control of what gets deleted and what gets preserved.

Related #3844

5/6/23 - backlog refinement conversation summation: This is no longer needed, but the decided improvement is a 'self repairing mechanism' whereby pullsync resyncs the intervals or complete historical resync periodically (every day). This should also be randomised that this event is not happening in parallel en masse.


Created documentation of calloc

this file used influence from other files in the c docs for consistency

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

I made the changes suggested by camperBot on your behalf.
LGTM

@brianacowles I made the changes suggested by camperBot on your behalf.
LGTM

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Add testing

Removed testing at beginning for simplicty, but try to add testing back in + understand testing itself.
Perhaps I could make it optional with an input parameter
Check this cookiecutter-python-library  template

Another example repo with a few more tests
Found through this article


How to edit font etc

Forgive me for such a trivial question but was curious if there is a config file or something like that to change the font etc. Really awesome project, thank you!

Currently there is no way to set the font, or really set anything at all.

So what would be the desired approach? a css file in Library/Application Support/black-screen/user-styles.cs If the file is present it gets included, then up to the user to inspect with devtools and change what they want with this "override" style sheet?
Thats pretty much how Atom does it, even has a button to open those files for editing.
Really cool project, just started using it. I think one of the main benefits of an electron terminal is that you CAN customize anything via css, would be great to expose that.

I'd like to see some end-user-friendly solution as we know from any other program. Either Settings -> Font Size or right in the menu (font size +-).


OSS Detect Backdoor only shows one finding per file.

It looks like we're setting AllowDupTags to true, but OSS Characteristic isn't passing it off to Application Inspector, so we get the default which is just unique tags per file.

Fixed in #237


Appstudio update test-custom-default-branch-cnfq

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nwtb-on-pull-request-
creating pipelinerun test-simple-build-nwtb-on-pull-request- in namespace build-e2e-ctpq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-mcmm-on-pull-request-
creating pipelinerun test-simple-build-mcmm-on-pull-request- in namespace build-e2e-ctpq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-ufgi-on-pull-request-
creating pipelinerun test-simple-build-ufgi-on-pull-request- in namespace build-e2e-ctpq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-yrqk-on-pull-request-
creating pipelinerun test-annotations-yrqk-on-pull-request- in namespace build-e2e-ctpq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-imul-on-pull-request-
creating pipelinerun test-simple-build-imul-on-pull-request- in namespace build-e2e-ctpq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-mijr-on-pull-request-
creating pipelinerun test-annotations-mijr-on-pull-request- in namespace build-e2e-ctpq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-lqle-on-pull-request-
creating pipelinerun test-simple-build-lqle-on-pull-request- in namespace build-e2e-ctpq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-wadk-on-pull-request-
creating pipelinerun test-simple-build-wadk-on-pull-request- in namespace build-e2e-ctpq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Fixed grammatical, punctuation and spelling errors in markdown files

Closes #284

I have made changes to the markdown files to fix grammatical, punctuation and spelling errors.

This is a minor contribution and I am working towards making more significant contributions to the project in the future.

/lgtm cancel
Squashing needed.

Thanks a lot for the contribution! Can you please squash your commits into one?

Sure!
I have squashed all changes into commit 2cd45b74856d86886c0390db460e4ceb6f6ce47e

Thanks a lot for the contribution! Can you please squash your commits into one?

Sure! I have squashed all changes into commit 2cd45b7

You now have 6 commits instead of 4 in this PR. Please try again.

@tuminoid
I think I have fixed it now.

/assign @lentzi90

Thank you very much for this! /approve
I saw a few changes that may not be necessary. Specifically changing of American English to British English (e.g. center -> centre, color -> colour). Not sure if we have a preference for that really, so it may be unnecessary to do those changes. But this contains so much good that I would rather take it in directly.

Thank you for the approval.
If it is really necessary to revert the changes, I could either:

Add another commit removing those changes, then squash
Revert the changes in a new PR

What do you suggest? I think I should go for the latter.
Also, I want to work on this project under Outreachy and I am trying to improve my application to get selected.
Since you are a member of the project, can you please direct me towards some issues I could work on or help with?

No need to revert anything. This should be already merged but the CI is acting up...
/retest all

Out of ideas what's wrong with tide here, I'm going to merge it manually.


USER STORY: QA Testing

As a developer, I will want to improve improve testing coverage.

[x] #380
[ ] Write 2 tests each of your choice for 2 separate pages that don’t have any tests.

Closing as we no longer user user story issues.


Injecting  self defined service into another self defined service

Sorry for posting an angular question here. Desperate times.
I'm struggling to inject a service into another service. I'm getting undefined.
Cannot resolve all parameters for 'PerkStoreService'(undefined).
Make sure that all the parameters are decorated with Inject...

When I inject both services into a component, they work just fine.
What could be the problem?

@sfabriece you can have a look here

i've done exactly that. But i'm still getting undefined. The injected service returning undefined is exported with a barrel. I don't know if that is the reason.

Not related to the seed. You can move the discussion to gitter.


Better preserve facet attributes

Fixes #208
This is a minimal fix to make sure that facet attributes (especially facet_grid and facet_nrow) are preserved until when they are actually needed. Otherwise they might get lost during split() or rbind() etc.
This is a minimal fix (just adds three lines of code) but not a good/robust fix.

In tinyplot.default I simply store attributes(facet) early on and then stick all attributes on again before doing the facet layout. Both lines are flagged with a TODO comment.
In tinyplot.density a very similar approach already existed. However, rather than sticking on all attributes it previously restored only facet_grid but not facet_nrow. So I've only added the latter now because I wasn't sure whether there would be situations when we do not want to restore all attributes.

At the very least, both approaches should be aligned in later iterations of the code, I think. Preferably, though, we would discuss the way how we pass on the facet information more generally. But I guess it makes more sense to reconsider this question when the code has been modularized. Then we will have a better picture of how the facets fit into this.

Looks good to me. Should we include a snapshot test?

Super, thanks @zeileis. Agree that we should revisit once the modularization is finished, but this is a nice workaround in the interim :-)
Please do a tinysnapshot test or two if you can (one for "hist" and another for "density"). Happy for you or Vincent to merge thereafter.

Thanks for the feedback. I have now aligned the attribute-handling code between tinyplot.density and tinyplot.default, including the TODO comments in both places and simplifying the density method a bit.
I'm happy to have a look at snapshots but I've never done that so far. Which steps do I need to follow for this?
Also, should I bump the version to <IP_ADDRESS>0?

For snapshots, the process looks something like this:
Install dependencies:
install.packages(c("tinysnapshot", "tinytest", "svglite"))

Insert snapshot code in one of the test files. Should look something like this:
f = function() tinyplot(Temp ~ Day | Month, data = aq, type = "b")
expect_snapshot_plot(f, label = "aesthetics_type_b")

Run the test suite to create the snapshot file:
library(tinytable)
tinytest::run_test_dir()

Run again to make sure the test passes:
tinytest::run_test_dir()

Commit the snapshot to git and push.

Also, should I bump the version to <IP_ADDRESS>0?

Yes, please. Fwiw I've taken to just using a trailing ".99" to signal dev versions now (i.e. <IP_ADDRESS>), instead of incrementing in the 900s. But fine with either.

Thank you both! I've bumped the version to <IP_ADDRESS> now and added the hist/density mtcars examples from #208 as snapshots.
Apologies, Vincent, I just didn't realize that I simply have to run tinytest twice to generate the new snapshot files (and didn't take the time to read the docs). Very elegant - as usual!
I think this should be ready to merge now.

Looks great! Thanks for a useful PR.


Cannot connect to RC

Is the DJI mini SE remote control supported by the released APKs?
Are the APKs referencing dji's sdk version 4 or 5?
Is there any option to get more verbose logs or to debug the USB connection problem?
I tried on two different Android devices:
One kept displaying the red icon ("no device connected") even when dmesg was showing that the USB device got connected
And the other device was showing a purple icon.

It was a problem with the android device not supporting AOA.


fix: Removed Hardcoded Locales and Improved Error Handlings

Purpose
Added some more error handling to the application when some fields are removed from the content model and fixed some bugs reported by users
Approach
When there is fields that are missing which was required to use this application we shown a message to the user what was the exact issue is.

Testing steps
If the user deletes some of the fields from the content model, it will show the actual issue that has to be fixed.

Hey Team, Any one can review the PR, because this is the blocker for us as of now.

Hi team,
This pull request is high-priority. Could you please review it at your earliest convenience? Your timely attention to this pull request would be greatly appreciated.
Thank you!

Hi @david-shibley-contentful ,
All the changes suggested was completed and Removed the config.yml file from .circeci folder.

@vijayaragavan462 we need you to remove .contentful/.circleci/config.yml but it looks like you removed .circleci/config.yml instead. Please correct these changes

Sure, I have corrected these changes its ready now


DANGEROUS: VEHICLE_CMD_DO_REPOSITION broken on master

Had a Drone fly into a pole and smash into the ground when issuing a VEHICLE_CMD_DO_REPOSITION. I looked at the log and everything looked fine. This is broken on current master 294af5daada2fa08da8d9f1c4310779683fe9750
https://review.px4.io/plot_app?log=cc6bccfc-44a5-4eca-bc2f-7a4be13a0a7a

As you can see we were just adjusting the altitude setpoint and nothing else.
I then tested this in SITL using make posix_sitl_default gazebo and issued via QGroundControl a goto command and saw the vehicle fly to the new location, but then fly super far an a random direction for a while, then returns back to that location, wanders around a bit more, flys off in the opposite direction... then I got bored of watching.

Isn't CI supposed to catch these types of bugs? This is extremely concerning to me.

Do you know when it last worked?

Isn't CI supposed to catch these types of bugs?

Yes, but it's mainly testing core controller and mission functionality so far. Every feature like this we care about needs targeted testing to prevent these kind of regressions.

This is extremely concerning to me.

Agreed.
Fix is here: https://github.com/PX4/Firmware/pull/10760

Awesome, thanks for the quick fix.


6.2 Togepi

Togetic breeding isn't producing togepi eggs at all. They don't have the custom togepi egg sprite and hatch as togetics.

This should be fixed as of 6.2.1.


-update

Skip to content

Pull requestsIssues
Marketplace
Explore

bgoonz/BGOONZ_BLOG_2.0

Unpin

Unwatch

Fork

Starred 11

Code

Issues6

Pull requests1

Discussions

Actions

Projects1

Wiki64

Settings

Releases2

More
Edit Code
Gitpod
-update #1449
Open
preview  master
Open
-update#1449
bgoonz wants to merge 2 commits into preview from master
+1 -1
 Conversation 0 Commits 2 Checks 27 Files changed 1
Conversation

Owner
bgoonz now
|
No description provided.
|
bgoonz added 2 commits 2 minutes ago
[-update](https://github.com/bgoonz/BGOONZ_BLOG_2.0/pull/1449/commits/5fbadbf6084b399b99a305644d0285ae6773e5c8 "-update")
[5fbadbf](https://github.com/bgoonz/BGOONZ_BLOG_2.0/pull/1449/commits/5fbadbf6084b399b99a305644d0285ae6773e5c8)

[Merge branch 'master' of](https://github.com/bgoonz/BGOONZ_BLOG_2.0/pull/1449/commits/8289fa80c8c634c3280803ce315c9aea1a9e82de "Merge branch 'master' of https://github.com/bgoonz/BGOONZ_BLOG_2.0") [bgoonz/BGOONZ_BLOG_2.0](https://github.com/bgoonz/BGOONZ_BLOG_2.0)
[8289fa8](https://github.com/bgoonz/BGOONZ_BLOG_2.0/pull/1449/commits/8289fa80c8c634c3280803ce315c9aea1a9e82de)

vercel bot commented now
|
This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.
🔍 Inspect: vercel.com/bgoonz/bgoonz-blog-2-0/4Gw1HPBUSgELyY1B7nDWZrGb5VcC
✅ Preview: In Progress
|
Merge state
Add more commits by pushing to the [master](https://github.com/bgoonz/BGOONZ_BLOG_2.0) branch on bgoonz/BGOONZ_BLOG_2.0.
Show environments
This branch is pending to be deployed
1 active and 1 pending deployments
Production--- 8289fa80 Pending 1 minute ago by vercel[bot]
View deployment
github-pages--- 8289fa80 Deployed 29 seconds ago by github-pages[bot] via deploy #146
View deployment
Hide all checks14 queued, 3 in progress, 23 successful, 2 failing, 2 pending, and 1 neutral checks

Tests: nvm install-latest-npm / nvm install-latest-npm (11) (pull_request) Queued --- Waiting to run this check...
Details

Tests: release process / release (pull_request) Queued --- Waiting to run this check...
Details

Tests: release process / release (push) In progress --- This check has started...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (11) (push) Successful in 25s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (10) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (10) (push) Successful in 36s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9) (push) Successful in 23s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.2) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.2) (push) Successful in 31s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.1) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.1) (push) Successful in 22s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.0) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.0) (push) Successful in 20s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (8) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (8) (push) Successful in 21s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (7) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (7) (push) Successful in 26s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (6) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (6) (push) Successful in 31s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (6.1) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (6.1) (push) Successful in 32s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (5) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (5) (push) Successful in 33s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (5.9) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (5.9) (push) Successful in 22s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4) (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4) (push) Successful in 23s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4.6) (push) Successful in 22s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4.5) (push) Successful in 22s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4.4) (push) Successful in 23s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (3) (push) Successful in 21s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (2) (push) Successful in 24s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (1) (push) Successful in 28s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (0.12) (push) Successful in 29s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (0.10) (push) Successful in 22s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (push)
Details

Build<PHONE_NUMBER>98) --- Summary
Details

Deadpendency Check Failing after 8s --- Deadpendency Check Result
Details

Cloudflare Pages In progress --- Building
Details

LGTM analysis: JavaScript In progress --- Fetching git commits
Details

Vercel Pending --- Vercel is deploying your app
Details

docs/readthedocs.org:bgoonzblog2 Pending --- Read the Docs build is in progress!
Details

Code scanning results / CodeQL Completed in 1s --- No analyses found
Details

CommitCheck --- All commit messages are valid!
Details
This branch has no conflicts with the base branch
Merge pull request
Convert to draft or view command line instructions.

Write Preview
Close pull request
Comment
Remember, contributions to this repository should follow its contributing guidelines and code of conduct.
ProTip! Add comments to specific lines under Files changed.
Reviewers -- review nowNo reviews
Still in progress? Convert to draft
AssigneesNo one---assign yourself
Labels
None yet
ProjectsNone yet
MilestoneNo milestone
Linked issues
Successfully merging this pull request may close these issues.
None yet
Notifications
Customize
Unsubscribe
You're receiving notifications because you're watching this repository.
1 participant

Lock conversation

© 2022 GitHub, Inc.

Terms

Privacy

Security

Status

Docs

Contact GitHub

Pricing

API

Training

Blog

About

vercel bot commented now This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit. 🔍 Inspect: vercel.com/bgoonz/bgoonz-blog-2-0/4Gw1HPBUSgELyY1B7nDWZrGb5VcC ✅ Preview: In Progress


Changed offer for SQL server and Windows server.

Existing setting results in error when script is ran.  Replaced with 2022 and was able to execute the script w/o errors. Changed value to: -Offer 'SQL2022-WS2022'

Description

Checklist

[x] This pull request was tested on both of:

[x] PowerShell 5.1 (Windows)
[x] PowerShell 6.x
[x] PowerShell 7.x (Latest PowerShell)

[x] Scripts do not contain static passwords or other secret tokens.
[x] All Azure resource identifiers which must be universally unique are guaranteed to be so.

Testing information

Platform and PowerShell version: [pscustomobject]$PSVersionTable | Select-Object OS, BuildVersion, PSVersion
Az version: (Get-InstalledModule -Name Az).Version

#sign-off


The 'Machine Learning using Azure Databricks' link broken

When I click the link, it goes to https://docs.azuredatabricks.net/spark/latest/mllib/decision-trees.html, which gets a 404 error

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

ID: 6fbd2747-9807-4c9b-e07f-819570e054f8
Version Independent ID: 03a288c1-c704-a2d8-b2d1-b200f1463e38
Content: Use Azure Databricks for sentiment analysis
Content Source: articles/azure-databricks/databricks-sentiment-analysis-cognitive-services.md
Service: azure-databricks
GitHub Login: @lenadroid
Microsoft Alias: alehall

@tomz Thanks for bringing this to our notice. We are investigating into the issue and will update you shortly.


Fix formatting on API docs Client.md

Checklist

[x] Use a separate branch in your local repo (not master).
[x] Test coverage is 100% (or you have a story for why it's ok).

@googlebot I signed it!

Thanks @haoyangnz!


Admin Page

Created admin page.
Created roles for each user.
To be an admin user just change the role in the database.
The admin button is hidden on navbar you have to type it expilcitly to view Admin page and all the confessions.

how can i access the admin panel please provide a video demonstration.

@Sayak-Bhunia, okay I will do that.

hey i forgot to discuss about this feature.
just give me a demo and i will merge it.
@mohdjami

Hey @Sayak-Bhunia @Kritika30032002 , Kindly label this PR under gssoc and level3 because I have not recieved any points of this PR.


Add some rebar3 / dialyzer related build stuff

I'm about 75% done with this PR -- Do you guys have any interest in incorporating these fixes?

I realize that there are issues with rebar3 looking at the other PR. No reason we can't use rebar3 for dialyzer, and rebar2 for the real builds.
Also, bumping 17.3->17.5 is because SSL is broken af with 17.1, and 17.3

I left some comments inline.
I'll be happy to incorporate fixes, as long as travis passes, the change set is clean and specific to the problem at hand, and that it doesn't break our production environment. 😉

One of the comments that seems to have gotten lost is the issue with erldns_resolver. I had changed the error case here: https://github.com/aetrion/erl-dns/blob/master/src/erldns_resolver.erl#L143-L153 to {error, _}. The problem is that erldns_zone_cache:get_authority has the typespec to return {'error','no_question'} as well.
What do you propose we do? How should that error case be handled?

@aeden So, the build passes on Erlang 18, not on 17. Do y'all run erl-dns on 17? (If so, you really should rock 18.3, it's a free performance boost). I'd like to remove Erlang 17 from the builds. In fact, we've been running erl-dns on every [DCOS](http://dcos.io) node in Erlang 18.1, and it's been working.

(Also, a review would be chill)

Also, probably a good idea to delete ebin/erldns.app from git.

Do y'all run erl-dns on 17?

We do. I am going to talk with other team members about moving to 18 in production.

So, it looks like it already wont work under Erlang 17 anyone since you're plugged into the new time API: https://github.com/aetrion/erl-dns/blob/master/src/erldns_storage.erl#L68-L70

I'm starting to review this now. I will let you know if I have questions.

Do you have a command or set of commands I should use to get dialyzer results?

./rebar3 dialyzer should get you the results. It's part of the Travis build as well to check the typespecs.

I don't know the actual commands it runs under the hood. You can probably get them by doing DEBUG=1 ./rebar3 dialyzer, @aeden

Thanks for all the work on this, I appreciate it!


.onOffsetChange not working for CollectionView

Correction, .onOffsetChange works with the most updated code, not release 0.0.6.

As noted, the fix is in master.


Feat: Improving styling of social media icons

Describe the feature
In footer ,the social media links are not properly styled.
so i want to style them and add hover effects so that they look atrractive.
@Anushkabh please assign this issue to me under GSSOC 2024 and add appropriate labels to it.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Anushkabh please assign this issue to me under gssoc 2024

Plz assign this task to me.

Fixed in #306


Can't use special character (Ø)

Due to me using a Norwegian keyboard the typical H,J,K,L-combo is moved one character to the right, making J,K,L,Ø the most comfortable keys to use as movement keys in VI. Ø is a unique Norwegian character. In my Neovim-config I have remapped everything to use these new keys and it works fine, but when I try to use Ø in my skhdrc it crashes the app, I don't know if this is intentional and me being a doofus or if it's a bug. The same happens if I try to the other Norwegian characters Æ or Å, so it seems to be a problem with language-specific characters.

The parser only does ascii chars for key literals; you can use the keycode instead:
ø = 0x29, æ = 0x27, å = 0x21

Thank you!