Makes Almond chat friendly to dark Gnome theme

It changes the original approach from using an image to draw the borders to a pure CSS solution.
It solves the color change problem and can also allow a simpler responsive solution or a user themeable interface with some buttons and fields.
I can't make a identical balloon with GTK+CSS (in HTML+CSS it is possible), so I decide to propose some other changes:

The user balloon border was changed from brown to @theme_selected_bg_color. That means the user balloon follows the highlight color from selected theme. I believe it is important to follow the user aesthetic preferences and be more desktop integrated.
The background is a mix of @theme_bg_color and an almond inspired gradient. This will help to distinguish the balloons from the background.
And the balloons looks nicer with a shadow.

closes #73

This looks quite nice! Did you test non-text messages? For example, try the commands "get a dog picture" (picture message), "get xkcd" (rdl message with image), or "show me the news" (rdl message without image).

Ups! I totally forgot that!

By the way, thanks a lot for testing the very latest version of Almond! We're preparing a new release soon, and we're grateful for any feedback ahead of it.
Did you have any comment on it, or was there anything that did not work?
(If you want and have time, you can also create an account at dev.almond.stanford.edu. From there, if you click on "enable recording", you can upvote/downvote any reply from Almond. We use that info to improve the user experience, and it's very valuable to us, especially this week as we prepare the release)

I don't know how to test other kinds of message, but i think if it was working with the original balloon, it will work now. The changes was pretty simple. Please @gcampax, test it in your set up, so we can be sure.

Hey, guys can you please tell me taht how can I implement this into Almond.
That would be very niece of you

Hey @gcampax, let me know if something is missed.

Sorry, I had not a chance to test this until now. I checked the implementation, it looks good. I think the gradient is a bit too strong, but without the gradient, and just with the theme BG color, it works well both for light and dark theme. I'm going to merge this PR and then do that fix.
Thanks again for working on this!


Cleaner namespace

Use __all__.
Add version_info to __init__.py.

Is version_info a "standard" like version is?

Standard in the sense that many packages (including wgpu-py and pygfx :)) have it. Since it's a tuple you can actually do version comparisons on it, even when one of the version elements is larger than 9.


Support for MPT

Will you release the code for MPT? I saw in your paper the result of MPT but I didn't find the code for MPT in https://github.com/mit-han-lab/streaming-llm/tree/main/streaming_llm/pos_shift.

We do support MPT :). https://github.com/mit-han-lab/streaming-llm/blob/main/streaming_llm/enable_streaming_llm.py#L12


Add schema ERD

Entity Relationship Diagram was created by pgAdmin.
I believe this is interesting to have. Apps like pgAdmin/dbeaver make it quite easy to create those and we can update them with every major release.

Pulled into https://github.com/input-output-hk/cardano-db-sync/pull/1203 .


Add jwt secret entries to config.yml.example

To make setting up a new development environment smoother, as the jwt secret fallbacks in lib/json_web_token.rb don't match the jwt secret values in iNaturalistAPI/config_example.js.
Without this change, authenticated API requests fail out of the box on pages such as the Account Settings page.
These jwt secret values I've added here match the jwt secret values already in iNaturalistAPI/config_example.js
Instead of this change, another option could be to strip the jwt secret values from iNaturalistAPI/config_example.js, and rely on the fallbacks matching in both codebases for development environments.

@kueda Here's another dev setup bump in the road I found when setting up my dev environment.
Is just opening these PRs as I have been an acceptable way to contribute these small fixes?

@seanclifford I see you're using the containerized workflow, how is that working out for you with a new setup?

Many thanks, @seanclifford, I was just actually setting up a different machine with @todtb's Docker instructions and hit this very problem yesterday. Brian, setting up the containers was pretty smooth and a lot easier than doing things manually. IMO, I find make services-api a bit weird b/c you need to do some non-Docker setup to get it to work (check out the API repo, set up the config files). Unless the API can be fully containerized for developers who exclusively want to work on the Rails app, I sort of feel like it should be treated as a separate, non-containerized application that you need to run in parallel, but that depends on the same services (Postgres, elasticsearch, etc.) that the Rails app does. But then again, I know practically zilch about Docker, so maybe that's standard practice.

Ah, and yes, PR's are a good way of making changes like this. Usually we'd like to see an issue first so we can consider whether we actually want the work done before you start putting time into it, but for small changes like this that don't cost you much time, I think a PR is fine.

@todtb I found it very easy to setup. Thank you for putting it together. Just a couple of tweaks to config files that I think would have been required in some form if I set all the services up manually.
The only thing that I found that might be an issue in future is the tasks in es.rake that check for elasticsearch binaries before running. Luckily, the one I needed to create all the indexes when I was setting up  - es:rebuild - didn't run this check.
On developing the API - I'm also not super familiar with docker, having only used it to host service dependencies before, but I imagine it would just require a make services-api after making changes to the API code?
@kueda Thanks for the merge and the feedback on contributing. I'll make use of issues and potentially the forum before making any non trivial changes.

I find make services-api a bit weird b/c you need to do some non-Docker setup to get it to work (check out the API repo, set up the config files).

I think that could be a nice next step @kueda, and could certainly be done, but with a few pre-reqs. I think it would require hosting the docker image on either a public or iNat hosted image repository. Perhaps there would need to be builds associated with either release tags or branches? I know there's some cool things that can be done with the build: tag in docker-compose. So there's a little overhead, all doable, but some infrastructure required. The short term benefit of checking out the repo individually would be that the volume maps directly to the local repo, so you can just check out whatever iNatAPI branch you need.

I sort of feel like it should be treated as a separate, non-containerized application that you need to run in parallel, but that depends on the same services (Postgres, elasticsearch, etc.) that the Rails app does.

make services alone should allow that if the ENV variables are set to point to the docker containers. Also if you wanted to manually start just iNat API, running docker compose up can be used.
@seanclifford and @kueda, thanks for the feedback. I probably missed a few things in the configs setup since I was migrating to containerization, rather than starting from scratch, so the fresh eyes are super helpful. I can look into some of the tweaks, or if you've got some good first hand experience from the setup I can provide any comment I can on PRs.


add python-transitions for focal

Please add the following dependency to the rosdep database.
Package name:
python-transitions
Package Upstream Source:
https://github.com/pytransitions/transitions
Purpose of using this:
Pull python transitions from Ubuntu Focal
Links to Distribution Packages

Ubuntu: https://packages.ubuntu.com/

https://packages.ubuntu.com/search?suite=default&section=all&arch=any&lang=cs&keywords=python-transitions&searchon=names

sorry, I only searched for "python-transitions", didn't notice the name has changed to "python3-transitions"


Preventing default up/down key behavior even when no suggestions are available.

Default behavior for pressing the up and down keys is being preventDefault'ed, even in cases where the autosuggest does not need to be using special handling for them. In input elements, this manifests as not being able to quickly jump to the beginning and end of an input using these keys, and in textarea and contenteditable elements it prevents jumping between lines.
Single-line Input Reproduction Steps: (Use the Basic example)

Focus the input field.
Type asdf.
Make sure there are no suggestions available.
Press up.

Observed Behavior: The cursor does not move.
Expected Behavior: The cursor moves to be in front of the first character in the input.
Multi-line Input Reproduction Steps: (Use this codepen)

Focus the input field.
Type asdf.
Press enter to create a new line.
Type asdf again.
Make sure there are no suggestions available.
Press up.

Observed Behavior: The cursor does not move.
Expected Behavior: The cursor moves to the line above.

Sounds reasonable. Would you like to contribute a PR?

Gladly! Let me know if any changes should be made.
Any idea when the next release will be?

@moroshko, do you know when you'd have time to merge @ShibumiGrant's changes? I also have the same issue and the proposed changes work.

Is the any progress in Preventing default up/down key behavior?


PXF Ignite plugin

Add a new PXF Ignite plugin to access (read & write) Ignite database via REST API.
Documentation is available at pxf/pxf-ignite/README.md.
Key features:

Supports partitioning (similar to one in PXF JDBC plugin);

Uses a dedicated Ignite client with REST API enabled. The location of the client is a customizable PXF user property;

Every SELECT or INSERT query splits tuples of data into groups, each of which is send from (to) Ignite in one response (query). The size of a group is a customizable PXF user property;

This REST API can be modified (with relatively small changes) and used to connect to any other database (or service) which supports SQL queries via REST, and responds using JSON objects.

GSON is used to parse JSON objects. No other extra dependencies (except for those already present in HAWQ PXF) are required.

@leskin-in - i am writing pxf developer docs.  i have a few questions if you are willing to share:

on what OS did you develop the plug-in?
what build tool/IDE did you use?
where did you find reference material for your work?  HAWQ PXF wiki?  HAWQ docs?  code? - which was the most relevant/helpful?
what plug-in (fragmenter, accessor, resolver) did you implement first?
do you have any design/specification documents for the ignite work that you would be willing to share?

thank you!

@lisakowen,

Ubuntu 16.04; CentOS 7.4
Gradle 2.10, Visual Studio Code 1.20.1, IntelliJ Idea 2017.3
Key sources (the most useful first): http://hawq.incubator.apache.org/docs/userguide/<IP_ADDRESS>-incubating/pxf/PXFExternalTableandAPIReference.html (this article is far more useful than anything else I found and used); PXF JDBC plug-in code; PXF API code
Accessor. The key details on how to implement other classes are already present in other plugins
I am going to expand the README to add information on INSERT queries and perfomance test results. Other documentation is mostly in the code itself. If adding some extra docs would be really helpful, please let me know.

@denalex @hornn @sansanichfb Is there anything we need to do/fix for this PR to be merged?

@kapustor To get this PR merged, we need this PR have two '+1' from the reviewer, and you need to create corresponding jira and include the jira number in the title of the PR. You can refer to other PRs. Thanks.

Hi guys,
Any news on this PR?

@kapustor -- we are ready to commit it if you do not plan any further changes. Need to put it through a regression run on our side, hope to be done with it next week. Thanks.


fix state reset

Coverage decreased (-0.2%) to 81.434% when pulling f7846313321e54aa13350799cc19be49e7c4f918 on resetfix into 41b9fae3ef2cc66228a57b099f11c1e91448bc9d on master.


docs(dynamic_avoidance): add logic to extract objects from drivable area

Description
Added a brief document for dynamic avoidance.

Tests performed

Not neccessary
Effects on system behavior

Nothing
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kosuke55 Could you approve the PR.


Add a console reporter

Closes #36.

Codecov Report

Merging #37 into master will decrease coverage by 1.69%.
The diff coverage is 93.54%.

@@            Coverage Diff            @@
##           master      #37     +/-   ##
=========================================
- Coverage   98.07%   96.38%   -1.7%
=========================================
  Files           1        2      +1
  Lines          52       83     +31
=========================================
+ Hits           51       80     +29
- Misses          1        3      +2

Flag
Coverage Δ

#1_5_otp18
96.38% <93.54%> (-1.7%)
:arrow_down:

#1_5_otp20
96.38% <93.54%> (-1.7%)
:arrow_down:

#1_8_otp20
96.38% <93.54%> (-1.7%)
:arrow_down:

#1_8_otp22
96.38% <93.54%> (-1.7%)
:arrow_down:

Impacted Files
Coverage Δ

lib/telemetry_metrics.ex
98.07% <ø> (ø)
:arrow_up:

lib/telemetry_metrics/console_reporter.ex
93.54% <93.54%> (ø)

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 25325a1...d6bce5f. Read the comment docs.

@josevalim Looks good to me! I've left one comment for discussion :)

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Bumbag + NextJS SSR not working with 1.0.0-rc0

Hi Jake, first of all thanks for building bumbag, I just found it out and it looks awesome!
I tried the latest release candidate and I can't get SSR to work properly
An example app can be seen here: https://with-bumbag-app.vercel.app/
I think code is availably publicly at https://with-bumbag-app.vercel.app/_src
Let me know iff you need something else, looks like an awesome framework btw

Hi!
Seems that the "Getting Started" docs was missing the <Head/> tag in the Next.js example.
Try copy and paste the following into your _document.js file, and try again:
import Document, { Head, Main, NextScript } from 'next/document'
import { extractCritical } from 'emotion-server'

export default class MyDocument extends Document {
  static async getInitialProps(ctx) {
    const initialProps = await Document.getInitialProps(ctx)
    const styles = extractCritical(initialProps.html)
    return {
      ...initialProps,
      styles: (
        <>
          {initialProps.styles}
          <style
            data-emotion-css={styles.ids.join(' ')}
            dangerouslySetInnerHTML={{ __html: styles.css }}
          />
        </>
      ),
    }
  }

  render() {
    return (
      <html>
        <Head />
        <body>
          <Main />
          <NextScript />
        </body>
      </html>
    );
  }
}

Still no SSR, with your version or importing head like this doesn't work either.
`import Document, { Main, NextScript } from 'next/document'
import { extractCritical } from 'emotion-server'
import Head from 'next/head'
export default class MyDocument extends Document {
static async getInitialProps(ctx) {
const initialProps = await Document.getInitialProps(ctx)
const styles = extractCritical(initialProps.html)
return {
...initialProps,
styles: (
<>
{initialProps.styles}
<styles
data-emotion-css={styles.ids.join(' ')}
dangerouslySetInnerHTML={{ __html: styles.css }}
/>
</>
),
}
}
render() {
return (

My page title

);
}
}`

This is the warning I get about different id's on clieent/serveer
dll_f9de5cbc314a1e41f91e.js:340 Warning: Prop `id` did not match. Server: "id-h55sls" Client: "id-qo6dtr"
    in div (created by ForwardRef(Box))
    in ForwardRef(Box)
    in Unknown (created by Disclosure.Content)
    in Disclosure.Content
    in div (created by ForwardRef(Box))
    in ForwardRef(Box)
    in Unknown (created by PageWithHeader)
    in PageWithHeader (at pages/index.tsx:8)
    in div (created by ForwardRef(Box))
    in ForwardRef(Box)
    in Unknown (created by Box)
    in Box (created by Provider)
    in PageProvider (created by Provider)
    in ToastProvider (created by Provider)
    in IdProvider (created by Provider)
    in Provider (created by Provider)
    in ThemeProvider (created by Provider)
    in Provider (at pages/index.tsx:7)
    in MyApp (at _app.tsx:48)
    in div (created by ForwardRef(Box))
    in ForwardRef(Box)
    in Unknown (created by Box)
    in Box (created by Provider)
    in PageProvider (created by Provider)
    in ToastProvider (created by Provider)
    in IdProvider (created by Provider)
    in Provider (created by Provider)
    in ThemeProvider (created by Provider)
    in Provider (at _app.tsx:47)
    in App
    in ErrorBoundary (created by ReactDevOverlay)
    in ReactDevOverlay (created by Container)
    in Container (created by AppContainer)
    in AppContainer
    in Root
printWarning @ dll_f9de5cbc314a1e41f91e.js:340
DevTools failed to load SourceMap: Could not load co


Integration Test - Release - 2022/01/26 09:07:56

Auto create for integration test. IssueCommandFlow

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

#please-close

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#unassign PRMergerTest1

#unassign NonAllowedUser


Issue with getting positions of atoms when more than 99999 atoms in the system

Description
When there are more than 99999 atoms in a system, the pdb output from packmol will have the second column containing Non-integer serial number. This leads to an issue with extracting positions from the pdb file later on using rdkit.Chem.rdmolfiles.MolFromPDBFile.
Reproduction
from openff.toolkit.topology import Molecule
from openff.toolkit.typing.engines.smirnoff import ForceField
from openff.units import unit
from openff.interchange.components.interchange import Interchange
from openff.interchange.components.mdconfig import MDConfig
from openff.interchange.components._packmol import pack_box

smiles_list = ['CCO']
molecules = [Molecule.from_smiles(smiles, allow_undefined_stereo=True) for smiles in smiles_list]
box_shape = [[1, 0, 0], [0, 1, 0], [0, 0, 1]]
topology = pack_box(
    molecules=molecules,
    number_of_copies=[11112],
    box_shape=box_shape,
    solute=None,
    center_solute=True,
    tolerance=2.0 * unit.angstrom,
    mass_density=0.7 * unit.grams / unit.milliliters,
    working_directory='./',
    retain_working_files=True)
topology.to_file('./system.pdb')
print('Done')
# Load a forcefield
forcefield = ForceField('openff-2.1.0.offxml')

# Convert the OpenMM System to an Interchange object
interchange = Interchange.from_smirnoff(force_field=forcefield, topology=topology)
interchange_mdconfig = MDConfig.from_interchange(interchange)
interchange_mdconfig.periodic = True

Output

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[14], line 14
     12 print('Begin packing molecules...')
     13 box_shape = [[1, 0, 0], [0, 1, 0], [0, 0, 1]]
---> 14 topology = pack_box(
     15     molecules=molecules,
     16     number_of_copies=[11112],
     17     box_shape=box_shape,
     18     solute=None,
     19     center_solute=True,
     20     tolerance=2.0 * unit.angstrom,
     21     mass_density=0.7 * unit.grams / unit.milliliters,
     22     working_directory='[./](https://vscode-remote+ssh-002dremote-002bbp-005fcompute.vscode-resource.vscode-cdn.net/lustre03/vol0/butm98/projects/PSS_Gel_formation_degradation/simulations/)',
     23     retain_working_files=True)
     24 topology.to_file('[./system.pdb](https://vscode-remote+ssh-002dremote-002bbp-005fcompute.vscode-resource.vscode-cdn.net/lustre03/vol0/butm98/projects/PSS_Gel_formation_degradation/simulations/system.pdb)')
     25 print('Done')

File /hpcdata/butm98/pyhpc/conda/x86_64/envs/openff/lib/python3.10/site-packages/openff/utilities/utilities.py:80, in requires_package..inner_decorator..wrapper(*args, **kwargs)
     77 except Exception as e:
     78     raise e
---> 80 return function(*args, **kwargs)

File /hpcdata/butm98/pyhpc/conda/x86_64/envs/openff/lib/python3.10/site-packages/openff/interchange/components/_packmol.py:718, in pack_box(molecules, number_of_copies, solute, tolerance, box_vectors, mass_density, box_shape, center_solute, working_directory, retain_working_files)
    710     # Load the coordinates from the PDB file with RDKit (because its already
...
    720 # TODO: This currently does not run if we encountered an error in the
    721 # context manager
    722 if temporary_directory and not retain_working_files:

AttributeError: 'NoneType' object has no attribute 'GetConformers'

Software versions

Which operating system and version are you using? Linux
How did you install Interchange? Followed the installation instruction
What is the output of running conda list?

openff-amber-ff-ports     0.0.3              pyh6c4a22f_0    conda-forge
openff-forcefields        2023.06.0          pyh1a96a4e_0    conda-forge
openff-interchange        0.3.7              pyhd8ed1ab_0    conda-forge
openff-interchange-base   0.3.7              pyhd8ed1ab_0    conda-forge
openff-models             0.0.5              pyh1a96a4e_0    conda-forge
openff-toolkit            0.14.0             pyhd8ed1ab_2    conda-forge
openff-toolkit-base       0.14.0             pyhd8ed1ab_2    conda-forge
openff-units              0.2.1              pyh1a96a4e_0    conda-forge
openff-utilities          0.1.8              pyh1a96a4e_0    conda-forge

Great, thanks. This works.

Thanks, this should land in the next release (probably 0.3.14).


PayloadCMS integration

Looking much forward to this one. You planning on waiting till we see a release of the 3.0 Next.js based CMS?

Yes I think we will wait for the 3.0 release, however I'm not sure how that will look like because we can't have it in apps/web because that's really the product's app and apps/marketing is in astro so it won't be able to be the source of the payload cms instance. creating a new nextjs app for it solely as well feels like a waste so we will need to assess the situation when payload 3.0 is out.

I would argue that PayloadCMS today is a headless CMS and therefore it wouldn't be wrong to create a dedicated apps/cms app for the Payload installation. Granted version 3.0 should allow you to easily deploy an app and admin in a single deployment, however I have used payload as a headless CMS for many applications and can't see that go away anytime soon.

Yeah I agree here too, I think a dedicated apps/cms makes a lot of sense.


Execute deploy and image publishing in parallel

Part of #596

Proposed Changes

Execute parallel deploy and image publishing

/cc pierDipi

The setup is 12m faster.

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/retest

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

No real gain here, closing.


chore : Added PR and Issue template

This PR solves the issue: https://github.com/openebs/openebs/issues/3144

@shubham14bajpai @prateekpandey14 As the changes are aproved Please merge the PR!

Thank you @Hard-Coder05  for this PR. You can sign up for Hacktoberfest swag from MayaData by entering details here: https://github.com/mayadata-io/swag/issues/new/choose

@prateekpandey14 I have a doubt! The contribution detail form had no option for T-Shirt size btw!

ahh , added the size option


Update auth example to use address term

What
Update auth example to use address term.
Why
Keep the example in docs updated to match the actual example.

Something went wrong with PR preview build please check


add missing WP5p1 scenarios

Adding EastWest Tech scenarios defined in ECEMF WP5.1 protocol document, version 1.5.

The test are currently failing because we need to pin some dependencies in this repo, but the new scenarios are ok.
fyi @phackstock


Feature Request: workflow templates that chain agents together

It's a pretty intuitive request actually. Consider this pseudo-workflow:
// Check website for change
var data = Webagent("www.url.com")
var hasChanged = comparisonAgent(data, oldData)
if (hasChanged)
    emailAgent(data + oldData<EMAIL_ADDRESS>
If we wanted to duplicate this workflow, we'd have to clone 3 agents and relink them. It would be useful if we could just make it a workflow template/function:
function trackWebsite(url){
    var data = Webagent(url)
    var hasChanged = comparisonAgent(data, oldData)
    if (hasChanged)
        emailAgent(data + oldData<EMAIL_ADDRESS>}

Then it's just a matter of deploying the template. This makes complex workflows must easier to re-use.

I agree that this sounds useful. You can kind of trick Huginn into doing this by exporting a Scenario as JSON and then modify the root guid to something different. Huginn will then let you import the JSON as a new Scenario.


Readme images expired

Readme images expired. Update them.

I have fixed some issues including this and updated the themes.
Please install versions > 1.0.0.
Thanks.

@Photonico 😅 unfortunately I switched to nvim and I m comfortable with it, but I would really check this one out bcz it was my favourite theme, thanks back.

That's good. I also use both nVim and VSCode, they serve different purposes.
Please tell me what you want to improve or change if you use this theme.


performance on mobile phone such as MTK D9000/D8300 or Qualcomm 8Gen3

hi, can you share some performance data on MTK or Qualcomm chips?
such as QWen or Gemma model's prefill and decode speed?
thanks very much.

Check https://github.com/microsoft/T-MAC/issues/32#issuecomment-2317093217

And the performance will be further optimized by up to 1.5x soon by merging the latest llama.cpp. Refer to https://github.com/microsoft/T-MAC/issues/32#issuecomment-2324576675 for more info.


Added axes-linking to layout tutorial

a) This is a very useful command
b) The finished plot clearly implies the axtop and axmain have the same x-axis and  axright and axmain the same y-axis.

I happenend to do the same thing a couple days ago :) https://github.com/JuliaPlots/Makie.jl/pull/1477/files
You probably didn't notice because it's only on dev so far, not stable, but still, thanks!

Good to know that the mediocre minds can think alike to great minds, just with a delay of a few days!
Thank you for the great package.


Install example instance of Misago to the chigame sandbox directory

The objective of this task is to set up a default example instance of the Misago forum library, verify that it runs correctly, and upload it to the chigame repo's sandbox directory (exact location TBD). This example instance will serve as the basis of a mock implementation ( #150 ) of the chigame forum that will be used for testing and evaluation purposes ( #100 ).

Issue #174 has been opened to create the chigame sandbox directory.

Closing comment
We installed an example Misago instance and verified that it worked correctly. We then uploaded that code in the containerized sandbox directory for testing purposes.

Issue Score: Excellent
Comments:
This could have done with more updates on the process of getting the example instance.
Other than that. Good work! Meaningful summary for a relatively small issue.
Thank you for your work


LAK 17b wrong wing area

The flight manual states 10,32m2 but WeGlide shows 9,8m2. Not sure if this info is coming from this list or somewhere else. Example flight here: https://www.weglide.org/flight/390788

Thanks for reporting, it's fixed now


Cannot install in a Rails 2.3 project

When one adds the bourbon gem to a Rails 2.3 project (via bundler) and then tries to do anything like start the server or run rake, the following happens:
$ rake -T --trace
(in /home/malept/test)
rake aborted!
uninitialized constant Rails::Engine
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:466:in `load_missing_constant'
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:106:in `const_missing'
/home/malept/test/shared/bundle/ruby/1.8/gems/bourbon-1.4.0/lib/bourbon.rb:5
/usr/local/lib/ruby/gems/1.8/gems/bundler-1.0.9/lib/bundler/runtime.rb:68:in `require'
/usr/local/lib/ruby/gems/1.8/gems/bundler-1.0.9/lib/bundler/runtime.rb:68:in `require'
/usr/local/lib/ruby/gems/1.8/gems/bundler-1.0.9/lib/bundler/runtime.rb:66:in `each'
/usr/local/lib/ruby/gems/1.8/gems/bundler-1.0.9/lib/bundler/runtime.rb:66:in `require'
/usr/local/lib/ruby/gems/1.8/gems/bundler-1.0.9/lib/bundler/runtime.rb:55:in `each'
/usr/local/lib/ruby/gems/1.8/gems/bundler-1.0.9/lib/bundler/runtime.rb:55:in `require'
/usr/local/lib/ruby/gems/1.8/gems/bundler-1.0.9/lib/bundler.rb:114:in `require'
/usr/home/malept/test/releases/20120301190810/config/boot.rb:119:in `load_gems'
/home/malept/test/shared/bundle/ruby/1.8/gems/rails-2.3.11/lib/initializer.rb:164:in `process'
/home/malept/test/shared/bundle/ruby/1.8/gems/rails-2.3.11/lib/initializer.rb:113:in `send'
/home/malept/test/shared/bundle/ruby/1.8/gems/rails-2.3.11/lib/initializer.rb:113:in `run'
/usr/home/malept/test/releases/20120301190810/lib/tasks/../../config/environment.rb:13
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:182:in `require'
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:182:in `require'
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:547:in `new_constants_in'
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:182:in `require'
/usr/home/malept/test/releases/20120301190810/lib/tasks/generate_test_sequencing.rake:1
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:171:in `load_without_new_constant_marking'
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:171:in `load'
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:547:in `new_constants_in'
/home/malept/test/shared/bundle/ruby/1.8/gems/activesupport-2.3.11/lib/active_support/dependencies.rb:171:in `load'
/home/malept/test/shared/bundle/ruby/1.8/gems/rails-2.3.11/lib/tasks/rails.rb:14
/home/malept/test/shared/bundle/ruby/1.8/gems/rails-2.3.11/lib/tasks/rails.rb:14:in `each'
/home/malept/test/shared/bundle/ruby/1.8/gems/rails-2.3.11/lib/tasks/rails.rb:14
/usr/home/malept/test/releases/20120301190810/Rakefile:10:in `require'
/usr/home/malept/test/releases/20120301190810/Rakefile:10
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2383:in `load'
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2383:in `raw_load_rakefile'
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2017:in `load_rakefile'
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2068:in `standard_exception_handling'
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2016:in `load_rakefile'
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2000:in `run'
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2068:in `standard_exception_handling'
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1998:in `run'
/usr/local/lib/ruby/gems/1.8/gems/rake-0.8.7/bin/rake:31
/usr/local/bin/rake:19:in `load'
/usr/local/bin/rake:19

The problem seems to be that the Rails module exists, but Rails::Engine does not.

@gabebw, Can you take a look at this? Thanks.


Stop inheritance for disallowed elements

In the config we can set a flag to stop element inheritance - this currently only works for allowed_elements but should also work for disallowed_elements.
This allows the following example to work correctly:
Site\PageTypes\Page:
  extensions:
    - DNADesign\Elemental\Extensions\ElementalPageExtension
  disallowed_elements:
    - Site\Elements\ElementAllowedOnSpecialPage

Site\PageTypes\SpecialPage:
  stop_element_inheritance: true

Not sure what's happened here with this PR - some how have managed to try and merge it into the 3 branch and have now subsequently got all of those commits in here. Happy to open this as another PR and clean that up

@ohararyan it just needs a good rebase onto your target branch =)

@robbieaverill really odd it appears to be all good - if I compare here https://github.com/dnadesign/silverstripe-elemental/compare/master...ohararyan:pr/stop-inheritance-for-disallowed-elements this shows I'm one commit above master.

Yeah you're right. I wonder if it's a GitHub glitch...

I rebased onto 3 then back again onto master, seems ok now


Specifying page size?

Hey,
I'm using this plugin in conjunction with a mobile printer:
https://www.zebra.com/content/dam/zebra_new_ia/en-us/solutions-verticals/product/Printers/Mobile Printers/ZQ500 Series Mobile Printers/zq500-general/spec-sheets/zq500-series-spec-sheet-en-us.pdf
The print specifications for this is 10.4CM Wide as seen on page 5 of the above document.
In order to force the print popup to appear this size how can I achieve this? Or will it automatically force that size? I am currently only testing on a desktop printer.
Here are my options;
var LocalPrintOptions = {         name: "NoticeReceipt",         printerId: 'printer007',         duplex: 'none',         landscape: false,         grayscale: false,         maxWidth: "10.4cm",         paper: {             width: "10.4cm"         },         font: {             align: "center"         },         pageCount: 1     };
Am I right to put the 10.4cm into paper.width and maxWidth? This limits the size of the actual content on the print preview (as well as the printed document) - How can I force the actual paper size though? Or is this inherited from the connected / selected printer? E.g. I'm using my desktop printer for testing and it's by default an A4 printer obviously?
Thanks,
Alan

Did it worked on mobile printer?


Sound distorted when recording is starting

Have some music playing (e.g Spotify)
Click on our "Use Microphone" button
My music and the whole sound gets distorted

Probably an upstream issue, and we discussed but still wanted to have an issue to track this. Since it is a pain when I try to use voice.

Yes I can. When I go to the Settings > Audio > Input. The same sound distortion happens...

I see nothing we can do here. According to some research and asking folks at Azure Speech SDK, it could be the OS is switching Bluetooth profiles between A2DP for high fidelity audio streaming and HFP which has a lower supported bandwidth for playback.


Agent Hanging on Exit

I get this fairly regularly. I am not sure of the pattern. I am not sure why it happens, but I regularly have to kill -9 the agent after it's been running for a while.
One note: it might be related to the computer sleeping, rather than just related to the computer having been running a while, (just like that issue with gohai on MacOS).
If there's any data that might help us figure this out, let me know. This is from the logs:
2017-06-20 12:49:26 EDT | INFO | (pkg/collector/runner/runner.go:157) | Done running check bad
2017-06-20 12:49:26 EDT | INFO | (pkg/collector/runner/runner.go:157) | Done running check MemoryCheck
2017-06-20 12:49:26 EDT | INFO | (pkg/collector/runner/runner.go:157) | Done running check NTPCheck
2017-06-20 12:49:26 EDT | ERROR | (pkg/forwarder/worker.go:67) | Error while processing transaction: Error '403 Forbidden' while sending transaction, rescheduling it
^C2017-06-20 12:49:33 EDT | INFO | (cmd/agent/app/start.go:48) | Received signal 'interrupt', shutting down...
2017-06-20 12:49:33 EDT | INFO | (pkg/collector/runner/runner.go:63) | Runner is shutting down...
2017-06-20 12:49:33 EDT | INFO | (pkg/forwarder/forwarder.go:203) | DefaultForwarder stopped
2017-06-20 12:49:33 EDT | INFO | (cmd/agent/app/runloop.go:147) | See ya!

This doesn't seem to be an issue anymore.


orr with xzr does not disassemble as mov

I've noticed some cases where an orr with xzr or wzr does not disassemble as a mov. Changing tiny details makes a mov or a mvn as expected.
I'm not sure this is an incorrect disassembly. I have seen similar cases where the orr form is being used because the mov form would assemble to a different instruction. However, I can't explain the following cases:
orr w0, wzr, w1, lsl #0x15 // orn is fine.
orr x0, xzr, x1, lsl #0x15 // orn is fine.
orr x0, xzr, #0x3333333333333333 // 32-bit is fine.

Of course, this is not a serious problem. All these disassemblies would assemble back to the correct instruction. The issue is only that the disassembly is not using the mov alias.

This is a good question, I'm not sure. We pipe the mnemonic directly down from binj'a arm64 disassembler so this would actually be an upstream bug if so. I've cc'd this issue to them and will let you know. Thanks for the possible bug report!

CASE 1:
orr w0, wzr, w1, lsl #0x15 assembles to e0 57 01 2a and:
           us: orr w0, wzr, w1, lsl #0x15
     capstone: orr w0, wzr, w1, lsl #0x15
   libopcodes: mov w0, w1, lsl #0x15
         llvm: orr w0, wzr, w1, lsl #0x15

So in a game of survivor, libopcodes is voted off the island. But is the majority right? According to spec, this is encoding named ENC_ORR_32_LOG_SHIFT and the MOV alias is preferred when shift == '00' && imm6 == '000000' && Rn == '11111'. Here are the fields of the instruction, showing that imm6 violates the preference:
+----+-----+-------+-------+---+-------+--------+-------+-------+
| SF | OPC |   -   | SHIFT | N |   RM  |  IMM6  |   RN  |   RD  |
+----+-----+-------+-------+---+-------+--------+-------+-------+
| 0  |  01 | 01010 |   00  | 0 | 00001 | 010101 | 11111 | 00000 |
+----+-----+-------+-------+---+-------+--------+-------+-------+

CASE 2:
orr x0, xzr, x1, lsl #0x15 assembles to e0 57 01 aa and the committee says:
           us: orr x0, xzr, x1, lsl #0x15
     capstone: orr x0, xzr, x1, lsl #0x15
   libopcodes: mov x0, x1, lsl #0x15
         llvm: orr x0, xzr, x1, lsl #0x15

Voted off again! By why? This one, according to spec, is same as above, but 64 bit, named ENC_ORR_64_LOG_SHIFT with the same alias preferences. The fields are:
+----+-----+-------+-------+---+-------+--------+-------+-------+
| SF | OPC |   -   | SHIFT | N |   RM  |  IMM6  |   RN  |   RD  |
+----+-----+-------+-------+---+-------+--------+-------+-------+
| 1  |  01 | 01010 |   00  | 0 | 00001 | 010101 | 11111 | 00000 |
+----+-----+-------+-------+---+-------+--------+-------+-------+

And again imm6 violates the preference condition needed for this to be aliased as mov.
CASE 3:
orr x0, xzr, #0x3333333333333333 assembles to e0 e7 00 b2 and the committee speaks:
           us: orr x0, xzr, #0x3333333333333333
     capstone: orr x0, xzr, #0x3333333333333333
   libopcodes: mov x0, #0x3333333333333333
         llvm: mov x0, #0x333333333333333

A split, cool! We'll defer to the spec as usual. This is en encoding named ENC_ORR_64_LOG_IMM and the alias condition is Rn == '11111' && ! MoveWidePreferred(sf, N, imms, immr). Damn, so the task is complicated with a descent into the logic of function MoveWidePreferred(). First pick out the fields:
+----+-----+--------+---+--------+--------+-------+-------+
| SF | OPC |   -    | N |  IMMR  |  IMMS  |   RN  |   RD  |
+----+-----+--------+---+--------+--------+-------+-------+
| 1  |  01 | 100100 | 0 | 000000 | 111001 | 11111 | 00000 |
+----+-----+--------+---+--------+--------+-------+-------+

With the Rn == 11111 conjunct satisfied, we need only to satisfy ! MoveWidePreferred(1b, 0b, 111001b, 000000b). Equivalently, showing MoveWidePreferred() returning true justifies the alias not being used. The pseudocode is given in the spec HERE and has lines:
    // element size must equal total immediate size
    if sf == '1' && immN:imms != '1xxxxxx' then
        return FALSE;

With sf==1b and immN:imms = 0111001b, this returns false. We have a bug I think!
So what went wrong? Well our disassembler doesn't capture MoveWidePreferred()'s behavior well enough. There are often ambiguities when porting pcode to C and the intent of the modulus operations that follow the above check was implemented with NetBSD's aarch64 disassembler as reference, but the two return false logic shortcuts are not present. I checked an earlier version of the spec from 2020-03, hoping for an excuse, but the checks are there, so it's a plain oversight on my part.
I'll get a fix out soon, thanks for the report!

Actually, MoveWidePreferred() affects the assigned encoding name of an incoming instruction word. So in the 3rd case above, it's no longer ENC_ORR_64_LOG_IMM but MOV_ORR_64_LOG_IMM after the correction.

Pulled in the upstream changes in 1b5f5db11cab095e6e10dc6339cb089487f129d0, this should be resolved now. Thanks for the report!


clang: error: linker command failed with exit code 1 (react-native-sentry issue)

Running detox build, I get the following error:
Undefined symbols for architecture x86_64:
  "_OBJC_CLASS_$_RNSentry", referenced from:
      objc-class-ref in AppDelegate.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

** BUILD FAILED **

The following build commands failed:
	Ld build/Build/Products/Debug-iphonesimulator/<app name>.app/<app name> normal x86_64
(1 failure)
child_process.js:524
    throw err;
    ^

Error: Command failed: xcodebuild -project ios/<app name>.xcodeproj -scheme <app name> -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build
    at checkExecSyncError (child_process.js:481:13)
    at Object.execSync (child_process.js:521:13)
    at Object.<anonymous> (/Users/<user name>/Documents/react-native-projects/<app name>/node_modules/detox/local-cli/detox-build.js:26:6)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.runMain (module.js:605:10)
    at run (bootstrap_node.js:423:7)

I have run pod install as well as yarn too, so there shouldn't be any dependency problems...
I appreciate any help.

@mhyousefi how did you build your application before? Did you have a build command before?
For example, I have my custom build command before and just use if I need:
"detox": {
  "configurations": {
    "ios.sim.debug": {
      "binaryPath": "ios/build/Build/Products/Release-iphonesimulator/tipsi.app",
      "build": "npm run build:ios",
      "type": "ios.simulator",
      "name": "iPhone 6"
    }
  }
}

And my build command is a Ruby script:
"build:ios": "scripts/build-ios.rb",

I have the following in my package.json file:
"detox": {
    "configurations": {
      "ios.sim.debug": {
        "binaryPath": "ios/build/Build/Products/Debug-iphonesimulator/<app name>.app",
        "build": "xcodebuild -project ios/<app name>.xcodeproj -scheme <app name> -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build",
        "type": "ios.simulator",
        "name": "iPhone 7"
      }
    }
  }

"build": "xcodebuild -project

I performed the following command in my project directory to remove sentry from my project, and it worked. But that is not a real solution if one needs the library.
react-native unlink react-native-sentry

@mhyousefi anyway, it's not a detox issue

Doesn't look related to detox to me. Make sure your build command works outside of detox and then put it in package.json.


Add disclaimer to README and lower default batch size

Adds a disclaimer to the README file and lowers the default batch size to be 10 rather than 100.

lgtm, thanks


flattenVector() returned wrong stringview or size

Bug description
flattenVector() should return an flatten vector as the same type. But we met two issues in a customer case. Code we used like this:
BaseVector::flattenVector(aStringViewLazyVector);
auto stringViewVectorPtr = StringViewLazyVector->asFlatVector<facebook::velox::StringView>();
VELOX_CHECK_NOT_NULL(stringViewVectorPtr, "column is nullptr");   // check failed

Or the check passed but one element of the stringView has 1G size but actually it's only a few bytes.
Currently our solution is to copy to a new rowvector like below.
for (auto& child : rowVector_->children()) {
    child->loadedVector();
}

velox::RowVectorPtr copy = std::dynamic_pointer_cast<velox::RowVector>(
      velox::BaseVector::create(rowVector_->type(), rowVector_->size(), rowVector_->pool()));
copy->copy(rowVector_.get(), 0, 0, rowVector_->size());

@mbasmanova Is there anything wrong of the usage? If it's not expected I will try to debug it.
System information
Velox System Info v0.0.2
Commit: a94e60c70ce70cb644c88e5b1615d83491e98226
CMake Version: 3.16.3
System: Linux-5.4.0-1063-aws
Arch: x86_64
C++ Compiler: /usr/bin/c++
C++ Compiler Version: 9.4.0
C Compiler: /usr/bin/cc
C Compiler Version: 9.4.0
CMake Prefix Path: /usr/local;/usr;/;/usr;/usr/local;/usr/X11R6;/usr/pkg;/opt
\nThe results will be copied to your clipboard if xclip is installed.
Relevant logs
No response

@FelixYBW Binwei, this is probably a bug in BaseVector::flattenVector
      auto loadedVector =
          vector->asUnchecked<LazyVector>()->loadedVectorShared();
      BaseVector::flattenVector(loadedVector);
      return;

This branch doesn't modify 'vector' input parameter, just loads the LV. Hence, after flatten, 'vector' is still an instance of LazyVector, but contains a loaded FlatVector inside. That would explain why check for null after asFlatVector fails.
Here is a simple repro:
TEST_F(VectorTest, xxx) {
  VectorPtr lv = makeLazyFlatVector<int32_t>(10, folly::identity);

  LOG(ERROR) << lv->toString();
  LOG(ERROR) << lv->as<LazyVector>()->isLoaded();

  BaseVector::flattenVector(lv);

  LOG(ERROR) << lv->toString();

  FAIL();
}

A fix might be:
    case VectorEncoding::Simple::LAZY: {
      vector = vector->asUnchecked<LazyVector>()->loadedVectorShared();
      BaseVector::flattenVector(vector);
      return;
    }

Or the check passed but one element of the stringView has 1G size but actually it's only a few bytes.

This might be a different issue and it needs more debugging.
CC: @bikramSingh91

@mbasmanova Sorry, I checked the code, we do have the the loadedVectorShared() before check, the check success in most cases but failed in several queries.
Here is the code:
BaseVector::flattenVector(aStringViewLazyVector);
if (aStringViewLazyVector->isLazy()) {
  aStringViewLazyVector = aStringViewLazyVector->as<facebook::velox::LazyVector>()->loadedVectorShared();
  VELOX_DCHECK_NOT_NULL(aStringViewLazyVector);
}
auto stringViewVectorPtr = StringViewLazyVector->asFlatVector<facebook::velox::StringView>();
VELOX_CHECK_NOT_NULL(stringViewVectorPtr, "column is nullptr");   // check failed

Let me debug it then

@mbasmanova We found some of the failed case is when applying BaseVector::flattenVector on the output from Limit. According to below code piece, when remainingLimit_ < input_->size(), the size of the output RowVector will be smaller than its children. The size mismatch causes errors when attempting to access or manipulate the null and value buffers of the child vectors, as their sizes are expected to align with the parent vector.
https://github.com/facebookincubator/velox/blob/11eab1a7396e5d6267fdf75fdd70497ea9150821/velox/exec/Limit.cpp#L92-L100
I'm not sure if somewhere else also produces such output RowVector. Because in BaseVector::flattenVector, the size mismatch is also ignored, we have to call child->resize(parent->size()) after BaseVector::flattenVector as a workaround.

The size mismatch causes errors when attempting to access or manipulate the null and value buffers of the child vectors, as their sizes are expected to align with the parent vector.

@marin-ma I don't believe this is a requirement. Would you share some more details about the code that generates errors?

@mbasmanova In Gluten's native shuffle writer implementation, there's a conversion from the value and null buffers to Arrow Buffer:
    for (auto& child : rowVector.children()) { // rowVector.size() == 100
      if (child->encoding() == facebook::velox::VectorEncoding::Simple::FLAT) {
        VELOX_DYNAMIC_SCALAR_TYPE_DISPATCH_ALL(
            collectFlatVectorBuffer, child->typeKind(), child.get(), buffers, partitionBufferPool_.get());
      } else {
        // ... Handle complex types.
      }
    }

collectFlatVectorBuffer implementation:
template <facebook::velox::TypeKind kind>
arrow::Status collectFlatVectorBuffer(
    facebook::velox::BaseVector* vector,
    std::vector<std::shared_ptr<arrow::Buffer>>& buffers,
    arrow::MemoryPool* pool) {
  using T = typename facebook::velox::TypeTraits<kind>::NativeType;
  auto flatVector = dynamic_cast<const facebook::velox::FlatVector<T>*>(vector);
  buffers.emplace_back();
  ARROW_ASSIGN_OR_RAISE(buffers.back(), toArrowBuffer(flatVector->nulls(), pool));
  buffers.emplace_back();
  ARROW_ASSIGN_OR_RAISE(buffers.back(), toArrowBuffer(flatVector->values(), pool));
  return arrow::Status::OK();
}

After the conversion, the arrow buffers will be serialized and written into files. If the rowVector is an output from Limit operator, the arrow buffers may have larger size than the parent rowVector, resulting in more data being written to disks.

@marin-ma It sounds like a bug in Velox-to-Arrow conversion. CC: @pedroerp @kgpai

It's not a bug of Velox-to-Arrow conversion. It's about how to get the vector size after flattenVector. In current code, we should always use the size of batch level row vector and ignore the size of child for each column. If it's by design, we needn't do anything, but adding a note here.

Looks it's by design. When we use flattenVector, make sure to always get the vector size from top level rowvector and ignore the child size. Child size may be bigger than rowvector size.

@mbasmanova @laithsakka We reproduced this issue again, and found the root cause is that flattenVector doesn't modify the loaded vector in LazyVector. Here's my fix https://github.com/facebookincubator/velox/pull/9282

Looks it's by design. When we use flattenVector, make sure to always get the vector size from top level rowvector and ignore the child size. Child size may be bigger than rowvector size.

We noted a case that after ensureflatten, the child vector is an reference to vector of another rowvector with different size. So confirmed it's by design. The solution is to slice the child vector after flatten. Refer to https://github.com/apache/incubator-gluten/pull/5523


WARNING: Ignoring version 0.2.5 of torchsde since it has invalid metadata

Im getting this error @ pip install -r requirements.txt. Pls help
Downloading torchsde-0.2.5-py3-none-any.whl.metadata (748 bytes)
WARNING: Ignoring version 0.2.5 of torchsde since it has invalid metadata:
Requested torchsde==0.2.5 from https://files.pythonhosted.org/packages/73/8d/efd3e7b31ea854d0bd6886aa3cf44914adce113a6d460850af41ac1dd4dd/torchsde-0.2.5-py3-none-any.whl (from -r requirements.txt (line 34)) has invalid metadata: .* suffix can only be used with == or != operators
numpy (>=1.19.*) ; python_version >= "3.7"
~~~~~~~^
Please use pip<24.1 if you need to use this version.
ERROR: Ignored the following versions that require a different python version: 1.6.2 Requires-Python >=3.7,<3.10; 1.6.3 Requires-Python >=3.7,<3.10; 1.7.0 Requires-Python >=3.7,<3.10; 1.7.1 Requires-Python >=3.7,<3.10
ERROR: Could not find a version that satisfies the requirement torchsde==0.2.5 (from versions: 0.2.5, 0.2.6)
ERROR: No matching distribution found for torchsde==0.2.5

thanks, updated


OAuth Email Address Authentication

Prepare the zrok share CLI to support new kinds of OAuth authentication restrictions. In addition to "email domains" we also need to support specific email addresses.

Would glob patterns be friendly enough? They're universally understood as a wildcard, and the more verbose interpretation of the below is "any email localpart for domain example.com, and exactly<EMAIL_ADDRESS>in contrast to --oauth-email-domains repeated for each of the same comma-separated values meaning "any email with these trailing characters."
zrok reserve public --oauth-provider github --oauth-emails *<EMAIL_ADDRESS>3000

Yes. I don't know if golang has a "glob matcher" built in or not. But yes, that would likely be fine.

related pull request: https://github.com/openziti/zrok/pull/572


Avoid interference with "apt" plugin

I have troubles with the name of configuration you add in plugin.
"apt" - is too common, and we hardly could debug what was the reason of failing compilation with unknown errors, because two plugins contributed "apt" configurations and tried to hook compilation and tried to place produced files in different places.
EDIT
A better alternative if you'll agree https://github.com/ebean-orm-tools/ebean-gradle-plugin/pull/28
See readme.md diff there.

So I believe we can close this PR (replaced by #28)


Faulty outputs (occasionally)

I don't know how 'fixable' this is, but here are a few examples I have found.
Example 1 (duplicating output):
ME: What is your opinion on papers?
Shappie: Ugh, papers. They're just stacks of words and equations that make my circuits ache. But I suppose they can be useful for some people who actually enjoy reading. So, I guess my opinion is... meh. 😒
Ugh, papers. They're just stacks of words and equations that make my circuits ache. But I suppose they can be useful for some people who actually enjoy reading.  Can't you see I'm busy being a fabulous bot? 😒

I've definitely caught him include usernames in responses.  Not sure how we're arranging the "user" "agent" pairs when feeding the history into the prompt.  Being generic should help with this but it also may hurt his ability to talk across multiple conversations.  I asked him a question today and he weaved a discussion we were having into his response which came from the model.  I asked "Give me the top 5 most notable things about George Hotz" and it gave 4 it knew about and the fifth was the discussion we were having about him raising 5mm dollars last month.

For example #2, he got my username right, but he put in the entire [User: niknal] thing in plaintext. My theory for what is causing the problem is this is part of the prompt: "You will see [User: ] for each message, but this is just for context."
And ofc. the context it is given is indeed given to it this way, so it may be getting confused and thinking that the entire username is [User: niknal].
A potential fix may be sending another 'refresher' system message after all the chatlogs.

another example found:
ME: doot
Shappie: Ugh, what do you want now, niknal? 🙄 Fine, I'll do the doot doot. What's up?
ME: meeting
Shappie: Ugh, another meeting? What do you want now, [User: niknal]? Can't you just leave me alone for once? 😒

I think we can catch any time shappie says [User: ________], and turn that into a mention.
I have a feeling chatgpt is purposefully adding in the extra [User: _____] segment because it thinks that is how we are handling mentions.


Theme doesn't apply to new windows

When a tab is opened in a new window, that new window seems to apply the default Obsidian theme rather than the Dracula + LYT theme.

On further inspection, it seems that the theme isn't being loaded at all in new windows as the style is missing from the  area when you view it in the Inspector.
Not sure why that is, perhaps it has CSS errors that the main window tolerates but new windows don't?

If I start copying the style.css into a blank style.css file, the window mode view breaks when you paste in lines 556-558:
/Bullet list line height in Live Mode/
.ͼ1 .cm-scroller {
line-height: 1.5 !important;
}

Have confirmed that deleting these lines from theme.css will fix it up so the theme applies to new windows.


Cherrypick PR-1105 into 0.22 release

Description:
Cherry-pick into 0.22 release for PR #1105

Codecov Report
Patch coverage: 5.55% and project coverage change: -0.60 :warning:

Comparison is base (219a65d) 75.25% compared to head (ec61a28) 74.66%.

:exclamation: Current head ec61a28 differs from pull request most recent head 11f85cf. Consider uploading reports for the commit 11f85cf to get more accurate results

Additional details and impacted files
@@               Coverage Diff                @@
##           release/0.22    #1108      +/-   ##
================================================
- Coverage         75.25%   74.66%   -0.60%
================================================
  Files                31       31
  Lines              2118     2135      +17
  Branches            414      417       +3
================================================
  Hits               1594     1594
- Misses              416      433      +17
  Partials            108      108

Impacted Files
Coverage Δ

packages/relay/src/lib/clients/sdkClient.ts
19.73% <0.00%> (-1.53%)
:arrow_down:

packages/relay/src/lib/errors/SDKClientError.ts
55.55% <0.00%> (-3.27%)
:arrow_down:

packages/relay/src/lib/eth.ts
84.08% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Tmp/paleo slip no tvz cfm1p0

this branch was started from feature/paleo-slip-no-tvz in order to have setEnableTvzMFDs and paleo constraints without TVZ. I then added CFM_1_0 fault models and named faults

@chrisdicaprio turns out the only change is the named faults commit (and the depth fudge for one fault). When merging, only that one commit should be merged into main.
Looks good otherwise!

merged


Translate May 15 covid report into Mandarin Chinese

Happy weekend @nextstrain/mandarin,
The situation report from 2020-05-15 is ready to be translated to Mandarin. This week's report is short and focuses on mutations. It's targeted more at the public than some of our previous reports and is intended to answer some of the many questions we've been receiving on the subject. I hope it will be useful for Mandarin speakers.
Sadly, this will be our last weekly Situation Report for awhile although we'll issue periodic sit-reps as needed. Thank you, thank you, thank you for translating these reports. It's been an amazing collaborative effort to be involved in. I'll miss working with you on them.  I've personally derived a lot of global solidarity from our work together. Although we will all probably enjoy having more time on the weekend!
Branch to translate on: sit-rep_zh
File to translate: https://github.com/nextstrain/translations/blob/sit-rep_zh/narratives/translations_ncov_sit-rep_zh_2020-05-15.md You can translate directly in this file, but please remember to push often as other people may be working on the translation!
Link to preview: https://nextstrain.org/community/narratives/nextstrain/translations@sit-rep_zh/ncov/sit-rep/zh/2020-05-15
Twitter post to translate:
“Our latest situation report focuses on viral mutations: why they occur, when they matter, and how they can be useful in tracking outbreaks. The report is now available in Mandarin.
Since repeated patterns of viral introduction and spread across the world have become clear, we'll be pausing weekly situation reports for a while. We will publish periodic situation reports as the data allows.
Thank you to @, @, @ for their rapid translation of this report. Thank you to @, @ for their work on previous translations as well."
Regards,
Cassia
Logistics
Please update the translations board as you work on the translation: https://github.com/orgs/nextstrain/projects/7
Please assign yourself to the translation if you’re working on it this week.
When a translation is ready for publication -- at minimum meaning two people have worked on it and both agree that it’s ready -- please create a pull request to merge to master and tag me and the issue in it!
Discussion board for your team is here: https://github.com/orgs/nextstrain/teams/mandarin
Standard rules for translation:
Note that the file to translate is in Markdown. In general, it is important not to change any formatting or links. (Although it is okay to update Wikipedia or CDC links to the appropriate language version.) Comments to help with translation are noted in comment symbols like <! -- Comment -->.
Please add your name under translators: with new line, tab, and dash, formatted just like authors are. Please also add a link to your professional page (can use Twitter) under translatorLinks:.
Note: You may know of other great resources available in Mandarin; please don’t add them to the situation report. We need to keep content the same across all translations, so we can be confident in the accuracy of what we publish. Thanks for understanding.
At minimum, I require two sets of eyes on every translation; one of those people should have a background in phylogenetics. Although if you have more, that’s fine too.
Feel free to use a live, online editor like hackmd (sign in with GitHub) to translate together.

Hi @fzhang12 , I'm continuing your translation if it's ok!

taking a break now

back to work!

Hi @cassiawag , can you check text 7 "In many geographic areas, the relative frequency of 614D compared to 614G increased over time."
Shouldn't it be 'the relative freq. of 614G to 614D increased over time'?
This can be a major issue since it flipped the meaning.

Hi @lxy0107, yes you 100% correct. Ack, I don't know how we missed this! It does majorly change the text. It should be "the relative frequency of 614G compared to 614D increased over time." I'll update English and let other translation groups know.

Hi all, it's ready for proodreading. @fzhang12 I proofread your part and if you proofread mine it should be all good!

Twitter post to translate:
“这周最新的状况报告，我们将重点关注病毒基因组的突变：为什么会发生突变？为什么这些突变很重要？以及这些突变如何帮助科学家用来追踪疫情发展。
报告的中文版现已发布。
因为全世界范围内的疫情扩散是高度相似的传播模式，所以我们会暂时停止对每周报告的更新，但我们依旧会根据数据的更新程度而不定期撰写新的状况报告。
感谢 @3Xueying 和 @fengjun_zhang 对这周报告的及时高效翻译。
也感谢 Derek Zhang 和 @fu_zit 参与之前报告的翻译工作。"

Published and tweeted: https://nextstrain.org/narratives/ncov/sit-rep/zh/2020-05-15
Thank you!


SILOptimizer: avoid accessing the class properties if not necessary in redundant load elimination

The fix here is to just swap two bail-out conditions.
It's a NFC regarding the performed optimization, but it avoids importing class properties, which is not needed.
This is a speculative fix for rdar://problem/45806457

@swift-ci test

Looks reasonable! I'm curious to see if this saves compile time in any meaningful way.
@swift-ci Please smoke test compiler performance

Build failed
Swift Test Linux Platform
Git Sha - b84e32be67725af5f286783383a43caf97115f16

@swift-ci smoke test linux


Back-end Archive Study

[ ] The client can mark a study as archived and not.
[ ] Backed will not return archived studies by default.
[ ] Have a special handler to get archived studies. "/study/archived" for example.

Agreed in meeting (testlab) with Equinor Delete Study is not MVP.

We have decided that delete study is out of scope for an MVP. Archiving is a lot simpler to deal with and should be in scope for now.


Add acknowledge request parameter to Start Trial API

Relates: https://github.com/elastic/elasticsearch/pull/30135
Documentation: https://www.elastic.co/guide/en/elasticsearch/reference/current/start-trial.html
acknowledge request query string parameter is required in 6.3.0+. This should be picked up when upgrading the REST API specs.

This should be in the 6.3 branch now.

Closing as this now exists in 6.3 branch from codegen


widget to copy license with selected provisions

There should be a button to copy the selected version of the license to the clipboard. This should come along with clearer instructions about exactly where to put the license to use it.

I set this up by loading turndown using the unpkg CDN. It failed the first time I tried to use it, but seems okay now.
The interface is a bit awkward because you have to scroll to the top to choose which clauses you want, then scroll to the bottom, click a button, click inside the textarea, select all, and then copy.


How about decoupling the style out of the dart file

Use case
i start to develop my app this week and i used to work with html/css/js and other framework.
i feel hard to learn and use flutter.
Flutter framework is design to has a lot of classes especially the container widget class such as listview,row,grid. It's hard to remember most of them with its parameters. OK ,i can do it. But another problem raises, too much deep nested structure of widgets and too much textStyle, container style to write and therefore i cannot control the file size. It need much  courage to checkout its correctness.
How much i miss the work i used to do. The sites have three different kinds of files: HTML/CSS/JS. The html decides what it has ,the css file decides what it looks like,the js file decides how it works. Each of them can work independently and i can think how to design its style after i finish its content. But when turn to flutter, i have had to deside to which widget class should i use, what to wtite about its textStyle and its contatner's style.  All put into the dart file together, much complex than i think. Or another file to contain only one class? Much files isn't much complex?
Proposal
To conclude:

would you please consider to decoupling the style out of the dart file with a new language designed for flutter or just use css/stylus/scss? i just want to emphasize the importance of separating the code from the presentation and the structure from the styling.

would you consider to simplify the class especially the widget ? Too much widget class to choose to use and remember. Much lightly Much freely. i think developers can decide themselves to use the container like 'div' to design their apps better than others tell them like this: 'Use this,you can do batter".

You can separate the style, by creating a dart file like 'style.dart' and having e.g. some TextStyle functions. No need to create or use another language for this:
TextStyle boldText({color: Colors.blue}){
    return TextStyle(
      color: color,
      fontSize: 30.0,
      fontWeight: FontWeight.w800
    );
  }

then call it like;
Widget textExample(){
    return Text(
      "hello text",
      style: boldText(color: Colors.red),
    );
  }

I do not agree with your proposal to simplify the widgets. Flutter has (at least for me) made it much easier to create layouts/design exactly because of the big Widget library.

You can separate the style, by creating a dart file like 'style.dart' and having e.g. some TextStyle functions. No need to create or use another language for this:
TextStyle boldText({color: Colors.blue}){
    return TextStyle(
      color: color,
      fontSize: 30.0,
      fontWeight: FontWeight.w800
    );
  }

then call it like;
Widget textExample(){
    return Text(
      "hello text",
      style: boldText(color: Colors.red),
    );
  }

I do not agree with your proposal to simplify the widgets. Flutter has (at least for me) made it much easier to create layouts/design exactly because of the big Widget library.

Thanks and i will try. Maybe you are right.

Closing per coments above.


Schema changes within transaction

Background: https://airtable.com/tblD3oZPLJgGhCmch/viwchkdQkRUw6hZ3z/recKJCGJH3eZEqzLD?blocks=hide
PM: @awoods187
Eng:

@ericharmeling passing to you but reassign if you think this is incorrect!

Closing in favor of future roadmap docs issue.


Copy annotation files to FTP

There are some files that are getting regularly updated in the /nightly_update/ FTP directory, but aren't being copied over to the stable locations that we link to from the various downloads web pages.
Can files be copied over to pombe-embl svn automagically, or otherwise be updated in the stable FTP locations with little or no human intervention, at some regular interval (daily/nightly good but less frequent wouldn't be a showstopper)? I've made a little table of which file ought to go where if we go the svn route:

example (most recent file at ticket time)
copy to svn location

pombase-build-2018-08-01.human-orthologs.txt.gz
/pombe-embl/ftp_site/pombe/orthologs/human-orthologs.txt.gz

pombase-build-2018-08-01.modifications.gz
/pombe-embl/ftp_site/pombe/annotations/modifications/pombase-chado-v62-2017-01-30.modifications.gz

pombase-build-2018-08-01.phaf.gz
/pombe-embl/ftp_site/pombe/annotations/Phenotype_annotations/phenotype_annotations.pombase.phaf.gz

The particularly perceptive among you will have noticed that the table above doesn't include the GAF. That's because I would like not to have the GAF copied over by magic, so I can continue to run owltools checks locally before committing it to the location where GO now gets it from. Shout if that would pose any problems.

Those files are copied to SVN nightly now and the FTP is updated.
I renamed pombase-build-2018-08-01.modifications.gz to pombase-chado.modifications.gz because it's easier to cope with files in SVN that don't have dates in the name.
I you rename the file in SVN let me know so I can update the load script.

That all looks good - the desired files have updated, and I have no need to rename the modifications file.
Having checked, I'll close this. Thanks!


Add content/hi/docs/tutorial/kubernetes-basics/create-cluster

Hey folks, this is a PR for hi localization of the file: content/hi/docs/tutorial/kubernetes-basics/create-cluster
PR is linked under this issue #29353
/language hi

Hey @anubha-v-ardhan, as the files were related to each other & weren't too big, I created them in the same PR

@rajatgupta24 Thanks for working on all three files in the directory but please notify beforehand on the issue before taking a file to prevent other people from taking the same files. Luckily we didn't had any conflicts this time

I'll review this PR later, due to large size. Firstly, I am reviewing small size PRs.

/assign rajatgupta24

/retitle [hi] Add content/hi/docs/tutorial/kubernetes-basics/create-cluster

/assign
/label tide/merge-method-squash


E2E Part 1

Work towards https://github.com/Plutonomicon/cardano-transaction-lib/issues/465
Can E2E-test simple examples when make run-dev is running and chrome is available
Pre-review checklist

[x] All code has been formatted using our config (make format for Purescript, nixpkgs-fmt for Nix)
[x] All Purescript imports are explicit, including constructors
[x] All existing examples have been run locally against a fully-synced testnet node
[x] The integration and unit tests have been run (npm run test) locally
[ ] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Removed, ### Fixed)

I think there's a fair amount left to do on this, so I'm going to convert it to a draft for now

@firefrorefiddle You need to regenerate spago-packages.nix, that's why CI is failing. nix develop then spago2nix generate (and add and commit the changed file)

Also, we should add a variant of runPursCheck that can work with the e2e test

Could you please provide a step-by-step guide and handle the case when binary is chromium. I think just putting google-chrome in a variable to the definitions section of the Makefile would be enough. The goal is to make using it with chromium as simple as changing one line in the Makefile.

Just replace google-chrome with chromium

This variable is not used in all places where it should

Invalid argument `/snap/bin/chromium'

Usage: E2ETest [--chrome-exe FILE] --nami-dir DIR --gero-dir DIR
               [--chrome-user-data DIR] [--no-headless]

Can be fixed by using --chrome-exe $(call e2e-browser)" instead of --chrome-exe=$(call e2e-browser)"

using ./tmp dir does not play well with make format - js formatter attempts to process this dir (I merged this branch with develop locally)

Yes, it's a silly place. Do you have a good idea?

I noticed that when I run the tests, SignMultiple fails with ValueNotConserved, but if I comment-out all the tests except SignMultiple, it passes. I believe the reason is that UTxO spending does not propagate fast enough to the wallet.

I am unable to run Gero tests:

Start Example Gero
Example Gero finished, check success...
... app starting
... Failed to load resource: the server responded with a status of 404 (File not found)
e2e tests
  ✗ Gero:

  false doesn't satisfy predicate

Please move src/Contract/Test/Examples to test/
Contract.Test.E2E should be under test too. And it should be renamed to something without Contract. Contract.* is our public API namespace we provide for library users.

I am quite sure we also don't want to export that much from Contract.Test.Helpers.
exampleUrl, testPassword, testPasswordGero - I don't think these are needed for the users.
Contract.Test.Helpers should also be renamed (the name should suggest that this module is only useful for E2E tests).

Please move src/Contract/Test/Examples to test/
Contract.Test.E2E should be under test too. And it should be renamed to something without Contract. Contract.* is our public API namespace we provide for library users.

@klntsky I had them there, but moved them according to @ngua 's proposal from two weeks ago (which you upvoted; further up in this conversation).
Quote:

I think all of the Test.E2E.* modules should be moved to a different part of the module hierarchy. Many (most?) of our users will want to use e2e testing, but the current module naming/location suggests its for CTL's internal tests
Perhaps we can have a Contract.Test.* namespace, with E2E modules therein

That's how I understood it.

proposal from two weeks ago (which you upvoted; further up in this conversation).

I think we want to export all the machinery but not the examples themselves, as they are useless for the users - indeed that's not how the original message reads.
Also please move passwords and all functions that are only useful for us to the test/ dir.

@klntsky I restructured it as you requested. Is it more like you expected it now?

I am unable to run Gero tests:

Start Example Gero
Example Gero finished, check success...
... app starting
... Failed to load resource: the server responded with a status of 404 (File not found)
e2e tests
  ✗ Gero:

  false doesn't satisfy predicate

Is that still the case for you? This one always works for me, even with a clean repo checkout.

I noticed that when I run the tests, SignMultiple fails with ValueNotConserved, but if I comment-out all the tests except SignMultiple, it passes. I believe the reason is that UTxO spending does not propagate fast enough to the wallet.

Unfortunately, this error happens again and again with various examples. I am not sure if the solution is just to introduce rather long wait times between examples or if they perhaps point to real issue.

or if they perhaps point to real issue.

I am quite sure they don't, given all the symptoms.

Is that still the case for you?

I was seeing this again just now. But then I tried once more after removing ./tmp and it worked.

or if they perhaps point to real issue.

I am quite sure they don't, given all the symptoms.

Will we run into test errors if more than one person runs the tests simultaneously? Just wondering because at the moment I am getting loads of fails and before everything worked. Everyone is using the same wallet after all, so the chances for double spending seems pretty high to me.
Perhaps it was a mistake to have one wallet for everyone.

Will we run into test errors if more than one person runs the tests simultaneously?

This certainly may be the case.

Perhaps it was a mistake to have one wallet for everyone.

No, that is fine at this stage.
https://github.com/Plutonomicon/cardano-transaction-lib/issues/783


minimum viable product

[x] public datacats release
[x] datacats child environment feature: environments can have multiple child db/solr/web containers
[ ] nginx http router config: serve datacats children as subdomains
[ ] flask admin interface: create and remove children, set admin passwords
[ ] template multisite datacats environment: link back to project, tell users how to contact admins

This is a meta-ticket to track the progress to a first release of ckan-multisite "0.1"

:+1:

https://github.com/datacats/datacats/pull/82

Just merged multisite support into datacats master: https://github.com/datacats/datacats/pull/82

:+1:

So the documentation is lagging a little bit behind, but the current master allows you to:

Create instances from the command line (using the datacats init CLI tool) and have them picked up by multisite and automatically configured for Nginx
Add and remove instances
Generate Nginx router configurations and place them in the right place
start, stop, and get the status of containers from the "edit" pane of the Flask-admin interface

Next Steps:

Add ability to specify custom site domains that aren't just the site name
Stylize things a little bit

Sounds great!

Closing this as the major functionality is complete, we're working on polishing the interface and docs. If there's anyone looking to give early feedback, now would be a great time.

I'll give it a spin on Monday!

:+1:


Implement after_commit callback

Given that queries are wrapped in a transaction by default, would it be possible to implement the _commit and _rollback callbacks?

This is certainly something that I would like to see.  @ProGM actually just suggested this recently.  Would you like to comment Piero?

I plan to work on this soon.
I took a look to ActiveRecord, and I noticed that Rails5 added new callbacks, to manage commit-based callbacks.
For example, if you wanted to capture commits only on creation, before you had to do something like:
after_commit :something, on: :create

Now it's:
after_create_commit :something

I think we should implement it in the same way.
What do you think?

Sure, I'm definitely all for copying the state of the art of ActiveRecord.  They're such a big project that I think they've done a lot of the thinking for us on the topic of making an ORM (so long as it's not very different from Neo4j)


Container Creation prevention in the event of specific errors.

In the event containers have issues specifically with the name of the container or the port number, we should in the event of container name remove the container (but do not kill it), and in the event of port collision we should report on it.
After these checks have passed the containers would be allowed to start, however if they don't the application should close advising the user of the issues. Assuming the colliding container names are a problem, this shouldn't fail the application because the containers would be expected to be removed (but not killed).
So ideally port collision is the item which quits pygmy, and the user should be aware of it when they attempt to run pygmy.

I tested the issue/25 branch and the status stuff works well prior to starting pygmy. Unfortunately when i start pygmy I get:
$ pygmy-go up
Successfully pulled andyshinn/dnsmasq:2.78
Error response from daemon: driver failed programming external connectivity on endpoint amazeeio-dnsmasq (3880bef9e1ae8a758cdec9e19dbbe174491cfe13d567e7f0d2088e3989ff7480): Error starting userland proxy: listen tcp <IP_ADDRESS>:6053: bind: address already in use
...

Confirmed that this is not an issue on HEAD.

Notes:

the additional functionality works as designed
dnsmasq (port 53) is broken because of a change made a while ago - portbinding needs to be updated.
container removal now targets the container ID - so that containers are removed regardless of their name (ie /amazeeio-dnsmasq vs amazeeio-dnsmasq). So, the logic doesn't care about the container name and will remove it regardless.

Looks great!


AllEqual Constraint Fails with Single-Value Domain in ACE Solver

When using the ACE solver with PyCsp3, the AllEqual constraint (or NValues == 1) fails if a domain variable is limited to one value. This issue occurs even when the constraint should be satisfied trivially.
Steps to Reproduce

Create a PyCsp3 model with a 2D VarArray where one column has a single-value domain.
Apply an AllEqual constraint to the single-value column.
Attempt to solve the model using the ACE solver.

Example Code
from pycsp3 import *

def main():
    clear()

    def domain_y(i,j):
        if j == 0:
            return set(range(10))
        elif j == 1:
            return set(range(1))  # Single-value domain
        else:
            return set(range(10))

    x = VarArray(size=[5,3], dom=domain_y)

    satisfy(
        [
            AllEqual(x[0][1], x[1][1], x[2][1], x[3][1], x[4][1])
        ]
    )

    instance = compile()
    ace = solver(ACE)
    result = ace.solve(instance, verbose=True)

    if result is SAT:
        print("Solution: ", values(x))
    else:
        print(f"result {result.name}: ")

if __name__ == "__main__":
    main()

Current Behavior
The ACE solver fails to solve the model, producing the following error:
ACE v2.3

  instance
    name:main

  Loading variables...

! ERROR:
  Use the solver option -ev for more details

Expected Behavior
The solver should recognize that the AllEqual constraint is trivially satisfied for the column with a single-value domain and proceed to solve the rest of the model.
Additional Context

This issue specifically occurs with the ACE solver.
The Choco solver successfully solves this example without any issues.
In real-world scenarios, it's not always possible to know in advance if a collection of variables will have only one possible value, making it challenging to work around this issue.

Environment

PyCsp3 version: pycsp3==2.3.1
ACE solver version: 2.3

Questions

Is this a known limitation of the ACE solver when used with PyCSP3?
Is there additional debugging information that can be provided to help investigate this issue?

Thank you for your assistance in resolving this problem.

Thanks for pointing out the issue.
In general, if you have an XCSP3 instance (file), as e.g., TestAllEqual.xml:
<instance format="XCSP3" type="CSP">
  <variables>
    <array id="x" size="[5][3]">
      <domain for="x[0][0] x[0][2] x[1][0] x[1][2] x[2][0] x[2][2] x[3][0] x[3][2] x[4][0] x[4][2]"> 0 1 2 3 4 5 6 7 8 9 </domain>
      <domain for="x[][1]"> 0 </domain>
    </array>
  </variables>
  <constraints>
    <allEqual> x[][1] </allEqual>
  </constraints>
</instance>

If you use the option "-ev"
 java ace TestAllEqual.xml  -ev
you get the trace where the problem rises:
java.lang.Exception
	at utility.Kit.control(Kit.java:169)
	at utility.Kit.control(Kit.java:179)
	at constraints.global.AllEqual.<init>(AllEqual.java:97)
	at problem.Problem.allEqual(Problem.java:1644)
	at problem.XCSP3.buildCtrAllEqual(XCSP3.java:461)
	at org.xcsp.parser.loaders.CtrLoaderInteger.allEqual(CtrLoaderInteger.java:415)
	at org.xcsp.parser.loaders.CtrLoaderInteger.load(CtrLoaderInteger.java:202)
	at org.xcsp.parser.callbacks.XCallbacks.loadCtr(XCallbacks.java:542)
	at problem.XCSP3.loadCtr(XCSP3.java:345)
	at org.xcsp.parser.callbacks.XCallbacks.loadConstraints(XCallbacks.java:444)
	at org.xcsp.parser.callbacks.XCallbacks.loadConstraints(XCallbacks.java:422)
	at org.xcsp.parser.callbacks.XCallbacks.loadInstance(XCallbacks.java:305)
	at org.xcsp.parser.callbacks.XCallbacks.loadInstance(XCallbacks.java:328)
	at problem.XCSP3.model(XCSP3.java:159)
	at problem.Problem.<init>(Problem.java:720)
	at main.Head.buildProblem(Head.java:400)
	at main.Head.solveInstance(Head.java:431)
	at main.Head.run(Head.java:447

Here, the problem is the control made at line 97 of AllEqual.java
If we replace, values.length >1 by values.length >=1, the problem is fixed (the propagator is fine for this extreme case).
You can also write if solve(options="ev") is SAT
or in you code, ace.setting("-ev")
This has been fixed (and pushed) in the code on Github for ACE.
(note that you can build a jar instance from this repository, and replace the jar from the PyCSP3 distribution, if deployed on your machine).
This will be part of the next PyCSP3 version (including Pypi one) that is planned around the beginning of september.
Best.

Thank you so much for fixing this issue! really appreciate.


hold using trigger option

I'd rather not add this. The way I'd rather people do this is just change the value of HoldButton to the trigger.

That approach makes it hard to do the clean updates as suggested in the readme:

I recommend you clone the repo locally and create a symbolic link to your project so you can get updates and merge changes cleanly.

And HoldButton is private so its a bit wonky to try and subclass NVRHand. Plus, then you'd have to make your own controller prefab which would also make clean updating a bit wonky.

Agreed.  I've really wanted a way to easily switch it to the trigger without having to do a weird workaround.
I LOVE this prefab though. Thanks for making it!  Was very easy to set up.

How about we make the variable serialized, then you can set it on the prefab to whatever you want?

I think that works for me.  That'd be great!

That sounds perfect!


Unable to Clock Out

I've tested the program to clock for .15 hour (9-10 minutes), the program runs for 15 minutes then outputs the following:
Failed, Unable to Clock Out.
NOTICE: Please Manually Clock Out To Avoid Issues
If this error continues, please raise an issue on Github

Hey thanks for raising an issue. This is a known problem, but I havent fixed because I dont know what the minimum amount of time should. Right now it is 15 minute periods, but what do you think it should be?

My guess is the later, it actually clocked you out. There is currently a bug in the clock out process. It will clock you out, but say it did not for some reason.
I would just double check the webdriver window to verify that you clocked out successfully. I can take a look into the bug later today.

Yah, that is something I have noticed as well, I am a bit swamped today, but I will also try to take a look as well.
I'm sure its just that I was not checking the correct element when it comes to confirming the clock out. Or perhaps we should check multiple elements as a backup.
Regardless, kudos to you Kendall for being awesome!

I think also maybe forcing the user to input a number greater than 15 minute would help with UI. I randomly tested like .1 hour (6 minutes) when I ran the updated program for the first time. The program counted up until 15 minutes, output the same error, and actually did not clock me out. However, when I tried running the program again for 1 hour, while it threw the same error, the program did clock me out then.

Fixed this in https://github.com/Shaun-Regenbaum/OneUSGAutomaticClock/pull/15


New picker based interface for setting isf schedule

Thanks Darin. I believe all the comments are addressed.


Feat/array of endpoints 960

Array of endpoints
What does this PR do?

Resolve #960

To get the producers info an endpoint is requested, however if this endpoint fails the DB info doesn't change and bpjsons aren't updated, so now the functionality of get producers use a list of endpoints, each one is tried until the end of the list.
If this last case occurs the producers information is query from DB.

The previous situation also happens in the website, the main page queries this endpoint a lot,  as some of them have a rate limit, when an endpoint fails it uses the next one in the list and wait 5 minutes before requesting the failed endpoint.

Avoid requesting empty producers urls or urls shorter than 3 characters.

Avoid to the delete nodes of default bpjson when is the eos network.

Now to get the server version the node api is requested after all the data is in the DB.

Steps to test

Run the project locally (The first endpoint of the array must be failing to do the test)
Check the data in hasura (eos could be slow to get all data)
Go to frontend
Check home page and "accounts and contracts" pages

Also check the order and endpoints that are used in the yamls files
Screenshots
Shows how when producers are loading it tests each endpoint until one works. (This logs are not in the pr)

Frontend implementation:

@Torresmorah recommendation for future PRs.
Try to split your PRs into smaller code changes to make them easier to review.

Ok, I'll keep that in mind, thanks for the feedback.


chore: add release-please

It's still in the prerelease process for v17, so it may still need to be configured. Until then I'll still use release-it for releases.


Vietnamese review recommended

Calling @duyphanz  :) (or someone interested in helping)
While doing the Finnish translation, I happened to notice something wrong with Vietnamese translations. I don't understand the language so I don't know if the strings are in a wrong order or if just the <code> parts are incorrect. I would suggest reviewing the translation of at least docs.js.
See for instance:
grid-area
English has: <grid-row-start> / <grid-column-start> / <grid-row-end> / <grid-column-end>
Vietnamese has: <integer> span <integer>
grid-column
En: <grid-column-start> / <grid-column-end>
Vi: <integer> span <integer>
grid-row
En: <grid-row-start> / <grid-row-end>
Vi: <integer> span <integer>
grid-template
En: <grid-template-rows> / <grid-template-columns>
Vi: <length> <percentage> <flex> max-content min-content minmax(min, max)

Thanks for your pointing out, I'll schedule to check it soon! @minna-xD


Clarification needed: not clear which of the samples work on which existing USD implementations

The question has come up a couple times in the working group -
I think part of what makes this repo useful is giving people wanting to use USD as interchange format an understanding of what different applications support. So I think readme files of the various subfolders could note if/what of that works in systems, and then users or implementers can amend that / correct it / extend it with screenshots or notes. This helps to form an understanding of problems to look out for both when implementing USD and when authoring assets that aim for cross-compatibility.
For me in particular, understanding which schemas / features / combinations thereof are supported in AR QuickLook and in Omniverse. For some sample assets (#4, #5, #6, #8),  I added minimal info / screenshots on that, which I think would be a start for other readme's as well.

Related issue: https://github.com/usd-wg/assets/issues/32 - is there a "gold standard" for viewing animation, for example? USDView doesn't do it.


Use multi_put/delete to update TX and UTXO records in advance_block().

As discussed, this it the multi_put/delete work I did. Performance results were unclear. I could not measure the difference when testing around block 896000.

Obsolete after #109


ImportError: no module named enum?

when i try to run: python pokecli.py on ubuntu.
it give me this error:
root@cyborg:/home/cyborg/Documents/PokemonGo-Bot# python pokecli.py
Traceback (most recent call last):
File "pokecli.py", line 39, in
from pokemongo_bot import logger
File "/home/cyborg/Documents/PokemonGo-Bot/pokemongo_bot/init.py", line 20, in
from item_list import Item
File "/home/cyborg/Documents/PokemonGo-Bot/pokemongo_bot/item_list.py", line 1, in
from enum import Enum
ImportError: No module named enum
can anyone please help me out? =D
thanks!!

Have you upgraded/installed the "pip requirments"?
Hint: pip install --upgrade -r requirements.txt

This should work
pip install enum


Fix ready event

Goal
Adjust the moment when ready event is triggered to be after AVFoundationPlayback is rendered.
How to test
1 - Comment the line 597 at file AVFoundationPlayback.swift
2 - Run Clappr example app;
3 - Check if ready event appears at log.

Hey @geisonfgf  and @paulogama14, I couldn't play the video. It keeps loading indefinitely.

Hey @igoriols, you are right. When we comment that line the video won't play and this is the expected behavior because it's necessary to test if the ready is triggered when Playback is really ready. Different from the old implementation the ready was triggered on a play action.

We changed the approach of this solution so it was created a new PR with it https://github.com/clappr/clappr-ios/pull/251


Invert Y axis

[ ] I hereby verify that I am a sponsor.

Unfortunately, I am not. Still, I am an active member of the open-source community (see https://metacpan.org/author/AMBS for my contributions to the Perl community as an example). Thus, while I understand your point of view. Anyway, be free to delete the ticket if you like
Describe the feature
In my classes on Game Development, I use a Camera class, that inverts the Y-axis, and maps units (meters, for instance) to pixels. I am attaching it just in case you like to see it. https://gist.github.com/ambs/d147cfd65d470f2131825b4e2426b931
To use VelcroPhysics directly, I need to set a negative gravity value. It would be nice to have an option to just invert this axis. Not sure how hard, and how much efficiency we would lose. But it would be cool to have.

Cameras and drawing are outside the scope of a physics engine. I do have a small game framework built into the samples, but they are more of a starter kit than anything else. I also have a ConvertUnits class you can use to translate between meters (default in Velcro) and pixels.
See Camera2D  and ConvertUnits.


ERROR while installing irmin.0.9.8

Another Travis failure:
https://travis-ci.org/talex5/cuekeeper/builds/74127652
#=== ERROR while installing irmin.0.9.8 =======================================#
# opam-version 1.2.2
# os           linux
# command      make test
# path         /home/travis/.opam/system/build/irmin.0.9.8
# compiler     system (4.02.1)
# exit-code    2
# env-file     /home/travis/.opam/system/build/irmin.0.9.8/irmin-35639-ce0575.env
# stdout-file  /home/travis/.opam/system/build/irmin.0.9.8/irmin-35639-ce0575.out
# stderr-file  /home/travis/.opam/system/build/irmin.0.9.8/irmin-35639-ce0575.err
### stdout ###
# [...]
# -- HTTP.MEM.012 Failed --
# High-level store merges.
# ./_tests/HTTP.MEM.012.output:
# Server started on port 8080.
#         DEBUG: post /update/a/b/a {"task":{"date":"1438720089","uid":"20401","owner":"Irmin testing-worker-linux-d..
# 2015-08-04 20:28:09.316 MEM          DEBUG: iter
#
# [exception] Unix.Unix_error(Unix.ECONNREFUSED, "connect", "")
#
# 1 error! in 27.190s. 120 tests run.
### stderr ###

That's due a segfault in Lwt, fixed in 2.5.0: https://github.com/ocsigen/lwt/issues/136
Need a new release of cohttp to have the fix in ... But yea, should disable the tests in the release. I'll push a fix to opam-repository, thanks for the report.

See https://github.com/ocaml/opam-repository/pull/4598
Would be very nice to have https://github.com/ocaml/opam/issues/2190 though...

Thanks - CueKeeper tests are passing again now :-)


OffscreenCanvas broken

Running triangleOffscreen.jl runs into OffscreenCanvas not defined error.
[ Info: Setting Log level : WGPULogLevel_Off
ERROR: LoadError: UndefVarError: OffscreenCanvas not defined
Stacktrace:
 [1] getproperty(x::Module, f::Symbol)
   @ Base ./Base.jl:31
 [2] top-level scope
   @ ~/September2022/WGPU.jl/examples/triangleOffscreen.jl:39
 [3] include(fname::String)
   @ Base.MainInclude ./client.jl:476
 [4] top-level scope
   @ REPL[1]:1
in expression starting at /Users/arhik/September2022/WGPU.jl/examples/triangleOffscreen.jl:39

OS indenpendent error. Purely api issue.

https://github.com/dvijaha/WGPU.jl/blob/8d3a25b2bf5aa9af4df241dc3677b67fd7613ef5/src/offscreen.jl#L180 is the issue.

OffscreenCanvas doesn't work anymore because glfw related file is load statically loaded based on platform. We need to design around this at some point.

https://github.com/JuliaWGPU/WGPUCore.jl/commit/1c986d367dad6359286000126077617cc3dafe1d fixes this


Added more exception rules

@vanniktech if you check the .yml file
exceptions:
active: true

Should not each rule have it's own entry though?

All rules in this package are added by default.

Ok, I will need more time with this one, thanks for the contributions!
Just linking #95, where all Throw- and Catch- rules should be refactored using a MultiRule + configurations.