I tried all 3 possible options :

If both mentioned:
--vpc-security-group-ids sg-0614fbf3b6c8efc95
--db-security-groups vpc-dfed7cbb \
gives following error:
{code}
root@percona03-251:/data/xfs/scripts # sh rds_creation.sh
An error occurred (InvalidParameterCombination) when calling the RestoreDBInstanceFromS3 operation: DB Security Groups and Vpc Security Groups cannot both be provided.
{code}

If only --db-security-groups vpc-dfed7cbb is mentioned:
{code}
root@percona03-251:/data/xfs/scripts # sh rds_creation.sh
An error occurred (InvalidParameterCombination) when calling the RestoreDBInstanceFromS3 operation: DB Security Groups can only be associated with VPC DB Instances using API versions 2012-01-15 through 2012-09-17.
root@percona03-251:/data/xfs/scripts #
{code}

If only  vpc-security-group-ids sg-0614fbf3b6c8efc95 is mentioned:
{code}
root@percona03-251:/data/xfs/scripts # sh rds_creation.sh
An error occurred (InvalidParameterCombination) when calling the RestoreDBInstanceFromS3 operation: The DB instance and EC2 security group are in different VPCs. The DB instance is in vpc-fe7f8a94 and the EC2 security group is in vpc-dfed7cbb
root@percona03-251:/data/xfs/scripts #
{code}

please use security group belong to the above mentioned vpc

please use 3rd option and use same vpc


Doesn't boot to GUI after installing VNC

I downloaded the latest stable release (2.1.0), flashed to SD, put in Raspberry Pi 3 model B, booted, pressed ctrl-alt-F2, entered raspi-config, chose "Interfacing options" and VNC. When I enabled VNC server, I got a lot of errors, and VNC didn't work.
So I ran a sudo apt update, sudo apt upgrade and then enabled VNC in raspi-config. The output can be seen here.
Now, when I boot, it starts up with a text login on top of the welcome image background.png. So the GUI doesn't boot.
If I go into raspi-config and choose "Boot Options" - "Desktop / CLI" - "Desktop Autologin", it tells me that I need to install lightdm, so that seems like the wrong way to go.
Help would be appreciated!

Hey there!
There's another issue (#38) with a lot of info regarding your use case. Please check it out.
With regard to updates, it's not really something we support (#11), and I wonder how that could be made more clear... :thinking:


[r19.09]: python27Packages.cherrypy: fix build

Motivation for this change
Due to the recent update of python 2.7 from 2.7.16 to 2.7.17 (see 05dfc5063928030483ad5f5e468d3b9ec8632a1d) the StaticTest::test_null_bytes test case started failing on 2.7 too.
This is because additional validation has been backported to httplib in PR 13315 in https://github.com/python/cpython to to address a security issue.

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 nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Has this been fixed already on master? Different fix here?

@FRidh: I was about to do a PR for master too, but then encountered some issue that prevents building entirely;
https://github.com/NixOS/nixpkgs/pull/73922
> nix-build -I nixpkgs=$(pwd) '<nixpkgs>' -A pkgs.pythonPackages.cherrypy                       e8249a905b1 [e8249a905b1] untracked

error: black-19.10b0 not supported for interpreter python2.7
(use '--show-trace' to show detailed location information)

So I'm looking into that for a minute.

@FRidh: I think it would be nice to merge this one for 19.09 as work on deprecating the 2.7 version is going on and this has a few downstream failures.


[GUVNOR-2278] fix compilation error

+1 to merge.

master: 44e9d9f62b898fa82e5fc0cee21c1e1ff2fced6f

master: 44e9d9f62b898fa82e5fc0cee21c1e1ff2fced6f


Fix for directed tests failure

SWDEV-203971:
directed_tests/runtimeApi/module/hipLaunchCooperativeKernel.tst - Disabling test temporarily until driver support is available.
directed_tests/runtimeApi/memory/hipArray.tst

Disabled hipArray.tst as behavior is random on different GPU's. It needs clean up as it uses driver and runtime API's. Disabling test temporarily.


[ISSUE #2321] Resolve security issues of Go SDK

Fixes #2321
Motivation
#2321
Modifications
#2321
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

LGTM


blog. 3d transform aspect ratio on blog items

https://github.com/drewwilson/atvImg
https://codepen.io/thomasxiii/pen/MaByJa

http://freebies.lorenzobocchi.com/


Next language

What language BERT model would you like to be released next?

Swedish:
https://github.com/af-ai-center/bert

ALBERT in Danish ;-)

Or you could do knowledge distillation on this model. Here's a ton of synopses and links: https://blog.inten.to/speeding-up-bert-5528e18bb4ea
Both the Huawei TinyBERT and the single layer model are both about a magnitude faster at inference than the base model.

Agree that a compressed Danish BERT (ALBERT/DistilBERT/TinyBERT/...),  would be great!

Agree that a compressed Danish BERT (ALBERT/DistilBERT/TinyBERT/...), would be great!

Joachim, I was suggesting that @emillykkejensen make one, not BotXO =]
But you could do it too! You're at a uni - you have lots of time.
ALBERT requires the source material but any kind of knowledge distillation method can run from the BERT weights posted here.
What would be really nice though would be if BotXO would post their code for text preprocessing here. I can see that they do everything lower-case but don't reduce repeated characters. That's a bit sad in my mind. I don't need a token for '=======' or whatever it was I saw in there. This function should do it:
def squeeze_strings(s):
  if type(s) == float:
    return s
  else:
    s = re.sub(r'(?P<rep>.)(?P=rep){2,}', r'\g<rep>\g<rep>', s)
    s = re.sub(r'(?P<rep>..)(?P=rep){2,}', r'\g<rep>\g<rep>', s)
    return s

There are only 25 words in the Danish language that should be affected by squeeze_strings() and in all cases it would just change the conjugation (e.g. from "bortopererer" to "bortoperer", that is from present case to imperative).

I would be surprised if the optimisation described above have any measurable impact on performance. Is it something that's been mentioned in the litterature? If so, I've missed it.
I don't have permission to open source the data fetching/preprocessing code (yet) but it's currently quite hacky, cleaning up bad stuff from the internet (there is a surprising amount of NSFW stuff on Common Crawl)
I'm currently in dialog with a norwegian proffessor about training AlBERT models :-)

The blog post that @grofte links to, and the ALBERT paper linked in there (https://openreview.net/pdf?id=H1eA7AEtvS) do state those speedups quite clearly. Fewer parameters of course also means a smaller burden on memory.
(And answering @grofte -- you're wrong, I'm not at a uni at this very point in time, and even if I was, that wouldn't mean I had lots of time. :) )

@jbingel sorry for being unclear: I was refering to the proposed squeeze_strings function.

@mollerhoj Oh, I don't think you would see a difference in performance. But you would probably train faster. The function is just stripping out nonsense filler with no semantic content.
But I'm guessing that you guys do normalization through decomposition, sentence splitting (how though?), lower case, strip everything not in ascii+æøå. BERT tokenizer should take take of everything else once it has the vocab file. The other stuff you do to prepare the data for pre-training shouldn't be that interesting for employing the model.

Danish cased model!


Create an Equality Package

Please describe the problem you have
Currently Contour equality is being duplicated across managed dependent resources. Create a pkg that performs the equality matching that can be leveraged across the project.

What I would suggest here is to write comparison functions that you can provide to conversion.EqualitiesOrDie
func Semantic() conversion.Equalities {
    e := Equalities{equalities.Semantic} // take base equalities from kubernetes

    e.AddFuncs(
        operatorCompareJob,
        operatorCompareDeployment,
    )

    return e
}

xref #47

Equality pkg created here.


mat-drawer-container triggers angular detection cycles on window:resize

Reproduction
https://stackblitz.com/edit/angular-se13bg?file=src%2Fapp%2Fsidenav-drawer-overview-example.html
Steps to reproduce:

<mat-drawer-container></mat-drawer-container>

Expected Behavior
mat-drawer-container should not trigger angular detection cycle when the window is resized
Actual Behavior
mat-drawer-container triggers angular detection cycles on window:resize
Environment

Angular: 9
CDK/Material: 9

Sounds like it could be resolved by https://github.com/angular/components/pull/15814 since the sidenav is set up to only trigger change detection if its margin has changed.


Suggestion: create a "class generator" for the Adonis CLI

I think is a good idea, to introduce a class generator for the Adonis CLI make command.

@NeroOficial thanks for the suggestion.
I just release version 1.5.0 in which you can run adonis make:transformer to generate a transformer class.


Protocol URLs throw mixed content warnings

When I try to load fonts in a WebView via a protocol URL, I get mixed content warnings:

Mixed Content: The page at 'https://TEAM.slack.com/messages/general/' was loaded over HTTPS, but requested an insecure font 'slack-resources:fonts/TTF/Lato-Hairline.ttf'. This content should also be served over HTTPS.

This prevents the font from being loaded unless disablewebsecurity is passed to the WebView; is there a way in Chromium to indicate that a particular protocol is "trusted"?

There are also allow_displaying_insecure_content and allow_running_insecure_content options in Chromium, I guess that's what you want?

@zcbenz That might work, but I don't want to disable this warning entirely since it seems like a Generally Good Idea™, only to mark my protocol handler as secure. If that involves a huge amount of work, I could also live with allow_displaying_insecure_content

Yesssssssssss :sparkles: :sparkles: :sparkles: :sparkles: :sparkles: :sparkles: :sparkles: :sparkles: :sparkles:

Is there a way to do the same for webview? I con only get this to work for the webframe my application is running in. I am embedding an web app that uses HTTPS and can not display images that are served via HTTP. Is there a way to fix that?

@pcomans any update on it? did you find the way to load http urls? I show external images and can't make it work :(

Accessing webFrame works in Webviews too, run it in your WebView preload script

Do you have any example of it? Where's the webview preload script? Thanks ;)

@bitomule you can instruct the webview to use a preload script by using the preload attribute.
In your preload script you should be able to use webFrame.registerURLSchemeAsSecure('yourscheme').

@etiktin but I'm not using webview. I just load my url in the window. Is it possible to allow http content if web is loaded that way?

@bitomule I'm not sure I understand you. If you're using a custom protocol and you want to register it as secure, you can use a preload script also with BrowserWindow and not just webviews and call webFrame.registerURLSchemeAsSecure('yourscheme') from there (see BrowserWindow options).
If that's not what you were asking, I suggest you ask the community.

@etiktin I'm using BrowserWindow and just loading a remote https url. The issue is that site loads non https sites entered by users. I tried BrowserWindow options to disable security but it doesn't work. If you have any idea it would be great, if not I'll try community.

@bitomule it does for me. What version are using? The docs in the master branch of github use the new options syntax introduced in v0.35 (e.g. web-preferences is now webPreferences), so if you are using an older version you should use the old option names or upgrade to the latest and use the new ones.

@etiktin thanks! Updated electron version and now it works. Thanks thanks thanks!!

@bitomule you're welcome.


Stereochemistry

How should we handle stereochemistry?
The phase angle in the Fourier series is restricted to 0 or 180. This allows the same torsion types to be used for enantiomers. If that's the case, do we need to consider enantiomers different molecules?
For the time being I'll add both the canonical isomeric SMILES and a canonical SMILES to the JSON specs for fragments.

I presume we currently want to use the canonical isomeric SMILES string as our way of judging whether fragments are unique. Later on, we may relax that.

For the incoming molecules to fragment - should we enforce isomeric SMILES? If it's not isomeric, should we generate an isomeric SMILES before we generate conformations of the molecule to fragment?
In general I need to add a cleanup step for incoming molecules - generating an isomeric SMILES can be part of that step. However, shouldn't we also enumerate enantiomers if several stereocenters exist? The fragments can be different  for different enantiomers.

To clarify here:

For incoming molecules, we need one stage that expands ambiguous stereochemistry and enumerates likely protonation/tautomeric states. At this point, all molecules will then have specific stereochemistry and explicit hydrogens.
We should expand the JSON generated to specify both explicit-hydrogen canonical isomeric SMILES and non-isomeric SMILES so that either can be used as a key for indexing.
The resulting molecules can then be distributed in parallel to another function, which fragments each of these molecules and passes fragments on to the next stage.

I think all five will be useful! But it's possible 3 and 5 are redundant if the tautomeric and charge state are both encoded by 2 and 4. This is probably a question for @bannanc or @cbayly13

Stereochemistry enumeration was addressed with https://github.com/openforcefield/fragmenter/pull/10.


Lint rules v 0.52

Also can you remove 0.52 from .alexignore?

I hope, that I changed all your requests. Sorry for the trouble. Hopefully, I could still help and everything is fine by now.

@mats-codes no need to apologize, all of your changes were great and no trouble at all!


{Authn, ...}::API.host method names are misleading

They really return URLs. I propose to have .url method instead and deprecate .host.

No plan to implement.


Cargo and crates signatures

There is no signature verification to authenticate the crates.io database. An attacker who compromise the crate.io repository/server/Github account (as happened to kernel.org) could spread the infection to every Rust developer and application user.
Here are some complementary solutions:

The hard step: only allow crates with signed Git tags (from their owner) and store their public keys in a dedicated repository only updatable (and signed) by some Rust core developers. This repository only need to be updated for new or updated keys.
The easier step: make @bors sign every commits (without tags). Obviously, this @bors instance should not run on the same server as the crates.io-index (e.g. Github) or be protected/isolated.
Make Cargo warn and ask user for confirmation about new or updated public keys.

The @bors' public key should be shipped with Cargo, which should be signed with a Rust core developer… Contrary to Rust, the Cargo repository isn't actually signed at all!

Following the RubyGems.org compromission, there is currently some discussions about protecting Python package downloads: Surviving a Compromise of PyPI (PEP 458 and PEP 480).

@l0kod what you're describing sounds a lot like you're trying to reinvent TUF, and PEP 458/PEP 480 is implementation of TUF.
Specifically TUF packages can be "unclaimed" and signed by an online key held by the packaging system itself, and "claimed" and signed by the author(s) of the packages (using a threshold signature scheme)
If you don't intend to implement TUF wholesale, I'd suggest reading their paper, sketching out a threat model, designing a usability story, and coming up with something a bit more concrete:
http://freehaven.net/~arma/tuf-ccs2010.pdf

I'll leave a note on this specifically:

The hard step: only allow crates with signed Git tags (from their owner)

The problem with using Git tags (beyond the actual mechanics of how you intend to use them) is they rely on the SHA1 hash function for integrity, as this is what's used for Git's Merkleized DAG.
SHA1 is already known to be problematic, and SHA256+ is recommended for new projects.
If a second preimage attack were found against SHA1, it would be catastrophic for anything using git tags in this capacity.
Now granted that a second preimage attack is yet to be found against MD5 which is known to be much more broken than SHA1. However, if we don't rely on git for computing digests, we can pick a much stronger hash function.
I'd also note that Linus does not consider Git's usage of SHA1 to be a "security feature", so it's questionable as to whether Git will end up using a stronger hash function in the future:

Nobody has been able to break SHA-1, but the point is the SHA-1, as far as Git is concerned, isn't even a security feature. It's purely a consistency check. The security parts are elsewhere, so a lot of people assume that since Git uses SHA-1 and SHA-1 is used for cryptographically secure stuff, they think that, OK, it's a huge security feature. It has nothing at all to do with security

-- Linus Torvalds

I'm going to close this in favor of https://github.com/rust-lang/crates.io/issues/75 in light of @tarcieri's comment. I also wouldn't mind creating a more general "securing crates.io" issue, but it sounds like https://github.com/rust-lang/crates.io/issues/75 is quickly becoming accepted practice and may be the best route for that regardless.

As an aside, I'd advocate using a memory hard hash function like Argon2 for tools like git.  If quantum computers are possible, then Grovers algorithm half's your security parameter.  Just because you adopt 256+ bit hashes doesn't mean people use all the bits when discussing commits.  Argon2 should protect against Grover unless adversary to build a quantum computer with millions or billions of qubits.

@burdges that doesn't make any sense. Argon2 is a password hashing algorithm. Its construction is intended to mitigate time-memory tradeoffs when brute forcing low-entropy passwords. This is completely irrelevant for selecting the hash function for a content addressable storage system like git.
For a hash that tolerates Grover and targets a 128-bit security level, we:

Square once for the birthday bound
Square again for Grover

This puts us at a 512-bit hash function. SHA2-512 should be sufficient.

Ain't so likely anyone will change git soon, but if one considers reinforcing it by keeping records in the package manager, then one needs a better hash function.   Argon2 is susceptible to Grover like everything, but it'd still protect against time-memory trade offs when brute forcing higher-entropy data, thus requiring a ridiculously large quantum computer.  That'd double the security of the shorter substrings that humans sometimes handle.  I suspect it's easier to protect against time-memory trade offs in a quantum computer, so maybe you don't need anything as fancy as Argon2, but I haven't thought about it carefully.

Argon2 is susceptible to Grover like everything, but it'd still protect against time-memory trade offs when brute forcing higher-entropy data

A 128-bit post-quantum security level is outside the realm of brute force attacks. SHA2-512 is sufficient.
The NSA recently updated Suite B to harden it against potential future attacks by quantum computers. This is their recommendation for a hash function:
https://www.nsa.gov/ia/programs/suiteb_cryptography/index.shtml
Use SHA-384 to protect up to TOP SECRET.

Instead of Argon2, I think BLAKE2b would be a compelling alternative, but as @tarcieri notes, SHA2-384 and SHA2-512 are sufficient.

+1 for BLAKE2b-512. It is also a good candidate and performs very well in software.

Okay, this ticket is closed but I couldn't help myself from bike shedding on hashing algos.  First of all, if we used Argon2 we would burn the sh!t out of our laptops because we would never finish hashing the files 😱!
BLAKE2 is a good replacement for SHA1, but we really shouldn't put all of our eggs in one basket as BLAKE2 has a ~112 bit security level and is too similar to SHA2.  KangarooTwelve is looking like a good partner for BLAKE2 as it's very fast and very different from SHA2.

There's really no reason to use off-the-rails hash functions (despite my +1 to BLAKE2 above).
SHA(2)-256 is used almost universally due to widespread hardware support, including but not limited to AVX2 implementations (not used in the BLAKE2 benchmark chart or benchmarks showing SHA-512 is faster than SHA-256, for example, but supporting CPUs dating back to 2008) as well as the new SHA Extensions and AVX-512 introduced in e.g. Skylake. SHA-256 is also pretty much going to be the only hash function you'll commonly find in hardware on MCUs.
Keccak-based hash functions have an internal state of 1600-bits, making them much less amenable to e.g. AVX-512 optimizations (which can run 4 SHA-256 rounds in parallel using simultaneous hashing). The use of a sponge function makes them nicely distinct from the SHA2 family, however.
If you want a Keccak-based alternative to SHA2-256, there's little reason not to use SHA3-256. This is the only Keccak-based construction likely to be implemented in hardware at some point in the future, and also the one CPU manufacturers will be targeting for optimized software implementation.
There is also little reason to suspect quantum computers will be able to attack either SHA2-256 or SHA3-256, despite the NSA recommending a move to SHA2-384+. In a post-quantum scenario, both will still have 128-bits of preimage resistance and quantum attacks are unlikely to improve the performance of collision-finding.

This is the only Keccak-based construction likely to be implemented in hardware at some point in the future

at this point It would make more sense to implement keccak-f in hardware instead.


Show only a specific cluster in resulted image

Hi,
After using  provided google Colab demo, I just wanted to know if this could be possible to show only a desired cluster on the final resulted image?(Not all the detected ones.) I saw Issue 18 but did not understand how to this task. As an example this is an image I tried to segment:

As you can see there are almost 4 clusters on the image for each method. How could I only segment/plot the animal(s) cluster on the final image for each method?
Thanks

You can edit the plotting code with a quick predicate which checks whether yours preds are equal to the class you care about. Heres a quick idea (you might nd to edit this a bit to get it to work)
import matplotlib.pyplot as plt
from utils import unnorm, remove_axes
fig, ax = plt.subplots(1,3, figsize=(5*3,5))
ax[0].imshow(unnorm(img)[0].permute(1,2,0).cpu())
ax[0].set_title("Image")
ax[1].imshow(cluster_pred == MY_CLUSTER_CLASS)
ax[1].set_title("Cluster Predictions")
ax[2].imshow(linear_pred == MY_LINEAR_CLASS)
ax[2].set_title("Linear Probe Predictions")
remove_axes(ax)


Permission to access pypi / setup release / push action and semantic versioning

hi @hozn are you comfortable with giving me super power access to this repo to add secrets and such? And also with potentially sharing pypi credentials so I can setup an action to release and push there?
If you are you could email me at leah at pyopensci.org . i'm happy to help move things along here and to continue to contribute where I can! i think this package is really fantastic and i'd like to see it grow and become stable with a small maintenance team supporting it.
I am working on a project now called pyopensci full time. the goal is to support maintainers working on projects like this.

Hi @lwasser  -- yes, I'm happy to do that.  I'm unsure how to grant additional privs in github, but I'll reach out by email and we can coordinate that + pypi.


Usage of setConsumption()

In one-to-one conversation how to set read / setconsumption? What should be the value of horizon?  can anyone provide an example
Account type

Conversation details

Steps to reproduce

Result or traceback

It might help to know why you want to do this.  SkypeChat.setConsumption() exists more as an implementation detail -- the only real use of it is to mark all messages as read, which SkypeMsg.read() does for you:
https://github.com/OllieTerrance/SkPy/blob/03042f1732dd103ba252af9e27e2bf76b62a3b4e/skpy/msg.py#L262-L266

I want to mark the messages from a particular user as read till a time t, since when i'm having a one to one conversation the read messages are trigged as newmessage event when i use getEvents().

Calling SkypeMsg.read() will emit a SkypeChatUpdateEvent, which you should then ignore -- you've already consumed up to the horizon it specifies as you were the one to set it; the event exists to allow other clients to update their local state.

What value should i pass to the horizon argument. can you give an example for this ,, .

I already did, see SkypeMsg.read() above.  It's also stated in the docs:

horizon (str) -- new horizon string, of the form <id>,<timestamp>,<id>

 and  values

Once again, from the sample above:

a message ID, such as SkypeMsg.clientId
a UNIX timestamp, such as time.time() for the current time, in milliseconds

Sorry,still i'm getting the same error.
ch.setConsumption(horizon=(z[0].clientId,time.time(),z[0].clientId))
File skpy/chat.py", line 207, in setConsumption
json={"consumptionhorizon": horizon})
File "/skpy/conn.py", line 219, in call
raise SkypeApiException("{0} response from {1} {2}".format(resp.status_code, method, url), resp)
skpy.core.SkypeApiException: ('400 response from PUT https://hk2-2-client-s.gateway.messenger.live.com/v1/users/ME/conversations/8:live:abcd/properties', <Response [400]>)

You're sending a tuple, not a string.  The response will probably hint at invalid parameters if you look at it.

I checked type of horizon it returning a str / string.

horizon=(z[0].clientId,time.time(),z[0].clientId)

That is most definitely a tuple of three items, not a string, so I don't know where you attempted to check it.

It appears you're not very familiar with Python. I would suggest brushing up on the basics before trying to use a library like this one, it's not a particularly good resource for newcomers.  Learn Python the Hard Way may be a good place to start.
Questions or bugs specific to the library are of course welcome here, but you're expected to bring a basic understanding of the language you're attempting to use.  Until then, there are better places to ask for general Python support.

I can't tell you much more unless you actually post your code and the API output as requested in the issue template.

Steps to reproduce
Explain how to (reliably) recreate the issue.  A short code sample may be useful.
If applicable, test the corresponding action on https://web.skype.com to see if it works there.
Result or traceback
Please set SKPY_DEBUG_HTTP=1 in your environment to capture API requests to Skype's servers.
Include the relevant parts (not the entire output), ```enclosed in backticks``` for readability.
Make sure to remove any personal information such as passwords or tokens before posting.


Problem with image position

There seems to be some incompatibility, disabling Copy byttons for code blocks fixed the width problem.

Ah yeah, this is a known issue for plugins that modify code blocks in a specific way. It's already mentioned in #4, but basically, since I attach the banner to the frontmatter (which acts like a code block in the preview view) with some styling, any plugin that uses their own styling can clash with mine, & it messes things up from there. Unfortunately I (currently) have no way to apply the banner outside of the frontmatter block, so the best mitigation would be to have the other plugin ignore this plugin's styling.
As a hacky workaround though (since I messed with that plugin a little bit), you can create and enable a small CSS snippet with this:
.obsidian-banner-wrapper.has-copy-button {
	position: initial;
}

And it should put the banner back into place, with a copy button that's a tad misleading on the top right, so hopefully that's an alright workaround for now

Gonna close this due to inactivity


Merge of OCW-8 and OCW-9

Coverage decreased (-3.5%) to 66.24% when pulling d9fb074bd178479c7bd6c6e049c9cd8244bfc3a3 on master into 73058300a534b0b0d9e247fa56d63f24af173cef on production.


Mention tape-describe in README

Hi James. I've written a really simple package called tape-describe that prefixes test names with a given description. Think it's worth a mention?
NPM: https://www.npmjs.com/package/tape-describe
GitHub: https://github.com/mattriley/tape-describe

Sure, seems fine.


libqalculate, qalculate-gtk: 4.1.1 -> 4.2.0

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thanks!

This PR broke the build on aarch64-linux, transitively breaking plasma.  The issue was shown even before merging in the OfBorg CI.

The issue was shown even before merging in the OfBorg CI.

It is very confusing that a build failure is listed as "28 checks passed".

I think OfBorg never makes CI red, maybe because for many packages it runs into timeouts or something. I'm not even sure if I've ever seen nixpkgs CI red.
Anyway, the interesting part of config.log is probably
configure:6828: checking whether the C compiler works
configure:6850: aarch64-unknown-linux-gnu-gcc -g -O2   conftest.c  >&5
/nix/store/mskx817hz1irhv9q0b6lvdaqbwimv4fp-binutils-2.38/bin/ld: cannot find crt1.o: No such file or directory
/nix/store/mskx817hz1irhv9q0b6lvdaqbwimv4fp-binutils-2.38/bin/ld: cannot find crti.o: No such file or directory
collect2: error: ld returned 1 exit status
configure:6854: $? = 1
configure:6894: result: no
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME "libqalculate"
| #define PACKAGE_TARNAME "libqalculate"
| #define PACKAGE_VERSION "4.2.0"
| #define PACKAGE_STRING "libqalculate 4.2.0"
| #define PACKAGE_BUGREPORT ""
| #define PACKAGE_URL ""
| #define PACKAGE "libqalculate"
| #define VERSION "4.2.0"
| /* end confdefs.h.  */
|
| int
| main (void)
| {
|
|   ;
|   return 0;
| }
configure:6899: error: in `/build/source':
configure:6901: error: C compiler cannot create executables
See `config.log' for more details


Use ${CC} instead of hard-coded gcc

${CC} is the predominantly used "compiler", but there are some hard-coded references to gcc which this removes.

Thanks for the change, that makes sense.


Every fetch request gets stuck in Pending mode (iOS)

Description
Every-time I try to do an HTTP request on iOS, the request gets stuck in pending mode and nothing happens. It works fine on android and it previously also worked for iOS but then I bundled up the android app for play store and now iOS is randomly broken...
I installed a library even to visually show that's stuck:

I have tried to ping different API's but every Fetch request just fails. The metro bundler has also become slower and the startup of the app is also slower than before. To me it's seems like a XCODE/iOS simulator issue perhaps?
React Native version:
System:
OS: macOS 10.15.6
CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Memory: 781.70 MB / 16.00 GB
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 14.10.1 - ~/.nvm/versions/node/v14.10.1/bin/node
Yarn: 1.22.5 - /usr/local/bin/yarn
npm: 6.14.8 - ~/.nvm/versions/node/v14.10.1/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.8.3 - /Users/dkjogrpe/.rbenv/shims/pod
SDKs:
iOS SDK:
Platforms: iOS 14.0, DriverKit 19.0, macOS 10.15, tvOS 14.0, watchOS 7.0
Android SDK:
API Levels: 23, 28, 29
Build Tools: 28.0.3, 29.0.2
System Images: android-28 | Google Play Intel x86 Atom, android-29 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312
Xcode: 12.0.1/12A7300 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_222 - /usr/bin/javac
Python: 3.7.1 - /Users/dkjogrpe/.pyenv/shims/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: 0.63.3 => 0.63.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Released an android version and then suddenly iOS broken, seems like a random occurence

Expected Results
I expected iOS to work as before
Snack, code example, screenshot, or link to a repository:
I can send over the repo, it's currently private and it's not possible to make a minimal viable example since it's just all of a sudden happened. It could be a problem with the iOS simulator/XCODE though or something in the iOS setup, I have tried to roll back a bunch of commits and even on an old version, the problem is still there

Did some more digging,
It works everywhere else but the iOS simulator. As a result i tried to reinstall XCode with the newest version but still same issue :(

Has same issue, but its work well on ios (both simulator and real device), but in one of my Android app which is in store and hasn't been updated some of requests suddenly stopped receive answers

For those people with similar problem, Temporarily solved this my opening Xcode and running the simulator from there. Everything updated to latest version.
Maybe close this issue

Hi @jonasgroendahl appreciate investigating and sharing your solution. I'll close for now as it seemed to be an environment issue.

I had this exact issue where fetch requests would simply not complete on mac in the ios simulator. I tried everything: updating npm packages, clearing all caches, creating a new react native app using the cli, rebooting my computer, pulling out my hair. What finally resolved it was updating osx from catalina to big sur. Now xcode wont start but my app works.

I had this exact issue where fetch requests would simply not complete on mac in the ios simulator. I tried everything: updating npm packages, clearing all caches, creating a new react native app using the cli, rebooting my computer, pulling out my hair. What finally resolved it was updating osx from catalina to big sur. Now xcode wont start but my app works.

The issue returned. Found this issue, removing @babel/plugin-proposal-class-properties ultimately resolved the issue for me.


A query of a TPH throws exception when selecting a property via discriminator and cast

I have a query that looks a a TPH DbSet consist of Book and BookwithOffer. I try to extract a value in the BookWithOffer, with a test on the discriminator. This works for a string, but not for a decimal. The SQL looks OK (see near the end of this comment), but I get the following exception.
I can confirm that the TPH table works property, as I have unit tests that checks that.
System.NullReferenceException : Object reference not set to an instance of an object.
   at lambda_method(Closure , QueryContext , ValueBuffer )
   at Microsoft.EntityFrameworkCore.Query.ExpressionVisitors.Internal.ProjectionShaper.TypedProjectionShaper`3.Shape(QueryContext queryContext, ValueBuffer valueBuffer)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.Enumerator.BufferlessMoveNext(Boolean buffer)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.Enumerator.MoveNext()
   at Microsoft.EntityFrameworkCore.Query.Internal.LinqOperatorProvider.ExceptionInterceptor`1.EnumeratorExceptionInterceptor.MoveNext()
   at System.Collections.Generic.List`1.AddEnumerable(IEnumerable`1 enumerable)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Test.UnitTests.ServiceLayer.Ch13_ListBooksService.TestBookListDtoSelectOk() in C:\Users\Jon\Documents\Visual Studio 2017\Projects\EfCoreInAction\Test\UnitTests\ServiceLayer\Ch13_ListBooksService.cs:line 39

Steps to reproduce
Here is a snippet of the code I am trying to develop.
public class Book
{
    public int BookId { get; set; }

    [Required] //#A
    [MaxLength(256)] //#B
    public string Title { get; set; }
    public string Description { get; set; }
    public DateTime PublishedOn { get; set; }
    [MaxLength(64)] //#B
    public string Publisher { get; set; }
    public decimal ActualPrice { get; set; }
    //This controls the TPH - true if its a BookWithOffer
    public bool HasOffer { get; set; }

    [MaxLength(512)] //#B
    public string ImageUrl { get; set; }
    public bool SoftDeleted { get; set; }

    //-----------------------------------------------
    //relationships

    //public BookWithOffer Promotion { get; set; }
    public ICollection<Review> Reviews { get; set; }
    public ICollection<BookAuthor>
        AuthorsLink { get; set; }
}

public class BookWithOffer : Book
{
    public const int PromotionalTextLength = 200;
    public decimal OldPrice { get; set; }
    [Required]
    [MaxLength(PromotionalTextLength)]
    public string PromotionalText { get; set; }
}

My DTO looks like this
public class BookListDto
{
    public int BookId { get; set; }
    public string Title { get; set; }
    public bool HasOffer { get; set; }
    public decimal OldPrice { get; set; }
    public decimal ActualPrice { get; set; }
    public string PromotionPromotionalText { get; set; }
}

My query looks like this
var books = context.Books.Select(p => new BookListDto
{
    BookId = p.BookId,
    Title = p.Title,
    ActualPrice = p.ActualPrice,
    HasOffer = p.HasOffer,
    OldPrice = p.HasOffer
        ? p.ActualPrice
        : ((BookWithOffer)p).OldPrice,
    PromotionPromotionalText =
            p.HasOffer
              ? "unit test!"
              : ((BookWithOffer)p).PromotionalText
}).ToList()

My config is
public void Configure
    (EntityTypeBuilder<Book> entity)
{
     //... other configs removed for clarity

    entity.HasDiscriminator(b => b.HasOffer)
        .HasValue<Book>(false)
        .HasValue<BookWithOffer>(true);

    entity.Property(p => p.HasOffer)
        .Metadata.AfterSaveBehavior = PropertySaveBehavior.Save;
}

The SQL that I captured looks like this.
SELECT "b"."BookId", "b"."Title", "b"."ActualPrice", "b"."HasOffer", CASE
    WHEN "b"."HasOffer" = 1
    THEN "b"."ActualPrice" ELSE "b"."OldPrice"
END AS "OldPrice", CASE
    WHEN "b"."HasOffer" = 1
    THEN 'unit test!' ELSE "b"."PromotionalText"
END AS "PromotionPromotionalText"
FROM "Books" AS "b"
WHERE "b"."HasOffer" IN (1, 0) AND ("b"."SoftDeleted" = 0)

Further technical details
EF Core version: 2.0.0
Database Provider: Microsoft.EntityFrameworkCore.Sqlite (for unit test)
Operating system: Window 10
IDE: VS2017 15.3

The above error suggest that, when reading values from database the value was NULL but it was being stored in a non-nullable property. So NRE.
In your model, you have set discriminator (HasOffer) to be false for book & true for BookWithOffer.
But in your query you are calling OldPrice = p.HasOffer ? p.ActualPrice : ((BookWithOffer)p).OldPrice
If p.HasOffer is false (means its a Book) then cast to BookWithOffer will fail. Since all of this is happening in database as you can see in SQL, for a book "b"."HasOffer" will be 0 hence, it will get value of "b"."OldPrice" column. Since that column does not exist for Book, in TPH it will have null value. When this returned NULL value is materialized into BookListDto.OldPrice which is decimal (non-nullable), NRE is thrown. Probably you need to invert conditions for OldPrice & PromotionPromotionalText both.
Following query works correctly
                var books = db.Set<Book>().Select(p => new BookListDto
                {
                    BookId = p.BookId,
                    Title = p.Title,
                    ActualPrice = p.ActualPrice,
                    HasOffer = p.HasOffer,
                    OldPrice = !p.HasOffer
                        ? p.ActualPrice
                        : ((BookWithOffer) p).OldPrice,
                    PromotionPromotionalText =
                        !p.HasOffer
                            ? "unit test!"
                            : ((BookWithOffer) p).PromotionalText
                }).ToList();

Doh! So sorry - I just didn't see that.


Potential README word correction

In the README:

Elm-Ui is optioned. Buttons, Input fields and Labels comes with a predefined style - that is not documented.

For the final word in first sentence, optioned. Do you mean opinionated?

Thanks a lot for pointing this out!  🙏🙏


zappa deploy should check if lambda function already existed?

When updating function, I should call zappa update dev instead of zappa deploy dev. But I think a nice tweak would be to check if function already existed so it won't waste time uploading the zip?

+1

Related to https://github.com/Miserlou/Zappa/issues/78


Error in Stripe Payment

If the admin has a payment method other than stripe (for example, manual), an error occurs here:
https://github.com/medusajs/nextjs-starter-medusa/blob/214595863e16e2eabfb5e10202215f61f3a2e7bd/components/checkout/injectable-payment-card.jsx#L29-L36
This is because payment_session is not part of the received data in the cart from the request /store/carts/{id}/payment-sessions

After investigating a little more I think this endpoint should be called before the stripe payment is processed.


average value in log

[x] I have marked all applicable categories:

[ ] exception-raising bug
[ ] RL algorithm bug
[ ] system worker bug
[ ] system utils bug
[ ] code design/refactor
[x] documentation request
[ ] new feature request

[x] I have visited the readme and doc
[x] I have searched through the issue tracker and pr tracker
[x] I have mentioned version numbers, operating system and environment, where applicable:import ding, torch, sys
print(ding.__version__, torch.__version__, sys.version, sys.platform)

v0.4.6 1.12.0 3.7.13 (default, Mar 29 2022, 02:18:16)  [GCC 7.5.0] linux
after a few iterations, the training iteration result will be printed and recorded in tensorboard. When it comes to the total_loss_avg, in what dimension is this average taken? Is it the average for every 100 iterations or the average for the entire training process from start to now?

The specific average interval is set in this line, the current argument is expire=10, which means it is the average for each 10 iterations.

The specific average interval is set in this line, the current argument is expire=10, which means it is the average for each 10 iterations.

Thanks! This is very clear.


Doubts for installation

Hello, Good morning excellent work I discovered it and I found it excellent. Congratulations.
But I have a question at the time of installation, the command "npx wpgulp" has to be executed in the root folder or inside the root / src folder, I confused that part a bit which would be the right thing.

root -> npx wpgulp
or
root / src -> npx wpgulp

Thank you.
PD: I speak Spanish, and the comment was made using google Translator, sorry if there is something wrong
-----------^ Click "Preview" for a nicer view!
This is a FOSS (Free & Open Source Software) project. Providing support is a full-time job and so it's not provided for free. No promises there. 😇
If you'd like to support this project or hire Ahmad Awais (maker of WPGulp) then feel free to contact here → https://AhmadAwais.com/contact/.

If you'd like us to keep producing professional free and open source software (FOSS). Consider paying for an hour of our dev-time. We'll spend two hours on open source for each contribution. Yeah, that's right, you pay for one hour and get both of us (Awais & Maedah) to spend an hour as a thank you.
🚀 $99.99 — Support for one hour or more →
🔰 $49.99 — Support half an hour maintenance →
☕ $9.99 — Buy us lunch or coffee to keep us trucking #OpenSource →

Follow the directions here https://github.com/ahmadawais/WPGulp#getting-started
Root of your theme or plugin that is where run this.


Constrain resizing the cropping rectangle

Hi,
Is there a way to constrain the resizing of the cropping rectangle to always be a square instead of freeform?
Regards,

see wiki


updated spade tutorial to work with viziphant 0.2.0 (#88)

This PR updates the spade tutorial to work with viziphant v0.2.0.
With the new version of viziphant a new modul called patterns was introduced, see: https://github.com/INM-6/viziphant/releases/tag/v0.2.0 .
With this PR, cell [6] from the spade tutorial
viziphant.spade.plot_patterns(spiketrains, patterns)

is changed to:
viziphant.patterns.plot_patterns(spiketrains, patterns)

All Travis CI jobs running Python 3.6 seem to fail.
The reason might be, that pipv22.0 does no longer support Python 3.6 ?, see: https://pip.pypa.io/en/stable/news/


on launch: "failed to spawn stub: search: not found"

Since upgrading recently, every time elvish is launched it prints: failed to spawn stub: search: not found on launch
That is on NetBSD 7. On Debian wheezy, similar error occurs:
Warning: cannot connect to store: failed to initialize table cmd: database is locked
failed to spawn stub: search: not found

After rm -Rf ~/.elvish/*, failed to spawn stub: search: not found

Please refer to the updated building section of README.

To fix the first error, do rmdir ~/.elvish/db.lock. There used to be bugs that will leave the database locked when elvish exits, but hopefully they have been fixed.

Also, NetBSD is likely also affected by #152.


Update examples code

---------- WIP -----------
When adjusting the cylinder orientation to the new one, I discovered that the examples are implemented in the Z up left-handed coordinate system.
This PR updates the examples to use the OpenGL and WebGL standard coordinate system Y up right-handed.
While at it i decided also to

replace TrackballControls with OrbitControls, they make more sense
convert the examples to modern ES6 syntax

To test this PR locally:
npx serve

The visual result should be the same, since only the code changes.

This is awesome, I think it will be much easier for newcomers to understand the coordinate system

I didn't read this clearly the first time. The ES6 conversion is even more awesome than the adjusted coordinates. I'm super excited to have this.

@codynova this PR is finally ready for review 🥳

Amazing work, this is massive 🎺


pythonPackages.nixpart0: fix nixops deploy

re-introduce the unused "udevSoMajor" variable for nixops
backward-compatibilty (nixops master still uses it unlike claimed by previous comment).
This was removed by the treewide 52f53c69ce6dbc5538f7e4cd22f9d93baf1f64a2 cleanup
Motivation for this change
Fix nixops deployments
Things done

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

[ ] 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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

Is there also a pull request hat removes the override from NixOps?

There is this closed PR: https://github.com/NixOS/nixops/pull/347
Since nixpart0 is legacy I think it's fine to keep this around. NixOps should be upgraded to the latest nixpart instead.


freecad_dict format

About the dictionary we use to feed Geometry3D.
A
Recently we changed from file_path to serial_fcdoc. This means instead of
freecad_dict = {
    'pyenv': 'python2',
    'file_path': 'geometry_sweep_showcase.fcstd'
}

the user has to write
freecad_dict = {
    'pyenv': 'python2',
    'serial_fcdoc': Geometry3D.serialize_fc_file('geometry_sweep_showcase.fcstd')
}

Is the template doc required to be serialised? Might be a future proof choice in case it is not transfered on disk anymore, so I agree with that.
Is the doc required to be serialised by the user? We could call serialize_fc_file in the Geometry3D.__init__, then the user can write the more convenient top snippet.

On a related note, we have that kind of serialisation code in various places (qmt/geometry/freecad/fileIO.py, qmt/data/geo_data.py and now qmt/tasks/geometry.py). What would be a good place that is importable by Python2 and Python3, to hold a set of unified "deal with pipeable serial data" functions?
B
The Geometry3D docstring states that the dict requires
"parts_dict": parts_dict = {part_name: Part3D}

The current implementation is
"input_parts":input_parts_list = [Part3D]

Since Geometry3D "consumes" (or is at least the sole user of) these parts, and the returned geo object contains the part names, why is there a need to set part_name at this point?
I plan to remove parts_dict and document input_parts as the build order spec.

For point A - yes, it's required to be serialized. I actually changed this after many hours of debugging cryptic errors :). The problem was that all tests that had been run used a shared directory, which we don't want to assume using docker. Everything that a task needs we want to assume is either handed to it by previous task output (and is serializable) or is explicitly passed in through the options dictionary.
I agree with unifying the serialization routines and putting them somewhere. Perhaps in a new file in the data module? However, I think it's a good idea to (at this stage at least) require the user to explicitly serialize the data. The reason is that we want the options dictionary to really be the object that gets passed to _solve_instance. This makes debugging a lot easier. Maybe a compromise would be putting an explicit check to give a helpful error message if type str is passed in?
For B - I totally agree. part_name is obsolete now that the parts are objects. I noted this in a Todo when I noticed that wire shells actually targets parts now (rather than a str corresponding to the part name). Internally, I think that we always access the part_name's from the items in build_order, so it's important that the names of the step files correspond to the strings that are written in there.
Perhaps an easy fix for now would just be to make the label field be automatically set by the name of the Part3DData class being instantiated? That way we wouldn't break any dependencies.

I'll deal with the unified serialisation tomorrow.
I disagree with the rationale behind requiring the user to serialise manually. The debugging is clear enough (you get an additional big blob attr quite early along the way instead of at the very beginning) and the difference in user convenience and "example niceness" is substantial IMO.
In case you're firm on this, I will add a more helpful error message. If a user gets the idea to put in a string despite the current variable naming and us only showing examples that manually call serialize_fc_file, the current error cryptically complains about padding. (then again, auto-serialising in the init preempts this ;) )
Part3D.label holds the pretty human-given name of each part at construction (in the user run.py). Each part knows it's own name.
This name is then added to Geo3DData.build_order during the construction of Geo3DData in the Geometry3D task, used as key in the Geo3DData.parts dict (since Kevin needs exactly that kind of access) and in the export naming.
The question about part_name and parts_dict here is resolved, the rest about build order pertains to #33.

Looks good, thanks!


Allows the launcher to set a min memory amount of 1GB instead of 3GB.

7c40327 Basically what the title says. Been bugging me for a while so I figured I'd just create a pull request.
697eb97 Updates everything to the latest versions. No issues that I could find.

Thanks, although the ram can be changed per fork if your modpack demands less memory (which it probably should). Not going to change this at the moment, but I will include this in a todo list for the eventual rewrite.
As for the dependencies, I'm going to give electron-builder some time to catch up before moving to  electron 6.


Save Parameter key and values

I am curious if there is a way to save a query parameters value for later use instead of having to re-enter it each time one goes to the Query Detail page? I only ask because I have not been able to find an answer to that with Google and I do not see anything in the codebase to do it, but maybe I am missing something. Thank you

@dwmorrison33 could you provide an example of what you mean please?

@marksweb Thank you for your response, below is an example. I create a new query with id as the parameter and execute it:

After navigating back to the Query List page and clicking again on this query, the parameter id no longer has the value 10000 populated:

Is there a way to persist this value? Also, on a side note, I am using version 1.1.3 of your package since my company is on Django 1.11 and that is the last version that is compatible.

@dwmorrison33 Right, I'm with you - and I thought that was what you meant. I'm fairly sure the app can't do that, at least as it is now.
My suggestion would be to create bookmarks with the query params defined in them.

@dwmorrison33 Right, I'm with you - and I thought that was what you meant. I'm fairly sure the app can't do that, at least as it is now.
My suggestion would be to create bookmarks with the query params defined in them.

Would you mind if I implemented a feature to save the query Params so that we can save a default value and then put in a pull request, as saving bookmarks will become a bit laborious across the organizations users who are using these queries each day. I just want to check before I work on this feature and you all not wanting it, but I think it would be useful to others as well.

@dwmorrison33 Yeah that sounds like it'd be really useful.
Create a branch from the 1.1.3 tag and when you come to a PR, target this new branch I've setup; https://github.com/groveco/django-sql-explorer/tree/release/1.1.4
Then I can do a 1.1.4 release and then, hopefully, merge it to the main branch for it to benefit the users of the latest version as well 👍

@marksweb I had a chance to work on the changes yesterday and finished them up this morning, but do not have permission to push up the code to the repo. Is there another way to push the code so that I can create a PR and request to merge into https://github.com/groveco/django-sql-explorer/tree/release/1.1.4?

@dwmorrison33 ok, so not having permission is correct.
The workflow for contributing is among the lines of...

fork the repo
clone your fork to your local machine
add the main repo as a remote (this allows you to pull changes in to your fork and base branches from upstream branches like we want here)
create a branch for your contribution and remember to check for guidelines on the repo
once complete and all changes are pushed to your fork github will allow you to create a pull request from the branch.
the PR will be reviewed & merged if accepted

See my comment on the PR. I believe this is already possible via this feature (as per the docs):

You can combine both a default and label to your parameter but you must start with the label: $$paramName|label:defaultValue$$.


[pixman] Build error - Windows

Package: pixman:x86-windows -> 0.42.2#2
Host Environment

Host: x64-windows
Compiler: MSVC 19.27.29120.0
vcpkg-tool version: 2023-06-22-f19f3d9939100085fe2f7e41502d60b08e2083b6
vcpkg-scripts version: 64adda19c 2023-07-02 (6 hours ago)

To Reproduce
vcpkg install freetype giflib gtest libjpeg-turbo libpng libwebp pixman tinyxml zlib
Failure logs
-- Downloading https://gitlab.freedesktop.org/pixman/pixman/-/archive/37216a32839f59e8dcaa4c3951b3fcfc3f07852c/pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz -> pixman-pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Disabling metrics because vcpkg.disable-metrics exists
[DEBUG] Trying to load bundleconfig from C:\src\libresprite\vcpkg\vcpkg-bundle.json
[DEBUG] Failed to open: C:\src\libresprite\vcpkg\vcpkg-bundle.json
[DEBUG] Bundle config: readonly=false, usegitregistry=false, embeddedsha=nullopt, deployment=Git, vsversion=nullopt
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Feature flag 'dependencygraph' unset
Downloading https://gitlab.freedesktop.org/pixman/pixman/-/archive/37216a32839f59e8dcaa4c3951b3fcfc3f07852c/pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz
warning: Download failed -- retrying after 1000ms
warning: Download failed -- retrying after 2000ms
warning: Download failed -- retrying after 4000ms
error: Failed to download from mirror set
error: https://gitlab.freedesktop.org/pixman/pixman/-/archive/37216a32839f59e8dcaa4c3951b3fcfc3f07852c/pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz: WinHttpSendRequest failed with exit code 12175
error: https://gitlab.freedesktop.org/pixman/pixman/-/archive/37216a32839f59e8dcaa4c3951b3fcfc3f07852c/pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz: WinHttpSendRequest failed with exit code 12175
error: https://gitlab.freedesktop.org/pixman/pixman/-/archive/37216a32839f59e8dcaa4c3951b3fcfc3f07852c/pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz: WinHttpSendRequest failed with exit code 12175
error: https://gitlab.freedesktop.org/pixman/pixman/-/archive/37216a32839f59e8dcaa4c3951b3fcfc3f07852c/pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz: WinHttpSendRequest failed with exit code 12175
[DEBUG] D:\a\_work\1\s\src\vcpkg\base\downloads.cpp(1003):
[DEBUG] Time in subprocesses: 0us
[DEBUG] Time in parsing JSON: 31us
[DEBUG] Time in JSON reader: 0us
[DEBUG] Time in filesystem: 506us
[DEBUG] Time in loading ports: 0us
[DEBUG] Exiting after 9.1 s (9077405us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):

      Failed to download file with error: 1
      If you are using a proxy, please check your proxy setting. Possible causes are:

      1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
         to `https://address:port`. This is not correct, because `https://` prefix
         claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
         , etc..) is an HTTP proxy. Try setting `http://address:port` to both
         HTTP_PROXY and HTTPS_PROXY instead.

      2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
         set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
         The value set by your proxy might be wrong, or have same `https://` prefix issue.

      3. Your proxy's remote server is out of service.

      If you've tried directly download the link, and believe this is not a temporary
      download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
      to report this upstream download server failure.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
  scripts/cmake/vcpkg_from_gitlab.cmake:118 (vcpkg_download_distfile)
  ports/pixman/portfile.cmake:45 (vcpkg_from_gitlab)
  scripts/ports.cmake:147 (include)

Additional context

The download link seems to work fine when copied & pasted into a browser...
https://gitlab.freedesktop.org/pixman/pixman/-/archive/37216a32839f59e8dcaa4c3951b3fcfc3f07852c/pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz

Which windows version do you have?

I don't know, but I guess not :/

But as a workaround you can download the file manually and place it inside the downloads folder of vcpkg and name it pixman-pixman-37216a32839f59e8dcaa4c3951b3fcfc3f07852c.tar.gz

Adding the extra -pixman- to the filename and placing it in vcpkg\downloads\ did the trick!


merge from arlac77/template-github,arlac77/template-kronos-module

.npmignore

chore: add  ()
chore: remove  ()

Codecov Report

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

@@            Coverage Diff            @@
##            master      #738   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           30        30
  Branches         7         7
=========================================
  Hits            30        30

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 045fa1a...820b9f4. Read the comment docs.


Azure go-autorest causing ambiguous import

Got an ambiguous import error when run go get ./... under a go module enabled project:
build github.com/.../cmd/manager: cannot load github.com/Azure/go-autorest/autorest: ambiguous import: found github.com/Azure/go-autorest/autorest in multiple modules:
	github.com/Azure/go-autorest v11.2.8+incompatible<EMAIL_ADDRESS>	github.com/Azure/go-autorest/autorest v0.3.0<EMAIL_ADDRESS>
go-autorest is introduced by client-go.
here is the line in my go.mod:
github.com/Azure/go-autorest/autorest v0.3.0 // indirect

but in the go.sum, there are many lines:
github.com/Azure/go-autorest v11.2.8+incompatible h1:Q2feRPMlcfVcqz3pF87PJzkm5lZrL+x6BDtzhODzNJM=
github.com/Azure/go-autorest v11.2.8+incompatible/go.mod h1:r+4oMnoxhatjLLJ6zxSWATqVooLgysK6ZNox3g/xq24=
github.com/Azure/go-autorest/autorest v0.3.0 h1:yOmXNB2qa2Kx40wMZB19YyafzjCHacXPk8u0neqa+M0=
github.com/Azure/go-autorest/autorest v0.3.0/go.mod h1:AKyIcETwSUFxIcs/Wnq/C+kwCtlEYGUVd7FPNb2slmg=
github.com/Azure/go-autorest/autorest/adal v0.1.0 h1:RSw/7EAullliqwkZvgIGDYZWQm1PGKXI8c4aY/87yuU=
github.com/Azure/go-autorest/autorest/adal v0.1.0/go.mod h1:MeS4XhScH55IST095THyTxElntu7WqB7pNbZo8Q5G3E=
github.com/Azure/go-autorest/autorest/date v0.1.0/go.mod h1:plvfp3oPSKwf2DNjlBjWF/7vwR+cUD/ELuzDCXwHUVA=
github.com/Azure/go-autorest/autorest/mocks v0.1.0/go.mod h1:OTyCOPRA2IgIlWxVYxBee2F5Gr4kF2zd2J5cFRaIDN0=
github.com/Azure/go-autorest/logger v0.1.0/go.mod h1:oExouG+K6PryycPJfVSxi/koC6LSNgds39diKLz7Vrc=
github.com/Azure/go-autorest/tracing v0.1.0/go.mod h1:ROEEAFwXycQw7Sn3DXNtEedEvdeRAgDr0izn4z5Ij88=

What's triggering the dependency on v11.2.8+incompatible?  That's the source of the problem.

You might need to use a replace directive in your go.mod file.  We recently added some documentation about this here https://github.com/azure/go-autorest#using-with-go-modules.  Can you try it?

A replace directive should never be necessary. A more appropriate resolution would be to modify the nested modules to require a version of the parent module that removes the ambiguous package.
(See, for example, the requirement on cloud.google.com/go in https://code-review.googlesource.com/c/gocloud/+/43891.)

@jhendrixMSFT would it make sense to switch back to using one tag for the entire repository (as previously) - rather than doing so for individual packages, since they've all got to be used together anyway?

Not all of them need to be used together, e.g. some modules are only consumed via code generation (e.g. validation), so having them split up allows us to make breaking changes without impacting unrelated modules (e.g. auth).
It's still not clear to me where the dependency on v11.2.8+incompatible came from and if it can be removed; this version predates the release of modules in this repo.  @27149chen can you provide complete repro steps and a link to your source code?

@bcmills you're right, using replace was bad advice and I admit it stems from still not fully understanding how modules interact with pre-modules dependencies.
I'm looking at the reference you provided and read the associated wiki section.  To make sure I correctly understand, I'd want to update module ./autorest/validation to have a dependency on its parent module ./autorest, and apply this same logic to all sub-modules.  Going forward does that mean that every time a parent module has a minor/patch update all of its sub-modules must be updated to require the new version?  Can this "fake" requirement ever be removed (perhaps in v2+)?

Going forward does that mean that every time a parent module has a minor/patch update all of its sub-modules must be updated to require the new version?

Nope. They only need to refer to some version of the parent module that doesn't contain conflicting package definitions. (They only need to upgrade beyond that if they actually use packages from the parent module, which does not seem to be the case here.)

Can this "fake" requirement ever be removed (perhaps in v2+)?

If we address https://github.com/golang/go/issues/27899, then you can remove the fake requirement when you drop support for versions of the go command that do not include the fix.

Awesome thanks for the feedback, will work on this today.

For module ./tracing/opencensus I believe the workaround is not required since it wasn't "carved out from its parent module (was added later), correct?

Another question, this repo  has no go.mod in its root as it seemed unnecessary given that it would contain no content.  Is this ok or do I need to add a root go.mod associated with a root version (e.g. v13.1.0) so I can carve out everything underneath it?

For module ./tracing/opencensus I believe the workaround is not required since it wasn't "carved out" from its parent module (was added later), correct?

That sounds right. You only need the reverse edge to remove ambiguous packages, so if there are no such packages to remove then you're good without it.

this repo has no go.mod in its root as it seemed unnecessary given that it would contain no content.

If the root is already above v1, then you really do not want to add a go.mod file there. When you add a go.mod file you have to comply with semantic import versioning (by adding a major-version suffix), and that makes it an entirely different module altogether (and thus of no use for removing the ambiguous import).

That's right, I remember having this epiphany when I originally added modules to this repo.
Unfortunately the ./autorest/adal module is imported by its parent, so I cannot implement the workaround for it (I had debated keeping adal within the autorest module but decided against it as I wanted it to version independently).  I'm unsure how to proceed in this case, do you have any advice?

The go.mod file is allowed to be incomplete, so you can have either (or both) of autorest/adal and autorest depend on each other, or you can provide only the dependency from autorest/adal on the compatible autorest, in which case users of autorest may end up adding an indirect requirement on autorest/adal in their own module.

Hmm maybe I'm misunderstanding something here.  I'm using https://code-review.googlesource.com/c/gocloud/+/43891 as a template and added a blank import on github.com/Azure/go-autorest/autorest from new file ./autorest/adal/workaround.go however when I go build I receive the following error.
can't load package: import cycle not allowed
package github.com/Azure/go-autorest/autorest/adal
        imports github.com/Azure/go-autorest/autorest
        imports github.com/Azure/go-autorest/autorest/adal

Did you remember to valve off the new file with a +build constraint? You want it to be visible to go mod tidy (which mostly ignores constraints) but not to go build.

Ah no I totally missed that, it's working now.  Thanks a bunch for your help :).

This should now be resolved with the latest module versions.  @27149chen please let me know if you continue to have problems.

To anyone who is struggling with this issue, I managed to avoid this problem by removing the following two lines in go.mod.
github.com/Azure/go-autorest/autorest v0.9.2 // indirect
github.com/Azure/go-autorest/autorest/to v0.3.0 // indirect

go mod tidy

Something is wrong with v0.3.0, I suppose.
Reference: https://github.com/kubernetes/client-go/issues/628

Has this problem been solved, I also encountered this problem and did not find a solution


Specify march for Darwin x86

The oldest Macs on which Swift 5 can be deployed have Sandy Bridge processors

@xymus since you've touched this code in the past, could you please take a look?

CC: @stephentyrone - IIRC @stephentyrone had some concerns around bumping the minimum supported CPU level in the past.


Manually upgrade deps

Fixes #

Proposed Changes

Manually upgrade deps
Can't wait till tomorrow

Release Note

Docs

/test unit-tests_eventing-kafka-broker_main
/test channel-integration-tests-sasl-ssl_eventing-kafka-broker_main


Can it support NFT-AR?

Hey there, I just want to exploit that new feild of Web AR, somebody help me plz, and THX

@Simon1059770342 Check out Blair McIntyre at Mozilla for WebXR (AR/VR):
https://github.com/mozilla/webxr-polyfill
Examples:
https://examples.webxrexperiments.com/


Hook fails to run

The hook is failing to run for me in sails v0.10.5. Is there any additional setup I'm missing? I ran
npm install -s sails-hook-newrelic
vim config/newrelic  # added valid config

I added a console.log to index.js of this lib and it doesn't appear to be called on lift.

You need 0.11 version of sails
On Thursday, March 12, 2015, Jason Sims<EMAIL_ADDRESS>wrote:

The hook is failing to run for me in sails v0.10.5. Is there any
additional setup I'm missing? I ran
npm install -s sails-hook-newrelic
vim config/newrelic  # added valid config
I added a console.log to index.js of this lib and it doesn't appear to be
called on lift.
—
Reply to this email directly or view it on GitHub
https://github.com/Kikobeats/sails-hook-newrelic/issues/1.

Ok cool, thanks for the quick response!


slow search pane

Maybe I have too many fields marked as searchable in my .cxml and lots of content, but search box could probably be made faster.
probably search directly as you type instead of searching in the background - maybe could use background workers in newer browsers or see if search is taking more than some msec and cancel it, with a timeout set at every char pressed (and resetting countdown at next one if not fired yet) so that if you type and pause your typing speed a bit it would then search (but not search if you type too fast or try searching and abort if some max time has elapsed [this is separate from the timeout to search if user paused or slowed down typing])
or maybe else the search algorithm or structures could be optimized

Agreed, there's probably a lot of room for improvement here. The search box is using a pretty simple implementation, because it happened to be fast enough on the collections I was testing and I was much more focused on the perf of zoom/pan/animate scenarios.


Remove a number of old scrolling widgets:

ScrollableList
ScrollableLazyList
LazyBlock
MaterialList

Clients should use ListView instead.

@Hixie

/cc @tvolkert

All hail the code deletion!

Oh, uh, LGTM also.


Add introduction to Elixir video in the Newsroom page

Please include this Elixir introduction video in the Newsroom section of the https://elixir.redcross.lk/ website:
https://user-images.githubusercontent.com/44461004/219546506-2b9fb3cb-0716-46c8-be95-c4cc2cea6a5f.MP4
Position: above the news section. It should be one of the first things they see before they need to scroll down to the news articles.
Sharing: We want anyone to be able to share a link to the video in social media text, on social media posts/stories etc.

I would like to work on this

I would like to work on this

Go Ahead @Tharindu37

@anjula-sack what do you think about this one?

@anjula-sack what do you think about this one?

Can you send the draft PR? @Tharindu37 Then we can decide.

@anjula-sack  Ok


feat/panel-container

add panel-container, wizard and tabs elements

Task linked: CU-33z1kkm Make updateHeight method of collapsible element public

Task linked: CU-33z1kgc Create tab custom element for illuminate frontend

Task linked: CU-33z1kdt Move wizard custom element to ui library

Task linked: CU-34ajv40 Create demo pages for new ui components


Unable to rename Accessories

Issue

I'm unable to rename accessories exposed from homebridge-hue. Somehow these jump back to the ones defined in Deconz after a few minutes. Has this always been like that?
This only happens with devices exposed through homebridge-hue, I had no issues renaming other devices exposed through homebridge (e.g. MQTT devices).
Running v0.12.21
https://user-images.githubusercontent.com/5174440/106485916-6156ce80-64b1-11eb-8959-2776d43e6e8f.mov
Log Messages

Debug Files

homebridge-hue.json.gz

homebridge.log.gz

Ah, seems to be related to https://github.com/ebaauw/homebridge-hue/issues/910 ... just curious: Why is this the default behaviour and not the other way around?

I don’t want to expose Configured Name for new installations.


cant move two crashed trains

Describe the Bug
two trains crahed together and now i cant move them
Reproduction Steps
1.create 2 trains
2. have one assibemabled and the other inside that train
3. try to move the trains
...
Expected Result
being able to move the train
Screenshots and Videos

Crash Report or Log
No response
Operating System
Windows 11
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods
No response
Additional Context
No response

You can't drive derailed trains, and crash derails them. You need to relocate back in track in order to drive

Yes. I'm aware. I can't relocate it. It's just stuck.

Please send a log

Closing due to no response - most likely a issue with some addon


Remote Data Encoder

Provide a framework for serving and consuming data encoding via HTTP across all our SDKs.

Frees users to write their encryption code in whichever of our SDK-supported languages they prefer.
Allows users to restrict access to their encryption keys or KMS API usage to only the remote decoder process rather than developer workstations.
Allows tctl to encode payloads for tctl workflow start. Note: This will not work for any encoders which require workflow context. Fixing this is outside the scope of the MVP

Remote_Data_Encoder_protocol.pdf
GoSDK reference implementation: https://github.com/temporalio/sdk-go/pull/717

Please note protocol update (Payloads vs Payload): https://github.com/temporalio/sdk-go/pull/733

The linked PDF is correct and matches https://github.com/temporalio/sdk-go/pull/733

Go and TS SDKs both have nice APIs for this now, would be great to get this in Java too.

Not really our preferred or advised method to do that. It's more of a workaround if the standard approach to data converters is not applicable. Is there a specific client and usecase asking?

The majority of users should stick to standard data converters for a variety of reasons as long as they could. And security is one of them.

Hey! Temporal Cloud customer here, we'd love for the Java SDK to have parity with the other SDKs in this specific area.


ui: display search page size changer

INSPIR-3352

@harunurhan maybe i missed that but in the embed search, at least in the screenshots, there is not a test with the page size selection

@drjova because this changes the default size for those from 10 to 25

could we use another record that has more than 25? To make sure we render properly the dropdown in embeded search as well

we could, but screenshots are not intended to capture all cases (currently at least).

Sure, IMHO it's a core feature and we should test both cases i.e. single page and multiple pages.

pagination is core feature for search, and right now both cases are tested.. but if you want to test both cases for each collection, that could get unproductive very easily.

I don't want to test each collection, I just want to make sure the select box for search results size is rendered properly with the correct style.

Well there are some search pages that have it, no?

I'm talking about the embed search, please correct me if I'm wrong but I didn't see any screenshot for this

embedded literature search doesn't have its own layout.


Miniaturizarion fluid drop added recipe is not accesible anymore

http://puu.sh/vJo8p/1044a5c192.jpg

This seems to be specific to our server, this line was missing in the server configs.
Added the recipe back in again.


Increase QDrant timeout

To fix bug: Error in deleting file from Qdrant: timed out
Doubel check qdrant database size, ensure we have a big enough EC2 node to handle it.

EC2 is big enough, we're only using 2 of 16GB of ram, but I think 15GB of disk. Should carefully alter our Qdrant config to make use of more ram.


Prevent SYSTEM_ROLES to be removed when login

Created from https://github.com/communitiesuk/funding-service-design-authenticator/pull/326
Change description
Prevent SYSTEM_ROLES to be removed when login.
At the moment when we login we refresh the account role in the database
with the roles we get from AD.
This prevents that. We check the system roles the user had, and adds
them back to the array of roles before we trigger the update.

[ ] Unit tests and other appropriate tests added or updated
[ ] README and other documentation has been updated / added (if needed)
[ ] Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

Closing this. If needed again, re-open.


Add option to start budget week on non-Monday

Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do?

Per correspondence, BRAF is experimenting with weekly budgets and planning on having their budget start day be on something that isn't monday, which isn't as helpful since there's a day shift. To that effect, we should make it so that this is config-able.

What feature or behavior is this required for?

proper budget bar functioning if you are not dcaf or otherwise do not have monday/sunday budgets.

How could we solve this issue? (Not knowing is okay!)

my hope is that this is a pretty minor thing, and that we can continue to keep mondays as the defaults. To that end, I'd say let's try adding a config called like budget_start_day that should be a day name (e.g. monday, tuesday, wednesday). The budget bar functions should use that day if exists, otherwise default to monday.
I don't know how tangled a web this is, I don't think very, but you know.

Anything else?

not a terrible thing if this doesn't get done for a bit but we probably owe BRAF a solid.
This is probably a fun fishing expedition thru the budget bar code.

I'll take a look at this.

@colinxfleming I'm looking around and not finding where the budgets are calculated based on date, just the individual patient pledges. Would you be able to point me to a file in the codebase or a page in the app where I could see?

@valeriecodes sorry, my bad for not adding more detail for this! Should have added more of a grand tour.
The method that generates the report is here: https://github.com/DCAFEngineering/dcaf_case_management/blob/master/app/models/patient.rb#L143
The place this actually shows up in a view is the patient dashboard here: https://github.com/DCAFEngineering/dcaf_case_management/blob/master/app/views/dashboards/_budget_bar.html.erb and the helper that calls that actual method is here: https://github.com/DCAFEngineering/dcaf_case_management/blob/master/app/controllers/concerns/budget_bar_calculable.rb
My guess is that the move here is to add a start day optional argument to Patient#pledged_status_summary and alter BudgetBarCalculable#budget_bar_calculations so that it passes in a config. If that doesn't wind up being a viable option or not super straightforward let me know!

@valeriecodes are you still working on this one? Mind if I grab it?

Resolved by #1709


Parser doesn't allow function calls as statements...

Expressions can make function calls, like...
x = myfunction();

...but bare expressions are not permitted as statements, so this won't parse by itself...
myfunction();

We need to add a new statement type to the parser.

...we probably need "lvalue++;" and friends to optionally function as a statement, too. And then change the third for-loop section to accept a statement instead of an expression.


test: convert InsertPosition from actions to MarketOrder

We have a test Action (actions take in an app and context and mutate them) called InsertPosition in the perp module. Because the action allows us to insert positions directly into state without making a trade on the market, it enables the creation of highly unrealistic resulting states/conditions.
We should replace instances of InsertPosition with real market orders to mirror what will happen with calls from the module's MsgServer.

^ Another reason using InsertPosition is bad is because it makes the AMM go out of sync with the actual trader state. The current logic for InsertPosition().Do is:
func (i insertPosition) Do(app *app.NibiruApp, ctx sdk.Context) (sdk.Context, error, bool) {
	traderAddr := sdk.MustAccAddressFromBech32(i.position.TraderAddress)
	app.PerpKeeperV2.SavePosition(ctx, i.position.Pair, 1, traderAddr, i.position)
	return ctx, nil, true
}

where
func (k Keeper) SavePosition(ctx sdk.Context, pair asset.Pair, version uint64, account sdk.AccAddress, position types.Position) {
	k.Positions.Insert(ctx, collections.Join(collections.Join(position.Pair, version), account), position)
}

This is a raw insert to state without updating the AMM, making AMM.Bias, AMM.TotalLong, and AMM.TotalShort incorrect in any test that uses InsertPosition.

@matthiasmatt pointed this out in https://github.com/NibiruChain/nibiru/issues/1526, I think


Clipboard not working in some OSX environments

Context
I'm building an app on atom-shell and a portion of atom.  Copy/paste does not work for me, but it does for my other team members running identical code, identical versions of atom-shell and on OSX.
Example
c = require("clipboard")
c.writeText("text")
c.readText()
# undefined

This example sets the clipboard text and reads it back correctly if executed in atom on my machine, but shows the behaviour above in our atom-shell based app.  It neither sets the clipboard, nor can it read it.
Environment
Atom-shell 0.19.4
OSX 10.9.5

Probably unrelated, but also installed systemwide:
Atom 0.152.0
NPM 2.1.6
Node 0.10.33

I see the same behavior as @zipcode - copy+paste works from a .app invocation but not an Atom invocation. Seeing this on 0.20.3 atom-shell on osx 10.9.5

Were you using any custom shell like screen or tmux in the terminal?

@zcbenz Yep - using tmux. Issue goes away when not using tmux. Good call

Same - using tmux.

OS X's clipboard API requires the process to run in a UI session, which might not be true if you are using tmux, you can also try if pbcopy/pbpaste work in your terminal.
See also the link bellow for a better explanation, it also provides some work-arounds:
https://github.com/ChrisJohnsen/tmux-MacOSX-pasteboard#mac-os-x-pasteboard-access-under-tmux-and-screen
I'm closing this because this is not an issue of atom-shell.


alphaTest should be respected by pc.Picker and shadow mapping

Currently alphaTest is not respected by pc.Picker, so it is possible to pick on occluded pixels.
As well as for shadow mapping, it has hardcoded static alphaTest value instead of using the one from material if it is set.

Alpha test cutoff threshold is already respected in shadow mapping. Not in picker though.

The problem with the picker is when you drag-n-drop an alpha-tested material onto object. If your mouse happens to be on a transparent pixel, then the next frame picker will select a different object behind the first. Then, again, first object becomes opaque again, and picker will select it, and so it flickers back and forth...

I think that's an Editor problem, not an engine problem. Although I'm not entirely sure how I'd fix it in the Editor codebase.

Added picker alpha-test support in https://github.com/playcanvas/engine/pull/654
In Editor, if you drag a material, you must set
picker._ignoreOpacityFor = last picked meshInstance
to prevent flickering.


TGRSIDetectorHit Manual Streamers

I'm going to play around today, while waiting for some data to sort, with writing some streamers for the TGRSIDetector and TGRSIDetectorHit Classes. It is possible that I will be able to get energy, detector, position to fill as the data is read from the file. This would be excellent to have in the code.

So you want to fill the energy in GriffinHit from DetectorHIt?  This is
pretty simple. Add
void TGriffinHit::Streamer(TBuffer &R__b)
{
// Stream an object of class TGriffinHit.
if (R__b.IsReading()) {
R__b.ReadClassBuffer(TGriffinHit::Class(),this);
energy   = this->GetEnergy();
detector = this->GetDetector();
position = this->GetPosition();
} else {
R__b.WriteClassBuffer(TGriffinHit::Class(),this);
}
}
to GriffinHit.cxx and change
#pragma C++ class TGriffinHit+
to
#pragma C++ class TGriffinHit-
in the TGriffinLinkDef.h file.
That should do it.
On Tue, Jun 23, 2015 at 10:30 AM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

I'm going to play around today, while waiting for some data to sort, with
writing some streamers for the TGRSIDetector and TGRSIDetectorHit Classes.
It is possible that I will be able to get energy, detector, position to
fill as the data is read from the file. This would be excellent to have in
the code.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/371.

Yeah, that was the plan! Thanks for ruining my learning Peter…. :p

On Jun 23, 2015, at 10:50 AM, pcbend<EMAIL_ADDRESS>wrote:
So you want to fill the energy in GriffinHit from DetectorHIt? This is
pretty simple. Add
void TGriffinHit::Streamer(TBuffer &R__b)
{
// Stream an object of class TGriffinHit.
if (R__b.IsReading()) {
R__b.ReadClassBuffer(TGriffinHit::Class(),this);
energy = this->GetEnergy();
detector = this->GetDetector();
position = this->GetPosition();
} else {
R__b.WriteClassBuffer(TGriffinHit::Class(),this);
}
}
to GriffinHit.cxx and change
#pragma C++ class TGriffinHit+
to
#pragma C++ class TGriffinHit-
in the TGriffinLinkDef.h file.
That should do it.
On Tue, Jun 23, 2015 at 10:30 AM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

I'm going to play around today, while waiting for some data to sort, with
writing some streamers for the TGRSIDetector and TGRSIDetectorHit Classes.
It is possible that I will be able to get energy, detector, position to
fill as the data is read from the file. This would be excellent to have in
the code.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/371.

—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/371#issuecomment-114532577.

I'll be more careful next time.  :)
On Tue, Jun 23, 2015 at 10:57 AM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

Yeah, that was the plan! Thanks for ruining my learning Peter…. :p

On Jun 23, 2015, at 10:50 AM, pcbend<EMAIL_ADDRESS>wrote:
So you want to fill the energy in GriffinHit from DetectorHIt? This is
pretty simple. Add
void TGriffinHit::Streamer(TBuffer &R__b)
{
// Stream an object of class TGriffinHit.
if (R__b.IsReading()) {
R__b.ReadClassBuffer(TGriffinHit::Class(),this);
energy = this->GetEnergy();
detector = this->GetDetector();
position = this->GetPosition();
} else {
R__b.WriteClassBuffer(TGriffinHit::Class(),this);
}
}
to GriffinHit.cxx and change
#pragma C++ class TGriffinHit+
to
#pragma C++ class TGriffinHit-
in the TGriffinLinkDef.h file.
That should do it.
On Tue, Jun 23, 2015 at 10:30 AM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

I'm going to play around today, while waiting for some data to sort,
with
writing some streamers for the TGRSIDetector and TGRSIDetectorHit
Classes.
It is possible that I will be able to get energy, detector, position to
fill as the data is read from the file. This would be excellent to
have in
the code.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/371.

—
Reply to this email directly or view it on GitHub <
https://github.com/GRIFFINCollaboration/GRSISort/issues/371#issuecomment-114532577
.

—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/371#issuecomment-114534211
.

So this is interesting. I can get the Streamers to be called when using a function such as Clone(). However, the streamers are not being called when I am accessing the tree. Are we doing something to the TKey of the branches of our tree that would keep this from happening?

I found this in my searches
'If you have MyClass *obj and obj is given as a top Tree branch in split mode, then MyClass::Streamer is not called. '
Rene Brun

ok I think I have figured it out. @pcbend, we cannot do this without breaking the Draw command in the analysis tree. Root only calls the streamer when loading a branch if the split level is set to 0. If we set the split level to 0, I do not believe that we can use the Draw command as we do not have direct access to each member as it's own branch. I'll just make these variables fill on their first call. I don't think this is a big deal, it would have just been nice to get it all done in the streamer.

Yeah, I see the issue.  Apart from making a custom class to hold an Int_t
am not sure if there is a straight forward answer.   At the least, we
should make the GetFunctions() re-calculate there values every time for now
(if they calculate anything at all).
On Thu, Jul 9, 2015 at 1:57 PM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

ok I think I have figured it out. @pcbend https://github.com/pcbend, we
cannot do this without breaking the Draw command in the analysis tree. Root
only calls the streamer when loading a branch if the split level is set to
0. If we set the split level to 0, I do not believe that we can use the
Draw command as we do not have direct access to each member as it's own
branch. I'll just make these variables fill on their first call. I don't
think this is a big deal, it would have just been nice to get it all done
in the streamer.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/371#issuecomment-120087168
.

Yeah, I'm just going to put some is_modified flags in and call a Modified() function when it needs to be called. Not parsing mnemonic for the detector on every get detector call will be nice.

On Jul 9, 2015, at 5:22 PM, pcbend<EMAIL_ADDRESS>wrote:
Yeah, I see the issue. Apart from making a custom class to hold an Int_t
am not sure if there is a straight forward answer. At the least, we
should make the GetFunctions() re-calculate there values every time for now
(if they calculate anything at all).
On Thu, Jul 9, 2015 at 1:57 PM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

ok I think I have figured it out. @pcbend https://github.com/pcbend, we
cannot do this without breaking the Draw command in the analysis tree. Root
only calls the streamer when loading a branch if the split level is set to
0. If we set the split level to 0, I do not believe that we can use the
Draw command as we do not have direct access to each member as it's own
branch. I'll just make these variables fill on their first call. I don't
think this is a big deal, it would have just been nice to get it all done
in the streamer.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/371#issuecomment-120087168
.

—
Reply to this email directly or view it on GitHub.

I think I have solved this problem with my setters and const/non-const getters.


Links to external sources are not clickable.

I have this issue on Firefox, and Vivaldi, and it appears to be a layout issue.
It seems to be because of the Divs that hold the entries are overlapping with the actual posts as they span the whole width of the page, this is supported by the fact that if I change the padding or margin so that the posts no longer overlap on a vertical level the links are again accessible.
The simplest possible fix is to disable the css rule that adds a -150px top margin in the class "timeline-entry"

Oh I'm sorry this appears to be duplicate of #86 I only checked open issues.


TypeError: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received an instance of ArrayBuffer

Environment
solc-typed-ast version: 15.0.0.
node version: 16.16.9.
typescript version: 4.9.3.
Scenario
import { compileSol } from "solc-typed-ast";

export async function compile (){
  const result = await compileSol("./src/sample.sol", "auto");
  console.log(result);
  return result;
}

Expected behavior

Current behavior
The following error is reported:
TypeError: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received an instance of ArrayBuffer
        at new NodeError (node:internal/errors:372:5)
        at Hash.update (node:internal/crypto/hash:105:11)
        at getCompilerForVersion (/home/maltabba/repos/experiments/web3-abi-generator/node_modules/solc-typed-ast/src/compile/kinds/compiler.ts:141:14)
        at runMicrotasks (<anonymous>)
        at processTicksAndRejections (node:internal/process/task_queues:96:5)
        at compile (/home/maltabba/repos/experiments/web3-abi-generator/node_modules/solc-typed-ast/src/compile/utils.ts:154:22)
        at compileSol (/home/maltabba/repos/experiments/web3-abi-generator/node_modules/solc-typed-ast/src/compile/utils.ts:326:22) {
      code: 'ERR_INVALID_ARG_TYPE'
    }

Description [optional]
https://github.com/nodejs/node/blob/49a77a5a996a49e8cb728eed42e55a7c1a9eef6e/lib/internal/crypto/hash.js#L105
Suggested solution [optional]
Use "string" instead of "arraybuffer" at this line:
https://github.com/ConsenSys/solc-typed-ast/blob/63828e53551f1a61b4a71103aaa4eeab4f6d9173/src/compile/kinds/compiler.ts#L136

@Muhammad-Altabba Hello there. I will investigate the issue and currently opened pull request with fix. Will respond with results when there would be any information to share. Thank you.

@Muhammad-Altabba Hello again. I checked the possible issue, but not yet able to reproduce it on my end.
First of all, I didn't found NodeJS v16.16.9 release. So I tried to downgrade to v16.14.0 instead. In this version, there is no conflict in typesystem. TypeScript compiler invocation (via tsc) still manages to compile the codebase.
I also checked through documentation for a particular release. It seems that the crypto API supports Buffer (that is returned by axios request). Also I do not see any related failures in our CI anywhere, except your opened PR (that is failing).
The kind of problem, that you expressing, regularly related to a difference in environment.  Is it related to NodeJS of Browser?

@blitz-1306 ,
Sorry for the typo regarding the node version. I used 16.16.0. But I am still facing the same issue after I downgraded to version 16.14.0.
And I fixed the MR. And I hope they will pass after you approve the workflow to run.
Regarding the environment, I am using node and not the browser.

@Muhammad-Altabba Hello there. I pulled the mentioned repository, removed .skip suffix from tests. Comple test succeeded for me without any issues. I'm currently running NodeJS v16.14.0. Maybe it is something in your environment.

It would be great if you will be able to make CI work to show the issue there. If issue would manifest in clean environment it would be easier to catch and fix.

Hi @blitz-1306 ,
I am still facing the issue locally. And the CI used to pass but is now failing at https://github.com/conx3/web3-plugin-craftsman
I suggest that it could be something wrong with the server deployment. If I could guess there are multiple servers that are supposed to serve the exact same version but they are not.Or what do you think?

@Muhammad-Altabba
Hello there.
First of all, sorry for late response - I took some time for vacation and was dedicated to issues IRL.
I looked through mentioned jobs and was finally able to reproduce the issue. I created a small PR #216 with possible fix. Could I ask you to help checking that things are smooth with these fixes on your end (at least locally)?
Following steps are needed:

In solc-typed-ast repo clone, switch to https://github.com/Consensys/solc-typed-ast/tree/fix-downloading-for-newer-nodejs branch. Then perform local installation with commands:

npm install
npm link

Remove any predownloaded compilers, that may be present in compiler cache. You can discover cache location by running command:

sol-ast-compile --locate-compiler-cache

In repos with mentioned issues tweak package.json to use local solc-typed-ast like following:

"dependencies": {
  "solc-typed-ast": "file:path/to/solc-typed-ast/"
  // ...
}

Then try to run package build steps and testing them.
I also seen that you try to run package on Windows-based hosts. Beware that we had issues with it and do not yet fully support this platform. I had a few attempts in #177, but this for now is out of our priority list.

Hello @blitz-1306,
Sorry also for the late reply :)
I followed the steps and yes it worked.
And I am not currently running on Windows-based hosts. But I thought and now I hope that it will be supported in the future :)

Next release should have a in it, as #216 is merged.