FW-4418 Search Permissions

Description of Changes

Adds view permission filters to the ES search query that function the same way as the is_visible_object filter in backend/permissions/filters/view.py
Bonus change: adds python manage.py rebuild_index to the reset database script.

Checklist

[x] README / documentation has been updated if needed
[x] PR title / commit messages contain Jira ticket number if applicable
[x] Tests have been updated / created if applicable
[x] Admin Panel has been updated if models have been added or modified
[x] Migrations have been updated and committed if applicable
[x] Team Postman workspace has been updated if applicable

Reviewers Should Do The Following:

[x] Review the code changes here
[x] Pull the branch and test locally

Additional Notes
Apologies about the commit history, I had accidentally been working in the wrong branch for a bit.

Nice implementation. One small thing that I noticed missing is the update signal for sites, in a scenario where a site's visibility is changed, e.g. from 'team' to 'public', all the existing entries in the index needs to be updated with the new visibility else the search will produce incorrect results.
This will be a heavy operation since it requires fetching all entries from the index associated with a site. Similar issue was present with site_slug, and thus we moved to indexing site_id instead. Verification that visibility was changed if a site update signal is fired is recommended before continuing with the operation.


Prettier (or any async formatter) sometimes load an old version of the file

On a TS file, i could delete everything in the file, when the format run, it reload the previous version (and i need to restart neovim to fix, just closing and opening the file doesn't fix it)

how can i reproduce this issue ?

how can i reproduce this issue ?

its very random, try to create new buffer, save, reload, paste something, do undo, its very hard to happen (happened 2 times in one week of work)

let me try it with many times. hope i can reproduce this.

tried this many times and unable to repro.... think we should close issues with limited repro otherwise they'll be up forever.

Screencast from 2023-08-23 16-40-28.webm
Could repro, but I imagine this would hardly happen in real work, right?
If more people complain about this maybe buffer locking needs to be added.

@xiaoshihou514 can you be clear about how this happens? Is it that as a formatter runs and you try to save changes they're lost?

@barrett-ruth fyi

I think we ought to implement buffer locking. I think this would happen just enough to be relevant and we ought to just fix it now.

However, this seems like a core problem of asychronous formatting in general... if we're locking a buffer during asynchronous formatting isn't that quite similar to the whole process being synchronous?

Well you could still move around, open splits, etc.

Furthermore, how does vim.lsp.buf.format with { async = false } handle this? These are things to consider.

It just blocks and freezes the editor, much like read !prettier "%" or something similar.

a clean reproduce step here  ?

edit after the format, because async the buffer  content changed so the buffer lose new lines right ? there need  check changedtick


Catch scroll event regardless of where it is dispatched ?

Hello, thanks for this awesome tooltip package.
I am currently facing a small issue where scroll events that are not directly on the document object are not registered by the package and hence breaks the position of the tooltip relative to the anchor.
The anchor scrolls away leaving the tooltip. Thus it doesn't recalculate its position relative to the anchor.
window.addEventListener('scroll', ...) this line here
I think we are missing the third argument which is very important in this case.
window.addEventListener('scroll', this.listenResizeScroll, true)
it tells the browser to capture the event on dispatch, even if that event does not normally bubble.
In effect, we capture scroll events everywhere which may or may not be on the document but elements in the subtree and position the tooltip correctly.
I can open a PR to fix this?

I am closing this issue because that doesn't seem to be the case. It doesn't recalculate it's position because it is not being done :)

Hey @Gregjarvez I'll take a look at this soon, been a busy fall, but I'm getting back to stuff now.

Hey @Gregjarvez I wasn't able to repro this, can you explain more how this happens, is it with internal scroll bars?  Can you fork the demo pen with a repro?

Hi @Gregjarvez I belive this may be similar to the issue with: https://github.com/bsidelinger912/react-tooltip-lite/issues/58.  Where we didn't listen for scrolls on internal scrollbars.  That's been fixed with v1.11.2, please let me know if this addresses your issue.


refactor (vscode): migrate from submodule to subtree

WIP: This PR changes our current approach of using vscode as a submodule to using it as a subtree.
Changes

remove submodule
add subtree
apply patch
remove old files and scripts

ci/dev/vscode.patch
ci/dev/patch-vscode.sh and vscode:patch from package.json
ci/dev/diff-vscode.sh and vscode:diff from package.json

Remaining Todos

[ ] update ci/dev scripts
[ ] update ci/build scripts
[ ] update docs

Screenshots
todo
Checklist

[ ] tested locally
[ ] updated the docs

Fixes #1587
Many thanks to @SPGoding for testing out this approach and outlining the steps to start this effort! 🎉

Okay made some initial progress. The migration piece was the easy part. The harder part is understanding how to update our vscode.sh script.
How do we keep our subtree up to date with vscode?
Thinking a bit out loud here...

when someone is making a change to our codebase, they don't need to run any funky commands because the vscode code is already there (thanks to the subtree)
when we (maintainers) need to update the subtree (as in update vscode from 1.5.1 to 1.5.2, how do we do it?

If it were a normal repository, like a fork I'd probably fetch the latest changes then rebase my changes on top of those. But subtree's don't support rebase, only merge strategies. So we have to figure that out.
Resources that may help:

How to use the subtree merge strategy
Why doesn't Git subtree use rebase?
bash script: git rebase to subtree

Maybe something like this?
  # Add vscode as a new remote
  # TODO only do once
  git remote add -f vscode https://github.com/microsoft/vscode.git

  # Prep to merge with version we're currently on
  git merge -s ours --no-commit --allow-unrelated-histories vscode/release/1.51

  # Read changes to our sub directory
  git read-tree --prefix=lib/vscode -u vscode/release/1.51

  # Commit changes
  # git commit -m "chore: update vscode"

  # git pull -s subtree Bproject master

I think all we need to do in vscode.sh is install VS Code's Node modules. Actually maybe we should just move that to postinstall in package.json so you don't need to run a separate yarn vscode.
That way development would just be:
yarn
yarn watch

As for updating VS Code once the remote is added I think all we need is something like:
git subtree pull --prefix lib/vscode vscode release/1.52 --squash --message "Update VS Code to 1.52.1"

I'm not sure we need a script for that, maybe we just add it to the contributing docs.

Actually maybe we should just move that to postinstall in package.json so you don't need to run a separate yarn vscode

Great idea! Didn't think of that.

I'm not sure we need a script for that, maybe we just add it to the contributing docs.

Ah, if it's only one line, I agree, we probably don't need that. I'll add to contributing. Thanks @code-asher

Alright a few errors I'm seeing.
Error in fmt script
In our ci / fmt workflow, it's upset with the let declaration inside the for loop. Maybe when that gets called $1 is undefined?
lib/vscode/src/vs/base/node/cpuUsage.sh:9:11: = must follow a name

The code there reads:
#!/bin/bash

function get_total_cpu_time() {
  # Read the first line of /proc/stat and remove the cpu prefix
  CPU=(`sed -n 's/^cpu\s//p' /proc/stat`)

  # Sum all of the values in CPU to get total time
  for VALUE in "${CPU[@]}"; do
    let $1=$1+$VALUE
  done
}

(see file on remote)
Error in lint script
It seems like it can't find our .eslint.yaml or rather the node_modules where these rule definitions come from.
/github/workspace/lib/vscode/test/automation/src/explorer.ts
  1:1  error  Definition for rule 'code-no-unused-expressions' was not found        code-no-unused-expressions
  1:1  error  Definition for rule 'code-translation-remind' was not found           code-translation-remind
  1:1  error  Definition for rule 'code-no-nls-in-standalone-editor' was not found  code-no-nls-in-standalone-editor
  1:1  error  Definition for rule 'code-no-standalone-editor' was not found         code-no-standalone-editor
  1:1  error  Definition for rule 'code-no-unexternalized-strings' was not found    code-no-unexternalized-strings

Re: Feedback in Slack from @code-asher

we might need to use something other than git ls-files for determining what to run our formatter and linter on
we can do this by adding | grep -v "lib/vscode" to all the git ls-files invocations

For the space issue it seems to run out when copying to the GCP directory (build-packages.sh). I think we just remove that for now since we don't currently upload to GCP anyway and I think it might make sense to add a post-publish script that uploads the releases from GitHub to GCP instead of doing it this way anyway.

For the space issue it seems to run out when copying to the GCP directory (build-packages.sh). I think we just remove that for now since we don't currently upload automatically to GCP anyway

Good call. I removed it in 96576bd

and I think it might make sense to add a separate step that uploads the releases from GitHub to GCP instead of doing it this way anyway.

Do you want me to open an issue for us to do that in the future?

$ ./ci/build/build-packages.sh
done (release-packages/code-server-3.7.4-linux-arm64)
guessing packager from target file extension...
using deb packager...
nfpm: error: cannot add data.tar.gz to deb: write release-packages/code-server_3.7.4_arm64.deb: no space left on device

That's interesting...It's still trying to add it. I must have removed the wrong step.

Do you want me to open an issue for us to do that in the future?

No need, we're tracking it internally.

That's interesting...It's still trying to add it. I must have removed the wrong step.

Nah you got it right, it's just running out of space later down the line now.

May submodules burn in hell!


Fix paragraph actions dropdown styling for Firefox

Before on Firefox

After on Firefox

Tested and LGTM.
Merge if there're no more discussions


Amend KEGG database parser

commit
7d9ab00
Proposed change
Parsing KEGG database works for KEGG Orthology but is still not very reliable. Does not really work for the other KEGG database or works poorly. The main problem is probably a proper way of handling multiline in regex.

The KEGG Orthology database approach should probably be different. Using link/genes/KO fetches all the genes and KEGG Organism ID from given KO. That may give more complete information without the overhead #54


use absolute GH link for benchmark dir

Description :
It seems that in Github relative links only work when you point to a file, update with an absolute GH link to benchmark dir

Well spotted, thanks! :pray:


Crash on self signed "certificate_bundle"

Hi,
I found the webhook v0.1.1 & v0.1.2 crashing:
thread 'main' panicked at src/main.rs:132:10:
2024-03-18T11:04:40.198062390+01:00 called `Result::unwrap()` on an `Err` value: Error { tag: Tag::Default, profile: Some(Profile(Uncased { string: "default" })), metadata: None, path: [], kind: MissingField("certificate_bundle"), prev: None }
2024-03-18T11:04:40.198068050+01:00 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

My Helm chart looks like this:
apiVersion: helm.toolkit.fluxcd.io/v2beta1
kind: HelmRelease
metadata:
  name: external-dns
  namespace: external-dns
spec:
  chart:
    spec:
      chart: external-dns
      reconcileStrategy: ChartVersion
      sourceRef:
        kind: HelmRepository
        name: external-dns
      version: 6.x.x
  interval: 12h0m
  values:
    provider: webhook
    registry: noop
    extraArgs:
      webhook-provider-url: http://localhost:8800
    managedRecordTypesFilters:
      - A
    sources:
      - ingress
      - service
    sidecars:
      - name: opnsense-unbound-external-dns-webhook
        image: ghcr.io/jobs62/opnsense_unbound_external-dns_webhook:v0.1.2
        ports:
          - containerPort: 8800
            name: http
        # livenessProbe:
        #   httpGet:
        #     path: /healthz
        #     port: http
        #   initialDelaySeconds: 10
        #   timeoutSeconds: 5
        # readinessProbe:
        #   httpGet:
        #     path: /healthz
        #     port: http
        #   initialDelaySeconds: 10
        #   timeoutSeconds: 5
        env:
          - name: OPNSENSE_BASE
            value: "https://<IP_ADDRESS>/"
          - name: OPNSENSE_ALLOW_INVALID_CERTS
            value: "true"
          - name: OPNSENSE_DOMAIN_FILTERS
            value: "[\"exmaple.com\"]"
          - name: OPNSENSE_KEY
            valueFrom:
              secretKeyRef:
                name: opnsense-api
                key: key
          - name: OPNSENSE_SECRET
            valueFrom:
              secretKeyRef:
                name: opnsense-api
                key: secret

Do you want a full stack trace? I also disabled the probes, since they wouldn't become healthy, but the webhook still works.

This is also present in 0.1.1 not only 0.1.2

i have been able to reproduce it. i will have a look.

could you try v0.1.3-rc1 tag ?

Yep, no more crash so far.


Add splash/loading screen

Should come up quickly to cover for potentially slow load

Not currently needed to cover load times, but it might be nice to add a splash of color.


Update configure-measure-group-properties.md

Description for DataAggregation property was outdated - for current SSAS versions, DataAggregation is a supported setting. The sentence I added is a description from the 20768 training.

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

@amirchandandani Can you take a look at the proposed change? If it looks ok, please let me know and I'll sign off on this PR.

@vstrien Thanks for contributing. However, this property is actually not used. The engine ignores it. What description from the 20768 training are you referring to? Could you provide me with more detail about this training and who provided it?
Thanks

@vstrien Thanks for contributing. However, this property is actually not used. The engine ignores it. What description from the 20768 training are you referring to? Could you provide me with more detail about this training and who provided it?
Thanks

Sure, I am an MCT, and this is right in the Microsoft-provided Trainer Handbook for the 20768 course, page 111.

@vstrien Thanks for letting me know. You're the second person in two weeks who suggested the same change. Now I know where it's coming from. It will take a while to look into this. Let's leave this issue open while I do some digging. Thanks for your patience.

@vstrien I'm not able to get into the actual course, but I found it described here, https://www.microsoft.com/en-us/learning/course.aspx?cid=20768. This course is terribly outdated. I remember being consulted for help with creating the DAX module, probably seven or eight years ago. I strongly recommend not suggesting or presenting this course. You might want to bring it up with your contacts in MS Learning. I'll do what I can internally to get this course removed. I cannot merge your suggested change as stated earlier. If you could close this PR, that would be great. Again, thanks for the information. I'm sorry I cannot provide a better outcome.

As requested by @Minewiskan, proposed update was incorrect.


add metric for tsdb size retention bytes

Signed-off-by: YaoZengzeng<EMAIL_ADDRESS>fixes: prometheus/prometheus#5790

@brian-brazil PTAL

Why do you need this?

@krasi-georgiev Just fix issue prometheus/prometheus#5790

@brian-brazil Updated, naming is always hard for me 😄
But as @krasi-georgiev said, I'm also wondering if this metric is really necessary.

It's generally useful to have metrics like this for flags that set limits.

Yes I understand the use case, but seems a bit strange to add metrics that are constants like - version numbers, config flags etc.
Anyway if @brian-brazil 	is ok with this feel free to merge.

Config flags like this which are limits are useful for alerting (though this specific one isn't quite, as retention handling is automatic).

Can you please add a test for this metric as well to ensure that the value is the same as the set value when opening a db.
look at the db size tests for some ideas.

@brian-brazil @krasi-georgiev PTAL

:+1:

@brian-brazil @krasi-georgiev Any new comments?

nothing on my side
LGTM


Ustreamer

Motivation for this change
This package is useful (not only) on raspberry pi 4 devices in combination with a tc358743 HDMI-to-CSI-2 converter.
https://github.com/NixOS/nixos-hardware/pull/269
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 124955 at 7b1ab204 run on x86_64-linux 1

1 package built successfully:

ustreamer

2 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/video/ustreamer/default.nix:18:3:
   |
18 |   installPhase = ''
   |   ^

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/applications/video/ustreamer/default.nix:33:5:
   |
33 |     license = licenses.gpl3;
   |     ^


Question on next release

Hi Timon
Thanks for all. One question, when is the next release going out? E.g. 0.14 ?
Thanks

That's long overdue due, I plan to get it ready the next days.

Hi Timon!, sorry to be pushy about this, but, do you have any estimated time to release ? as the home bridge code depends on it to support the Wemo Dimmer. Thanks Much!
Martin.

Thanks for the friendly reminder :-) v0.14.0 is out now.


chore(main): release 0.2.0

:robot: I have created a release beep boop
0.2.0 (2024-05-13)
Features

generate interval-based hierarchies (8500823)

Bug Fixes

record suppression supported (f050a6e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/IFCA-Advanced-Computing/anjana/releases/tag/v0.2.0 :sunflower:


go/printer: consider using last position for nodes with no position

Reminder issue to experiment with using the last known (end) position for nodes where we don't have a position (and where NoPos is not used to indicate absence of a token).
From e-mail conversation with @aclements:

On Wed, Nov 28, 2018 at 8:34 AM Austin Clements<EMAIL_ADDRESS>wrote:
Thanks! Assigning the new Specs I created in the const block to the position of the const worked perfectly.
Is there a reason we don't treat NoPos nodes as having the position of the nearest ancestor when printing?

NoPos sometimes indicates that a token is absent (e.g., the position of '(' or ')' is NoPos if those parentheses are missing).
But I suppose in other cases it might work - interesting thought. I'm slightly worried that making such a change might break existing carefully tuned code though. But worthwhile an experiment.

I was discussing this a bit with @rsc and the thought we had was that since NoPos indicates (usually) that you don't know the position of the token, then the printer should also think of that token as consuming no space. Hence, the printer simply shouldn't update its notion of where it is in the output file.


Fix for Awaited inference

This addresses an issue where we are potentially too aggressive when unwrapping a promise-like type when inferring the return type of an async function from its function body.
Fixes #48320

@typescript-bot test this
@typescript-bot user test this


Publish Script adds SET ANSI_NULLS OFF;

modify any database object in project, e.g., save change to a stored procedure
perform Schema Compare against or opt to Publish to a target database
Generate script

Result:
Prior to the ALTER PROCEDURE ... block, 2 SET statements are scripted:
SET QUOTED_IDENTIFIER ON;
SET ANSI_NULLS OFF;
This latter setting is non-standard and is not what a tool like Visual Studio generates during its Publish operations.  I can't find where to control this behavior from ADS.
It should be noted that the ANSI_NULLS OFF setting can be highly detrimental to data operations, so a warning should be explicitly communicated to the user rather than have it tacitly embedded into the publish script.

Does unselecting the option "Ignore ANSI NULLS" not work?

Please refer to this other ANSI_NULLS OFF issue
De-selecting the "Ignore ANSI_NULLS" option appears to be rather counterintuitive as it generates a lot of verbosity switching ANSI_NULLS OFF several times for nearly each object in the script, so this is not the answer:
GO
SET QUOTED_IDENTIFIER ON;

SET ANSI_NULLS OFF;

GO
SET QUOTED_IDENTIFIER ON;

SET ANSI_NULLS OFF;

GO
BEGIN TRANSACTION;

SET TRANSACTION ISOLATION LEVEL SERIALIZABLE;

SET XACT_ABORT ON;

CREATE TABLE [dbo].[tmp_ms_xx_Foo] (


Does not compile

The project does not compile using the provided INSTALL file, it complains about a checksum not passing as shown in this gist.
I'm running 64-bit Arch linux with the latest TeXLive package.
latex --version returns:
pdfTeX 3.14159265-2.6-1.40.17 (TeX Live 2016/Arch Linux)
kpathsea version 6.2.2
Copyright 2016 Han The Thanh (pdfTeX) et al.
There is NO warranty.  Redistribution of this software is
covered by the terms of both the pdfTeX copyright and
the Lesser GNU General Public License.
For more information about these matters, see the file
named COPYING and the pdfTeX source.
Primary author of pdfTeX: Han The Thanh (pdfTeX) et al.
Compiled with libpng 1.6.24; using libpng 1.6.26
Compiled with zlib 1.2.8; using zlib 1.2.8
Compiled with poppler version 0.46.0

It seems the actual package compiles fine but the documentation fails. The installation script is referencing a directory kauguide that doesn't actually exist in the repo, perhaps this is related to the error? Nevermind, kauguide.tex is a file.

Hi Adam,
I missed to adjust the checksum in the kautools.dtx after recent changes. This is now done in aba8db2fb3b1f87ad3df5335693d69e55ef0febb.
Thanks for pointing out!
Cheers,
Stefan.

Did it work?

Pulled the changes and now it installs successfully complete with the documentation!
Unrelated to this issue but is the KAU logo supposed to be shown in any of the templates? There are some references to kau-logo-tryck but it's not included from what I've seen.


Embedding improvements

This PR allows the creation of embedding as follows:

sumary PR:

Added sc.pl.embedding to the API. This makes possible to select any 'basis' for plotting. This name is consistent with other functions that accept 'basis' as parameter: sc.pl.embedding_density and scvelo.velocity_embedding (#762)
vmax and vmin now could be a list to indicate individual vmin/vmax range when using multiple plots (#775).
vmax and vmin can indicate a quantile using the format qNN to indicate the NN quantile.  Eg. vmax=['q90', 'q80'] (#775)
vmax and vmin can be a user defined function. (#775)
added the add_contour parameter which produces figures like the one on top (I will be happy to know if there is a better name for this type of scatter plot as contour may refer to some specific type of plots)

TODO:

[x] Correct style
[x] Pass tests
[x] Add tests for new parameters
[x] Find better name for 'contour'

Awesome! maybe “outline” would be a good word?

This looks very cool! I wonder if there could also be an option to put a contour between cluster that have a fluid boundary (for example what I assume are clusters 6 and 7 above). I like the above plot, but I imagine this might help with colours being difficult to discern in certain colour maps.

@flying-sheep 'outline' could be good. We already have font outline that works similar. Can you take a look at the new type hints that I added. I am not sure if I did it right?
@LuckyMD you mean to do something like this (top image)?

Exactly!

@LuckyMD I will think about it. The implementation does not seem trivial at the moment and will require yet a new parameter which I am hesitant to add, unless add_contour can accept something line add_contour='separate categories'. But at least, using overlapping images and without further modifications the issue is solved.

The typing is pretty good! There’s one rule I follow, which is to be specific:

If your function has parameter a, does for elem in a, and expects elems to be strs, you can say Iterable[str]. If you use a[i], say Sequence[str]. You don’t want to artificially limit the user by saying you need a List[str] if a Tuple[str] can be passed or even any Iterator[str] is sufficient.
If you say what you return, be concrete, e.g. List[str]. You know what exact type you return.
If you accept a callable, specify its signature: Callable[[ArgType1, ArgType2], RetType]

So you should change

callable→Callable[[???], ?]
Sequence→Sequence[?]
Optional[dict]→Optional[Mapping[?, ?]]

Also stylewise: Once ( and ) aren on separate lines, never have anything after (, and before ):
def _get_vmin_vmax(
    […]
    color_vector: Sequence[float],
):
    '''
    […]

logg.error(
    "The parameter […]"
    […]
    "of plots."
)

@flying-sheep I quite like your style guidelines. It might be a good idea to designate a particular function that does it well and is complex enough to include all of these options. That way it's easy to follow style when writing a new function. Something like contributors template.

Good idea. Done in 740c4a510ec598ab03ff3de1d9b1c091f0aac292

This looks great!
A few ideas:

For having an outline to separate overlapping clusters, I don't think I like that one of the outlines would be plotted over the other cluster. In the plots shown above (https://github.com/theislab/scanpy/pull/794#issuecomment-523515331) I think the upper image is less clear about the extent of the overlap than the lower one, and suggests a greater importance of group 3. Maybe there could be some indication of ambiguity for the region of overlap?
For the string based quantile selection, is there another package which allows writing operations like this? My concern is that string based DSLs can get messy. It would be nice to make sure we're choosing a unambiguous spec which we can extend in the future and use in other functions. An example of a spec would be SQL reduction operations (like PERCENTILE_DISC), but hopefully there would be something less verbose.
For the basis argument, could we not require the key in obsm start with X_? I'm thinking the key would just go through a check like:

if basis in adata.obsm:
    basis_key = basis
elif f"X_{basis}" in adata.obsm:
    basis_key = f"X_{basis}"
else:
    raise KeyError(
        f"Could not find entry in `obsm` for '{basis}'.\n"
        f"Available keys are: {list(adata.obsm.keys())}."
    )

@flying-sheep regarding the typing I ended up with the following line:
 vmax: Union[float, str, Callable[[Sequence[float]], float], Sequence[Union[str, float, Callable[[Sequence[float]], float]]], None] = None,

Does it looks ok to you?

This looks very cool Fidel. It's not a big deal but if we use numbers in 0-100 range for vmax-vmin, it's a percentile, so p80 makes more sense.

I ended up with […] Does it looks ok to you?

Yeah! You could simplify though:
VMinMax = Union[str, float, Callable[[Sequence[float]], float]]

def ...(
    vmin: Union[VMinMax, Sequence[VMinMax], None] = None,
    vmax: Union[VMinMax, Sequence[VMinMax], None] = None,
):

It's not a big deal but if we use numbers in 0-100 range for vmax-vmin, it's a percentile, so p80 makes more sense.

you can also be more or less precise: q001→0.001, q9→0.9

@ivirshup I looked around and could not find any example in other packages that use input strings as in this case. I agree that the current solution is not ideal, but other solutions seem more complicated. For example, we can add a new parameter called percentile that then interprets vmin and vmax as percentiles/quantiles.  However, this limits the option to mix different types of values.
In seaborn they use the parameter robust to set vmin and vmax as the .02 and .98 quantiles but the quantiles can not be changed.
Unless we have a strong opinion against this solution I suggest we keep it.

Looks great! Optimally we’d add a test image like the bottom one in https://github.com/theislab/scanpy/pull/794#issuecomment-523515331, but I’d be up for merging this as-is.

For having an outline to separate overlapping clusters, I don't think I like that one of the outlines would be plotted over the other cluster. In the plots shown above (#794 (comment)) I think the upper image is less clear about the extent of the overlap than the lower one, and suggests a greater importance of group 3. Maybe there could be some indication of ambiguity for the region of overlap?

I see your point regarding the accurate representation of what is shown. My thoughts were more along the lines of making it easy to distinguish clusters. That would be particularly useful when you have >20 clusters that are hard to distinguish by colours alone. And the overlaps may not be as important for a large-scale overview.

Great stuff, thank you!

Hmm, there’s this code in embedding which probably doesn‘t work as List here is typing.List instead of collections.abc.List.
if not isinstance(vmin, List):
    vmin = [vmin]

Don’t know how I missed this. The correct check would of course be:
if isinstance(vmin, str) or not isinstance(vmin, collections.abc.Sequence):
    vmin = [vmin]

I am surprised that this is coming out. I thought I had solved the issue as I don't get the error. Thanks @flying-sheep for addressing this.

Hmm, the code in this PR isn’t the current code anymore. I’m fixing it in #839


split out actual combining logic into a module

the gulp plugin should just feed the css into this module and receive out new css. this will clean up the code base a lot and let other people use the code for other build systems

:+1:
Seems like this project and https://github.com/frontendfriends/node-combine-mq (https://github.com/frontendfriends/gulp-combine-mq) have similar goals.


Object.create example output might be misleading

The Object.create() example shows the creation of a new object using obj as prototype. As output, a copy of obj is shown:

While the properties are of course still accessible through the prototype chain, it might be a good idea to explicitly show the __proto__ property, like chrome does:

Hm, I think that might be too confusing for beginners, which is the point of this resource. Thanks though!


Set DOCKER_HOST through testcontainers.properties

For some reasons, I cannot set the environment variables on the CI machine. Can I set DOCKER_HOST through testcontainers.properties to access the Docker server?
It seems only support environment variables but not properties files
https://www.testcontainers.org/features/configuration/#customizing-docker-host-detection

Properties as follow in my classpath are not woring
# testcontainers.properties
DOCKER_HOST=tcp://X.X.X.X:2375
docker.client.strategy=org.testcontainers.dockerclient.EnvironmentAndSystemPropertyClientProviderStrategy
# docker-java.properties
DOCKER_HOST=tcp://X.X.X.X:2375

Today I pushed my code to repository and then the Jenkis was running. I got the exception as below.
2021-03-21 19:15:53.996  WARN 12056 --- [           main] o.t.u.TestcontainersConfiguration        : Attempted to read Testcontainers configuration file at file:/root/.testcontainers.properties but the file was not found. Exception message: FileNotFoundException: /root/.testcontainers.properties (No such file or directory)
2021-03-21 19:15:54.010  INFO 12056 --- [           main] .t.d.DockerMachineClientProviderStrategy : docker-machine executable was not found on PATH ([/usr/local/sbin, /usr/local/bin, /usr/sbin, /usr/bin, /sbin, /bin, /usr/local/jeci-pts, /usr/local/jvm/jdk1.8.0_151/bin, /usr/local/jvm/jdk1.8.0_151/jre/bin, /usr/local/maven/apache-maven-3.5.4/bin, /usr/local/jeci-pts/jeci-pts])
2021-03-21 19:15:54.052 ERROR 12056 --- [           main] o.t.d.DockerClientProviderStrategy       : Could not find a valid Docker environment. Please check configuration. Attempted configurations were:
2021-03-21 19:15:54.052 ERROR 12056 --- [           main] o.t.d.DockerClientProviderStrategy       :     UnixSocketClientProviderStrategy: failed with exception InvalidConfigurationException (Could not find unix domain socket). Root cause NoSuchFileException (/var/run/docker.sock)
2021-03-21 19:15:54.053 ERROR 12056 --- [           main] o.t.d.DockerClientProviderStrategy       : As no valid configuration was found, execution cannot continue
2021-03-21 19:15:55.053 ERROR 12056 --- [           main] c.z.h.p.HikariPool                       : HikariPool-1 - Exception during pool initialization.

java.lang.IllegalStateException: Could not find a valid Docker environment. Please see logs and check configuration
	at org.testcontainers.dockerclient.DockerClientProviderStrategy.lambda$getFirstValidStrategy$7(DockerClientProviderStrategy.java:215) ~[testcontainers-1.15.2.jar:?]
	at java.util.Optional.orElseThrow(Optional.java:290) ~[?:1.8.0_151]
	at org.testcontainers.dockerclient.DockerClientProviderStrategy.getFirstValidStrategy(DockerClientProviderStrategy.java:207) ~[testcontainers-1.15.2.jar:?]
	at org.testcontainers.DockerClientFactory.getOrInitializeStrategy(DockerClientFactory.java:136) ~[testcontainers-1.15.2.jar:?]
	at org.testcontainers.DockerClientFactory.client(DockerClientFactory.java:178) ~[testcontainers-1.15.2.jar:?]
	at org.testcontainers.LazyDockerClient.getDockerClient(LazyDockerClient.java:14) ~[testcontainers-1.15.2.jar:?]
	at org.testcontainers.LazyDockerClient.authConfig(LazyDockerClient.java:12) ~[testcontainers-1.15.2.jar:?]
	at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:310) ~[testcontainers-1.15.2.jar:?]
	at org.testcontainers.jdbc.ContainerDatabaseDriver.connect(ContainerDatabaseDriver.java:118) ~[jdbc-1.15.2.jar:?]
	at com.zaxxer.hikari.util.DriverDataSource.getConnection(DriverDataSource.java:121) ~[HikariCP-3.4.2.jar:?]
	at com.zaxxer.hikari.pool.PoolBase.newConnection(PoolBase.java:354) ~[HikariCP-3.4.2.jar:?]
	at com.zaxxer.hikari.pool.PoolBase.newPoolEntry(PoolBase.java:202) ~[HikariCP-3.4.2.jar:?]

It seems like reading ~/.testcontainers.properties directly but not reading classpath:testcontainers.properties, does any method to resolve this problem ?

I have read the relevant code. In this line testcontainers only read userProperties, but missed classpathProperties
https://github.com/testcontainers/testcontainers-java/blob/8965712f239ae3c449cec579264af5e79265f314/core/src/main/java/org/testcontainers/utility/TestcontainersConfiguration.java#L171
It should use the following line instead
    public String getDockerClientStrategyClassName() {
        return getEnvVarOrProperty("docker.client.strategy", null);
    }

@rnorth Hey, I'm trying to fix this problem to read DOCKER_HOST from classpath, but there seems to be a test case 'shouldNotReadDockerClientStrategyClasspathProperties' to avoid this, what's the historical factor?

Hi,
I met the same issue with yours.
Did you resolve it finally?
Due the CI machine, my code only can use the classpath way.
0:11:25  02:11:24.777 [main] INFO org.testcontainers.utility.ImageNameSubstitutor - Image name substitution will be performed by: DefaultImageNameSubstitutor (composite of 'ConfigurationFileImageNameSubstitutor' and 'PrefixingImageNameSubstitutor') 10:11:25  02:11:24.785 [main] DEBUG org.testcontainers.utility.PrefixingImageNameSubstitutor - No prefix is configured 10:11:25  02:11:24.786 [main] DEBUG org.testcontainers.utility.ImageNameSubstitutor - Did not find a substitute image for docker.elastic.co/elasticsearch/elasticsearch:7.9.2 (using image substitutor: DefaultImageNameSubstitutor (composite of 'ConfigurationFileImageNameSubstitutor' and 'PrefixingImageNameSubstitutor')) 10:11:25  02:11:24.803 [main] DEBUG org.testcontainers.dockerclient.RootlessDockerClientProviderStrategy - $XDG_RUNTIME_DIR is not set. 10:11:25  02:11:24.803 [main] DEBUG org.testcontainers.dockerclient.RootlessDockerClientProviderStrategy - '/root/.docker/run' does not exist. 10:11:25  02:11:24.835 [main] DEBUG org.testcontainers.dockerclient.RootlessDockerClientProviderStrategy - '/run/user/0' does not exist. 10:11:25  02:11:24.836 [main] DEBUG org.testcontainers.dockerclient.DockerClientProviderStrategy - Trying out strategy: UnixSocketClientProviderStrategy 10:11:25  02:11:24.837 [main] DEBUG org.testcontainers.dockerclient.DockerClientProviderStrategy - UnixSocketClientProviderStrategy: failed with exception InvalidConfigurationException (Could not find unix domain socket). Root cause NoSuchFileException (/var/run/docker.sock) 10:11:25  02:11:24.839 [main] INFO org.testcontainers.dockerclient.DockerMachineClientProviderStrategy - docker-machine executable was not found on PATH ([/opt/java/openjdk/bin, /usr/local/sbin, /usr/local/bin, /usr/sbin, /usr/bin, /sbin, /bin, /opt/apache-maven/bin]) 10:11:25  02:11:24.840 [main] ERROR org.testcontainers.dockerclient.DockerClientProviderStrategy - Could not find a valid Docker environment. Please check configuration. Attempted configurations were: 10:11:25  02:11:24.840 [main] ERROR org.testcontainers.dockerclient.DockerClientProviderStrategy -     UnixSocketClientProviderStrategy: failed with exception InvalidConfigurationException (Could not find unix domain socket). Root cause NoSuchFileException (/var/run/docker.sock) 10:11:25  02:11:24.840 [main] ERROR org.testcontainers.dockerclient.DockerClientProviderStrategy - As no valid configuration was found, execution cannot continue

Hi, I met the same issue with yours. Did you resolve it finally? Due the CI machine, my code only can use the classpath way.

Using maven-surefire-plugin
<plugin>
    <artifactId>maven-surefire-plugin</artifactId>
    <configuration>
        <environmentVariables>
            <DOCKER_HOST>${docker.host}</DOCKER_HOST>
        </environmentVariables>
    </configuration>
</plugin>

Hi @jizhuozhi ,
if we use the maven-surefire-plugin, testcontainers.properties is still needed?
Could you share the content? Thanks in advance!

There is no need to use any properties, just write any thing in the
maven-surefire-plugin as the issues ahead of this.
swift0303 @.***>于2022年5月13日 周五11:13写道：

Hi @jizhuozhi https://github.com/jizhuozhi ,
if we use the maven-surefire-plugin, testcontainers.properties is still
needed?
Could you share the content? Thanks in advance!
—
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-java/issues/3887#issuecomment-1125614717,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHN3C5WQXRUYHDXIAXW526LVJXCEDANCNFSM4ZH7RJQA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @jizhuozhi ,
Thanks a lot! I got it!
Now  i met the other issue.


Text boxes and buttons

This page makes no mention of text boxes and buttons. Is it possible to customize text boxes and buttons using the template json file? What would the visualName be for those? I tried "text", "textBox", "button", "buttonBox" and none of those worked. Please add documentation for those if the functionality exists.

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

ID: 8d9a8511-6ff9-78c6-5a8c-3f5ac68224d0
Version Independent ID: f4fd28c7-71fa-a776-7154-e76eb9bd3649
Content: Use Report Themes in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-report-themes.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @ttplaya! We’ve forwarded your feedback to the writer, who should respond to this thread shortly. Thanks!

Thanks for your feedback @ttplaya - you may find additional information about using themes in the community portal found at https://community.powerbi.com. We'll continue to add more content to the article you mention as our engineering team makes the information available.


"relative" fileLocation problem matchers don't take cwd into account

If I have a task with the following attribute:
"options": {
        "cwd": "${workspaceFolder}/Tests"
      },

and the problem matcher includes
        "fileLocation": [
          "relative", "${cwd}"
        ],

when the problem matcher runs, it doesn't take the specified folder into account, i.e. the files are looked for in the workspace folder. If I add /Tests to the fileLocation, e.g. "${cwd}/Tests", it finds the correct file.

${cwd} is the current working directory of the VS Code process, not the terminal or task's process.
The docs here try to say that, but they aren't very clear. I'll update it.

Thanks for clarifying the docs. Is there any way to get the current working directory for the task?

Not using ${} style variables. However, since you set the cwd using options, you presumably know what your cwd is (unless the task changes it).

The problem is that I'm trying to write a reusable problem matcher (provided by an extension), so although I do know the cwd, the problem matcher doesn't. Unfortunately the task output doesn't provide full paths.
Thanks for the explanation.


Implement Lab6  CRUD puppystore

Fixes #606
Review of colleague's PR: #653
Changes proposed in this PR:

Rework create logic to handle puppy ID creation as per PR review suggestion.
Updated error messages as per code review comments.
Created an executable go program in directory 06_puppy/sirigithub
Implemented a Puppy struct containing ID, Breed, Colour, Value
Created Storer interface with crud methods for Puppy
Created a MapStore implementation of Storer backed by a map
Created a SyncStore implementation of Storer backed by a sync.Map
Written tests against the Storer interface for both implementations
Implemented a main class and basic tests for both Storer implementations.

I can see a lot of good work over here. Although i have made few comments, but they are optional. Thanks

@juliaogris .. i was a bit late looking for a PR to review not many left , a couple of the higher labs I would like to attempt them to get more context. I have otherwise addresses all your comments.

Ignoring the golangci issue as this is golangci-lint version specific.

Thank you @camh-anz @juliaogris


Do you plan on adding this to PyPI?

I'm loading requirements of my app from requirements.txt and would like to avoid using the editable packages syntax if at all possible.
I envision my requirements looking something like this:
RNCryptor-python==3.2.0
Sorry if this is asked in the wrong place. Please feel free to close if that's the case, and let me know where I can submit questions like this.

Hi,
I think it is the right place. Actually all is already prepared for PyPI. I've just waited for someone who is going to use it :) I will try to do it at weekend. Feel free to ping me If I am lost.
BTW, the name will be just rncryptor.

Hi,
I've uploaded RNCryptor to PyPI: https://pypi.python.org/pypi/rncryptor

Perfect. Thanks, @ykalchevskiy!

You are welcome :)


Vertx does not show up on current and outdated commands

Related to #345, arrays are not handled correctly and this results in truncation. Vertx is last in the list so drops off.

@programmist I had a closer look on my system, and I seem to have vertx showing up on the current command. Could you try having another look, maybe after opening a new terminal?

I closed my terminal emulator and opened it up again.  When I run gvm current I see
Using:
gradle: 2.5
groovy: 2.4.4

When I run gvm current vertx I see
Using vertx version 3.0.0

I'm still new to gvm, but is there a local cache that might need to be cleared?  I tried gvm flush candidates and opened a new terminal, but it didn't change anything.
I also tried uninstalling gvm by deleting the .gvm directory and removing the entries from .bashrc, .bash_profile, and .zshrc.  Upon reinstalling gvm and Groovy, Gradle, and Vert.x I still don't see vertx in the current list.
I'm running OSX 10.10.4

Confirmed, I can recreate this on a docker ubuntu instance under the same conditions as what you stated.

Will roll out as release 2.4.5.


Added README.md file for mosquitto broker addon

To resolve https://github.com/home-assistant/hassio-addons/issues/730

@jensihnow I've added some commas and stuff, made the suboptions separated with a . (that matches service calls from our general documentation) and removed a bunch of (optional) things, that wasn't optional according to the add-on configuration schema.
I think it looks good to go now!

Thx @frenck

You don't have to thank me sir, I have to thank you! Great job on a first-time contribution m8! 🎖
And feel free to contribute more often! 👍
Happy Hacktoberfest 🎉


Finalize Components thumbnail illustrations

@JennySanchez commented on Mon Jul 27 2020
As an IBM.com library adopter, I need to see not only the list but the image of the components so I can quickly find what I need on the Components overview pages.
Tasks

[x] Review the images with Wonil and Linda for feedback
[x] Make changes as needed.
[ ] Finalize wireframes images.

Acceptance criteria

[ ] Images are ready for dev implementation.

All wireframes can be viewed here: https://ibm.ent.box.com/folder/118968988056

Component thumbnails images exported and uploaded to Box: https://ibm.box.com/s/hx2xg8kp7hardtzjstgypcda3qjzcccd


gRPC-Core and gRPC-C-C++ get Semantic issue after Update Xcode to 12.5 (12E262)

Step 0: Are you in the right place?

For issues or feature requests related to the code in this repository
file a Github issue.

If this is a feature request please use the Feature Request template.

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general (non-iOS) Firebase discussion, use the firebase-talk
google group.
For backend issues, console issues, and other non-SDK help that does not fall under one
of the above categories, reach out to
Firebase Support.
Once you've read this section and determined that your issue is appropriate for
this repository, please delete this section.

[REQUIRED] Step 1: Describe your environment

Xcode version: 12.5
Firebase SDK version: 8.0.0
Installation method: CocoaPods
Firebase Component: Firestore

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
What happened? How can we make the problem occur?
This could be a description, log/console output, etc.
If you have a downloadable sample project that reproduces the bug you're reporting, you will
likely receive a faster response on your issue.
Relevant Code:
gRPC-Core  framework have this problem

typedef struct {.         // This line have problem.  Anonymous non-C-compatible type given name for linkage purposes by typedef declaration; add a tag name here

  grpc_mdelem* md = nullptr;
  size_t size = 0;
} grpc_credentials_mdelem_array;

// TODO(you): code here to reproduce the problem

These warnings can probably be suppressed using the techniques already in place here:
https://github.com/firebase/firebase-ios-sdk/blob/38b4592cd939a482f1095bdbdceb447940a25346/Firestore/core/src/util/warnings.h#L20-L36.
I'll look into this.

After investigation, I found that the offending code is not directly or indirectly included from Firestore's source code. As a result, these warnings cannot be fixed here in this GitHub repository. As a result, I'm going to close this issue. I believe the correct place to open a ticket is against grpc (https://github.com/grpc).


Footnote links does not work

Description
Links to footnotes do not work (i.e. clicking on them does nothing). On the contrary clicking the "return" button on a footnote correctly takes you to the referring position.
P.S. As footnotes are core to academic articles, arXiv HTML should have world class support for footnotes and references (e.g. in line with what's outlined at https://gwern.net/sidenote)
(Optional:) Please add any files, screenshots, or other information here.
I'm using the Readwise Highlighter extension
(Required) What is this issue most closely related to? Select one.
References
Internal issue ID
5974da50-c631-435b-9f86-ef56475a66b7
Paper URL
https://browse.arxiv.org/html/2401.01883v1/
Browser
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>
Device Type
Windows Laptop

Some more information about it issue - it seems that the reference links point to https://browse.arxiv.org/html/2401.01883v1/#bib.bib1 instead of https://browse.arxiv.org/html/2401.01883v1#bib.bib1 (note the erroneous / before the anchor hash). Removing the slash manually takes to the proper location.


Update README.md

Umask parameter does not need in case of master (.tac file has it by default).

@azhogov Sorry, I fixed documentation. 😄


Moneyflow2

Use "MoneflowCentral2" and convert it to the production ready code.
DO NOT remove Moneyflow1 please.

Moneyflow2:

Prepare Use Cases ->> doc

Requirements ->> [tests, doc]

Compare pros/cons of concepts
https://gist.github.com/AnthonyAkentiev/23652768acf7dc6c28f19a256146ad4f

Client usage pseudoCode ->> doc
https://gist.github.com/AnthonyAkentiev/1856ff8389c60124a2f82972cf147d32


chore(deps): update dependency sqlalchemy to v2.0.17

This PR contains the following updates:

Package
Update
Change

sqlalchemy (changelog)
patch
==2.0.9 -> ==2.0.17

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

changes were merged with https://github.com/bcgov/nr-rfc-grib-copy/pull/70


Add TinyMenu To Library

@ArduinoBot

@ArduinoBot

Hello @Yuki-Kurosawa. The automated system here only operates on open pull requests. Since your submission was already accepted and the pull request merged, the bot won't pay any more attention to this pull request.
But this human is happy to provide assistance. Was there something you needed from the bot?


Merge millions of neighbouring squares?

Thanks for an extremely useful tool. Apologies if this is a naive question or the wrong place to ask.
Is there a way to ask tippecanoe to merge attributes-less neighbouring polygons? I have ~ 15 millions adjacent tiny squares, each of them corresponding to a Level 18 slippy tile, which do not render efficiently. I would be very happy for those of them who are consecutive to be merged into one big polygon -- doing away with individual squares and boundaries.
This is a somewhat simpler case than for general polgons, as those shapes are non-overlapping squares, parallel to the axes, etc. Usual geometric toolboxes like Python's shapely are overkill and die on it. The annoying bit is that the union of the squares can still be non-convex, and have 'holes' in it.
Any help much welcome! Thanks :)

Have you tried doing this in a proper GIS before converting your data to tilesets? Something like QGIS should be able to handle this, I would have thought.

@jucor The --coalesce option will try to combine adjacent (defined by quadkey order, not necessarily by real adjacency, and currently stopping at 700 points per coalesced feature) that have the same attributes. I hope this is close enough for what you are looking for.
I also have some experimental code for more general polygon merging by true adjacency, but it is slow and complicated.

@ericfischer : thanks a lot, --coalesce did the trick perfectly! My .mbtiles file went from 106M to  9.9M, and the webmap is now much more reactive. Amazing! Closing this issue. Thanks again!
@dylanparry : thanks for the suggestion. I am trying to stay entirely programmatic, because this rendering is just one part of an automated pipeline post processing the output of a machine learning classifier. So jumping through QGIS, which is clunky and highly manual, would be a very last resort. However, if you do know some good programmatic GIS libraries (ideally in Python), I'd love to hear them -- I'm coming at this form the machine learning angle, not from the GIS one.


BOM Weather component rarely updates (not 30 minutely as described)

Home Assistant release with the issue:

Last working Home Assistant release (if known):
unsure
Operating environment (Hass.io/Docker/Windows/etc.):
Docker inside Ubuntu 18 VM

Component/platform:
BOM Weather (not forecast)
https://www.home-assistant.io/components/bom/

Description of problem:
BOM Weather component seems to only update every 7-8 hours or so. My 'current temps' during the day read some temperature from the middle of the night which is way out.
Looking at my logs, it appears that the timeframe check is finding my last update is -1 day ago? seems wrong to me
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
sensor:
    - platform: bom
    station: IDQ60901.94569
    name: "Sunshine Coast"
    monitored_conditions:
      - apparent_t
      - delta_t
      - gust_kmh
      - gust_kt
      - air_temp
      - dewpt
      - press_msl
      - rain_trace
      - rel_hum
      - wind_dir
      - wind_spd_kmh
      - wind_spd_kt

Traceback (if applicable):
2019-09-02 03:30:29 DEBUG (SyncWorker_8) [homeassistant.components.bom.sensor] BOM was updated -1 day, 15:30:29.007135 minutes ago, skipping update as < 35 minutes, Now: 2019-09-02 03:30:29.007139, LastUpdate: 2019-09-02 12:00:00

Additional information:

I resolved this by fixing the timezone on my Ubuntu VM plenty of instructions online if you google. Wouldn't have thought this should be timezone dependent, but it sorts out the issue. See below, no longer says "-1 days"
[homeassistant.components.bom.sensor] BOM was updated 0:28:15.005026 minutes ago, skipping update as < 35 minutes, Now: 2019-09-03 08:28:15.005029, LastUpdate: 2019-09-03 08:00:00
Thankyou


setup.py  on  master branch corrupt ?

I am trying to create a release and after creating a new release, I do pip3 install JPype1 and I get the following error.
setup.py", line 4, in <module>
        import setupext
    ModuleNotFoundError: No module named 'setupext'

On 30.10.18 22:30, Anil Pai wrote:

I am trying to create a release and after creating a new release, I do pip3 install JPype1 and I get the following error.
setup.py", line 4, in <module>
        import setupext
    ModuleNotFoundError: No module named 'setupext'

it seems setupext has not been added to master, but resides only in devel.

It appears to be there to me.
https://github.com/jpype-project/jpype/tree/master/setupext

Is it something missing from MANIFEST.in?  I don't know python packaging tool well enough to know how to change what ships with pip.

I did not replicate the issue.
$ python3 -m pip install --force-reinstall JPype1
Collecting JPype1
  Downloading https://files.pythonhosted.org/packages/c4/4b/60a3e63d51714d4d7ef1b1efdf84315d118a0a80a5b085bb52a7e2428cdc/JPype1-0.6.3.tar.gz (168kB)
    100% |████████████████████████████████| 174kB 605kB/s
Building wheels for collected packages: JPype1
  Running setup.py bdist_wheel for JPype1 ... done
  Stored in directory: /home/nelson85/.cache/pip/wheels/0e/2b/e8/c0b818ac4b3d35104d35e48cdc7afe27fc06ea277feed2831a
Successfully built JPype1
Installing collected packages: JPype1
  Found existing installation: JPype1 0.6.3
    Uninstalling JPype1-0.6.3:
      Successfully uninstalled JPype1-0.6.3
Successfully installed JPype1-0.6.3

I created a new release and gave that a different name..
$ python3 -m pip install --force-reinstall ha_JPype1
Collecting ha_JPype1
  Downloading https://files.pythonhosted.org/packages/38/9c/04d462192b8e897a3f50581f5ae284f9047631cc7a65c9b4e0e22f202e41/ha_JPype1-1.6.7.tar.gz (168kB)
    100% |████████████████████████████████| 174kB 2.6MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/7j/ty_pggwd15lcml_pnw3ghmq00000gn/T/pip-build-05z8v158/ha-JPype1/setup.py", line 4, in <module>
        import setupext
    ModuleNotFoundError: No module named 'setupext'

On 31.10.18 17:53, Anil Pai wrote:

I created a new release and gave that a different name..
$ python3 -m pip install --force-reinstall ha_JPype1
Collecting ha_JPype1
  Downloading https://files.pythonhosted.org/packages/38/9c/04d462192b8e897a3f50581f5ae284f9047631cc7a65c9b4e0e22f202e41/ha_JPype1-1.6.7.tar.gz (168kB)
    100% |████████████████████████████████| 174kB 2.6MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/7j/ty_pggwd15lcml_pnw3ghmq00000gn/T/pip-build-05z8v158/ha-JPype1/setup.py", line 4, in <module>
        import setupext
    ModuleNotFoundError: No module named 'setupext'

So you package some non working version and then you blame us for it?

No I checked.  It gave me a version without setupext from a download site. It appears like python setup uses the same install directory.  So if you install successfully once it finds resources, but not if you pip.  Must be a missing line in the dist building script.

I see a lot of changes to setup file recently.
https://github.com/jpype-project/jpype/commit/490eb6511a0a0b57be2191cbbf3a90bac45e5879#diff-2eeaed663bd0d25b7e608891384b7298L9
I am trying to understand what that means..

Can I also know how to generate a distribution ? how do you do that ? are you guys using twine ?
Any code or instructions will help. Thanks !

It appears the command we use to build the source distribution ("python setup.py sdist") had a missing line in the manifest file.  I put in a pull request to fix it #367.

@anilpai you are aware that you can also install from an git archive, or commit id from github itself. If you upload non-working versions on PyPI some users will stumble upon it and finally blame us for something we never would have released.
You can just use
pip install https://github.com/jpype-project/jpype/archive/161541e229b311f99f75c301404324d3e72a1d49.zip

to install a specific version. Why do you need to "release" something in the first place?


Should Softdelete use deleted_at

Problem
I was reviewing https://django-model-utils.readthedocs.io/en/latest/models.html#softdeletablemodel and see that it uses a boolean. Could we add another model that uses created_at that is a timestamp? thoughts?

Hi!
I think the TimeStampedModel does exactly that.
https://github.com/jazzband/django-model-utils/blob/eb00f65c81fb26dc73826d0e33d564d1b079f9a0/model_utils/models.py#L28

not quite. If timestamped model also added a deleted_at that was a timestamp then yes.

Maybe you can combine TimeStampedModel and SoftDeletableModel.
For example, if an object is deleted, the modified field should have the timestamp of the deletion.
Obviously you can still modify it, and that timestamp would not reference anymore the time of deletion.
That being said, I see a valid reason on the field you want to add.


gpt-3.5-turbo-16k Not Supported

Can you please add support for gpt-3.5-turbo-16k? I get errors when I try to do:
var encoding = GptEncoding.GetEncodingForModel("gpt-3.5-turbo-16k");

I will have a look, thanks for your question


fix: Activity event improvements

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #5775
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at e83d93e
Refactored and improved the layout and responsiveness of the collection activity event components. Used props, utility classes, and flexbox to control the size, alignment, and overflow of the avatar, event, and text elements. Modified EventRowAvatar.vue, EventRowDesktop.vue, EventRowTablet.vue, and Events.vue.

🤖 Generated by Copilot at e83d93e

Sing, O Muse, of the skillful coder who refines
The EventRowAvatar component with a size prop divine
And who enhances the event row for desktop view sublime
With utility classes and overflow control, a master of design

hey hey, no jumping to second row please

hey hey, no jumping to second row please

Fixed

hmm... not sure about that NeoAvatar this name would refer to Identity Avatar to me. I would keep it outside libs/ui for now

IMO, this component could also be used for displaying the Identity avatar if we give it proper props. So, let's leave it here for now ?

What if we want to use lazy loading on these images?

Then we need to introduce nuxt-img or some other lazy load package. There is already an issue for nuxt-image.

Then we need to introduce nuxt-img or some other lazy load package. There is already an issue for nuxt-image.

fyi libs/ui isn't a nuxt project (vue 3)

Thanks for correcting me. Then it would need another lazy load package supported vue3.

pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 30.55 USD/KSM ~ 1.637 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0xe56f6966f8b7e589316594e113eb2e189e3805b6bdcd83feef7ac9a72158c638
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Fix  duplicates in guidance group options

Fixes #1796  .
Changes proposed in this PR:

Restructure the has_many association to use SQL's DISTINCT function

looks good. tests are failing due to JS issues on dev branch that have since been resolved


[fix][client] Specify the init parameters value in the OpSendMsg

Motivation
Some parameters in OpSendMsg are not specified with initialized values.
These initial values are all relied upon by other codes. Such as here:
https://github.com/apache/pulsar/blob/eec46ddcba4d2b4f956e1b4d63154cc43087f507/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ProducerImpl.java#L1409-L1422
Modifications

Specify the init value for these parameters according to the value in the OpSendMsg.recyle().

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

Please explain what's the affect of these fields without initial values.

I have explained it in the PR description. Is there something unclear?

In addition, adding initial values to a recyclable class is dangerous. For a recyclable class, we should make constructors private and only expose some factory methods and set the value in these factory methods.
The reason is that if the object was reused from the pool, the initial value should be what was set in recycle() method.
For example,
@Data
class RecyclableInteger {
    private static final Recycler<RecyclableInteger> RECYCLER = new Recycler<RecyclableInteger>() {
        @Override
        protected RecyclableInteger newObject(Handle<RecyclableInteger> handle) {
            return new RecyclableInteger(handle);
        }
    };

    private final Recycler.Handle<RecyclableInteger> handle;
    private int x = 100; // initial value: 100

    private RecyclableInteger(Recycler.Handle<RecyclableInteger> handle) {
        this.handle = handle;
    }

    public static RecyclableInteger create() {
        return RECYCLER.get(); // retrieve an object without modifying the value of `x`
    }

    public void recycle() {
        this.x = 0; // recycled value: 0
        handle.recycle(this);
    }
}

        RecyclableInteger i = RecyclableInteger.create();
        System.out.println(i.getX()); // 100 as we expected
        i.recycle();
        i = RecyclableInteger.create();
        System.out.println(i.getX()); // 0, which is unexpected

I have explained it in the PR description. Is there something unclear?

Sorry at the time I replied, the PR description was not updated. Then, please see my another comment.

The best practice might be adding an initialize() method to a recyclable class and calling this method at the beginning of each factory method, like:
    private void initialize() {
        x = 100;
    }

    public static RecyclableInteger create() {
        final RecyclableInteger i = RECYCLER.get();
        i.initialize();
        return i;
    }

In addition, adding initial values to a recyclable class is dangerous. For a recyclable class, we should make constructors private and only expose some factory methods and set the value in these factory methods.
The reason is that if the object was reused from the pool, the initial value should be what was set in recycle() method.

Before this PR, there is already an inconsistent issue in the OpSendMsg. The value of the OpSendMsg just initialized is not the same as the OpSendMsg just being recycled. This PR is to fix this inconsistency. I was wondering if we could have the recyle method call some kind of initialization method so we don't need to maintain the same initialization value in two places.

if we could have the recyle method call some kind of initialization method so we don't need to maintain the same initialization value in two places.

Yes.

@BewareMyPower Hi, I have added the initialization for the OpSendMsg, PTAL again. Thanks.


Parsing "javascript" to json (tailing commas)

I think it is not possible now to parse valid javascript objects if they are containing tailing commas.
Is that right?
[1,2,3,] and {"key":"value",} sometimes "came out" from not very polished systems...

Thx the response (and sorry that I forgot to react when I first read this)! I saw that there are some "global" params in the json parser, and I didn't found at first that the parsers are not implemented here.
For future readers:

I fixed my problem using a small nodejs script... (so no code for reproduce)
jackson sure has this feature: https://github.com/FasterXML/jackson-core/issues/118
also jackson can parse unquoted object keys: https://stackoverflow.com/a/36582975
so you can parse {nice: true,} if the api returns "javascript" titled by "json"


Nick: Watch annongland.com

Nick requests the watch of the watch_keyword annongland\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

annongland\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Add spinner to details loading state

Added a PatternFly spinner to the details page loading state. This replaces the existing binocular icon.
Fixes https://github.com/project-koku/koku-ui/issues/1186

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1203   +/-   ##
=======================================
  Coverage   74.57%   74.57%
=======================================
  Files         227      227
  Lines        4453     4453
  Branches      870      870
=======================================
  Hits         3321     3321
  Misses       1013     1013
  Partials      119      119

Continue to review full report at Codecov.

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


Be more explicit about how we use memory

The fuzzy algorithm was initially written fairly quickly with only a passing concern for what our memory use looked like. As such:

there is no consistent principle governing the use of copied values vs. references to values other then "hey, this struct looks pretty small and immutable", and
there might be a nice way to avoid creating a new Vec<Next> every time we want to consider the possible future steps from a given position.

Review memory use and use of references, but only after implementing benchmarks.

My short-term goal for fuzzy is to complete #12. Before I do that, I want to take my own stab at being less naive about memory use / moving / copying.

I've removed succ and it's temporary vector. Other then that, I think using references to patt and text items (which I did earlier) was the only thing that jumps out at me.
I could potentially pre-compute all Ix values (probably in the Config) and just pass around references to them, but it's really not obvious that this would be an improvement ... let's not do any more on this before implementing benchmarks or asking for feedback.


Something went wrong

Starting a new 3 nodes cluster today, have deployed some apps successfuly and another which result in a failing state. Since this fail, got a unknown_error: Something went wrong while creating or deleting apps.
Here is a flynn-host collect-debug-info
Let me know if I can provide you more informations.
Thanks in advance for incoming help.

@Natsuke it looks like there is a service discovery issue:
15:13:25.529998 heartbeat.go:138: discoverd: heartbeat flynn-host (<IP_ADDRESS>:1113) failed: unknown_error: Something went wrong

Can you please gist the contents of the following file:
/tmp/flynn-host-logs/flynn-etcd-fa0ba973dbb1476986f6e13e8b95d8d9/flynn-etcd-fa0ba973dbb1476986f6e13e8b95d8d9.log

Thanks for fast reply,
here is the log content of flynn-etcd-fa0ba973dbb1476986f6e13e8b95d8d9.log

@Natsuke there are a lot of network related errors in the etcd logs, for example:
2015/03/30 13:09:06 rafthttp: encountered error reading the client log stream: read tcp <IP_ADDRESS>:2380: use of closed network connection
2015/03/30 13:13:22 etcdhttp: unexpected error: etcdserver: request timed out
2015/03/30 13:13:48 rafthttp: encountered error reading the client log stream: read tcp <IP_ADDRESS>:2380: i/o timeout
2015/03/30 14:13:57 rafthttp: encountered error writing to server log stream: write tcp <IP_ADDRESS>:33279: broken pipe

This is then causing service discovery issues, and ultimately leading to components unable to find each other.
Are all your nodes in the same availability zone? etcd is known not to work well when there is potential for high latency between nodes.

@lmars For now our 3 nodes run on a Proxmox hypervisor on the same machine with these spec which is our testing machine before switch to our dev cluster.
We noticed poor i/o performance which maybe cause etcd failure.

@Natsuke ok yes, poor i/o performance could indeed cause issues. You may want to consider running a single node Flynn cluster until you have tracked down the i/o issues.

@lmars Okay i'll give a try. Thanks a lot for time sparing. I'll reopen new issue if we encounter another failure.


implement REST API versioning?

We're planning on making some breaking changes to the API, and I was thinking about the best way to handle this so that we don't have to worry too much about pushing our changes on the client and server at the same time for this and future versions. Working on different branches helps, but implementing an API versioning feature would pretty much eliminate the problem.
I was thinking from the API side the easiest way to do it would be to place a version slug in the URL like:
http://<IP_ADDRESS>:3001/api/v1/[...]
Looks like there's at least one gem that provides this feature:
https://github.com/bploetz/versionist
and a few articles that show how to roll your own:
http://www.bignerdranch.com/blog/adding-versions-rails-api/

Yup, I've done this for both DGI and Docm, so we should definitely do the same here!

+1

On Mar 10, 2015, at 10:38 AM, Adam Coffman<EMAIL_ADDRESS>wrote:
Yup, I've done this for both DGI and Docm, so we should definitely do the same here!
—
Reply to this email directly or view it on GitHub https://github.com/genome/civic-server/issues/43#issuecomment-78080047.

I'm going to close this issue for the sake of tidiness. Once we're officially out of beta we'll pin the API at v1 and from that point on, any breaking changes will require a new "version."


Make stuff in IR.scala package private

Contributor Checklist

[ ] Did you add Scaladoc to every public function/method?
[ ] Did you add at least one test demonstrating the PR?
[x] Did you delete any extraneous printlns/debugging code?
[x] Did you specify the type of improvement?
[ ] Did you add appropriate documentation in docs/src?
[x] Did you state the API impact?
[x] Did you specify the code generation impact?
[x] Did you request a desired merge strategy?
[x] Did you add text to be included in the Release Notes for this change?

Type of Improvement

code cleanup

API Impact
Stuff that should not have been part of the public API in IR.scala was made package private.

Backend Code Generation Impact

None
Desired Merge Strategy

Squash: The PR will be squashed and merged (choose this if you have no preference.

Release Notes

Classes in IR.scala were made package private and removed from the public API.
Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (Bug fix: 3.3.x, [small] API extension: 3.4.x, API modification or big change: 3.5.0)?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you mark as Please Merge?

Unfortunately, Width is actually used in the public API (which is annoying): https://github.com/chipsalliance/chisel3/blob/392ea3c9b5b04e374eeb1bf3b0d87ac9fbf45513/core/src/main/scala/chisel3/package.scala#L154
So at least that needs to stay public but separately we should pull that into package chisel3.

Should these API being deprecated at first? Although I believe there will be no user using those API.

Should these API being deprecated at first? Although I believe there will be no user using those API.

I would usually say yes, but I think in this case we can just go for it and see if anything breaks. We don't publish anything in chisel3.internal to the public API docs. Unfortunately a few of these nodes are used in the public API (Width, BinaryPoint, and Circuit) but for the rest, maybe we should just see if we can make them private.

Tested with all upstream Chisel projects, I'm going to merge this for RC2 and hope for the best since this stuff never should have been public. If we get push back we can revert before .0.


Add semigroup instance for Action

This would add Semigroup and Monoid instances for Action, keeping compatibility with past compiler versions.

Thanks - a good idea! Shake already depends on the semigroups library on old versions of GHC, so I was able to delete a lot of the CPP.


Improve render performance

Ive noticed two cases where render performance could be improved.

When the calendar loads, it initially draws the entire calendar from 1980 and then immediately redraws the entire thing with the proper dates. This is an easy 2x perf boost if we can avoid this.

When you are using the withRange() HOC, the entire calendar re-renders on every mouse movement. It should be possible to only update the header and the dates that have changed.

I'll update this issue if I find anything else.

Regarding your first point, that was an issue related to react-tiny-virtual-list. Just pushed 2.1.3 which should address this.
Regarding the second point, all components use PureComponent and will only update if their props change.

For item 2, I put a console.log inside the render method of day/index.js and on every mouse movement it spewed out a hundred messages. Maybe you could try this and see if it does the same for you. Ill double check tomorrow.

Ah, it could be an issue specific to range selection, since the selected state for range selection is an object

I can confirm that this performance problem exists only when doing a range selection and only when a selection is in progress.

I think the problem is in withRange.js:60
The functions onClick and onMouseOver are allocated on every component update, therefore triggering a re-render.

Yeah, @lap123 is right. Based on the current implementation, I'm not sure there's a way around it, but PRs are most welcome if any of you want to take a stab at it. Going to close out this issue since it isn't something that's actionable at the moment and I don't have the bandwidth to work on it myself.

I'm having this issue, but it doesn't seem to have performance issues when I use it in the demo: http://clauderic.github.io/react-infinite-calendar/#/enhance-default-functionality/range-selection?_k=lq88j9
What could be the difference?


Ability to pick between Icon and Entity Picture in Media Players

Is your feature request related to a problem? Please describe.
When I select a Media Player, I can't have the icon instead of the entity picture. I have multiple media speakers which I differentiate using icons (like a sofa for the one in the living room and things like that), but when using this button the album cover is shown instead of the icon and we don't have the ability to pick which of the two we want.
Describe the solution you'd like
A simple boolean option in the configuration would be enough to pick between showing the entity picture or the icon. Instead of only checking if there is an entity picture, you'd check also for this boolean when rendering the button.
Describe alternatives you've considered
The alternative to this is not having the icon at all and use the name of the entity to know which speaker tye button is for. However I believe dashboards with icons instead of text are much easier to use and in smaller screens having all the names of the speakers is not even possible.
Additional context
Not needed.

I can help with the JS but I'm not so familiar with all the home assistant stuff (such as the UI menu to configure the button).


Eslint breaks with v5.10.0

When upgrading eslint from v5.9 to v5.10, it failed due to mandatory plugin eslint-plugin-dollar-sign that it cannot find.
Removing eslint-config-with-prettier seems to fix the issue.
Related issue: https://github.com/eslint/eslint/issues/11177

eslint-config-with-prettier shouldn't have a dependency to jquery, we do not use it for our react project. Otherwise, please rename it as eslint-jquery-config-with-prettier and/or highlight the dependendency in the readme, thank you!

Thanks for the ISSUE.
I will remove the dependency, and make it optional in a readme.


fix: Use generic column types in resize-handle and data-sorting plugins (#247)

Proposed fix for #247 for Resize-handle and data-sorting

I had to wait with the sort, sortDescending and sortAscending helpers:
https://github.com/CrowdStrike/ember-headless-table/blob/8111252521665b76957bf3741678f0a8d6e5b10e/ember-headless-table/src/plugins/data-sorting/helpers.ts#L38-L50
For some reason, I didn't get the generics to work for the meta.forTable(column.table, Sorting).handleSort(column), see failing test here: https://github.com/CrowdStrike/ember-headless-table/actions/runs/7989097110/job/21815050164?pr=248#step:4:67
@NullVoxPopuli perhaps you know how to pass on the generic Column through meta.forTable? thanks,


reactive and ref type infer is Wrong

Version
3.0.0-beta.7
Reproduction link
https://github.com/shaonialife/vue-next-test
Steps to reproduce
const state = reactive({
  foo: {
    value: 1,
    label: 'bar',
  },
});

console.log(state.foo.label); // Property 'label' does not exist on type 'number'

What is expected?
no lint error
What is actually happening?
Property 'label' does not exist on type 'number'

ref has same problem

Seems it interprets the foo object as a ref because of the value property. If it's renamed, it doesn't happen anymore.
That problem has been fixed before, Ref's have a special marker attribute to prevent that from happening.
Thanks for reporting, we'll look into it!


Trap Stackoverflow Test

My other PR (#151) was failing CI on MacOS only, I think this is due to a bug in wasmtime dotnet, although I'm not sure what and unfortunately I cannot debug it further since I don't own a Mac.
This PR includes the minimal set of changes to (hopefully) reproduce the problem.
Note that the nuint change is necessary to prevent an OverflowException (as mentioned in #150).

As expected this PR has failed CI. As mentioned above I can't debug it any further, I hope this helps someone with access to a Mac debug the issue.

I have been testing this, and found that Native.wasmtime_func_call crashes the test runner if the executed wasm overflows the stack. I have seen this happen on master, so this is a bug unrelated to this PR.

I've just put together a test of this in Rust which is fine when run on macos (using GH actions). So it seems to not be a bug in the underlying library.

I've been doing even more testing with GH actions.
As you can see from this execution log this program works when you run it with:

dotnet run --project ConsoleApp/ConsoleApp/ConsoleApp.csproj

However, the same code copied into a test and run with:

dotnet test wasmtime-dotnet/tests/Wasmtime.Tests.csproj

Crashes!
I don't really have any ideas how to debug this further...

I'll dive into this soon and figure out why it's failing in the test host; perhaps it is installing a vectored exception handler that is interfering with Wasmtime's.

I'll still be looking into this issue, but I'm not holding up the 1.0.0 release for it as I believe it's a test-host only problem.


high variance in output of sampler

We have several reports of high variance in predictions and learned weights between multiple runs (observed by Toshiba, 341 team, and myself).
We want to fully understand what is causing this and how to best deal with it.

I did some measurements of variance on the simple spouse example (/deepdive/examples/spouse_example). In each case, I ran 20 iterations of the sampler using the default settings.
I tried both the sampler before April 29th (old) and the sampler after April 29th (new). Zifei also suggested to try switching the symmetry factor (Equal(A,B) with learned weight) on and off. Below are variance of expectations, and variance of learned weights for all combinations:
new sampler, with symmetry
Average variance expectations  : 0.05173602954174087
Average variance weights       : 2.868037752352032

old sampler, with symmetry
Average variance expectations  : 0.00468157241586358
Average variance weights       : 0.0030183971679320445

new sampler, without symmetry
Average variance expectations  : 0.08879336425759929
Average variance weights       : 0.21628128085863527

old sampler, without symmetry
Average variance expectations  : 0.0011099367678029117
Average variance weights       : 0.0012100521803007904

We can conclude

variance goes up when we add the symmetry factor
variance goes up when we use the new sampler

@feiranwang Here are some numbers on the Chunking Example. This example has 3 factors: 1. logistic regression, 2. linear chain CRF, 3. skip-chain CRF. All weights are learned.
Details below, but the high-level message is that the results are largely consistent with the previous experiment.
(The only surprise is that adding skip-chain CRF factors reduces variance in both cases !?)
old sampler, 3 factors
Average variance expectations  : 3.478433970813495e-05
Average variance weights       : 0.0013675461712029696

old sampler, 2 factors
Average variance expectations  : 3.2690989117771345e-05
Average variance weights       : 0.0015932013418684728

old sampler, 1 factor
Average variance expectations  : 1.547835036452601e-05
Average variance weights       : 0.0007665820282586536

new sampler, 3 factors
Average variance expectations  : 9.510826048157778e-05
Average variance weights       : 0.003406234197422947

new sampler, 2 factors
Average variance expectations  : 0.00011861153511291191
Average variance weights       : 0.005766963065798057

new sampler, 1 factor
Average variance expectations  : 2.1697729640831944e-05
Average variance weights       : 0.001199654528888731

@feiranwang What changes have been done on Apr 29?
@raphaelhoffmann For the spouse example, does the old sampler (the one with lower variance) produces the "same quality" result as the "new sampler"?
Ce

@zhangce Here is the commit https://github.com/HazyResearch/sampler/commit/ca3e8d45ef1c1c7d8badd16f82eb02847118d744. The change is sampling all variables during learning.

@feiranwang I am confused by this code.
The learning works as follows. There are two set of possible worlds, one contains
all possible worlds that are consistent with the evidence (call it I_e), another are
free to choose values on evidence values (call it I).
The goal of learning is to make sure the expectation of factors matches between I_e and I.
Therefore, we need to draw samples from I_e and I.
 1. if v is evidence, you consider to flip it in I; you do not care I_e because it cannot change in I_e
 2. if v is not evidence, you consider to flip it in I; then you consider to flip it in I_e

Now, in 2, you only flip it in I but not I_e? Is this where the bug sits?
Ce

@zhangce I think the code follows the above procedure. Case 2 is reflected in https://github.com/HazyResearch/sampler/blob/master/src/app/gibbs/single_thread_sampler.cpp#L50.
If v is not evidence, we flip it in I_e (line 50 - 66), and flip it in I (line 68-77), and update weight.

@feiranwang ah sorry, I understand it now.
How are the weight being initialized now? Random?
Ce

@zhangce They're all initialized to 0 for unknown weights.
Do you think it makes sense in case 2, we flip the variables, but do not use them to update weights, since they do not give information about the weights? Maybe we can talk in more detail.

This is open in two issues. Closing this one.


Don't run sbt-requirejs all the time

sbt-requirejs should use information from sbt-js and decide if rebuild of bokeh(.min).js is necessary.

This issue isn't relevant anymore, because we switched to browserify and the the entire build system is different now.


pageSize is ignore when request fields has parents.

Hello,
When I add "parents" to my request then I get max 460 file regardless pageSize setting.
Google Drive Api v3 v1.15.0 (and earlier)
Example
1 - Without "parents"
HTTP
GET https://www.googleapis.com/drive/v3/files?pageSize=1000&q='Me'+in+owners&spaces=drive&fields=nextPageToken%2C+files(name%2C+mimeType)&key={YOUR_API_KEY}
C#
 var request = service.Files.List();
                    request.Q = "'Me' in owners";
                    request.Spaces = "drive";
                    request.Fields = "nextPageToken, files(name, mimeType)";
                    request.PageToken = pageToken;
                    request.PageSize = 1000;
                    var result = request.Execute();

This two request give me list of 1000 files from drive.
2 - With "parents"
HTTP
GET https://www.googleapis.com/drive/v3/files?pageSize=1000&q='Me'+in+owners&spaces=drive&fields=nextPageToken%2C+files(name%2C+parents)&key={YOUR_API_KEY}
C#
 var request = service.Files.List();
                    request.Q = "'Me' in owners";
                    request.Spaces = "drive";
                    request.Fields = "nextPageToken, files(name, parents)";
                    request.PageToken = pageToken;
                    request.PageSize = 1000;
                    var result = request.Execute();

This two request give me only 460 files from drive, regardless pageSize setting

The pageSize specified in the request is the maximum number of the items to return per page. The server may return fewer items if (for example) the query is taking too long, or taking too many server resources.
See the description of pageSize in the documented here: https://developers.google.com/drive/v3/reference/files/list

Hey,
Thx for reply. I don't see this information in documentation. I had read this page before I wrote this issue and I used it to test this.
But if you are right this 460 is not constant right? I've done some tests and for me it's constant. I don't see dependency. I have 1036197 files on my Google Drive and always when I add "parents" I got this list in 460 parts.
 request.Fields = "nextPageToken, files(id, name, md5Checksum, mimeType, size)";
1000 items per Page
 request.Fields = "nextPageToken, files(id, name, parents, md5Checksum, mimeType, size)";
460 items per Page
Better example:
request.Fields = "nextPageToken, files(createdTime,description,id,md5Checksum,mimeType,modifiedTime,name,originalFilename,ownedByMe,size,version)";
I request 12 values and I get 1000 items per page.
request.Fields = "nextPageToken, files(parents)";
I request only "parents" and I always get exactly 460 items per page.
I don't understand why :)

The contract is that you will get back <=pageSize items, which is happening correctly.
There are many reasons why the server may choose to return a particular quantity, but that is down to implementation details of the server which I don't know - and may change at any time.
Sorry, but you're very unlikely to be able to get an answer to your question "why" ;)

:disappointed: OK

I think the number of fields requested has an effect on number of items returned per page. They're probably trying to keep the response size from growing too large.


npmjs reports different version

The package.json reports version 3.0.0 but npmjs only serves version 2.0.1. This causes confusion as the docs are different too.

@FrozenFalcon thanks for bringing this to my attention again. The reason for the difference is that I was having some issues with the GeoIP API that is used in versions <= 2.x and I found an alternative provider and bumped the version to 3.0.0, but both actually still work, it was a temporary issue with the original provider; so I never actually pushed 3.0.0 to NPM as it wasn't really necessary. I haven't used either provider that much. If you use this module perhaps you could help me find out which is better if there is a difference at all, or maybe it would be a good idea to have a primary and fallback provider using both. I don't necessary have the time to look at this now, but maybe you could contribute a pull request to somehow resolve the inconsistency.


Ordered lists number shouldn't be specified

You get that for free

Nice, I see the updates:

Page reference


Dynomite::DbConfig.check_local! misditects when jets running on Docker

I got troubles on my local development environment.

Jets is running on Docker container
DynamoDB-local is running on Docker container: amazon/dynamodb-local named dynamodb
both containers belongs to same docker network

DynamoDB-local running on http://dynamodb:8000
jets-container $ aws --endpoint-url http://dynamodb:8000 dynamodb list-tables
{
    "TableNames": []
}

config/dynamodb.yml in jets:
development:
  table_namespace: <%= Jets.config.table_namespace %>
  endpoint: http://dynamodb:8000

But jets dynamodb:migrate [path] failed.
/home/morimori/.anyenv/envs/rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/dynomite-1.2.2/lib/dynomite/db_config.rb:48:in `check_dynamodb_local!': You have configured your app to use DynamoDB local, but it is not running.  Please start DynamoDB local. Example: brew cask install dynamodb-local && dynamodb-local (RuntimeError)

I checked check_dynamodb_local!
https://github.com/tongueroo/dynomite/blob/master/lib/dynomite/db_config.rb#L43
    def check_dynamodb_local!(endpoint)
      return unless endpoint && endpoint.include?("8000")

      open = port_open?("<IP_ADDRESS>", 8000, 0.2)
      unless open
        raise "You have configured your app to use DynamoDB local, but it is not running.  Please start DynamoDB local. Example: brew cask install dynamodb-local && dynamodb-local"
      end
    end

Because Port: 8000 and host: <IP_ADDRESS> is hardcoded, jets cannot connect http://dynamodb:8000.
Can I fix this behavior and post PR?

RE: Can I fix this behavior and post PR?

Yes please. Thanks!

Thanks for comment!
I'll try to fix it :)


normalize confusion matrix

Kindly pinging @strsamue here, perhaps we can implement and add this to the final report soon 🤩


Scribblets eventually exhaust swap space

Swap and free RAM slowly decrease at each tick.  Exhaustion of swap noted ~2 hours in.

Logs attached
scribe-logs-issue11.tar.gz

Note that on kill, CPUScribblet dies and MemScribblet runs a final time, showing full swap recovery.


[Feature Request] Custom Schemas For Auto-Setup (Postgres)

Is your feature request related to a problem? Please describe.
We use Supabase (i.e., built on top of Postgres) as our database provider and backend-as-a-service. As part of their service, they automatically backup (for a price) the postgres database. Currently, we are able to set the database using docker-compose and changing our docker-compose.yml to:
temporal:
    container_name: temporal
    ...
    environment:
      - DB=postgresql
      - DBNAME=postgres
      - VISIBILITY_DBNAME=postgres
      - DB_PORT=5432
      - POSTGRES_USER=postgres
      - POSTGRES_PWD=
      - POSTGRES_SEEDS=db.<supabase-project-id>.supabase.co
      ...

The issue we're facing is that we need to have everything inside of the postgres database, but the auto-setup.sh automatically uses the public schema inside of whatever database is set, without the option to override this via an environment variable set in the docker-compose.yml. We need to be able to set the schema of the temporal database and the temporal visibility database.
Describe the solution you'd like
I would like to be able to pass the environment variables DB_SCHEMA and VISIBILITY_DB_SCHEMA, or something like them (such as just SCHEMA and VISIBILITY_SCHEMA) in the docker-compose.yml file.
These two variables would be passed down to the auto-setup script, and would execute the named schemas when the auto-setup  script runs the temporal-sql-tool. It looks like, from here, that the commands would need to be changed to look something like:
SQL_DATABASE=temporal ./temporal-sql-tool --ca search_path=temporal setup-schema -v 0.0

Additional context
It also looks like this change would need to be reflected in the server itself, I just don't know enough about Docker to give nay suggestion. The earlier link seems to suggest it, but I'm just not sure.
Edit 1
From further research, it just looks like there would need to be a way to pass down the --ca flag to the temporal-sql-tool, but I could be wrong. I also saw that the connectAttributes for temporal and visibility will likely need to be changed to reflect the same search_path=SCHEMA_NAME_HERE, but I'm not sure how to incorporate that into this mix.

Postgres supports giving table names like: myschema.table_name. You should be able to use that do use different schemas without the need for new variables.


Long startup activation time on the first start

🐛 Describe the bug
It takes significant amount of time to start the extension on first, fresh start of VS Code. This occurs only on the first start in the day. This leads to "Extension runner is unresponsive" error notification in VS Code. Any consequent startup activation time is at least 10x faster, ~950ms.
🌴 Visual Studio Code Version :  1.28.0-insider (system setup)
Commit: 2558a72d6902015b142ef45c57b664ce1f09e03f
🌴 Code Settings Sync Version : 3.1.2
🌴 Operating System : Win 10 Home (1803)
🌴 Occurs On: Start of VS Code
🌴 Proxy Enabled: No
📰 To Reproduce
Steps to reproduce the behavior:

Start fresh VS Code.
Open Running Extensions
Find Settings Sync
See error from VS Code

💪 Expected behavior
The first startup activation time should be similar to the consequent (~950ms).
📺 Console Error Log
First start:

Second start:

Actually code is the same on first and second startup ...
What is really affecting the time might be the Visual Studio Code extension host.
Can you verify does it happens every time ~25900ms on the first startup of day ?

So far, I tested this three times and always with positive results -> ~25900ms.

I can confirm this issue.

This issue is available for bounty. Fix and earn!

@alexdevero this issue is fixed.


Set Image From Url

Can I set image which I have url to load image from this url. I am currently using KingFisher for getting images from URL.

Hey, Toast-Swift is supporting UIImage, not URL.
You can load the image from KingFisher and then pass it to the toast.

The default Toast view will not support this. You can use the showToast methods to show any view as toast. Just make it your own UIView subclass that can load an image using Kingfisher (if that's what works for your app) and then use showToast to present the view.


Bucket Fill option:

Bucket Fill option:
This is a great app, but I am really missing the option to fill an area with a single colour.

Thanks! Due to how the program works (it's based on vectors rather than pixels) it's impossible to implement a traditional bucket fill operation. So i'm closing this.